Skip to content

fix(tui): keep retry off cancel queue and sanitize lists - #88

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/fix-retry-cancel-mgmt-catalog-7291
Sep 5, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/fix-retry-cancel-mgmt-catalog-7291

Conversation

@jkyberneees

Copy link
Copy Markdown
Contributor

What

Eighth self-improvement pass after v1.4.7. Three bugs, RED-first, reviewer-validated.

  1. /retry and alt+r re-queued during cancelling. v1.4.4’s Enter guard keys off status=="cancelling", but slash commands run before that guard and alt+r calls retryLast(), which only checked busy. Cancel had already restored the queue into the composer; retry put lastPrompt back on the drain path. retryLast now no-ops the enqueue while cancelling.

  2. Management drawer lists painted raw wire fields. Skill name/source, memory facts, MCP command text, config keys/values, and agent goals/tools/status went to the terminal without collapse. Description already did; the list rows now match.

  3. Model picker and palette painted raw catalog IDs/descriptions. v1.4.5 collapsed the header and applyModelChoice; modelEntries still copied the catalog verbatim, so ^O and ^K leaked CSI. Collapse at the mapper.

Tests

  • TestRetryDuringCancelDoesNotQueue
  • TestMgmtListRowsSanitizeWireFields
  • TestModelCatalogSanitizesPickerAndPalette

make fmt / vet / lint 0 issues. make test race green.

Expert review: APPROVE.

Open in Web Open in Cursor 

/retry and alt+r queued during cancelling, bypassing the enter
guard. Management drawer lists and the model catalog painted
raw wire fields.

Co-authored-by: admin <admin@21no.de>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bodek f6d09af Commit Preview URL

Branch Preview URL
Sep 05 2026, 08:33 PM

@jkyberneees
jkyberneees marked this pull request as ready for review September 5, 2026 20:36
@cursor
cursor Bot merged commit f6d09af into main Sep 5, 2026
8 checks passed
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.

2 participants