@erclx/canon 5.34.0 → 5.36.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (248) hide show
  1. package/README.md +15 -17
  2. package/claude/.claude-plugin/plugin.json +1 -1
  3. package/claude/skills/api-design/REQUIREMENT.md +1 -1
  4. package/claude/skills/auto-ship/references/draft-mark.md +1 -1
  5. package/claude/skills/canon-cli/REQUIREMENT.md +2 -2
  6. package/claude/skills/canon-cli/SKILL.md +12 -12
  7. package/claude/skills/canvas-board/REQUIREMENT.md +1 -1
  8. package/claude/skills/canvas-board/SKILL.md +2 -2
  9. package/claude/skills/canvas-board/references/design.md +53 -36
  10. package/claude/skills/codebase-layout/REQUIREMENT.md +1 -1
  11. package/claude/skills/codebase-layout/SKILL.md +2 -2
  12. package/claude/skills/create-rule/REQUIREMENT.md +6 -8
  13. package/claude/skills/create-rule/SKILL.md +8 -8
  14. package/claude/skills/create-rule/references/rule.md +11 -23
  15. package/claude/skills/create-skill/SKILL.md +2 -1
  16. package/claude/skills/create-skill/references/skill-paths.md +12 -15
  17. package/{standards/standard.md → claude/skills/create-skill/references/skill-reference.md} +50 -45
  18. package/claude/skills/create-skill/references/skill.md +9 -3
  19. package/claude/skills/design-taste/REQUIREMENT.md +6 -6
  20. package/claude/skills/design-taste/SKILL.md +2 -2
  21. package/claude/skills/design-taste/references/craft.md +1 -1
  22. package/claude/skills/design-taste/references/preflight.md +2 -2
  23. package/claude/skills/design-taste/references/tells.md +1 -1
  24. package/claude/skills/draft-and-pick/SKILL.md +1 -1
  25. package/claude/skills/draft-identity/SKILL.md +3 -2
  26. package/claude/skills/git-commit/references/versioning.md +13 -13
  27. package/claude/skills/git-pr/SKILL.md +1 -1
  28. package/claude/skills/git-pr/references/publish.md +1 -1
  29. package/claude/skills/markdown-craft/references/markdown.md +3 -2
  30. package/claude/skills/plan-feature/SKILL.md +1 -1
  31. package/claude/skills/review-address/REQUIREMENT.md +1 -1
  32. package/claude/skills/review-address/SKILL.md +2 -2
  33. package/claude/skills/review-craft/references/ui.md +1 -1
  34. package/claude/skills/review-pr/SKILL.md +3 -3
  35. package/claude/skills/review-pr/references/post.md +1 -1
  36. package/claude/skills/review-ui/REQUIREMENT.md +1 -1
  37. package/claude/skills/review-ui/SKILL.md +1 -1
  38. package/claude/skills/role-orchestrator/REQUIREMENT.md +2 -2
  39. package/claude/skills/role-orchestrator/SKILL.md +11 -12
  40. package/claude/skills/role-orchestrator/references/orchestrator-cloud-launch.md +3 -1
  41. package/claude/skills/role-orchestrator/references/orchestrator-dispatch.md +11 -8
  42. package/claude/skills/role-orchestrator/references/orchestrator-handback.md +10 -4
  43. package/claude/skills/role-orchestrator/references/orchestrator-launch.md +2 -2
  44. package/claude/skills/role-orchestrator/references/orchestrator-poll.md +11 -12
  45. package/claude/skills/role-orchestrator/references/orchestrator-review-dispatch.md +12 -15
  46. package/claude/skills/role-orchestrator/references/orchestrator-reviewer-launch.md +6 -4
  47. package/claude/skills/role-orchestrator/scripts/poll.ts +50 -1
  48. package/claude/skills/role-reviewer/REQUIREMENT.md +17 -7
  49. package/claude/skills/role-reviewer/SKILL.md +17 -10
  50. package/claude/skills/role-worker/REQUIREMENT.md +1 -1
  51. package/claude/skills/role-worker/SKILL.md +6 -6
  52. package/claude/skills/session-compact/references/handoff-note.md +1 -1
  53. package/claude/skills/target-setup/SKILL.md +2 -2
  54. package/claude/skills/target-setup/references/detect.md +3 -3
  55. package/claude/skills/test-craft/SKILL.md +2 -2
  56. package/claude/skills/ux-audit/REQUIREMENT.md +1 -1
  57. package/claude/skills/ux-audit/SKILL.md +2 -2
  58. package/claude/skills/ux-measure/REQUIREMENT.md +1 -1
  59. package/claude/skills/write-human/references/machine-tells.md +12 -0
  60. package/docs/agents/audits.md +1 -1
  61. package/docs/agents/commands.md +14 -13
  62. package/docs/agents/comments.md +1 -1
  63. package/docs/agents/counts.md +5 -6
  64. package/docs/agents/drift-surfaces.md +15 -19
  65. package/docs/agents/gov-detect.md +1 -1
  66. package/docs/agents/index.md +2 -3
  67. package/docs/agents/install-and-sync.md +10 -34
  68. package/docs/agents/markdown-audit.md +22 -12
  69. package/docs/agents/migrate.md +1 -1
  70. package/docs/agents/output-shape.md +1 -1
  71. package/docs/agents/overview.md +2 -2
  72. package/docs/agents/records-stale.md +1 -1
  73. package/docs/agents/records-validate.md +11 -23
  74. package/docs/agents/records.md +1 -1
  75. package/docs/agents/routing.md +1 -1
  76. package/docs/agents/rule-citations.md +9 -17
  77. package/docs/agents/scripting.md +2 -20
  78. package/docs/agents/state-scoped-risk.md +1 -1
  79. package/docs/agents/tasks-validate.md +1 -1
  80. package/docs/agents/teach.md +1 -1
  81. package/docs/target/index.md +1 -0
  82. package/docs/target/migrations.md +4 -0
  83. package/docs/target/projects.md +10 -10
  84. package/docs/target/scenarios.md +5 -6
  85. package/docs/target/standards-moved.md +32 -0
  86. package/docs/target/sync.md +5 -5
  87. package/docs/workflow/ai-workflow.md +1 -1
  88. package/docs/workflow/operating-model.md +16 -18
  89. package/docs/workflow/skill-map.md +1 -1
  90. package/docs/workflow/visual-design-workflow.md +3 -3
  91. package/governance/detect.toml +32 -36
  92. package/governance/renames.toml +44 -1
  93. package/governance/rules/lib/{305-e2e-reliability.md → e2e-reliability.md} +1 -1
  94. package/governance/rules/lib/{306-test-scope.md → test-scope.md} +2 -2
  95. package/governance/rules/ui/{410-a11y.md → a11y.md} +1 -1
  96. package/governance/rules/ui/{440-surface-capture.md → surface-capture.md} +3 -3
  97. package/governance/stacks/base.toml +15 -8
  98. package/governance/stacks/go.toml +2 -0
  99. package/governance/stacks/node.toml +1 -1
  100. package/governance/stacks/php.toml +2 -0
  101. package/governance/stacks/python.toml +1 -1
  102. package/package.json +1 -4
  103. package/scripts/core/install-check.sh +4 -10
  104. package/src/audits/catalog.ts +0 -9
  105. package/src/capture/frame-values.ts +3 -6
  106. package/src/claude/routing.ts +1 -1
  107. package/src/claude/skills-reach.ts +4 -6
  108. package/src/cli/help-all.ts +96 -0
  109. package/src/cli/help.ts +29 -29
  110. package/src/cli.ts +8 -3
  111. package/src/commands/claude.ts +29 -0
  112. package/src/commands/design.ts +22 -1
  113. package/src/commands/gov.ts +29 -11
  114. package/src/commands/init.ts +10 -1
  115. package/src/commands/labels.ts +4 -44
  116. package/src/commands/markdown.ts +96 -3
  117. package/src/commands/migrate.ts +1 -1
  118. package/src/commands/records.ts +5 -21
  119. package/src/commands/sync.ts +11 -0
  120. package/src/commands/tooling.ts +10 -4
  121. package/src/counts/catalogs.ts +0 -6
  122. package/src/design/budget.ts +14 -10
  123. package/src/design/document.ts +5 -5
  124. package/src/design/parse.ts +21 -4
  125. package/src/design/render.ts +3 -3
  126. package/src/gate/boundaries.ts +2 -2
  127. package/src/gate/measures.ts +7 -29
  128. package/src/gate/sequencer.ts +51 -2
  129. package/src/gate/stages.ts +0 -18
  130. package/src/gov/citations.ts +16 -71
  131. package/src/gov/stacks.ts +3 -3
  132. package/src/inventory/subjects.ts +1 -1
  133. package/src/markdown/bans.ts +1 -1
  134. package/src/markdown/ceiling.ts +6 -1
  135. package/src/markdown/colons.ts +86 -0
  136. package/src/markdown/gate.ts +15 -0
  137. package/src/markdown/heading-case.ts +74 -0
  138. package/src/migrate/scratch-evidence.ts +2 -2
  139. package/src/records/validate.ts +6 -194
  140. package/src/sandbox/provision.ts +0 -5
  141. package/src/secrets/shipped.ts +1 -2
  142. package/src/serve/static.ts +1 -2
  143. package/src/shipped/references.ts +42 -93
  144. package/src/sync/check.ts +18 -14
  145. package/src/sync/copy.ts +4 -5
  146. package/src/tooling/inject.ts +6 -37
  147. package/src/tooling/read.ts +2 -3
  148. package/src/tooling/scan.ts +4 -17
  149. package/src/tooling/subfolder.ts +2 -66
  150. package/tooling/astro/manifest.toml +0 -1
  151. package/tooling/astro/reference.md +0 -8
  152. package/tooling/base/configs/scripts/verify.json +1 -1
  153. package/tooling/base/configs/scripts/verify.ts +22 -17
  154. package/tooling/base/manifest.toml +7 -7
  155. package/tooling/base/reference.md +19 -32
  156. package/tooling/base/seeds/.github/workflows/verify.yml +0 -9
  157. package/tooling/base/seeds/.lintstagedrc +2 -4
  158. package/tooling/base/seeds/canon/context/ci.md +4 -6
  159. package/tooling/base/seeds/canon/context/development.md +2 -2
  160. package/tooling/claude/reference.md +1 -1
  161. package/tooling/claude/seeds/.claude/hooks/review-post-scan.sh +13 -1
  162. package/tooling/claude/seeds/.claude/hooks/scratch-guard.sh +1 -1
  163. package/tooling/claude/seeds/.claude/hooks/standards-audit.sh +27 -2
  164. package/tooling/claude/seeds/CLAUDE.md +1 -1
  165. package/tooling/claude/seeds/canon/DESIGN.md +24 -37
  166. package/tooling/go/reference.md +1 -1
  167. package/tooling/php/reference.md +1 -1
  168. package/tooling/python/reference.md +2 -2
  169. package/tooling/shell/configs/.github/workflows/shell.yml +30 -0
  170. package/tooling/shell/manifest.toml +12 -0
  171. package/tooling/shell/reference.md +46 -0
  172. package/tooling/vite-react/manifest.toml +0 -1
  173. package/tooling/vite-react/reference.md +0 -8
  174. package/tooling/web/configs/.github/workflows/readme-screenshot.yml +3 -5
  175. package/tooling/web/configs/.github/workflows/verify.yml +0 -9
  176. package/tooling/web/manifest.toml +1 -2
  177. package/tooling/web/reference.md +4 -6
  178. package/docs/agents/standards-audit.md +0 -40
  179. package/governance/rules/tooling/765-spelling.md +0 -19
  180. package/governance/stacks/astro.toml +0 -2
  181. package/governance/stacks/nextjs.toml +0 -2
  182. package/governance/stacks/node-server.toml +0 -2
  183. package/governance/stacks/python-fastapi.toml +0 -2
  184. package/governance/stacks/react.toml +0 -2
  185. package/src/commands/standards.ts +0 -278
  186. package/src/labels/spelling.ts +0 -120
  187. package/src/standards/audit.ts +0 -146
  188. package/src/standards/list.ts +0 -33
  189. package/src/standards/read.ts +0 -177
  190. package/src/standards/scope.ts +0 -56
  191. package/standards/index.md +0 -10
  192. package/tooling/base/configs/.vscode/extensions.json +0 -9
  193. package/tooling/base/configs/.vscode/settings.json +0 -3
  194. package/tooling/base/configs/scripts/verify.sh +0 -2
  195. package/tooling/base/seeds/.cspell/project-terms.txt +0 -0
  196. package/tooling/base/seeds/.cspell/tech-stack.txt +0 -25
  197. package/tooling/base/seeds/cspell.json +0 -20
  198. package/tooling/go/seeds/.cspell/tech-stack.txt +0 -16
  199. package/tooling/nextjs/seeds/.cspell/tech-stack.txt +0 -3
  200. package/tooling/php/seeds/.cspell/tech-stack.txt +0 -5
  201. package/tooling/python/seeds/.cspell/tech-stack.txt +0 -18
  202. package/tooling/vite-react/seeds/.cspell/project-terms.txt +0 -1
  203. package/tooling/vite-react/seeds/.cspell/tech-stack.txt +0 -2
  204. package/tooling/web/configs/.vscode/extensions.json +0 -13
  205. package/tooling/web/configs/.vscode/settings.json +0 -10
  206. package/tooling/web/seeds/.cspell/tech-stack.txt +0 -21
  207. /package/governance/rules/canon/{606-git.md → git.md} +0 -0
  208. /package/governance/rules/canon/{611-indexes.md → indexes.md} +0 -0
  209. /package/governance/rules/canon/{604-scratch.md → scratch.md} +0 -0
  210. /package/governance/rules/canon/{602-tasks.md → task-scope.md} +0 -0
  211. /package/governance/rules/canon/{605-worktrees.md → worktrees.md} +0 -0
  212. /package/governance/rules/claude/{566-output.md → output.md} +0 -0
  213. /package/governance/rules/code/{000-code.md → routing.md} +0 -0
  214. /package/governance/rules/code/{010-testing.md → testing.md} +0 -0
  215. /package/governance/rules/framework/{210-astro.md → astro.md} +0 -0
  216. /package/governance/rules/framework/{220-fastapi.md → fastapi.md} +0 -0
  217. /package/governance/rules/framework/{230-nextjs.md → nextjs.md} +0 -0
  218. /package/governance/rules/framework/{200-react.md → react.md} +0 -0
  219. /package/governance/rules/framework/{260-shadcn.md → shadcn.md} +0 -0
  220. /package/governance/rules/framework/{250-tailwind.md → tailwind.md} +0 -0
  221. /package/governance/rules/lang/{130-go.md → go.md} +0 -0
  222. /package/governance/rules/lang/{140-php.md → php.md} +0 -0
  223. /package/governance/rules/lang/{110-python.md → python.md} +0 -0
  224. /package/governance/rules/lang/{100-typescript.md → typescript.md} +0 -0
  225. /package/governance/rules/lib/{370-database.md → database.md} +0 -0
  226. /package/governance/rules/lib/{340-pydantic.md → pydantic.md} +0 -0
  227. /package/governance/rules/lib/{360-security-server.md → security-server.md} +0 -0
  228. /package/governance/rules/lib/{350-security-web.md → security-web.md} +0 -0
  229. /package/governance/rules/lib/{320-tanstack-query.md → tanstack-query.md} +0 -0
  230. /package/governance/rules/lib/{335-testing-go.md → testing-go.md} +0 -0
  231. /package/governance/rules/lib/{336-testing-php.md → testing-php.md} +0 -0
  232. /package/governance/rules/lib/{330-testing-py.md → testing-py.md} +0 -0
  233. /package/governance/rules/lib/{300-testing-ts.md → testing-ts.md} +0 -0
  234. /package/governance/rules/lib/{310-zod.md → zod.md} +0 -0
  235. /package/governance/rules/tooling/{700-ci-workflow.md → ci-workflow.md} +0 -0
  236. /package/governance/rules/tooling/{790-code-comments.md → code-comments.md} +0 -0
  237. /package/governance/rules/tooling/{780-config-comments.md → config-comments.md} +0 -0
  238. /package/governance/rules/tooling/{775-dependencies.md → dependencies.md} +0 -0
  239. /package/governance/rules/ui/{480-canvas.md → canvas.md} +0 -0
  240. /package/governance/rules/ui/{400-ui.md → copy.md} +0 -0
  241. /package/governance/rules/ui/{460-design-taste.md → design-taste.md} +0 -0
  242. /package/governance/rules/ui/{420-forms.md → forms.md} +0 -0
  243. /package/governance/rules/ui/{465-interface-casing.md → interface-casing.md} +0 -0
  244. /package/governance/rules/ui/{450-link-behavior.md → link-behavior.md} +0 -0
  245. /package/governance/rules/ui/{470-motion.md → motion.md} +0 -0
  246. /package/governance/rules/ui/{430-ux-completeness.md → ux-completeness.md} +0 -0
  247. /package/tooling/{base → shell}/configs/.editorconfig +0 -0
  248. /package/tooling/{base → shell}/configs/.shellcheckrc +0 -0
