Skip to content

Commit cb2d37f

Browse files
committed
fix: skill follow-ups from review
allowed-tools now pre-grants make and gh, matching the commands the skill body runs. The raw-URL fallback tells agents to fetch reference files from the same ref as SKILL.md instead of pointing at main, so a pinned skill can't mix with newer references. updating.md gains skill activation smoke checks (positive, negative, explicit invocation) before any release touching the skill. https://claude.ai/code/session_01T3RGHgchJZBpvuiesrrE7X
1 parent a718536 commit cb2d37f

2 files changed

Lines changed: 14 additions & 4 deletions

File tree

skills/simple-modern-uv/SKILL.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ compatibility: Requires git and uv (https://docs.astral.sh/uv/); network access
1212
metadata:
1313
author: jlevy (github.com/jlevy)
1414
source: https://github.com/jlevy/simple-modern-uv
15-
allowed-tools: Bash(uvx:*), Bash(uv:*), Bash(git:*), Read, Write, Edit
15+
allowed-tools: Bash(uvx:*), Bash(uv:*), Bash(git:*), Bash(make:*), Bash(gh:*), Read, Write, Edit
1616
---
1717
# simple-modern-uv: Modern Python Project Setup
1818

@@ -37,9 +37,9 @@ For all workflows: when something fails, check [references/faq.md](references/fa
3737
before improvising. For post-setup customization (license change, private/unpublished
3838
package, entry points), see [references/customize.md](references/customize.md).
3939

40-
If this file was fetched from a URL rather than installed as a folder, the reference
41-
files are at
42-
`https://raw.githubusercontent.com/jlevy/simple-modern-uv/main/skills/simple-modern-uv/references/<name>.md`.
40+
If this file was fetched from a URL rather than installed as a folder, fetch the
41+
reference files from the **same ref**: replace `SKILL.md` in the URL you used with
42+
`references/<name>.md` (don’t mix a pinned `SKILL.md` with references from `main`).
4343

4444
## The Interview: What to Ask the User (and What Not to Ask)
4545

updating.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,16 @@ and template code. This is the real end-to-end validation that the template work
202202

203203
If CI fails, fix issues in the template repo and repeat from Step 2.
204204

205+
### Skill Activation Checks
206+
207+
Before releasing changes that touch `skills/simple-modern-uv/`, smoke-test activation in
208+
at least one target agent (CI validates structure and links, not activation):
209+
210+
- A new-project prompt ("start a new Python project called X") activates the skill
211+
- A migration prompt ("convert this Poetry project to uv") activates the skill
212+
- An unrelated Python prompt ("debug this stack trace") does *not* activate it
213+
- Explicit invocation (`/simple-modern-uv` or the agent’s equivalent) loads cleanly
214+
205215
## Step 7: Create a Release on the Template Repo
206216

207217
Once CI passes downstream (and this repo’s own CI is green on the candidate commit),

0 commit comments

Comments
 (0)