GitHub check
256 KB of 616 KB loaded Download the full log (616 KB)
360 KB of earlier output not loaded
Z ⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-codex-app-server/src/schema.test.ts:51:18]
50 │ assert.equal(Schema.is(CodexSchema.ServerNotification__CollabAgentTool)(tool), true);
51 │ assert.equal(Schema.is(CodexSchema.V2ThreadResumeResponse__CollabAgentTool)(tool), true);
· ─────────
52 │ }
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-codex-app-server/src/schema.test.ts:55:5]
54 │ assert.equal(
55 │ Schema.is(CodexSchema.ServerNotification__CollabAgentToolCallStatus)("interrupted"),
· ─────────
56 │ true,
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-codex-app-server/src/schema.test.ts:59:5]
58 │ assert.equal(
59 │ Schema.is(CodexSchema.V2ThreadResumeResponse__CollabAgentToolCallStatus)("interrupted"),
· ─────────
60 │ true,
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-codex-app-server/src/schema.test.ts:103:16]
102 │
103 │ assert.equal(Schema.is(CodexSchema.V2ThreadResumeResponse)(resumeResponse), true);
· ─────────
104 │ });
╰────
⚠ 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.
╭─[packages/effect-acp/src/client.ts:1270:11]
1269 │ extNotificationHandlers.set(CLIENT_METHODS.mcp_message, (params) =>
1270 │ Schema.decodeUnknownEffect(AcpSchemaV2.MessageMcpNotification)(params).pipe(
· ──────────────────────────
1271 │ Effect.mapError(() => AcpError.AcpRequestError.invalidParams()),
╰────
⚠ react(memo-dependencies): Found extra memoization dependencies
╭─[apps/mobile/src/features/keyboard/HardwareKeyboardCommandProvider.tsx:122:34]
121 │ return [...commands];
122 │ }, [activeThreadRef, pathname, registrationVersion, navigation]);
· ─────────┬─────────
· ╰── Unnecessary dependency `registrationVersion`
123 │
╰────
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
⚠ 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.`);
╰────
⚠ 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.`);
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-acp/src/schema.test.ts:18:12]
17 │ const command = { name: "plan", description: "Make a plan", input: { hint: "Task" } };
18 │ expect(Schema.is(V1.AvailableCommand)(command)).toBe(true);
· ─────────
19 │ expect(
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.is(...) to module scope: the compiled function is rebuilt on every call. Move it to a module-level const.
╭─[packages/effect-acp/src/schema.test.ts:20:7]
19 │ expect(
20 │ Schema.is(V2.AvailableCommand)({ ...command, input: { type: "text", hint: "Task" } }),
· ─────────
21 │ ).toBe(true);
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.decodeUnknownEffect(...) 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.
╭─[packages/effect-acp/src/protocol.test.ts:500:31]
499 │
500 │ const response = yield* Schema.decodeUnknownEffect(
· ──────────────────────────
501 │ Schema.fromJsonString(JsonRpcErrorResponse),
╰────
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/review/useReviewCommentSelectionController.ts:107:5]
104 │
105 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
106 │ clearReviewCommentTarget();
107 │ setPendingNativeCommentSelection(null);
· ────────────────┬───────────────
· ╰── Avoid calling setState() directly within an effect
108 │ }, [selectedSection?.id]);
╰────
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/review/useReviewCommentSelectionController.ts:112:7]
109 │
110 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
111 │ if (activeCommentTarget === null) {
112 │ setPendingNativeCommentSelection(null);
· ────────────────┬───────────────
· ╰── Avoid calling setState() directly within an effect
113 │ }
╰────
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/review/useReviewCommentSelectionController.ts:108:7]
107 │ setPendingNativeCommentSelection(null);
108 │ }, [selectedSection?.id]);
· ─────────┬─────────
· ╰── Unnecessary dependency `selectedSection?.id`
109 │
╰────
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
⚠ 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:222:5]
221 │ let idlePasses = 0;
222 │ ╭─▶ for (;;) {
223 │ │ yield* Effect.promise(() => Promise.resolve());
224 │ │ const pending = backgroundRuntime.pending.shift();
225 │ │ if (!pending) {
226 │ │ idlePasses++;
227 │ │ if (idlePasses >= 3) {
228 │ │ return;
229 │ │ }
230 │ │ continue;
231 │ │ }
232 │ │ idlePasses = 0;
233 │ │ const exit = yield* Effect.exit(
234 │ │ pending.operation as Effect.Effect<unknown, unknown, ManagedRelay.ManagedRelayClient>,
235 │ │ );
236 │ │ yield* Effect.sync(() => {
237 │ │ pending.resolve(exit);
238 │ │ });
239 │ ╰─▶ }
240 │ },
╰────
⚠ t3code(no-inline-schema-compile): Hoist Schema.decodeUnknownEffect(...) 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.
╭─[packages/effect-acp/src/errors.test.ts:129:28]
128 │ const secret = "acp-summary-secret";
129 │ const cause = yield* Schema.decodeUnknownEffect(
· ──────────────────────────
130 │ Schema.Struct({
╰────
⚠ react(no-array-index-key): Usage of Array index in keys is not allowed
╭─[apps/mobile/src/features/review/ReviewCommentCard.tsx:223:31]
222 │ ? snippetTokens.map((line, lineIndex) => (
223 │ <NativeText key={lineIndex}>
· ───────────────
224 │ {lineIndex > 0 ? "\n" : ""}
╰────
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/review/ReviewCommentCard.tsx:227:25]
226 │ <NativeText
227 │ key={tokenIndex}
· ────────────────
228 │ 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/modules/t3-source-text/src/SourceText.tsx:36:9]
35 │ <T3SourceTextRunNativeComponent
36 │ key={index}
· ───────────
37 │ style={flattenedStyle}
╰────
help: Use a unique data-dependent key to avoid unnecessary rerenders
⚠ eslint(no-empty-pattern): Empty object binding pattern
╭─[apps/server/src/processRunner.test.ts:264:14]
263 │
264 │ for (const {} of ["timeout", "success"] as const) {
· ──
265 │ }
╰────
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
⚠ react(exhaustive-effect-dependencies): Found extra effect dependencies
╭─[apps/mobile/src/features/review/ReviewSheet.tsx:456:7]
455 │ showAuxiliaryPane("inspector");
456 │ }, [environmentId, showAuxiliaryPane, threadId]);
· ──────┬────── ────┬───
· │ ╰── Unnecessary dependency `threadId`
· ╰── Unnecessary dependency `environmentId`
457 │ const {
╰────
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(static-components): Cannot create components during render
╭─[apps/mobile/src/features/review/ReviewSheet.tsx:724:20]
491 │ // Expo may return null before registration or in older binaries; use the fallback.
492 │ const NativeReviewDiffView = resolveNativeReviewDiffView();
· ─────────────┬─────────────
· ╰── The component is created during render here
493 │ const nativeReviewDiffViewRef = useRef<NativeReviewDiffViewHandle>(null);
494 │ const showcasedReviewDrawRef = useRef<string | null>(null);
495 │ const [isPullRefreshing, setIsPullRefreshing] = useState(false);
496 │ const handlePullToRefresh = async () => {
497 │ setIsPullRefreshing(true);
498 │ await refreshSelectedSection();
499 │ setIsPullRefreshing(false);
500 │ };
501 │ const reviewFileNavigatorRef = useRef<ReviewFileNavigatorHandle>(null);
502 │ const reviewFiles = parsedDiff.kind === "files" ? parsedDiff.files : [];
503 │ const fileVisibility = useReviewFileVisibility({
504 │ threadKey: reviewCache.threadKey,
505 │ sectionId: selectedSection?.id ?? null,
506 │ files: reviewFiles,
507 │ cachedExpandedFileIds: selectedSection?.id
508 │ ? reviewCache.expandedFileIdsBySection[selectedSection.id]
509 │ : undefined,
510 │ cachedViewedFileIds: selectedSection?.id
511 │ ? reviewCache.viewedFileIdsBySection[selectedSection.id]
512 │ : undefined,
513 │ });
514 │ const { collapsedFileIds, toggleExpandedFile, toggleViewedFile, viewedFileIds } = fileVisibility;
515 │ const commentSelection = useReviewCommentSelectionController({
516 │ environmentId,
517 │ threadId,
518 │ selectedSection,
519 │ nativeReviewDiffData,
520 │ });
521 │ const nativeBridge = useNativeReviewDiffBridge({
522 │ threadKey: reviewCache.threadKey,
523 │ sectionId: selectedSection?.id ?? null,
524 │ diff: selectedSection?.diff,
525 │ data: nativeReviewDiffData,
526 │ collapsedFileIds,
527 │ viewedFileIds,
528 │ selectedRowIds: commentSelection.selectedRowIds,
529 │ canHighlight: parsedDiff.kind === "files",
530 │ });
531 │ const showcaseReviewKey =
532 │ SHOWCASE_ENABLED && parsedDiff.kind === "files" && selectedSection
533 │ ? `${reviewCache.threadKey}:${selectedSection.id}:${nativeBridge.tokensResetKey}:${nativeBridge.themeId}`
534 │ : null;
535 │ const handleNativeDebug = (event: NativeSyntheticEvent<Record<string, unknown>>) => {
536 │ nativeBridge.onDebug(event);
537 │ if (
538 │ showcaseReviewKey === null ||
539 │ showcasedReviewDrawRef.current === showcaseReviewKey ||
540 │ !isNativeReviewDiffDrawEvent(event.nativeEvent)
541 │ ) {
542 │ return;
543 │ }
544 │ showcasedReviewDrawRef.current = showcaseReviewKey;
545 │ reportShowcaseSceneRendered({ scene: "review", themeId: nativeBridge.themeId });
546 │ };
547 │
548 │ const handleSelectFile = async (fileId: string | null) => {
549 │ loadVisibleFile(fileId, true);
550 │ commentSelection.clearSelection();
551 │ if (fileId !== null && collapsedFileIds.includes(fileId)) {
552 │ toggleExpandedFile(fileId);
553 │ }
554 │ const nativeView = nativeReviewDiffViewRef.current;
555 │ if (!nativeView) return;
556 │ const result = await settlePromise(() =>
557 │ fileId === null ? nativeView.scrollToTop(true) : nativeView.scrollToFile(fileId, true),
558 │ );
559 │ if (AsyncResult.isFailure(result)) {
560 │ console.error("[review] Failed to navigate to diff file", squashAtomCommandFailure(result));
561 │ }
562 │ };
563 │ const handleVisibleFileChange = (
564 │ event: NativeSyntheticEvent<{ readonly fileId?: string | null }>,
565 │ ) => {
566 │ loadVisibleFile(event.nativeEvent.fileId ?? null);
567 │ reviewFileNavigatorRef.current?.setVisibleFile(event.nativeEvent.fileId ?? null);
568 │ };
569 │ const renderInspector = () => (
570 │ <ReviewFileNavigator
571 │ ref={reviewFileNavigatorRef}
572 │ files={nativeReviewDiffData.files}
573 │ headerInset={insets.top}
574 │ sectionId={selectedSection?.id ?? null}
575 │ onSelectFile={(fileId) => void handleSelectFile(fileId)}
576 │ />
577 │ );
578 │
579 │ const handleNativeToggleFile = (event: NativeSyntheticEvent<{ readonly fileId?: string }>) => {
580 │ const { fileId } = event.nativeEvent;
581 │ if (fileId) {
582 │ loadVisibleFile(fileId, true);
583 │ toggleExpandedFile(fileId);
584 │ }
585 │ };
586 │
587 │ const handleNativeToggleViewedFile = (
588 │ event: NativeSyntheticEvent<{ readonly fileId?: string }>,
589 │ ) => {
590 │ const { fileId } = event.nativeEvent;
591 │ if (fileId) {
592 │ toggleViewedFile(fileId);
593 │ }
594 │ };
595 │
596 │ const parsedDiffNotice =
597 │ parsedDiff.kind === "files" || parsedDiff.kind === "raw" ? parsedDiff.notice : null;
598 │ const hasCachedSelectedDiff = selectedSection?.diff != null;
599 │ const hasAnyCachedDiff = reviewSections.some((section) => section.diff != null);
600 │ const sectionMenu = buildReviewSectionMenu(reviewSections);
601 │ const { showConnectionNotice, showSectionToolbar } = resolveReviewAvailability({
602 │ hasEnvironmentPresentation: environment.isReady,
603 │ isEnvironmentConnected: isEnvironmentReady,
604 │ hasCachedSelectedDiff,
605 │ hasAnyCachedDiff,
606 │ });
607 │ const handleRetryEnvironment = () => {
608 │ void retryEnvironment(environmentId);
609 │ };
610 │ const handleReturnToThread = () => {
611 │ if (navigation.canGoBack()) {
612 │ navigation.goBack();
613 │ return;
614 │ }
615 │ selectDetail({
616 │ name: "Thread",
617 │ params: { environmentId: String(environmentId), threadId: String(threadId) },
618 │ });
619 │ };
620 │ const androidHeaderSubtitle = [
621 │ selectedSection?.title,
622 │ headerDiffSummary.additions,
623 │ headerDiffSummary.deletions,
624 │ ]
625 │ .filter((part): part is string => Boolean(part))
626 │ .join(" · ");
627 │
628 │ const showChangedFilesPane =
629 │ !showConnectionNotice &&
630 │ selectedSection !== null &&
631 │ parsedDiff.kind === "files" &&
632 │ NativeReviewDiffView !== null;
633 │ useRegisterWorkspaceInspector(showChangedFilesPane ? renderInspector : undefined);
634 │ const showChangedFilesToggle = panes.supportsAuxiliaryPane && showChangedFilesPane;
635 │
636 │ const listHeader = (() => {
637 │ const children: ReactElement[] = [];
638 │
639 │ if (error) {
640 │ children.push(
641 │ <View
642 │ key="review-error"
643 │ className={cn(
644 │ "bg-card px-4 py-3",
645 │ Platform.OS === "android" ? "m-2 rounded-[20px]" : "border-b border-border",
646 │ )}
647 │ >
648 │ <Text className="text-sm font-t3-bold text-foreground">Review unavailable</Text>
649 │ <Text className="text-xs leading-normal text-foreground-muted">{error}</Text>
650 │ </View>,
651 │ );
652 │ }
653 │
654 │ if (parsedDiffNotice) {
655 │ children.push(<ReviewNotice key="review-notice" notice={parsedDiffNotice} />);
656 │ }
657 │
658 │ if (children.length === 0) {
659 │ return null;
660 │ }
661 │
662 │ return <>{children}</>;
663 │ })();
664 │ const headerSubtitle = [
665 │ headerDiffSummary.additions,
666 │ headerDiffSummary.deletions,
667 │ pendingReviewCommentCount > 0
668 │ ? `${pendingReviewCommentCount} comment${pendingReviewCommentCount === 1 ? "" : "s"}`
669 │ : null,
670 │ ]
671 │ .filter(Boolean)
672 │ .join(" · ");
673 │ const headerTitleText = selectedSection?.title ?? "Review changes";
674 │
675 │ return (
676 │ <>
677 │ <ReviewHeader
678 │ environmentId={environmentId}
679 │ threadId={threadId}
680 │ title={headerTitleText}
681 │ subtitle={headerSubtitle}
682 │ androidSubtitle={androidHeaderSubtitle}
683 │ iconColor={headerIcon}
684 │ selectedThreadCwd={selectedThreadCwd}
685 │ sectionMenu={sectionMenu}
686 │ selectedSection={selectedSection}
687 │ showSectionToolbar={showSectionToolbar}
688 │ showChangedFilesToggle={showChangedFilesToggle}
689 │ onRefresh={handlePullToRefresh}
690 │ onSelectSection={selectSection}
691 │ onReturnToThread={handleReturnToThread}
692 │ />
693 │
694 │ <ScreenContent>
695 │ <View className="flex-1 bg-sheet android:bg-sheet-solid">
696 │ {showConnectionNotice ? (
697 │ <View className="flex-1" style={{ paddingTop: topContentInset }}>
698 │ <EnvironmentConnectionNotice
699 │ environmentLabel={environment.presentation?.entry.target.label ?? "Environment"}
700 │ connection={
701 │ environment.presentation?.connection ?? {
702 │ phase: "available",
703 │ error: null,
704 │ traceId: null,
705 │ }
706 │ }
707 │ resourceName="review"
708 │ onRetry={handleRetryEnvironment}
709 │ />
710 │ </View>
711 │ ) : selectedSection && parsedDiff.kind === "files" && NativeReviewDiffView ? (
712 │ <View
713 │ className="flex-1"
714 │ style={{
715 │ backgroundColor: nativeBridge.theme.background,
716 │ }}
717 │ >
718 │ <View
719 │ className="min-w-0 flex-1"
720 │ style={listHeader ? { paddingTop: topContentInset } : undefined}
721 │ >
722 │ {listHeader}
723 │ <View className="min-w-0 flex-1" collapsable={false}>
724 │ <NativeReviewDiffView
· ──────────┬─────────
· ╰── This component is created during render
725 │ collapsable={false}
╰────
help: Components created during render will reset their state each time they are created. Declare components outside of render
note: React Compiler skipped optimizing this component or hook. Additional guidance: https://react.dev/reference/eslint-plugin-react-hooks/lints/static-components
⚠ eslint(no-unused-vars): Identifier 'Duration' is imported but never used.
╭─[packages/provider-pi/src/server/adapter.test.ts:24:13]
23 │ import * as Deferred from "effect/Deferred";
24 │ import * as Duration from "effect/Duration";
· ────┬───
· ╰── 'Duration' is imported here
25 │ import * as Effect from "effect/Effect";
╰────
help: Consider removing this import.
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/showcase/ShowcaseCaptureCoordinator.tsx:124:7]
119 │
120 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
121 │ if (orientationSettled) return;
122 │ const orientation = getNativeShowcaseOrientation();
123 │ if (orientation === null) {
124 │ setOrientationSettled(true);
· ──────────┬──────────
· ╰── Avoid calling setState() directly within an effect
125 │ 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:299:7]
287 │
288 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
289 │ if (
290 │ scene === null ||
291 │ requestedScene === null ||
292 │ scene !== requestedScene ||
293 │ !hasFixture ||
294 │ !orientationSettled ||
295 │ !themeApplied ||
296 │ (scene === "agent-activity" && !agentActivityStaged) ||
297 │ !isShowcaseNativeContentReady({ scene, themeId, renderSignal })
298 │ ) {
299 │ setReadyScene(null);
· ──────┬──────
· ╰── Avoid calling setState() directly within an effect
300 │ 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:129:9]
128 │ let cancelled = false;
129 │ let lastOutcome: string | null = null;
· ─────┬─────
· ╰── 'lastOutcome' is declared here
130 │ void retryShowcaseOperation(async () => applyNativeShowcaseOrientation(orientation), {
╰────
help: Consider removing this declaration.
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/review/useNativeReviewDiffHighlighting.ts:60:5]
50 │
51 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
52 │ traceDiffHighlight("viewport-reset", {
53 │ reset: diffHighlightTraceId(resetKey),
54 │ enabled,
55 │ rows: rows.length,
56 │ });
57 │ const previousRange = scheduler.reset();
58 │ highlightedRowIdsRef.current = new Set();
59 │ pendingPatchesRef.current = [];
60 │ setViewport({ scope: viewportScope, range: null });
· ─────┬─────
· ╰── Avoid calling setState() directly within an effect
61 │ setTokensPatchJson(createEmptyTokenPatch(resetKey));
╰────
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-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
⚠ eslint(no-unused-vars): Variable 'layer' is declared but never used. Unused variables should start with a '_'.
╭─[packages/provider-pi/src/server/adapter.ts:3337:7]
3336 │
3337 │ const layer: Layer.Layer<ProviderAdapter.ProviderAdapterV2, never, PiAdapterV2DriverEnv> =
· ──┬──
· ╰── 'layer' is declared here
3338 │ Layer.effect(
╰────
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.
╭─[packages/client-runtime/src/connection/supervisor.test.ts:135:3]
134 │ let lastState = yield* SubscriptionRef.get(state);
135 │ ╭─▶ for (let iteration = 0; iteration < 100; iteration += 1) {
136 │ │ lastState = yield* SubscriptionRef.get(state);
137 │ │ if (predicate(lastState)) {
138 │ │ return lastState;
139 │ │ }
140 │ │ yield* Effect.yieldNow;
141 │ ╰─▶ }
142 │ return yield* Effect.die(
╰────
⚠ eslint(no-unused-vars): Variable 'layerTest' is declared but never used. Unused variables should start with a '_'.
╭─[apps/server/src/usage/UsageService.ts:162:7]
161 │
162 │ const layerTest = Layer.succeed(
· ────┬────
· ╰── 'layerTest' is declared here
163 │ UsageService,
╰────
help: Consider removing this declaration.
⚠ eslint(no-unused-vars): Identifier 'serverEnvironment' is imported but never used.
╭─[apps/web/src/components/LegacySidebar.tsx:211:40]
210 │ import { useClientSettings, updateClientSettings } from "~/hooks/useSettings";
211 │ import { primaryServerKeybindingsAtom, serverEnvironment } from "../state/server";
· ────────┬────────
· ╰── 'serverEnvironment' is imported here
212 │ import { appAtomRegistry } from "../rpc/atomRegistry";
╰────
help: Consider removing this import.
⚠ 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:2613:7]
2612 │ let orphanedAt: number | undefined;
2613 │ ╭─▶ for (let interruptAt = 1; interruptAt <= 31; interruptAt += 1) {
2614 │ │ const tasks: Array<() => void> = [];
2615 │ │ const dispatcher: Scheduler.SchedulerDispatcher = {
2616 │ │ scheduleTask: (task) => tasks.push(task),
2617 │ │ flush: () => {
2618 │ │ let task: (() => void) | undefined;
2619 │ │ while ((task = tasks.shift()) !== undefined) task();
2620 │ │ },
2621 │ │ };
2622 │ │
2623 │ │ let requestFiber: Fiber.Fiber<unknown, unknown> | undefined;
2624 │ │ let requestChecks = 0;
2625 │ │ const scheduler: Scheduler.Scheduler = {
2626 │ │ executionMode: "async",
2627 │ │ makeDispatcher: () => dispatcher,
2628 │ │ shouldYield: (fiber) => {
2629 │ │ if (fiber !== requestFiber) return false;
2630 │ │ requestChecks += 1;
2631 │ │ if (requestChecks !== interruptAt) return false;
2632 │ │ fiber.interruptUnsafe();
2633 │ │ return true;
2634 │ │ },
2635 │ │ };
2636 │ │
2637 │ │ const input: UsageSummaryInput = {
2638 │ │ ...WINDOW,
2639 │ │ sinceDay: UsageDay.make("2026-09-01"),
2640 │ │ untilDay: UsageDay.make(`2026-08-${String(interruptAt).padStart(2, "0")}`),
2641 │ │ };
2642 │ │ const first = yield* service
2643 │ │ .readSummary(input)
2644 │ │ .pipe(
2645 │ │ Effect.exit,
2646 │ │ Effect.provideService(Scheduler.Scheduler, scheduler),
2647 │ │ Effect.forkChild,
2648 │ │ );
2649 │ │ requestFiber = first;
2650 │ │ yield* Effect.yieldNow;
2651 │ │ dispatcher.flush();
2652 │ │
2653 │ │ const second = yield* service.readSummary(input).pipe(
2654 │ │ Effect.match({
2655 │ │ onFailure: (error) => error.reason,
2656 │ │ onSuccess: () => "success" as const,
2657 │ │ }),
2658 │ │ Effect.provideService(Scheduler.Scheduler, scheduler),
2659 │ │ Effect.forkChild,
2660 │ │ );
2661 │ │ yield* Effect.yieldNow;
2662 │ │ dispatcher.flush();
2663 │ │ const secondExit = second.pollUnsafe();
2664 │ │ if (secondExit === undefined) {
2665 │ │ second.interruptUnsafe();
2666 │ │ orphanedAt = interruptAt;
2667 │ │ break;
2668 │ │ }
2669 │ │ if (Exit.isFailure(secondExit)) {
2670 │ │ assert.fail("the matching request fiber was interrupted");
2671 │ │ }
2672 │ │ assert.strictEqual(secondExit.value, "invalidWindow");
2673 │ ╰─▶ }
2674 │
╰────
⚠ typescript(no-extraneous-class): Unexpected class with only a constructor.
╭─[apps/desktop/src/preview/Manager.test.ts:150:18]
149 │ BrowserWindow: browserWindowConstructor,
150 │ ╭─▶ ClipboardItem: class {
151 │ │ constructor(data: Record<string, unknown>) {
152 │ │ clipboardItemConstructor(data);
153 │ │ }
154 │ ╰─▶ },
155 │ clipboard: {
╰────
help: Try replacing this class with a standalone function or deleting it entirely
⚠ unicorn(no-new-array): Do not use `new Array(singleArgument)`.
╭─[apps/web/src/terminal/ghostty/core.ts:186:19]
185 │ const count = Math.min(CHUNK_SIZE, graphemeLength - start);
186 │ const codes = new Array<number>(count);
· ────────────────────────
187 │ for (let index = 0; index < count; index += 1) {
╰────
help: It's not clear whether the argument is meant to be the length of the array or the only element. If the argument is the array's length, consider using `Array.from({ length: n })`. If the argument is the only element, use `[element]`.
⚠ 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
⚠ 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 missing/extra effect dependencies
╭─[apps/mobile/src/features/threads/ThreadComposer.tsx:767:7]
766 │ if (settingsSheetPresentation.isActive) {
767 │ settingsRoutePresentation.present(settingsRouteSession);
· ────────────┬────────────
· ╰── Missing dependency `settingsRoutePresentation`
768 │ }
769 │ }, [settingsRoutePresentation.present, settingsRouteSession, settingsSheetPresentation.isActive]);
· ────────────────┬────────────────
· ╰── Overly precise dependency `settingsRoutePresentation.present`, use `settingsRoutePresentation` instead
770 │
╰────
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:388:5]
384 │ typeof props.style.borderRadius === "number" ? props.style.borderRadius : 0;
385 │ const animatedBorderRadius = useSharedValue(targetBorderRadius);
· ──────────┬─────────
· ╰── `animatedBorderRadius` originates here
386 │ const shouldAnimate = props.animateLayout !== false && Platform.OS !== "android";
387 │ useLayoutEffect(() => {
388 │ animatedBorderRadius.value = shouldAnimate
· ──────────┬─────────
· ╰── `animatedBorderRadius` cannot be modified
389 │ ? 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
⚠ unicorn(no-useless-fallback-in-spread): Empty fallbacks in spreads are unnecessary
╭─[scripts/lib/cli-external-packages.test.ts:104:11]
103 │ const declared = {
104 │ ...(manifest.dependencies ?? {}),
· ────────────────────────────────
105 │ ...(manifest.optionalDependencies ?? {}),
╰────
help: Spreading falsy values in object literals won't add any unexpected properties, so it's unnecessary to add an empty object as fallback.
⚠ unicorn(no-useless-fallback-in-spread): Empty fallbacks in spreads are unnecessary
╭─[scripts/lib/cli-external-packages.test.ts:105:11]
104 │ ...(manifest.dependencies ?? {}),
105 │ ...(manifest.optionalDependencies ?? {}),
· ────────────────────────────────────────
106 │ ...(manifest.peerDependencies ?? {}),
╰────
help: Spreading falsy values in object literals won't add any unexpected properties, so it's unnecessary to add an empty object as fallback.
⚠ unicorn(no-useless-fallback-in-spread): Empty fallbacks in spreads are unnecessary
╭─[scripts/lib/cli-external-packages.test.ts:106:11]
105 │ ...(manifest.optionalDependencies ?? {}),
106 │ ...(manifest.peerDependencies ?? {}),
· ────────────────────────────────────
107 │ };
╰────
help: Spreading falsy values in object literals won't add any unexpected properties, so it's unnecessary to add an empty object as fallback.
⚠ react(immutability): This value cannot be modified
╭─[apps/mobile/src/features/threads/PendingUserInputCard.tsx:90:5]
79 │
80 │ const cardCoverage = props.cardCoverage;
· ──────┬─────
· ╰── `cardCoverage` originates here
81 │ const barHeightRef = useRef(0);
82 │ const cardHeightRef = useRef(0);
83 │ const cardHeight = useSharedValue(0);
84 │ const notifyCoverage = useCallback(() => {
85 │ if (!cardCoverage) {
86 │ return;
87 │ }
88 │ // The card resizes with the keyboard every frame, so easing each sample would leave the
89 │ // feed inset and floating controls trailing it.
90 │ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
· ──────┬─────
· ╰── `cardCoverage` cannot be modified
91 │ }, [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:102:7]
82 │ const cardHeightRef = useRef(0);
83 │ const cardHeight = useSharedValue(0);
· ─────┬────
· ╰── `cardHeight` originates here
84 │ const notifyCoverage = useCallback(() => {
85 │ if (!cardCoverage) {
86 │ return;
87 │ }
88 │ // The card resizes with the keyboard every frame, so easing each sample would leave the
89 │ // feed inset and floating controls trailing it.
90 │ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
91 │ }, [cardCoverage]);
92 │ const handleBarLayout = useCallback(
93 │ (event: LayoutChangeEvent) => {
94 │ barHeightRef.current = event.nativeEvent.layout.height;
95 │ notifyCoverage();
96 │ },
97 │ [notifyCoverage],
98 │ );
99 │ const handleCardLayout = useCallback(
100 │ (event: LayoutChangeEvent) => {
101 │ cardHeightRef.current = event.nativeEvent.layout.height;
102 │ cardHeight.value = event.nativeEvent.layout.height;
· ─────┬────
· ╰── `cardHeight` cannot be modified
103 │ 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:90:5]
83 │ const cardHeight = useSharedValue(0);
84 │ const notifyCoverage = useCallback(() => {
· ───┬───
· ╰── This function may (indirectly) reassign or modify `cardCoverage` after render
85 │ if (!cardCoverage) {
86 │ return;
87 │ }
88 │ // The card resizes with the keyboard every frame, so easing each sample would leave the
89 │ // feed inset and floating controls trailing it.
90 │ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
· ──────┬─────
· ╰── This modifies `cardCoverage`
91 │ }, [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:90:5]
89 │ // feed inset and floating controls trailing it.
90 │ cardCoverage.value = Math.max(0, cardHeightRef.current - barHeightRef.current);
· ──────┬─────
· ╰── This modifies `cardCoverage`
91 │ }, [cardCoverage]);
92 │ const handleBarLayout = useCallback(
93 │ (event: LayoutChangeEvent) => {
· ───────────────┬───────────────
· ╰── This function may (indirectly) reassign or modify `cardCoverage` after render
94 │ 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:102:7]
99 │ const handleCardLayout = useCallback(
100 │ (event: LayoutChangeEvent) => {
· ───────────────┬───────────────
· ╰── This function may (indirectly) reassign or modify `cardHeight` after render
101 │ cardHeightRef.current = event.nativeEvent.layout.height;
102 │ cardHeight.value = event.nativeEvent.layout.height;
· ─────┬────
· ╰── This modifies `cardHeight`
103 │ 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
⚠ unicorn(prefer-array-find): Prefer `find` over filtering and accessing the first result.
╭─[apps/server/src/orchestration-v2/Adapters/AcpAdapterV2.test.ts:2143:15]
2142 │ assert.equal(
2143 │ items.filter((item) => item.type === "assistant_message").at(-1)?.text,
· ──────
2144 │ "authoritative answer",
╰────
help: Use `find(predicate)` instead of `filter(predicate)[0]` or similar patterns.
⚠ 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/orchestration-v2/Adapters/AcpAdapterV2.test.ts:4245:7]
4244 │ const closeFiber = yield* Scope.close(sessionScope, Exit.void).pipe(Effect.forkScoped);
4245 │ ╭─▶ while (!responseLifecycle.includes("failed")) {
4246 │ │ yield* Effect.yieldNow;
4247 │ ╰─▶ }
4248 │ const responseExit = yield* Fiber.join(responseFiber);
╰────
⚠ 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/orchestration-v2/Adapters/AcpAdapterV2.test.ts:6355:9]
6354 │ .pipe(Effect.forkScoped);
6355 │ ╭─▶ while (promptGates.length < 1) {
6356 │ │ yield* Effect.yieldNow;
6357 │ ╰─▶ }
6358 │ assert.isDefined(sessionUpdateHandler, "session update handler must be wired");
╰────
⚠ 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/orchestration-v2/Adapters/AcpAdapterV2.test.ts:6490:9]
6489 │ .pipe(Effect.forkScoped);
6490 │ ╭─▶ while (promptGates.length < 2) {
6491 │ │ yield* Effect.yieldNow;
6492 │ ╰─▶ }
6493 │ if (sessionUpdateHandler === undefined || capturedMutation.current === null) {
╰────
⚠ eslint(no-unused-vars): Identifier 'serverEnvironment' is imported but never used.
╭─[apps/web/src/components/Sidebar.tsx:135:40]
134 │ import { environmentPresentations } from "../state/presentation";
135 │ import { primaryServerKeybindingsAtom, serverEnvironment } from "../state/server";
· ────────┬────────
· ╰── 'serverEnvironment' is imported here
136 │ import { environmentProjects } from "../state/projects";
╰────
help: Consider removing this import.
⚠ react(purity): Cannot call impure function during render
╭─[apps/web/src/components/Sidebar.tsx:1177:11]
1176 │ foldSearchQuery(threadSearchQuery),
1177 │ Date.now(),
· ─────┬────
· ╰── Cannot call impure function
1178 │ // 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(capitalized-calls): Capitalized function called without JSX
╭─[apps/mobile/src/features/threads/ThreadQueueControl.tsx:617:27]
616 │ const onEnd = useEffectEvent((y: number, success: boolean) => props.onEnd(y, success));
617 │ const gesture = Gesture.Pan()
· ─┬─
· ╰── `Pan` may be a component
618 │ .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
⚠ eslint(no-unused-vars): Function 'layerClaudeAgentSdkReplayQueryRunner' is declared but never used.
╭─[apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.testkit.ts:956:10]
955 │
956 │ function layerClaudeAgentSdkReplayQueryRunner(
· ──────────────────┬─────────────────
· ╰── 'layerClaudeAgentSdkReplayQueryRunner' is declared here
957 │ transcript: ClaudeAgentSdkReplayTranscript,
╰────
help: Consider removing this declaration.
⚠ 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:434:5]
424 │ : null;
425 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
426 │ if (
427 │ props.active === false ||
428 │ activeDraftKey !== null ||
429 │ editingPendingTask !== null ||
430 │ selectedProject === null
431 │ ) {
432 │ return;
433 │ }
434 │ setActiveDraftKey(
· ────────┬────────
· ╰── Avoid calling setState() directly within an effect
435 │ 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:1217:36]
1216 │ <ActiveNewTaskContext value={activeNewTask}>
1217 │ <NewTaskFlowContext.Provider value={value}>{props.children}</NewTaskFlowContext.Provider>
· ─────────────
1218 │ </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.
⚠ 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/state/shell-sync.test.ts:319:7]
318 │ });
319 │ ╭─▶ for (let index = 0; index < 100; index += 1) {
320 │ │ if ((yield* SubscriptionRef.get(shellState)).status === "synchronizing") break;
321 │ │ yield* Effect.yieldNow;
322 │ ╰─▶ }
323 │
╰────
⚠ 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/state/shell-sync.test.ts:328:7]
327 │ yield* Deferred.succeed(releaseSave, undefined);
328 │ ╭─▶ for (let index = 0; index < 100; index += 1) {
329 │ │ if ((yield* Ref.get(saved)).length > 0) break;
330 │ │ yield* Effect.yieldNow;
331 │ ╰─▶ }
332 │
╰────
⚠ 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/state/shell-sync.test.ts:457:7]
456 │ yield* Deferred.succeed(releaseUnenrichedSave, undefined);
457 │ ╭─▶ for (let index = 0; index < 100; index += 1) {
458 │ │ const values = yield* Ref.get(saved);
459 │ │ if (values.length > 0 && values.at(-1)?.projects[0]?.repositoryIdentity != null) {
460 │ │ break;
461 │ │ }
462 │ │ yield* Effect.yieldNow;
463 │ ╰─▶ }
464 │
╰────
⚠ eslint(no-unused-vars): Variable 'layerCodexAppServerClientFactoryCommand' is declared but never used. Unused variables should start with a '_'.
╭─[apps/server/src/orchestration-v2/Adapters/CodexAdapterV2.ts:1409:7]
1408 │
1409 │ const layerCodexAppServerClientFactoryCommand = (
· ───────────────────┬───────────────────
· ╰── 'layerCodexAppServerClientFactoryCommand' is declared here
1410 │ options: CodexClient.CodexAppServerClientOptions & {
╰────
help: Consider removing this declaration.
⚠ eslint(no-unused-vars): Variable 'layer' is declared but never used. Unused variables should start with a '_'.
╭─[apps/server/src/orchestration-v2/Adapters/CodexAdapterV2.ts:1645:7]
1644 │
1645 │ const layer: Layer.Layer<
· ──┬──
· ╰── 'layer' is declared here
1646 │ ProviderAdapter.ProviderAdapterV2,
╰────
help: Consider removing this declaration.
⚠ eslint(no-unused-vars): Variable 'layer' is declared but never used. Unused variables should start with a '_'.
╭─[apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.ts:10633:7]
10632 │
10633 │ const layer: Layer.Layer<
· ──┬──
· ╰── 'layer' is declared here
10634 │ ProviderAdapter.ProviderAdapterV2,
╰────
help: Consider removing this declaration.
⚠ 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): Type 'AuthSessionState' is imported but never used.
╭─[packages/client-runtime/src/state/usageAccess.ts:3:8]
2 │ AuthDiagnosticsReadScope,
3 │ type AuthSessionState,
· ────────┬───────
· ╰── 'AuthSessionState' is imported here
4 │ sessionGrantsScope,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Parameter 'stopSessions' is declared but never used. Unused parameters should start with a '_'.
╭─[apps/server/src/provider/AntigravityAuth.ts:303:20]
302 │ },
303 │ (effect, flow, stopSessions) =>
· ──────┬─────
· ╰── 'stopSessions' is declared here
304 │ effect.pipe(
╰────
help: Consider removing this parameter.
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/threads/ThreadArrangementSheet.tsx:287:5]
285 │ .join("|");
286 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
287 │ stop();
· ──┬─
· ╰── Avoid calling setState() directly within an effect
288 │ }, [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:287:5]
286 │ useEffect(() => {
287 │ stop();
· ──┬─
· ╰── Missing dependency `stop`
288 │ }, [orderVersion]);
· ──────┬─────
· ╰── Unnecessary dependency `orderVersion`
289 │ 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:105:9]
104 │ const latest = useRef(props);
105 │ latest.current = props;
· ───┬──────┬───
· │ ╰── Cannot update ref value during render
· ╰── This value is a ref
106 │ 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:113:18]
112 │ .runOnJS(true)
113 │ .onStart(() => latest.current.onStart())
· ───────────────┬──────────────
· ╰── Passing a ref to a function may read its value during render
114 │ .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:114:19]
113 │ .onStart(() => latest.current.onStart())
114 │ .onUpdate((event) => latest.current.onMove(event.translationY))
· ──────────────────────────┬─────────────────────────
· ╰── Passing a ref to a function may read its value during render
115 │ .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:115:16]
114 │ .onUpdate((event) => latest.current.onMove(event.translationY))
115 │ .onEnd((event) => latest.current.onMove(event.translationY))
· ──────────────────────────┬─────────────────────────
· ╰── Passing a ref to a function may read its value during render
116 │ .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:116:21]
115 │ .onEnd((event) => latest.current.onMove(event.translationY))
116 │ .onFinalize((_, success) => latest.current.onEnd(!success)),
· ───────────────────────┬──────────────────────
· ╰── Passing a ref to a function may read its value during render
117 │ [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:275:9]
274 │ const latest = useRef({ rows, planners, moveThread });
275 │ latest.current = { rows, planners, moveThread };
· ───┬──────┬───
· │ ╰── Cannot update ref value during render
· ╰── This value is a ref
276 │
╰────
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:273:22]
272 │ const [preview, setPreview] = useState<Drag | null>(null);
273 │ const translateY = useRef(new Animated.Value(0)).current;
· ──────────────────┬──────────────────
· ╰── Cannot access ref value during render
274 │ 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:350:16]
349 │ update(0);
350 │ let last = performance.now();
· ────────┬────────
· ╰── Cannot call impure function
351 │ 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:108:15]
107 │ () =>
108 │ Gesture.Pan()
· ─┬─
· ╰── `Pan` may be a component
109 │ .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:117:5]
112 │ .runOnJS(true)
113 │ .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
114 │ .onUpdate((event) => latest.current.onMove(event.translationY))
115 │ .onEnd((event) => latest.current.onMove(event.translationY))
116 │ .onFinalize((_, success) => latest.current.onEnd(!success)),
117 │ [props.disabled],
· ────────┬───────
· ╰── This dependency list is missing values used by the callback
118 │ );
╰────
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(exhaustive-effect-dependencies): Found missing/extra effect dependencies
╭─[apps/mobile/src/features/threads/NewTaskContextPickerScreens.tsx:250:7]
249 │ mountedRef.current = false;
250 │ flow.setBranchQuery("");
· ──┬─
· ╰── Missing dependency `flow`
251 │ };
252 │ }, [flow.setBranchQuery]);
· ─────────┬─────────
· ╰── Overly precise dependency `flow.setBranchQuery`, use `flow` instead
253 │
╰────
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
⚠ unicorn(no-useless-fallback-in-spread): Empty fallbacks in spreads are unnecessary
╭─[apps/mobile/src/state/use-model-option-memory.ts:21:7]
20 │ [instanceId]: {
21 │ ...(state[instanceId] ?? {}),
· ────────────────────────────
22 │ [model]: options,
╰────
help: Spreading falsy values in object literals won't add any unexpected properties, so it's unnecessary to add an empty object as fallback.
⚠ 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.
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/threads/ThreadDetailScreen.tsx:565:5]
563 │ }, []);
564 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
565 │ setKeyboardStateSuspect(false);
· ───────────┬───────────
· ╰── Avoid calling setState() directly within an effect
566 │ }, [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:977:7]
974 │ const [lastKnownKeyboardHeight, setLastKnownKeyboardHeight] = useState(0);
975 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
976 │ if (liveKeyboardHeight > 0 && liveKeyboardHeight !== lastKnownKeyboardHeight) {
977 │ setLastKnownKeyboardHeight(liveKeyboardHeight);
· ─────────────┬────────────
· ╰── Avoid calling setState() directly within an effect
978 │ }
╰────
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:1220:5]
1218 │
1219 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
1220 │ setAnchorMessageId(null);
· ─────────┬────────
· ╰── Avoid calling setState() directly within an effect
1221 │ 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:566:7]
565 │ setKeyboardStateSuspect(false);
566 │ }, [isKeyboardVisible, liveKeyboardHeight]);
· ────────┬──────── ─────────┬────────
· │ ╰── Unnecessary dependency `liveKeyboardHeight`
· ╰── Unnecessary dependency `isKeyboardVisible`
567 │ 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:1168:7]
1167 │ userInputInsetProgress.set(1);
1168 │ }, [activeUserInputRequestId, userInputCardProgress, userInputInsetProgress]);
· ────────────┬───────────
· ╰── Unnecessary dependency `activeUserInputRequestId`
1169 │ 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:1226:15]
1225 │ freeze.set(false);
1226 │ }, [freeze, selectedThreadKey]);
· ────────┬────────
· ╰── Unnecessary dependency `selectedThreadKey`
1227 │
╰────
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:1063:5]
1058 │ const composerPopoverHeight = useSharedValue(0);
1059 │ const floatingControlCoverage = useSharedValue(
· ───────────┬───────────
· ╰── `floatingControlCoverage` originates here
1060 │ showFloatingStatus ? FLOATING_WORKING_CONTROL_COVERAGE : 0,
1061 │ );
1062 │ useEffect(() => {
1063 │ floatingControlCoverage.value = withTiming(
· ───────────┬───────────
· ╰── `floatingControlCoverage` cannot be modified
1064 │ 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:1079:7]
1069 │ const userInputCoverageApplies = Platform.OS === "ios" && activeUserInputRequestId !== null;
1070 │ const combinedContentInsetEndAdjustment = useSharedValue(
· ────────────────┬────────────────
· ╰── `combinedContentInsetEndAdjustment` originates here
1071 │ Math.max(0, estimatedOverlayHeight - nativeInsetOvercount),
1072 │ );
1073 │ useAnimatedReaction(
1074 │ () =>
1075 │ contentInsetEndAdjustment.value +
1076 │ floatingControlCoverage.value +
1077 │ (userInputCoverageApplies ? userInputInsetProgress.value * userInputCardCoverage.value : 0),
1078 │ (value) => {
1079 │ combinedContentInsetEndAdjustment.value = value;
· ────────────────┬────────────────
· ╰── `combinedContentInsetEndAdjustment` cannot be modified
1080 │ },
╰────
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
⚠ eslint(no-empty-pattern): Empty object binding pattern
╭─[apps/server/src/provider/Drivers/CodexDriver.test.ts:460:14]
459 │
460 │ for (const {} of ["direct", "wrapper"] as const) {
· ──
461 │ }
╰────
help: Passing `null` or `undefined` will result in runtime error because `null` and `undefined` cannot be destructured.
⚠ react(jsx-no-constructed-context-values): The Context `value` prop should not be constructed.
╭─[apps/mobile/src/features/threads/ThreadSettingsSheet.tsx:287:50]
286 │ return (
287 │ <ExistingThreadSettingsRouteContext.Provider value={value}>
· ─────────────
288 │ {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:651:44]
650 │ return (
651 │ <ThreadSettingsSessionContext.Provider value={value}>
· ─────────────
652 │ {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:1460:55]
1459 │ return (
1460 │ <ThreadSettingsPickerPresentationContext.Provider value={presentation}>
· ────────────────────
1461 │ <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.
⚠ eslint(no-unused-vars): Identifier 'androidWorkspaceNavigationState' is imported but never used.
╭─[apps/mobile/src/lib/android-workspace-navigation.test.ts:9:3]
8 │ import {
9 │ androidWorkspaceNavigationState,
· ───────────────┬───────────────
· ╰── 'androidWorkspaceNavigationState' is imported here
10 │ linkedThreadStep,
╰────
help: Consider removing this import.
⚠ eslint(no-unsafe-optional-chaining): Unsafe usage of optional chaining
╭─[apps/server/src/orchestration-v2/ThreadLaunchService.test.ts:1849:8]
1848 │ assert.equal(
1849 │ (stored.context?.records[0] as { attachmentId: string }).attachmentId,
· ──────────────────────────
1850 │ stored.attachments[0]?.id,
╰────
help: If this short-circuits with 'undefined' the evaluation will throw TypeError
⚠ 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."));
╰────
⚠ 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);
╰────
⚠ 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:339:38]
338 │ visit(parser.parse(markdown));
339 │ for (const lineStart of lineStarts.reverse()) {
· ───────
340 │ // Lazy headers can lack container markers; copy the delimiter's markers to the header and blank line.
╰────
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/components/CommandPalette.tsx:2057:20]
2056 │ title: inferProjectTitleFromPath(destinationPath),
2057 │ createdAt: new Date().toISOString(),
· ─────┬────
· ╰── Cannot call impure function
2058 │ remoteUrl: addProjectCloneFlow.remoteUrl,
╰────
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
⚠ unicorn(prefer-array-find): Prefer `find` over filtering and accessing the first result.
╭─[packages/client-runtime/src/operations/projects.ts:238:51]
237 │ ): string {
238 │ const folderName = workspaceRoot.split(/[\\/]/).filter(Boolean).at(-1) ?? "";
· ──────
239 │ return target.owner ? `${target.owner}/${folderName}` : folderName;
╰────
help: Use `find(predicate)` instead of `filter(predicate)[0]` or similar patterns.
⚠ 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
⚠ react(immutability): This value cannot be modified
╭─[apps/mobile/src/features/threads/thread-work-log.tsx:174:5]
170 │ const expandedAngle = props.collapsedDirection === "right" ? 90 : 180;
171 │ const rotation = useSharedValue(props.expanded ? expandedAngle : 0);
· ────┬───
· ╰── `rotation` originates here
172 │
173 │ useLayoutEffect(() => {
174 │ rotation.value = withTiming(props.expanded ? expandedAngle : 0, {
· ────┬───
· ╰── `rotation` cannot be modified
175 │ duration: THREAD_DISCLOSURE_TRANSITION_MS,
╰────
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(no-array-index-key): Usage of Array index in keys is not allowed
╭─[apps/mobile/src/features/threads/thread-work-log.tsx:1239:23]
1238 │ <Text
1239 │ key={`${index}:${line}`}
· ────────────────────────
1240 │ selectable
╰────
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/ThreadMarkdownImage.tsx:52:5]
50 │
51 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
52 │ setDecodedSize(null);
· ───────┬──────
· ╰── Avoid calling setState() directly within an effect
53 │ }, [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:56:5]
54 │
55 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
56 │ setFailedUri(null);
· ──────┬─────
· ╰── Avoid calling setState() directly within an effect
57 │ }, [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:53:7]
52 │ setDecodedSize(null);
53 │ }, [props.sourceKey]);
· ───────┬───────
· ╰── Unnecessary dependency `props.sourceKey`
54 │
╰────
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:57:7]
56 │ setFailedUri(null);
57 │ }, [props.uri]);
· ────┬────
· ╰── Unnecessary dependency `props.uri`
58 │
╰────
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
⚠ 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.
╭─[packages/contracts/src/auth.test.ts:17:12]
16 │ it("decodes legacy review credentials without offering them in new grants", () => {
17 │ expect(Schema.decodeUnknownSync(AuthEnvironmentScopes)(["review:write"])).toEqual([
· ────────────────────────
18 │ "review:write",
╰────
⚠ 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.
╭─[packages/contracts/src/auth.test.ts:20:18]
19 │ ]);
20 │ expect(() => Schema.decodeUnknownSync(AuthGrantScopes)(["review:write"])).toThrow();
· ────────────────────────
21 │ expect(AuthStandardClientScopes).not.toContain("review:write");
╰────
⚠ 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.
╭─[packages/contracts/src/auth.test.ts:57:21]
56 │ it("ignores unknown response permissions without falling back to broader scopes", () => {
57 │ const session = Schema.decodeUnknownSync(AuthSessionState)({
· ────────────────────────
58 │ authenticated: true,
╰────
⚠ react(purity): Cannot call impure function during render
╭─[apps/web/src/components/sidebar/SidebarThreadRow.tsx:373:47]
372 │ const { open, onOpenChange, onSnooze, timestampFormat } = props;
373 │ const presets = open ? resolveSnoozePresets(new Date(), timestampFormat) : [];
· ─────┬────
· ╰── Cannot call impure function
374 │ 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:860:39]
859 │ if (snoozeGateClock.thread !== thread) {
860 │ setSnoozeGateClock({ thread, now: new Date().toISOString() });
· ─────┬────
· ╰── Cannot call impure function
861 │ }
╰────
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
⚠ eslint(no-unused-vars): Variable 'layer' is declared but never used. Unused variables should start with a '_'.
╭─[apps/server/src/serverRuntimeStartup.ts:720:7]
719 │
720 │ const layer = layerWithOptions();
· ──┬──
· ╰── 'layer' is declared here
╰────
help: Consider removing this declaration.
⚠ 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(exhaustive-effect-dependencies): Found extra effect dependencies
╭─[apps/mobile/src/components/ComposerEditor.tsx:82:6]
81 │ },
82 │ [draftKey],
· ────┬───
· ╰── Unnecessary dependency `draftKey`
83 │ );
╰────
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
⚠ t3code(no-inline-schema-compile): Hoist Schema.encodeUnknownSync(...) 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.
╭─[packages/contracts/src/composerContext.test.ts:181:18]
180 │ it("sends a message without the records the wire cannot carry", () => {
181 │ const wire = Schema.encodeUnknownSync(Schema.toCodecJson(OrchestrationMessageContext))({
· ────────────────────────
182 │ version: 1,
╰────
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/threads/ThreadRouteScreen.tsx:413:5]
411 │
412 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
413 │ setInspectorSelection((current) => {
· ──────────┬──────────
· ╰── Avoid calling setState() directly within an effect
414 │ 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:541:24]
540 │ const inspectorHeaderInset = Platform.OS === "ios" ? 0 : safeAreaInsets.top;
541 │ ╭─▶ const GitInspector = () => (
542 │ │ <GitOverviewSheet
543 │ │ headerInset={inspectorHeaderInset}
544 │ │ presentation="inspector"
545 │ │ route={{ params: { environmentId: environmentId!, threadId: threadId! } }}
546 │ │ />
547 │ ╰─▶ );
548 │ 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:548:26]
547 │ );
548 │ ╭─▶ const FilesInspector = () =>
549 │ │ selectedThread !== null && selectedThreadCwd !== null ? (
550 │ │ <ThreadFileNavigatorPane
551 │ │ cwd={selectedThreadCwd}
552 │ │ environmentId={selectedThread.environmentId}
553 │ │ headerInset={inspectorHeaderInset}
554 │ │ projectName={selectedThreadProject?.title ?? "Files"}
555 │ │ selectedPath={null}
556 │ │ onSelectFile={handleSelectInspectorFile}
557 │ │ />
558 │ ╰─▶ ) : null;
559 │ 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/components/VideoPreviewModal.tsx:67:5]
65 │ const { id: attachmentId, fileUri } = attachment;
66 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
67 │ setUri(null);
· ───┬──
· ╰── Avoid calling setState() directly within an effect
68 │ setError(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/components/VideoPreviewModal.tsx:90:7]
89 │ };
90 │ }, [attachmentId, fileUri]);
· ──────┬───── ───┬───
· │ ╰── Unnecessary dependency `fileUri`
· ╰── Unnecessary dependency `attachmentId`
91 │ const mimeType = videoMimeType(attachment) ?? attachment.mimeType;
╰────
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/components/VideoPreviewModal.tsx:212:34]
211 │ useEffect(() => {
212 │ if (!isFocused && hasSource) props.onRequestClose();
· ──┬──
· ╰── Missing dependency `props`
213 │ }, [isFocused, hasSource, props.onRequestClose]);
· ──────────┬─────────
· ╰── Overly precise dependency `props.onRequestClose`, use `props` instead
214 │ const { source } = props;
╰────
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/floating-working-control.tsx:108:5]
104 │ );
105 │ const separationProgress = useSharedValue(props.showScrollToEnd ? 1 : 0);
· ─────────┬────────
· ╰── `separationProgress` originates here
106 │
107 │ useEffect(() => {
108 │ separationProgress.value = withTiming(props.showScrollToEnd ? 1 : 0, CONTROL_TIMING);
· ─────────┬────────
· ╰── `separationProgress` cannot be modified
109 │ }, [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:132:5]
122 │ // Animate an in-flow sizer for native glass layout. Measure labels at fixed width to avoid repeated animation retargeting.
123 │ const capsuleWidth = useSharedValue<number | null>(null);
· ──────┬─────
· ╰── `capsuleWidth` originates here
124 │ const measuredWidthRef = useRef<number | null>(null);
125 │ const handleLabelLayout = (event: LayoutChangeEvent) => {
126 │ const width = event.nativeEvent.layout.width;
127 │ if (width === measuredWidthRef.current) {
128 │ return;
129 │ }
130 │ const first = measuredWidthRef.current === null;
131 │ measuredWidthRef.current = width;
132 │ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
· ──────┬─────
· ╰── `capsuleWidth` cannot be modified
133 │ };
╰────
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:139:7]
122 │ // Animate an in-flow sizer for native glass layout. Measure labels at fixed width to avoid repeated animation retargeting.
123 │ const capsuleWidth = useSharedValue<number | null>(null);
· ──────┬─────
· ╰── `capsuleWidth` originates here
124 │ const measuredWidthRef = useRef<number | null>(null);
125 │ const handleLabelLayout = (event: LayoutChangeEvent) => {
126 │ const width = event.nativeEvent.layout.width;
127 │ if (width === measuredWidthRef.current) {
128 │ return;
129 │ }
130 │ const first = measuredWidthRef.current === null;
131 │ measuredWidthRef.current = width;
132 │ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
133 │ };
134 │ const hasStatus = props.status !== null;
135 │ const hasCapsule = hasStatus || hasQueue || hasAgents || hasPreview;
136 │ useEffect(() => {
137 │ if (!hasStatus) {
138 │ measuredWidthRef.current = null;
139 │ capsuleWidth.value = null;
· ──────┬─────
· ╰── `capsuleWidth` cannot be modified
140 │ }
╰────
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:139:7]
135 │ const hasCapsule = hasStatus || hasQueue || hasAgents || hasPreview;
136 │ useEffect(() => {
· ───┬───
· ╰── This function may (indirectly) reassign or modify `capsuleWidth` after render
137 │ if (!hasStatus) {
138 │ measuredWidthRef.current = null;
139 │ capsuleWidth.value = null;
· ──────┬─────
· ╰── This modifies `capsuleWidth`
140 │ }
╰────
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:132:5]
124 │ const measuredWidthRef = useRef<number | null>(null);
125 │ const handleLabelLayout = (event: LayoutChangeEvent) => {
· ───────────────┬───────────────
· ╰── This function may (indirectly) reassign or modify `capsuleWidth` after render
126 │ const width = event.nativeEvent.layout.width;
127 │ if (width === measuredWidthRef.current) {
128 │ return;
129 │ }
130 │ const first = measuredWidthRef.current === null;
131 │ measuredWidthRef.current = width;
132 │ capsuleWidth.value = first ? width : withTiming(width, CONTROL_TIMING);
· ──────┬─────
· ╰── This modifies `capsuleWidth`
133 │ };
╰────
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
⚠ eslint(no-empty-pattern): Empty array binding pattern
╭─[apps/server/src/pullRequest/PullRequestService.test.ts:1436:12]
1435 │
1436 │ for (const [] of [
· ──
1437 │ ["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(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:2223:5]
2221 │ );
2222 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
2223 │ setExpandedVideo(null);
· ────────┬───────
· ╰── Avoid calling setState() directly within an effect
2224 │ 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:2716:9]
2707 │ const stopFollowingEnd = useEffectEvent(() => setEndFollow(false));
2708 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
2709 │ if (!jumpToMessageId || loadedListKey !== listMountKey) return;
2710 │ const index = presentedFeed.findIndex(
2711 │ (entry) => entry.type === "message" && entry.id === jumpToMessageId,
2712 │ );
2713 │ if (index === -1) {
2714 │ const runId = messageFoldRunId(props.feed, jumpToMessageId);
2715 │ if (runId && !expandedTurnIds.has(runId)) {
2716 │ setInteractionState((current) => ({
· ─────────┬─────────
· ╰── Avoid calling setState() directly within an effect
2717 │ ...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:72:10]
71 │ import { resolveAssetUrl } from "@t3tools/client-runtime/state/assets";
72 │ import { isMarkdownFileLinkLabel } from "@t3tools/shared/markdownLinks";
· ───────────┬───────────
· ╰── 'isMarkdownFileLinkLabel' is imported here
73 │ 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:98:3]
97 │ import {
98 │ createContext,
· ──────┬──────
· ╰── 'createContext' is imported here
99 │ memo,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'ActivityIndicator' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:114:3]
113 │ import {
114 │ ActivityIndicator,
· ────────┬────────
· ╰── 'ActivityIndicator' is imported here
115 │ Image,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'ScrollView' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:121:3]
120 │ Pressable,
121 │ ScrollView,
· ─────┬────
· ╰── 'ScrollView' is imported here
122 │ StyleSheet,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'NativeText' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:123:11]
122 │ StyleSheet,
123 │ Text as NativeText,
· ─────┬────
· ╰── 'NativeText' is imported here
124 │ 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:146:10]
145 │ import { copyTextWithHaptic } from "../../lib/copyTextWithHaptic";
146 │ import { tryOpenExternalUrl } from "../../lib/openExternalUrl";
· ─────────┬────────
· ╰── 'tryOpenExternalUrl' is imported here
147 │ 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:149:10]
148 │ import { hasWideMarkdownBlock } from "../../lib/wideMarkdownBlocks";
149 │ import { faviconUrlForOrigin } from "@t3tools/shared/favicon";
· ─────────┬─────────
· ╰── 'faviconUrlForOrigin' is imported here
150 │ import {
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'MediaVideoPlayer' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:163:10]
162 │ import { VideoAttachmentTile } from "../../components/VideoAttachmentTile";
163 │ import { MediaVideoPlayer } from "../../components/MediaVideoPlayer";
· ────────┬───────
· ╰── 'MediaVideoPlayer' is imported here
164 │ 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:167:3]
166 │ attachmentVideoPreviewSource,
167 │ mediaVideoPreviewUri,
· ──────────┬─────────
· ╰── 'mediaVideoPreviewUri' is imported here
168 │ mediaVideoThumbnailKey,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'mediaVideoThumbnailKey' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:168:3]
167 │ mediaVideoPreviewUri,
168 │ mediaVideoThumbnailKey,
· ───────────┬──────────
· ╰── 'mediaVideoThumbnailKey' is imported here
169 │ 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:169:8]
168 │ mediaVideoThumbnailKey,
169 │ type MediaVideoPreviewSource,
· ───────────┬───────────
· ╰── 'MediaVideoPreviewSource' is imported here
170 │ } from "../../lib/videoPreviewSource";
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'resolveMarkdownFontSizes' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:191:3]
190 │ import {
191 │ resolveMarkdownFontSizes,
· ────────────┬───────────
· ╰── 'resolveMarkdownFontSizes' is imported here
192 │ resolveNativeMarkdownTypography,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'resolveMarkdownInlineCodePresentation' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:198:3]
197 │ normalizeNativeMarkdownUrl,
198 │ resolveMarkdownInlineCodePresentation,
· ──────────────────┬──────────────────
· ╰── 'resolveMarkdownInlineCodePresentation' is imported here
199 │ resolveMarkdownLinkPresentation,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Type 'PendingThreadFeedEntry' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:237:8]
236 │ isCompactionCommandFeedEntry,
237 │ type PendingThreadFeedEntry,
· ───────────┬──────────
· ╰── 'PendingThreadFeedEntry' is imported here
238 │ type ThreadFeedRow,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Identifier 'useAssetUrlState' is imported but never used.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:248:3]
247 │ useAssetUrl,
248 │ useAssetUrlState,
· ────────┬───────
· ╰── 'useAssetUrlState' is imported here
249 │ useRefreshAssetUrl,
╰────
help: Consider removing this import.
⚠ eslint(no-unused-vars): Variable 'navigation' is declared but never used. Unused variables should start with a '_'.
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:438:9]
437 │ const forkFromRun = useAtomCommand(threadEnvironment.forkFromRun, "fork from response");
438 │ const navigation = useNavigation();
· ─────┬────
· ╰── 'navigation' is declared here
439 │ const selectDetail = useSelectWorkspaceDetail();
╰────
help: Consider removing this declaration.
⚠ react(exhaustive-effect-dependencies): Found extra effect dependencies
╭─[apps/mobile/src/features/threads/ThreadFeed.tsx:2225:7]
2224 │ setExpandedFile(null);
2225 │ }, [props.environmentId, props.threadId, props.contentPresentation.kind]);
· ─────────┬───────── ───────┬────── ───────────────┬──────────────
· │ │ ╰── Unnecessary dependency `props.contentPresentation.kind`
· │ ╰── Unnecessary dependency `props.threadId`
· ╰── Unnecessary dependency `props.environmentId`
2226 │ 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:2685:7]
2684 │ reportHeaderMaterialVisibility(false);
2685 │ }, [feedThreadKey, reportHeaderMaterialVisibility]);
· ──────┬──────
· ╰── Unnecessary dependency `feedThreadKey`
2686 │
╰────
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:2701:7]
2700 │ }
2701 │ }, [listMountKey, props.contentInsetEndAdjustment, listRef]);
· ──────┬─────
· ╰── Unnecessary dependency `listMountKey`
2702 │ // 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:2895:7]
2894 │ }
2895 │ }, [expandedTurnIds, expandedWorkGroups, expandedWorkRows, settleDisclosureAfterLayout]);
· ───────┬─────── ─────────┬──────── ────────┬───────
· │ │ ╰── Unnecessary dependency `expandedWorkRows`
· │ ╰── Unnecessary dependency `expandedWorkGroups`
· ╰── Unnecessary dependency `expandedTurnIds`
2896 │
╰────
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:628:9]
627 │ };
628 │ }, [props.environmentId, attachment.id, httpBaseUrl]),
· ─────────┬───────── ──────┬────── ─────┬─────
· │ │ ╰── Unnecessary dependency `httpBaseUrl`
· │ ╰── Unnecessary dependency `attachment.id`
· ╰── Unnecessary dependency `props.environmentId`
629 │ );
╰────
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/use-composer-command-menu.ts:221:7]
217 │ };
218 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
219 │ const inserted = ownerKey ? readComposerDraftSelection(ownerKey, draftMessage) : null;
220 │ if (inserted) {
221 │ setSelection((current) =>
· ──────┬─────
· ╰── Avoid calling setState() directly within an effect
222 │ 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:301:7]
289 │ }, [hasWorkspaceSnapshot]);
290 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
291 │ if (!environmentId || !projectCwd || !selectedProviderInstanceId) return;
292 │ const key = `${environmentId}:${selectedProviderInstanceId}:${projectCwd}`;
293 │ const now = Date.now();
294 │ const lastRequest = workspaceRefreshKeyRef.current;
295 │ if (
296 │ lastRequest?.key === key &&
297 │ now - lastRequest.requestedAt < PROVIDER_WORKSPACE_SNAPSHOT_TTL_MS
298 │ )
299 │ return;
300 │ if (hasCurrentProviderWorkspaceSnapshot(selectedProviderStatus, projectCwd, now)) {
301 │ setWorkspaceRefreshRetry(null);
· ────────────┬───────────
· ╰── Avoid calling setState() directly within an effect
302 │ 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:331:5]
330 │ }, [
331 │ draftMessage,
· ──────┬─────
· ╰── Unnecessary dependency `draftMessage`
332 │ 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
⚠ react(set-state-in-effect): Calling setState synchronously within an effect can trigger cascading renders
╭─[apps/mobile/src/features/threads/NewTaskDraftRouteScreen.tsx:40:5]
37 │ const [initialized, setInitialized] = useState(false);
38 │ useLayoutEffect(() => {
· ───────┬───────
· ╰── This is the containing effect
39 │ reset();
40 │ setInitialized(true);
· ───────┬──────
· ╰── Avoid calling setState() directly within an effect
41 │ }, [reset]);
╰────
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/NewTaskDraftRouteScreen.tsx:93:5]
88 │
89 │ useEffect(() => {
· ────┬────
· ╰── This is the containing effect
90 │ if (!needsPreparation || !initialProjectRef.branch || waitingForProject) return;
91 │ const branchName = initialProjectRef.branch;
92 │ let active = true;
93 │ setPendingCheckouts((count) => count + 1);
· ─────────┬─────────
· ╰── Avoid calling setState() directly within an effect
94 │ // Serialize replacements: ignoring a stale result cannot undo its Git mutation.
╰────
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(purity): Cannot call impure function during render
╭─[apps/mobile/src/features/threads/thread-list-v2-items.tsx:712:76]
711 │ const [snoozeGateClock, setSnoozeGateClock] = useState(() => ({ thread, now: Date.now() }));
712 │ if (snoozeGateClock.thread !== thread) setSnoozeGateClock({ thread, now: Date.now() });
· ─────┬────
· ╰── Cannot call impure function
713 │ 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:733:71]
732 │ const snoozePresets = useMemo(
733 │ () => (swipeActions.secondary === "snooze" ? resolveSnoozePresets(new Date()) : ([] as const)),
· ─────┬────
· ╰── Cannot call impure function
734 │ [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:734:6]
733 │ () => (swipeActions.secondary === "snooze" ? resolveSnoozePresets(new Date()) : ([] as const)),
734 │ [props.snoozePresetMinute, swipeActions.secondary],
· ────────────┬───────────
· ╰── Unnecessary dependency `props.snoozePresetMinute`
735 │ );
╰────
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:780:7]
779 │ thread.pinnedAt,
780 │ variant,
· ───┬───
· ╰── Unnecessary dependency `variant`
781 │ ],
╰────
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:1229:21]
1228 │ {providerDrivers.slice(0, -1).map((driver, index) => (
1229 │ <View key={`${driver}:${index}`} className="-mr-1 opacity-30">
· ──────────────────────────
1230 │ <ProviderIcon provider={driver} size={12} />
╰────
help: Use a unique data-dependent key to avoid unnecessary rerenders
Found 0 errors and 383 warnings in 5589 files (5.8s, 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=__cdc1a910-3d38-401a-8b1b-103b2321eb93.__actions_cache]
Post job cleanup.
Cache hit occurred on the primary key pnpm-full-v1-Linux-X64-1af206af93d733fa7c4e35fa5de35961f61c0d15b9f10a3bf0b6b76380c4d2df, not saving cache.
##[end-action id=__cdc1a910-3d38-401a-8b1b-103b2321eb93.__actions_cache;outcome=success;conclusion=success;duration_ms=160]
##[start-action display=Cache Vite+;id=__cdc1a910-3d38-401a-8b1b-103b2321eb93.vp_cache]
Post job cleanup.
Cache hit occurred on the primary key vite-plus-home-v2-Linux-X64-1.4.0-node26.12.1-pnpm@12.14.0, not saving cache.
##[end-action id=__cdc1a910-3d38-401a-8b1b-103b2321eb93.vp_cache;outcome=success;conclusion=success;duration_ms=69]
Post job cleanup.
[command]/usr/local/bin/git version
git version 2.55.0
Temporarily overriding HOME='/home/runner/_work/_temp/7611a110-3880-4aff-b7eb-0cec05687e29' 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: 146
BLACKSMITH_RUNNER_ACQUIRE_JOB_MS: 552
BLACKSMITH_CHECK_RUN_ID: 121182404788
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