List workflow stage threads - #380
Conversation
c2e1498 to
49ad1a5
Compare
49ad1a5 to
42c0366
Compare
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 6 files (changes from recent commits).
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
🟡 Changes recommended
Markdown output renders nested topics as a map value rather than a listing table.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds workflow stage view for listing threads within a workflow stage.
Changes:
- Supports JSON, styled, ID-only, and count output.
- Adds command tests and CLI documentation.
- Updates the command surface snapshot.
[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or rungh pr ready --undo.
Click "Ready for review" or rungh pr readyto reengage.
File summaries
| File | Description |
|---|---|
.surface |
Registers the new command surface. |
docs/cli.md |
Documents stage thread listing. |
internal/cmd/workflow.go |
Implements the command and output formats. |
internal/cmd/workflow_test.go |
Tests retrieval and primary formats. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Adds
hey workflow stage view <workflow-id> <stage-id>to list the threads currently in a workflow stage. Table output shows thread ID, subject, and email count; JSON includes stage metadata and topic IDs forhey thread read. Markdown output renders a thread table.--ids-onlyand--countsupport scripting.Uses the upstream HEY SDK v0.30.0, which includes basecamp/hey-sdk#141. Removes the temporary personal-fork replacement, merges current
main, and moves the command documentation intodocs/cli.md.Validation:
make check(golangci-lint v2.13.2 locally),make build replace-check, and workflow tests with the race detector. A read-only live check of an existing stage returned four threads, with matching JSON and--countoutput.