@erclx/canon 4.62.0 → 4.62.1

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

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

package/package.json

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

package/src/migrate/scratch-evidence.ts

@@ -98,14 +98,18 @@ function citationPattern(folder: string): RegExp {
   )
 }
 
-/** One promoted folder's citation pattern, paired with its replacement. */
-const REWRITES: readonly {
+interface Rewrite {
   readonly pattern: RegExp
   readonly folder: string
-}[] = PROMOTED_FOLDERS.map((folder) => ({
-  pattern: citationPattern(folder),
-  folder,
-}))
+}
+
+/** One rewrite per folder, paired with its citation pattern. */
+function buildRewrites(folders: readonly string[]): readonly Rewrite[] {
+  return folders.map((folder) => ({
+    pattern: citationPattern(folder),
+    folder,
+  }))
+}
 
 /**
  * Marks a line naming a promoted folder's old path on purpose, the same
@@ -125,40 +129,43 @@ function isKept(lines: readonly string[], index: number): boolean {
   return above >= 0 && (lines[above]?.includes(KEEP_MARKER) ?? false)
 }
 
-function rewriteLine(line: string): string {
-  return REWRITES.reduce(
+function rewriteLine(line: string, rewrites: readonly Rewrite[]): string {
+  return rewrites.reduce(
     (current, { pattern, folder }) =>
       current.replace(pattern, `.canon/review/evidence/${folder}`),
     line,
   )
 }
 
-/**
- * Rewrites every unmarked citation of a promoted folder into its destination
- * under `.canon/review/evidence/`, absolute regardless of how the source
- * citation was spelled. A file naming no promoted folder returns
- * byte-identical, and a marked line is returned unchanged.
- */
-export function rewriteScratchEvidence(text: string): string {
-  const lines = text.split('\n')
-  return lines
-    .map((line, index) => (isKept(lines, index) ? line : rewriteLine(line)))
-    .join('\n')
+interface RewriteOutcome {
+  readonly text: string
+  readonly count: number
 }
 
-/** How many unmarked citations `rewriteScratchEvidence` would change. */
-export function countScratchEvidenceCitations(text: string): number {
+/**
+ * Rewrites every unmarked citation of a folder `rewrites` covers into its
+ * destination under `.canon/review/evidence/`, absolute regardless of how the
+ * source citation was spelled, counting each as it goes. A file naming no
+ * such folder returns byte-identical with a count of zero, and a marked line
+ * is returned unchanged and uncounted.
+ */
+function applyRewrites(
+  text: string,
+  rewrites: readonly Rewrite[],
+): RewriteOutcome {
   const lines = text.split('\n')
   let count = 0
 
-  for (const [index, line] of lines.entries()) {
-    if (isKept(lines, index)) continue
-    for (const { pattern } of REWRITES) {
+  const rewritten = lines.map((line, index) => {
+    if (isKept(lines, index)) return line
+
+    for (const { pattern } of rewrites) {
       count += [...line.matchAll(pattern)].length
     }
-  }
+    return rewriteLine(line, rewrites)
+  })
 
-  return count
+  return { text: rewritten.join('\n'), count }
 }
 
 /** The files under every `BACKED_FOLDERS` entry, archives included. */
@@ -261,17 +268,17 @@ export function planScratchEvidence(
   sources: readonly ScratchEvidenceSource[],
 ): ScratchEvidencePlan {
   const { moves, collisions } = planFolderMoves(root)
+  const folders = PROMOTED_FOLDERS.filter(
+    (folder) => !collisions.includes(destinationPath(root, folder)),
+  )
+  const rewrites = buildRewrites(folders)
   const entries: CitationEntry[] = []
 
   for (const source of sources) {
-    const rewritten = countScratchEvidenceCitations(source.text)
-    if (rewritten === 0) continue
-
-    entries.push({
-      path: source.path,
-      text: rewriteScratchEvidence(source.text),
-      rewritten,
-    })
+    const { text, count } = applyRewrites(source.text, rewrites)
+    if (count === 0) continue
+
+    entries.push({ path: source.path, text, rewritten: count })
   }
 
   return {

package/tooling/astro/configs/eslint.config.js

@@ -26,7 +26,7 @@ export default defineConfig([
   ...tseslint.configs.recommended,
   ...astro.configs.recommended,
   {
-    files: ['**/*.{ts,tsx,js,jsx}'],
+    files: ['**/*.{ts,tsx,js,jsx,astro}'],
     plugins: {
       'simple-import-sort': simpleImportSort,
       'check-file': checkFile,

package/tooling/astro/reference.md

@@ -21,7 +21,7 @@ The astro stack covers Astro + TypeScript projects: content sites, marketing sit
 - `vitest.config.ts`: uses `getViteConfig` from `astro/config` (not `mergeConfig`). jsdom, globals, setup file, `passWithNoTests: true`, v8 coverage, `**/*.astro` in coverage excludes.
 - `playwright.config.ts`: all browsers, `webServer` runs `bun run build && bun run preview` on port `4321` plus `WORKTREE_PORT_OFFSET`, `reuseExistingServer: false`. Astro's dev/prod gap is wide (MDX, island hydration, asset optimization), so E2E always tests the built `dist/`. `DIST_PREBUILT` set in the environment drops the `build` half, running `bun run preview` alone against a `dist/` a prior CI job already produced.
 - `tsconfig.json`: extends `astro/tsconfigs/strict`, adds `skipLibCheck`, `vitest/globals` and `@testing-library/jest-dom` in types, `@/` paths.
-- `eslint.config.js`: overrides the web layer. Adds `eslint-plugin-astro` (`.astro` parser via `astro-eslint-parser`). React-hooks scoped to `.jsx`/`.tsx` only (`.astro` is not React). `src/pages/**` exempt from filename and folder naming conventions because Astro's file-based routing ties names to URL segments.
+- `eslint.config.js`: overrides the web layer. Adds `eslint-plugin-astro` (`.astro` parser via `astro-eslint-parser`). React-hooks scoped to `.jsx`/`.tsx` only (`.astro` is not React). The shared block's `files` selector includes `.astro`, so `check-file/filename-naming-convention` reaches `.ts`, `.tsx`, and `.astro` under `KEBAB_CASE`, overriding Astro's own PascalCase component convention deliberately, on the ground that a component's name in markup comes from the import binding rather than the filename. `.js` and `.jsx` stay out of the rule's own pattern, matching `web` and `nextjs`. `check-file/folder-naming-convention`'s `src/**/!(__tests__|pages)` pattern reaches no folder under a real ESLint run, `.astro`-only or otherwise. See `.claude/context/tooling.md` for the measurement.
 
 ## Typecheck