Sign up
View stargazers13Sign up

refactor(web): prune unused UI and provider code#9959

merged1
t3code/setup-nip-prune-unused-code
t3code/knip-prune-web-model-exports
openedSep 5, 2026byJJulius Marminge
5 files changed
View on GitHubTwo-way sync enabledPR last imported 4d ago

Knip found unused web code left alongside the current provider/model implementation.

Remove the unreferenced ColorSelector, obsolete provider-picker metadata, and an unused interaction-mode helper. Keep four still-used helpers private to their modules. No production caller or test changes.

Verification: 279 existing tests passed across modelSelection, providerInstances, providerModels, composerDraftStore, and session-logic. Web typecheck, targeted lint, and formatting passed. The integrated web production build passed. No reachable UI changed, so screenshots do not apply.

Leave this stack unmerged for maintainer review.

Model: gpt-6 astra. Harness: Codex in T3 Code.

Added 1 commit

  • refactor(web): prune unused UI and provider codeb92d329
Ggithub-actionsBotSep 5, 2026
Synced from GitHub

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire—13.5 KiB—15.1 KiB✅
CodexThread snapshot wire—7.0 KiB—7.3 KiB✅
CodexLive turn WebSocket wire—6.5 KiB—7.8 KiB✅
CodexLive turn WebSocket decoded—57.0 KiB—66.4 KiB✅
CodexLive turn messages—8—21✅
ClaudeTotal thread wire—13.5 KiB—15.1 KiB✅
ClaudeThread snapshot wire—7.0 KiB—7.3 KiB✅
ClaudeLive turn WebSocket wire—6.5 KiB—7.8 KiB✅
ClaudeLive turn WebSocket decoded—57.8 KiB—66.4 KiB✅
ClaudeLive turn messages—8—21✅

Baseline: unavailable · PR result: b92d329 · Source CI: success

Scenario and decoded snapshot size

10 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.

  • Codex decoded thread snapshot: 113.8 KiB
  • Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

MmacroscopeappBotSep 5, 2026
Synced from GitHub

Approvability

Verdict: Approved at b92d329

Macroscope's review found this PR approvable — This is a contained unused-code cleanup: the deleted component and removed metadata/helpers have no repository callers, while retained helpers keep the same implementations and internal call paths. No product defaults or runtime behavior are changed.

You can add or adjust custom eligibility rules. Learn more.