Skip to content

refactor(web): remove unused legacy client modules - #271

Merged
aaditagrawal merged 6 commits into
mainfrom
refactor/alignment-client-cleanup
Sep 5, 2026
Merged

refactor(web): remove unused legacy client modules#271
aaditagrawal merged 6 commits into
mainfrom
refactor/alignment-client-cleanup

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Removes four unreferenced legacy web modules (916 lines) and the now-unused ghostty-web dependency/optimizer entry. Keeps the active preference-restoration paths and current terminal implementation.

Validation: reference search, full web test suite (3,939 tests), workspace check/typecheck, and integrated web/Android terminal verification at the final stack tip. A command entered on web was observed live on Android; hiding/reopening the web terminal retained its output.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XL 500-999 effective changed lines (test files excluded in mixed PRs). labels Sep 5, 2026
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

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

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.6 KiB 15.1 KiB
Codex Thread snapshot wire 7.0 KiB 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 7.8 KiB
Codex Live turn WebSocket decoded 57.0 KiB 66.4 KiB
Codex Live turn messages 9 21
Claude Total thread wire 13.5 KiB 15.1 KiB
Claude Thread snapshot wire 7.0 KiB 7.3 KiB
Claude Live turn WebSocket wire 6.5 KiB 7.8 KiB
Claude Live turn WebSocket decoded 57.8 KiB 66.4 KiB
Claude Live turn messages 8 21

Baseline: unavailable · PR result: b6693c9 · 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.9 KiB
  • Claude decoded thread snapshot: 114.6 KiB

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

cursoragent and others added 2 commits September 5, 2026 06:41
…nto refactor/alignment-client-cleanup

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
cursoragent and others added 2 commits September 5, 2026 06:49
…nto refactor/alignment-client-cleanup

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
…nto refactor/alignment-client-cleanup

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>

aaditagrawal commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

@aaditagrawal
aaditagrawal changed the base branch from perf/alignment-cursor-history to main September 5, 2026 07:48
@aaditagrawal
aaditagrawal merged commit 685973e into main Sep 5, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL 500-999 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants