@erclx/canon 5.67.0 → 5.69.0

package/src/claude/cases/workflow.ts

@@ -143,11 +143,11 @@ export const WORKFLOW_CASES: readonly SkillCase[] = [
   {
     prompt:
       'Look over the interface and tell me what feels unfinished or confusing.',
-    expect: 'ux-audit',
+    expect: 'ui-audit',
   },
   {
     prompt: 'Tell me the paint and layout cost of this page right now.',
-    expect: 'ux-measure',
+    expect: 'ui-measure',
   },
   {
     prompt:
@@ -162,12 +162,12 @@ export const WORKFLOW_CASES: readonly SkillCase[] = [
   {
     prompt:
       'I need real footage of the board import flow for the launch video. Film it from our app so HyperFrames can zoom in on the clicks.',
-    expect: 'ui-record',
+    expect: 'ui-film',
   },
   {
     prompt:
       'Shoot the takes for the storyboard we wrote, one clip per beat of the app doing its thing.',
-    expect: 'ui-record',
+    expect: 'ui-film',
   },
   {
     prompt:
@@ -216,10 +216,10 @@ export const WORKFLOW_CASES: readonly SkillCase[] = [
   },
   {
     prompt: 'Find the rough, unfinished-feeling spots in this interface.',
-    expect: 'ux-audit',
+    expect: 'ui-audit',
   },
   {
     prompt: 'Tell me the render cost of this page in the browser.',
-    expect: 'ux-measure',
+    expect: 'ui-measure',
   },
 ]

package/src/commands/fixtures/gh.sh

@@ -0,0 +1,64 @@
+#!/usr/bin/env bash
+# Fake gh for src/commands/pr.test.ts. Reads FAKE_GH_FX, FAKE_GH_COMMENTS, and FAKE_GH_API_LOG.
+set -o pipefail
+fx="$FAKE_GH_FX"
+f=.
+p=
+for a in "$@"; do
+  [ "$p" = --jq ] && f=$a
+  p=$a
+done
+case "$*" in
+*graphql* | "pr "* | "repo view"*) exit 1 ;;
+api*pulls/7/files*)
+  echo "$*" >>"$fx/files-calls.log"
+  [ -f "$fx/files-fail" ] && exit 1
+  {
+    if [ -f "$fx/files.tsv" ]; then
+      cat "$fx/files.tsv"
+    else
+      git diff --name-status --no-renames main...HEAD | sed -e 's/^A/added/' -e 's/^M/modified/' -e 's/^D/removed/'
+    fi
+  } | jq -Rn '[inputs | select(length > 0) | split("\t") | {status: .[0], filename: .[1]}]' | jq -r "$f"
+  ;;
+api*git/trees/*)
+  [ -f "$fx/base-tree.json" ] || exit 1
+  jq -r "$f" "$fx/base-tree.json"
+  ;;
+api*contents/*)
+  a="$*"
+  p=${a##*contents/}
+  p=${p%%\?*}
+  r=${a##*ref=}
+  [ -f "$fx/img/$r-$(basename "$p")" ] && cat "$fx/img/$r-$(basename "$p")" || exit 1
+  ;;
+api*compare/*)
+  if [ -f "$fx/merge-base" ]; then cat "$fx/merge-base"; else git merge-base main HEAD; fi
+  ;;
+"api -X PUT repos/{owner}/{repo}/pulls/7/reviews/"*)
+  id=${4##*/}
+  echo "$id" >>"$fx/review-puts.log"
+  [ -f "$fx/review-put-fail-$id" ] && exit 1
+  printf '%s\n' "${@: -1}" >>"$fx/review-bodies.log"
+  echo "{}"
+  ;;
+api*pulls/7/reviews*) [ -f "$fx/ui-reviews.json" ] && jq -r "$f" "$fx/ui-reviews.json" ;;
+api*issues/7/comments*) jq '[.comments[] | {html_url: .url, body}]' "$FAKE_GH_COMMENTS" | jq -r "$f" ;;
+"api -X PATCH repos/{owner}/{repo}/pulls/7 "*)
+  [ -f "$fx/body-fail" ] && exit 1
+  printf '%s' "${@: -1}" >"$fx/body-write.log"
+  echo "{}"
+  ;;
+"api repos/{owner}/{repo}/pulls/7")
+  if [ -f "$fx/head" ]; then h=$(cat "$fx/head"); else h=$(git rev-parse HEAD); fi
+  b=
+  [ -f "$fx/pr-body" ] && b=$(cat "$fx/pr-body")
+  jq -n --arg h "$h" --arg b "$b" '{number:7,head:{ref:"feat/x",sha:$h},base:{ref:"main"},mergeable_state:"clean",body:$b}'
+  ;;
+"api repos/{owner}/{repo}") echo "{\"full_name\":\"o/r\"}" ;;
+api*)
+  printf '%s' "${@: -1}" >"$FAKE_GH_API_LOG"
+  echo "{}"
+  ;;
+*) exit 1 ;;
+esac

package/src/commands/gov.ts

