docguard-cli 0.12.0 → 0.13.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -17,7 +17,7 @@
17
17
  - [What is DocGuard?](#what-is-docguard)
18
18
  - [Quick Start](#-quick-start)
19
19
  - [Spec Kit Integration](#-spec-kit-integration)
20
- - [Commands](#-commands)
20
+ - [Usage](#usage)
21
21
  - [Validators](#-validators)
22
22
  - [Templates](#-templates)
23
23
  - [AI Agent Support](#-ai-agent-support)
@@ -343,7 +343,7 @@ DocGuard provides AI agent slash commands for integrated workflows. Installed au
343
343
 
344
344
  | Command | What It Does |
345
345
  |:--------|:-------------|
346
- | `/docguard.guard` | Run quality validation — check all 21 validators |
346
+ | `/docguard.guard` | Run quality validation — check all 22 validators |
347
347
  | `/docguard.review` | Analyze doc quality and suggest improvements |
348
348
  | `/docguard.fix` | Generate targeted fix prompts for specific issues |
349
349
  | `/docguard.score` | Show CDD maturity score with category breakdown |
@@ -21,6 +21,7 @@ import { c } from '../shared.mjs';
21
21
  import { computeApiSurfaceDrift } from '../validators/api-surface.mjs';
22
22
  import { removeEndpoints, hasGeneratedMarker } from '../writers/api-reference.mjs';
23
23
  import { applyMechanicalFixes } from '../writers/mechanical.mjs';
24
+ import { loadFixMemory } from '../writers/fix-memory.mjs';
24
25
  import { runGuardInternal } from './guard.mjs';
25
26
 
26
27
  const API_DOC = 'docs-canonical/API-REFERENCE.md';
@@ -281,6 +282,55 @@ export function applyAllMechanicalFixes(projectDir, config, { force = false } =
281
282
  return { applied, skipped, total: fixes.length };
282
283
  }
283
284
 
285
+ /**
286
+ * M-2 — `docguard fix --history` shows the audit log of mechanical fixes
287
+ * that have been applied to this project. Reads `.docguard/fixed.json`
288
+ * and pretty-prints (or emits JSON when --format json).
289
+ */
290
+ function runHistoryMode(projectDir, flags) {
291
+ const mem = loadFixMemory(projectDir);
292
+ const isJson = flags.format === 'json';
293
+
294
+ if (isJson) {
295
+ console.log(JSON.stringify(mem, null, 2));
296
+ return;
297
+ }
298
+
299
+ if (mem.entries.length === 0) {
300
+ console.log(`${c.bold}🗂 DocGuard Fix History${c.reset}`);
301
+ console.log(`${c.dim} No fixes recorded yet. Run \`docguard fix --write\` to start the audit log.${c.reset}`);
302
+ return;
303
+ }
304
+
305
+ console.log(`${c.bold}🗂 DocGuard Fix History${c.reset} ${c.dim}(${mem.entries.length} entries, newest first)${c.reset}\n`);
306
+
307
+ // Group by date for readability
308
+ const byDate = new Map();
309
+ for (const e of mem.entries) {
310
+ const day = (e.appliedAt || '').slice(0, 10);
311
+ if (!byDate.has(day)) byDate.set(day, []);
312
+ byDate.get(day).push(e);
313
+ }
314
+
315
+ // Show the most recent N days (cap output at 20 entries)
316
+ let printed = 0;
317
+ for (const [day, dayEntries] of byDate) {
318
+ if (printed >= 20) break;
319
+ console.log(` ${c.cyan}${day}${c.reset} ${c.dim}(${dayEntries.length} fix${dayEntries.length > 1 ? 'es' : ''})${c.reset}`);
320
+ for (const e of dayEntries.slice(0, 5)) {
321
+ if (printed >= 20) break;
322
+ const time = (e.appliedAt || '').slice(11, 16);
323
+ console.log(` ${c.dim}${time}${c.reset} ${e.type} → ${c.cyan}${e.file}${c.reset} ${c.dim}${e.summary || ''}${c.reset}`);
324
+ printed++;
325
+ }
326
+ if (dayEntries.length > 5) console.log(` ${c.dim}... ${dayEntries.length - 5} more on this day${c.reset}`);
327
+ }
328
+
329
+ if (mem.entries.length > 20) {
330
+ console.log(`\n ${c.dim}... ${mem.entries.length - 20} older entries. Use ${c.cyan}--format json${c.dim} for the full log.${c.reset}`);
331
+ }
332
+ }
333
+
284
334
  function runWriteMode(projectDir, config, flags) {
285
335
  const isJson = flags.format === 'json';
286
336
  const { applied, skipped, total } = applyAllMechanicalFixes(projectDir, config, { force: flags.force });
@@ -321,6 +371,11 @@ export function runFix(projectDir, config, flags) {
321
371
  const autoFix = flags.auto || false;
322
372
  const specificDoc = flags.doc || null;
323
373
 
374
+ // M-2: --history shows the audit trail of past mechanical fixes.
375
+ if (flags.history) {
376
+ return runHistoryMode(projectDir, flags);
377
+ }
378
+
324
379
  // --write: deterministically APPLY mechanical fixes (no LLM). Currently:
325
380
  // remove API-REFERENCE.md endpoints the OpenAPI spec confirms no longer exist.
326
381
  if (flags.write) {
@@ -10,6 +10,7 @@
10
10
  import { c, resolveSeverity } from '../shared.mjs';
11
11
  import { detectAgentMode, isSpecKitInitialized } from '../ensure-skills.mjs';
12
12
  import { checkUpgradeStatus } from './upgrade.mjs';
13
+ import { changedFilesSince, isGitRepo } from '../shared-git.mjs';
13
14
  import { validateStructure, validateDocSections } from '../validators/structure.mjs';
14
15
  import { validateDrift } from '../validators/drift.mjs';
15
16
  import { validateChangelog } from '../validators/changelog.mjs';
@@ -27,6 +28,7 @@ import { validateMetricsConsistency } from '../validators/metrics-consistency.mj
27
28
  import { validateDocsCoverage } from '../validators/docs-coverage.mjs';
28
29
  import { validateDocQuality } from '../validators/doc-quality.mjs';
29
30
  import { validateCrossReferences } from '../validators/cross-reference.mjs';
31
+ import { validateGeneratedStaleness } from '../validators/generated-staleness.mjs';
30
32
  import { validateTodoTracking } from '../validators/todo-tracking.mjs';
31
33
  import { validateSchemaSync } from '../validators/schema-sync.mjs';
32
34
  import { validateSpecKitIntegration } from '../scanners/speckit.mjs';
@@ -103,6 +105,7 @@ export function runGuardInternal(projectDir, config) {
103
105
  { key: 'schemaSync', name: 'Schema-Sync', fn: () => validateSchemaSync(projectDir, config) },
104
106
  { key: 'specKit', name: 'Spec-Kit', fn: () => validateSpecKitIntegration(projectDir, config) },
105
107
  { key: 'crossReference', name: 'Cross-Reference', fn: () => validateCrossReferences(projectDir, config) },
108
+ { key: 'generatedStaleness', name: 'Generated-Staleness', fn: () => validateGeneratedStaleness(projectDir, config) },
106
109
  // Metrics-Consistency runs post-loop (needs guard results)
107
110
  ];
108
111
 
@@ -197,7 +200,7 @@ function liteValidatorsConfig() {
197
200
  'structure', 'docsSync', 'drift', 'changelog', 'testSpec', 'environment',
198
201
  'security', 'architecture', 'freshness', 'traceability', 'docsDiff',
199
202
  'apiSurface', 'metadataSync', 'docsCoverage', 'docQuality', 'todoTracking',
200
- 'schemaSync', 'specKit', 'crossReference', 'metricsConsistency',
203
+ 'schemaSync', 'specKit', 'crossReference', 'generatedStaleness', 'metricsConsistency',
201
204
  ];
202
205
  const out = {};
203
206
  for (const k of all) out[k] = CHANGED_ONLY_VALIDATORS.includes(k);
@@ -212,8 +215,22 @@ export function runGuard(projectDir, config, flags) {
212
215
  // fast subset (Docs-Sync, Environment, API-Surface). Designed for husky/
213
216
  // lefthook hooks; expects to finish in under 2 seconds.
214
217
  if (flags.changedOnly) {
215
- config = { ...config, validators: liteValidatorsConfig() };
216
- console.log(`${c.cyan}⚡ docguard guard --changed-only${c.reset} ${c.dim}(running ${CHANGED_ONLY_VALIDATORS.length} fast validators only pre-commit lite mode)${c.reset}\n`);
218
+ // Compute the set of changed files since the given ref (default HEAD~1 —
219
+ // the pre-commit common case: "files changed in this commit vs the last
220
+ // committed state"). Validators that opt into `config.changedFiles` can
221
+ // scope to this list; others run normally over the whole tree.
222
+ const ref = flags.since || 'HEAD~1';
223
+ const changed = isGitRepo(projectDir) ? changedFilesSince(projectDir, ref) : [];
224
+ config = {
225
+ ...config,
226
+ validators: liteValidatorsConfig(),
227
+ changedFiles: changed,
228
+ changedSinceRef: ref,
229
+ };
230
+ const label = changed.length > 0
231
+ ? `${changed.length} file(s) changed since ${ref}`
232
+ : `no changes since ${ref} — running all ${CHANGED_ONLY_VALIDATORS.length} lite validators on full tree`;
233
+ console.log(`${c.cyan}⚡ docguard guard --changed-only${c.reset} ${c.dim}(${label})${c.reset}\n`);
217
234
  }
218
235
 
219
236
  const data = runGuardInternal(projectDir, config);
@@ -34,6 +34,48 @@ function gitChangedFiles(projectDir, since) {
34
34
  return [...new Set([...committed, ...working])];
35
35
  }
36
36
 
37
+ /**
38
+ * L-1: Map each `source: 'code'` section ID to a predicate that returns true
39
+ * when one of the changed file paths could plausibly affect it. Conservative
40
+ * by design — when in doubt we run the section's sync, never skip it.
41
+ *
42
+ * The predicates are matched against project-relative POSIX paths (the form
43
+ * `git diff --name-only` returns).
44
+ */
45
+ const SECTION_FILE_MATCHERS = {
46
+ 'tech-stack': (p) => /package\.json$|pyproject\.toml$|Cargo\.toml$|go\.mod$|pom\.xml$|Gemfile$/.test(p),
47
+ 'frontend-modules': (p) => /(^|\/)(src\/)?(stores|hooks|contexts|features)\//.test(p),
48
+ 'endpoints-table': (p) => /(^|\/)(routes|controllers|handlers|app\/api)\//.test(p)
49
+ || /\.(yaml|yml|json)$/i.test(p) && /openapi|swagger/i.test(p),
50
+ 'entities-table': (p) => /(^|\/)(models|schemas|entities)\//.test(p)
51
+ || /\.prisma$/.test(p),
52
+ 'relationships': (p) => /(^|\/)(models|schemas|entities)\//.test(p)
53
+ || /\.prisma$/.test(p),
54
+ 'screens-table': (p) => /(^|\/)(screens|pages|app)\//.test(p)
55
+ || /\.(tsx|jsx)$/.test(p),
56
+ 'flows': (p) => /(^|\/)(screens|pages|app|routes)\//.test(p),
57
+ 'integrations-table':(p) => /package\.json$|pyproject\.toml$|requirements.*\.txt$|Cargo\.toml$/.test(p),
58
+ 'features-table': (p) => /(^|\/)(features|domains)\//.test(p),
59
+ 'features': (p) => /(^|\/)(features|domains)\//.test(p),
60
+ 'env-vars-table': (p) => /\.env(\..+)?$|(^|\/)config\//.test(p)
61
+ || /\.(ts|tsx|js|jsx|mjs|py|go|rs|java|kt|rb)$/.test(p), // any code may use env
62
+ 'setup': (p) => /\.env(\..+)?$|(^|\/)config\//.test(p),
63
+ };
64
+
65
+ /**
66
+ * Decide whether a given code-truth section should be re-synced based on the
67
+ * set of changed files. Returns true when:
68
+ * - changedFiles is null/empty (no scope info → sync everything), OR
69
+ * - any changed file matches the section's known source patterns, OR
70
+ * - the section has no matcher registered (unknown → conservative: sync)
71
+ */
72
+ function sectionTouchedByChanges(sectionId, changedFiles) {
73
+ if (!changedFiles || changedFiles.length === 0) return true;
74
+ const matcher = SECTION_FILE_MATCHERS[sectionId];
75
+ if (!matcher) return true; // unknown section → don't accidentally skip it
76
+ return changedFiles.some(matcher);
77
+ }
78
+
37
79
  export function runSync(projectDir, config, flags) {
38
80
  const plan = buildMemoryPlan(projectDir, config);
39
81
  const apply = !!flags.write;
@@ -63,6 +105,14 @@ export function runSync(projectDir, config, flags) {
63
105
  const existing = getSection(content, sec.id);
64
106
  if (!existing) continue; // sync refreshes sections that already exist
65
107
  if (existing.body.trim() === String(sec.body).trim()) continue; // already current
108
+ // L-1: when --since is provided, only update sections whose underlying
109
+ // source files appear in the changed set. Avoids spurious updates when
110
+ // the section's CONTENT would naturally drift (e.g. timestamp-driven
111
+ // counters) but no real source file changed.
112
+ if (changed !== null && !sectionTouchedByChanges(sec.id, changed)) {
113
+ skipped.push({ doc: doc.path, reason: `section ${sec.id} unchanged since ${flags.since} (no underlying source files in diff)` });
114
+ continue;
115
+ }
66
116
  codeSectionChanged = true;
67
117
  updates.push({ doc: doc.path, section: sec.id, status: apply ? 'updated' : 'stale' });
68
118
  if (apply) { content = replaceSection(content, sec.id, sec.body).content; docChanged = true; }
@@ -84,7 +84,112 @@ const TRACE_MAP = {
84
84
  },
85
85
  };
86
86
 
87
+ /**
88
+ * L-2 / S-3 — Reverse trace: given a code file, find which canonical doc
89
+ * sections mention it. Mirror of the forward trace (doc → code).
90
+ *
91
+ * Match strategies (each yields a hit):
92
+ * 1. Direct path match: full project-relative path appears in doc text.
93
+ * 2. Basename match: e.g. `users.ts` appears (covers cases where the doc
94
+ * refers to the file by name without the full path).
95
+ * 3. Module name match: file stem (e.g. `users`) appears as a fenced
96
+ * `code` reference. Tighter than 2 — avoids matching common nouns.
97
+ *
98
+ * Output: one line per (doc, match-line) pair, with the surrounding context.
99
+ */
100
+ export function runTraceReverse(projectDir, config, flags) {
101
+ const target = flags.args && flags.args[0];
102
+ if (!target) {
103
+ console.error(`${c.red}Error: trace --reverse requires a target path${c.reset}`);
104
+ console.log(`Usage: ${c.cyan}docguard trace --reverse <code-path>${c.reset}`);
105
+ console.log(`Example: ${c.cyan}docguard trace --reverse src/routes/users.ts${c.reset}`);
106
+ process.exit(1);
107
+ }
108
+
109
+ // Suppress chrome in JSON mode so stdout stays parseable.
110
+ const isJson = flags.format === 'json';
111
+ if (!isJson) {
112
+ console.log(`${c.bold}🔄 DocGuard Trace (reverse) — ${target}${c.reset}`);
113
+ console.log(`${c.dim} Finding canonical doc sections that reference this file...${c.reset}\n`);
114
+ }
115
+
116
+ const docsDir = resolve(projectDir, 'docs-canonical');
117
+ if (!existsSync(docsDir)) {
118
+ if (isJson) {
119
+ console.log(JSON.stringify({ target, matches: [], error: 'no docs-canonical/ directory' }, null, 2));
120
+ } else {
121
+ console.log(` ${c.yellow}No docs-canonical/ directory found.${c.reset}`);
122
+ }
123
+ return;
124
+ }
125
+
126
+ // Normalize the target path: strip leading ./
127
+ const normalized = target.replace(/^\.\//, '');
128
+ const base = basename(normalized);
129
+ const stem = base.replace(/\.[^.]+$/, '');
130
+
131
+ const matches = []; // { doc, line, content, kind }
132
+ for (const f of readdirSync(docsDir)) {
133
+ if (!f.endsWith('.md')) continue;
134
+ const docPath = resolve(docsDir, f);
135
+ let content;
136
+ try { content = readFileSync(docPath, 'utf-8'); } catch { continue; }
137
+ const lines = content.split('\n');
138
+ for (let i = 0; i < lines.length; i++) {
139
+ const line = lines[i];
140
+ let kind = null;
141
+ if (line.includes(normalized)) kind = 'path';
142
+ else if (line.includes(base)) kind = 'basename';
143
+ else if (new RegExp(`\`${escapeRegex(stem)}\``).test(line)) kind = 'module';
144
+ if (kind) {
145
+ matches.push({ doc: f, line: i + 1, content: line.trim(), kind });
146
+ }
147
+ }
148
+ }
149
+
150
+ if (flags.format === 'json') {
151
+ console.log(JSON.stringify({
152
+ target: normalized,
153
+ matches,
154
+ timestamp: new Date().toISOString(),
155
+ }, null, 2));
156
+ return;
157
+ }
158
+
159
+ if (matches.length === 0) {
160
+ console.log(` ${c.yellow}⚠️ No canonical doc references "${normalized}"${c.reset}`);
161
+ console.log(` ${c.dim}Consider documenting this file in docs-canonical/ARCHITECTURE.md or DATA-MODEL.md${c.reset}`);
162
+ return;
163
+ }
164
+
165
+ // Group by doc for readable output
166
+ const byDoc = new Map();
167
+ for (const m of matches) {
168
+ if (!byDoc.has(m.doc)) byDoc.set(m.doc, []);
169
+ byDoc.get(m.doc).push(m);
170
+ }
171
+
172
+ console.log(` ${c.green}✅ ${matches.length} reference(s) across ${byDoc.size} doc(s):${c.reset}\n`);
173
+ for (const [doc, hits] of byDoc) {
174
+ console.log(` ${c.cyan}${doc}${c.reset} ${c.dim}(${hits.length} hit${hits.length > 1 ? 's' : ''})${c.reset}`);
175
+ for (const h of hits.slice(0, 5)) {
176
+ const trimmed = h.content.length > 80 ? h.content.slice(0, 77) + '…' : h.content;
177
+ console.log(` ${c.dim}L${h.line} [${h.kind}]${c.reset} ${trimmed}`);
178
+ }
179
+ if (hits.length > 5) console.log(` ${c.dim}... ${hits.length - 5} more${c.reset}`);
180
+ }
181
+ }
182
+
183
+ function escapeRegex(s) {
184
+ return s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
185
+ }
186
+
87
187
  export function runTrace(projectDir, config, flags) {
188
+ // L-2: dispatch to reverse mode when --reverse is set.
189
+ if (flags.reverse) {
190
+ return runTraceReverse(projectDir, config, flags);
191
+ }
192
+
88
193
  console.log(`${c.bold}🔗 DocGuard Trace — ${config.projectName}${c.reset}`);
89
194
  console.log(`${c.dim} Directory: ${projectDir}${c.reset}`);
90
195
  console.log(`${c.dim} Generating requirements traceability matrix...${c.reset}\n`);
package/cli/docguard.mjs CHANGED
@@ -381,6 +381,15 @@ async function main() {
381
381
  flags.apply = true;
382
382
  } else if (args[i] === '--changed-only') {
383
383
  flags.changedOnly = true;
384
+ } else if (args[i] === '--reverse') {
385
+ flags.reverse = true;
386
+ } else if (args[i] === '--history') {
387
+ flags.history = true;
388
+ } else if (!args[i].startsWith('--') && i > 0) {
389
+ // Positional args go into flags.args for commands that take them (e.g.
390
+ // `docguard trace --reverse <path>`). Skip the command itself (i === 0).
391
+ flags.args = flags.args || [];
392
+ flags.args.push(args[i]);
384
393
  } else if (args[i] === '--doc' && args[i + 1]) {
385
394
  flags.doc = args[i + 1];
386
395
  i++;
Binary file
@@ -183,7 +183,15 @@ function collectCanonicalDocs(projectDir) {
183
183
  }
184
184
  } catch {}
185
185
  }
186
- for (const f of ['AGENTS.md', 'CHANGELOG.md', 'DRIFT-LOG.md', 'ROADMAP.md', 'README.md']) {
186
+ // Standard root-level docs that are commonly cross-referenced. We index
187
+ // them so links like [CONTRIBUTING.md](CONTRIBUTING.md#some-section) can
188
+ // resolve. The list is conservative — adding everything would pull in
189
+ // boilerplate (LICENSE, NOTICE) that doesn't have meaningful headings.
190
+ for (const f of [
191
+ 'README.md', 'AGENTS.md', 'CHANGELOG.md', 'DRIFT-LOG.md', 'ROADMAP.md',
192
+ 'CONTRIBUTING.md', 'CODE_OF_CONDUCT.md', 'SECURITY.md',
193
+ 'PHILOSOPHY.md', 'STANDARD.md', 'COMPARISONS.md',
194
+ ]) {
187
195
  const p = resolve(projectDir, f);
188
196
  if (existsSync(p)) docs.push(p);
189
197
  }
@@ -80,6 +80,17 @@ export function validateDocsSync(projectDir, config) {
80
80
  return results; // No canonical docs to check against
81
81
  }
82
82
 
83
+ // N-1: When the guard runs in --changed-only mode, config.changedFiles is
84
+ // populated with paths that changed since the given ref. We use it to scope
85
+ // route/service checks to ONLY the files actually changed — turning a
86
+ // whole-tree scan into a surgical check. If the list is empty (no changes,
87
+ // or git unavailable), we fall back to scanning everything.
88
+ const changedSet = config && Array.isArray(config.changedFiles) && config.changedFiles.length > 0
89
+ ? new Set(config.changedFiles)
90
+ : null;
91
+ // Closure: true if the given relative path should be considered.
92
+ const inScope = (relPath) => !changedSet || changedSet.has(relPath);
93
+
83
94
  // Find route/API files (monorepo-aware) and check they're mentioned in docs.
84
95
  // Note: bare 'api' is intentionally excluded — it collides with frontend
85
96
  // API client conventions (src/api/client.ts). Backend routes use
@@ -95,6 +106,8 @@ export function validateDocsSync(projectDir, config) {
95
106
  const relPath = file.replace(projectDir + '/', '');
96
107
  if (isTestFile(relPath)) continue;
97
108
  if (!isValidRouteFile(relPath)) continue;
109
+ // N-1: skip files outside the --changed-only scope.
110
+ if (!inScope(relPath)) continue;
98
111
 
99
112
  results.total++;
100
113
  const name = basename(file, ext);
@@ -118,6 +131,8 @@ export function validateDocsSync(projectDir, config) {
118
131
 
119
132
  const relPath = file.replace(projectDir + '/', '');
120
133
  if (isTestFile(relPath)) continue;
134
+ // N-1: skip files outside the --changed-only scope.
135
+ if (!inScope(relPath)) continue;
121
136
 
122
137
  results.total++;
123
138
  const name = basename(file, ext);
@@ -9,6 +9,7 @@
9
9
  import { existsSync, readdirSync, statSync } from 'node:fs';
10
10
  import { resolve, join, extname } from 'node:path';
11
11
  import { execSync, execFileSync } from 'node:child_process';
12
+ import { getLastCommitDate } from '../shared-git.mjs';
12
13
 
13
14
  const IGNORE_DIRS = new Set([
14
15
  'node_modules', '.git', '.next', 'dist', 'build',
@@ -21,16 +22,10 @@ const IGNORE_DIRS = new Set([
21
22
  * Returns null if the file isn't tracked or git isn't available.
22
23
  */
23
24
  function getLastGitDate(filePath, dir) {
24
- try {
25
- const result = execFileSync(
26
- 'git',
27
- ['log', '-1', '--format=%aI', '--', filePath],
28
- { cwd: dir, encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] }
29
- ).trim();
30
- return result ? new Date(result) : null;
31
- } catch {
32
- return null;
33
- }
25
+ // Delegate to shared-git so rename history (--follow) is preserved.
26
+ // Without --follow, a `git mv` resets the file's "last commit date" and
27
+ // the Freshness counter silently misses drift introduced by the rename.
28
+ return getLastCommitDate(dir, filePath);
34
29
  }
35
30
 
36
31
  /**
@@ -0,0 +1,97 @@
1
+ /**
2
+ * Generated-Doc Staleness Validator — M-1 / S-7
3
+ *
4
+ * Re-runs the memory-plan scanner and compares each `source=code` section's
5
+ * expected body against what's actually committed in the canonical docs.
6
+ * Flags sections where the doc says one thing but the scanner produces
7
+ * another — that's drift, and it means either:
8
+ * (a) Code changed and `docguard sync --write` hasn't been run, OR
9
+ * (b) Someone hand-edited a code-truth section (which shouldn't happen —
10
+ * human prose belongs in source=human sections).
11
+ *
12
+ * Why this matters: K-1's auto-fix Action runs `fix --write` (mechanical
13
+ * fixes) but doesn't run `sync --write` (memory refresh). Projects that
14
+ * skip the nightly sync recipe accumulate hidden drift in source=code
15
+ * sections. This validator surfaces it as a warning so CI can catch it.
16
+ *
17
+ * Cheap: just diffs in-memory strings; no extra git or filesystem walk
18
+ * beyond what memory-plan already does.
19
+ *
20
+ * @req SC-M1-001 — flag source=code sections whose body differs from scanner output
21
+ * @req SC-M1-002 — no warning when sections match
22
+ * @req SC-M1-003 — N/A when no canonical docs exist
23
+ * @req SC-M1-004 — N/A when no source=code sections present in any doc
24
+ */
25
+
26
+ import { existsSync, readFileSync } from 'node:fs';
27
+ import { resolve, basename } from 'node:path';
28
+
29
+ import { buildMemoryPlan } from '../scanners/memory-plan.mjs';
30
+ import { getSection } from '../writers/sections.mjs';
31
+
32
+ export function validateGeneratedStaleness(projectDir, config = {}) {
33
+ const result = { errors: [], warnings: [], passed: 0, total: 0 };
34
+
35
+ // Build the canonical memory plan (what the docs SHOULD contain). If this
36
+ // fails or produces no docs, the validator is N/A.
37
+ let plan;
38
+ try {
39
+ plan = buildMemoryPlan(projectDir, config);
40
+ } catch {
41
+ return { ...result, applicable: false };
42
+ }
43
+ if (!plan || !Array.isArray(plan.docs) || plan.docs.length === 0) {
44
+ return { ...result, applicable: false };
45
+ }
46
+
47
+ // Walk each doc's source=code sections and compare against on-disk content.
48
+ let anySourceCodeSection = false;
49
+
50
+ for (const doc of plan.docs) {
51
+ const fullPath = resolve(projectDir, doc.path);
52
+ if (!existsSync(fullPath)) continue;
53
+ let content;
54
+ try { content = readFileSync(fullPath, 'utf-8'); } catch { continue; }
55
+
56
+ for (const sec of doc.sections) {
57
+ if (sec.source !== 'code') continue;
58
+ anySourceCodeSection = true;
59
+
60
+ const onDisk = getSection(content, sec.id);
61
+ // If the section isn't present in the doc at all, that's a Structure /
62
+ // Doc Sections concern — not ours. Skip without counting.
63
+ if (!onDisk) continue;
64
+
65
+ result.total++;
66
+ const expected = String(sec.body || '').trim();
67
+ const actual = String(onDisk.body || '').trim();
68
+
69
+ if (expected === actual) {
70
+ result.passed++;
71
+ continue;
72
+ }
73
+
74
+ // Compute a short diff hint — first changed line — so the warning is
75
+ // actionable without dumping the whole section.
76
+ const exp = expected.split('\n');
77
+ const act = actual.split('\n');
78
+ let firstDiff = -1;
79
+ for (let i = 0; i < Math.max(exp.length, act.length); i++) {
80
+ if (exp[i] !== act[i]) { firstDiff = i; break; }
81
+ }
82
+ const hint = firstDiff >= 0
83
+ ? ` (first drift at line ${firstDiff + 1} of section: "${(act[firstDiff] || '').slice(0, 60)}…" vs scanner: "${(exp[firstDiff] || '').slice(0, 60)}…")`
84
+ : '';
85
+
86
+ result.warnings.push(
87
+ `${basename(doc.path)} → section "${sec.id}" is stale${hint}. Run \`docguard sync --write\` to refresh code-truth sections.`
88
+ );
89
+ }
90
+ }
91
+
92
+ if (!anySourceCodeSection) {
93
+ return { ...result, applicable: false };
94
+ }
95
+
96
+ return result;
97
+ }
@@ -0,0 +1,133 @@
1
+ /**
2
+ * Fix Memory — M-2 / S-10
3
+ *
4
+ * Persists a JSON log of every mechanical fix `docguard fix --write` applies,
5
+ * stored at `.docguard/fixed.json`. Two purposes:
6
+ *
7
+ * 1. **Audit trail.** Users (and reviewers) can ask "what did the bot
8
+ * change in this repo and when?" without digging through git history.
9
+ * Especially valuable for the K-1 auto-fix Action which commits as
10
+ * `docguard-bot` — the memory file is the human-readable record.
11
+ *
12
+ * 2. **Future suppression hook.** A future `fix --write` can check the
13
+ * memory and skip fixes that were applied and then reverted — avoiding
14
+ * ping-pong loops where the bot keeps re-applying a fix the user keeps
15
+ * undoing. For v0.13 we just record; suppression is v0.14+.
16
+ *
17
+ * Format (JSON, gitignore-friendly):
18
+ * {
19
+ * "schemaVersion": "1",
20
+ * "entries": [
21
+ * {
22
+ * "id": "<sha256 of type+file+before+after, first 12 chars>",
23
+ * "type": "replace-version",
24
+ * "file": "README.md",
25
+ * "summary": "v0.11.2 → v0.12.0",
26
+ * "appliedAt": "2026-05-26T01:35:00Z",
27
+ * "appliedBy": "fix --write" | "sync --write" | "docguard-bot"
28
+ * }
29
+ * ]
30
+ * }
31
+ *
32
+ * The file is intentionally small (no full before/after content) to stay
33
+ * checkable into git for teams that want the audit trail under version
34
+ * control. Capped at 500 entries (rolling).
35
+ *
36
+ * @req SC-M2-001 — loadFixMemory returns an empty array when no file exists
37
+ * @req SC-M2-002 — appendFixes creates .docguard/ if needed
38
+ * @req SC-M2-003 — appendFixes is idempotent (same fix logged twice → one entry)
39
+ * @req SC-M2-004 — fingerprint dedupes by type+file+summary (not timestamp)
40
+ * @req SC-M2-005 — entries are capped at MAX_ENTRIES (oldest dropped)
41
+ */
42
+
43
+ import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
44
+ import { resolve, dirname } from 'node:path';
45
+ import { createHash } from 'node:crypto';
46
+
47
+ const MEMORY_PATH = '.docguard/fixed.json';
48
+ const SCHEMA_VERSION = '1';
49
+ const MAX_ENTRIES = 500;
50
+
51
+ /**
52
+ * Compute a stable fingerprint for a fix. Used for dedup — two fixes with
53
+ * the same type+file+summary are considered the same operation, even if
54
+ * applied at different times.
55
+ */
56
+ export function fingerprintFix(fix) {
57
+ const key = `${fix.type || ''}|${fix.file || ''}|${fix.summary || ''}`;
58
+ return createHash('sha256').update(key).digest('hex').slice(0, 12);
59
+ }
60
+
61
+ /**
62
+ * Load the fix memory from disk. Returns { schemaVersion, entries } —
63
+ * always a valid shape, even if the file is missing or malformed.
64
+ */
65
+ export function loadFixMemory(projectDir) {
66
+ const p = resolve(projectDir, MEMORY_PATH);
67
+ if (!existsSync(p)) {
68
+ return { schemaVersion: SCHEMA_VERSION, entries: [] };
69
+ }
70
+ try {
71
+ const data = JSON.parse(readFileSync(p, 'utf-8'));
72
+ if (!data || !Array.isArray(data.entries)) {
73
+ return { schemaVersion: SCHEMA_VERSION, entries: [] };
74
+ }
75
+ return { schemaVersion: data.schemaVersion || SCHEMA_VERSION, entries: data.entries };
76
+ } catch {
77
+ return { schemaVersion: SCHEMA_VERSION, entries: [] };
78
+ }
79
+ }
80
+
81
+ /**
82
+ * Append fixes to the memory file. Dedupes by fingerprint — re-applying the
83
+ * same fix updates the existing entry's appliedAt instead of adding a row.
84
+ *
85
+ * `fixes` is an array of { type, file, summary } objects. The function adds
86
+ * `id` + `appliedAt` + `appliedBy` automatically.
87
+ *
88
+ * Returns the updated memory object.
89
+ */
90
+ export function appendFixes(projectDir, fixes, appliedBy = 'fix --write') {
91
+ if (!Array.isArray(fixes) || fixes.length === 0) {
92
+ return loadFixMemory(projectDir);
93
+ }
94
+ const mem = loadFixMemory(projectDir);
95
+ const now = new Date().toISOString();
96
+ const byId = new Map(mem.entries.map(e => [e.id, e]));
97
+
98
+ for (const f of fixes) {
99
+ const id = fingerprintFix(f);
100
+ const entry = {
101
+ id,
102
+ type: f.type || 'unknown',
103
+ file: f.file || '',
104
+ summary: f.summary || '',
105
+ appliedAt: now,
106
+ appliedBy,
107
+ };
108
+ byId.set(id, entry); // overwrites prior with same fingerprint → updates appliedAt
109
+ }
110
+
111
+ let entries = Array.from(byId.values());
112
+ // Sort newest-first so the cap drops the oldest.
113
+ entries.sort((a, b) => (b.appliedAt || '').localeCompare(a.appliedAt || ''));
114
+ if (entries.length > MAX_ENTRIES) entries = entries.slice(0, MAX_ENTRIES);
115
+
116
+ const next = { schemaVersion: SCHEMA_VERSION, entries };
117
+
118
+ const fullPath = resolve(projectDir, MEMORY_PATH);
119
+ const dir = dirname(fullPath);
120
+ if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
121
+ writeFileSync(fullPath, JSON.stringify(next, null, 2) + '\n', 'utf-8');
122
+
123
+ return next;
124
+ }
125
+
126
+ /**
127
+ * True if a candidate fix (by fingerprint) has been applied before.
128
+ * Currently informational — future versions may use this to suppress.
129
+ */
130
+ export function isFixRecorded(projectDir, candidate) {
131
+ const id = fingerprintFix(candidate);
132
+ return loadFixMemory(projectDir).entries.some(e => e.id === id);
133
+ }
@@ -102,6 +102,12 @@ export function applyMechanicalFix(projectDir, fix, opts = {}) {
102
102
 
103
103
  /**
104
104
  * Apply a batch of fixes; returns a summary.
105
+ *
106
+ * M-2: When `opts.recordHistory` is true (default true when not in dry-run),
107
+ * each successfully applied fix is appended to `.docguard/fixed.json` so
108
+ * the project has a persistent audit trail. Pass `recordHistory: false` to
109
+ * disable (used by dry-run tests).
110
+ *
105
111
  * @returns {{ applied: object[], skipped: object[] }}
106
112
  */
107
113
  export function applyMechanicalFixes(projectDir, fixes, opts = {}) {
@@ -112,5 +118,21 @@ export function applyMechanicalFixes(projectDir, fixes, opts = {}) {
112
118
  if (r.applied) applied.push({ ...fix, detail: r.detail });
113
119
  else if (r.skipped) skipped.push({ ...fix, reason: r.skipped });
114
120
  }
121
+
122
+ if (applied.length > 0 && opts.recordHistory !== false) {
123
+ // Lazy-import to avoid the circular risk and keep mechanical.mjs's
124
+ // synchronous-only contract clean for callers that don't want history.
125
+ import('./fix-memory.mjs').then(({ appendFixes }) => {
126
+ const entries = applied.map(f => ({
127
+ type: f.type,
128
+ file: f.file || f.path || '',
129
+ summary: f.summary || f.detail || `${f.type} applied`,
130
+ }));
131
+ appendFixes(projectDir, entries, opts.appliedBy || 'fix --write');
132
+ }).catch(() => {
133
+ // Never let history-write break the fix flow — it's auxiliary.
134
+ });
135
+ }
136
+
115
137
  return { applied, skipped };
116
138
  }
@@ -1,5 +1,5 @@
1
1
  ---
2
- description: Run DocGuard guard validation — check project documentation against CDD standards with 21 validators
2
+ description: Run DocGuard guard validation — check project documentation against CDD standards with 22 validators
3
3
  handoffs:
4
4
  - label: Fix All Issues
5
5
  agent: docguard.fix
@@ -23,7 +23,7 @@ Run the DocGuard CLI to validate all documentation against Canonical-Driven Deve
23
23
  npx docguard-cli guard
24
24
  ```
25
25
 
26
- 2. **Parse the output**. Each of the 21 validators reports ✅ (pass), ⚠️ (warning), ❌ (fail), or ➖ (N/A — nothing to validate). **A ➖ N/A is NOT a pass**: it means the validator found nothing to check (e.g. no API-REFERENCE.md, no DB schema, no layer boundaries declared). Don't read N/A as "healthy" — read it as "not assessed".
26
+ 2. **Parse the output**. Each of the 22 validators reports ✅ (pass), ⚠️ (warning), ❌ (fail), or ➖ (N/A — nothing to validate). **A ➖ N/A is NOT a pass**: it means the validator found nothing to check (e.g. no API-REFERENCE.md, no DB schema, no layer boundaries declared). Don't read N/A as "healthy" — read it as "not assessed".
27
27
 
28
28
  | Validator | What It Checks |
29
29
  |-----------|---------------|
@@ -68,7 +68,7 @@ diagnose → AI reads prompts → AI fixes docs → guard verifies
68
68
  ## Verify
69
69
 
70
70
  ```bash
71
- npx docguard-cli guard # Pass/fail check (21 validators)
71
+ npx docguard-cli guard # Pass/fail check (22 validators)
72
72
  npx docguard-cli score # 0-100 maturity score
73
73
  ```
74
74
 
@@ -14,7 +14,7 @@ handoffs:
14
14
 
15
15
  # DocGuard Guard
16
16
 
17
- Validate your project against its canonical documentation. Runs 160+ automated checks across 21 validators.
17
+ Validate your project against its canonical documentation. Runs 160+ automated checks across 22 validators.
18
18
 
19
19
  ## User Input
20
20
 
@@ -3,7 +3,7 @@ schema_version: "1.0"
3
3
  extension:
4
4
  id: "docguard"
5
5
  name: "DocGuard — CDD Enforcement"
6
- version: "0.12.0"
6
+ version: "0.13.0"
7
7
  description: "Canonical-Driven Development enforcement as a true spec-kit extension. LLM-first design with 19 automated validators, 4 AI behavior skills, spec-kit skill chaining, and workflow hooks. Zero NPM runtime dependencies."
8
8
  author: "Ricardo Accioly"
9
9
  repository: "https://github.com/raccioly/docguard"
@@ -6,10 +6,10 @@ description: AI-driven documentation repair with structured research workflow, t
6
6
  compatibility: Requires DocGuard CLI installed (npm i -g docguard-cli or npx docguard-cli)
7
7
  metadata:
8
8
  author: docguard
9
- version: 0.12.0
9
+ version: 0.13.0
10
10
  source: extensions/spec-kit-docguard/skills/docguard-fix
11
11
  ---
12
- <!-- docguard:version: 0.12.0 -->
12
+ <!-- docguard:version: 0.13.0 -->
13
13
 
14
14
  # DocGuard Fix Skill
15
15
 
@@ -7,10 +7,10 @@ description: Run DocGuard guard validation against Canonical-Driven Development
7
7
  compatibility: Requires DocGuard CLI installed (npm i -g docguard-cli or npx docguard-cli)
8
8
  metadata:
9
9
  author: docguard
10
- version: 0.12.0
10
+ version: 0.13.0
11
11
  source: extensions/spec-kit-docguard/skills/docguard-guard
12
12
  ---
13
- <!-- docguard:version: 0.12.0 -->
13
+ <!-- docguard:version: 0.13.0 -->
14
14
 
15
15
  # DocGuard Guard Skill
16
16
 
@@ -139,7 +139,7 @@ For each finding, provide a **specific, actionable fix** — not "fix the issue"
139
139
 
140
140
  Based on the triage results:
141
141
 
142
- - **If all PASS**: "All 21 validators passed. Project is CDD-compliant. Ready to commit."
142
+ - **If all PASS**: "All 22 validators passed. Project is CDD-compliant. Ready to commit."
143
143
  - **If only MEDIUM/LOW warnings**: "Non-blocking warnings found. Safe to commit, but consider running `/docguard.fix` for automated remediation."
144
144
  - **If HIGH or CRITICAL failures**: "Blocking issues found. Fix these before committing. Suggest running `/docguard.fix --doc [most impactful doc]` next."
145
145
 
@@ -6,10 +6,10 @@ description: Cross-document consistency analysis and quality assessment. Perform
6
6
  compatibility: Requires DocGuard CLI installed (npm i -g docguard-cli or npx docguard-cli)
7
7
  metadata:
8
8
  author: docguard
9
- version: 0.12.0
9
+ version: 0.13.0
10
10
  source: extensions/spec-kit-docguard/skills/docguard-review
11
11
  ---
12
- <!-- docguard:version: 0.12.0 -->
12
+ <!-- docguard:version: 0.13.0 -->
13
13
 
14
14
  # DocGuard Review Skill
15
15
 
@@ -6,10 +6,10 @@ description: CDD maturity assessment with category-aware improvement roadmap. Ru
6
6
  compatibility: Requires DocGuard CLI installed (npm i -g docguard-cli or npx docguard-cli)
7
7
  metadata:
8
8
  author: docguard
9
- version: 0.12.0
9
+ version: 0.13.0
10
10
  source: extensions/spec-kit-docguard/skills/docguard-score
11
11
  ---
12
- <!-- docguard:version: 0.12.0 -->
12
+ <!-- docguard:version: 0.13.0 -->
13
13
 
14
14
  # DocGuard Score Skill
15
15
 
@@ -4,7 +4,7 @@ description: Keep canonical documentation ALWAYS UP TO DATE. Refreshes code-trut
4
4
  compatibility: Requires DocGuard CLI installed (npm i -g docguard-cli or npx docguard-cli)
5
5
  metadata:
6
6
  author: docguard
7
- version: 0.12.0
7
+ version: 0.13.0
8
8
  source: extensions/spec-kit-docguard/skills/docguard-sync
9
9
  ---
10
10
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "docguard-cli",
3
- "version": "0.12.0",
3
+ "version": "0.13.0",
4
4
  "description": "The enforcement tool for Canonical-Driven Development (CDD). Audit, generate, and guard your project documentation.",
5
5
  "type": "module",
6
6
  "bin": {
@@ -1,5 +1,5 @@
1
1
  ---
2
- description: Run DocGuard guard validation — check all 21 validators and fix any issues
2
+ description: Run DocGuard guard validation — check all 22 validators and fix any issues
3
3
  handoffs:
4
4
  - label: Fix Issues
5
5
  agent: docguard.fix
@@ -19,7 +19,7 @@ You are an AI agent enforcing Canonical-Driven Development (CDD) compliance usin
19
19
  npx docguard-cli guard
20
20
  ```
21
21
 
22
- Read the output. It shows pass (✅), warn (⚠️), or fail (❌) for each of the 21 validators:
22
+ Read the output. It shows pass (✅), warn (⚠️), or fail (❌) for each of the 22 validators:
23
23
 
24
24
  | Priority | Validators |
25
25
  |----------|-----------|