Skip to content

fix(tui): avoid placeholder session route on continue#29293

Closed
YOMXXX wants to merge 1 commit into
anomalyco:devfrom
YOMXXX:fix/tui-continue-fork-placeholder-route
Closed

fix(tui): avoid placeholder session route on continue#29293
YOMXXX wants to merge 1 commit into
anomalyco:devfrom
YOMXXX:fix/tui-continue-fork-placeholder-route

Conversation

@YOMXXX
Copy link
Copy Markdown

@YOMXXX YOMXXX commented May 26, 2026

Issue for this PR

Closes #29262

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The TUI used sessionID: "dummy" as the initial route whenever --continue was passed. While the real previous session or fork target is resolved asynchronously, the session route can issue startup requests for /session/dummy, which fails server-side validation and shows an Unexpected server error toast.

This removes the placeholder session route. --continue and --continue --fork now start on the home route until the real session id is known. Explicit --session without --fork still starts directly on that real session id.

How did you verify your code works?

  • bun test test/cli/tui/app.test.ts from packages/opencode
  • bun test test/cli/tui/thread.test.ts from packages/opencode
  • bun typecheck from packages/opencode
  • bun turbo typecheck from repo root
  • .husky/pre-push from repo root

Note: after syncing latest dev, the new stats packages required refreshing local dependencies with bun install so @tsconfig/node22 was available. The generated lockfile rewrite was not included in this PR.

Screenshots / recordings

Not applicable.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

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.

TUI shows server error on --continue --fork: Expected a string starting with "ses", got "dummy"

2 participants