Skip to content

notification: document pull SSE as the notification.pullStream action#35

Merged
acoshift merged 1 commit into
mainfrom
notification-pull-stream
Jun 20, 2026
Merged

notification: document pull SSE as the notification.pullStream action#35
acoshift merged 1 commit into
mainfrom
notification-pull-stream

Conversation

@acoshift

Copy link
Copy Markdown
Member

Summary

Follows apiserver#171 / api#95, which moved the pull SSE stream from the raw GET /notification/pull/sse endpoint to a first-class arpc action. Updates the non-CLI consumer example in the Pull section to the action shape:

POST https://api.deploys.app/notification.pullStream
Content-Type: application/json

{ "project": "<id>", "name": "<channel>", "ack": <last-handled-id> }

Same POST + JSON request as notification.pull; the response is an event stream. Also notes that a pre-stream failure returns the usual {ok:false,error} JSON, and that the ack resume token rides in the request body now (not a query param / Last-Event-ID). The CLI --follow behavior is unchanged.

hugo --gc --minify builds clean.

Rollout

PR 3 of 3 (final follow-up). Follows apiserver#171 → api#95.

🤖 Generated with Claude Code

apiserver moved the pull SSE stream from a raw GET endpoint to a first-class arpc
action (notification.pullStream). Update the non-CLI consumer example to the
POST+JSON action shape and note the arpc {ok:false,error} pre-stream error
envelope. CLI --follow behavior is unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011d4bVuGLnCbcJD9ZvastPH
@deploys-app deploys-app Bot temporarily deployed to pr-35 June 20, 2026 12:33 Destroyed
@deploys-app

deploys-app Bot commented Jun 20, 2026

Copy link
Copy Markdown

Preview deleted (PR closed).

@acoshift acoshift merged commit ffe1700 into main Jun 20, 2026
1 check passed
@acoshift acoshift deleted the notification-pull-stream branch June 20, 2026 12:40
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