Skip to content

feat(triage+recall): full lifecycle actions + status filter browse#13

Merged
nokternol merged 1 commit into
mainfrom
feat/triage-recall-mar-08
Mar 8, 2026
Merged

feat(triage+recall): full lifecycle actions + status filter browse#13
nokternol merged 1 commit into
mainfrom
feat/triage-recall-mar-08

Conversation

@nokternol

Copy link
Copy Markdown
Owner

Summary

  • /review: Commit ✓ / Park / Discard / Promote → Plan inline per item; Promote only shown for ideas; per-item busy state prevents double-clicks
  • Browse: Status filter row (all / draft / active / parked / promoted / done / dismissed); also fixes URLSearchParams bug that was serialising undefined as the string "undefined"
  • Chat: Park and Promote → Plan buttons in header alongside Commit; Promote only visible for idea type
  • API: Inflight endpoint now includes type from cache entry when matched

Test plan

  • bash scripts/check.sh passes locally (7/7)
  • CI quality gate green on this PR
  • Capture idea → visit /review → confirm Commit / Park / Discard / Promote buttons present
  • Park from /review → item removed from list, shows in browse with status=parked filter
  • Commit from /review → item removed, shows in browse as done
  • Discard from /review → item removed, shows in browse as dismissed
  • Promote from /review → navigates to new plan chat
  • Chat (idea): Promote → Plan button visible; Park button visible
  • Chat (plan): Promote → Plan button NOT visible; Park button visible
  • Browse: select 'parked' status filter → only parked items shown

🤖 Generated with Claude Code

- /review: Commit, Park, Discard, Promote actions per in-flight item;
  Promote only shown for ideas, Discard available without id match;
  per-item busy state prevents double-clicks
- Browse: status filter row (all/draft/active/parked/promoted/done/dismissed);
  fix URLSearchParams dropping undefined params (was sending status=undefined)
- Chat: Park and Promote → Plan buttons alongside Commit;
  Promote only visible for idea type
- API: inflight response now includes type when cache entry matches
- Tests: review action tests, browse filter tests, chat button tests,
  inflight type assertion; MSW park handler added to defaults
- Stories: review triage variants (idea/plan/no-id/mixed), browse stories

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nokternol nokternol merged commit 7a18b9c into main Mar 8, 2026
3 checks passed
@nokternol nokternol deleted the feat/triage-recall-mar-08 branch March 8, 2026 20:36
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