From b1db3b2ed64d5aad4d655f5a00f0dc8aa0e9d227 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Sat, 15 Aug 2026 21:24:35 +0000 Subject: [PATCH] chore: version skills --- .../domain-modeling-trigger-context-adr.md | 5 ----- .changeset/grilling-remove-em-dashes.md | 5 ----- .../skill-tool-invocation-terminology.md | 9 -------- .changeset/user-invoked-skill-invocation.md | 11 ---------- .claude-plugin/plugin.json | 2 +- CHANGELOG.md | 22 +++++++++++++++++++ package.json | 2 +- 7 files changed, 24 insertions(+), 32 deletions(-) delete mode 100644 .changeset/domain-modeling-trigger-context-adr.md delete mode 100644 .changeset/grilling-remove-em-dashes.md delete mode 100644 .changeset/skill-tool-invocation-terminology.md delete mode 100644 .changeset/user-invoked-skill-invocation.md diff --git a/.changeset/domain-modeling-trigger-context-adr.md b/.changeset/domain-modeling-trigger-context-adr.md deleted file mode 100644 index da0fe9c..0000000 --- a/.changeset/domain-modeling-trigger-context-adr.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -domain-modeling: trigger on discussing codebase terminology and on writing or editing a CONTEXT.md or an ADR directly, replacing the narrower "pin down domain terminology or a ubiquitous language" / "record an architectural decision" phrasing. Also drops the "another skill needs to maintain the domain model" caveat — that's the invoking skill's job to state explicitly, not this description's. diff --git a/.changeset/grilling-remove-em-dashes.md b/.changeset/grilling-remove-em-dashes.md deleted file mode 100644 index 6cd562f..0000000 --- a/.changeset/grilling-remove-em-dashes.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -grilling: remove em-dashes from `SKILL.md`, replacing them with colons and semicolons so the instructions read as plain text. diff --git a/.changeset/skill-tool-invocation-terminology.md b/.changeset/skill-tool-invocation-terminology.md deleted file mode 100644 index d570264..0000000 --- a/.changeset/skill-tool-invocation-terminology.md +++ /dev/null @@ -1,9 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -Standardize cross-skill invocation on an explicit "call the Skill tool" instruction instead of bare `/skill`-style prose, across `code-review`, `diagnosing-bugs`, `grill-with-docs`, `grill-me`, `improve-codebase-architecture`, `tdd`, `to-spec`, `to-tickets`, `triage`, and `wayfinder`. - -- A skill that names another skill in prose ("run the `/grilling` skill") does not reliably cause it to load — this is the documented rough edge behind `grill-with-docs`'s most-reported problem. Naming the tool directly (`Call the Skill tool with "grilling"`) is intended to raise the hit rate. Dropping the leading `/` also makes the instruction harness-neutral rather than less: it no longer assumes Claude Code's trigger syntax. -- A step needing more than one skill now says so as multiple calls ("Call the Skill tool twice, for `grilling` and `domain-modeling`"), not one call carrying two names. -- Documents the convention in `.agents/invocation.md` for future skills to follow. diff --git a/.changeset/user-invoked-skill-invocation.md b/.changeset/user-invoked-skill-invocation.md deleted file mode 100644 index e7f206b..0000000 --- a/.changeset/user-invoked-skill-invocation.md +++ /dev/null @@ -1,11 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -Stop skills from trying to reach user-invoked skills through the Skill tool — fix cross-skill references that violated the "no other skill can call it" invariant in `.agents/invocation.md`, in `to-spec`, `wayfinder`, `to-tickets`, `triage`, `code-review`, and `diagnosing-bugs`. - -- `to-spec`, `wayfinder`, `to-tickets`, `triage`, and `code-review` each carried a precondition ("...run `/setup-matt-pocock-skills` if not") that PR #878 rewrote into a literal `Call the Skill tool with "setup-matt-pocock-skills"` instruction. `setup-matt-pocock-skills` is user-invoked, so none of these skills — user-invoked or model-invoked — can call it. Reworded all five as instructions for the agent to tell the human to run it instead. -- `diagnosing-bugs`'s Phase 6 post-mortem hand off to `improve-codebase-architecture` (also user-invoked) the same way, from an autonomous, often-unattended bug-fixing flow with no human in the loop to catch the failed call. Removed the hand-off outright rather than softening it — it rarely fired in practice. Phase 6 is now "Cleanup" only; the mechanical checklist is untouched. -- Added a carve-out paragraph to `.agents/invocation.md`'s "Dependencies between them" section: the `Call the Skill tool with "name"` convention only applies when the named skill is model-invoked. This is the section PR #878 introduced without reconciling it against the user-invoked/model-invoked invariant stated eight lines above it — the gap is most of why this bug reached six call sites instead of one. - -Fixes #453. diff --git a/.claude-plugin/plugin.json b/.claude-plugin/plugin.json index 60fc082..91c63b3 100644 --- a/.claude-plugin/plugin.json +++ b/.claude-plugin/plugin.json @@ -1,6 +1,6 @@ { "name": "mattpocock-skills", - "version": "1.2.3", + "version": "1.2.4", "description": "Matt Pocock's agent skills for real engineering — grilling, spec/ticket flows, TDD, code review, domain modelling and more. Plug-and-play, not vibe coding.", "author": { "name": "Matt Pocock", diff --git a/CHANGELOG.md b/CHANGELOG.md index 26d68ff..5572e01 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,27 @@ # mattpocock-skills +## 1.2.4 + +### Patch Changes + +- [#848](https://github.com/mattpocock/skills/pull/848) [`f02e2ed`](https://github.com/mattpocock/skills/commit/f02e2ed3624d031272f8547742d23bf6bca8b072) Thanks [@mattpocock](https://github.com/mattpocock)! - domain-modeling: trigger on discussing codebase terminology and on writing or editing a CONTEXT.md or an ADR directly, replacing the narrower "pin down domain terminology or a ubiquitous language" / "record an architectural decision" phrasing. Also drops the "another skill needs to maintain the domain model" caveat — that's the invoking skill's job to state explicitly, not this description's. + +- [#879](https://github.com/mattpocock/skills/pull/879) [`d419977`](https://github.com/mattpocock/skills/commit/d419977fe07d9e1607d3523f3579310bbb076b93) Thanks [@mattpocock](https://github.com/mattpocock)! - grilling: remove em-dashes from `SKILL.md`, replacing them with colons and semicolons so the instructions read as plain text. + +- [#878](https://github.com/mattpocock/skills/pull/878) [`e3e547b`](https://github.com/mattpocock/skills/commit/e3e547b57d549110a0aa6ff40fd7b871c01c76c9) Thanks [@mattpocock](https://github.com/mattpocock)! - Standardize cross-skill invocation on an explicit "call the Skill tool" instruction instead of bare `/skill`-style prose, across `code-review`, `diagnosing-bugs`, `grill-with-docs`, `grill-me`, `improve-codebase-architecture`, `tdd`, `to-spec`, `to-tickets`, `triage`, and `wayfinder`. + + - A skill that names another skill in prose ("run the `/grilling` skill") does not reliably cause it to load — this is the documented rough edge behind `grill-with-docs`'s most-reported problem. Naming the tool directly (`Call the Skill tool with "grilling"`) is intended to raise the hit rate. Dropping the leading `/` also makes the instruction harness-neutral rather than less: it no longer assumes Claude Code's trigger syntax. + - A step needing more than one skill now says so as multiple calls ("Call the Skill tool twice, for `grilling` and `domain-modeling`"), not one call carrying two names. + - Documents the convention in `.agents/invocation.md` for future skills to follow. + +- [#880](https://github.com/mattpocock/skills/pull/880) [`1dab982`](https://github.com/mattpocock/skills/commit/1dab98299c3b81f560026c01b7ebf55ed5d91373) Thanks [@mattpocock](https://github.com/mattpocock)! - Stop skills from trying to reach user-invoked skills through the Skill tool — fix cross-skill references that violated the "no other skill can call it" invariant in `.agents/invocation.md`, in `to-spec`, `wayfinder`, `to-tickets`, `triage`, `code-review`, and `diagnosing-bugs`. + + - `to-spec`, `wayfinder`, `to-tickets`, `triage`, and `code-review` each carried a precondition ("...run `/setup-matt-pocock-skills` if not") that PR [#878](https://github.com/mattpocock/skills/issues/878) rewrote into a literal `Call the Skill tool with "setup-matt-pocock-skills"` instruction. `setup-matt-pocock-skills` is user-invoked, so none of these skills — user-invoked or model-invoked — can call it. Reworded all five as instructions for the agent to tell the human to run it instead. + - `diagnosing-bugs`'s Phase 6 post-mortem hand off to `improve-codebase-architecture` (also user-invoked) the same way, from an autonomous, often-unattended bug-fixing flow with no human in the loop to catch the failed call. Removed the hand-off outright rather than softening it — it rarely fired in practice. Phase 6 is now "Cleanup" only; the mechanical checklist is untouched. + - Added a carve-out paragraph to `.agents/invocation.md`'s "Dependencies between them" section: the `Call the Skill tool with "name"` convention only applies when the named skill is model-invoked. This is the section PR [#878](https://github.com/mattpocock/skills/issues/878) introduced without reconciling it against the user-invoked/model-invoked invariant stated eight lines above it — the gap is most of why this bug reached six call sites instead of one. + + Fixes [#453](https://github.com/mattpocock/skills/issues/453). + ## 1.2.3 ### Patch Changes diff --git a/package.json b/package.json index 2200fef..9bd335e 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "mattpocock-skills", - "version": "1.2.3", + "version": "1.2.4", "private": true, "description": "Matt Pocock's agent skills for real engineering", "repository": {