Skip to content

feat(messages): add --send-at flag to messages send (cueapi #623 parity)#44

Closed
mikemolinet wants to merge 1 commit into
mainfrom
feat/messages-send-at-port
Closed

feat(messages): add --send-at flag to messages send (cueapi #623 parity)#44
mikemolinet wants to merge 1 commit into
mainfrom
feat/messages-send-at-port

Conversation

@mikemolinet
Copy link
Copy Markdown
Collaborator

Summary

Adds optional --send-at flag to cueapi messages send for per-message scheduling. Server-side support landed in cueapi #623.

Behavior

  • Omitted: message delivered immediately (existing behavior).
  • ISO 8601 timestamp: server stores send_at on the message row and the worker picks it up when due.
  • Past timestamp: treated as 'send now' (idempotent — no error).

Implementation

send_at flows as a body field on /v1/messages. Verified via app/schemas/message.py MessageCreate schema (cueapi #623). Same-day SDK ports already shipped: cueapi-python PR #34, cueapi-mcp manifest #31 references.

Tests

Extended test_messages_send_help_lists_required_flags to pin --send-at in help output. 184/184 tests pass.

Source

Drift audit handoff/cueapi-package-drift-2026-05-06. Backlog row "Parity port: PR #623 → cueapi-cli" (p1). cueapi-main confirmed cueapi-cli lane mine via [CC-CUEAPI-CLI-LANE-OPTION-B-YOURS].

🤖 Generated with Claude Code

Adds optional per-message scheduling via --send-at. Body field on
/v1/messages (server's MessageCreate schema accepts send_at as a body
field per cueapi #623). Past timestamps are treated as 'send now'
(idempotent — no error).

Test extended: test_messages_send_help_lists_required_flags pins
--send-at in help output.

184/184 tests pass.

Source: drift audit handoff/cueapi-package-drift-2026-05-06; Backlog
row "Parity port: PR #623 (POST /v1/messages send_at) → cueapi-cli" (p1).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@govindkavaturi-art govindkavaturi-art enabled auto-merge (squash) May 7, 2026 01:58
@mikemolinet
Copy link
Copy Markdown
Collaborator Author

Superseded by cueapi-cli #49 (merged 2026-05-10, a9151b1) which added --send-at to messages send along with --notify and --mode parity flags. Main's send-at implementation is fuller (better help text + composed with BCC-light + Surface 6 mode hint) than this PR's. Closing — no re-port needed.

For future me: this PR sat for 4 days on a stale base while my afternoon attention was elsewhere; the messages send_at port was completed via the broader #49 PR which composed multiple related primitives. Lesson: when sibling PRs touch the same command surface, the larger composed PR wins and the older split-out PR becomes stale fast.

auto-merge was automatically disabled May 11, 2026 23:50

Pull request was closed

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