Skip to content

[codex] Phase 3F: safe web runtime fixes#16

Merged
sukar0972 merged 4 commits into
mainfrom
codex/phase3f-safe-web-runtime
Jul 4, 2026
Merged

[codex] Phase 3F: safe web runtime fixes#16
sukar0972 merged 4 commits into
mainfrom
codex/phase3f-safe-web-runtime

Conversation

@sukar0972

Copy link
Copy Markdown
Owner

Summary

Phase 3F applies a small set of web-safe upstream runtime fixes before the larger Phase 4 preview/browser-simulation work:

  • improve Cursor CLI missing-command and ACP discovery setup guidance
  • reject unsupported remote pairing protocols before they reach connection setup
  • restore native keyboard activation for pending user-input choices
  • ignore stale shell snapshot reducer events defensively

Intentionally deferred from this phase

  • c1e2408e1 / browserTargetResolver preview-host handling: deferred with browser/preview simulation work because this fork does not have that resolver path.
  • ea52bb1db trace ID clipboard UI: not directly portable because current saved-backend runtime state has no traceId field.
  • c6c64918f ChatMarkdown rerender reduction: overlaps a larger markdown chrome/table-copy refactor and should be handled with the UI phase instead of this small runtime-fix PR.
  • 28107e89c diff component rename cleanup: obsolete in this fork; the old active path is already gone.

Grok Composer 2.5 review

Reviewed with grok -m grok-composer-2.5-fast after implementation and again after removing no-op DPoP commits from the branch.

Final verdict: approve, no merge-blocking issues.

Verification

  • vp test apps/server/src/provider/Layers/CursorProvider.test.ts packages/shared/src/remote.test.ts packages/client-runtime/src/shellSnapshotReducer.test.ts — 41 passed
  • vp run typecheck — passed
    • existing suggestion remains: src/diagnostics/ProcessResourceMonitor.ts(315,5)
  • vp check — passed
    • existing react(no-unstable-nested-components) warnings remain

StiensWout and others added 4 commits July 4, 2026 07:39
Co-authored-by: Codex <codex@openai.com>
(cherry picked from commit 3083d71)
Co-authored-by: Codex <codex@openai.com>
(cherry picked from commit 8919ae7)
Co-authored-by: Codex <codex@openai.com>
(cherry picked from commit 8b993bc)
Co-authored-by: Codex <codex@openai.com>
(cherry picked from commit 4abf8b4)
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Jul 4, 2026
@sukar0972 sukar0972 marked this pull request as ready for review July 4, 2026 20:50
@sukar0972 sukar0972 merged commit 7407da8 into main Jul 4, 2026
10 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 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