fix: correct skill frontmatter and Slack search modifiers
allowed-tools in commit and pr skills used wrong syntax — array
brackets and glob separators now match the parser's expectations.
Slack search `@` prefix on `from:`/`to:` silently returns wrong
results; bare names work reliably.
Assisted-by: Claude Opus 4.6 via Claude Code
diff --git a/ai/skills/commit/SKILL.md b/ai/skills/commit/SKILL.md
index 444efa56..b40fe9bb 100644
--- a/ai/skills/commit/SKILL.md
+++ b/ai/skills/commit/SKILL.md
@@ -2,7 +2,7 @@
name: commit
description: Use when the user asks to "commit changes", "commit my work", "/commit", "create a commit", "jj commit", "describe a change", "describe a revision", "/describe", "rewrite commit message", or "update revision description"
argument-hint: <revision or fileset>
-allowed-tools: Bash(jj diff *), Bash(jj show *), Bash(jj log *)
+allowed-tools: [Bash(jj diff *), Bash(jj show *), Bash(jj log *)]
---
# Commit or Describe Changes
diff --git a/ai/skills/mcporter-slack/SKILL.md b/ai/skills/mcporter-slack/SKILL.md
index 95cd1dc1..089041c4 100644
--- a/ai/skills/mcporter-slack/SKILL.md
+++ b/ai/skills/mcporter-slack/SKILL.md
@@ -52,8 +52,10 @@ For private channels, the Slack MCP app must be invited to the channel.
## Common Mistakes
-- Parentheses in OR queries silently fail. Write `from:@me OR to:@me`,
- not `(from:@me OR to:@me)`.
+- The `@` prefix on `from:` and `to:` modifiers doesn't work reliably.
+ Use bare names: `from:me to:martin.emde`, not `from:@me to:@Martin Emde`.
+- Parentheses in OR queries silently fail. Write `from:me OR to:me`,
+ not `(from:me OR to:me)`.
- Passing timestamps as `key=value` coerces them to numbers and fails
schema validation. Use `--args` JSON instead.
- Sending to user IDs instead of channel IDs. Always resolve the channel
@@ -62,11 +64,11 @@ For private channels, the Slack MCP app must be invited to the channel.
## Search Queries
```bash
-mcporter call slack.slack_search_public query="from:@user OR to:@user after:2025-01-01 before:2025-01-07"
-mcporter call slack.slack_search_public query="from:@user on:2025-01-15"
+mcporter call slack.slack_search_public query="from:user OR to:user after:2025-01-01 before:2025-01-07"
+mcporter call slack.slack_search_public query="from:user on:2025-01-15"
```
-Modifiers: `from:@user`, `to:@user`, `mentions:@user`, `in:#channel`,
+Modifiers: `from:user`, `to:user`, `mentions:user`, `in:#channel`,
`before:` / `after:` / `on:` with `YYYY-MM-DD` or `today`,
`is:thread`, `has:pin`.
diff --git a/ai/skills/pr/SKILL.md b/ai/skills/pr/SKILL.md
index 79b71671..b3a08e74 100644
--- a/ai/skills/pr/SKILL.md
+++ b/ai/skills/pr/SKILL.md
@@ -2,7 +2,7 @@
name: pr
description: Use when the user asks to "create a pull request", "create a PR", "/pr", "open a PR", "submit for review", or "push and create PR"
argument-hint: <revision> [ready|web]
-allowed-tools: Bash(jj diff:*), Bash(jj log:*), Bash(jj show:*), Bash(jj bookmark list:*)
+allowed-tools: [Bash(jj diff *), Bash(jj log *), Bash(jj show *), Bash(jj bookmark list *)]
---
# Create Pull Request