fix(app): sync upstream and harden development web session state - #8
Merged
Conversation
Preserve native prompt payloads and task metadata, refresh after early interruption, bridge retry state, and share rollback mutations with keyboard commands. Extend native browser fixtures and replacement consistency tests. This is a draft audit checkpoint: unresolved browser and performance failures are recorded in the audit report before merge.
henry701
marked this pull request as ready for review
September 5, 2026 12:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[ai-agent-generated] This PR description was LLM-generated.
Scope
Integrate upstream through
70b4ca8c181e4c1ac6d8993b86249d824487ec65while preserving fork-specific web behavior. Save the session-state, native projection, early Stop, rollback, provider-discovery and CI/test improvements developed during the audit. Use focused adapters and point integrations rather than reverting fork features.Owner-approved merge with follow-ups
The repository owner explicitly requested committing/pushing all current improvements and merging PRs #7 and #8 now, with remaining issues fixed in follow-up PRs. This supersedes the earlier draft hold. It is not a claim that browser tests or hosted CI are all green.
The owner confirmed that CPU throttling caused the slower tab-switch benchmark timings. The throttled measurements are not comparable with the earlier baseline and do not establish a code-induced performance regression.
Verification
These are separate, overlapping runs, not an aggregate all-green browser suite. Prior CI failures included real subprocess/portability failures and installation/cancellation problems; quota exhaustion was not established. Windows clean-install validation remains incomplete.
Recorded follow-ups
See
docs/fork/web-audit.mdfor evidence, reproduction commands and exit criteria:ENOTEMPTY, clean Windows tests and hosted CI parity.e2894562f8ba943d72172d10b727c24d5f650c16(console usage normalization/tier configuration) and later changes remain follow-up work.Failing tests were retained, not skipped or weakened. No deployment, service restart or binary reinstall is included.
Post-merge verification — 2026-09-05
Both PRs are merged: production
327ce4d046b14334f9d221c90841bbf7d1c72308, development1c504d299ca8ded867ee7ebc8a5ca216d83b82d3. Both original pushed improvement heads are verified ancestors of their fork target branches; the resulting dev and production trees are identical.SYNC-01 update: the fork's upstream-sync workflow merged
e2894562f8ba943d72172d10b727c24d5f650c16into both target branches before these PR merges. Thus the audit file's pre-merge SYNC-01 checkpoint is now resolved for that commit. The merged console changes passed five provider-usage tests, two tier-threshold tests, and both affected console package typechecks. Future upstream updates remain ordinary follow-up work.Both pre-push hooks passed 30/30 typecheck tasks. Development independently passed the 800 + 51 app tests and e2e typechecking. The final affected core run including MCP passed 39 tests / 125 assertions. Hosted checks were pending at the merge; no hosted all-green claim is made. Remaining WEB/CI/deployment follow-ups are unchanged.