GitHub check
256 KB of 604 KB loaded Download the full log (604 KB)
348 KB of earlier output not loaded
026-10-24T12:44:50.7023145Z ยท โฐโโ `loadingVisibility` cannot be modified
270 โ }, [loadingVisibility, props.loading]);
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/voice-input/ComposerDictationControl.tsx:328:5]
325 โ }) {
326 โ const recordingVisibility = useSharedValue(props.phase === "recording" ? 1 : 0);
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `recordingVisibility` originates here
327 โ useLayoutEffect(() => {
328 โ recordingVisibility.value = withTiming(props.phase === "recording" ? 1 : 0, DICTATION_TIMING);
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `recordingVisibility` cannot be modified
329 โ }, [props.phase, recordingVisibility]);
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/components/VideoPreviewModal.ios.tsx:63:55]
61 โ }, []);
62 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
63 โ if (playbackUrl === null && resolvedUrl !== null) setPlaybackUrl(resolvedUrl);
ยท โโโโโโโโฌโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
64 โ }, [playbackUrl, resolvedUrl]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(no-unstable-nested-components): Do not define components during render.
โญโ[apps/mobile/src/Stack.tsx:1079:21]
1078 โ <Navigator
1079 โ screenLayout={(props) => <GuardedScreenLayout {...props} />}
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
1080 โ screenOptions={({ route }) =>
โฐโโโโ
help: Move this component definition out of the parent component `AdaptiveRootStack` and pass data as props. If you want to allow component creation in props, set `allowAsProps` option to true.
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/voice-input/GlobalVoiceInputControl.tsx:68:5]
65 โ const [expanded, setExpanded] = useState(false);
66 โ const progress = useSharedValue(0);
ยท โโโโโฌโโโ
ยท โฐโโ `progress` originates here
67 โ useEffect(() => {
68 โ progress.value = withTiming(expanded ? 1 : 0, MORPH_TIMING);
ยท โโโโโฌโโโ
ยท โฐโโ `progress` cannot be modified
69 โ }, [expanded, progress]);
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ t3code(no-inline-schema-compile): Hoist Schema.decodeEffect(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
โญโ[apps/server/src/vcs/GitVcsDriverCore.test.ts:1786:34]
1785 โ const stat = branch.files!.find((file) => file.path === path)!;
1786 โ const request = yield* Schema.decodeEffect(ReviewDiffPreviewInput)({
ยท โโโโโโโโโโโโโโโโโโโ
1787 โ cwd,
โฐโโโโ
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/showcase/ShowcaseCaptureCoordinator.tsx:123:7]
118 โ
119 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
120 โ if (orientationSettled) return;
121 โ const orientation = getNativeShowcaseOrientation();
122 โ if (orientation === null) {
123 โ setOrientationSettled(true);
ยท โโโโโโโโโโโฌโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
124 โ return;
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/showcase/ShowcaseCaptureCoordinator.tsx:304:7]
292 โ
293 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
294 โ if (
295 โ scene === null ||
296 โ requestedScene === null ||
297 โ scene !== requestedScene ||
298 โ !hasFixture ||
299 โ !orientationSettled ||
300 โ !themeApplied ||
301 โ (scene === "agent-activity" && !agentActivityStaged) ||
302 โ !isShowcaseNativeContentReady({ scene, themeId, renderSignal })
303 โ ) {
304 โ setReadyScene(null);
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
305 โ return;
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ eslint(no-unused-vars): Variable 'lastOutcome' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/mobile/src/features/showcase/ShowcaseCaptureCoordinator.tsx:128:9]
127 โ let cancelled = false;
128 โ let lastOutcome: string | null = null;
ยท โโโโโโฌโโโโโ
ยท โฐโโ 'lastOutcome' is declared here
129 โ void retryShowcaseOperation(async () => applyNativeShowcaseOrientation(orientation), {
โฐโโโโ
help: Consider removing this declaration.
โ unicorn(prefer-string-starts-ends-with): Prefer String#startsWith over a regex with a caret.
โญโ[apps/server/src/vcs/GitVcsDriverCore.ts:517:11]
516 โ GIT_DIAGNOSTIC_LINE_PATTERN.test(line) ||
517 โ (!/^remote:/.test(line) && SSH_TRANSPORT_REFUSAL_PATTERN.test(line)),
ยท โโโโโโโโโโโโโโโ
518 โ )
โฐโโโโ
help: Replace `/^remote:/.test(line)` with `line.startsWith('remote:')`.
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[apps/mobile/src/features/agent-awareness/remoteRegistration.test.ts:223:5]
222 โ let idlePasses = 0;
223 โ โญโโถ for (;;) {
224 โ โ yield* Effect.promise(() => Promise.resolve());
225 โ โ const pending = backgroundRuntime.pending.shift();
226 โ โ if (!pending) {
227 โ โ idlePasses++;
228 โ โ if (idlePasses >= 3) {
229 โ โ return;
230 โ โ }
231 โ โ continue;
232 โ โ }
233 โ โ idlePasses = 0;
234 โ โ const exit = yield* Effect.exit(
235 โ โ pending.operation as Effect.Effect<unknown, unknown, ManagedRelay.ManagedRelayClient>,
236 โ โ );
237 โ โ yield* Effect.sync(() => {
238 โ โ pending.resolve(exit);
239 โ โ });
240 โ โฐโโถ }
241 โ },
โฐโโโโ
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/layout/AdaptiveWorkspaceLayout.tsx:155:7]
154 โ }
155 โ }, [wrappedRender]);
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ Unnecessary dependency `wrappedRender`
156 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
โญโ[apps/mobile/src/features/layout/AdaptiveWorkspaceLayout.tsx:800:42]
799 โ >
800 โ <AdaptiveWorkspaceContext.Provider value={contextValue}>
ยท โโโโโโโโโโโโโโโโโโโโ
801 โ <WindowHingeContext value={windowHinge}>
โฐโโโโ
help: Wrap the `value` prop in useMemo() or useCallback(), or use a constant value to prevent unnecessary re-renders.
Alternatively, move the value outside the render function if it doesn't depend on props or state.
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/layout/workspace-inspector-pane.tsx:42:5]
35 โ
36 โ const inspectorProgress = useSharedValue(inspectorVisible ? 1 : 0);
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ `inspectorProgress` originates here
37 โ const { renderedInspectorWidth } = props;
38 โ const renderedContentWidth = useSharedValue(inspectorWidth ?? 0);
39 โ
40 โ const onClosed = props.onClosed;
41 โ useEffect(() => {
42 โ inspectorProgress.value = withTiming(
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ `inspectorProgress` cannot be modified
43 โ inspectorVisible ? 1 : 0,
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/layout/workspace-inspector-pane.tsx:67:7]
37 โ const { renderedInspectorWidth } = props;
38 โ const renderedContentWidth = useSharedValue(inspectorWidth ?? 0);
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `renderedContentWidth` originates here
39 โ
40 โ const onClosed = props.onClosed;
41 โ useEffect(() => {
42 โ inspectorProgress.value = withTiming(
43 โ inspectorVisible ? 1 : 0,
44 โ WORKSPACE_PANE_TIMING,
45 โ (finished) => {
46 โ if (finished === true && !inspectorVisible && onClosed !== undefined) {
47 โ runOnJS(onClosed)();
48 โ }
49 โ },
50 โ );
51 โ const targetWidth = inspectorVisible ? (inspectorWidth ?? 0) : 0;
52 โ renderedInspectorWidth.set(
53 โ resizing ? targetWidth : withTiming(targetWidth, WORKSPACE_PANE_TIMING),
54 โ );
55 โ }, [
56 โ inspectorProgress,
57 โ inspectorVisible,
58 โ inspectorWidth,
59 โ onClosed,
60 โ renderedInspectorWidth,
61 โ resizing,
62 โ ]);
63 โ
64 โ useEffect(() => {
65 โ const targetWidth = inspectorWidth ?? 0;
66 โ if (!inspectorVisible || resizing) {
67 โ renderedContentWidth.value = targetWidth;
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `renderedContentWidth` cannot be modified
68 โ return;
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/client.test.ts:131:9]
130 โ ) {
131 โ โญโโถ for (let attempt = 0; attempt < 100; attempt += 1) {
132 โ โ if (subscriptions.length >= count) {
133 โ โ return;
134 โ โ }
135 โ โ yield* Effect.yieldNow;
136 โ โฐโโถ }
137 โ return yield* Effect.die(new Error(`Expected ${count} durable subscriptions.`));
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/client.test.ts:252:7]
251 โ yield* SubscriptionRef.set(activeSession, Option.some(session(firstClient)));
252 โ โญโโถ for (let attempt = 0; attempt < 100 && subscriptions.length < 1; attempt += 1) {
253 โ โ yield* Effect.yieldNow;
254 โ โฐโโถ }
255 โ yield* SubscriptionRef.set(activeSession, Option.none());
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/client.test.ts:258:7]
257 โ
258 โ โญโโถ for (let attempt = 0; attempt < 100 && subscriptions.length < 2; attempt += 1) {
259 โ โ yield* Effect.yieldNow;
260 โ โฐโโถ }
261 โ yield* Fiber.interrupt(subscriptionFiber);
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/client.test.ts:322:7]
321 โ );
322 โ โญโโถ for (let attempt = 0; attempt < 100 && observedFailures.length < 1; attempt += 1) {
323 โ โ yield* Effect.yieldNow;
324 โ โฐโโถ }
325 โ
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/client.test.ts:330:7]
329 โ yield* SubscriptionRef.set(activeSession, Option.some(session(secondClient)));
330 โ โญโโถ for (let attempt = 0; attempt < 100 && subscriptions.length < 2; attempt += 1) {
331 โ โ yield* Effect.yieldNow;
332 โ โฐโโถ }
333 โ yield* Fiber.interrupt(subscriptionFiber);
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/session.test.ts:226:3]
225 โ ) {
226 โ โญโโถ for (let attempt = 0; attempt < 100; attempt += 1) {
227 โ โ const socket = sockets[index];
228 โ โ if (socket) {
229 โ โ return socket;
230 โ โ }
231 โ โ yield* Effect.yieldNow;
232 โ โฐโโถ }
233 โ return yield* Effect.die(new Error("Expected the RPC protocol to create a websocket."));
โฐโโโโ
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[packages/client-runtime/src/rpc/session.test.ts:240:3]
239 โ ) {
240 โ โญโโถ for (let attempt = 0; attempt < 100; attempt += 1) {
241 โ โ const request = socket.sent.map((message) => decodeJson(message)).filter(isRpcRequest)[index];
242 โ โ if (request) {
243 โ โ return request;
244 โ โ }
245 โ โ yield* Effect.yieldNow;
246 โ โฐโโถ }
247 โ return yield* Effect.die(new Error("Expected the RPC protocol to send a request."));
โฐโโโโ
โ react(capitalized-calls): Capitalized function called without JSX
โญโ[apps/mobile/src/lib/useHoverGesture.ts:8:15]
7 โ () =>
8 โ Gesture.Hover()
ยท โโโฌโโ
ยท โฐโโ `Hover` may be a component
9 โ .manualActivation(true)
โฐโโโโ
help: Render `Hover` with JSX if it is a component; otherwise rename it to start with a lowercase letter or allowlist it in the compiler configuration
note: `Hover` is treated as a component because it begins with an uppercase letter; React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/static-components
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/processRunner.test.ts:266:14]
265 โ
266 โ for (const {} of ["timeout", "success"] as const) {
ยท โโ
267 โ }
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ unicorn(require-module-specifiers): Empty import specifier is not allowed
โญโ[apps/server/src/sourceControl/SourceControlProviderRegistry.ts:15:13]
14 โ } from "@t3tools/contracts";
15 โ import type {} from "@t3tools/contracts";
ยท โโ
16 โ
โฐโโโโ
help: Remove empty braces
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/settings/scheduled-task-editor.tsx:100:85]
99 โ const startEditor = startScheduledTaskEditor;
100 โ const resetEditor = useCallback(() => startEditor(defaultEditor), [defaultEditor, startEditor]);
ยท โโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `startEditor`
101 โ const readEditor = useCallback(
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/settings/scheduled-task-editor.tsx:121:7]
120 โ setEditor,
121 โ startEditor,
ยท โโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `startEditor`
122 โ hasChanges,
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/settings/SettingsNotificationsRouteScreen.tsx:119:10]
117 โ
118 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
119 โ void refreshNotifications();
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
120 โ const subscription = AppState.addEventListener("change", (state) => {
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/settings/SettingsNotificationsRouteScreen.tsx:128:7]
125 โ
126 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
127 โ if (!isLoaded) {
128 โ setLiveActivityStatus("checking");
ยท โโโโโโโโโโโฌโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
129 โ return;
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ eslint(no-unused-vars): Function 'nonEmptyString' is declared but never used.
โญโ[packages/client-runtime/src/work-log/presentation.ts:237:10]
236 โ
237 โ function nonEmptyString(value: unknown): string | null {
ยท โโโโโโโโฌโโโโโโ
ยท โฐโโ 'nonEmptyString' is declared here
238 โ return typeof value === "string" && value.trim().length > 0 ? value : null;
โฐโโโโ
help: Consider removing this declaration.
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/workspace/WorkspaceToolbar.ios.tsx:114:33]
113 โ return () => cancelAnimationFrame(frame);
114 โ }, [restoreSearch, tabActive, bottomSearch]);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `bottomSearch`
115 โ const composeButton = nativeToolbarCompose ? (
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/provider/Drivers/CodexDriver.test.ts:466:14]
465 โ
466 โ for (const {} of ["direct", "wrapper"] as const) {
ยท โโ
467 โ }
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ eslint(no-unused-vars): Variable 'layerFromProviderAdapter' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/orchestration-v2/ProviderAdapterRegistry.ts:346:7]
345 โ
346 โ const layerFromProviderAdapter: Layer.Layer<
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ 'layerFromProviderAdapter' is declared here
347 โ ProviderAdapterRegistryV2,
โฐโโโโ
help: Consider removing this declaration.
โ eslint(no-unused-vars): Variable 'layerDaemon' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/orchestration-v2/EffectWorker.ts:914:7]
913 โ
914 โ const layerDaemon: Layer.Layer<never, never, OrchestrationEffectWorkerV2> = Layer.effectDiscard(
ยท โโโโโโฌโโโโโ
ยท โฐโโ 'layerDaemon' is declared here
915 โ runDaemon.pipe(Effect.forkScoped),
โฐโโโโ
help: Consider removing this declaration.
โ eslint(no-new): Do not use 'new' for side effects.
โญโ[apps/server/src/mcp/McpToolAccess.test.ts:168:3]
167 โ // @ts-expect-error a declaration constructed directly
168 โ new McpToolAccess.Declaration(unchecked);
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
169 โ // @ts-expect-error a handlers layer built outside McpToolAccess
โฐโโโโ
help: Assign the result of 'new' to a variable or compare it to a reference.
โ eslint(no-unused-vars): Type 'Scope' is imported but never used.
โญโ[apps/server/src/provider/ProviderAuthService.ts:15:18]
14 โ import * as Layer from "effect/Layer";
15 โ import type * as Scope from "effect/Scope";
ยท โโโฌโโ
ยท โฐโโ 'Scope' is imported here
16 โ import * as Stream from "effect/Stream";
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Type 'ThreadRunSummary' is imported but never used.
โญโ[apps/web/src/types.ts:22:3]
21 โ EnvironmentThreadShell,
22 โ ThreadRunSummary,
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ 'ThreadRunSummary' is imported here
23 โ ThreadRuntimeSummary,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'HostProcess' is imported but never used.
โญโ[apps/server/src/orchestration-v2/ProviderTurnStartService.ts:25:13]
24 โ import * as Schema from "effect/Schema";
25 โ import * as HostProcess from "@t3tools/shared/HostProcess";
ยท โโโโโโฌโโโโโ
ยท โฐโโ 'HostProcess' is imported here
26 โ
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Variable 'serverConfig' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/orchestration-v2/ProviderTurnStartService.ts:124:11]
123 โ const path = yield* Path.Path;
124 โ const serverConfig = yield* ServerConfig.ServerConfig;
ยท โโโโโโโฌโโโโโ
ยท โฐโโ 'serverConfig' is declared here
125 โ const serverSettings = yield* ServerSettings.ServerSettingsService;
โฐโโโโ
help: Consider removing this declaration.
โ eslint(no-unused-vars): Variable 'serverSettings' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/orchestration-v2/ProviderTurnStartService.ts:125:11]
124 โ const serverConfig = yield* ServerConfig.ServerConfig;
125 โ const serverSettings = yield* ServerSettings.ServerSettingsService;
ยท โโโโโโโโฌโโโโโโ
ยท โฐโโ 'serverSettings' is declared here
126 โ const gitWorkflow = yield* GitWorkflowService.GitWorkflowService;
โฐโโโโ
help: Consider removing this declaration.
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/diffs/nativeReviewDiffSurface.ts:277:39]
276 โ
277 โ โญโโถ return createElement(RawNativeView, {
278 โ โ ...nativeProps,
279 โ โ refreshEnabled: props.onPullToRefresh !== undefined,
280 โ โ ref: nativeRef,
281 โ โโโถ });
ยท โฐโโโโ Passing a ref to a function may read its value during render
282 โ }
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ unicorn(no-array-reverse): Use `Array#toReversed()` instead of `Array#reverse()`.
โญโ[apps/mobile/src/lib/markdownDocument.ts:259:28]
258 โ visit(parser.parse(markdown), false, false);
259 โ for (const edit of edits.reverse())
ยท โโโโโโโ
260 โ markdown = markdown.slice(0, edit.start) + edit.content + markdown.slice(edit.end);
โฐโโโโ
help: `Array#reverse()` mutates the original array. Use `Array#toReversed()` to return a new reversed array without modifying the original.
โ unicorn(no-array-reverse): Use `Array#toReversed()` instead of `Array#reverse()`.
โญโ[apps/mobile/src/lib/markdownDocument.ts:290:28]
289 โ visit(parser.parse(markdown));
290 โ for (const edit of edits.reverse())
ยท โโโโโโโ
291 โ markdown = markdown.slice(0, edit.start) + edit.content + markdown.slice(edit.end);
โฐโโโโ
help: `Array#reverse()` mutates the original array. Use `Array#toReversed()` to return a new reversed array without modifying the original.
โ unicorn(no-array-reverse): Use `Array#toReversed()` instead of `Array#reverse()`.
โญโ[apps/mobile/src/lib/markdownDocument.ts:344:38]
343 โ visit(parser.parse(markdown));
344 โ for (const lineStart of lineStarts.reverse()) {
ยท โโโโโโโ
345 โ // The header row may be a lazy line without its container markers (`>` or list indent),
โฐโโโโ
help: `Array#reverse()` mutates the original array. Use `Array#toReversed()` to return a new reversed array without modifying the original.
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/hooks/useThreadActions.ts:821:56]
820 โ const resolved = resolveThreadTarget(target);
821 โ if (resolved && !canSnooze(resolved.thread, { now: new Date().toISOString() })) {
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
822 โ return AsyncResult.failure(
โฐโโโโ
help: `Date` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/hooks/useThreadActions.ts:709:46]
708 โ const wokeAt = resolved
709 โ ? threadWokeAt(resolved.thread, { now: new Date().toISOString() })
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
710 โ : null;
โฐโโโโ
help: `Date` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ typescript(no-this-alias): Unexpected aliasing of 'this' to local variable.
โญโ[apps/web/src/contextMenuFallback.test.ts:38:9]
37 โ get isConnected() {
38 โ let current: FakeElement | null = this;
ยท โโโโโโโ
39 โ while (current?.parent) {
โฐโโโโ
help: Assigning a variable to this instead of properly using arrow lambdas may be a symptom of pre-ES2015 practices or not managing scope well.
โ eslint(no-unused-vars): Variable 'layerTest' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/usage/UsageService.ts:169:7]
168 โ
169 โ const layerTest = Layer.succeed(
ยท โโโโโฌโโโโ
ยท โฐโโ 'layerTest' is declared here
170 โ UsageService,
โฐโโโโ
help: Consider removing this declaration.
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[apps/server/src/usage/UsageService.test.ts:2615:7]
2614 โ let orphanedAt: number | undefined;
2615 โ โญโโถ for (let interruptAt = 1; interruptAt <= 31; interruptAt += 1) {
2616 โ โ const tasks: Array<() => void> = [];
2617 โ โ const dispatcher: Scheduler.SchedulerDispatcher = {
2618 โ โ scheduleTask: (task) => tasks.push(task),
2619 โ โ flush: () => {
2620 โ โ let task: (() => void) | undefined;
2621 โ โ while ((task = tasks.shift()) !== undefined) task();
2622 โ โ },
2623 โ โ };
2624 โ โ
2625 โ โ let requestFiber: Fiber.Fiber<unknown, unknown> | undefined;
2626 โ โ let requestChecks = 0;
2627 โ โ const scheduler: Scheduler.Scheduler = {
2628 โ โ executionMode: "async",
2629 โ โ makeDispatcher: () => dispatcher,
2630 โ โ shouldYield: (fiber) => {
2631 โ โ if (fiber !== requestFiber) return false;
2632 โ โ requestChecks += 1;
2633 โ โ if (requestChecks !== interruptAt) return false;
2634 โ โ fiber.interruptUnsafe();
2635 โ โ return true;
2636 โ โ },
2637 โ โ };
2638 โ โ
2639 โ โ const input: UsageSummaryInput = {
2640 โ โ ...WINDOW,
2641 โ โ sinceDay: UsageDay.make("2026-09-01"),
2642 โ โ untilDay: UsageDay.make(`2026-08-${String(interruptAt).padStart(2, "0")}`),
2643 โ โ };
2644 โ โ const first = yield* service
2645 โ โ .readSummary(input)
2646 โ โ .pipe(
2647 โ โ Effect.exit,
2648 โ โ Effect.provideService(Scheduler.Scheduler, scheduler),
2649 โ โ Effect.forkChild,
2650 โ โ );
2651 โ โ requestFiber = first;
2652 โ โ yield* Effect.yieldNow;
2653 โ โ dispatcher.flush();
2654 โ โ
2655 โ โ const second = yield* service.readSummary(input).pipe(
2656 โ โ Effect.match({
2657 โ โ onFailure: (error) => error.reason,
2658 โ โ onSuccess: () => "success" as const,
2659 โ โ }),
2660 โ โ Effect.provideService(Scheduler.Scheduler, scheduler),
2661 โ โ Effect.forkChild,
2662 โ โ );
2663 โ โ yield* Effect.yieldNow;
2664 โ โ dispatcher.flush();
2665 โ โ const secondExit = second.pollUnsafe();
2666 โ โ if (secondExit === undefined) {
2667 โ โ second.interruptUnsafe();
2668 โ โ orphanedAt = interruptAt;
2669 โ โ break;
2670 โ โ }
2671 โ โ if (Exit.isFailure(secondExit)) {
2672 โ โ assert.fail("the matching request fiber was interrupted");
2673 โ โ }
2674 โ โ assert.strictEqual(secondExit.value, "invalidWindow");
2675 โ โฐโโถ }
2676 โ
โฐโโโโ
โ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
โญโ[apps/mobile/src/features/threads/ThreadSettingsSheet.tsx:290:50]
289 โ return (
290 โ <ExistingThreadSettingsRouteContext.Provider value={value}>
ยท โโโโโโโโโโโโโ
291 โ {props.children}
โฐโโโโ
help: Wrap the `value` prop in useMemo() or useCallback(), or use a constant value to prevent unnecessary re-renders.
Alternatively, move the value outside the render function if it doesn't depend on props or state.
โ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
โญโ[apps/mobile/src/features/threads/ThreadSettingsSheet.tsx:654:44]
653 โ return (
654 โ <ThreadSettingsSessionContext.Provider value={value}>
ยท โโโโโโโโโโโโโ
655 โ {props.children}
โฐโโโโ
help: Wrap the `value` prop in useMemo() or useCallback(), or use a constant value to prevent unnecessary re-renders.
Alternatively, move the value outside the render function if it doesn't depend on props or state.
โ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
โญโ[apps/mobile/src/features/threads/ThreadSettingsSheet.tsx:1462:55]
1461 โ return (
1462 โ <ThreadSettingsPickerPresentationContext.Provider value={presentation}>
ยท โโโโโโโโโโโโโโโโโโโโ
1463 โ <ThreadSettingsPickerStack.Navigator
โฐโโโโ
help: Wrap the `value` prop in useMemo() or useCallback(), or use a constant value to prevent unnecessary re-renders.
Alternatively, move the value outside the render function if it doesn't depend on props or state.
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:261:5]
259 โ .join("|");
260 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
261 โ stop();
ยท โโโฌโ
ยท โฐโโ Avoid calling setState() directly within an effect
262 โ }, [orderVersion]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found missing/extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:261:5]
260 โ useEffect(() => {
261 โ stop();
ยท โโโฌโ
ยท โฐโโ Missing dependency `stop`
262 โ }, [orderVersion]);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `orderVersion`
263 โ useEffect(
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should. Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:96:9]
95 โ const latest = useRef(props);
96 โ latest.current = props;
ยท โโโโฌโโโโโโโฌโโโ
ยท โ โฐโโ Cannot update ref value during render
ยท โฐโโ This value is a ref
97 โ const gesture = useMemo(
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:104:18]
103 โ .runOnJS(true)
104 โ .onStart(() => latest.current.onStart())
ยท โโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโ
ยท โฐโโ Passing a ref to a function may read its value during render
105 โ .onUpdate((event) => latest.current.onMove(event.translationY))
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:105:19]
104 โ .onStart(() => latest.current.onStart())
105 โ .onUpdate((event) => latest.current.onMove(event.translationY))
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโโโโโโโโ
ยท โฐโโ Passing a ref to a function may read its value during render
106 โ .onEnd((event) => latest.current.onMove(event.translationY))
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:106:16]
105 โ .onUpdate((event) => latest.current.onMove(event.translationY))
106 โ .onEnd((event) => latest.current.onMove(event.translationY))
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโโโโโโโโ
ยท โฐโโ Passing a ref to a function may read its value during render
107 โ .onFinalize((_, success) => latest.current.onEnd(!success)),
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:107:21]
106 โ .onEnd((event) => latest.current.onMove(event.translationY))
107 โ .onFinalize((_, success) => latest.current.onEnd(!success)),
ยท โโโโโโโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโโโโโ
ยท โฐโโ Passing a ref to a function may read its value during render
108 โ [props.disabled],
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:249:9]
248 โ const latest = useRef({ rows, planners, moveThread });
249 โ latest.current = { rows, planners, moveThread };
ยท โโโโฌโโโโโโโฌโโโ
ยท โ โฐโโ Cannot update ref value during render
ยท โฐโโ This value is a ref
250 โ
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(refs): Cannot access refs during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:247:22]
246 โ const [preview, setPreview] = useState<Drag | null>(null);
247 โ const translateY = useRef(new Animated.Value(0)).current;
ยท โโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโ
ยท โฐโโ Cannot access ref value during render
248 โ const latest = useRef({ rows, planners, moveThread });
โฐโโโโ
help: React refs are values that are not needed for rendering. Refs should only be accessed outside of render, such as in event handlers or effects. Accessing a ref value (the `current` property) during render can cause your component not to update as expected
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/refs
โ react(purity): Cannot call impure function during render
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:324:16]
323 โ update(0);
324 โ let last = performance.now();
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Cannot call impure function
325 โ const tick = () => {
โฐโโโโ
help: `performance.now` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(capitalized-calls): Capitalized function called without JSX
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:99:15]
98 โ () =>
99 โ Gesture.Pan()
ยท โโฌโ
ยท โฐโโ `Pan` may be a component
100 โ .enabled(!props.disabled)
โฐโโโโ
help: Render `Pan` with JSX if it is a component; otherwise rename it to start with a lowercase letter or allowlist it in the compiler configuration
note: `Pan` is treated as a component because it begins with an uppercase letter; React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/static-components
โ react(memo-dependencies): Found missing memoization dependencies
โญโ[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:108:5]
103 โ .runOnJS(true)
104 โ .onStart(() => latest.current.onStart())
ยท โโโโฌโโ
ยท โฐโโ Missing dependency `latest`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
105 โ .onUpdate((event) => latest.current.onMove(event.translationY))
106 โ .onEnd((event) => latest.current.onMove(event.translationY))
107 โ .onFinalize((_, success) => latest.current.onEnd(!success)),
108 โ [props.disabled],
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ This dependency list is missing values used by the callback
109 โ );
โฐโโโโ
help: Missing dependencies can cause a value to update less often than it should, resulting in stale UI
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadRouteScreen.tsx:406:5]
404 โ
405 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
406 โ setInspectorSelection((current) => {
ยท โโโโโโโโโโโฌโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
407 โ if (props.renderInspector === undefined) {
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(no-unstable-nested-components): Do not define components during render.
โญโ[apps/mobile/src/features/threads/ThreadRouteScreen.tsx:534:24]
533 โ const inspectorHeaderInset = Platform.OS === "ios" ? 0 : safeAreaInsets.top;
534 โ โญโโถ const GitInspector = () => (
535 โ โ <GitOverviewSheet
536 โ โ headerInset={inspectorHeaderInset}
537 โ โ presentation="inspector"
538 โ โ route={{ params: { environmentId: environmentId!, threadId: threadId! } }}
539 โ โ />
540 โ โฐโโถ );
541 โ const FilesInspector = () =>
โฐโโโโ
help: Move this component definition out of the parent component `ThreadRouteContent` and pass data as props.
โ react(no-unstable-nested-components): Do not define components during render.
โญโ[apps/mobile/src/features/threads/ThreadRouteScreen.tsx:541:26]
540 โ );
541 โ โญโโถ const FilesInspector = () =>
542 โ โ selectedThread !== null && selectedThreadCwd !== null ? (
543 โ โ <ThreadFileNavigatorPane
544 โ โ cwd={selectedThreadCwd}
545 โ โ environmentId={selectedThread.environmentId}
546 โ โ headerInset={inspectorHeaderInset}
547 โ โ projectName={selectedThreadProject?.title ?? "Files"}
548 โ โ selectedPath={null}
549 โ โ onSelectFile={handleSelectInspectorFile}
550 โ โ />
551 โ โฐโโถ ) : null;
552 โ const RouteInspector = () => props.renderInspector?.(inspectorHeaderInset);
โฐโโโโ
help: Move this component definition out of the parent component `ThreadRouteContent` and pass data as props.
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/pull-requests/PullRequestRouteScreen.tsx:333:7]
328 โ };
329 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
330 โ if (!guidesEnabled && section === "guide") {
331 โ requestedSection.current = "overview";
332 โ currentScrollOffset.current = scrollOffsets.current.overview ?? -insets.top;
333 โ setSection("overview");
ยท โโโโโโฌโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
334 โ }
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found missing effect dependencies
โญโ[apps/mobile/src/features/pull-requests/PullRequestRouteScreen.tsx:335:6]
330 โ if (!guidesEnabled && section === "guide") {
331 โ requestedSection.current = "overview";
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ Missing dependency `requestedSection`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
332 โ currentScrollOffset.current = scrollOffsets.current.overview ?? -insets.top;
ยท โโโโโโโโโโฌโโโโโโโโโ โโโโโโโฌโโโโโโ
ยท โ โฐโโ Missing dependency `scrollOffsets`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
ยท โฐโโ Missing dependency `currentScrollOffset`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
333 โ setSection("overview");
334 โ }
335 โ }, [guidesEnabled, section, insets.top]);
ยท โโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโ
ยท โฐโโ This dependency list is missing values used by the callback
336 โ const tabContent = activityContent;
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found missing effect dependencies
โญโ[apps/mobile/src/features/pull-requests/PullRequestRouteScreen.tsx:877:6]
860 โ if (
861 โ lastSelection.current.path === route.params.path &&
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ Missing dependency `lastSelection`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
862 โ lastSelection.current.id === route.params.fileSelectionId
863 โ )
864 โ return;
865 โ lastSelection.current = { path: route.params.path, id: route.params.fileSelectionId };
866 โ if (route.params.focus) navigation.setParams({ focus: undefined });
867 โ setJumpPath(route.params.path);
868 โ setSelectedRowId(null);
869 โ jumpStarted.current = false;
870 โ const file = data.files.find((entry) => entry.path === route.params.path);
871 โ if (!file || !nativeView.current) return;
ยท โโโโโโฌโโโโ
ยท โฐโโ Missing dependency `nativeView`. Refs, setState functions, and other "stable" values generally do not need to be added as dependencies, but this variable may change over time to point to different values
872 โ setCollapsedIds((ids) => ids.filter((id) => id !== file.id));
873 โ jumpStarted.current = true;
874 โ void nativeView.current.scrollToFile(file.id, false).catch(() => {
875 โ jumpStarted.current = false;
876 โ });
877 โ }, [route.params.path, route.params.fileSelectionId, route.params.focus, data.files, navigation]);
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
ยท โฐโโ This dependency list is missing values used by the callback
878 โ const appliedFilter = useRef(filter.hideTestFiles);
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ t3code(no-inline-schema-compile): Hoist Schema.decodeUnknownSync(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
โญโ[apps/server/src/auth/RpcAuthorization.ts:237:17]
236 โ const requiredScopesForSettingsUpdate = (payload: unknown) => {
237 โ const input = Schema.decodeUnknownSync(SettingsUpdate)(payload);
ยท โโโโโโโโโโโโโโโโโโโโโโโโ
238 โ const scopes = requiredScopesForServerSettingsPatch(input.patch);
โฐโโโโ
โ t3code(no-inline-schema-compile): Hoist Schema.decodeUnknownSync(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
โญโ[apps/server/src/auth/RpcAuthorization.ts:256:26]
255 โ if (method === WS_METHODS.assetsCreateUrl) {
256 โ const { resource } = Schema.decodeUnknownSync(AssetCreateUrlInput)(payload);
ยท โโโโโโโโโโโโโโโโโโโโโโโโ
257 โ return [
โฐโโโโ
โ eslint(no-unused-vars): Identifier 'identity' is imported but never used.
โญโ[apps/server/src/auth/http.ts:28:10]
27 โ import * as Schema from "effect/Schema";
28 โ import { identity } from "effect/Function";
ยท โโโโโฌโโโ
ยท โฐโโ 'identity' is imported here
29 โ import * as Layer from "effect/Layer";
โฐโโโโ
help: Consider removing this import.
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/PendingUserInputCard.tsx:94:5]
83 โ
84 โ const cardCoverage = props.cardCoverage;
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `cardCoverage` originates here
85 โ const barHeightRef = useRef(0);
86 โ const cardHeightRef = useRef(0);
87 โ const cardHeight = useSharedValue(0);
88 โ const notifyCoverage = useCallback(() => {
89 โ if (!cardCoverage) {
90 โ return;
91 โ }
92 โ // The card resizes with the keyboard every frame, so easing each sample would leave the
93 โ // feed inset and floating controls trailing it.
94 โ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `cardCoverage` cannot be modified
95 โ }, [cardCoverage]);
โฐโโโโ
help: Do not mutate component props or hook arguments. If the value should change, update it where it is owned and pass an update callback, or use local state
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/PendingUserInputCard.tsx:106:7]
86 โ const cardHeightRef = useRef(0);
87 โ const cardHeight = useSharedValue(0);
ยท โโโโโโฌโโโโ
ยท โฐโโ `cardHeight` originates here
88 โ const notifyCoverage = useCallback(() => {
89 โ if (!cardCoverage) {
90 โ return;
91 โ }
92 โ // The card resizes with the keyboard every frame, so easing each sample would leave the
93 โ // feed inset and floating controls trailing it.
94 โ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
95 โ }, [cardCoverage]);
96 โ const handleBarLayout = useCallback(
97 โ (event: LayoutChangeEvent) => {
98 โ barHeightRef.current = event.nativeEvent.layout.height;
99 โ notifyCoverage();
100 โ },
101 โ [notifyCoverage],
102 โ );
103 โ const handleCardLayout = useCallback(
104 โ (event: LayoutChangeEvent) => {
105 โ cardHeightRef.current = event.nativeEvent.layout.height;
106 โ cardHeight.value = event.nativeEvent.layout.height;
ยท โโโโโโฌโโโโ
ยท โฐโโ `cardHeight` cannot be modified
107 โ notifyCoverage();
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): Cannot modify local variables after render completes
โญโ[apps/mobile/src/features/threads/PendingUserInputCard.tsx:94:5]
87 โ const cardHeight = useSharedValue(0);
88 โ const notifyCoverage = useCallback(() => {
ยท โโโโฌโโโ
ยท โฐโโ This function may (indirectly) reassign or modify `cardCoverage` after render
89 โ if (!cardCoverage) {
90 โ return;
91 โ }
92 โ // The card resizes with the keyboard every frame, so easing each sample would leave the
93 โ // feed inset and floating controls trailing it.
94 โ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ This modifies `cardCoverage`
95 โ }, [cardCoverage]);
โฐโโโโ
help: This argument is a function which may reassign or mutate `cardCoverage` after render, which can cause inconsistent behavior on subsequent renders. Consider using state instead
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): Cannot modify local variables after render completes
โญโ[apps/mobile/src/features/threads/PendingUserInputCard.tsx:94:5]
93 โ // feed inset and floating controls trailing it.
94 โ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ This modifies `cardCoverage`
95 โ }, [cardCoverage]);
96 โ const handleBarLayout = useCallback(
97 โ (event: LayoutChangeEvent) => {
ยท โโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโ
ยท โฐโโ This function may (indirectly) reassign or modify `cardCoverage` after render
98 โ barHeightRef.current = event.nativeEvent.layout.height;
โฐโโโโ
help: This argument is a function which may reassign or mutate `cardCoverage` after render, which can cause inconsistent behavior on subsequent renders. Consider using state instead
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): Cannot modify local variables after render completes
โญโ[apps/mobile/src/features/threads/PendingUserInputCard.tsx:106:7]
103 โ const handleCardLayout = useCallback(
104 โ (event: LayoutChangeEvent) => {
ยท โโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโ
ยท โฐโโ This function may (indirectly) reassign or modify `cardHeight` after render
105 โ cardHeightRef.current = event.nativeEvent.layout.height;
106 โ cardHeight.value = event.nativeEvent.layout.height;
ยท โโโโโโฌโโโโ
ยท โฐโโ This modifies `cardHeight`
107 โ notifyCoverage();
โฐโโโโ
help: This argument is a function which may reassign or mutate `cardHeight` after render, which can cause inconsistent behavior on subsequent renders. Consider using state instead
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/new-task-flow-provider.tsx:444:5]
434 โ : null;
435 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
436 โ if (
437 โ props.active === false ||
438 โ activeDraftKey !== null ||
439 โ editingPendingTask !== null ||
440 โ selectedProject === null
441 โ ) {
442 โ return;
443 โ }
444 โ setActiveDraftKey(
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
445 โ createNewTaskDraft({
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
โญโ[apps/mobile/src/features/threads/new-task-flow-provider.tsx:1244:36]
1243 โ <ActiveNewTaskContext value={activeNewTask}>
1244 โ <NewTaskFlowContext.Provider value={value}>{props.children}</NewTaskFlowContext.Provider>
ยท โโโโโโโโโโโโโ
1245 โ </ActiveNewTaskContext>
โฐโโโโ
help: Wrap the `value` prop in useMemo() or useCallback(), or use a constant value to prevent unnecessary re-renders.
Alternatively, move the value outside the render function if it doesn't depend on props or state.
โ eslint(no-unused-vars): Variable 'useDragRegion' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/web/src/components/preview/PreviewPanelShell.tsx:61:9]
60 โ ) {
61 โ const useDragRegion = isElectron && props.mode !== "sheet" && props.mode !== "embedded";
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ 'useDragRegion' is declared here
62 โ const isInline = props.mode === "inline";
โฐโโโโ
help: Consider removing this declaration.
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[apps/server/src/scheduledTasks/ScheduledTaskService.webhook.test.ts:521:13]
520 โ })).delivery;
521 โ โญโโถ while (delivery.outcome === "accepted") {
522 โ โ yield* Effect.yieldNow;
523 โ โ delivery = (yield* service.getWebhookDelivery({
524 โ โ id: task.id,
525 โ โ deliveryId: deliveryId!,
526 โ โ })).delivery;
527 โ โฐโโถ }
528 โ assert.equal(delivery.outcome, "dispatch_failed");
โฐโโโโ
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/files/FileTreeBrowser.tsx:155:5]
150 โ
151 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
152 โ if (!controlledSelectedPath) {
153 โ return;
154 โ }
155 โ setExpandedPaths((current) => {
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
156 โ const ancestors = ancestorPaths(controlledSelectedPath);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found missing/extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadComposer.tsx:773:7]
772 โ if (settingsSheetPresentation.isActive) {
773 โ settingsRoutePresentation.present(settingsRouteSession);
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโโ
ยท โฐโโ Missing dependency `settingsRoutePresentation`
774 โ }
775 โ }, [settingsRoutePresentation.present, settingsRouteSession, settingsSheetPresentation.isActive]);
ยท โโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโ
ยท โฐโโ Overly precise dependency `settingsRoutePresentation.present`, use `settingsRoutePresentation` instead
776 โ
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should. Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/ThreadComposer.tsx:391:5]
387 โ typeof props.style.borderRadius === "number" ? props.style.borderRadius : 0;
388 โ const animatedBorderRadius = useSharedValue(targetBorderRadius);
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `animatedBorderRadius` originates here
389 โ const shouldAnimate = props.animateLayout !== false && Platform.OS !== "android";
390 โ useLayoutEffect(() => {
391 โ animatedBorderRadius.value = shouldAnimate
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ `animatedBorderRadius` cannot be modified
392 โ ? withTiming(targetBorderRadius, {
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/components/CommandPaletteThreadItems.tsx:152:58]
151 โ if (threadSearchQuery.trim().length > 0 && !threadSearchQuery.startsWith(">") && !isOpen) {
152 โ claimThreadOpen(thread.environmentId, thread.id, Date.now());
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
153 โ }
โฐโโโโ
help: `Date.now` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/files/SourceFileSurface.tsx:257:7]
256 โ return () => cancelAnimationFrame(frame);
257 โ }, [props.path, targetIndex]);
ยท โโโโโโฌโโโโ
ยท โฐโโ Unnecessary dependency `props.path`
258 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(no-array-index-key): Usage of Array index in keys is not allowed
โญโ[apps/mobile/src/features/files/SourceFileSurface.tsx:293:23]
292 โ <SourceText
293 โ key={`${index}:${tokenIndex}`}
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
294 โ style={{
โฐโโโโ
help: Use a unique data-dependent key to avoid unnecessary rerenders
โ react(no-array-index-key): Usage of Array index in keys is not allowed
โญโ[apps/mobile/src/features/files/SourceFileSurface.tsx:309:27]
308 โ return (
309 โ <SourceText key={index}>
ยท โโโโโโโโโโโ
310 โ {body}
โฐโโโโ
help: Use a unique data-dependent key to avoid unnecessary rerenders
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/use-composer-command-menu.ts:217:7]
213 โ };
214 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
215 โ const inserted = ownerKey ? readComposerDraftSelection(ownerKey, draftMessage) : null;
216 โ if (inserted) {
217 โ setSelection((current) =>
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
218 โ current.start === inserted.start && current.end === inserted.end ? current : inserted,
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/use-composer-command-menu.ts:298:7]
286 โ }, [hasWorkspaceSnapshot]);
287 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
288 โ if (!environmentId || !projectCwd || !selectedProviderInstanceId) return;
289 โ const key = `${environmentId}:${selectedProviderInstanceId}:${projectCwd}`;
290 โ const now = Date.now();
291 โ const lastRequest = workspaceRefreshKeyRef.current;
292 โ if (
293 โ lastRequest?.key === key &&
294 โ now - lastRequest.requestedAt < PROVIDER_WORKSPACE_SNAPSHOT_TTL_MS
295 โ )
296 โ return;
297 โ if (hasCurrentProviderWorkspaceSnapshot(selectedProviderStatus, projectCwd, now)) {
298 โ setWorkspaceRefreshRetry(null);
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
299 โ return;
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/use-composer-command-menu.ts:328:5]
327 โ }, [
328 โ draftMessage,
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `draftMessage`
329 โ environmentId,
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ eslint(no-unused-vars): Function 'describeSchedule' is declared but never used.
โญโ[apps/server/src/scheduledTasks/Schedule.ts:79:10]
78 โ
79 โ function describeSchedule(schedule: ScheduledTaskSchedule): string {
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ 'describeSchedule' is declared here
80 โ if (schedule.type === "webhook") return "On webhook";
โฐโโโโ
help: Consider removing this declaration.
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/keyboard/HardwareKeyboardCommandProvider.tsx:116:34]
115 โ return [...commands];
116 โ }, [activeThreadRef, pathname, registrationVersion, navigation]);
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `registrationVersion`
117 โ
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:361:7]
355 โ }, [cancelledIncomingShareId, navigation, props.incomingShareId]);
356 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
357 โ if (!isReturningToProjectPicker) {
358 โ return;
359 โ }
360 โ if (requestedInitialProjectAvailable) {
361 โ setIsReturningToProjectPicker(false);
ยท โโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
362 โ return;
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:504:9]
447 โ
448 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
449 โ if (props.pendingTaskId || props.draftId) {
450 โ return;
451 โ }
452 โ if (lastInitialProjectRefRef.current !== props.initialProjectRef) {
453 โ lastInitialProjectRefRef.current = props.initialProjectRef;
454 โ appliedInitialProjectKeyRef.current = null;
455 โ }
456 โ const initialEnvironmentId = props.initialProjectRef?.environmentId;
457 โ const initialProjectId = props.initialProjectRef?.projectId;
458 โ if (initialEnvironmentId && initialProjectId) {
459 โ const directProject =
460 โ projects.find(
461 โ (project) =>
462 โ project.environmentId === initialEnvironmentId && project.id === initialProjectId,
463 โ ) ?? null;
464 โ
465 โ if (directProject) {
466 โ // Apply the route's project once. Re-applying on every change would
467 โ // instantly revert environment/project switches made in the picker.
468 โ const directProjectKey = `${directProject.environmentId}:${directProject.id}`;
469 โ if (appliedInitialProjectKeyRef.current === directProjectKey) {
470 โ return;
471 โ }
472 โ if (props.initialProjectRef?.branch) {
473 โ if (
474 โ selectedProject?.environmentId !== directProject.environmentId ||
475 โ selectedProject.id !== directProject.id
476 โ ) {
477 โ setProject(directProject);
478 โ return;
479 โ }
480 โ if (!flow.draftKey) return;
481 โ // The route completes checkout before mounting this composer. Local
482 โ // mode reuses an existing worktree; worktree mode would create another.
483 โ updateComposerDraftSettings(flow.draftKey, {
484 โ workspaceSelection: {
485 โ mode: "local",
486 โ branch: props.initialProjectRef.branch,
487 โ worktreePath: props.initialProjectRef.worktreePath ?? null,
488 โ startFromOrigin: false,
489 โ },
490 โ });
491 โ }
492 โ appliedInitialProjectKeyRef.current = directProjectKey;
493 โ if (
494 โ selectedProject?.environmentId === directProject.environmentId &&
495 โ selectedProject.id === directProject.id
496 โ ) {
497 โ return;
498 โ }
499 โ setProject(directProject);
500 โ return;
501 โ }
502 โ
503 โ if (projects.length > 0) {
504 โ setIsReturningToProjectPicker(true);
ยท โโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
505 โ }
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:604:5]
545 โ
546 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
547 โ const shareId = props.incomingShareId;
548 โ const draftKey = flow.draftKey;
549 โ const destinationProject = selectedProject;
550 โ const initialEnvironmentId = props.initialProjectRef?.environmentId;
551 โ const initialProjectId = props.initialProjectRef?.projectId;
552 โ const selectedProjectMatchesRoute =
553 โ !initialEnvironmentId ||
554 โ !initialProjectId ||
555 โ (destinationProject?.environmentId === initialEnvironmentId &&
556 โ destinationProject.id === initialProjectId);
557 โ if (
558 โ !shareId ||
559 โ !draftKey ||
560 โ !destinationProject ||
561 โ !selectedProjectMatchesRoute ||
562 โ cancelledIncomingShareId === shareId
563 โ ) {
564 โ return;
565 โ }
566 โ const importKey = `${shareId}:${draftKey}`;
567 โ if (
568 โ startedShareImportKeyRef.current === importKey ||
569 โ cancellingShareImportKeyRef.current === importKey
570 โ ) {
571 โ return;
572 โ }
573 โ
574 โ if (!incomingShare) {
575 โ if (isIncomingShareUnavailable && alertedUnavailableIncomingShareIdRef.current !== shareId) {
576 โ alertedUnavailableIncomingShareIdRef.current = shareId;
577 โ Dialog.alert(
578 โ "Shared content unavailable",
579 โ "The shared content is no longer in the inbox. You can continue editing this task draft.",
580 โ );
581 โ }
582 โ return;
583 โ }
584 โ
585 โ if (
586 โ incomingShare.attachments.some((attachment) => attachment.type === "file") &&
587 โ selectedEnvironmentServerConfig === null
588 โ ) {
589 โ return;
590 โ }
591 โ
592 โ if (alertedUnavailableIncomingShareIdRef.current === shareId) {
593 โ alertedUnavailableIncomingShareIdRef.current = null;
594 โ }
595 โ startedShareImportKeyRef.current = importKey;
596 โ const draftBackup =
597 โ shareImportDraftBackupRef.current.get(importKey) ?? getComposerDraftSnapshot(draftKey);
598 โ shareImportDraftBackupRef.current.set(importKey, draftBackup);
599 โ const importToken = Symbol(importKey);
600 โ let didReserveShare = false;
601 โ let didConsumeShare = false;
602 โ let needsDraftRestore = false;
603 โ activeShareImportTokenRef.current = importToken;
604 โ setImportingShareKey(importKey);
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
605 โ void (async () => {
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:524:5]
523 โ props.initialProjectRef,
524 โ props.incomingShareId,
ยท โโโโโโโโโโโฌโโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `props.incomingShareId`
525 โ props.pendingTaskId,
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found missing/extra effect dependencies
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:543:5]
542 โ loadedBranchesProjectKeyRef.current = projectKey;
543 โ flow.loadBranches();
ยท โโโฌโ
ยท โฐโโ Missing dependency `flow`
544 โ }, [flow.loadBranches, selectedProject]);
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Overly precise dependency `flow.loadBranches`, use `flow` instead
545 โ
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should. Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/NewTaskDraftScreen.tsx:766:5]
765 โ flow.draftKey,
766 โ hasImportedIncomingShare,
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `hasImportedIncomingShare`
767 โ incomingShare,
768 โ isIncomingShareInboxLoading,
ยท โโโโโโโโโโโโโโฌโโโโโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `isIncomingShareInboxLoading`
769 โ isIncomingShareUnavailable,
770 โ props.incomingShareId,
771 โ props.initialProjectRef?.environmentId,
772 โ props.initialProjectRef?.projectId,
773 โ releaseShareReservation,
774 โ reserveShare,
775 โ selectedEnvironmentServerConfig,
776 โ selectedProject,
777 โ shareImportAttempt,
ยท โโโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Unnecessary dependency `shareImportAttempt`
778 โ ]);
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ eslint(no-empty-pattern): Empty array binding pattern
โญโ[apps/server/src/pullRequest/PullRequestService.test.ts:1437:12]
1436 โ
1437 โ for (const [] of [
ยท โโ
1438 โ ["github", "github.com"],
โฐโโโโ
help: Passing non-iterable values (null, undefined, numbers, booleans, etc.) will result in a runtime error because these values are not iterable.
โ react(no-object-type-as-default-prop): Do not use an array literal as default prop value. Use a stable reference instead.
โญโ[apps/web/src/components/pullRequest/PullRequestListFilters.tsx:473:19]
472 โ onFilters,
473 โ authorOptions = [],
ยท โโ
474 โ labelOptions = [],
โฐโโโโ
help: Default values are re-created on every render and break referential equality, causing unnecessary re-renders. Move the value out of the component or memoize it.
โ react(no-object-type-as-default-prop): Do not use an array literal as default prop value. Use a stable reference instead.
โญโ[apps/web/src/components/pullRequest/PullRequestListFilters.tsx:474:18]
473 โ authorOptions = [],
474 โ labelOptions = [],
ยท โโ
475 โ optionScope,
โฐโโโโ
help: Default values are re-created on every render and break referential equality, causing unnecessary re-renders. Move the value out of the component or memoize it.
โ eslint(no-unused-vars): Variable 'layer' is declared but never used. Unused variables should start with a '_'.
โญโ[apps/server/src/serverRuntimeStartup.ts:728:7]
727 โ
728 โ const layer = layerWithOptions();
ยท โโโฌโโ
ยท โฐโโ 'layer' is declared here
โฐโโโโ
help: Consider removing this declaration.
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/components/sidebar/SidebarThreadRow.tsx:372:47]
371 โ const { open, onOpenChange, onSnooze, timestampFormat } = props;
372 โ const presets = open ? resolveSnoozePresets(new Date(), timestampFormat) : [];
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
373 โ return (
โฐโโโโ
help: `Date` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/components/sidebar/SidebarThreadRow.tsx:853:39]
852 โ if (snoozeGateClock.thread !== thread) {
853 โ setSnoozeGateClock({ thread, now: new Date().toISOString() });
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
854 โ }
โฐโโโโ
help: `Date` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadMarkdownImage.tsx:57:5]
55 โ
56 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
57 โ setDecodedSize(null);
ยท โโโโโโโโฌโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
58 โ }, [props.sourceKey]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadMarkdownImage.tsx:61:5]
59 โ
60 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
61 โ setFailedUri(null);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
62 โ }, [props.uri]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ eslint(no-unused-vars): Identifier 'ActivityIndicator' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadMarkdownImage.tsx:7:10]
6 โ import { parse, SvgAst, type JsxAST } from "react-native-svg";
7 โ import { ActivityIndicator, Pressable, StyleSheet, View, type ViewStyle } from "react-native";
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ 'ActivityIndicator' is imported here
8 โ
โฐโโโโ
help: Consider removing this import.
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadMarkdownImage.tsx:58:7]
57 โ setDecodedSize(null);
58 โ }, [props.sourceKey]);
ยท โโโโโโโโฌโโโโโโโ
ยท โฐโโ Unnecessary dependency `props.sourceKey`
59 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadMarkdownImage.tsx:62:7]
61 โ setFailedUri(null);
62 โ }, [props.uri]);
ยท โโโโโฌโโโโ
ยท โฐโโ Unnecessary dependency `props.uri`
63 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/floating-working-control.tsx:111:5]
107 โ );
108 โ const separationProgress = useSharedValue(props.showScrollToEnd ? 1 : 0);
ยท โโโโโโโโโโฌโโโโโโโโ
ยท โฐโโ `separationProgress` originates here
109 โ
110 โ useEffect(() => {
111 โ separationProgress.value = withTiming(props.showScrollToEnd ? 1 : 0, CONTROL_TIMING);
ยท โโโโโโโโโโฌโโโโโโโโ
ยท โฐโโ `separationProgress` cannot be modified
112 โ }, [props.showScrollToEnd, separationProgress]);
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/floating-working-control.tsx:138:5]
128 โ // and repeatedly retargets the animation as it grows.
129 โ const capsuleWidth = useSharedValue<number | null>(null);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `capsuleWidth` originates here
130 โ const measuredWidthRef = useRef<number | null>(null);
131 โ const handleLabelLayout = (event: LayoutChangeEvent) => {
132 โ const width = event.nativeEvent.layout.width;
133 โ if (width === measuredWidthRef.current) {
134 โ return;
135 โ }
136 โ const first = measuredWidthRef.current === null;
137 โ measuredWidthRef.current = width;
138 โ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `capsuleWidth` cannot be modified
139 โ };
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/floating-working-control.tsx:145:7]
128 โ // and repeatedly retargets the animation as it grows.
129 โ const capsuleWidth = useSharedValue<number | null>(null);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `capsuleWidth` originates here
130 โ const measuredWidthRef = useRef<number | null>(null);
131 โ const handleLabelLayout = (event: LayoutChangeEvent) => {
132 โ const width = event.nativeEvent.layout.width;
133 โ if (width === measuredWidthRef.current) {
134 โ return;
135 โ }
136 โ const first = measuredWidthRef.current === null;
137 โ measuredWidthRef.current = width;
138 โ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
139 โ };
140 โ const hasStatus = props.status !== null;
141 โ const hasCapsule = hasStatus || hasQueue || hasAgents || hasPreview;
142 โ useEffect(() => {
143 โ if (!hasStatus) {
144 โ measuredWidthRef.current = null;
145 โ capsuleWidth.value = null;
ยท โโโโโโโฌโโโโโ
ยท โฐโโ `capsuleWidth` cannot be modified
146 โ }
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): Cannot modify local variables after render completes
โญโ[apps/mobile/src/features/threads/floating-working-control.tsx:145:7]
141 โ const hasCapsule = hasStatus || hasQueue || hasAgents || hasPreview;
142 โ useEffect(() => {
ยท โโโโฌโโโ
ยท โฐโโ This function may (indirectly) reassign or modify `capsuleWidth` after render
143 โ if (!hasStatus) {
144 โ measuredWidthRef.current = null;
145 โ capsuleWidth.value = null;
ยท โโโโโโโฌโโโโโ
ยท โฐโโ This modifies `capsuleWidth`
146 โ }
โฐโโโโ
help: This argument is a function which may reassign or mutate `capsuleWidth` after render, which can cause inconsistent behavior on subsequent renders. Consider using state instead
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): Cannot modify local variables after render completes
โญโ[apps/mobile/src/features/threads/floating-working-control.tsx:138:5]
130 โ const measuredWidthRef = useRef<number | null>(null);
131 โ const handleLabelLayout = (event: LayoutChangeEvent) => {
ยท โโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโ
ยท โฐโโ This function may (indirectly) reassign or modify `capsuleWidth` after render
132 โ const width = event.nativeEvent.layout.width;
133 โ if (width === measuredWidthRef.current) {
134 โ return;
135 โ }
136 โ const first = measuredWidthRef.current === null;
137 โ measuredWidthRef.current = width;
138 โ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ This modifies `capsuleWidth`
139 โ };
โฐโโโโ
help: This argument is a function which may reassign or mutate `capsuleWidth` after render, which can cause inconsistent behavior on subsequent renders. Consider using state instead
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2143:5]
2141 โ );
2142 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
2143 โ setExpandedVideo(null);
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
2144 โ setExpandedFile(null);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2644:9]
2635 โ const stopFollowingEnd = useEffectEvent(() => setEndFollow(false));
2636 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
2637 โ if (!jumpToMessageId || loadedListKey !== listMountKey) return;
2638 โ const index = presentedFeed.findIndex(
2639 โ (entry) => entry.type === "message" && entry.id === jumpToMessageId,
2640 โ );
2641 โ if (index === -1) {
2642 โ const runId = messageFoldRunId(props.feed, jumpToMessageId);
2643 โ if (runId && !expandedTurnIds.has(runId)) {
2644 โ setInteractionState((current) => ({
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
2645 โ ...current,
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ eslint(no-unused-vars): Identifier 'isMarkdownFileLinkLabel' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:66:10]
65 โ import { resolveAssetUrl } from "@t3tools/client-runtime/state/assets";
66 โ import { isMarkdownFileLinkLabel } from "@t3tools/shared/markdownLinks";
ยท โโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ 'isMarkdownFileLinkLabel' is imported here
67 โ import { formatAttachmentSize } from "@t3tools/client-runtime/state/attachments";
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'createContext' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:92:3]
91 โ import {
92 โ createContext,
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ 'createContext' is imported here
93 โ memo,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'ActivityIndicator' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:108:3]
107 โ import {
108 โ ActivityIndicator,
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ 'ActivityIndicator' is imported here
109 โ Image,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'ScrollView' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:115:3]
114 โ Pressable,
115 โ ScrollView,
ยท โโโโโโฌโโโโ
ยท โฐโโ 'ScrollView' is imported here
116 โ StyleSheet,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'NativeText' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:117:11]
116 โ StyleSheet,
117 โ Text as NativeText,
ยท โโโโโโฌโโโโ
ยท โฐโโ 'NativeText' is imported here
118 โ type ColorValue,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'tryOpenExternalUrl' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:140:10]
139 โ import { copyTextWithHaptic } from "../../lib/copyTextWithHaptic";
140 โ import { tryOpenExternalUrl } from "../../lib/openExternalUrl";
ยท โโโโโโโโโโฌโโโโโโโโ
ยท โฐโโ 'tryOpenExternalUrl' is imported here
141 โ import { downloadAndShareAttachment } from "../../lib/attachmentDownload";
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'faviconUrlForOrigin' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:143:10]
142 โ import { hasWideMarkdownBlock } from "../../lib/wideMarkdownBlocks";
143 โ import { faviconUrlForOrigin } from "@t3tools/shared/favicon";
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ 'faviconUrlForOrigin' is imported here
144 โ import {
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'MediaVideoPlayer' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:157:10]
156 โ import { VideoAttachmentTile } from "../../components/VideoAttachmentTile";
157 โ import { MediaVideoPlayer } from "../../components/MediaVideoPlayer";
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ 'MediaVideoPlayer' is imported here
158 โ import { resolveMarkdownMediaPreview } from "../../lib/markdownMedia";
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'mediaVideoPreviewUri' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:161:3]
160 โ attachmentVideoPreviewSource,
161 โ mediaVideoPreviewUri,
ยท โโโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ 'mediaVideoPreviewUri' is imported here
162 โ mediaVideoThumbnailKey,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'mediaVideoThumbnailKey' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:162:3]
161 โ mediaVideoPreviewUri,
162 โ mediaVideoThumbnailKey,
ยท โโโโโโโโโโโโฌโโโโโโโโโโ
ยท โฐโโ 'mediaVideoThumbnailKey' is imported here
163 โ type MediaVideoPreviewSource,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Type 'MediaVideoPreviewSource' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:163:8]
162 โ mediaVideoThumbnailKey,
163 โ type MediaVideoPreviewSource,
ยท โโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ 'MediaVideoPreviewSource' is imported here
164 โ } from "../../lib/videoPreviewSource";
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Type 'ReviewDiffTheme' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:171:15]
170 โ import { parseReviewCommentMessageSegments } from "../review/reviewCommentSelection";
171 โ import type { ReviewDiffTheme } from "../review/shikiReviewHighlighter";
ยท โโโโโโโโฌโโโโโโโ
ยท โฐโโ 'ReviewDiffTheme' is imported here
172 โ import {
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'resolveMarkdownFontSizes' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:186:3]
185 โ import {
186 โ resolveMarkdownFontSizes,
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ 'resolveMarkdownFontSizes' is imported here
187 โ resolveNativeMarkdownTypography,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'resolveMarkdownInlineCodePresentation' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:193:3]
192 โ normalizeNativeMarkdownUrl,
193 โ resolveMarkdownInlineCodePresentation,
ยท โโโโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโโโ
ยท โฐโโ 'resolveMarkdownInlineCodePresentation' is imported here
194 โ resolveMarkdownLinkPresentation,
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'useMarkdownCodeHighlight' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:233:10]
232 โ import { ThreadMcpApp } from "./McpAppWebView";
233 โ import { useMarkdownCodeHighlight } from "./markdownCodeHighlightState";
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ 'useMarkdownCodeHighlight' is imported here
234 โ import {
โฐโโโโ
help: Consider removing this import.
โ eslint(no-unused-vars): Identifier 'useAssetUrlState' is imported but never used.
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:237:3]
236 โ useAssetUrl,
237 โ useAssetUrlState,
ยท โโโโโโโโโฌโโโโโโโ
ยท โฐโโ 'useAssetUrlState' is imported here
238 โ useRefreshAssetUrl,
โฐโโโโ
help: Consider removing this import.
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2145:7]
2144 โ setExpandedFile(null);
2145 โ }, [props.environmentId, props.threadId, props.contentPresentation.kind]);
ยท โโโโโโโโโโฌโโโโโโโโโ โโโโโโโโฌโโโโโโ โโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโ
ยท โ โ โฐโโ Unnecessary dependency `props.contentPresentation.kind`
ยท โ โฐโโ Unnecessary dependency `props.threadId`
ยท โฐโโ Unnecessary dependency `props.environmentId`
2146 โ const horizontalPadding = props.layoutVariant === "split" ? 20 : 16;
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2613:7]
2612 โ reportHeaderMaterialVisibility(false);
2613 โ }, [feedThreadKey, reportHeaderMaterialVisibility]);
ยท โโโโโโโฌโโโโโโ
ยท โฐโโ Unnecessary dependency `feedThreadKey`
2614 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2629:7]
2628 โ }
2629 โ }, [listMountKey, props.contentInsetEndAdjustment, listRef]);
ยท โโโโโโโฌโโโโโ
ยท โฐโโ Unnecessary dependency `listMountKey`
2630 โ // Opening at a message: its run unfolds, the list stops following the end
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:2825:7]
2824 โ }
2825 โ }, [expandedTurnIds, expandedWorkGroups, expandedWorkRows, settleDisclosureAfterLayout]);
ยท โโโโโโโโฌโโโโโโโ โโโโโโโโโโฌโโโโโโโโ โโโโโโโโโฌโโโโโโโ
ยท โ โ โฐโโ Unnecessary dependency `expandedWorkRows`
ยท โ โฐโโ Unnecessary dependency `expandedWorkGroups`
ยท โฐโโ Unnecessary dependency `expandedTurnIds`
2826 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/threads/ThreadFeed.tsx:617:9]
616 โ };
617 โ }, [props.environmentId, attachment.id, httpBaseUrl]),
ยท โโโโโโโโโโฌโโโโโโโโโ โโโโโโโฌโโโโโโ โโโโโโฌโโโโโ
ยท โ โ โฐโโ Unnecessary dependency `httpBaseUrl`
ยท โ โฐโโ Unnecessary dependency `attachment.id`
ยท โฐโโ Unnecessary dependency `props.environmentId`
618 โ );
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:569:5]
567 โ }, []);
568 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
569 โ setKeyboardStateSuspect(false);
ยท โโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
570 โ }, [isKeyboardVisible, liveKeyboardHeight]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:965:7]
962 โ const [lastKnownKeyboardHeight, setLastKnownKeyboardHeight] = useState(0);
963 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
964 โ if (liveKeyboardHeight > 0 && liveKeyboardHeight !== lastKnownKeyboardHeight) {
965 โ setLastKnownKeyboardHeight(liveKeyboardHeight);
ยท โโโโโโโโโโโโโโฌโโโโโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
966 โ }
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:1221:5]
1219 โ
1220 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
1221 โ setAnchorMessageId(null);
ยท โโโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
1222 โ setSubmittedMessageId(null);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:570:7]
569 โ setKeyboardStateSuspect(false);
570 โ }, [isKeyboardVisible, liveKeyboardHeight]);
ยท โโโโโโโโโฌโโโโโโโโ โโโโโโโโโโฌโโโโโโโโ
ยท โ โฐโโ Unnecessary dependency `liveKeyboardHeight`
ยท โฐโโ Unnecessary dependency `isKeyboardVisible`
571 โ const handleOwnedInputFocusChange = (focused: boolean) => {
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:1169:7]
1168 โ userInputInsetProgress.set(1);
1169 โ }, [activeUserInputRequestId, userInputCardProgress, userInputInsetProgress]);
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `activeUserInputRequestId`
1170 โ const showContent = props.showContent ?? true;
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(exhaustive-effect-dependencies): Found extra effect dependencies
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:1227:15]
1226 โ freeze.set(false);
1227 โ }, [freeze, selectedThreadKey]);
ยท โโโโโโโโโฌโโโโโโโโ
ยท โฐโโ Unnecessary dependency `selectedThreadKey`
1228 โ
โฐโโโโ
help: Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:1057:5]
1052 โ const composerPopoverHeight = useSharedValue(0);
1053 โ const floatingControlCoverage = useSharedValue(
ยท โโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ `floatingControlCoverage` originates here
1054 โ showFloatingStatus ? FLOATING_WORKING_CONTROL_COVERAGE : 0,
1055 โ );
1056 โ useEffect(() => {
1057 โ floatingControlCoverage.value = withTiming(
ยท โโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ `floatingControlCoverage` cannot be modified
1058 โ showFloatingStatus ? FLOATING_WORKING_CONTROL_COVERAGE : 0,
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(immutability): This value cannot be modified
โญโ[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:1075:7]
1065 โ const userInputCoverageApplies = Platform.OS === "ios" && activeUserInputRequestId !== null;
1066 โ const combinedContentInsetEndAdjustment = useSharedValue(
ยท โโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโ
ยท โฐโโ `combinedContentInsetEndAdjustment` originates here
1067 โ Math.max(0, estimatedOverlayHeight - nativeInsetOvercount),
1068 โ );
1069 โ useAnimatedReaction(
1070 โ () =>
1071 โ contentInsetEndAdjustment.value +
1072 โ floatingControlCoverage.value +
1073 โ (userInputCoverageApplies ? userInputInsetProgress.value * userInputCardCoverage.value : 0),
1074 โ (value) => {
1075 โ combinedContentInsetEndAdjustment.value = value;
ยท โโโโโโโโโโโโโโโโโฌโโโโโโโโโโโโโโโโ
ยท โฐโโ `combinedContentInsetEndAdjustment` cannot be modified
1076 โ },
โฐโโโโ
help: Modifying a value returned from a hook is not allowed. Consider moving the modification into the hook where the value is constructed
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/immutability
โ react(purity): Cannot call impure function during render
โญโ[apps/mobile/src/features/threads/thread-list-v2-items.tsx:698:76]
697 โ const [snoozeGateClock, setSnoozeGateClock] = useState(() => ({ thread, now: Date.now() }));
698 โ if (snoozeGateClock.thread !== thread) setSnoozeGateClock({ thread, now: Date.now() });
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
699 โ const snoozeGateNow = new Date(snoozeGateClock.now).toISOString();
โฐโโโโ
help: `Date.now` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(purity): Cannot call impure function during render
โญโ[apps/mobile/src/features/threads/thread-list-v2-items.tsx:719:71]
718 โ const snoozePresets = useMemo(
719 โ () => (swipeActions.secondary === "snooze" ? resolveSnoozePresets(new Date()) : ([] as const)),
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
720 โ [props.snoozePresetMinute, swipeActions.secondary],
โฐโโโโ
help: `Date` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/threads/thread-list-v2-items.tsx:720:6]
719 โ () => (swipeActions.secondary === "snooze" ? resolveSnoozePresets(new Date()) : ([] as const)),
720 โ [props.snoozePresetMinute, swipeActions.secondary],
ยท โโโโโโโโโโโโโฌโโโโโโโโโโโ
ยท โฐโโ Unnecessary dependency `props.snoozePresetMinute`
721 โ );
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(memo-dependencies): Found extra memoization dependencies
โญโ[apps/mobile/src/features/threads/thread-list-v2-items.tsx:766:7]
765 โ thread.pinnedAt,
766 โ variant,
ยท โโโโฌโโโ
ยท โฐโโ Unnecessary dependency `variant`
767 โ ],
โฐโโโโ
help: Extra dependencies can cause a value to update more often than it should, resulting in performance problems such as excessive renders or effects firing too often
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ react(no-array-index-key): Usage of Array index in keys is not allowed
โญโ[apps/mobile/src/features/threads/thread-list-v2-items.tsx:1216:21]
1215 โ {providerDrivers.slice(0, -1).map((driver, index) => (
1216 โ <View key={`${driver}:${index}`} className="-mr-1 opacity-30">
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโ
1217 โ <ProviderIcon provider={driver} size={12} />
โฐโโโโ
help: Use a unique data-dependent key to avoid unnecessary rerenders
โ react(purity): Cannot call impure function during render
โญโ[apps/web/src/components/Sidebar.tsx:1136:11]
1135 โ foldSearchQuery(threadSearchQuery),
1136 โ Date.now(),
ยท โโโโโโฌโโโโ
ยท โฐโโ Cannot call impure function
1137 โ // A list filter, as on mobile: a pick would move rows unexplained.
โฐโโโโ
help: `Date.now` is an impure function. Calling an impure function can produce unstable results that update unpredictably when the component re-renders
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/purity
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/use-worktree-setup.ts:30:40]
28 โ const snapshot = resolveWorktreeSetupSnapshot(input.threadId, query.data, live);
29 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
30 โ if (snapshot && snapshot !== live) setHeld({ key, snapshot });
ยท โโโโฌโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
31 โ }, [key, live, snapshot]);
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[apps/server/src/project/ProjectService.test.ts:94:3]
93 โ ) {
94 โ โญโโถ for (let attempt = 0; attempt < 100; attempt += 1) {
95 โ โ const project = Option.getOrThrow(yield* service.getById(projectId));
96 โ โ if (predicate(project)) return project;
97 โ โ yield* Effect.yieldNow;
98 โ โฐโโถ }
99 โ return yield* Effect.die(`Project ${projectId} was not enriched in time.`);
โฐโโโโ
โ react(exhaustive-effect-dependencies): Found missing/extra effect dependencies
โญโ[apps/mobile/src/features/threads/NewTaskContextPickerScreens.tsx:249:7]
248 โ mountedRef.current = false;
249 โ flow.setBranchQuery("");
ยท โโโฌโ
ยท โฐโโ Missing dependency `flow`
250 โ };
251 โ }, [flow.setBranchQuery]);
ยท โโโโโโโโโโฌโโโโโโโโโ
ยท โฐโโ Overly precise dependency `flow.setBranchQuery`, use `flow` instead
252 โ
โฐโโโโ
help: Missing dependencies can cause an effect to fire less often than it should. Extra dependencies can cause an effect to fire more often than it should, resulting in performance problems such as excessive renders and side effects
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/exhaustive-deps
โ t3code(no-polling-loop-in-tests): Polling loop in a test. Spinning on `yieldNow`/`sleep` is scheduler- and load-dependent. Await the `Deferred`, `Queue.take`, `Stream` element, `drain()`, or `expect.poll` that marks the milestone.
โญโ[apps/server/src/project/ProjectEnrichmentService.test.ts:28:3]
27 โ ) {
28 โ โญโโถ for (let attempt = 0; attempt < 100; attempt += 1) {
29 โ โ const available = yield* service.peek(workspaceRoot);
30 โ โ if (predicate(available)) return available;
31 โ โ yield* Effect.yieldNow;
32 โ โฐโโถ }
33 โ return yield* Effect.die(`Project metadata for ${workspaceRoot} was not resolved in time.`);
โฐโโโโ
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/process/externalLauncher.test.ts:186:12]
185 โ
186 โ for (const {} of ["darwin", "linux"] as const) {
ยท โโ
187 โ }
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/process/externalLauncher.test.ts:189:12]
188 โ
189 โ for (const {} of [
ยท โโ
190 โ { kind: "browser", folderVerb: "none" },
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/process/externalLauncher.test.ts:311:12]
310 โ
311 โ for (const {} of ["rejected", "spawn-error", "stalled"] as const) {
ยท โโ
312 โ }
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ eslint(no-empty-pattern): Empty object binding pattern
โญโ[apps/server/src/process/externalLauncher.test.ts:314:12]
313 โ
314 โ for (const {} of [
ยท โโ
315 โ {
โฐโโโโ
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
โ t3code(no-inline-schema-compile): Hoist Schema.encodeEffect(...) to module scope: both the inline schema literal and the compiled function are rebuilt on every call. Move the compiled function to a module-level const.
โญโ[apps/server/src/serverSettings.test.ts:1741:32]
1740 โ const model = createModelSelection(ProviderInstanceId.make("codex"), "gpt-5.5");
1741 โ const modelJson = yield* Schema.encodeEffect(Schema.fromJsonString(ModelSelection))(model);
ยท โโโโโโโโโโโโโโโโโโโ
1742 โ const scriptsJson = yield* Schema.encodeEffect(
โฐโโโโ
โ t3code(no-inline-schema-compile): Hoist Schema.encodeEffect(...) to module scope: both the inline schema literal and the compiled function are rebuilt on every call. Move the compiled function to a module-level const.
โญโ[apps/server/src/serverSettings.test.ts:1742:34]
1741 โ const modelJson = yield* Schema.encodeEffect(Schema.fromJsonString(ModelSelection))(model);
1742 โ const scriptsJson = yield* Schema.encodeEffect(
ยท โโโโโโโโโโโโโโโโโโโ
1743 โ Schema.fromJsonString(Schema.Array(ProjectScript)),
โฐโโโโ
โ react(capitalized-calls): Capitalized function called without JSX
โญโ[apps/mobile/src/features/threads/ThreadQueueControl.tsx:602:27]
601 โ const onEnd = useEffectEvent((y: number, success: boolean) => props.onEnd(y, success));
602 โ const gesture = Gesture.Pan()
ยท โโฌโ
ยท โฐโโ `Pan` may be a component
603 โ .enabled(!props.disabled)
โฐโโโโ
help: Render `Pan` with JSX if it is a component; otherwise rename it to start with a lowercase letter or allowlist it in the compiler configuration
note: `Pan` is treated as a component because it begins with an uppercase letter; React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/static-components
โ react(no-array-index-key): Usage of Array index in keys is not allowed
โญโ[apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx:818:17]
817 โ <div
818 โ key={`${index}:${check.name}:${check.url ?? ""}`}
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโโ
819 โ className="group flex items-center gap-2 rounded-md pr-1 hover:bg-accent/60"
โฐโโโโ
help: Use a unique data-dependent key to avoid unnecessary rerenders
โ t3code(no-inline-schema-compile): Hoist Schema.decodeUnknownEffect(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
โญโ[apps/server/src/assets/AssetAccess.test.ts:975:35]
974 โ // Every client and server release spells the resource this way on the wire.
975 โ const { resource } = yield* Schema.decodeUnknownEffect(AssetCreateUrlInput)({
ยท โโโโโโโโโโโโโโโโโโโโโโโโโโ
976 โ resource: { _tag: "github-media", cwd: "/repo", url },
โฐโโโโ
โ t3code(no-inline-schema-compile): Hoist Schema.encodeEffect(...) to module scope: both the inline schema literal and the compiled function are rebuilt on every call. Move the compiled function to a module-level const.
โญโ[apps/server/src/assets/AssetAccess.test.ts:1012:32]
1011 โ expect(error._tag).toBe("AssetGitHubMediaUrlValidationError");
1012 โ const encoded = yield* Schema.encodeEffect(Schema.fromJsonString(AssetAccessError))(error);
ยท โโโโโโโโโโโโโโโโโโโ
1013 โ expect(encoded).not.toContain(url);
โฐโโโโ
โ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
โญโ[apps/mobile/src/features/threads/thread-inspector-content-stack.tsx:60:5]
58 โ
59 โ useEffect(() => {
ยท โโโโโฌโโโโ
ยท โฐโโ This is the containing effect
60 โ setMountedModes((current) => {
ยท โโโโโโโโฌโโโโโโโ
ยท โฐโโ Avoid calling setState() directly within an effect
61 โ if (current.has(props.mode)) {
โฐโโโโ
help: Effects should synchronize React with external systems. Calling setState synchronously inside an effect starts another render and is usually unnecessary. Derive the value during render, initialize state directly, or update it from the event that caused the change. Use an effect only when synchronizing with an external system.
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect
Found 0 errors and 367 warnings in 5478 files (5.9s, 8 threads)
Post job cleanup.
Cache hit occurred on the primary key electron-Linux-X64-854e11ddd491bd4f350eebb67a74265df119376a3984c8e153cfe5fc8ced6309, not saving cache.
Post job cleanup.
##[start-action display=Cache full pnpm store;id=__c17805a4-df81-4837-925b-a1d05d7aa5c8.__actions_cache]
Post job cleanup.
Cache hit occurred on the primary key pnpm-full-v1-Linux-X64-fa50cf9b003d547cd32552f61b874eb4abe5dace3fdb4b1abd9610da2da24074, not saving cache.
##[end-action id=__c17805a4-df81-4837-925b-a1d05d7aa5c8.__actions_cache;outcome=success;conclusion=success;duration_ms=106]
##[start-action display=Cache Vite+;id=__c17805a4-df81-4837-925b-a1d05d7aa5c8.vp_cache]
Post job cleanup.
Cache hit occurred on the primary key vite-plus-home-v2-Linux-X64-1.3.0-node26.12.1-pnpm@12.14.0, not saving cache.
##[end-action id=__c17805a4-df81-4837-925b-a1d05d7aa5c8.vp_cache;outcome=success;conclusion=success;duration_ms=70]
Post job cleanup.
[command]/usr/local/bin/git version
git version 2.55.0
Temporarily overriding HOME='/home/runner/_work/_temp/0c3b734d-d689-4807-a317-88263583a476' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/local/bin/git config --global --add safe.directory /home/runner/_work/t3code/t3code
Removing SSH command configuration
[command]/usr/local/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/local/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
fatal: No url found for submodule path '.repos/alchemy-effect/submodules/distilled' in .gitmodules
Warning: The process '/usr/local/bin/git' failed with exit code 128
A job completed hook has been configured by the self-hosted runner administrator
โธ Run '/job_completed.sh'
shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
env:
BLACKSMITH_RUNNER_MESSAGE_WAIT_MS: 147
BLACKSMITH_RUNNER_ACQUIRE_JOB_MS: 453
BLACKSMITH_CHECK_RUN_ID: 120875719752
BLACKSMITH_JOB_DISPLAY_NAME: Lint
GITHUB_ACTIONS_CACHE_MODE: write
Docker container cache not activated, skipping hash computation.
No active SSH sessions detected. Proceeding with shutdown immediately.
Cleaning up orphan processes