Skip to content

sync: finish upstream alignment through bc8584bf8e - #273

Merged
aaditagrawal merged 12 commits into
mainfrom
sync/alignment-20260905-final-refresh
Sep 5, 2026
Merged

sync: finish upstream alignment through bc8584bf8e#273
aaditagrawal merged 12 commits into
mainfrom
sync/alignment-20260905-final-refresh

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Incorporates eight upstream commits published during the alignment audit, through bc8584bf8e967ecc1cd215d42bd7a47faf51a862. The final stack now contains all 156 upstream commits missing after #266.

Adopts file grammar readiness before editing, selected diff-file reveal, deduplicated PR project choices, private test helpers, obsolete preview-helper cleanup, and browser keybinding notice spacing. Retains the fork SDK dependency patch and all preceding provider/runtime adaptations.

Validation: 43 focused tests across six affected web/mobile/client-runtime files; full workspace check/typecheck; preserved exact cold/warm highlighting regression. Integrated browser verification opened a cold syntax-highlighted JSON editor with editing enabled and inspected keybinding settings. Final branch is zero commits behind the stated upstream snapshot. No release or merge to main is included in this PR.

@aaditagrawal aaditagrawal changed the title fix(web): load file grammar before enabling edits (#9947) sync: finish upstream alignment through bc8584bf8e Sep 5, 2026
…tings' into sync/alignment-20260905-final-refresh

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 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.5 KiB 15.1 KiB
Codex Thread snapshot wire 7.0 KiB 7.3 KiB
Codex Live turn WebSocket wire 6.4 KiB 7.8 KiB
Codex Live turn WebSocket decoded 56.2 KiB 66.4 KiB
Codex Live turn messages 8 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: 6420917 · 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.

…tings' into sync/alignment-20260905-final-refresh

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 fix/alignment-provider-model-settings to main September 5, 2026 07:48
@aaditagrawal
aaditagrawal merged commit c4f2093 into main Sep 5, 2026
23 checks passed
@aaditagrawal aaditagrawal mentioned this pull request Sep 5, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 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.

3 participants