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

storkit: create 400_bug_whatsapp_and_slack_missing_reset_command_handler

Browse Source
This commit is contained in:
Timmy
2026-03-26 11:09:24 +00:00
parent feb340beba
commit 3777042ad3
2 changed files with 4 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.storkit/work/1_backlog/400_bug_whatsapp_and_slack_missing_reset_command_handler.md
+4
View File
@@ -8,6 +8,10 @@ name: "WhatsApp and Slack missing reset command handler"
The reset command has a fallback handler in chat/commands/mod.rs that returns None with a comment saying it's handled before try_handle_command. This is only true for Matrix. WhatsApp and Slack don't have pre-dispatch handling, so None causes fallthrough to LLM. This caused a real outage when stale session IDs couldn't be cleared via the bot after switching from Docker to bare-metal. The reset command has a fallback handler in chat/commands/mod.rs that returns None with a comment saying it's handled before try_handle_command. This is only true for Matrix. WhatsApp and Slack don't have pre-dispatch handling, so None causes fallthrough to LLM. This caused a real outage when stale session IDs couldn't be cleared via the bot after switching from Docker to bare-metal.
## Implementation Note
Follow the existing code patterns used by commands that already work on WhatsApp and Slack (e.g. "rebuild", "status"). Study how those commands are dispatched and handled, and implement the reset handler in the same way.
## How to Reproduce ## How to Reproduce
1. Configure bot with transport = "whatsapp" or "slack"\n2. Send "reset" to the bot\n3. Check server logs 1. Configure bot with transport = "whatsapp" or "slack"\n2. Send "reset" to the bot\n3. Check server logs
.storkit/work/1_backlog/402_bug_whatsapp_and_slack_missing_rebuild_command_handler.md
-26
View File
@@ -1,26 +0,0 @@
---
name: "WhatsApp and Slack missing rebuild command handler"
---
# Bug 402: WhatsApp and Slack missing rebuild command handler
## Description
The rebuild command has a fallback handler in chat/commands/mod.rs that returns None. Only Matrix has pre-dispatch handling for this command. On WhatsApp and Slack, the command falls through to the LLM path.
## How to Reproduce
1. Configure bot with transport = "whatsapp" or "slack"\n2. Send "rebuild" to the bot\n3. Check server logs
## Actual Result
Command falls through to LLM instead of triggering a server rebuild.
## Expected Result
The bot triggers a server rebuild and replies with confirmation.
## Acceptance Criteria
- [ ] WhatsApp transport handles rebuild command: triggers rebuild and replies with confirmation
- [ ] Slack transport handles rebuild command: triggers rebuild and replies with confirmation