fix(queue-store): clear phase hints on auto-dismiss

[fathiraz]

Fixes review feedback on #38 (discussion_r3332138647).

Problem

scheduleDismiss removed entries from processes but not phaseHintsMap. Manual queueStore.dismiss() already cleared hints; auto-dismiss did not. Reusing the same processId could surface stale Undo/Retry from a prior run.

Fix

• Add removeProcess() to clear dismiss timer (when applicable), phaseHintsMap, and the process entry.
• Call it from queueStore.dismiss() and from the auto-dismiss timer callback.
• Regression test: after undo window auto-dismiss, hints are gone and a new in-flight run for the same id has no stale undo phase.

Verification

pnpm test
pnpm typecheck
pnpm build:chrome && pnpm build:firefox && pnpm build:edge

All passed (378 tests).

  

---

Summary by cubic

Fixes a leak where auto-dismiss left phase hints behind, causing stale Undo/Retry when reusing a processId. Centralizes process removal to clear timers, hints, and entries for consistent behavior.

• Bug Fixes
  • Add removeProcess() to clear dismiss timer, phaseHintsMap, and the process entry.
  • Use it in queueStore.dismiss() and the auto-dismiss path in scheduleDismiss().
  • Add regression test to ensure hints are cleared after the undo window and new runs start clean.

^{Written for commit 8eed2ac. Summary will update on new commits.}

[github-actions]

Cursor auto review

No actionable issues found on changed lines.

No actionable issues found. The removeProcess helper correctly centralizes timer, phaseHintsMap, and process cleanup for both manual and auto-dismiss paths, and the regression test covers the stale-hint scenario described in the PR.

Generated automatically when this PR was submitted using Cursor CLI with --model auto.

[cubic-dev-ai]

No issues found across 2 files

_{Re-trigger cubic}