From 1a24b763ea566340cc3eb9d11d002063eb90bb70 Mon Sep 17 00:00:00 2001 From: aaron Date: Mon, 10 Aug 2026 19:08:50 -0400 Subject: [PATCH] amico-slack: dedup footer and default send to Aaron Fix double _Authored by Amico_ (CLI now strips existing footer loop-idempotent) and default send to Aaron (--as-bot to opt out). Update skill to document idempotent footer and chat:write.customize requirement. CLI fix at ~/.local/bin/amico-slack already live; this PR tracks the versioned doc. Fixes #330 --- .../extension/skills/amico-slack/SKILL.md | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/packages/extension/skills/amico-slack/SKILL.md b/packages/extension/skills/amico-slack/SKILL.md index 4cab162..6b75c09 100644 --- a/packages/extension/skills/amico-slack/SKILL.md +++ b/packages/extension/skills/amico-slack/SKILL.md @@ -37,13 +37,16 @@ Use this skill when reading channel discussions, sending updates to Harmoniqs Sl - Strikethrough uses tildes `~strikethrough~`. - Code blocks use triple backticks ` ```code``` `. -4. **Subtext Attribution & File Passing (`--as-user`)**: - - When posting on Aaron's behalf, pass `--as-user` to `amico-slack send`. - - It appends a subtle `_Authored by Amico_` context block at the bottom of the message. +4. **Subtext Attribution & File Passing (sender identity)**: + - `amico-slack send` **defaults to Aaron** — it posts with Aaron's profile (`--as-user` behavior) and appends a subtle `_Authored by Amico_` context block. Pass `--as-bot` only when you intentionally want the bot identity without the footer. `--as-user` is still accepted for backwards-compat but is no longer required. + - The footer is **idempotent** — if the message text already ends with `_Authored by Amico_` the CLI strips the duplicate before appending, so do NOT manually add the footer in the message file; let the CLI do it. Manually adding it was the source of the double-footer bug. + - Requires `chat:write.customize` on the Slack app (reinstall after adding the scope) or the `username` override is silently ignored and the message appears as `amicobot`. - For long/multiline text or text containing backticks/quotes, **always pass the message via a temporary file** (`--file /tmp/msg.txt`) to avoid shell backtick/quote expansion bugs. - Example: ```bash - amico-slack send "#calibration" --file /tmp/cal_update.txt --as-user + amico-slack send "#calibration" --file /tmp/cal_update.txt + # --as-bot only when you want the bot: + # amico-slack send "#calibration" --file /tmp/cal_update.txt --as-bot ``` 5. **Threaded Discussions**: @@ -60,10 +63,10 @@ Use this skill when reading channel discussions, sending updates to Harmoniqs Sl | Task | Command | |---|---| -| Send update as Aaron (from file) | `amico-slack send "" --file --as-user` | -| Send update as Aaron (text) | `amico-slack send "" "" --as-user` | -| Send update in thread | `amico-slack send "" --file --thread --as-user` | -| Send update as Bot | `amico-slack send "" ""` | +| Send update as Aaron (from file) | `amico-slack send "" --file ` | +| Send update as Aaron (text) | `amico-slack send "" ""` | +| Send update in thread | `amico-slack send "" --file --thread ` | +| Send update as Bot | `amico-slack send "" "" --as-bot` | | Read channel history | `amico-slack read "" [limit]` | | Read thread replies | `amico-slack read "" --thread ` | | Search / verify team users | `amico-slack whois [name]` |