@erclx/canon 5.53.0 → 5.55.0

package/src/commands/design.ts

@@ -1,90 +1,60 @@
-import { copyFileSync, existsSync, mkdirSync } from 'node:fs'
-import { basename, join, relative, resolve } from 'node:path'
+import { existsSync } from 'node:fs'
+import { join, relative } from 'node:path'
 import type { Command } from 'commander'
-import {
-  createDesignAdapter,
-  DESIGN_INSTALL_DIR,
-  DESIGN_PROJECT_SUBDIR,
-} from '@/design/adapter'
 import { buildDesignCss } from '@/design/css'
 import { HAND_DRAWN_FONT_FACES } from '@/design/fonts'
-import { addProjectFace, listProjectFaces } from '@/design/project-fonts'
-import { renderDesignDoc } from '@/design/render'
-import { DESIGN_BASE_CSS, DESIGN_DOCUMENT, regenDesign } from '@/design/regen'
+import { DESIGN_DOCUMENT, regenDesign } from '@/design/regen'
 import { checkoutMismatchWarning, PROJECT_ROOT } from '@/roots/project'
-import { creationRel, SCRATCH } from '@/roots/record'
-import { surfaceDir } from '@/roots/surface'
-import { recordStamp, runDomainSync } from '@/sync/engine'
-import { isDirectory, resolveTarget } from '@/targets/validate'
-import {
-  intro,
-  logAdd,
-  logError,
-  logInfo,
-  logWarn,
-  outro,
-  palette,
-} from '@/cli/ui'
+import { intro, logError, logInfo, logWarn, outro } from '@/cli/ui'
 
 export function register(program: Command): void {
   const design = program
     .command('design')
     .helpOption('-h, --help', 'Show this help message')
-    .description(
-      'Design system commands (regen, css, render, install, sync, fonts)',
-    )
+    .description('Design system commands (regen, css)')
 
   design
     .command('regen')
     .helpOption('-h, --help', 'Show this help message')
-    .description(
-      'Rewrite canon/DESIGN.md and the base stylesheet from src/design/tokens.ts',
-    )
+    .description('Rewrite canon/DESIGN.md from src/design/tokens.ts')
     .addHelpText(
       'after',
       [
         '',
-        'Unlike install and sync, this runs against the toolkit checkout rather',
-        'than a target. It rewrites the record and the base stylesheet from the',
-        'token module and refuses where the record is absent, which is what the',
-        'CLI installed into a project as a dependency looks like.',
+        'This runs against the toolkit checkout rather than a target. It',
+        'rewrites the record from the token module and refuses where the record',
+        'is absent, which is what the CLI installed into a project as a',
+        'dependency looks like.',
         '',
       ].join('\n'),
     )
     .action(() => {
-      const { GREEN, GREY, NC, RED, WHITE } = palette(process.stderr)
-
       // The guard below catches a `PROJECT_ROOT` with no record at all, which
       // is an installed package. It cannot catch a second real checkout that
       // has one, so both paths carry the warning. It is a frame-interior line,
       // so each emits it after its own opener rather than ahead of the branch.
       const mismatch = checkoutMismatchWarning(process.cwd())
 
-      // Both outputs resolve from `PROJECT_ROOT`, which is the installed
+      // The output resolves from `PROJECT_ROOT`, which is the installed
       // package directory when the CLI runs out of a target's `node_modules`.
-      // The record is the one output that is already committed here and ships
-      // with no package, so its absence is what separates the two.
+      // The record is already committed here and ships with no package, so
+      // its absence is what separates the two.
       if (!existsSync(join(PROJECT_ROOT, DESIGN_DOCUMENT))) {
-        process.stderr.write(`${GREY}┌${NC}\n`)
+        intro('Regenerate design source')
         if (mismatch !== undefined) logWarn(mismatch)
-        process.stderr.write(
-          `${GREY}│${NC} ${RED}✗${NC} No ${DESIGN_DOCUMENT} at ${PROJECT_ROOT}. Regen runs in the toolkit checkout, not against a target.\n${GREY}└${NC}\n`,
+        logError(
+          `No ${DESIGN_DOCUMENT} at ${PROJECT_ROOT}. Regen runs in the toolkit checkout, not against a target.`,
         )
+        outro()
         process.exitCode = 1
         return
       }
 
-      process.stderr.write(
-        `${GREY}┌${NC}\n${GREY}│${NC} ${WHITE}Regenerate design source${NC}\n`,
-      )
+      intro('Regenerate design source')
       if (mismatch !== undefined) logWarn(mismatch)
       const result = regenDesign(PROJECT_ROOT)
-      for (const path of [result.documentPath, result.cssPath]) {
-        process.stderr.write(
-          `${GREY}│${NC} ${GREEN}✓${NC} ${relative(PROJECT_ROOT, path)}\n`,
-        )
-      }
-      process.stderr.write(`${GREY}└${NC}\n`)
+      logInfo(relative(PROJECT_ROOT, result.documentPath))
+      outro()
     })
 
   design
@@ -107,278 +77,4 @@ export function register(program: Command): void {
         }),
       )
     })
