xdrs-core 0.24.0 → 0.24.1

package/lib/lint.js

@@ -28,6 +28,8 @@ const SKILL_PACKAGE_OPTIONAL_DIRS = new Set(['scripts', 'references', RESOURCE_D
 const XDR_ALLOWED_FRONTMATTER_KEYS = new Set(['name', 'description', 'apply-to', 'valid-from', 'license', 'metadata']);
 const SKILL_ALLOWED_FRONTMATTER_KEYS = new Set(['name', 'description', 'license', 'metadata', 'compatibility', 'allowed-tools']);
 const IMAGE_EXTENSIONS = new Set(['.png', '.jpg', '.jpeg', '.gif', '.svg', '.webp', '.bmp']);
+const SLIDE_FILE_RE = /^.+-slides(?:-[a-z0-9-]+)?\.md$/;
+const SLIDE_MAX_NAME_LENGTH = 64;
 const EMOJI_RE = /\p{Extended_Pictographic}/u;
 const XDR_MAX_WORDS = 2600;
 const ARTICLE_MAX_WORDS = 5000;
@@ -808,6 +810,51 @@ function lintOrphanAssets(assetsDir, documentPaths, xdrsRoot, errors) {
       errors.push(`Orphan asset file not referenced by any document: ${toDisplayPath(assetPath)}`);
     }
   }
+
+  // Lint slide/presentation files inside .assets
+  for (const assetPath of assetTree.files) {
+    const fileName = path.basename(assetPath);
+    if (SLIDE_FILE_RE.test(fileName)) {
+      lintSlideFile(assetPath, documentPaths, xdrsRoot, errors);
+    }
+  }
+}
+
+function lintSlideFile(filePath, documentPaths, xdrsRoot, errors) {
+  const fileName = path.basename(filePath);
+
+  if (fileName !== fileName.toLowerCase()) {
+    errors.push(`Slide file name must be lowercase: ${toDisplayPath(filePath)}`);
+  }
+
+  if (fileName.length > SLIDE_MAX_NAME_LENGTH) {
+    errors.push(`Slide file name must be ${SLIDE_MAX_NAME_LENGTH} characters or fewer: ${toDisplayPath(filePath)}`);
+  }
+
+  const content = fs.readFileSync(filePath, 'utf8');
+
+  const fm = extractFrontmatter(content);
+  if (!fm.present) {
+    errors.push(`Slide file must start with a YAML frontmatter block containing marp: true: ${toDisplayPath(filePath)}`);
+  } else {
+    const hasMarp = /^marp:\s*true$/m.test(content.match(/^---\r?\n([\s\S]*?)\r?\n---/)[1]);
+    if (!hasMarp) {
+      errors.push(`Slide frontmatter must include marp: true: ${toDisplayPath(filePath)}`);
+    }
+  }
+
+  lintNoEmojis(content, filePath, 'Slide', errors);
+
+  // Check that the slide links back to at least one parent document
+  const repoRoot = path.dirname(xdrsRoot);
+  const slideDir = path.dirname(filePath);
+  const slideLinks = parseLocalLinks(content, slideDir, repoRoot);
+  const slideLinkSet = new Set(slideLinks.map(normalizePath));
+
+  const linksToParent = documentPaths.some((docPath) => slideLinkSet.has(normalizePath(docPath)));
+  if (!linksToParent) {
+    errors.push(`Slide file must contain a link back to its parent document: ${toDisplayPath(filePath)}`);
+  }
 }
 
 function collectAssetTree(dirPath, errors) {

package/lib/lint.test.js

@@ -1238,8 +1238,158 @@ test('passes when research ## Introduction contains Question: line', () => {
   expect(result.errors.join('\n')).not.toContain('Research ## Introduction must contain');
 });
 
+// ─── Slide / Presentation checks ─────────────────────────────────────────────
+
+test('passes for valid slide file in .assets with marp frontmatter and backlink', () => {
+  const workspaceRoot = createWorkspace('marp-valid', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/index.md': '# _local Scope Overview\n\n## Content\n\nLocal scope.\n\n- [ADRs](adrs/index.md)\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Slides](.assets/001-main-slides.md)'),
+    '.xdrs/_local/adrs/principles/.assets/001-main-slides.md': slideDocument('[Parent](../001-main.md)'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  const slideErrors = result.errors.filter((e) => /\bSlide\b/.test(e) || /\bSlide\b/i.test(e) && e.includes('slides'));
+  expect(slideErrors).toHaveLength(0);
+});
+
+test('reports slide file missing marp: true in frontmatter', () => {
+  const workspaceRoot = createWorkspace('slide-no-marp', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Slides](.assets/001-main-slides.md)'),
+    '.xdrs/_local/adrs/principles/.assets/001-main-slides.md': [
+      '---',
+      'theme: default',
+      '---',
+      '',
+      '# Slides',
+      '',
+      '[Parent](../001-main.md)',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide frontmatter must include marp: true');
+});
+
+test('reports slide file without any frontmatter', () => {
+  const workspaceRoot = createWorkspace('slide-no-frontmatter', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Slides](.assets/001-main-slides.md)'),
+    '.xdrs/_local/adrs/principles/.assets/001-main-slides.md': '# Slides\n\n[Parent](../001-main.md)\n',
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide file must start with a YAML frontmatter block containing marp: true');
+});
+
+test('reports slide file missing backlink to parent document', () => {
+  const workspaceRoot = createWorkspace('slide-no-backlink', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Slides](.assets/001-main-slides.md)'),
+    '.xdrs/_local/adrs/principles/.assets/001-main-slides.md': slideDocument('No link back here.'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide file must contain a link back to its parent document');
+});
+
+test('reports slide file name exceeding 64 characters', () => {
+  const longName = '001-main-slides-' + 'a'.repeat(46) + '.md'; // > 64 chars
+  const workspaceRoot = createWorkspace('slide-name-too-long', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument(`Body.\n\n[Slides](.assets/${longName})`),
+    [`.xdrs/_local/adrs/principles/.assets/${longName}`]: slideDocument('[Parent](../001-main.md)'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide file name must be 64 characters or fewer');
+});
+
+test('reports emojis in slide files', () => {
+  const workspaceRoot = createWorkspace('slide-emoji', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Slides](.assets/001-main-slides.md)'),
+    '.xdrs/_local/adrs/principles/.assets/001-main-slides.md': slideDocument('Great stuff! \u{1F680}\n\n[Parent](../001-main.md)'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide must not contain emojis');
+});
+
+test('does not lint non-slide files in .assets as slides', () => {
+  const workspaceRoot = createWorkspace('non-slide-asset', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-main](principles/001-main.md) - Main decision'
+    ]),
+    '.xdrs/_local/adrs/principles/001-main.md': xdrDocument('Body.\n\n[Schema](.assets/schema.json)'),
+    '.xdrs/_local/adrs/principles/.assets/schema.json': '{"type":"object"}',
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('Slide');
+  expect(result.errors.join('\n')).not.toContain('marp');
+});
+
+test('passes for slide file in article .assets with backlink', () => {
+  const workspaceRoot = createWorkspace('marp-article-valid', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/index.md': '# _local Scope Overview\n\n## Content\n\nLocal scope.\n\n- [ADRs](adrs/index.md)\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [001-guide](principles/articles/001-guide.md) - Guide'
+    ]),
+    '.xdrs/_local/adrs/principles/articles/001-guide.md': articleDocument('Body.\n\n[Slides](.assets/001-guide-slides.md)'),
+    '.xdrs/_local/adrs/principles/articles/.assets/001-guide-slides.md': slideDocument('[Parent](../001-guide.md)'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('Slide');
+  expect(result.errors.join('\n')).not.toContain('marp');
+});
+
 // ─── New helpers ──────────────────────────────────────────────────────────────
 
+function slideDocument(body) {
+  return [
+    '---',
+    'marp: true',
+    '---',
+    '',
+    '# Presentation',
+    '',
+    body,
+    ''
+  ].join('\n');
+}
+
 function articleDocument(body) {
   return [
     '# _local-article-001: Guide',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xdrs-core",
-  "version": "0.24.0",
+  "version": "0.24.1",
   "description": "A standard way to organize Decision Records (XDRs) across scopes, subjects, and teams so that AI agents can reliably query and follow them.",
   "repository": {
     "type": "git",