refactor(shared): remove unused runtime helpers and exports - #10166
Conversation
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a contained shared-code cleanup that narrows unused exports and keeps all internally used implementations and values unchanged. No new capability, product-default change, deployment change, or sensitive-area modification is introduced. You can add or adjust custom eligibility rules. Learn more. |
fd0c6e2 to
b74ac25
Compare
Dismissing prior approval to re-evaluate b74ac25
b74ac25 to
ea9d672
Compare
ea9d672 to
d5f8565
Compare
Bugbot is paused — on-demand spend limit reachedBugbot uses usage-based billing for this team and has hit its on-demand spend limit. A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue. |
Dismissing prior approval to re-evaluate d5f8565
Shared exports several implementation helpers that no consumer uses.
Remove the four unused model, composer, and source-control helpers and keep remaining implementation-only functions/constants private. Preserve exported types and Effect schemas under the revised Knip policy. Keep the public Net constructor required by the repository's Effect service convention.
Verification: shared typecheck and existing shared/source-control tests pass on the rebased stack. The integrated Knip gate passes. No existing tests were changed or removed.
Model: gpt-6 astra. Harness: Codex in T3 Code.