@erclx/canon 4.117.0 → 4.119.0

package/src/claude/cases/workflow.ts

@@ -163,11 +163,21 @@ export const WORKFLOW_CASES: readonly SkillCase[] = [
     prompt: 'Get me set up in a fresh Claude Code worktree for this branch.',
     expect: 'session-worktree',
   },
+  {
+    prompt:
+      'My Write to the plan file at the main root got refused for session isolation. How do I land it there?',
+    expect: 'session-worktree',
+  },
   {
     prompt:
       'Relay this over to the session that dispatched me. It gave me a sessionId rather than a name, so work out who that is and send it.',
     expect: 'session-relay',
   },
+  {
+    prompt:
+      'I am stuck on a question only the operator can answer. Before I stop and ask it, tell the session that dispatched me, it gave me a sessionId.',
+    expect: 'session-relay',
+  },
 
   // Negatives: pairs whose bodies already carry an explicit Do NOT boundary.
   {

package/src/commands/context.ts

@@ -56,12 +56,6 @@ import {
 } from '@/context/folders'
 import { isGating } from '@/context/gate'
 import { auditIndexes, type FolderDrift } from '@/context/index-drift'
-import {
-  loadNarration,
-  type Narration,
-  PRONOUN_HEADING,
-  VERB_HEADING,
-} from '@/context/narration'
 import {
   measureWireframeFolder,
   type WireframeStatesReport,
@@ -101,7 +95,7 @@ export function register(program: Command): void {
   context
     .command('audit')
     .description(
-      'Report required sections, entry length, citations, reference form, catalog tables, provenance, superseded-decision narration, index drift, the architecture record against its own ceiling, its own entry cap, and its word weight, and wireframe states against their evidence folders',
+      'Report required sections, entry length, citations, reference form, catalog tables, provenance, index drift, the architecture record against its own ceiling, its own entry cap, and its word weight, and wireframe states against their evidence folders',
     )
     .argument('[path]', 'Project root, defaulting to the current directory')
     .helpOption('-h, --help', 'Show this help message')
@@ -133,8 +127,8 @@ export function register(program: Command): void {
         'findings that are facts rather than judgments: a missing required',
         'section and index drift. A context entry requires Overview and',
         'Layout, and a wireframe requires Regions, States, Copy, and Not on',
-        'this surface. Entry length, reference form, table, provenance, narration,',
-        'and the record claim classification are judgments under both.',
+        'this surface. Entry length, reference form, table, provenance, and the',
+        'record claim classification are judgments under both.',
         '',
         'Depth and bullet weight are stated over every markdown file rather',
         'than over a context entry, so `canon markdown audit` measures them.',
@@ -653,24 +647,13 @@ async function runAudit(
     )
   }
 
-  const narration: Narration = gateOnly
-    ? { kind: 'absent' }
-    : await loadNarration(root)
-  const entries = gateOnly
-    ? []
-    : await measureFolders(
-        root,
-        folders,
-        narration.kind === 'loaded' ? narration : undefined,
-      )
+  const entries = gateOnly ? [] : await measureFolders(root, folders)
   const drift = gateOnly ? [] : await auditIndexes(folders)
   const sections = gateOnly ? [] : missingSections(root, folders, entries)
   const length = gateOnly ? undefined : lengthFindings(entries)
   // Absent under `--citations-only` and null when the project carries no
-  // record, for the reason `checkpoints.narration` states about its own two
-  // absences. A run that never looked and a project with nothing to look at
-  // are different answers, and one value for both reports the second as the
-  // first.
+  // record. A run that never looked and a project with nothing to look at are
+  // different answers, and one value for both reports the second as the first.
   const record = gateOnly ? undefined : await measureArchitecture(root)
   const wireframes = gateOnly
     ? []
@@ -693,7 +676,6 @@ async function runAudit(
     reportLength(length ?? [])
     reportTables(entries)
     reportProvenance(entries, folders)
-    reportNarration(entries, folders, narration)
     reportDrift(drift)
     reportRecord(record, root)
     reportWireframeStates(wireframes)
@@ -722,10 +704,11 @@ async function runAudit(
         // answers, and one wrong restatement is a cause reported against the
         // wrong entry.
         //
-        // Absent rather than empty under `--citations-only`, for the reason
-        // `checkpoints.narration` below carries. That mode measures no entry,
-        // so an empty array here reads as a corpus with nothing past the
-        // checkpoint rather than as a run that never looked.
+        // Absent rather than empty under `--citations-only`. That mode
+        // measures no entry, so an empty array here reads as a corpus with
+        // nothing past the checkpoint rather than as a run that never looked,
+        // and collapsing the two reports a measured answer for a run that
+        // never opened an entry.
         length,
         missingSections: sections,
         indexDrift: drift,
@@ -747,20 +730,6 @@ async function runAudit(
           // caller that read it before wireframes owed a section, since turning
           // it into this map would break that caller without an error.
           requiredSectionsByFolder: REQUIRED_SECTIONS_BY_FOLDER,
-          // Three states rather than two, so a record showing no finding says
-          // which terms were looked for. The key is absent when the run never
-          // scanned, which is `--citations-only`, and null when it scanned and
-          // no rule published both headings. Collapsing the first into null
-          // reports an absent rule against a run that never opened one.
-          narration: gateOnly
-            ? undefined
-            : narration.kind === 'loaded'
-              ? {
-                  source: narration.source,
-                  pronouns: narration.pronouns,
-                  verbs: narration.verbs,
-                }
-              : null,
         },
       })}\n`,
     )
@@ -1144,75 +1113,6 @@ function reportProvenance(
   )
 }
 
-/**
- * Reports the bullets narrating a decision the bullet above them replaced.
- *
- * The reach line names the rule the sets were read from, since the check is
- * silent when no rule publishes them and a run that scanned nothing otherwise
- * prints the same clean line as a run that scanned everything.
- *
- * The legitimate-hit line is here for the same reason the provenance section
- * says a marker is a judgment. The standard keeps a rejected alternative and
- * why it lost, which is a back-reference in the past tense by construction, and
- * no measure separates one from the shape the rule bans.
- */
-function reportNarration(
-  entries: readonly EntryReport[],
-  folders: readonly AuditedFolder[],
-  narration: Narration,
-): void {
-  logStep('Narration')
-
-  const governed = folders.filter(governsContent)
-  if (governed.length === 0) {
-    logInfo(
-      `Out of scope. The rule is stated in the standard governing canon/${PROVENANCE_FOLDER}/, and no audited folder is that one.`,
-    )
-    return
-  }
-
-  if (narration.kind === 'absent') {
-    logWarn(
-      `Not scanned. No rule under .claude/rules/ or governance/rules/ publishes both ${PRONOUN_HEADING} and ${VERB_HEADING}.`,
-    )
-    return
-  }
-
-  logInfo(
-    `Covers canon/${PROVENANCE_FOLDER}/ alone, reading ${plural(narration.pronouns.length, 'pronoun')} and ${plural(narration.verbs.length, 'verb')} from ${narration.source}.`,
-  )
-  logInfo(
-    'A rejected alternative is a legitimate hit, since the standard keeps what was tried and why it lost.',
-  )
-
-  const carrying = entries
-    .filter((entry) => entry.narration.length > 0)
-    .sort((a, b) => b.narration.length - a.narration.length)
-
-  if (carrying.length === 0) {
-    logInfo('No bullet narrates a decision the bullet above it replaced.')
-    return
-  }
-
-  const total = carrying.reduce((sum, entry) => sum + entry.narration.length, 0)
-  logWarn(
-    `${plural(total, 'bullet')} to read across ${carrying.length} ${carrying.length === 1 ? 'entry' : 'entries'}`,
-  )
-  pipeOutput(
-    carrying
-      .map(
-        (entry) =>
-          `${entry.rel}  ${plural(entry.narration.length, 'bullet')}\n${entry.narration
-            .map(
-              (found) =>
-                `  :${found.line}  ${found.pronoun} with ${found.verb}`,
-            )
-            .join('\n')}`,
-      )
-      .join('\n'),
-  )
-}
-
 /** How each classification reads in the report. */
 const CLAIM_LABEL: Record<string, string> = {
   countable: 'countable claim',

package/src/commands/sync.ts

@@ -133,9 +133,17 @@ function renderCheck(report: CheckReport): void {
       logWarn(entry.notice ?? `${entry.rel} (${entry.state})`)
     }
 
-    const { stale, customized, drifted, stranded, orphaned, missing } =
-      domain.counts
-    if (stale + customized + drifted + stranded === 0) {
+    const {
+      stale,
+      customized,
+      drifted,
+      stranded,
+      retired,
+      renamed,
+      orphaned,
+      missing,
+    } = domain.counts
+    if (stale + customized + drifted + stranded + retired + renamed === 0) {
       logInfo(orphaned === 0 ? 'up to date' : `up to date (${orphaned} local)`)
     }
     // A local file is not a deficiency and a missing rule is, so this never

package/src/commands/tasks.ts

@@ -409,12 +409,14 @@ export function register(program: Command): void {
       [
         '',
         'Exit codes:',
-        '  0  the line was added, corrected, or already correct',
+        '  0  the line was added, appended to, corrected, or already correct',
         '  1  refused, with the reason on stderr or in the JSON record',
         '',
         'It adds Pull request: #NNN under the origin lines the task carries,',
-        'and corrects the number in place when the line exists. The board is',
-        'shared scratch, so a linked worktree records against the same board.',
+        'and appends the number to the line when one exists, so a task shipped',
+        'in slices lists every pull request. A line that does not read as a',
+        'list is replaced. The board is shared scratch, so a linked worktree',
+        'records against the same board.',
         '',
         'Examples:',
         '  canon tasks pull-request 673 v28.1-trigger-escalation',
@@ -448,11 +450,10 @@ export function register(program: Command): void {
         '  0  the line was added, corrected, or already correct',
         '  1  refused, with the reason on stderr or in the JSON record',
         '',
-        'It writes Plan: [<label>](<target>) right after the H1, the same',
-        'add/correct/unchanged shape canon tasks pull-request writes with, and',
-        'resolves both a bare slug and a board-relative path the way',
-        'canon tasks plan-answers does. Safe from a linked worktree, since it',
-        'resolves the board root in-process.',
+        'It writes Plan: [<label>](<target>) right after the H1, corrects the',
+        'target in place when the line exists, and resolves both a bare slug',
+        'and a board-relative path the way canon tasks plan-answers does. Safe',
+        'from a linked worktree, since it resolves the board root in-process.',
         '',
         'Examples:',
         '  canon tasks plan-link v28.1-trigger-escalation dispatch-answer-gate',

package/src/context/audit.ts

@@ -2,7 +2,6 @@ import { readFile } from 'node:fs/promises'
 import { basename, relative } from 'node:path'
 import { BARE_NAME, IGNORE_MARKER } from '@/context/citations'
 import type { AuditedFolder } from '@/context/folders'
-import type { NarrationTerms } from '@/context/narration'
 import { type BodyLine, bodyLines, maskDisplayed } from '@/markdown/scan'
 import { documentHeight } from '@/markdown/structure'
 import { isStubSeed } from '@/seed-marker'
@@ -80,10 +79,6 @@ const TABLE_ROW = /^\s*\|/
 const TABLE_SEPARATOR = /^\s*\|[\s:|-]+\|\s*$/
 const NAMED_CELL = /`[^`]+`|\[[^\]]+\]\([^)]+\)/
 
-const TOP_BULLET = /^-\s+(.+)$/
-/** Any indented line, which is a nested bullet or a wrapped continuation. */
-const INSIDE_LIST = /^\s+\S/
-
 /**
  * Spellings of how the domain reached its shape rather than what it is now.
  *
@@ -156,9 +151,9 @@ const SENTENCE_END = /[.!?:;]\s(?=[^.!?:;]*$)/
 /**
  * The folder whose standard carries the exclusion above.
  *
- * Three measures narrow here rather than one: provenance, the required
- * sections, and the superseded-decision narration. The name is the first of
- * them because it was the first, and it stays because renaming a constant the
+ * Two measures narrow here rather than one: provenance and the required
+ * sections. The name is the first of them because it was the first, and it
+ * stays because renaming a constant the
  * JSON record publishes as `checkpoints.provenanceFolder` breaks every consumer
  * reading that field for a gain of one word.
  *
@@ -202,14 +197,6 @@ export interface BareReferenceFinding {
   readonly name: string
 }
 
-export interface NarrationFinding {
-  readonly line: number
-  /** The opening that points back at the bullet above. */
-  readonly pronoun: string
-  /** The past-tense verb that turns the back-reference into a narration. */
-  readonly verb: string
-}
-
 export interface EntryReport {
   readonly rel: string
   /**
@@ -229,12 +216,6 @@ export interface EntryReport {
   readonly catalogTables: readonly TableFinding[]
   /** Empty for an entry no standard bans a change narrative in. */
   readonly provenance: readonly ProvenanceFinding[]
-  /**
-   * Empty outside the governed folder, and empty on a run whose caller loaded
-   * no term sets. The report distinguishes the two from the vocabulary itself,
-   * since an empty list here is silent about which one produced it.
-   */
-  readonly narration: readonly NarrationFinding[]
   /**
    * References naming a sibling entry by bare filename, and empty for a caller
    * that passed no sibling names. Which entries have siblings worth matching is
@@ -434,125 +415,6 @@ function stampsMeasurement(text: string, index: number): boolean {
   return STAMPING.test(clause) || STAMPING_RUN.test(clause)
 }
 
-function escape(term: string): string {
-  return term.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-}
-
-/**
- * Copulas that turn the verb behind them into a present-tense passive.
- *
- * `used to` is the term this exists for, since `is used to resolve the folder`
- * is the passive of `use` rather than the past habitual the set means. This
- * list is English grammar rather than corpus vocabulary, so it stays in code
- * while the two tunable sets stay in the rule, and a rule publishing two of
- * three headings would be a fourth absent state to carry for no tuning anyone
- * wants.
- *
- * What it costs is the perfect passive. `has been superseded` narrates a
- * supersession and is rejected with the rest, which is a recall gap taken
- * knowingly on the trade this check already makes everywhere else.
- */
-const COPULA = ['is', 'are', 'be', 'been', 'being', 'was', 'were']
-
-/**
- * Finds the bullets narrating a decision the bullet above them replaced.
- *
- * The signal is structural rather than lexical, and the corpus is what decides
- * that. The terms carrying clean signal for a supersession are too rare to
- * catch anything, and the one term that would have caught the known case is
- * `now`, which appears 57 times across 24 entries in correct present-tense
- * prose. What separates the shape instead is a bullet pointing back at its
- * sibling and putting the sibling's design in the past: an opening pronoun with
- * no antecedent of its own, plus a past-tense verb, plus a bullet above it to
- * refer to. Precision is the whole value, so recall is the accepted exposure
- * and a narration written as a single bullet is not reached.
- *
- * The pronoun is matched cased and anchored, since a mid-sentence `this` is a
- * determiner rather than a back-reference. The verb is matched uncased anywhere
- * in the bullet, since the tense is what carries the signal wherever it sits,
- * and rejected behind a `COPULA` for the reason stated there. Displayed spans
- * are masked so a term quoted in backticks is not read as prose the entry
- * writes, and fenced blocks are skipped for the reason the scans above skip
- * them.
- *
- * A blank line does not end the run. Markdown reads two bullets around one as a
- * single loose list, so breaking there would leave the shape reachable by
- * anyone who spaced their bullets out. What ends a run is content that is
- * neither a bullet nor indented under one, which the fenced branch below has to
- * answer for itself because a fenced line is skipped before that test.
- */
-function narration(
-  lines: readonly BodyLine[],
-  terms: NarrationTerms,
-): NarrationFinding[] {
-  if (terms.pronouns.length === 0 || terms.verbs.length === 0) return []
-
-  const findings: NarrationFinding[] = []
-  let following = false
-  let fenceBlock: number | undefined
-  let fenceInsideList = false
-
-  for (const line of lines) {
-    // A fenced line is never a bullet, but an unindented block still ends the
-    // run. CommonMark reads a fence at column zero as interrupting the list, so
-    // the bullets around it are two lists and the second has no antecedent
-    // above it. A block indented under its bullet stays inside the item.
-    //
-    // The opening delimiter decides for every line of its own block. Reading
-    // each line instead ends the run on a blank line inside an indented fence,
-    // which has no indentation to read, and on a content line at column zero,
-    // which CommonMark permits since only the fence's own indent is stripped.
-    //
-    // The block index is what re-opens that decision, so two blocks written
-    // with nothing between them are answered separately rather than the second
-    // inheriting the first. Indentation stays the test here because what
-    // interrupts a list is this check's own judgment, and the walker only says
-    // which lines share a block.
-    if (line.fenced) {
-      if (line.fenceBlock !== fenceBlock) {
-        fenceBlock = line.fenceBlock
-        fenceInsideList = INSIDE_LIST.test(line.text)
-      }
-      if (!fenceInsideList) following = false
-      continue
-    }
-
-    if (line.text.trim() === '') continue
-
-    const bullet = line.text.match(TOP_BULLET)
-    if (!bullet) {
-      // A nested bullet and a wrapped continuation both sit inside the list, so
-      // neither ends the run. Anything else does, which is what keeps a bullet
-      // opening the list under a heading from reading as a reply to the last
-      // bullet of the list before it.
-      if (!INSIDE_LIST.test(line.text)) following = false
-      continue
-    }
-
-    const text = maskDisplayed(bullet[1])
-
-    if (following) {
-      const pronoun = terms.pronouns.find((term) =>
-        new RegExp(`^${escape(term)}\\b`).test(text),
-      )
-      const verb = pronoun
-        ? terms.verbs.find((term) =>
-            new RegExp(
-              `(?<!\\b(?:${COPULA.join('|')})\\s+)\\b${escape(term)}\\b`,
-              'i',
-            ).test(text),
-          )
-        : undefined
-
-      if (pronoun && verb) findings.push({ line: line.number, pronoun, verb })
-    }
-
-    following = true
-  }
-
-  return findings
-}
-
 /**
  * Finds which required sections the entry declares.
  *
@@ -639,7 +501,6 @@ export function measureEntry(
   rel: string,
   source: string,
   governsContent = true,
-  terms?: NarrationTerms,
   siblings: readonly string[] = [],
   required: readonly string[] = governsContent ? REQUIRED_SECTIONS : [],
 ): EntryReport {
@@ -650,7 +511,6 @@ export function measureEntry(
     lines: documentHeight(source),
     catalogTables: catalogTables(lines),
     provenance: governsContent ? provenance(lines) : [],
-    narration: governsContent && terms ? narration(lines, terms) : [],
     bareReferences: bareReferences(lines, siblings),
     sections: declaredSections(lines, required),
     stub: isStubSeed(source),
@@ -714,7 +574,6 @@ function historyCause(entry: EntryReport): LengthCause {
 export async function measureFolders(
   root: string,
   folders: readonly AuditedFolder[],
-  terms?: NarrationTerms,
 ): Promise<EntryReport[]> {
   const reports: EntryReport[] = []
 
@@ -731,7 +590,6 @@ export async function measureFolders(
           relative(root, path),
           await readFile(path, 'utf8'),
           governsContent(folder),
-          terms,
           names.filter((name) => name !== self),
           requiredSections(folder),
         ),

package/src/design/base.css

@@ -1,29 +1,29 @@
-/* Generated by `canon design regen` from src/design/tokens.ts. Do not edit.
+/* Generated by `canon design regen` from src/design/neutral.ts. Do not edit.
    This is the base half of a design install. A target overrides a value in
    .claude/design/project/, which the three-way merge leaves alone, so this
    file changes freely and nothing a project wrote lives in it. */
 
 :root {
-  --color-background: #0f0e0c;
-  --color-surface: #151412;
-  --color-chrome: #1b1a18;
-  --color-border: #2a2926;
-  --color-text: #d9d7d4;
-  --color-text-body: #aeada9;
-  --color-text-secondary: #8c8b86;
-  --color-muted: #7f7f7c;
-  --color-accent: #c76b5f;
+  --color-background: #0e0e0e;
+  --color-surface: #141414;
+  --color-chrome: #1a1a1a;
+  --color-border: #292929;
+  --color-text: #dddddd;
+  --color-text-body: #b2b2b2;
+  --color-text-secondary: #8e8e8e;
+  --color-muted: #7f7f7f;
+  --color-accent: #ececec;
   --color-success: #61c454;
-  --color-light-background: #fbfaf8;
-  --color-light-surface: #f1f1ee;
-  --color-light-chrome: #e9e8e5;
-  --color-light-text: #2c2c29;
-  --color-light-text-body: #4b4947;
-  --color-light-text-secondary: #666561;
-  --color-light-muted: #6e6d6c;
-  --color-light-accent: #ad4a4b;
+  --color-light-background: #fafafa;
+  --color-light-surface: #f1f1f1;
+  --color-light-chrome: #e8e8e8;
+  --color-light-text: #252525;
+  --color-light-text-body: #444444;
+  --color-light-text-secondary: #5f5f5f;
+  --color-light-muted: #6c6c6c;
+  --color-light-accent: #191919;
   --color-light-success: #2d6b22;
-  --color-light-border: #d5d4d1;
+  --color-light-border: #d4d4d4;
   --space-xs: 0.25rem;
   --space-sm: 0.5rem;
   --space-md: 0.875rem;

package/src/design/board.ts

@@ -10,6 +10,7 @@ import {
 import { join, relative, sep } from 'node:path'
 import { DESIGN_DOCUMENT } from '@/design/regen'
 import { renderDesignDoc } from '@/design/render'
+import { NEUTRAL_TOKENS } from '@/design/neutral'
 import { colorValue } from '@/design/tokens'
 import { parseFrontmatter, readField } from '@/indexes/frontmatter'
 import { recordDir } from '@/record-root'
@@ -131,6 +132,16 @@ const SCROLLSPY_SCRIPT = [
   '})();</script>',
 ].join('')
 
+/**
+ * A role's toolkit value, falling back to the neutral install base's where the
+ * toolkit record drops the role, so the board never restates a hex of its own.
+ */
+function roleColor(role: string): string {
+  const value = colorValue(role) ?? colorValue(role, NEUTRAL_TOKENS)
+  if (value === undefined) throw new Error(`No design token declares ${role}`)
+  return value
+}
+
 /**
  * The same brand mark `src/design/render.ts` embeds in its own preview,
  * colored with the dark accent since the board's default chrome is dark
@@ -139,7 +150,7 @@ const SCROLLSPY_SCRIPT = [
  * copies rather than a shared icon helper for the first second caller.
  */
 function faviconLink(): string {
-  const color = colorValue('accent') ?? '#e0724b'
+  const color = roleColor('accent')
   const href = `data:image/svg+xml,${encodeURIComponent(
     `<svg xmlns="http://www.w3.org/2000/svg" viewBox="10 10 80 80"><path d="M34,20 L15,28 L15,72 L34,80 Z M66,20 L85,28 L85,72 L66,80 Z" fill="${color}" /><rect x="44" y="15" width="12" height="70" rx="2" fill="${color}" /></svg>`,
   )}`
@@ -232,12 +243,12 @@ ${SCROLLSPY_SCRIPT}
 }
 
 function panelPage(title: string, body: string): string {
-  const accent = colorValue('light-accent') ?? '#a4471c'
-  const surface = colorValue('light-surface') ?? '#f4efe6'
-  const border = colorValue('light-border') ?? '#e4dcd0'
-  const muted = colorValue('light-muted') ?? '#726b62'
-  const text = colorValue('light-text') ?? '#1a1815'
-  const background = colorValue('light-background') ?? '#faf7f2'
+  const accent = roleColor('light-accent')
+  const surface = roleColor('light-surface')
+  const border = roleColor('light-border')
+  const muted = roleColor('light-muted')
+  const text = roleColor('light-text')
+  const background = roleColor('light-background')
 
   return `<!doctype html>
 <html lang="en">