Skip to content

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

Merged
matt2e merged 1 commit intomainfrom
allow-suggested-actions-even-if-note-not-last
May 8, 2026
Merged

fix: show suggested actions in note modal regardless of recency#696
matt2e merged 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
@matt2e matt2e merged commit e401cbc into main May 8, 2026
5 checks passed
@matt2e matt2e deleted the allow-suggested-actions-even-if-note-not-last branch May 8, 2026 03:02
matt2e added a commit that referenced this pull request May 8, 2026
Signed-off-by: Matt Toohey <contact@matttoohey.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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