-
-  design
-    .command('render')
-    .helpOption('-h, --help', 'Show this help message')
-    .description('Render DESIGN.md tokens to HTML and CSS preview')
-    .option(
-      '-s, --source <path>',
-      'Source DESIGN.md path',
-      relative(process.cwd(), surfaceDir(process.cwd(), 'DESIGN.md')),
-    )
-    .option(
-      '-o, --out <path>',
-      'Output directory',
-      creationRel(process.cwd(), SCRATCH, 'render', 'design'),
-    )
-    .option(
-      '--embed-fonts',
-      'Declare each vendored face the typography table names, and set the page body in the body stack',
-    )
-    .action((opts: { source: string; out: string; embedFonts?: boolean }) => {
-      const sourcePath = resolve(process.cwd(), opts.source)
-      const outDir = resolve(process.cwd(), opts.out)
-      const { GREEN, GREY, NC, RED, WHITE } = palette(process.stderr)
-      if (!existsSync(sourcePath)) {
-        process.stderr.write(
-          `${GREY}┌${NC}\n${GREY}│${NC} ${RED}✗${NC} ${opts.source} not found\n${GREY}└${NC}\n`,
-        )
-        process.exitCode = 1
-        return
-      }
-      process.stderr.write(
-        `${GREY}┌${NC}\n${GREY}│${NC} ${WHITE}Render design tokens${NC}\n`,
-      )
-      const result = renderDesignDoc(sourcePath, outDir, {
-        embedFonts: opts.embedFonts,
-      })
-      process.stderr.write(
-        `${GREY}│${NC} ${GREEN}✓${NC} ${result.htmlPath}\n${GREY}│${NC} ${GREEN}✓${NC} ${result.cssPath}\n${GREY}└${NC}\n`,
-      )
-    })
-
-  design
-    .command('install')
-    .description('Install the base stylesheet into a project')
-    .argument('[target]', 'Target directory', '.')
-    .helpOption('-h, --help', 'Show this help message')
-    .addHelpText(
-      'after',
-      [
-        '',
-        'Installs one toolkit-owned file and creates no override. A project',
-        `overrides a value by writing ${join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR)}/`,
-        'itself, which sync never touches. Nothing arrives on a project that',
-        'has not run this, so a design value reaches a target on an install',
-        'rather than on the next sync. A base stylesheet already installed is',
-        'replaced.',
-        '',
-        'Exit codes:',
-        '  0  the stylesheet was installed',
-        '  1  the target is missing or is the toolkit root, or the toolkit',
-        '     source is absent',
-        '',
-      ].join('\n'),
-    )
-    .action(async (target: string) => {
-      process.exitCode = await runInstall(target)
-    })
-
-  design
-    .command('sync')
-    .description(
-      'Update the base stylesheet already installed under .claude/design/',
-    )
-    .argument('[target]', 'Target directory', '.')
-    .helpOption('-h, --help', 'Show this help message')
-    .addHelpText(
-      'after',
-      [
-        '',
-        'Replaces the shipped design files under .claude/design/ with the',
-        'toolkit copy, edited or not, and leaves',
-        `${join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR)}/ alone. It deletes nothing it finds no`,
-        'source for.',
-        '',
-        'Exit codes:',
-        '  0  synced, already up to date, or the prompt was cancelled',
-        '  1  the target is missing or is the toolkit root',
-        '',
-      ].join('\n'),
-    )
-    .action(async (target: string) => {
-      process.exitCode = await runDomainSync(
-        createDesignAdapter(PROJECT_ROOT),
-        target,
-        { protectedRoot: PROJECT_ROOT },
-      )
-    })
-
-  registerFonts(design)
-}
-
-const FONTS_DIR = join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR, 'fonts')
-
-interface FontsRefused {
-  readonly ok: false
-  readonly reason: string
-  readonly detail: string
-}
-
-function writeJson(record: unknown): void {
-  process.stdout.write(`${JSON.stringify(record)}\n`)
-}
-
-function refuseFonts(
-  title: string,
-  refusal: FontsRefused,
-  emitJson: boolean,
-): number {
-  if (emitJson) writeJson(refusal)
-  intro(title)
-  logError(refusal.detail)
-  outro()
-  return 1
-}
-
-/** A target that is a directory, or the refusal naming the path given. */
-function fontsTarget(target: string): string | FontsRefused {
-  const resolved = resolve(target)
-  if (!isDirectory(resolved)) {
-    return {
-      ok: false,
-      reason: 'no-target',
-      detail: `Target directory not found: ${target}`,
-    }
-  }
-  return resolved
-}
-
-function registerFonts(design: Command): void {
-  const fonts = design
-    .command('fonts')
-    .helpOption('-h, --help', 'Show this help message')
-    .description(`Read and add the project's own faces under ${FONTS_DIR}/`)
-    .addHelpText(
-      'after',
-      [
-        '',
-        `Faces are declared as @font-face rules in ${join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR)}/fonts.css,`,
-        'which the canvas embeds into every frame. Sync never writes the folder.',
-        '',
-      ].join('\n'),
-    )
-
-  fonts
-    .command('list')
-    .helpOption('-h, --help', 'Show this help message')
-    .description('List each face the project override sheets declare')
-    .argument('[target]', 'Target directory', '.')
-    .option('--json', 'Emit the record on stdout')
-    .action((target: string, opts: { json?: boolean }) => {
-      const title = 'canon design fonts list'
-      const resolved = fontsTarget(target)
-      if (typeof resolved !== 'string') {
-        process.exitCode = refuseFonts(title, resolved, Boolean(opts.json))
-        return
-      }
-      const faces = listProjectFaces(resolved)
-      if (opts.json) writeJson({ ok: true, faces })
-      intro(title)
-      if (faces.length === 0) logInfo(`No faces declared under ${FONTS_DIR}/`)
-      for (const face of faces) {
-        const line = `${face.family} ${face.weight} ${face.style}: ${face.file} in ${face.sheet}`
-        if (face.present) logInfo(line)
-        else logWarn(`${line} (${face.problem ?? 'missing'})`)
-      }
-      outro()
-    })
-
-  fonts
-    .command('add')
-    .helpOption('-h, --help', 'Show this help message')
-    .description(
-      `Copy a woff2, woff, ttf, or otf face into ${FONTS_DIR}/ and declare it`,
-    )
-    .argument('<file>', 'Font file to copy')
-    .argument('[target]', 'Target directory', '.')
-    .requiredOption('--family <name>', 'Family name the face declares')
-    .option('--weight <n>', "Weight, or a range such as '100 900'", '400')
-    .option('--style <style>', 'normal or italic', 'normal')
-    .option('--json', 'Emit the record on stdout')
-    .action(
-      (
-        file: string,
-        target: string,
-        opts: { family: string; weight: string; style: string; json?: boolean },
-      ) => {
-        const title = 'canon design fonts add'
-        const emitJson = Boolean(opts.json)
-        const resolved = fontsTarget(target)
-        if (typeof resolved !== 'string') {
-          process.exitCode = refuseFonts(title, resolved, emitJson)
-          return
-        }
-        if (resolved === PROJECT_ROOT) {
-          process.exitCode = refuseFonts(
-            title,
-            {
-              ok: false,
-              reason: 'toolkit-root',
-              detail:
-                'Cannot run against toolkit root. Files here are the source of truth.',
-            },
-            emitJson,
-          )
-          return
-        }
-        const outcome = addProjectFace(resolved, resolve(file), {
-          family: opts.family,
-          weight: opts.weight,
-          style: opts.style,
-        })
-        if (!outcome.ok) {
-          process.exitCode = refuseFonts(title, outcome, emitJson)
-          return
-        }
-        if (emitJson) writeJson(outcome)
-        intro(title)
-        logAdd(join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR, outcome.file))
-        logInfo(
-          `Declared ${outcome.family} ${outcome.weight} ${outcome.style} in ${outcome.sheet}`,
-        )
-        outro()
-      },
-    )
-}
-
-/**
- * Copies the base and stops. The override folder is deliberately not created:
- * an empty override is a file the project did not ask for and did not write,
- * the three-way merge already handles a missing side, and an empty file invites
- * a target to fill it before it has an opinion.
- */
-async function runInstall(target: string): Promise<number> {
-  intro('canon design install')
-
-  const resolved = resolveTarget(target, PROJECT_ROOT)
-  if (typeof resolved === 'number') {
-    outro()
-    return resolved
-  }
-
-  const source = join(PROJECT_ROOT, DESIGN_BASE_CSS)
-  if (!existsSync(source)) {
-    logError(
-      `No base stylesheet at ${DESIGN_BASE_CSS}. Run canon design regen.`,
-    )
-    outro()
-    return 1
-  }
-
-  const dir = join(resolved, DESIGN_INSTALL_DIR)
-  mkdirSync(dir, { recursive: true })
-
-  const dest = join(dir, basename(source))
-  copyFileSync(source, dest)
-  logAdd(relative(resolved, dest))
-
-  await recordStamp(createDesignAdapter(PROJECT_ROOT), resolved, new Date())
-
-  logInfo(
-    `Override a value in ${join(DESIGN_INSTALL_DIR, DESIGN_PROJECT_SUBDIR)}/, which sync leaves alone.`,
-  )
-  outro()
-  return 0
 }

