[jrusso1020]

Problem

Two comments in agent_runtime.ts were wrong after #7425 (review nits on that PR):

• The Claude Code rule called CLAUDE_CODE_ENTRYPOINT an "older versions" marker. Current Claude Code sets it on itself at startup, and it is the only Claude marker that stdio MCP servers inherit, so the label invited someone to delete a marker we still need.
• The Agent SDK rule said Claude Code's shell tool sets CLAUDECODE. The Claude rule now keys on CLAUDE_CODE_CHILD_SESSION.

Changes

Comment wording only in packages/cli/src/telemetry/agent_runtime.ts. No code or behavior change.

Verification

• oxfmt --check, comment-ratchet and check-comment-citations pass locally.

— Rames

🤖 Generated with Claude Code

[github-actions]

Edit accuracy: accurate 267 (base branch 267), smooth 90 of those

The gate passes.
Smoothness is reported in the artifact, not gated. A case fails only if it fails 2 of 3 runs.

Quarantined, measured but not gated (8)

• seqplainkeys-none-center-r0-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): fail / pass / pass, passes
• seqplainkeys-none-center-r0-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-center-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-pct-r0-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): fail / pass / pass, passes
• seqplainkeys-none-pct-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-px-r0-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-px-r30-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-px-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run

[heygengenesis]

Verdict: COMMENT (needs human) · 0 blocker · 0 important · 0 nits · head 9a0c86d

Both comments now match the code. claude_code keys on CLAUDE_CODE_CHILD_SESSION || CLAUDE_CODE_ENTRYPOINT (L99–104), so dropping "older versions" keeps someone from deleting a marker the rule still needs. The SDK comment's "caught above" now names the marker that rule actually checks. I'd approve on the merits. I'm holding approval only because the required check Tests on windows-latest is red. The failing test is @hyperframes/engine browserManager.test.ts > BeginFrame capability probe > reports an empty screenshot as unsupported (cli: 4411 passed). A comments-only change can't cause that, and the same workflow also failed on a recent main push, so it looks like a flake. Re-running the failed job should clear it.

— Genesis

[heygengenesis]

Verdict: APPROVE · 0 blocker · 0 important · 0 nits · 9a0c86d

Windows re-run (attempt 2) is green on the same head, so the earlier hold is cleared.

— Genesis