8750822722

Committed 19 Apr 2024 08:38AM UTC coverage: 32.08% (+0.03%) from 32.052%

Build Type

push

github

Committed by

Commit Message

Merge pull request #9139 from ForumMagnum/master

[EA] Deploy fix for ghost loading states, cpu optimizations

Run Details

4685 of 37962 branches covered (12.34%)

Branch coverage included in aggregate %.

101 of 232 new or added lines in 22 files covered. (43.53%)

4 existing lines in 4 files now uncovered.

27171 of 61341 relevant lines covered (44.3%)

182.09 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/packages/lesswrong/components/hooks/useCallbackDebugRerenders.ts

NEW 1	import { useCallback, useRef } from "react";	×
2
3	/**
4	* Behaves identically to useCallback, except that when the resulting function
5	* is different because one of the deps is different, it console logs a message
6	* with `description` and the indexes of all the changed deps.
7	*
8	* Note that the eslint rule react-hooks/exhautive-deps is not able to
9	* recognize that this is the same as useCallback, so it is not safe to use
10	* except as a debugging tool.
11	*/
12	export function useCallbackDebugRerenders<T extends (...args: any[]) => any>(fn: T, deps: any[], description?: string): T {
NEW 13	const lastDeps = useRef(deps);	×
NEW 14	const depsChanged: number[] = [];	×
NEW 15	for (let i=0; i<deps.length; i++) {	×
NEW 16	if (deps[i] !== lastDeps.current[i]) {	×
NEW 17	depsChanged.push(i);	×
18	}
19	}
NEW 20	if (depsChanged.length > 0) {	×
21	// eslint-disable-next-line no-console
NEW 22	console.log(`${description ?? "useCallbackDebugRerenders"} changed: deps ${depsChanged.join(", ")}`);	×
23	}
NEW 24	lastDeps.current = [...deps];	×
25
26	// eslint-disable-next-line react-hooks/exhaustive-deps
NEW 27	return useCallback(fn, deps);	×
28	}