Skip to content

fix: show suggested actions in note modal regardless of recency#696

Open
matt2e wants to merge 1 commit intomainfrom
allow-suggested-actions-even-if-note-not-last
Open

fix: show suggested actions in note modal regardless of recency#696
matt2e wants to merge 1 commit intomainfrom
allow-suggested-actions-even-if-note-not-last

Conversation

@matt2e
Copy link
Copy Markdown
Contributor

@matt2e matt2e commented May 7, 2026

Summary

  • Removes the recency check that prevented suggested actions from appearing in the note modal when a newer timeline item (commit, note, or review) existed
  • Removes the active session check that also blocked suggested actions
  • Suggested actions from a note are now always shown when viewing that note, regardless of whether it's the latest timeline item

Test plan

  • Open a note modal for a note that has suggested actions
  • Verify suggested actions appear even when newer commits/notes/reviews exist on the timeline
  • Verify suggested actions still don't appear for notes without them

🤖 Generated with Claude Code

Remove the "active sessions" guard and the "is latest item" timestamp
check from computeNoteNextSteps so that suggested action buttons appear
in the note modal even when the note isn't the most recent timeline
item or sessions are already running. New sessions will simply be
queued, so hiding the buttons was unnecessarily restrictive.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e matt2e requested review from baxen and wesbillman as code owners May 7, 2026 07:41
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