Skip to content

fix(tui): hide stale retry status after model switch#1494

Open
netsky-prod wants to merge 1 commit into
XiaomiMiMo:mainfrom
netsky-prod:fix/1474-clear-retry-on-model-switch
Open

fix(tui): hide stale retry status after model switch#1494
netsky-prod wants to merge 1 commit into
XiaomiMiMo:mainfrom
netsky-prod:fix/1474-clear-retry-on-model-switch

Conversation

@netsky-prod

Copy link
Copy Markdown

Summary

  • Attach the provider/model that scheduled a retry status to SessionStatus.
  • Hide retry banners in the prompt footer when they belong to a previously selected model.
  • Keep legacy retry statuses without model metadata visible, preserving existing behavior for old state/events.
  • Add regression coverage for retry status model metadata and prompt status filtering.

Fixes #1474.

Verification

  • RED before fix: SessionStatus.Info.parse(...) dropped providerID/modelID, and test/cli/tui/prompt-status.test.ts could not import the missing helper.
  • bun test test/session/retry.test.ts test/cli/tui/prompt-status.test.ts --timeout 30000 - 42 pass, 0 fail.
  • bun typecheck
  • bunx oxlint packages/opencode/src/session/status.ts packages/opencode/src/session/processor.ts packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx packages/opencode/src/cli/cmd/tui/component/prompt/status.ts packages/opencode/test/session/retry.test.ts packages/opencode/test/cli/tui/prompt-status.test.ts - 0 errors, existing warnings only.
  • git diff --check
  • Pre-push bun turbo typecheck - 12 successful, 12 total.

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.

[Bug] switch model but status bar still shows old model quota limit

1 participant