Skip to content

fix: distinguish scheduled vs sent in output messaging#127

Merged
felipefreitag merged 2 commits intomainfrom
worktree-fix-scheduled-messaging
Mar 18, 2026
Merged

fix: distinguish scheduled vs sent in output messaging#127
felipefreitag merged 2 commits intomainfrom
worktree-fix-scheduled-messaging

Conversation

@felipefreitag
Copy link
Contributor

Spinner text, interactive output, and help text now correctly say "scheduled" instead of "sent" when --scheduled-at is used.

Copy link
Contributor

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

Choose a reason for hiding this comment

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

1 issue found across 3 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/commands/broadcasts/send.ts">

<violation number="1" location="src/commands/broadcasts/send.ts:37">
P2: Scheduled sends still use "sent" spinner text, so interactive output remains incorrect when `--scheduled-at` is set.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@felipefreitag felipefreitag force-pushed the worktree-fix-scheduled-messaging branch from dea3905 to eff696c Compare March 18, 2026 17:18
Spinner text, interactive output, and help text now correctly say
"scheduled" instead of "sent" when --scheduled-at is used.
@felipefreitag felipefreitag force-pushed the worktree-fix-scheduled-messaging branch from 6e3c64e to 6b8edcc Compare March 18, 2026 17:55
@felipefreitag felipefreitag merged commit 423d3e9 into main Mar 18, 2026
6 checks passed
@felipefreitag felipefreitag deleted the worktree-fix-scheduled-messaging branch March 18, 2026 17:56
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.

2 participants