cabloy 5.1.198 → 5.1.200

package/.cabloy-version

@@ -1 +1 @@
-5.1.198
+5.1.200

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 5.1.200
+
+### Features
+
+- Apply feature updates.
+
+## 5.1.199
+
+### Improvements
+
+- Update implementation chart tests.
+
 ## 5.1.198
 
 ### Features

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cabloy",
-  "version": "5.1.198",
+  "version": "5.1.200",
   "gitHead": "2c5c19284bab738e492856189acb6fad74b8a7b7",
   "description": "A Node.js fullstack framework",
   "keywords": [

package/repo-agent-governance/tests/spec-audit.test.mjs

@@ -208,6 +208,50 @@ test('local Markdown destinations, fragments and reference links are checked', a
   );
 });
 
+test('link destinations are not authority references but labels and prose remain audited', async t => {
+  const records = {
+    ...BASELINE,
+    'rollout.md':
+      '# Rollout\n\n[WBS-MISSING-01](./evidence/WBS-DEMO-10-01-current-source.md) and `ATP-MISSING-01`.\n\n![SRS-MISSING-01](<./evidence/WBS-DEMO-10-01-image.md>)\n\n[WBS-DEMO-10-01][proof]\n\n[proof]: ./evidence/WBS-DEMO-10-01-referenced.md\n',
+    'evidence/WBS-DEMO-10-01-current-source.md': '# Current source\n',
+    'evidence/WBS-DEMO-10-01-image.md': '# Image\n',
+    'evidence/WBS-DEMO-10-01-referenced.md': '# Referenced\n',
+  };
+  const result = await auditPlanning(await fixture(t, records));
+  assert.deepEqual(
+    result.diagnostics.map(diagnostic => [
+      diagnostic.code,
+      diagnostic.file,
+      diagnostic.line,
+      diagnostic.id,
+    ]),
+    [
+      ['undefined-reference', 'rollout.md', 3, 'ATP-MISSING-01'],
+      ['undefined-reference', 'rollout.md', 3, 'WBS-MISSING-01'],
+      ['undefined-reference', 'rollout.md', 5, 'SRS-MISSING-01'],
+    ],
+  );
+});
+
+test('destination masking preserves local link and reference diagnostics', async t => {
+  const records = {
+    ...BASELINE,
+    'rollout.md':
+      '# Rollout\n\n[Missing](./evidence/WBS-DEMO-10-01-missing.md)\n[Bad fragment](./evidence/WBS-DEMO-10-01-existing.md#absent)\n[Good fragment](./evidence/WBS-DEMO-10-01-existing.md#present)\n[Bad encoding](./evidence/WBS-DEMO-10-01-%ZZ.md)\n[Undefined][missing]\n\n[unused]: <./evidence/WBS-DEMO-10-01-unused.md>\n',
+    'evidence/WBS-DEMO-10-01-existing.md': '# Present\n',
+  };
+  const result = await auditPlanning(await fixture(t, records));
+  assert.deepEqual(
+    result.diagnostics.map(diagnostic => [diagnostic.code, diagnostic.line]),
+    [
+      ['link-target', 3],
+      ['link-fragment', 4],
+      ['link-format', 6],
+      ['link-reference', 7],
+    ],
+  );
+});
+
 test('exact range endpoints do not masquerade as exact traceability', async t => {
   const records = {
     ...BASELINE,

package/repo-agent-governance/tools/spec-audit/audit.mjs

@@ -142,17 +142,31 @@ function headingIds(markdown) {
   return slugs;
 }
 
+const LINK_DEFINITION = /^(\s*\[([^\]]+)\]:\s*)(<[^>]+>|\S+)/;
+const INLINE_LINK = /(!?\[[^\]]*\]\(\s*)(<[^>]+>|[^\s)]+)((?:\s+["'][^\n]*["'])?\s*\))/g;
+
+function referenceText(text) {
+  return text
+    .replace(
+      LINK_DEFINITION,
+      (_, start, _label, destination) => `${start}${' '.repeat(destination.length)}`,
+    )
+    .replace(
+      INLINE_LINK,
+      (_, start, destination, end) => `${start}${' '.repeat(destination.length)}${end}`,
+    );
+}
+
 function markdownLinks(markdown) {
   const lines = markdownLines(markdown);
   const references = new Map();
   const links = [];
   for (const { text, line } of lines) {
-    const declaration = text.match(/^\s*\[([^\]]+)\]:\s*(<[^>]+>|\S+)/);
+    const declaration = text.match(LINK_DEFINITION);
     if (declaration)
-      references.set(declaration[1].trim().toLowerCase(), declaration[2].replace(/^<|>$/g, ''));
-    const inline = /!?\[[^\]]*\]\(\s*(<[^>]+>|[^\s)]+)(?:\s+["'][^\n]*["'])?\s*\)/g;
-    for (const match of text.matchAll(inline))
-      links.push({ destination: match[1].replace(/^<|>$/g, ''), line });
+      references.set(declaration[2].trim().toLowerCase(), declaration[3].replace(/^<|>$/g, ''));
+    for (const match of text.matchAll(INLINE_LINK))
+      links.push({ destination: match[2].replace(/^<|>$/g, ''), line });
     const referenced = /!?\[([^\]]+)\]\[([^\]]*)\]/g;
     for (const match of text.matchAll(referenced))
       links.push({ reference: (match[2] || match[1]).trim().toLowerCase(), line });
@@ -260,7 +274,7 @@ export async function auditPlanning(
   for (const [file, markdown] of records) {
     for (const { text, line } of markdownLines(markdown)) {
       for (const prefix of OWNERS.keys()) {
-        for (const id of findIdentifiers(text, prefix)) {
+        for (const id of findIdentifiers(referenceText(text), prefix)) {
           if (!definitions.has(id)) {
             report(
               'undefined-reference',

package/repo-agent-governance/tools/spec-charts/generate-implementation-charts.test.mjs

@@ -1,5 +1,5 @@
 import assert from 'node:assert/strict';
-import { mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
+import { mkdtemp, readFile, readdir, rm, writeFile } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { resolve } from 'node:path';
 import { test } from 'node:test';
@@ -440,20 +440,19 @@ test('ATP catalogue role headers define scenarios but coverage and evidence tabl
   );
 });
 
-test('existing business ATP catalogues remain compatible with strict definitions', async () => {
+test('repository ATP catalogues remain compatible with strict definitions', async () => {
   const root = resolve(import.meta.dirname, '../../..');
-  const commerce = parseAtpIds(
-    await readFile(resolve(root, 'repo-specs/a-commerce/test-plan.md'), 'utf8'),
-  );
-  const account = parseAtpIds(
-    await readFile(resolve(root, 'repo-specs/home-user/test-plan.md'), 'utf8'),
-  );
-  assert.ok(commerce.has('ATP-SPC-04'));
-  assert.ok(commerce.has('ATP-SPC-05'));
-  assert.ok(account.has('ATP-HUA-ACC-01'));
-  assert.ok(account.has('ATP-HUA-RST-03'));
-  assert.ok(commerce.size > 10);
-  assert.ok(account.size > 10);
+  const suiteEntries = await readdir(resolve(root, 'repo-specs'), { withFileTypes: true });
+  const suiteNames = suiteEntries
+    .filter(entry => entry.isDirectory() && !entry.name.startsWith('.'))
+    .map(entry => entry.name)
+    .sort();
+  assert.ok(suiteNames.length > 0, 'repo-specs must contain at least one suite directory');
+  for (const suiteName of suiteNames) {
+    const testPlan = await readFile(resolve(root, 'repo-specs', suiteName, 'test-plan.md'), 'utf8');
+    const atpIds = parseAtpIds(testPlan);
+    assert.ok(atpIds.size > 0, `${suiteName}/test-plan.md must define formal ATP scenarios`);
+  }
 });
 
 test('progress rejects duplicate rows and multiple IDs in one row with source locations', () => {