@@ -1,17 +1,17 @@
1
1
  ---
2
2
  title: Skill path reference
3
- description: Which root a file a skill body names resolves against, and how a body cites a standard
3
+ description: Which root a file a skill body names resolves against, and how a body cites another skill's reference
4
4
  ---
5
5
 
6
6
  # Skill path reference
7
7
 
8
8
  Applies to every path a skill body names, under `skills/<name>/`. A skill reads from two roots, so the path it writes decides whether the file is found once the skill runs from another project.
9
9
 
10
- This is an attribute standard. It governs the path strings a body carries rather than a file of its own, so it carries no template.
10
+ This is an attribute reference. It governs the path strings a body carries rather than a file of its own, so it carries no template.
11
11
 
12
12
  ## Scope
13
13
 
14
- Governs how a `SKILL.md` body or a file under its `references/` names a bundled asset, an installed shared doc, or a standard.
14
+ Governs how a `SKILL.md` body or a file under its `references/` names a bundled asset, an installed shared doc, or another skill's reference.
15
15
 
16
16
  Does not govern:
17
17
 
@@ -23,7 +23,7 @@ Does not govern:
23
23
  A path in a skill body works when the skill resolves it from any project it runs in:
24
24
 
25
25
  - Does each bundled asset resolve against the skill's own directory rather than the session's working directory?