package/src/commands/gate.ts

@@ -19,6 +19,7 @@ import {
   logStep,
   logWarn,
   outro,
+  outroSuccess,
   palette,
   pipeOutput,
   plural,
@@ -216,20 +217,21 @@ function report(result: StageResult): void {
  * against.
  */
 function close(summary: Summary, code: number): void {
-  const { GREEN, NC, RED, YELLOW } = palette(process.stderr)
-  outro()
+  const { NC, RED, YELLOW } = palette(process.stderr)
 
   if (code !== 0) {
+    outro()
     process.stderr.write(`${RED}✗ Verification failed${NC}\n\n`)
     return
   }
 
   if (summary.unmeasured > 0) {
+    outro()
     process.stderr.write(
       `${YELLOW}! Verification passed, ${plural(summary.unmeasured, 'stage')} measured nothing${NC}\n\n`,
     )
     return
   }
 
-  process.stderr.write(`${GREEN}✓ Verification passed${NC}\n\n`)
+  outroSuccess('Verification passed')
 }

package/src/commands/gov.ts

@@ -75,7 +75,7 @@ import {
   logStep,
   logWarn,
   outro,
-  palette,
+  outroSuccess,
   select,
 } from '@/cli/ui'
 
@@ -184,9 +184,9 @@ export function register(program: Command): void {
         'Copies each rule into .claude/rules/canon/, replacing a copy already',
         'there. Rules under .claude/rules/project/ are never read.',
         '',
-        'The install confirm needs a TTY. --yes answers it, so a headless call',
-        'installs without setting CANON_NON_INTERACTIVE=1. It does not pick a',
-        'stack, so a headless call still names one.',
+        'The install confirm needs a TTY. --yes answers it, and a call with no',
+        'terminal on stdin takes Yes. It does not pick a stack, so a headless',
+        'call still names one.',
         '',
         'Exit codes:',
         '  0  installed, the stack held no rules, or the prompt was cancelled',
@@ -1160,7 +1160,7 @@ function runTestOrder(opts: TestOrderOptions): number {
   const report = readTestOrder(root, { base: opts.base })
   const emitJson = opts.json ?? false
 
-  // The frame renders on stderr in both modes and the record goes to stdout
+  // The refusal reaches stderr in both modes and the record goes to stdout
   // alone, which is the split `docs/agents/output-shape.md` fixes. A consumer
   // reading stdout sees pure data either way, and an operator reading the
   // terminal sees the refusal rather than a command that appeared to do nothing.
@@ -1455,8 +1455,8 @@ function displayPath(target: string, rel: string): string {
 }
 
 /**
- * Refuses rather than picking. `select_option` returned `options[0]` under
- * `CANON_NON_INTERACTIVE=1`, so a headless `canon gov install` with no stack
+ * Refuses rather than picking. `select_option` returned `options[0]` with no
+ * terminal to answer it, so a headless `canon gov install` with no stack
  * installed whichever stack sorted first, measured as 26 astro rules into an
  * empty directory. Every documented agent path already passes the argument.
  */
@@ -1558,16 +1558,13 @@ async function runInstall(
     now,
   )
 
-  const { GREEN, NC } = palette(process.stderr)
-  outro()
-  process.stderr.write(`${GREEN}✓ Rules installed${NC}\n`)
+  outroSuccess('Rules installed')
   return 0
 }
 
 /**
  * The install confirm. `--yes` answers it without a prompt, so the line it
- * would have printed is skipped rather than duplicated under
- * `CANON_NON_INTERACTIVE=1`.
+ * would have printed is skipped rather than duplicated on a headless call.
  */
 async function confirmInstall(
   count: number,
@@ -1582,7 +1579,6 @@ async function confirmInstall(
       { value: false, label: 'No' },
     ],
     nonInteractiveDefault: true,
-    ttyHint: 'Pass --yes',
   })
 }
 
@@ -1638,9 +1634,7 @@ async function installExtras(
 
   await recordStamp(createGovAdapter(PROJECT_ROOT), resolved, new Date())
 
-  const { GREEN, NC } = palette(process.stderr)
-  outro()
-  process.stderr.write(`${GREEN}✓ Rules installed${NC}\n`)
+  outroSuccess('Rules installed')
   return 0
 }
 
@@ -1689,10 +1683,6 @@ async function runBuild(target: string): Promise<number> {
   await writeFile(output, buildRulesPayload(files))
   logAdd(payload)
 
-  const { GREEN, NC } = palette(process.stderr)
-  outro()
-  process.stderr.write(
-    `${GREEN}✓ Rules built (${files.length} rules → ${payload})${NC}\n`,
-  )
+  outroSuccess(`Rules built (${files.length} rules → ${payload})`)
   return 0
 }

package/src/commands/labels.ts

@@ -311,7 +311,7 @@ async function runAudit(paths: string[], opts: AuditOptions): Promise<number> {
 
   intro('canon labels audit')
 
-  // The frame renders on stderr in both modes and the record goes to stdout
+  // The refusal reaches stderr in both modes and the record goes to stdout
   // alone, so an operator reading the terminal sees the refusal rather than a
   // command that appeared to do nothing.
   if (report.kind === 'refused') {

package/src/commands/pr.ts

@@ -3839,7 +3839,7 @@ async function readEvidenceGate(
       cwd: root,
       reject: false,
       timeout: GH_TIMEOUT_MS * 2,
-      env: { ...gitEnv(), CANON_NON_INTERACTIVE: '1' },
+      env: gitEnv(),
       extendEnv: false,
     },
   )

package/src/commands/sync.ts

@@ -35,7 +35,6 @@ import { describeSkew } from '@/version/skew'
 
 const SYNC_ARGS: Record<SyncDomain, readonly string[]> = {
   governance: ['gov', 'sync'],
-  design: ['design', 'sync'],
   claude: ['claude', 'sync'],
 }
 
@@ -58,10 +57,10 @@ export function register(program: Command): void {
       'after',
       [
         '',
-        'Runs `canon gov sync`, `canon design sync`, and `canon claude sync`',
-        'on each domain installed, so each carries its own replace behavior:',
-        'governance replaces an edited rule and deletes a retired one, design',
-        'replaces its shipped files, and claude writes .gitignore alone.',
+        'Runs `canon gov sync` and `canon claude sync` on each domain',
+        'installed, so each carries its own replace behavior: governance',
+        'replaces an edited rule and deletes a retired one, and claude writes',
+        '.gitignore alone.',
         'Tooling is not synced. Refuses a dirty working tree.',
         '',
         'Exit codes:',
@@ -424,7 +423,7 @@ async function runSync(target: string): Promise<number> {
 /**
  * Each domain sync runs as its own process. They open their own frames and
  * prompt for their own changes, so stdin stays inherited. Claude is the one
- * exception: it runs headless because the combined pull request preview is the
+ * exception: its stdin is closed because the combined pull request preview is the
  * single confirmation gate for the whole sync, and `canon claude sync` writes
  * only `.gitignore`.
  */

package/src/commands/tooling.ts

@@ -36,7 +36,7 @@ import {
   logStep,
   logWarn,
   outro,
-  palette,
+  outroSuccess,
   select,
 } from '@/cli/ui'
 
@@ -101,7 +101,7 @@ export function register(program: Command): void {
         'Examples:',
         '  canon tooling sync base',
         '  canon tooling sync base --check',
-        '  CANON_NON_INTERACTIVE=1 canon tooling sync base --write',
+        '  canon tooling sync base --write',
         '  canon tooling sync vite-react web --skip base',
         '',
         'A target below its git root syncs as a subfolder. It withholds',
@@ -412,7 +412,6 @@ async function runSync(
   }
 
   const mode = resolveWriteMode(opts)
-  const { GREEN, NC } = palette(process.stderr)
 
   if (result.totalChanges === 0) {
     // The stamp is a write like any other, so a run with no authority to write
@@ -421,8 +420,7 @@ async function runSync(
     if (mode === 'apply' || mode === 'prompt') {
       await stampChain(prepared.chain, prepared.target)
     }
-    outro()
-    process.stderr.write(`${GREEN}✓ Everything up to date${NC}\n`)
+    outroSuccess('Everything up to date')
     return 0
   }
 
@@ -450,8 +448,7 @@ async function runSync(
 
   await stampChain(prepared.chain, prepared.target)
 
-  outro()
-  process.stderr.write(`${GREEN}✓ Tooling sync complete${NC}\n`)
+  outroSuccess('Tooling sync complete')
   return 0
 }
 
@@ -607,7 +604,7 @@ async function promptForStack(): Promise<string | undefined> {
   return select({
     message: 'Select tooling stack:',
     options: stacks.map((name) => ({ value: name, label: name })),
-    nonInteractiveDefault: true,
+    ttyHint: `Pass a stack, one of: ${stacks.join(', ')}`,
   })
 }
 

package/src/commands/transcripts.ts

@@ -2,7 +2,7 @@ import { resolve } from 'node:path'
 import type { Command } from 'commander'
 import { recordDir } from '@/roots/record'
 import { ensureYtDlp, fetchOne } from '@/transcripts/fetch'
-import { palette } from '@/cli/ui'
+import { intro, logError, logInfo, outro } from '@/cli/ui'
 import { mainWorktreeRoot } from '@/git/worktree'
 
 interface TranscriptOptions {
@@ -37,10 +37,7 @@ export function register(program: Command): void {
     )
     .action(async (url: string, opts: TranscriptOptions) => {
       const outDir = await resolveOutDir(opts)
-      const { GREEN, GREY, NC, RED, WHITE } = palette(process.stderr)
-      process.stderr.write(
-        `${GREY}┌${NC}\n${GREY}│${NC} ${WHITE}canon transcripts${NC}\n`,
-      )
+      intro('canon transcripts')
       try {
         ensureYtDlp()
         const target = await fetchOne(url, {
@@ -48,14 +45,12 @@ export function register(program: Command): void {
           keepTimestamps: opts.keepTimestamps ?? false,
         })
         process.stdout.write(`${target}\n`)
-        process.stderr.write(
-          `${GREY}│${NC} ${GREEN}✓${NC} ${target}\n${GREY}└${NC}\n`,
-        )
+        logInfo(target)
+        outro()
       } catch (error) {
         const message = error instanceof Error ? error.message : String(error)
-        process.stderr.write(
-          `${GREY}│${NC} ${RED}✗${NC} ${message}\n${GREY}└${NC}\n`,
-        )
+        logError(message)
+        outro()
         process.exitCode = 1
       }
     })