[OrcaWin]

	Files	Added	Deleted	Net
Test	882	$\color{#1a7f37}{\Huge{\mathbf{+}}}$​2850	$\color{#cf222e}{\Huge{\mathbf{−}}}$​11981	$\color{#cf222e}{\Huge{\mathbf{−}}}$​9131
Prod	2118	$\color{#1a7f37}{\Huge{\mathbf{+}}}$​7591	$\color{#cf222e}{\Huge{\mathbf{−}}}$​33177	$\color{#cf222e}{\Huge{\mathbf{−}}}$​25586

ELI5

Remove comments that repeat the code and shorten useful explanations. Keep the reasons behind security, ownership, platform and compatibility decisions.

What Changed

Complete the review of 37,524/37,524 source/configuration files, reconciled with main a5e948dc3de. The diff changes 16,196 files and removes 128,867 net lines. Prose docs, binaries and data-only JSON are outside the inventory; generated/vendor ownership boundaries remain intact.

Preserve licenses, tool directives, SAFETY explanations and useful incident references. Verify executable equivalence across comment edits and upstream reconciliation. The tracker, scripts and verification artifacts stay outside the repository.

Supporting CI/test maintenance:

• Retain the two validated Ubuntu 22.04 IME runner-label repairs; reduce text-count and flaky-pattern allowances where findings decreased.
• Increase bounded audit-report capture from 8 to 64 MiB, with a real subprocess regression for reports above the old limit.
• Reuse timing-based E2E distribution for broad changed-spec selections, bounded at 14 shards. Native discovery verifies all 689 selected tests occur exactly once; older dispatched refs retain their native fallback.
• Refresh selected existing E2E expectations, selectors and probes for current upstream behavior, retaining behavioral assertions and existing convergence checks.
• Cancel pending terminal fit continuations during fixture teardown. A delayed-import reproduction confirmed retries from earlier tests contaminated later frame queues; cleanup preserves the original assertions and fixes that failure.

Consolidates #33011 and #33136, both closed. No application behavior changes are introduced by the audit.

Why

Finish the repository-wide audit in one merge, validating combined corrections locally before publication to reduce CI runs.

Visual Proof

N/A — no application behavior or design changes.

Testing

• pnpm check:changed --maxWorkers=2 incremental gates: 36/36, then 27/27 checks passed. Latest broad selection: 4,212 passed files, 19 skipped, zero unhandled errors. Terminal fixture cleanup passed all 21 affected checks and its delayed-import reproduction.
• Final upstream integration: 948 root tests passed across 35 files, three Windows-only skips; 144 mobile tests passed across 13 files. Full root and mobile production/test types pass; changed-code quality reports zero findings across 49 files. All 30 selected repository script guards and the dead design-system class lint pass.
• Subsequent backend integration: 79 root tests and 21 mobile tests pass; full root types and changed quality across 17 files pass.
• Full mobile CI Node/Bun selection: 11,310 passed, six skipped. Final production types and 1,110 test files pass with zero grandfathered type errors.
• Real React Doctor changed-lines gate: 0 errors, with the large report captured successfully.
• Source/emitted AST and executable-token proofs cover the comment-edited JS/TS files, with incremental reconciliation proofs and protected-comment checks. Latest integrations: 38, then 14 source/emitted/token and protected overlaps pass after commit hooks; the earlier native Rust noncomment proof also passes.
• Noncomment equivalence also covers 79 other-language files, eight stylesheets, 30 YAML files / 44 embedded scripts, and the generated Bash fixture. New AST-grep lint and 11 rule-test groups pass; baseline changes only reduce allowances.
• Broad local E2E: 689 cases, with targeted reruns accounting for every initial failure. Corrected selected tests and shell-sensitive cases pass; nine native-focus/foreground-timing cases require CI. Launches use ORCA_BACKGROUND_LAUNCH=1 and private Xvfb displays.

Linux validation supplies Node 24, shell tools and isolated prompt profiles. Windows, macOS and the remaining native-focus checks are CI validation. Merge requires all final-head required checks and the PR checks passed aggregate.

Agent skill upstream boundary

• Not applicable; no upstream skill-installer source is copied or translated.

[blacksmith-sh]

Found 10 test failures on Blacksmith runners:

Failures

Test	View Logs
[electron-headful] › tests/e2e/ pr11346-selected-runtime-add.spec.ts:751:5 › routes every Add Project path to a selecte d non-default headed runtime @headful	View Logs
[electron-headless] › tests/e2e/ diff-note-line-range.spec.ts:269:7 › Diff note line range › dragging from the "+" itsel f selects a range and the button rides the selection	View Logs
[electron-headless] › tests/e2e/ pr11346-selected-runtime-add.spec.ts:759:5 › keeps every selected-runtime Add Project p ath in hidden-window desktop parity	View Logs
[electron-headless] › tests/e2e/ ssh-orcad-explorer-watch-recovery.spec.ts:28:7 › remote folder explorer receives host-c reated files after server restart	View Logs
[electron-headless] › tests/e2e/ structured-chat-tabs-open-close.spec.ts:60:7 › structured chat tabs › open from the hos t and the New tab menu, show in the tab bar, and close	View Logs
Diff note line range/ dragging from the "+" itself selects a range and the button rides the selection	View Logs
structured chat tabs/ open from the host and the New tab menu, show in the tab bar, and close	View Logs
tests/e2e/pr11346-selected-runtime-add.spec.ts/ keeps every selected-runtime Add Project path in hidden-window desktop parity	View Logs
tests/e2e/pr11346-selected-runtime-add.spec.ts/ routes every Add Project path to a selected non-default headed runtime @headful	View Logs
tests/e2e/ssh-orcad-explorer-watch-recovery.spec.ts/ remote folder explorer receives host-created files after server restart	View Logs

^{Need help on this PR? Tag @codesmith-bot with what you need.}