Skip to content

fix(skill): make --reply-thread required when in Slack thread mode - #21

Merged
araa47 merged 2 commits into
araa47:mainfrom
csabapigler:fix/skill-reply-thread-required
May 11, 2026
Merged

fix(skill): make --reply-thread required when in Slack thread mode#21
araa47 merged 2 commits into
araa47:mainfrom
csabapigler:fix/skill-reply-thread-required

Conversation

@csabapigler

Copy link
Copy Markdown
Contributor

Problem

When OpenClaw is used in Slack thread mode, omitting --reply-thread causes the orca completion notification to land in the DM/channel root instead of the invoking thread. The flag was documented as optional (No), which leads to it being silently skipped.

Changes to skills/orca/SKILL.md

  • --reply-thread row: No**Yes, if in a thread** — with a note that it should be set to the topic_id from the inbound message metadata
  • New callout box explaining the thread mode rule and the distinction between --session-id (wakes the OpenClaw session) vs --reply-thread (routes the completion notification)
  • Updated openclaw message send example in the completion handler to include [--thread-id <thread>]

Why this matters

OpenClaw exposes the invoking thread's topic_id in inbound message metadata. Agents need to forward this as --reply-thread so that orca's completion wake-up message (and the subsequent openclaw message send summary) land back in the correct thread rather than at the top level of the DM.

When OpenClaw is used in Slack thread mode, omitting --reply-thread
causes the orca completion notification to land in the DM/channel root
instead of the invoking thread.

- Mark --reply-thread as required when topic_id is present in inbound metadata
- Add thread mode rule callout explaining topic_id → reply-thread mapping
- Clarify distinction between --session-id (wakes OpenClaw session) and
  --reply-thread (routes the completion notification)
- Update openclaw message send example to include --thread-id
@araa47
araa47 merged commit f9efbb4 into araa47:main May 11, 2026
2 of 3 checks passed
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