Skip to content

fix(workspaces): preserve resumed sessions and support text answers - #1391

Merged
luokerenx4 merged 1 commit into
devfrom
codex/web-session-recovery-input
Sep 6, 2026
Merged

fix(workspaces): preserve resumed sessions and support text answers#1391
luokerenx4 merged 1 commit into
devfrom
codex/web-session-recovery-input

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Opening an existing ACP conversation could silently allocate and bind a new native session when the runtime did not support history loading. Reject that restore with terminal guidance and preserve the original identity; fresh sessions still work.

Codex questions without options previously offered only an empty answer. Carry free text through the request card, domain hook, HTTP route and transport. Keep permission requests option-only, mask secret questions, retain drafts on errors, and clear an entire question sequence when its turn stops. Request IDs reset the card between questions.

The shared card retains its composer status placement and uses a labeled textarea plus explicit submission, without taking focus. Demo Codex conversations accept “Ask me for a project name” to exercise this interaction. Owner documentation records the contract and interaction choice.

Validation:

  • Full pnpm test: 737 files, 6649 passed, 3 skipped.
  • Targeted host, route, view and demo tests: 115 passed; additional request-card tests: 2 passed.
  • Root npx tsc --noEmit and UI npx tsc -b passed.
  • Real browser demo: Codex question → text answer → continued conversation; second question → Stop → card cleared. Visually inspected the rendered card.
  • Restarted the independent UI Setup project on this checkout and verified the real /chat page at localhost:5173.
  • PR feat(workspaces): generalize WebPi into a multi-runtime Web conversation surface #1390 post-merge CLI Installer Smoke passed before this follow-up.

This fixes the two reviewed defects after #1390. Credentialed tool turns and Web/TUI transcript continuity across all seven runtimes remain the acceptance gap already recorded in plans/web-conversation-surface.md; this PR does not claim that broader acceptance. No Electron-specific files or managed runtime launch contract changed.

@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openalice-demo Building Building Preview Sep 6, 2026 1:16pm UTC

Request Review

@luokerenx4
luokerenx4 merged commit f2468d6 into dev Sep 6, 2026
2 of 3 checks passed
@luokerenx4
luokerenx4 deleted the codex/web-session-recovery-input branch September 6, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant