From 914d9b2808084af0d7a08d8915f63a206c424738 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 19 Aug 2026 13:09:35 +0000 Subject: [PATCH] chore: version skills --- .../domain-modeling-trigger-context-adr.md | 5 ---- .changeset/fix-yaml-frontmatter-colons.md | 5 ---- .changeset/grilling-remove-em-dashes.md | 5 ---- .changeset/remove-em-dashes-repo-wide.md | 5 ---- .../skill-tool-invocation-terminology.md | 9 ------ .changeset/user-invoked-skill-invocation.md | 11 -------- .changeset/wait-what-context-map.md | 5 ---- .claude-plugin/plugin.json | 2 +- CHANGELOG.md | 28 +++++++++++++++++++ package.json | 2 +- 10 files changed, 30 insertions(+), 47 deletions(-) delete mode 100644 .changeset/domain-modeling-trigger-context-adr.md delete mode 100644 .changeset/fix-yaml-frontmatter-colons.md delete mode 100644 .changeset/grilling-remove-em-dashes.md delete mode 100644 .changeset/remove-em-dashes-repo-wide.md delete mode 100644 .changeset/skill-tool-invocation-terminology.md delete mode 100644 .changeset/user-invoked-skill-invocation.md delete mode 100644 .changeset/wait-what-context-map.md diff --git a/.changeset/domain-modeling-trigger-context-adr.md b/.changeset/domain-modeling-trigger-context-adr.md deleted file mode 100644 index 8ab6722..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, since that's the invoking skill's job to state explicitly, not this description's. diff --git a/.changeset/fix-yaml-frontmatter-colons.md b/.changeset/fix-yaml-frontmatter-colons.md deleted file mode 100644 index 64f830b..0000000 --- a/.changeset/fix-yaml-frontmatter-colons.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -Quote the `description` front matter in `to-spec`, `code-review`, `setup-matt-pocock-skills`, `writing-fragments`, `writing-shape`, and `wait-what`. An unquoted colon-space left over from the em-dash sweep in #905 made each block invalid YAML, so `skills.sh` skipped all six during discovery and they couldn't be listed or installed via `npx skills`. 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/remove-em-dashes-repo-wide.md b/.changeset/remove-em-dashes-repo-wide.md deleted file mode 100644 index a3909af..0000000 --- a/.changeset/remove-em-dashes-repo-wide.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -Remove every em-dash from the repo's prose (docs, `SKILL.md` files, ADRs, `README.md`, scripts, JSON/YAML metadata), hand-rewriting each sentence with a comma, colon, period, parentheses, or conjunction rather than mechanically substituting the character. `CLAUDE.md`/`AGENTS.md` now says not to reintroduce them. diff --git a/.changeset/skill-tool-invocation-terminology.md b/.changeset/skill-tool-invocation-terminology.md deleted file mode 100644 index 153b556..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 18e6a7c..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, since 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/.changeset/wait-what-context-map.md b/.changeset/wait-what-context-map.md deleted file mode 100644 index edaa4d6..0000000 --- a/.changeset/wait-what-context-map.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -"mattpocock-skills": patch ---- - -wait-what: follow `CONTEXT-MAP.md` to the right `CONTEXT.md` when a repo indexes multiple contexts that way instead of keeping a single root `CONTEXT.md`. diff --git a/.claude-plugin/plugin.json b/.claude-plugin/plugin.json index 0a2e308..67213f5 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..81f4e4c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,33 @@ # 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, since that's the invoking skill's job to state explicitly, not this description's. + +- [#911](https://github.com/mattpocock/skills/pull/911) [`4f28947`](https://github.com/mattpocock/skills/commit/4f289474bad013fe2be8f8769d733f59d9103d6b) Thanks [@mattpocock](https://github.com/mattpocock)! - Quote the `description` front matter in `to-spec`, `code-review`, `setup-matt-pocock-skills`, `writing-fragments`, `writing-shape`, and `wait-what`. An unquoted colon-space left over from the em-dash sweep in [#905](https://github.com/mattpocock/skills/issues/905) made each block invalid YAML, so `skills.sh` skipped all six during discovery and they couldn't be listed or installed via `npx skills`. + +- [#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. + +- [#905](https://github.com/mattpocock/skills/pull/905) [`e6e9577`](https://github.com/mattpocock/skills/commit/e6e957797d8cceb5b351c0dc840369523f9fb8fb) Thanks [@mattpocock](https://github.com/mattpocock)! - Remove every em-dash from the repo's prose (docs, `SKILL.md` files, ADRs, `README.md`, scripts, JSON/YAML metadata), hand-rewriting each sentence with a comma, colon, period, parentheses, or conjunction rather than mechanically substituting the character. `CLAUDE.md`/`AGENTS.md` now says not to reintroduce them. + +- [#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, since 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). + +- [#904](https://github.com/mattpocock/skills/pull/904) [`594f0f8`](https://github.com/mattpocock/skills/commit/594f0f83188921a60d45d63d6cdac509de20df2c) Thanks [@mattpocock](https://github.com/mattpocock)! - wait-what: follow `CONTEXT-MAP.md` to the right `CONTEXT.md` when a repo indexes multiple contexts that way instead of keeping a single root `CONTEXT.md`. + ## 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": {