Merge pull request #917 from mattpocock/grilling-add-hr-between-questions

grilling: separate questions in a round with an HR
2026-09-12 10:28:06 +07:00 · 2026-08-20 11:35:14 +01:00 · 2026-08-20 10:34:29 +00:00 · 2026-08-19 14:09:18 +01:00 · 2026-08-19 13:04:09 +00:00 · 2026-08-19 13:01:26 +00:00
9 changed files with 23 additions and 7 deletions
@@ -0,0 +1,5 @@
+---
+"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`.
@@ -0,0 +1,5 @@
+---
+"mattpocock-skills": patch
+---
+
+grilling: update the round template so consecutive questions are separated by a horizontal rule (`---`) instead of running together.
@@ -1,6 +1,6 @@
 ---
 name: code-review
-description: Review the changes since a fixed point (commit, branch, tag, or merge-base) along two axes: Standards (does the code follow this repo's documented coding standards?) and Spec (does the code match what the originating issue/spec asked for?). Runs both reviews in parallel sub-agents and reports them side by side. Use when the user wants to review a branch, a PR, work-in-progress changes, or asks to "review since X".
+description: "Review the changes since a fixed point (commit, branch, tag, or merge-base) along two axes: Standards (does the code follow this repo's documented coding standards?) and Spec (does the code match what the originating issue/spec asked for?). Runs both reviews in parallel sub-agents and reports them side by side. Use when the user wants to review a branch, a PR, work-in-progress changes, or asks to \"review since X\"."
 ---

 Two-axis review of the diff between `HEAD` and a fixed point the user supplies:
@@ -1,6 +1,6 @@
 ---
 name: setup-matt-pocock-skills
-description: Configure this repo for the engineering skills: set up its issue tracker, triage label vocabulary, and domain doc layout. Run once before first use of the other engineering skills.
+description: "Configure this repo for the engineering skills: set up its issue tracker, triage label vocabulary, and domain doc layout. Run once before first use of the other engineering skills."
 disable-model-invocation: true
 ---

@@ -1,6 +1,6 @@
 ---
 name: to-spec
-description: Turn the current conversation into a spec and publish it to the project issue tracker: no interview, just synthesis of what you've already discussed.
+description: "Turn the current conversation into a spec and publish it to the project issue tracker: no interview, just synthesis of what you've already discussed."
 disable-model-invocation: true
 ---

@@ -1,6 +1,6 @@
 ---
 name: writing-fragments
-description: Writing, explore: mine raw fragments, no structure yet.
+description: "Writing, explore: mine raw fragments, no structure yet."
 disable-model-invocation: true
 ---

@@ -1,6 +1,6 @@
 ---
 name: writing-shape
-description: Writing, exploit: shape raw material into an article, paragraph by paragraph.
+description: "Writing, exploit: shape raw material into an article, paragraph by paragraph."
 disable-model-invocation: true
 ---

@@ -7,11 +7,17 @@ Interview the user relentlessly until you reach a shared understanding. Map this

 Work the tree in **rounds**. The **frontier** is every decision whose prerequisites are already settled: the questions you can ask _now_ without guessing at answers you haven't heard yet. Ask the whole frontier in one round: number each question and give your recommended answer. Then wait for the user's answers before the next round.

-Each question should be formatted like so:
+Format a round like so:

 ```
 ❓ **Q1** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>

+➡️ <your recommended answer>
+
+---
+
+❓ **Q2** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>
+
 ➡️ <your recommended answer>
 ```

@@ -1,6 +1,6 @@
 ---
 name: wait-what
-description: Stop. That last message did not land: re-pitch it.
+description: "Stop. That last message did not land: re-pitch it."
 disable-model-invocation: true
 ---
Author	SHA1	Message	Date
Matt Pocock	0ab1b63a41	Merge pull request #917 from mattpocock/grilling-add-hr-between-questions grilling: separate questions in a round with an HR	2026-08-20 11:35:14 +01:00
remote-box	85f83d3fde	grilling: separate questions in a round with an HR Multi-question rounds ran straight into each other with no visual break. Fold the horizontal rule (---) directly into the round template so consecutive questions are shown as distinct blocks; the template demonstrates the separator, so no extra prose instruction is needed.	2026-08-20 10:34:29 +00:00
Matt Pocock	885e2ca4d8	Merge pull request #911 from mattpocock/fix/907-yaml-frontmatter-colons Fix invalid YAML front matter in six SKILL.md files	2026-08-19 14:09:18 +01:00
Claude Sonnet 5	4f289474ba	chore: add changeset for YAML front-matter fix Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-19 13:04:09 +00:00
Claude Sonnet 5	5c89081d4b	fix: quote SKILL.md descriptions with unquoted colons The colon-space sequences left behind by #905's em-dash-to-colon sweep make these six front-matter blocks invalid YAML, so skills.sh skips them during discovery and they can't be installed via `npx skills`. Quoting the description scalar fixes parsing without changing the wording. Fixes #907 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-19 13:01:26 +00:00
Matt Pocock	1bb95954ef	Merge pull request #905 from mattpocock/remove-em-dashes Remove all em-dashes from the repo	2026-08-19 11:04:49 +01:00