Skip to content

List workflow stage threads - #380

Open
almirsarajcic wants to merge 4 commits into
basecamp:mainfrom
almirsarajcic:feature/workflow-stage-view
Open

List workflow stage threads#380
almirsarajcic wants to merge 4 commits into
basecamp:mainfrom
almirsarajcic:feature/workflow-stage-view

Conversation

@almirsarajcic

@almirsarajcic almirsarajcic commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

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 for hey thread read. Markdown output renders a thread table. --ids-only and --count support 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 into docs/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 --count output.

@almirsarajcic
almirsarajcic force-pushed the feature/workflow-stage-view branch from 49ad1a5 to 42c0366 Compare September 1, 2026 12:45

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread internal/cmd/workflow.go
Comment thread internal/cmd/workflow_test.go

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread go.mod Outdated
@almirsarajcic
almirsarajcic marked this pull request as ready for review September 10, 2026 08:48
@almirsarajcic
almirsarajcic requested a review from a team as a code owner September 10, 2026 08:48
Copilot AI balanced review requested due to automatic review settings September 10, 2026 08:48
@github-actions github-actions Bot removed the deps label Sep 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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 run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to 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.

Comment thread internal/cmd/workflow.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants