@erclx/canon 4.30.1 → 4.30.2

package/claude/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "canon",
   "description": "Automated governance, versioning, and discovery tools for Claude Code.",
-  "version": "4.30.1",
+  "version": "4.30.2",
   "author": {
     "name": "Eric Le",
     "url": "https://github.com/erclx"

package/docs/agents/capture.md

@@ -5,7 +5,7 @@ description: Rendering HTML sources to PNG, what the command asserts about fonts
 
 # Capture
 
-`canon capture [source] --selector <sel>` renders HTML sources to PNG, which is how a generated documentation image is rebuilt from the markup it was generated out of. The source defaults to `assets/`, where a directory expands to every `.html` directly inside it, so adding a capture means dropping a file beside the first one and running the same command.
+`canon capture [source] --selector <sel>` renders HTML sources to PNG, which is how a generated documentation image is rebuilt from the markup it was generated out of. The source defaults to `assets/`, where a directory expands to every `.html` directly inside it, so adding a capture means dropping a file beside the first one and running the same command. That default names this repository's own folder rather than a convention every target shares, and it stays because a missing one still refuses loud, naming the argument, rather than failing silently.
 
 ```bash
 canon capture --selector .window
@@ -38,7 +38,7 @@ Each source renders at `deviceScaleFactor` 2 with a transparent background, and
 
 What is asserted is the font. The command reads the first family the captured element declares and fails when the browser did not resolve it, because a fallback face rewraps the block and silently changes the output. Sources therefore name a real font rather than relying on `monospace`. A source that cannot render reports its own line and exits 1 without dropping the rest of the batch.
 
-A run refuses before it reads anything else when `--selector` is absent, naming the flag and pointing at `--help`. Ordering it first is what keeps the message about the invocation: the source defaults to `assets`, so checking that first would answer `assets not found` from whatever directory the caller happened to be in and say nothing about the flag that was actually missing.
+A run refuses before it reads anything else when `--selector` is absent, naming the flag and pointing at `--help`. Ordering it first is what keeps the message about the invocation: the source defaults to `assets`, so checking that first would answer `assets not found` from whatever directory the caller happened to be in and say nothing about the flag that was actually missing. That source default is a deliberate choice rather than one inherited from some general convention: a missing folder still refuses loud, naming the argument, once the flag check passes, where the selector's dropped default cropped the wrong region in silence.
 
 The browser binary installs separately from the package. A first run does `bunx playwright install chromium` once, and a run that cannot launch one reports the engine's own remediation inside the frame and exits 1 rather than escaping as a stack trace.
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@erclx/canon",
   "type": "module",
-  "version": "4.30.1",
+  "version": "4.30.2",
   "description": "Infrastructure and quality tooling for developer workflows",
   "license": "MIT",
   "bin": {

package/src/commands/capture.ts

@@ -16,6 +16,11 @@ import {
   pipeOutput,
 } from '@/ui'
 
+/**
+ * Kept rather than removed, unlike the selector's default that came out for
+ * silently cropping the wrong region: a missing folder still refuses loud,
+ * naming the argument that failed to resolve.
+ */
 const DEFAULT_SOURCE = 'assets'
 
 /**

package/src/gov/citations.ts

@@ -6,6 +6,7 @@ import { isMarked } from '@/exempt-marker'
 import { gitEnv } from '@/git-env'
 import { listRuleFiles } from '@/gov/payload'
 import { parseFrontmatter } from '@/indexes/frontmatter'
+import { findBacktickSpans } from '@/markdown/backticks'
 
 /**
  * The inline token exempting one line from this sweep, shaped on the
@@ -114,13 +115,6 @@ const FENCE = /^\s*(?:```|~~~)/
 
 const FRONTMATTER_DELIMITER = /^---\s*$/
 
-/**
- * A backticked span, which is the only carrier a rule writes a citation in. No
- * rule in either corpus uses a markdown link, and matching running prose would
- * report every sentence that happens to name a file.
- */
-const BACKTICKED = /`([^`\n]+)`/g
-
 /**
  * The verb form, with the name captured. A leading letter or digit is required,
  * which is what leaves `canon standards <name>` unmatched: that line teaches the
@@ -254,8 +248,10 @@ export function collectCitations(text: string): RawCitation[] {
       })
     }
 
-    for (const match of line.matchAll(BACKTICKED)) {
-      const span = match[1] ?? ''
+    // A backticked span is the only carrier a rule writes a citation in. No
+    // rule in either corpus uses a markdown link, and matching running prose
+    // would report every sentence that happens to name a file.
+    for (const { content: span } of findBacktickSpans(line)) {
       const form = classifySpan(span)
       if (form === undefined) continue
       found.push({ line: index + 1, form, cited: span, preview, marked })

package/src/markdown/backticks.ts

@@ -0,0 +1,56 @@
+/** One code span, paired by matching backtick-run length rather than count. */
+export interface BacktickSpan {
+  /** Index of the opening run's first backtick. */
+  readonly start: number
+  /** Index one past the closing run's last backtick. */
+  readonly end: number
+  /** The text between the two runs, delimiters excluded. */
+  readonly content: string
+}
+
+/**
+ * Every code span in the text, pairing a run of backticks only with the next
+ * run of the same length.
+ *
+ * A single-backtick regex reads a doubled delimiter, such as
+ * ``` ``git status`` ```, as two unrelated single backticks: it opens on the
+ * second backtick of the pair, closes on the first backtick of the closing
+ * pair, and leaves one backtick before and after the span unconsumed. Every
+ * scan after that treats a stray leftover backtick as an opener, which
+ * absorbs the next real span's opening delimiter as its closer and drops the
+ * path inside past it. Pairing by run length rather than by single backtick
+ * keeps a doubled delimiter closed by a doubled delimiter, so nothing after
+ * it loses its pairing.
+ *
+ * An opening run with no same-length run after it is not a delimiter, per
+ * CommonMark, so it is left as literal text and the scan resumes at the next
+ * run rather than backtracking into the unmatched one.
+ */
+export function findBacktickSpans(text: string): BacktickSpan[] {
+  const runs = [...text.matchAll(/`+/g)].map((match) => ({
+    start: match.index ?? 0,
+    length: match[0].length,
+  }))
+
+  const spans: BacktickSpan[] = []
+  let i = 0
+  while (i < runs.length) {
+    const open = runs[i]
+    const closeIndex = runs.findIndex(
+      (run, index) => index > i && run.length === open.length,
+    )
+    if (closeIndex === -1) {
+      i += 1
+      continue
+    }
+    const close = runs[closeIndex]
+    spans.push({
+      start: open.start,
+      end: close.start + close.length,
+      content: text.slice(open.start + open.length, close.start),
+    })
+    i = closeIndex + 1
+  }
+
+  return spans
+}

package/src/pr/paths.ts

@@ -1,3 +1,5 @@
+import { findBacktickSpans } from '@/markdown/backticks'
+
 /**
  * The heading whose bullets state what a branch changed.
  *
@@ -114,63 +116,6 @@ function isDottedNumber(span: string): boolean {
   return /^\d+(?:\.\d+)+$/.test(segment)
 }
 
-/** One code span, paired by matching backtick-run length rather than count. */
-interface BacktickSpan {
-  /** Index of the opening run's first backtick. */
-  readonly start: number
-  /** Index one past the closing run's last backtick. */
-  readonly end: number
-  /** The text between the two runs, delimiters excluded. */
-  readonly content: string
-}
-
-/**
- * Every code span in a bullet, pairing a run of backticks only with the next
- * run of the same length.
- *
- * A single-backtick regex reads a doubled delimiter, such as
- * ``` ``git status`` ```, as two unrelated single backticks: it opens on the
- * second backtick of the pair, closes on the first backtick of the closing
- * pair, and leaves one backtick before and after the span unconsumed. Every
- * scan after that treats a stray leftover backtick as an opener, which
- * absorbs the next real span's opening delimiter as its closer and drops the
- * path inside past it. Pairing by run length rather than by single backtick
- * keeps a doubled delimiter closed by a doubled delimiter, so nothing after
- * it loses its pairing.
- *
- * An opening run with no same-length run after it is not a delimiter, per
- * CommonMark, so it is left as literal text and the scan resumes at the next
- * run rather than backtracking into the unmatched one.
- */
-function findBacktickSpans(text: string): BacktickSpan[] {
-  const runs = [...text.matchAll(/`+/g)].map((match) => ({
-    start: match.index ?? 0,
-    length: match[0].length,
-  }))
-
-  const spans: BacktickSpan[] = []
-  let i = 0
-  while (i < runs.length) {
-    const open = runs[i]
-    const closeIndex = runs.findIndex(
-      (run, index) => index > i && run.length === open.length,
-    )
-    if (closeIndex === -1) {
-      i += 1
-      continue
-    }
-    const close = runs[closeIndex]
-    spans.push({
-      start: open.start,
-      end: close.start + close.length,
-      content: text.slice(open.start + open.length, close.start),
-    })
-    i = closeIndex + 1
-  }
-
-  return spans
-}
-
 /** Blanks every backticked span so a cue search never fires inside one. */
 function maskSpans(text: string): string {
   let out = text