26
- - Does each cited standard resolve in a target that installed nothing but the plugin?
26
+ - Does each cited reference resolve in a target that installed nothing but the plugin?
27
27
  - Does a procedure two skills share live in one place, with each body citing it?
28
28
 
29
29
  A body failing any of these is non-conforming even when every path resolves in this repository.
@@ -34,18 +34,15 @@ Know which root a file lives under before referencing it.
34
34
 
35
35
  - Bundled skill assets (`references/`, `scripts/`, `assets/`) resolve against the skill's own directory. Reference them with `${CLAUDE_SKILL_DIR}/<path>`, never a bare relative path, which resolves against the session cwd and fails when a plugin skill runs from another project.
36
36
  - Installed shared docs (`.claude/rules/`, `canon/context/`) resolve against the target project cwd, where install placed them. Reference them by that path.
37
- - Do not hand-copy a standard into a skill. If a skill must carry its own copy, generate it from the single source and reference it through `${CLAUDE_SKILL_DIR}`, so one owner keeps every copy in sync.
37
+ - Do not hand-copy another skill's reference into a skill. If a skill must carry its own copy, generate it from the single source and reference it through `${CLAUDE_SKILL_DIR}`, so one owner keeps every copy in sync.
38
38
 
39
- ## Citing a standard
39
+ ## Citing another skill's reference
40
40
 
41
- No standard installs into a project, so a body cites one place rather than choosing between two.
41
+ A reference shared across skills lives in the `references/` of the skill that owns it, so a body cites it by the sibling path.
42
42
 
43
- - Cite `${CLAUDE_SKILL_DIR}/../../standards/X.md`. The plugin ships the whole standards folder beside `skills/`, so the path resolves in every install.
44
- - Cite another skill's reference as `${CLAUDE_SKILL_DIR}/../<skill>/references/<name>.md`. A standard that moved into the skill owning it lives there, and the plugin ships the skills side by side, so the sibling path resolves in every install. Name the owner skill rather than copying the file.
45
- - Never cite `.claude/standards/X.md` from a shipped body. A target holds no such folder. <!-- audit-ignore-citations: .claude/standards/X.md -->
46
- - Name `canon standards X` instead where the body wants the document rather than a path to open, such as a value it captures or reports. That verb resolves `standards/` at the project root and then the corpus inside the package.
47
- - State the path once per body, at the site that reads the standard. A later mention of a standard the body already read stays bare, since repeating the path at every mention is noise rather than instruction.
48
- - A guard on a standard's presence names the file rather than the folder holding it, since a folder test answers for a sibling that happens to be there.
43
+ - Cite another skill's reference as `${CLAUDE_SKILL_DIR}/../<skill>/references/<name>.md`. The plugin ships the skills side by side, so the sibling path resolves in every install. Name the owner skill rather than copying the file.
44
+ - State the path once per body, at the site that reads the reference. A later mention of a reference the body already read stays bare, since repeating the path at every mention is noise rather than instruction.
45
+ - A guard on a reference's presence names the file rather than the folder holding it, since a folder test answers for a sibling that happens to be there.
49
46
  - Use `${CLAUDE_SKILL_DIR}`, never a bare `../../` and never `${CLAUDE_PLUGIN_ROOT}`, since only the first expands before the body reaches the model.
50
- - Cite a shared procedure, never restate it. A procedure two or more skills execute gets one definition in a standard and a citation in each body.
51
- - Keep the trigger in the body and the procedure in the standard. The citing skill states when the procedure runs and what it runs against, since that varies per skill and the standard cannot know it.
47
+ - Cite a shared procedure, never restate it. A procedure two or more skills execute gets one definition in its owner's reference and a citation in each body.
48
+ - Keep the trigger in the body and the procedure in the reference. The citing skill states when the procedure runs and what it runs against, since that varies per skill and the reference cannot know it.
@@ -1,75 +1,83 @@
1
1
  ---
2
- title: Standard reference
3
- description: Shape and content rules for authoring a standard
2
+ title: Reference file reference
3
+ description: Shape and content rules for a reference that sets the shape of one artifact
4
4
  ---
5
5
 
6
- # Standard reference
6
+ # Reference file reference
7
7
 
8
- Applies to each authored standard in the folder. Skip for `index.md`, which is generated rather than authored.
8
+ Applies to each file under a skill's `references/` that sets the shape of one document type, or of one attribute carried across every document. A session reads it at the point the skill body names it, and skills and developers consume it alike. This file governs itself, so every rule below applies to it.
9
9
 
10
- ## Overview
10
+ ## Contents
11
11
 
