From 2aecca12ea9ff047f76c8178cb64dfeafd192abe Mon Sep 17 00:00:00 2001 From: Matt Pocock Date: Thu, 24 Sep 2026 12:04:54 +0100 Subject: [PATCH] changeset: split the graduation changeset into one per skill Co-Authored-By: Claude Opus 5.5 (1M context) --- .changeset/graduate-implement-spec-pr-retro.md | 12 ------------ .changeset/graduate-implement-spec.md | 11 +++++++++++ .changeset/graduate-pr.md | 7 +++++++ .changeset/graduate-retro.md | 7 +++++++ 4 files changed, 25 insertions(+), 12 deletions(-) delete mode 100644 .changeset/graduate-implement-spec-pr-retro.md create mode 100644 .changeset/graduate-implement-spec.md create mode 100644 .changeset/graduate-pr.md create mode 100644 .changeset/graduate-retro.md diff --git a/.changeset/graduate-implement-spec-pr-retro.md b/.changeset/graduate-implement-spec-pr-retro.md deleted file mode 100644 index c100774..0000000 --- a/.changeset/graduate-implement-spec-pr-retro.md +++ /dev/null @@ -1,12 +0,0 @@ ---- -"mattpocock-skills": minor ---- - -Graduate **`implement-spec`**, **`pr`** and **`retro`** into the **Engineering** bucket, so they ship in the Claude Code plugin, get docs pages, and are routed by `ask-matt`. - -- **`implement-spec`** (user-invoked) implements a whole spec in one run. It reads the tickets as a **task graph**, runs implementer subagents in their own worktrees across the ready **frontier**, and lands everything on one **integration branch**, closing out with `code-review`. Ahead of graduating: - - The goal is now the integration branch, not a PR. A draft PR opens only when the issue tracker closes work through PRs or you ask for one, and only after the first merge (a branch with no commits ahead of main can't open one). Without a PR, the tickets are resolved the way the tracker closes work. - - It points at the issue tracker like its siblings, telling you to run `/setup-matt-pocock-skills` when none has been provided, rather than silently defaulting to `gh`. - - Each implementer confirms its worktree is based on the integration branch, builds its ticket with `tdd`, and merges the integration tip into its own branch before reporting done, so each merge is a fast-forward. -- **`pr`** (model-invoked) is the shape a pull request body should take: a summary as the smallest visual that makes the change clear (pseudocode, a call tree, a file tree, Mermaid, a diff), before/after evidence that it works, and a merge-danger call (one-way or two-way door, plus blast radius). The Summary visuals are adapted from Dex Horthy's `show-me`, credited in the skill's `CREDITS.md`. -- **`retro`** (user-invoked) looks back at a coding session and suggests changes to the agent's environment rather than the code: navigation pointers, automated checks, coding standards, steering files, tool economy, information access. It classifies each coding-standards finding first: a mechanical violation gets a deterministic check (a linter rule, a pre-commit hook, or a CI job), and `CODING_STANDARDS.md` is kept for genuine judgement calls. A repo with no guardrail at all is a finding in its own right. `ask-matt` now routes it as the last step of the main flow, after `code-review`. diff --git a/.changeset/graduate-implement-spec.md b/.changeset/graduate-implement-spec.md new file mode 100644 index 0000000..0b3ff1b --- /dev/null +++ b/.changeset/graduate-implement-spec.md @@ -0,0 +1,11 @@ +--- +"mattpocock-skills": minor +--- + +Graduate **`implement-spec`** into the **Engineering** bucket, so it ships in the Claude Code plugin, gets a docs page, and is routed by `ask-matt` as the parallel alternative to per-ticket `implement`. + +`implement-spec` (user-invoked) implements a whole spec in one run. It reads the tickets as a **task graph**, runs implementer subagents in their own worktrees across the ready **frontier**, and lands everything on one **integration branch**, closing out with `code-review`. Ahead of graduating: + +- The goal is now the integration branch, not a PR. A draft PR opens only when the issue tracker closes work through PRs or you ask for one, and only after the first merge (a branch with no commits ahead of main can't open one). Without a PR, the tickets are resolved the way the tracker closes work. +- It points at the issue tracker like its siblings, telling you to run `/setup-matt-pocock-skills` when none has been provided, rather than silently defaulting to `gh`. +- Each implementer confirms its worktree is based on the integration branch, builds its ticket with `tdd`, and merges the integration tip into its own branch before reporting done, so each merge is a fast-forward. diff --git a/.changeset/graduate-pr.md b/.changeset/graduate-pr.md new file mode 100644 index 0000000..8763d99 --- /dev/null +++ b/.changeset/graduate-pr.md @@ -0,0 +1,7 @@ +--- +"mattpocock-skills": minor +--- + +Graduate **`pr`** into the **Engineering** bucket, so it ships in the Claude Code plugin, gets a docs page, and is routed by `ask-matt` as the PR-body close-out. + +`pr` (model-invoked) is the shape a pull request body should take: a summary as the smallest visual that makes the change clear (pseudocode, a call tree, a file tree, Mermaid, a diff), before/after evidence that it works, and a merge-danger call (one-way or two-way door, plus blast radius). The Summary visuals are adapted from Dex Horthy's `show-me`, credited in the skill's `CREDITS.md`. diff --git a/.changeset/graduate-retro.md b/.changeset/graduate-retro.md new file mode 100644 index 0000000..d6000ab --- /dev/null +++ b/.changeset/graduate-retro.md @@ -0,0 +1,7 @@ +--- +"mattpocock-skills": minor +--- + +Graduate **`retro`** into the **Engineering** bucket, so it ships in the Claude Code plugin, gets a docs page, and is routed by `ask-matt` as the last step of the main flow, after `code-review`. + +`retro` (user-invoked) looks back at a coding session and suggests changes to the agent's environment rather than the code: navigation pointers, automated checks, coding standards, steering files, tool economy, information access. It classifies each coding-standards finding first: a mechanical violation gets a deterministic check (a linter rule, a pre-commit hook, or a CI job), and `CODING_STANDARDS.md` is kept for genuine judgement calls. A repo with no guardrail at all is a finding in its own right.