Skip to content

refactor(tui): align prompt submit dispatch#276

Open
nicko-ai wants to merge 1 commit into
devfrom
codex/prompt-submit-upstream-align
Open

refactor(tui): align prompt submit dispatch#276
nicko-ai wants to merge 1 commit into
devfrom
codex/prompt-submit-upstream-align

Conversation

@nicko-ai

Copy link
Copy Markdown
Collaborator

Issue for this PR

N/A - refactor PR; the PR standards workflow skips issue linking for refactor: titles.

Type of change

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

What does this PR do?

Restores the prompt submit path to upstream-style fire-and-forget dispatch after the existing Run-mode auth and reconnect gates pass. The submitted prompt now clears immediately, editor context is marked sent immediately, and async prompt failures report telemetry/toast while Agency auth failures still reopen /auth.

Removes the completed prompt-submit cleanup item from FORK_CHANGELOG.md.

How did you verify your code works?

  • npx --yes bun@1.3.13 install --frozen-lockfile
  • npx --yes bun@1.3.13 test --timeout 30000 test/cli/tui/prompt.test.tsx
  • npx --yes bun@1.3.13 test --timeout 30000 test/cli/tui/session-error.test.ts test/cli/tui/dialog-auth-modal.test.tsx test/cli/tui/prompt-framework-mode.test.tsx
  • npx --yes bun@1.3.13 run typecheck from packages/opencode
  • pre-push hook: bun turbo typecheck with Bun 1.3.13

Screenshots / recordings

N/A - TUI submit-control refactor covered by focused tests.

Checklist

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

- restore upstream-style fire-and-forget prompt dispatch after Run-mode gates

- keep Agency auth failure recovery dialog without restoring submitted drafts

- remove the completed prompt-submit cleanup queue item
@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

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