The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	Jul 1, 2026 8:40pm

PR Summary

Low Risk
Documentation-only updates to Claude/agent rule files; no runtime, auth, or data-path changes.

Overview
Extends agent rule docs in .claude/rules/ so react-doctor / cleanup passes don’t repeat known pitfalls—no application code changes.

sim-hooks.md adds guidance after the prevX render-phase reset pattern: the initial useRef value controls first-mount behavior (use a sentinel when mount-time work must run; always update the ref unconditionally; place the block before early returns). It also tightens when useState → useRef is safe—only when the value isn’t read in render/JSX and isn’t a hook dependency, with a cross-reference to rerender-state-only-in-handlers.

sim-components.md lists no-render-in-render as a react-doctor false positive: inline helpers like {renderRow()} don’t remount by position, so extracting them is usually churn and can hurt focus/scroll unless the refactor is mechanical.

^{Reviewed by Cursor Bugbot for commit 5da3dab. Configure here.}

Greptile Summary

Documentation-only update to two agent rule files under .claude/rules/, recording a pair of React pitfalls uncovered during a react-doctor cleanup pass so future coding agents avoid repeating them.

• sim-hooks.md: The prevX idiom now explicitly documents the ref-seed/sentinel choice — seeding the current prop value suppresses the mount action (correct when the initial state is already set), while useRef(null) forces it to fire; also clarifies that useState→useRef is only safe when the value is never read in render and never appears in a hook dep array.
• sim-components.md: Adds no-render-in-render to the known react-doctor false-positive list, explaining that a helper called inline is reconciled by position and does not remount, so extracting it is usually churn unless the helper is genuinely a component defined during render.

Confidence Score: 5/5

Documentation-only additions to agent rule files; no executable code is changed and no runtime behavior is affected.

Both additions are technically accurate, consistent with the existing code examples in the files, and address real patterns (silent mount-action drop, incorrect useState→useRef conversion, no-render-in-render false positive) without contradicting any other rules in the codebase.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["prevX ref idiom needed"] --> B{"Does effect do real work\non mount?"}
    B -- "No (e.g. modal mounts closed)" --> C["useRef(currentPropValue)\nGuard = false on mount\nMount action skipped ✓"]
    B -- "Yes (panel already active,\nprop already matches)" --> D["useRef(null) — sentinel\nGuard = true on mount\nMount action fires ✓"]
    C --> E["Update ref unconditionally\ninside the block"]
    D --> E
    E --> F{"Value ever read in\nrender or dep array?"}
    F -- "Yes" --> G["Keep as useState\n(ref mutation won't re-render)"]
    F -- "No (handler / effect body only)" --> H["Safe to convert to useRef"]

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["prevX ref idiom needed"] --> B{"Does effect do real work\non mount?"}
    B -- "No (e.g. modal mounts closed)" --> C["useRef(currentPropValue)\nGuard = false on mount\nMount action skipped ✓"]
    B -- "Yes (panel already active,\nprop already matches)" --> D["useRef(null) — sentinel\nGuard = true on mount\nMount action fires ✓"]
    C --> E["Update ref unconditionally\ninside the block"]
    D --> E
    E --> F{"Value ever read in\nrender or dep array?"}
    F -- "Yes" --> G["Keep as useState\n(ref mutation won't re-render)"]
    F -- "No (handler / effect body only)" --> H["Safe to convert to useRef"]

_{Reviews (1): Last reviewed commit: "chore(rules): document render-phase ref-..." | Re-trigger Greptile}