Skip to content

Emit suggestion loading messages#37

Merged
royisme merged 1 commit intoclaude/review-cli-help-design-0168GRCcxP9NrLKNKaVzE7BVfrom
roy/fix-codex-review-issues-for-cli-help-output-2025-11-18
Nov 18, 2025
Merged

Emit suggestion loading messages#37
royisme merged 1 commit intoclaude/review-cli-help-design-0168GRCcxP9NrLKNKaVzE7BVfrom
roy/fix-codex-review-issues-for-cli-help-output-2025-11-18

Conversation

@royisme
Copy link
Copy Markdown
Owner

@royisme royisme commented Nov 18, 2025

Summary

  • send a suggestionsLoadedMsg from loadSuggestions instead of mutating state directly
  • update the dashboard Update loop to apply suggestion results and errors from messages

Testing

  • go test ./internal/...

Codex Task

@royisme royisme merged commit 29d8596 into claude/review-cli-help-design-0168GRCcxP9NrLKNKaVzE7BV Nov 18, 2025
@royisme royisme deleted the roy/fix-codex-review-issues-for-cli-help-output-2025-11-18 branch November 18, 2025 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant