Skip to content

Customize Slack in-flight response status#1362

Open
cyrusagent wants to merge 2 commits into
mainfrom
cypack-1367
Open

Customize Slack in-flight response status#1362
cyrusagent wants to merge 2 commits into
mainfrom
cypack-1367

Conversation

@cyrusagent

@cyrusagent cyrusagent commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements Cyrus-branded Slack in-flight status for chat sessions and adds Slack stream helpers that keep each stream in a single mode.

  • Sets assistant.threads.setStatus with Cyrus-specific status/loading copy when a Slack turn starts or receives a streamed follow-up.
  • Clears the active Slack status on successful result handling and on runner start/resume errors.
  • Adds Slack chat.startStream / chat.appendStream / chat.stopStream helpers with runtime guardrails for markdown-vs-chunks mode mixing and required channel recipient fields.
  • Removes prototype presentation from production behavior by omitting username/icon overrides by default so Slack uses the installed Cyrus app identity.

Validation

  • pnpm build
  • pnpm typecheck
  • pnpm test:packages:run
  • pnpm lint
  • pnpm --dir packages/slack-event-transport exec vitest run test/SlackMessageService.test.ts
  • pnpm --dir packages/edge-worker exec vitest run test/chat-sessions.test.ts
  • F1 synthetic Slack chat smoke documented in apps/f1/test-drives/2026-07-03-cypack-1367-slack-inflight-status.md

Notes

F1 validates the Slack chat handler path but cannot visually confirm Slack-rendered assistant status because it uses synthetic channels. A live Slack thread remains the final manual presentation check.

Linear: https://linear.app/ceedar/issue/CYPACK-1367/customize-cyrus-slack-in-flight-response-status-and-streaming


Tip: I will respond to comments that @ mention @cyrusagent on this PR/MR. You can also submit a review with all your feedback at once, and I will automatically wake up to address each comment.

@cyrusagent cyrusagent marked this pull request as ready for review July 10, 2026 16:45
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