12
- A standard is a target-facing authoring convention for one document type, or for one attribute carried across every document. Nothing installs it into a project, so a reader resolves it with `canon standards <name>` or reads it at `standards/<name>.md`, and skills and developers consume it alike. This file governs itself, so every rule below applies to it.
12
+ - [Scope](#scope)
13
+ - [What a working reference looks like](#what-a-working-reference-looks-like)
14
+ - [Scoping rules](#scoping-rules)
15
+ - [Frontmatter](#frontmatter)
16
+ - [Structure](#structure)
17
+ - [Rules](#rules)
18
+ - [Success criterion](#success-criterion)
19
+ - [Changing a reference](#changing-a-reference)
20
+ - [Examples](#examples)
21
+ - [Template](#template)
13
22
 
14
23
  ## Scope
15
24
 
16
- Governs each authored standard under `standards/`: its stated jurisdiction, success criterion, frontmatter, structure, and rule phrasing.
25
+ Governs a shape reference at `skills/<name>/references/<file>.md`: its stated jurisdiction, success criterion, frontmatter, structure, and rule phrasing.
17
26
 
18
27
  Does not govern:
19
28
 
20
- - The voice, rhythm, and sentence construction a standard is written in: the `write-human` skill
21
- - The punctuation, formatting, and word choice a standard is written in: `markdown.md`
22
- - The shape of any artifact a standard governs, which is that standard's own subject
29
+ - The `references/` folder's place in the skill, its line ceiling, and the body naming each file: `skill.md`
30
+ - How a body or a reference cites another skill's reference: `skill-paths.md`
31
+ - The voice, rhythm, and sentence construction a reference is written in: the `write-human` skill
32
+ - The punctuation, formatting, and word choice a reference is written in: `../markdown-craft/references/markdown.md`
33
+ - The shape of any artifact a reference governs, which is that reference's own subject
23
34
 
24
- ## What a working standard looks like
35
+ ## What a working reference looks like
25
36
 
26
- A standard answers these questions. Each can be answered wrong, which is what makes them a test rather than a preamble.
37
+ A shape reference answers these questions. Each can be answered wrong, which is what makes them a test rather than a preamble.
27
38
 
28
39
  - Which single document type or attribute does this govern, and where does it apply?
29
40
  - Can an author who has seen no example produce a conforming document from this file alone?
30
41
  - Does every rule state a shape the document must have, rather than a fact about the repository that happens to store it?
31
42
  - What does a conforming document achieve, stated so a reviewer can call one non-conforming without appealing to taste?
32
43
 
33
- A standard failing these questions is non-conforming even when it satisfies every shape rule below.
44
+ A reference failing these questions is non-conforming even when it satisfies every shape rule below.
34
45
 
35
46
  ## Scoping rules
36
47
 
37
48
  ### Declaring scope
38
49
 
39
- - Govern one document type per standard, or one attribute across every document. Split unrelated conventions into separate files.
40
- - Open with a `## Scope` section stating what the standard governs and what it does not, placed above the shape rules. A standard that specifies shape exhaustively and jurisdiction nowhere cannot refuse a rule, so the rule with no obvious owner lands in whichever standard sits nearest.
41
- - Write it as one line naming the artifact or attribute and where it applies, then a `Does not govern:` list. Give each entry the excluded concern and the owner it goes to. Name a sibling standard by bare filename, since the corpus is one flat folder, and name the surface instead where the owner is one, such as a coding rule, a project policy, or the code.
50
+ - Govern one document type per reference, or one attribute across every document. Split unrelated conventions into separate files.
51
+ - Open with a `## Scope` section stating what the reference governs and what it does not, placed above the shape rules. A reference that specifies shape exhaustively and jurisdiction nowhere cannot refuse a rule, so the rule with no obvious owner lands in whichever reference sits nearest.
52
+ - Write it as one line naming the artifact or attribute and where it applies, then a `Does not govern:` list. Give each entry the excluded concern and the owner it goes to.
53
+ - Name an owner in the same `references/` folder by bare filename and one in another skill by its sibling path, `../<skill>/references/<name>.md`. Name the surface instead where the owner is one, such as a coding rule, a project policy, or the code.
42
54
  - Put the governed path in backticks inside the first sentence of that line, anchored deep enough to resolve from a project root. A bare folder name matches a same-named folder elsewhere in the tree, and a path held back until a later sentence sits beside every other path the statement mentions, including the ones it excludes.
43
- - Say in that same sentence when the standard governs an attribute rather than a document type. An attribute is written across every document and has no path to name, so the silence reads as an omission unless the sentence claims it.
55
+ - Say in that same sentence when the reference governs an attribute rather than a document type. An attribute is written across every document and has no path to name, so the silence reads as an omission unless the sentence claims it.
44
56
  - Cut an entry that names no owner at all. It is either excluding something nothing was going to claim, or it is a content exclusion, which the rule below sends to the shape rules instead.
45
- - Declare a boundary from both sides. A yield, an exemption, or a handoff stated in one standard alone is never checked against the standard on the other side of it, which is how two files come to claim the same rule or neither does.
46
- - Separate a jurisdiction exclusion from a content exclusion. The first names a concern another standard owns and belongs in `## Scope`. The second names what does not belong inside the document and stays with the shape rules. Merging them puts a boundary claim where no sibling will read it.
47
- - Stay silent on a section the standard holds today but should not own. Claiming it makes the scope statement false the moment it moves, and the mismatch is the evidence that moves it.
57
+ - Declare a boundary from both sides. A yield, an exemption, or a handoff stated in one reference alone is never checked against the reference on the other side of it, which is how two files come to claim the same rule or neither does.
58
+ - Separate a jurisdiction exclusion from a content exclusion. The first names a concern another reference owns and belongs in `## Scope`. The second names what does not belong inside the document and stays with the shape rules. Merging them puts a boundary claim where no sibling will read it.
59
+ - Stay silent on a section the reference holds today but should not own. Claiming it makes the scope statement false the moment it moves, and the mismatch is the evidence that moves it.
48
60
 
49
61
  ### Naming the file
50
62
 
51
63
  - Name the file for the artifact the scope statement governs, taking the word from the deepest path segment that names the artifact rather than the folder holding it. `plan.md` over `.canon/plans/feature-<slug>.md` and `session.md` over `.canon/tasks/session-<slug>.md` both follow it.
52
- - Take the singular or the plural of that word, whichever reads as the artifact. A standard over one document takes the singular, and one over a set may take either.
53
- - Name an attribute standard for the attribute itself, since it governs no path to derive a word from.
54
- - Write the derived word alone. A standard installs as a sibling and is cited by bare filename, so a qualifier makes an outlier that every later citation has to carry.
55
- - Rename an outlier at the moment it is found, and state the reach in the change that does it. The name is installed in every target and cited in surfaces that never see the rename, so the cost only grows.
56
64
 
57
65
  ### Staying inside it
58
66
 
59
- - Name no path, filename, or folder outside the document type the standard governs. A standard reaches projects whose layout is their own, so a path borrowed from the authoring repository is wrong in a target and nothing reports it.
67
+ - Name no path, filename, or folder outside the document type the reference governs. A reference reaches projects whose layout is their own, so a path borrowed from the authoring repository is wrong in a target and nothing reports it.
60
68
  - State the rule, never the mechanism enforcing it. Hooks, scripts, checks, and skill catalogs are facts about one repository. Name the condition the document must meet and let the enforcing surface name its own case.
61
- - Invent inline examples rather than citing a real file elsewhere in the project. A cited file moves or is deleted and the standard goes stale in silence.
69
+ - Invent inline examples rather than citing a real file elsewhere in the project. A cited file moves or is deleted and the reference goes stale in silence.
62
70
 
63
71
  ## Frontmatter
64
72
 
65
73
  - Start the file with a frontmatter block carrying at least `title` and `description`. A consuming surface may require more.
66
74
  - `title`: names the doc type in sentence case, suffixed `reference` (`Commit reference`, `Branch reference`)
67
- - `description`: one line naming what the standard covers. It becomes the index link label on install.
75
+ - `description`: one line naming what the reference covers
68
76
 
69
77
  ## Structure
70
78
 
71
79
  - Use sentence case for every heading
72
- - Flat `##` rule groups for a single-topic standard. Group `##` headers by concern for a multi-topic one.
80
+ - Flat `##` rule groups for a single-topic reference. Group `##` headers by concern for a multi-topic one.
73
81
  - When one `##` section covers more than one sub-concern, split its bullets into `###` subgroups, one subgroup per sub-concern. A flat bullet list under an `##` covers a single sub-concern. Roughly seven bullets is a signal to split, not a hard cap.
74
82
  - Order groups from the most-used rule down to the edge case
75
83
 
@@ -83,13 +91,12 @@ A standard failing these questions is non-conforming even when it satisfies ever
83
91
 
84
92
  ## Success criterion
85
93
 
86
- - State what a conforming artifact achieves, not only what shape it takes. A standard that specifies structure exhaustively and success nowhere cannot be argued against, only edited on taste.
94
+ - State what a conforming artifact achieves, not only what shape it takes. A reference that specifies structure exhaustively and success nowhere cannot be argued against, only edited on taste.
87
95
  - Write the criterion as a small set of questions the artifact must answer, or a task a reader must be able to complete from it. Keep it checkable by a person in one sitting.
88
96
  - Place it near the top, above the shape rules it governs. The shape rules are the means and the criterion is the test.
89
97
  - Say that an artifact failing the criterion is non-conforming even when it satisfies every shape rule. Without that line the criterion reads as advice.
90
- - Add a criterion to an existing standard when that standard is next exercised, not in a sweep. A criterion written without a failure to point at is the taste-based edit this section exists to prevent, so a standard with no criterion yet is a known gap rather than a violation.
91
98
 
92
- ## Changing a standard
99
+ ## Changing a reference
93
100
 
94
101
  ### The checkpoint
95
102
 
@@ -98,12 +105,12 @@ Answer all four before editing, and carry the answers into wherever the change i
98
105
  - What problem does this solve? Name the artifact that went wrong, rather than the improvement the change makes.
99
106
  - Which surface owns the rule today? A rule already stated somewhere moves or is cited, never restated in a second place.
100
107
  - What deterministic check catches a regression? Name it, or say none exists and the rule holds on reading alone.
101
- - What does this collide with? Name the sibling standard, rule, or template it contradicts, or state that nothing does.
108
+ - What does this collide with? Name the sibling reference, rule, or template it contradicts, or state that nothing does.
102
109
 
103
110
  ### What justifies a change
104
111
 
105
- - Change a standard on a failure, not on a finding. A finding is that the docs say X or a paper suggests Y. A failure is a conforming artifact that satisfied every shape rule and still missed the success criterion.
106
- - Park findings wherever the project tracks pending work, or in the standard's own backlog section when it tracks none. They are hypotheses to test, not instructions to apply.
112
+ - Change a reference on a failure, not on a finding. A finding is that the docs say X or a paper suggests Y. A failure is a conforming artifact that satisfied every shape rule and still missed the success criterion.
113
+ - Park findings wherever the project tracks pending work, or in the reference's own backlog section when it tracks none. They are hypotheses to test, not instructions to apply.
107
114
  - Cite the failing artifact in the change that fixes it, so the next reader can tell which rules were paid for by evidence.
108
115
 
109
116
  ## Examples
@@ -115,20 +122,19 @@ Answer all four before editing, and carry the answers into wherever the change i
115
122
 
116
123
  ## Template
117
124
 
118
- - Carry one fenced template of the document the standard governs, so an author copies a shape instead of reconstructing it from prose.
119
- - Bind this to a standard governing a document type. An attribute standard governs a scan, a string, or a label rather than a file, so it has no document to template and carries none.
120
- - State that exemption in the sentence declaring the standard an attribute standard. An exemption left to inference reads as an omission, and the next author writes a template that teaches nothing.
121
- - Place the template below the rules it satisfies, under a `## Template` heading or inside the section holding those rules. A reader arrives at it having already read what each part means.
125
+ - Carry one fenced template of the document the reference governs, so an author copies a shape instead of reconstructing it from prose.
126
+ - Bind this to a reference governing a document type. An attribute reference governs a scan, a string, or a label rather than a file, so it has no document to template and carries none. State that exemption in the sentence declaring it an attribute reference.
127
+ - Place the template below the rules it satisfies, under a `## Template` heading or inside the section holding those rules.
122
128
  - Show the skeleton alone: every required section, one placeholder line for what each holds. A full worked document is a second document to maintain.
123
- - Write placeholders in angle brackets rather than copying a real document. A copied file goes stale, and its backticked filenames read as citations to anything parsing the standard.
124
- - Switch to a bare word where the fenced language reads `<` as syntax of its own, as Mermaid and HTML do, and say in the line above the fence that the names are placeholders. A template that does not parse in its own language teaches a shape the author cannot run.
125
- - Carry any header, key, or label a consuming tool parses verbatim. The template is the only place an author learns which strings are load-bearing, since prose calling them fixed does not say what they are.
126
- - Treat the template as authoritative where it and the prose disagree. An author copies the block, so a contradiction ships as the block, and naming the winner turns a drift into a visible defect rather than a silent one.
129
+ - Write placeholders in angle brackets rather than copying a real document. A copied file goes stale, and its backticked filenames read as citations to anything parsing the reference.
130
+ - Switch to a bare word where the fenced language reads `<` as syntax of its own, as Mermaid and HTML do, and say in the line above the fence that the names are placeholders.
131
+ - Carry any header, key, or label a consuming tool parses verbatim. The template is the only place an author learns which strings are load-bearing.
132
+ - Treat the template as authoritative where it and the prose disagree. An author copies the block, so a contradiction ships as the block.
127
133
 
128
134
  ````markdown
129
135
  ---
130
136
  title: <Document type> reference
131
- description: <one line naming what this standard covers>
137
+ description: <one line naming what this reference covers>
132
138
  ---
133
139
 
134
140
  # <Document type> reference
@@ -137,7 +143,7 @@ Applies to `<path/to/document>`. <One line on what the document is for and when
137
143
 
138
144
  ## Scope
139
145
 
140
- Governs `<path/to/document>`: <the aspects this standard sets>.
146
+ Governs `<path/to/document>`: <the aspects this reference sets>.
141
147
 
142
148
  Does not govern:
143
149
 
@@ -160,7 +166,6 @@ A <document type> failing these is non-conforming even when it satisfies every s
160
166
 
161
167
  ## <Shape rule group>
162
168
 
163
- - <imperative rule, one concern per bullet>
164
169
  - <imperative rule, one concern per bullet>
165
170
 
166
171
  ## Template
@@ -32,7 +32,8 @@ Does not govern:
32
32
 
33
33
  - What a `REQUIREMENT.md` must answer, its sections, and its template: `skill-requirement.md`
34
34
  - A practice skill's closing sections and ledger: `skill-practice.md`
35
- - Which root a path in a body resolves against, and how a body cites a standard: `skill-paths.md`
35
+ - Which root a path in a body resolves against, and how a body cites another skill's reference: `skill-paths.md`
36
+ - The jurisdiction, success criterion, and rule phrasing of a reference setting one artifact's shape: `skill-reference.md`
36
37
  - Path-scoped coding rules, which load on a file match rather than on a request match: `rule.md`
37
38
  - Voice, rhythm, and sentence construction in a skill body: the `write-human` skill
38
39
  - Punctuation, formatting, and word choice in a skill body: `markdown.md`
@@ -160,9 +161,14 @@ allowed-tools: <tools required>
160
161
  ## Scripts
161
162
 
162
163
  - Use `scripts/` for operations that must be deterministic or repetitive
164
+ - Ask whether a CLI verb the project already has does the job before writing a script. A verb is already installed and tested, and a script beside it is a second copy that drifts
165
+ - Write a script in TypeScript and run it as `bun ${CLAUDE_SKILL_DIR}/scripts/<name>.ts`. Keep bash for a few lines that start one command. The rule governs a new script, so bash kept by decision is no breach and is not ported unasked, such as `review-ui`'s `scripts/pw.sh` as glue and the shell scripts the web tooling stacks ship, whose port was deferred
166
+ - Import no package. A plugin loads from a cache that carries no `node_modules`, so only `node:` built-ins and the CLIs the skill already requires resolve
163
167
  - Use XML tags in script output for reliable parsing: `<SECTION>content</SECTION>`
164
- - Use `#!/usr/bin/env bash` shebang
165
- - Always include `2>/dev/null || echo "FALLBACK"` guards on git and shell commands
168
+
169
+ ### Inline commands
170
+
171
+ - Always include `2>/dev/null || echo "FALLBACK"` guards on git and shell commands a skill body runs inline. A TypeScript script handles failure with try and catch instead
166
172
 
167
173
  ## Invocation
168
174
 
@@ -27,7 +27,7 @@ A session handed either one unfiltered adopts a stack the project does not use,
27
27
  - Enumerate the layers in settle order, so an instruction to name one has a catalog behind it
28
28
  - State that a round varies one layer, and that a complaint is worked at the lowest layer it reaches rather than the cheapest one to change
29
29
  - Carry grey-boxing as the operational form of the ordering rule, since removing the higher layers from view is what makes a lower one judgeable
30
- - Cite the correctness floor rather than restating it. `400-ui`, `410-a11y`, `420-forms` and `430-ux-completeness` already carry accessibility, keyboard, forms, states and rendered copy on the rendered paths, and a second copy here drifts with nothing comparing the two.
30
+ - Cite the correctness floor rather than restating it. `copy`, `a11y`, `forms` and `ux-completeness` already carry accessibility, keyboard, forms, states and rendered copy on the rendered paths, and a second copy here drifts with nothing comparing the two.
31
31
  - Hold in the body what every design moment pays for, being the read, the layers, the ordering, grey-boxing and the locks, and defer everything a single moment needs.
32
32
  - State what has to stay constant across a whole surface, since coherence is a property of the whole and each of those is broken one section at a time.
33
33
  - Detect the mode before reading anything when a surface already exists, and name what cannot change silently, since a slug, a field name or an analytics event breaks something outside the surface.
@@ -66,11 +66,11 @@ A rule failing any of the four is recorded as considered and declined rather tha
66
66
 
67
67
  ## Out of scope
68
68
 
69
- - Accessibility, keyboard interaction, ARIA and alt text, which `410-a11y` owns on every rendered path
70
- - State coverage, empty and loading treatment, destructive confirmation and truncation, which `430-ux-completeness` owns
71
- - Validation timing, error placement and save blocking, which `420-forms` owns
72
- - Rendered copy casing, button labels and error wording, which `400-ui` owns
69
+ - Accessibility, keyboard interaction, ARIA and alt text, which `a11y` owns on every rendered path
70
+ - State coverage, empty and loading treatment, destructive confirmation and truncation, which `ux-completeness` owns
71
+ - Validation timing, error placement and save blocking, which `forms` owns
72
+ - Rendered copy casing, button labels and error wording, which `copy` owns
73
73
  - The candidate loop, the arm set, the render and the pick, which `draft-and-pick` owns and which this skill is loaded alongside rather than inside
74
- - The shape of the project's design document, its sections and its token tables, which `claude/skills/canvas-board/references/design.md` governs
74
+ - The shape of the project's design document, its sections and its role tables, which `claude/skills/canvas-board/references/design.md` governs
75
75
  - Prose on the surface being designed, which `write-human` and `markdown.md` govern between them
76
76
  - Measuring whether finished output followed these rules, which needs a check this skill does not carry and which no command reaches while a render is an image rather than a parse
@@ -113,7 +113,7 @@ Never change these without saying so and getting agreement, because each one bre
113
113
 
114
114
  Read `${CLAUDE_SKILL_DIR}/references/systems.md` when establishing or revising what a surface draws from, being the space scale, the type scale, and the relationships a palette holds. Every other reference here is read against a draft. That one is read before there is one, so a project reaches for it at the start and on a system revision rather than on an ordinary round.
115
115
 
116
- It states no values. A value belongs to the project's own design document.
116
+ It states no values. A value belongs to the project's own stylesheet, and its intent to the design document.
117
117
 
118
118
  ## The rest of the catalog
119
119
 
@@ -145,6 +145,6 @@ Run `${CLAUDE_SKILL_DIR}/references/preflight.md` against the output. It is the
145
145
  - States, empty and loading coverage, destructive confirmation, truncation: the same rules
146
146
  - Rendered copy casing, button labels, error message shape: the same rules
147
147
  - The candidate loop, the arm set, the render, and the pick: `draft-and-pick`
148
- - The shape of the project's design document and its token tables: `${CLAUDE_SKILL_DIR}/../canvas-board/references/design.md`
148
+ - The shape of the project's design document and where its values live: `${CLAUDE_SKILL_DIR}/../canvas-board/references/design.md`
149
149
  - Reading source for roughness: `ux-audit`. Measuring what a running interface costs to paint: `ux-measure`
150
150
  - Prose on the surface being designed: the `write-human` skill
@@ -39,7 +39,7 @@ Reaching for size is the first instinct and the weakest tool.
39
39
  - **Measure.** Body text running the full width of a wide container is unreadable however good the type is.
40
40
  - **Figures in columns.** Numbers that change, in a table, a price or a timer, need tabular figures or the column jitters.
41
41
  - **Tracking by size.** Large type wants tighter letter spacing, small uppercase labels want looser. Body text wants whatever the face was drawn with.
42
- - **Casing.** Sentence case for headings and labels. `465-interface-casing` states the rule and its exceptions.
42
+ - **Casing.** Sentence case for headings and labels. `interface-casing` states the rule and its exceptions.
43
43
 
44
44
  ## Depth and texture
45
45
 
@@ -28,7 +28,7 @@ Skip nothing on the ground that it is obviously fine. A rule is on this list bec
28
28
 
29
29
  - [ ] No layout family used twice. Eight sections carry at least four families
30
30
  - [ ] No three consecutive sections sharing the image-and-text split
31
- - [ ] Small uppercase labels above headings number no more than one per three sections, per `465-interface-casing`
31
+ - [ ] Small uppercase labels above headings number no more than one per three sections, per `interface-casing`
32
32
  - [ ] At most one horizontal marquee
33
33
  - [ ] Grid cells equal content items, with no blank tile filling a gap
34
34
  - [ ] Lists over five items use something other than a longer list
@@ -62,7 +62,7 @@ Skip this group where the surface draws on a system already settled.
62
62
  - [ ] Every number either real, labeled as sample, or removed
63
63
  - [ ] No placeholder that reads as placeholder: generic names, generic company names, reused portraits, identical dates
64
64
  - [ ] No filler verbs
65
- - [ ] Sentence case throughout, with exceptions per `465-interface-casing`
65
+ - [ ] Sentence case throughout, with exceptions per `interface-casing`
66
66
  - [ ] Every claim about what the product does is a thing it does
67
67
 
68
68
  ## Motion
@@ -61,7 +61,7 @@ These are the rules with numbers, so they are checked by counting rather than by
61
61
 
62
62
  - **A layout family appears once.** Once a section uses a shape, no other section on the surface uses it. Eight sections need at least four distinct families.
63
63
  - **Two consecutive splits, never three.** Alternating image-left and image-right reads as considered twice and as filler by the third. Break it with a full-width section, a vertical stack, a grid, or anything else.
64
- - **One small uppercase label per three sections.** The tiny wide-tracked line above a heading is the single most over-reached default in model output. `465-interface-casing` carries the count. The fix is usually to delete it, since a section's position already says what it is.
64
+ - **One small uppercase label per three sections.** The tiny wide-tracked line above a heading is the single most over-reached default in model output. `interface-casing` carries the count. The fix is usually to delete it, since a section's position already says what it is.
65
65
  - **One marquee.** Two sliding strips on one surface read as filler rather than as a device.
66
66
  - **Cells equal items.** A grid with a hole in it, or a blank tile at the end, means the grid was shaped before the content was counted. Reshape it.
67
67
  - **Over five items wants a different component.** A longer list is the lazy answer. Group into clusters, split into columns, give each item a card, make them scrollable, or collapse the tail behind a disclosure.
@@ -9,7 +9,7 @@ metadata:
9
9
 
10
10
  Some decisions are settled by looking rather than by reasoning, and no draft is wrong until one is picked. Every step here puts several candidates in front of the operator and keeps the real surface untouched until they have chosen.
11
11
 
12
- The candidates live on the local design canvas, the surface `canon:canvas-board` drives. Every verb below is `canon canvas`, and `canon standards canvas` fixes the content format a frame takes. Read that standard before writing the first frame.
12
+ The candidates live on the local design canvas, the surface `canon:canvas-board` drives. Every verb below is `canon canvas`, and `${CLAUDE_SKILL_DIR}/../canvas-board/references/canvas.md` fixes the content format a frame takes. Read that reference before writing the first frame.
13
13
 
14
14
  ## Guards
15
15
 
@@ -18,11 +18,12 @@ One identity rendered twice: the same mark sized down to an icon sequence and co
18
18
 
19
19
  Read these in parallel, skipping any that do not exist:
20
20
 
21
- - `canon/DESIGN.md`: the `## Personality`, `## Color`, and `## Typography` sections
21
+ - `canon/DESIGN.md`: the `## Overview` paragraph and the role intents under `## Colors` and `## Typography`, read under their earlier names `## Personality` and `## Color` in an older record
22
+ - The stylesheet `canon/DESIGN.md`'s lead names: the color and type values behind those roles
22
23
  - `canon/REQUIREMENTS.md`: the `## Personality` paragraph, when `canon/DESIGN.md` carries none
23
24
  - `CLAUDE.md`: the project's stated voice, when neither file above carries a personality signal
24
25
 
25
- No signal from any of the three is not a stop. Draft against a neutral default and tag the color and type choices `? verify` per `${CLAUDE_SKILL_DIR}/../canvas-board/references/design.md`'s uncertainty tag.
26
+ No signal from any of these is not a stop. Draft against a neutral default, and mark each guessed color and type value with a `/* ? verify */` comment beside its declaration in the `.mark` and `.card` markup this run writes, since the design document carries no value to tag.
26
27
 
27
28
  ## Step 2: detect the icon size sequence
28
29
 
@@ -39,19 +39,19 @@ External release identity used in git tags and release notes. Independent of pha
39
39
 
40
40
  ## Where each appears
41
41
 
42
- | Surface | Phase labels | Semver tags |
43
- | ------------------------------------------------------------------- | ------------ | ----------------------------------- |
44
- | `.canon/tasks/` | yes | no |
45
- | Chat with the operator | yes | no |
46
- | Tracked prose (context entries) | conditional | no |
47
- | Shipped prose (`standards/`, `claude/skills/`, `governance/rules/`) | no | no |
48
- | PR titles | no | only when the PR cuts a release |
49
- | PR bodies | no | only when the PR cuts a release |
50
- | Review comments | no | only when referencing a release |
51
- | Issue titles and bodies | no | only when referencing a release |
52
- | Commit messages | no | only when the commit cuts a release |
53
- | Git tags | no | yes |
54
- | README and `CHANGELOG.md` | no | yes |
42
+ | Surface | Phase labels | Semver tags |
43
+ | ----------------------------------------------------- | ------------ | ----------------------------------- |
44
+ | `.canon/tasks/` | yes | no |
45
+ | Chat with the operator | yes | no |
46
+ | Tracked prose (context entries) | conditional | no |
47
+ | Shipped prose (`claude/skills/`, `governance/rules/`) | no | no |
48
+ | PR titles | no | only when the PR cuts a release |
49
+ | PR bodies | no | only when the PR cuts a release |
50
+ | Review comments | no | only when referencing a release |
51
+ | Issue titles and bodies | no | only when referencing a release |
52
+ | Commit messages | no | only when the commit cuts a release |
53
+ | Git tags | no | yes |
54
+ | README and `CHANGELOG.md` | no | yes |
55
55
 
56
56
  ## Rules
57
57
 
@@ -87,7 +87,7 @@ canon labels scan --title "<title>" --body-file .canon/tmp/pr/body.md --json
87
87
 
88
88
  Branch on the JSON record rather than the exit code. An operator's shell can wrap `canon` in a function whose status comes from a trailing command, flattening a non-zero exit to 0, the same reason the labels step below branches on its own record rather than the exit.
89
89
 
90
- Stop and fix the title or body on a non-empty `phaseLabels`, `boardReferences`, `sessionLinks`, `unspelledWords`, `titleFormatIssues`, `bannedCharacters`, or a `sections` record with anything in `outside` or `missing`. Do not proceed to `### Final command` until a re-run comes back clean on all seven. `titleFormatIssues` names which `## Title` rule in `${CLAUDE_SKILL_DIR}/references/pr.md` the title breaks, structure, casing, or length, so fix the named rule rather than guessing. `sections.outside` lists each heading to rename or drop, `sections.missing` each required heading to add, and `bannedCharacters` names the character to restructure the sentence around. Leave `cutsRelease` and `semverTags` alone, since a release-please pull request legitimately carries version references its own fixed shape explains.
90
+ Stop and fix the title or body on a non-empty `phaseLabels`, `boardReferences`, `sessionLinks`, `titleFormatIssues`, `bannedCharacters`, or a `sections` record with anything in `outside` or `missing`. Do not proceed to `### Final command` until a re-run comes back clean on all six. `titleFormatIssues` names which `## Title` rule in `${CLAUDE_SKILL_DIR}/references/pr.md` the title breaks, structure, casing, or length, so fix the named rule rather than guessing. `sections.outside` lists each heading to rename or drop, `sections.missing` each required heading to add, and `bannedCharacters` names the character to restructure the sentence around. Leave `cutsRelease` and `semverTags` alone, since a release-please pull request legitimately carries version references its own fixed shape explains.
91
91
 
92
92
  The scan stays required even where a `pull_request` workflow job runs the same verb, since a project holding an older `canon` carries no such job and no `titleFormatIssues`, `sections`, or `bannedCharacters` key.
93
93
 
@@ -66,7 +66,7 @@ A number referring to a pull request or an issue takes the form its destination
66
66
 
67
67
  Text published to a remote is the auto-linking half: a pull request body, an issue body, a comment on either, and a commit message. Write `#123` there, and `owner/repo#123` where the reference crosses repositories, which the same auto-linking reaches. A markdown file browsed from the repository tree is the other half, where neither spelling links and the backticked one marks the number as an identifier. <!-- canon-allow-reference: the two spellings this section defines, which a qualified form would destroy -->
68
68
 
69
- Content that installs into another repository is the third destination, and it splits on which repository the number names. A skill body loads from a plugin cache, a docs page is read through `canon docs`, and a standard through `canon standards`, so the reader holds their own repository rather than the one a bare number belongs to. Qualifying a number against a different, resolvable repository fixes that, since the reader can open that repository even without this one.
69
+ Content that installs into another repository is the third destination, and it splits on which repository the number names. A skill body loads from a plugin cache and a docs page is read through `canon docs`, so the reader holds their own repository rather than the one a bare number belongs to. Qualifying a number against a different, resolvable repository fixes that, since the reader can open that repository even without this one.
70
70
 
71
71
  Qualifying it against `erclx/canon` fixes nothing, because a reader holding only the plugin cache or the published package still cannot open this repository's own history. A same-repository citation takes no number here at all, bare or qualified. State the fact the citation was standing in for instead, and where the identifier is the only trace of a one-off incident rather than a reproducible count, carry it into the `canon/context/` entry that owns the surface.
72
72
 
@@ -42,7 +42,7 @@ Does not govern:
42
42
  ## Headings
43
43
 
44
44
  - H1 for document title, H2 for main sections, H3 for subsections
45
- - Use sentence case for all headings (H1, H2, H3)
45
+ - Use sentence case for all headings (H1, H2, H3). The audit reports a heading or a frontmatter `title` with a capitalized word after the first, and a proper noun it reports stays as it is.
46
46
  - Proper nouns and product names retain their casing in headings
47
47
  - Past roughly 40 rendered lines with no heading of any level breaking them, add a subheading at the seam. Measure the longest such run rather than everything under one `##`, and exclude fenced code blocks. The number is a checkpoint, not a cap.
48
48
  - Count rendered lines rather than source lines, wrapping each source line at 80 columns and summing the heights. Source lines undercount a file authored one line per bullet, where a block of fifteen paragraph-bullets occupies fifteen lines and renders past sixty. A checkpoint another standard states counts the same unit, so a file measured one way never sits beside a run measured another.
@@ -80,7 +80,8 @@ Does not govern:
80
80
 
81
81
  ## Punctuation
82
82
 
83
- - Do not use em dashes (`—`) or semicolons (`;`). Rewrite or restructure the sentence to avoid them.
83
+ - Do not use em dashes (`—`) or semicolons (`;`). Rewrite or restructure the sentence to avoid them. A colon or a comma standing where the dash was is the same sentence, so split it or rewrite it.
84
+ - Do not use a colon to join two clauses in the middle of a sentence. A colon belongs after a label or at the end of a line introducing a list.
84
85
  - Do not use parenthetical asides in prose (`the config (which is optional) controls...`). Split into its own sentence or drop it. Parentheses in rule definitions for grouping examples are fine.
85
86
 
86
87
  ## Language
@@ -24,7 +24,7 @@ Read these in parallel from the project root, skipping any that do not exist:
24
24
 
25
25
  Also read these when the feature touches code or UI. Skip them for prose, docs, catalog, or config-only changes:
26
26
 
27
- - `canon/DESIGN.md`: tokens, typography, spacing, and component rules
27
+ - `canon/DESIGN.md`: visual intent and component rules, with values read from the stylesheet its lead names
28
28
 
29
29
  Coding standards live in `.claude/rules/`. Claude Code loads them automatically. Path-scoped rules apply to the files they match.
30
30
 
@@ -55,5 +55,5 @@ A declined finding fails on a third axis, which is where its reason ends up. A w
55
55
  - Writing the review, which `review-pr` owns. The split is by side of the channel: that one posts findings from an independent session and this one is the worker's return leg.
56
56
  - Staging, committing, and pushing the follow-up, which `git-followup` owns under this skill's direction
57
57
  - Refreshing the `.claude/` docs the fixes made stale, which `context-fold` owns
58
- - Re-reviewing its own fixes, which hands back to the orchestrator
58
+ - Re-reviewing its own fixes, which hands back to the reviewer
59
59
  - Re-reading a rewritten branch, which `review-pr` absorbs by testing whether the prior reviewed commit still reaches the head and paying for a full pass when it does not
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  name: review-address
3
- description: Pulls review findings and CI status on the current branch's open PR, fixes each in the working tree, refreshes any stale `.claude/` docs, replies with a summary comment, and pushes a follow-up commit. The worker's return leg after `review-pr`. Use when asked to "address the review", "fix the PR comments", "respond to review", or after an orchestrator posts findings. Do NOT use to write a review. That is `review-pr`.
3
+ description: Pulls review findings and CI status on the current branch's open PR, fixes each in the working tree, refreshes any stale `.claude/` docs, replies with a summary comment, and pushes a follow-up commit. The worker's return leg after `review-pr`. Use when asked to "address the review", "fix the PR comments", "respond to review", or after a reviewer posts findings. Do NOT use to write a review. That is `review-pr`.
4
4
  metadata:
5
5
  family: after-pr
6
6
  ---
@@ -247,7 +247,7 @@ addressed, and leads with the rebase instead:
247
247
  Rebased PR #<number> onto origin/main. <N> files resolved by hand, <N> rebuilt by the check.
248
248
  ```
249
249
 
250
- Do not merge. Hand back to the orchestrator for re-review.
250
+ Do not merge. Hand back to the reviewer for re-review, or to the orchestrator where no live reviewer answers.
251
251
 
252
252
  ## Post-review findings
253
253
 
@@ -59,7 +59,7 @@ A second theme earns a read only when the diff touches a color token. Name the s
59
59
 
60
60
  ## Comparing
61
61
 
62
- Check each opened image against the layout its checklist box claims, and check `canon/DESIGN.md` for the tokens the image makes visible: color, type scale, spacing, and component rules.
62
+ Check each opened image against the layout its checklist box claims, and check the tokens the image makes visible, being color, type scale, and spacing, against the stylesheet `canon/DESIGN.md`'s lead names, and its component rules against the record itself.
63
63
 
64
64
  Read the width and theme from whatever the image path carries, and fix no width list, since which widths matter belongs to the project. Skip the design file comparison silently when the project has none.
65
65
 
@@ -1,13 +1,13 @@
1
1
  ---
2
2
  name: review-pr
3
- description: Reviews an open pull request from an independent session and posts findings as a review comment on the PR. Posts a first pass against the whole change and every later pass against only the commits added since, under `## Review` while any finding is open and `## Review closed` once a pass carries none. Reads project docs and the task board for cross-feature context a self-review lacks. Use when asked to "review the PR", "review this feature's PR", "post a PR review", "re-review the PR", "close out the review", "confirm the findings are fixed", or acting as the orchestrator reviewing a worker's PR. Do NOT use to review local uncommitted changes.
3
+ description: Reviews an open pull request from an independent session and posts findings as a review comment on the PR. Posts a first pass against the whole change and every later pass against only the commits added since, under `## Review` while any finding is open and `## Review closed` once a pass carries none. Reads project docs and the task board for cross-feature context a self-review lacks. Use when asked to "review the PR", "review this feature's PR", "post a PR review", "re-review the PR", "close out the review", "confirm the findings are fixed", or acting as the dispatched reviewer of a worker's PR. Do NOT use to review local uncommitted changes.
4
4
  metadata:
5
5
  family: after-pr
6
6
  ---
7
7
 
8
8
  # Review PR
9
9
 
10
- This is the orchestrator's independent review. It reviews an open PR the session
10
+ This is a dispatched reviewer's independent review. It reviews an open PR the session
11
11
  did not write and posts the findings to the PR, so the vantage is independent
12
12
  and the output is durable.
13
13
 
@@ -19,7 +19,7 @@ none, so the most recent comment's heading reports whether any work is owed.
19
19
  Every pass is this skill, and which one it is gets detected from the thread
20
20
  rather than named by the caller.
21
21
 
22
- The threshold is stated here and nowhere else, and every other surface acting on it cites this skill rather than restating the grades. One rule governs both the heading and the dispatch: a pass carrying anything owed takes `## Review` and owes a dispatch to the session holding the branch, and a pass carrying nothing at all takes `## Review closed` and owes none. Owed covers a finding at any severity, a Testing question, and a reviewer request nobody has answered alike, which is what keeps the two halves from separating. Sending that dispatch is `role-orchestrator`'s step rather than this one, which posts and stops. Post the open heading whether it is the first pass or the fourth. A pull request thread then reads as `## Review`, the worker's answer under `## Review response` from `review-address`, another `## Review` while anything stays open, and `## Review closed` when nothing does.
22
+ The threshold is stated here and nowhere else, and every other surface acting on it cites this skill rather than restating the grades. One rule governs both the heading and the dispatch: a pass carrying anything owed takes `## Review` and owes a dispatch to the session holding the branch, and a pass carrying nothing at all takes `## Review closed` and owes none. Owed covers a finding at any severity, a Testing question, and a reviewer request nobody has answered alike, which is what keeps the two halves from separating. Sending that dispatch is `role-reviewer`'s step rather than this one, which posts and stops, and `role-orchestrator` sends it only as the fallback where no live reviewer can. Post the open heading whether it is the first pass or the fourth. A pull request thread then reads as `## Review`, the worker's answer under `## Review response` from `review-address`, another `## Review` while anything stays open, and `## Review closed` when nothing does.
23
23
 
24
24
  Both of this skill's headings anchor as a section distinct from human threads, and neither invents beyond what the whole set already states. That set is eight headings across four families, stated here once so `role-orchestrator`'s poll and every reply-posting skill cite it rather than carry a copy. The review family, `## Review` and `## Review closed`, belongs to this skill alone. The reply family, `## Review response`, `## Rebase`, and `## Post-review findings`, belongs to `review-address`. The third family is a single heading, `## Evidence`, which `canon pr evidence` posts through `git-pr` and `git-followup` and which answers no comment already on the thread, so the poll excludes it from the unclassified count without reading it as a reply that owes a re-review. The fourth, `## UI review` and `## UI review closed`, belongs to `review-ui`, which posts it as a review beside this one, and nothing in this skill reads it.
25
25
 
@@ -56,7 +56,7 @@ X critical, Y should-fix, Z minor. Reviewed against project docs and the board.
56
56
 
57
57
  A stale ticked box goes in a `**PR body**` block, in place of a `**`path/to/file.ext`**` block and ahead of every one of those, since it precedes the code the diff carries rather than sitting inside it.
58
58
 
59
- Run `canon labels scan --title "<title>" --body "<body>" --head <headRefName>` against the PR under review, since Step 1 already holds all three and this pass is the last human-shaped gate before merge. A hit lands in the same `**PR body**` block, `should-fix`, naming each finding the scan returns: a token quoted from the title or body for a phase label, a board identifier, a session link, or an unspelled word, and the broken rule's name, `structure`, a casing issue, or `length`, for a title-format hit. A heading in `sections.outside`, a heading in `sections.missing`, or a character in `bannedCharacters` is a hit of the same weight, naming the heading or the character. This reads the pull request being reviewed, distinct from the comment this pass is about to post, which the scan later in this step still covers.
59
+ Run `canon labels scan --title "<title>" --body "<body>" --head <headRefName>` against the PR under review, since Step 1 already holds all three and this pass is the last human-shaped gate before merge. A hit lands in the same `**PR body**` block, `should-fix`, naming each finding the scan returns: a token quoted from the title or body for a phase label, a board identifier, or a session link, and the broken rule's name, `structure`, a casing issue, or `length`, for a title-format hit. A heading in `sections.outside`, a heading in `sections.missing`, or a character in `bannedCharacters` is a hit of the same weight, naming the heading or the character. This reads the pull request being reviewed, distinct from the comment this pass is about to post, which the scan later in this step still covers.
60
60
 
61
61
  A later pass carrying findings keeps that shape and changes only the summary line:
62
62
 
@@ -54,7 +54,7 @@ A verdict also goes stale without saying so. A push after the pass changes what
54
54
 
55
55
  - Reviewing the code and the stills, which `review-pr` owns. The two run as separate sessions so each finishes on its own clock and neither hears the other.
56
56
  - Writing the checklist and its drivable format, which `ui-checklist` owns
57
- - Dispatching the pass, polling its verdict, and lifting the draft mark, which `role-orchestrator` owns
57
+ - Dispatching the pass and polling its verdict, which `role-orchestrator` owns, and the lift's gates, which `canon pr lift` reads whichever session runs it
58
58
  - A subagent inside the code reviewer, dropped because its cost stays invisible until it returns
59
59
  - Writing a tracked file or checking anything out to host a frame. The push goes through the GitHub API onto the frames branch alone, so the reviewer still enters no worktree and touches no branch a pull request holds.
60
60
  - Dropping a closed pull request's frames, which a scheduled workflow runs, and purging a dropped frame GitHub still serves by commit, which needs a platform request