@@ -34,6 +34,7 @@ import {
 } from '@/gov/restated'
 import {
   auditExit,
+  CORE_FOLDERS,
   hasFinding,
   PRACTICE_FOLDERS,
   readRuleAudit,
@@ -62,6 +63,11 @@ import {
   readTestOrder,
   type TestOrderReport,
 } from '@/gov/test-order'
+import {
+  type ChangedSet,
+  collectChangedFiles,
+  commandRunner,
+} from '@/gate/sequencer'
 import { recordStamp, runDomainSync } from '@/sync/engine'
 import { detectRules } from '@/gov/detect'
 import { writeChainStamp } from '@/sync/stamp'
@@ -138,6 +144,7 @@ interface AuditOptions {
   readonly root?: string
   readonly json?: boolean
   readonly gate?: boolean
+  readonly touched?: boolean
 }
 
 /** What a reader does about each way the sweep produced no reading. */
@@ -613,6 +620,10 @@ export function register(program: Command): void {
       '--gate',
       'Set 2 only for a finding in a gated folder, which the push stage reads',
     )
+    .option(
+      '--touched',
+      'Owe a ledger for each core rule the branch changed against origin/main',
+    )
     .addHelpText(
       'after',
       [
@@ -632,6 +643,9 @@ export function register(program: Command): void {
         `  asks a ledger of each rule under ${PRACTICE_FOLDERS.join(', ')}`,
         '  at governance/ledger/<folder>/<slug>.md, one table row per bullet',
         '  under a heading of the same name.',
+        `  A rule under ${CORE_FOLDERS.join(', ')} has its ledger checked where`,
+        '  one exists, and under --touched owes one once the branch changed it.',
+        '  With no baseline against main, --touched reads no rule as touched.',
         '  Any other tree reads .claude/rules/project/ alone and asks no ledger.',
         '  .claude/rules/canon/ is fixed at the toolkit source and never read',
         '  unless named. A named path resolves against --root and must sit',
@@ -654,6 +668,7 @@ export function register(program: Command): void {
         '  canon gov audit',
         '  canon gov audit .claude/rules/project/api.md',
         '  canon gov audit --json',
+        '  canon gov audit --gate --touched',
         '',
       ].join('\n'),
     )
@@ -672,7 +687,13 @@ async function runAudit(
   opts: AuditOptions,
 ): Promise<number> {
   const root = resolve(opts.root ?? process.cwd())
-  const report = await readRuleAudit(root, paths)
+  const changed = opts.touched
+    ? await collectChangedFiles(commandRunner(root))
+    : undefined
+  // An unscoped set lists nothing, and reading it as every file would owe a
+  // ledger of every core rule on a branch that touched none of them.
+  const touched = changed?.scoped ? changed.files : []
+  const report = await readRuleAudit(root, paths, { touched })
   const emitJson = opts.json ?? false
 
   if (report.kind === 'unreadable') {
@@ -690,7 +711,7 @@ async function runAudit(
     return 1
   }
 
-  reportAudit(report, root, paths.length > 0)
+  reportAudit(report, root, paths.length > 0, changed)
 
   if (emitJson) {
     process.stdout.write(
@@ -700,6 +721,9 @@ async function runAudit(
         limits: RULE_LIMITS,
         gated: RULE_GATED_FOLDERS,
         exempt: RULE_GATE_EXEMPT,
+        ...(changed === undefined
+          ? {}
+          : { touched: { scoped: changed.scoped } }),
         ...report,
       })}\n`,
     )
@@ -728,6 +752,7 @@ function reportAudit(
   report: Extract<RuleAuditReport, { kind: 'measured' }>,
   root: string,
   named: boolean,
+  changed: ChangedSet | undefined,
 ): void {
   intro('canon gov audit')
 
@@ -761,10 +786,15 @@ function reportAudit(
   }
 
   logStep('Ledgers')
+  if (changed !== undefined && !changed.scoped) {
+    logInfo(
+      'no baseline against main, so --touched reads no core rule as touched',
+    )
+  }
   if (report.scope === 'target') {
     logInfo('not read, since ledgers live in the toolkit and never ship')
   } else if (ledgers.length === 0) {
-    logInfo('every outside-practice rule has a ledger row per bullet')
+    logInfo('every rule owing a ledger has one, with a row per bullet')
   } else {
     for (const line of ledgers) logWarn(line)
   }

package/src/commands/migrate.ts

@@ -1309,6 +1309,13 @@ export function register(program: Command): void {
         'any ordinal evidence/ already carries. The dry run labels every',
         'derived destination, so check the split before passing --write.',
         '',
+        'Moves the design board to the folder Frameboard names:',
+        '  canvas/                         -> board/',
+        '',
+        'Stop a frameboard serve on canvas/ before --write. A project holding',
+        'both folders gets a collision, and a design.toml board key naming',
+        '.canon/canvas is left for you to update.',
+        '',
         'A citation reaching into a moved folder or naming a moved file is',
         'repointed, live or archived. A bare mention of an emptied folder, such',
         'as review/evidence/ with no slug, matches no row and stays as written.',
@@ -1389,8 +1396,11 @@ export function register(program: Command): void {
         `It also carries ${Object.keys(SKILL_CITATION_MAP).length} namespaced-only rows for a skill renamed from a single word.`,
         'The seven draft- skills move onto verb-first names, such as',
         // The old spellings below are the rename itself. canon-keep-retired
-        'draft-doc to write-doc and draft-and-pick to pick-by-eye, and a',
-        'pointer left under each old name is never moved.',
+        'draft-doc to write-doc and draft-slides to make-deck, and a',
+        'pointer left under each old name is never moved. Four skills then',
+        'moved into the family their job belongs to, such as',
+        // canon-keep-retired
+        'ux-audit to ui-audit and pick-by-eye to design-pick.',
         'It takes no scope. The skill folders it moves are authored in the',
         'toolkit and no target holds a copy of that catalog, so in a target it',
         'rewrites citations of a renamed skill and moves nothing.',