crashlabs/huskies
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 26 Wiki Activity

storkit: create 398_bug_whatsapp_and_slack_missing_async_command_handlers_for_start_rebuild_reset_rmtree_assign

Browse Source
This commit is contained in:
Timmy
2026-03-26 11:00:01 +00:00
parent 6476492caa
commit 0a67c28f8c
1 changed files with 33 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.storkit/work/1_backlog/398_bug_whatsapp_and_slack_missing_async_command_handlers_for_start_rebuild_reset_rmtree_assign.md
+33
View File
@@ -0,0 +1,33 @@
---
name: "WhatsApp and Slack missing async command handlers for start, rebuild, reset, rmtree, assign"
---
# Bug 398: WhatsApp and Slack missing async command handlers for start, rebuild, reset, rmtree, assign
## Description
The WhatsApp and Slack transports are missing async handlers for several bot commands: start, rebuild, reset, rmtree, and assign. These commands have fallback handlers in chat/commands/mod.rs that return None, with comments stating they are "detected and handled before try_handle_command is invoked". This is only true for the Matrix transport. On WhatsApp and Slack, the fallback returns None, causing the message to fall through to the LLM path instead of executing the intended command.
The reset command is the most impactful — it's supposed to clear the Claude Code session_id so the bot starts a fresh conversation, but on WhatsApp it silently forwards "reset" to the LLM instead. This caused a real outage when switching from Docker to bare-metal: stale session IDs persisted in .storkit/whatsapp_history.json could not be cleared via the bot.
## How to Reproduce
1. Configure the bot with transport = "whatsapp" (Twilio) or "slack".
2. Send "reset" to the bot via WhatsApp.
3. Observe the server logs.
## Actual Result
Log shows "No command matched, forwarding to LLM" — the reset command is sent to the LLM as a conversational message instead of clearing the session.
## Expected Result
The bot should clear the sender's session_id from conversation history and reply with a confirmation like "Session cleared."
## Acceptance Criteria
- [ ] WhatsApp transport handles reset command: clears sender session_id and replies with confirmation
- [ ] Slack transport handles reset command: clears channel session_id and replies with confirmation
- [ ] WhatsApp transport handles start, rebuild, rmtree, and assign commands (or returns a meaningful response)
- [ ] Slack transport handles start, rebuild, rmtree, and assign commands (or returns a meaningful response)
- [ ] Fallback handlers in chat/commands/mod.rs no longer silently swallow these commands