xdrs-core 0.49.1 → 0.51.0

package/lib/lint.test.js

@@ -338,8 +338,6 @@ test('accepts all known frontmatter fields in SKILL.md files', () => {
       'license: MIT',
       'metadata:',
       '  version: "1.0"',
-      'compatibility: node >= 18',
-      'allowed-tools: read_file grep_search',
       '---',
       '',
       '## Overview',
@@ -358,6 +356,553 @@ test('accepts all known frontmatter fields in SKILL.md files', () => {
   expect(result.errors.join('\n')).not.toContain('unknown field');
 });
 
+test('reports missing ### Inputs subsection in SKILL.md ## Overview', () => {
+  const workspaceRoot = createWorkspace('skill-missing-inputs', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ omitInputs: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "## Overview" is missing required subsection "### Inputs"');
+});
+
+test('reports missing ### Outputs subsection in SKILL.md ## Overview', () => {
+  const workspaceRoot = createWorkspace('skill-missing-outputs', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ omitOutputs: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "## Overview" is missing required subsection "### Outputs"');
+});
+
+test('reports missing ### Halt Conditions subsection in SKILL.md ## Overview', () => {
+  const workspaceRoot = createWorkspace('skill-missing-halt-conditions', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ omitHaltConditions: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "## Overview" is missing required subsection "### Halt Conditions"');
+});
+
+test('reports SKILL.md ### Inputs missing #### Optional subsection', () => {
+  const workspaceRoot = createWorkspace('skill-inputs-missing-optional', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ omitInputsOptional: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "### Inputs" section is missing "#### Optional"');
+});
+
+test('reports SKILL.md ### Inputs subsections out of order', () => {
+  const workspaceRoot = createWorkspace('skill-inputs-out-of-order', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ swapInputsOrder: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "### Inputs" subsections must appear in order (#### Required, #### Optional)');
+});
+
+test('reports SKILL.md ### Outputs missing #### Changes subsection', () => {
+  const workspaceRoot = createWorkspace('skill-outputs-missing-changes', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ omitOutputsChanges: true }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "### Outputs" section is missing "#### Changes"');
+});
+
+test('passes SKILL.md with valid None bullets in Inputs, Outputs, and Halt Conditions', () => {
+  const workspaceRoot = createWorkspace('skill-valid-none-bullets', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({
+      inputsRequired: '- None',
+      inputsOptional: '- None',
+      outputsContents: '- None',
+      outputsChanges: '- None',
+      haltConditions: '- None',
+    }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('must be under 10 words or a single "None"');
+});
+
+test('reports SKILL.md bullet over 10 words in #### Required', () => {
+  const workspaceRoot = createWorkspace('skill-required-bullet-too-long', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({
+      inputsRequired: '- A file path that is way too long to fit under the ten word limit',
+    }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "### Inputs" -> "#### Required" bullet must be under 10 words');
+});
+
+test('passes SKILL.md when both optional sections are omitted', () => {
+  const workspaceRoot = createWorkspace('skill-optional-sections-omitted', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument(),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('### Runtime Requirements');
+  expect(result.errors.join('\n')).not.toContain('### User Interaction');
+});
+
+test('reports empty ### Runtime Requirements section in SKILL.md', () => {
+  const workspaceRoot = createWorkspace('skill-empty-runtime-requirements', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ runtimeRequirements: '' }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill section "### Runtime Requirements" must not be empty');
+});
+
+test('reports empty ### User Interaction section in SKILL.md', () => {
+  const workspaceRoot = createWorkspace('skill-empty-user-interaction', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({ userInteraction: '' }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill section "### User Interaction" must not be empty');
+});
+
+test('passes SKILL.md with a long free-form Runtime Requirements bullet', () => {
+  const workspaceRoot = createWorkspace('skill-runtime-requirements-free-form', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({
+      runtimeRequirements: '- Write access to a directory that is not gitignored and has enough free disk space.',
+    }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('### Runtime Requirements');
+});
+
+test('reports SKILL.md User Interaction bullet over 10 words', () => {
+  const workspaceRoot = createWorkspace('skill-user-interaction-bullet-too-long', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': fullSkillDocument({
+      userInteraction: '- A clarifying question that is much too long to fit under the ten word limit',
+    }),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "### User Interaction" bullet must be under 10 words');
+});
+
+test('reports SKILL.md top-level sections out of template order', () => {
+  const workspaceRoot = createWorkspace('skill-sections-out-of-order', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': [
+      '---',
+      'name: check-links',
+      'description: Test skill document',
+      '---',
+      '',
+      '## Overview',
+      '',
+      'Test skill overview.',
+      '',
+      '### Inputs',
+      '',
+      '#### Required',
+      '- None',
+      '',
+      '#### Optional',
+      '- None',
+      '',
+      '### Outputs',
+      '',
+      '#### Contents',
+      '- None',
+      '',
+      '#### Changes',
+      '- None',
+      '',
+      '### Halt Conditions',
+      '- None',
+      '',
+      '## Anti-Patterns',
+      '',
+      '- **Mistake:** First mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Second mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Third mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '',
+      '## Instructions',
+      '',
+      'Test instructions.',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill sections must appear in template order');
+});
+
+test('reports SKILL.md ## Overview subsections out of template order', () => {
+  const workspaceRoot = createWorkspace('skill-overview-subsections-out-of-order', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': [
+      '---',
+      'name: check-links',
+      'description: Test skill document',
+      '---',
+      '',
+      '## Overview',
+      '',
+      'Test skill overview.',
+      '',
+      '### Halt Conditions',
+      '- None',
+      '',
+      '### Inputs',
+      '',
+      '#### Required',
+      '- None',
+      '',
+      '#### Optional',
+      '- None',
+      '',
+      '### Outputs',
+      '',
+      '#### Contents',
+      '- None',
+      '',
+      '#### Changes',
+      '- None',
+      '',
+      '## Instructions',
+      '',
+      'Test instructions.',
+      '',
+      '## Anti-Patterns',
+      '',
+      '- **Mistake:** First mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Second mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Third mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill "## Overview" subsections must appear in order');
+});
+
+test('reports skill Anti-Patterns section with fewer than 3 entries', () => {
+  const workspaceRoot = createWorkspace('skill-anti-patterns-too-few', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': [
+      '---',
+      'name: check-links',
+      'description: Test skill document',
+      '---',
+      '',
+      '## Overview',
+      '',
+      'Test skill overview.',
+      '',
+      '## Instructions',
+      '',
+      'Test instructions.',
+      '',
+      '## Anti-Patterns',
+      '',
+      '- **Mistake:** First mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Second mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('"## Anti-Patterns" section must include at least 3 entries (found 2)');
+});
+
+test('passes when skill Anti-Patterns section has at least 3 entries', () => {
+  const workspaceRoot = createWorkspace('skill-anti-patterns-enough', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': [
+      '---',
+      'name: check-links',
+      'description: Test skill document',
+      '---',
+      '',
+      '## Overview',
+      '',
+      'Test skill overview.',
+      '',
+      '## Instructions',
+      '',
+      'Test instructions.',
+      '',
+      '## Anti-Patterns',
+      '',
+      '- **Mistake:** First mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Second mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      '- **Mistake:** Third mistake. **Why it happens:** Reason. **Instead:** Fix.',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('"## Anti-Patterns" section must include at least');
+});
+
+test('reports skill bundling Makefile missing required targets', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-missing-targets', {
+    '.xdrs/index.md': rootIndex(),
+    '.gitignore': 'dist\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build:',
+      '\techo build',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill bundling Makefile is missing required "test" target');
+  expect(result.errors.join('\n')).toContain('Skill bundling Makefile is missing required "clean" target');
+  expect(result.errors.join('\n')).not.toContain('Skill bundling Makefile is missing required "build" target');
+});
+
+test('passes when skill bundling Makefile has build, test, and clean targets', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-complete', {
+    '.xdrs/index.md': rootIndex(),
+    '.gitignore': 'dist\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build: clean',
+      '\techo build',
+      'test:',
+      '\techo test',
+      'clean:',
+      '\trm -rf dist',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('Skill bundling Makefile is missing required');
+});
+
+test('reports skill bundling Makefile build target not depending on clean', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-build-no-clean-dep', {
+    '.xdrs/index.md': rootIndex(),
+    '.gitignore': 'dist\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build:',
+      '\trm -rf dist',
+      '\techo build',
+      'test:',
+      '\techo test',
+      'clean:',
+      '\trm -rf dist',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Skill bundling Makefile\'s "build" target must depend on "clean"');
+});
+
+test('passes when skill bundling Makefile build target depends on clean', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-build-clean-dep', {
+    '.xdrs/index.md': rootIndex(),
+    '.gitignore': 'dist\n',
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build: clean',
+      '\techo build',
+      'test:',
+      '\techo test',
+      'clean:',
+      '\trm -rf dist',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('must depend on "clean"');
+});
+
+test('reports skill bundling Makefile dist output not covered by any .gitignore', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-no-gitignore', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build: clean',
+      '\techo build',
+      'test:',
+      '\techo test',
+      'clean:',
+      '\trm -rf dist',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('is not covered by any .gitignore');
+});
+
+test('passes when skill bundling Makefile dist output is covered by a local .gitignore', () => {
+  const workspaceRoot = createWorkspace('skill-makefile-local-gitignore', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/Makefile': [
+      'build: clean',
+      '\techo build',
+      'test:',
+      '\techo test',
+      'clean:',
+      '\trm -rf dist',
+      ''
+    ].join('\n'),
+    '.xdrs/_local/adrs/principles/skills/check-links/.gitignore': 'dist/\n',
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('is not covered by any .gitignore');
+});
+
+test('reports scripts/ file missing a shebang or header comment', () => {
+  const workspaceRoot = createWorkspace('skill-script-missing-header', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/scripts/run.sh': [
+      'echo "no header here"',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Script must declare its runtime/interpreter at the top');
+});
+
+test('passes when scripts/ file declares a shebang', () => {
+  const workspaceRoot = createWorkspace('skill-script-with-shebang', {
+    '.xdrs/index.md': rootIndex(),
+    '.xdrs/_local/adrs/index.md': localAdrIndex([
+      '- [check-links](principles/skills/check-links/SKILL.md) - Check links'
+    ]),
+    '.xdrs/_local/adrs/principles/skills/check-links/SKILL.md': skillDocument('Test instructions.'),
+    '.xdrs/_local/adrs/principles/skills/check-links/scripts/run.sh': [
+      '#!/bin/bash',
+      'echo "has a shebang"',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('Script must declare its runtime/interpreter at the top');
+});
+
 test('accepts skill package names containing digits, including as a prefix', () => {
   const workspaceRoot = createWorkspace('skill-name-with-digits', {
     '.xdrs/index.md': rootIndex(),
@@ -2577,11 +3122,15 @@ function validCompiledMetaPolicy() {
     '',
     '## Sources',
     '',
-    '- [web] https://example.com/standards',
+    '- [web] example-standards: https://example.com/standards',
     '',
     '## Selectors',
     '',
     '- Include all requirement sections.',
+    '',
+    '## Sync Settings',
+    '',
+    '- Source re-sync period days: 30',
     ''
   ].join('\n');
 }
@@ -2591,17 +3140,258 @@ test('reports policy in compiled scope missing structured format', () => {
     '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
     '.xdrs/my-compiled/index.md': compiledScopeIndex(),
     '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
-      '- [001-a01](principles/001-a01.md) - A01'
+      '- [001-a01](principles/001-a01.md) - A01'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-001-a01',
+      'description: Compiled test policy.',
+      'apply-to: All teams',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: A01',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'External source.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Follow rule.',
+      '',
+      '## Source',
+      '',
+      '- source-1/a01.md',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('"### Details"');
+  expect(result.errors.join('\n')).toContain('019.04-structured-format-required');
+});
+
+test('reports compiled policy missing ## Source section', () => {
+  const workspaceRoot = createWorkspace('compiled-scope-missing-source', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-a01](principles/001-a01.md) - A01'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-001-a01',
+      'description: Compiled test policy.',
+      'apply-to: All teams',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: A01',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'External source.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Follow rule.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST comply.',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Compiled policy must include a "## Source" section');
+  expect(result.errors.join('\n')).toContain('019.10-source-section');
+});
+
+test('reports compiled policy with markdown links in ## Source section', () => {
+  const workspaceRoot = createWorkspace('compiled-scope-source-md-links', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-a01](principles/001-a01.md) - A01'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-001-a01',
+      'description: Compiled test policy.',
+      'apply-to: All teams',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: A01',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'External source.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Follow rule.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST comply.',
+      '',
+      '## Source',
+      '',
+      '- [source-1/a01.md](source-1/a01.md)',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('"## Source" section must not use markdown links');
+  expect(result.errors.join('\n')).toContain('019.10-source-section');
+});
+
+test('accepts valid compiled policy with ## Source section and no markdown links', () => {
+  const workspaceRoot = createWorkspace('compiled-scope-valid', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-a01](principles/001-a01.md) - A01'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-a01.md': validCompiledPolicy(),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('019');
+});
+
+test('reports compiled scope meta-policy missing ## Sources section', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-missing-sources', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-core](principles/001-core.md) - meta-policy',
+      '- [002-a01](principles/002-a01.md) - A01'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-core.md': [
+      '---',
+      'name: my-compiled-adr-policy-001-core',
+      'description: Compilation meta-policy.',
+      'apply-to: Compilation process',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: core',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'Defines compilation config.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Use configured sources.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST use configured sources.',
+      '',
+      '## Selectors',
+      '',
+      '- Include requirement sections.',
+      ''
+    ].join('\n'),
+    '.xdrs/my-compiled/adrs/principles/002-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-002-a01',
+      'description: Compiled test policy.',
+      'apply-to: All teams',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-002: A01',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'External source.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Follow rule.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST comply.',
+      '',
+      '## Source',
+      '',
+      '- source-1/a01.md',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('"## Sources" section');
+  expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
+});
+
+test('reports compiled scope meta-policy missing ## Selectors section', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-missing-selectors', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-core](principles/001-core.md) - meta-policy',
+      '- [002-a01](principles/002-a01.md) - A01'
     ]),
-    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+    '.xdrs/my-compiled/adrs/principles/001-core.md': [
       '---',
-      'name: my-compiled-adr-policy-001-a01',
+      'name: my-compiled-adr-policy-001-core',
+      'description: Compilation meta-policy.',
+      'apply-to: Compilation process',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: core',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'Defines compilation config.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Use configured sources.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST use configured sources.',
+      '',
+      '## Sources',
+      '',
+      '- [web] https://example.com/standards',
+      ''
+    ].join('\n'),
+    '.xdrs/my-compiled/adrs/principles/002-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-002-a01',
       'description: Compiled test policy.',
       'apply-to: All teams',
       'valid-from: 2026-01-01',
       '---',
       '',
-      '# my-compiled-adr-policy-001: A01',
+      '# my-compiled-adr-policy-002: A01',
       '',
       '## Context and Problem Statement',
       '',
@@ -2611,6 +3401,12 @@ test('reports policy in compiled scope missing structured format', () => {
       '',
       'Follow rule.',
       '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST comply.',
+      '',
       '## Source',
       '',
       '- source-1/a01.md',
@@ -2620,26 +3416,60 @@ test('reports policy in compiled scope missing structured format', () => {
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('"### Details"');
-  expect(result.errors.join('\n')).toContain('019.04-structured-format-required');
+  expect(result.errors.join('\n')).toContain('"## Selectors" section');
+  expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
 });
 
-test('reports compiled policy missing ## Source section', () => {
-  const workspaceRoot = createWorkspace('compiled-scope-missing-source', {
+test('reports compiled scope meta-policy missing ## Sync Settings section', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-missing-sync-settings', {
     '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
     '.xdrs/my-compiled/index.md': compiledScopeIndex(),
     '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
-      '- [001-a01](principles/001-a01.md) - A01'
+      '- [001-core](principles/001-core.md) - meta-policy',
+      '- [002-a01](principles/002-a01.md) - A01'
     ]),
-    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+    '.xdrs/my-compiled/adrs/principles/001-core.md': [
       '---',
-      'name: my-compiled-adr-policy-001-a01',
+      'name: my-compiled-adr-policy-001-core',
+      'description: Compilation meta-policy.',
+      'apply-to: Compilation process',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: core',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'Defines compilation config.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Use configured sources.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST use configured sources.',
+      '',
+      '## Sources',
+      '',
+      '- [web] example-standards: https://example.com/standards',
+      '',
+      '## Selectors',
+      '',
+      '- Include requirement sections.',
+      ''
+    ].join('\n'),
+    '.xdrs/my-compiled/adrs/principles/002-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-002-a01',
       'description: Compiled test policy.',
       'apply-to: All teams',
       'valid-from: 2026-01-01',
       '---',
       '',
-      '# my-compiled-adr-policy-001: A01',
+      '# my-compiled-adr-policy-002: A01',
       '',
       '## Context and Problem Statement',
       '',
@@ -2654,32 +3484,74 @@ test('reports compiled policy missing ## Source section', () => {
       '#### 01-rule',
       '',
       'MUST comply.',
+      '',
+      '## Source',
+      '',
+      '- source-1/a01.md',
       ''
     ].join('\n'),
   });
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('Compiled policy must include a "## Source" section');
-  expect(result.errors.join('\n')).toContain('019.10-source-section');
+  expect(result.errors.join('\n')).toContain('"## Sync Settings" section');
+  expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
 });
 
-test('reports compiled policy with markdown links in ## Source section', () => {
-  const workspaceRoot = createWorkspace('compiled-scope-source-md-links', {
+test('reports missing Source re-sync period days in ## Sync Settings', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-missing-resync-period', {
     '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
     '.xdrs/my-compiled/index.md': compiledScopeIndex(),
     '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
-      '- [001-a01](principles/001-a01.md) - A01'
+      '- [001-core](principles/001-core.md) - meta-policy',
+      '- [002-a01](principles/002-a01.md) - A01'
     ]),
-    '.xdrs/my-compiled/adrs/principles/001-a01.md': [
+    '.xdrs/my-compiled/adrs/principles/001-core.md': [
       '---',
-      'name: my-compiled-adr-policy-001-a01',
+      'name: my-compiled-adr-policy-001-core',
+      'description: Compilation meta-policy.',
+      'apply-to: Compilation process',
+      'valid-from: 2026-01-01',
+      '---',
+      '',
+      '# my-compiled-adr-policy-001: core',
+      '',
+      '## Context and Problem Statement',
+      '',
+      'Defines compilation config.',
+      '',
+      '## Decision Outcome',
+      '',
+      'Use configured sources.',
+      '',
+      '### Details',
+      '',
+      '#### 01-rule',
+      '',
+      'MUST use configured sources.',
+      '',
+      '## Sources',
+      '',
+      '- [web] example-standards: https://example.com/standards',
+      '',
+      '## Selectors',
+      '',
+      '- Include requirement sections.',
+      '',
+      '## Sync Settings',
+      '',
+      '- Source storage: temporary',
+      ''
+    ].join('\n'),
+    '.xdrs/my-compiled/adrs/principles/002-a01.md': [
+      '---',
+      'name: my-compiled-adr-policy-002-a01',
       'description: Compiled test policy.',
       'apply-to: All teams',
       'valid-from: 2026-01-01',
       '---',
       '',
-      '# my-compiled-adr-policy-001: A01',
+      '# my-compiled-adr-policy-002: A01',
       '',
       '## Context and Problem Statement',
       '',
@@ -2697,34 +3569,19 @@ test('reports compiled policy with markdown links in ## Source section', () => {
       '',
       '## Source',
       '',
-      '- [source-1/a01.md](source-1/a01.md)',
+      '- source-1/a01.md',
       ''
     ].join('\n'),
   });
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('"## Source" section must not use markdown links');
-  expect(result.errors.join('\n')).toContain('019.10-source-section');
-});
-
-test('accepts valid compiled policy with ## Source section and no markdown links', () => {
-  const workspaceRoot = createWorkspace('compiled-scope-valid', {
-    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
-    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
-    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
-      '- [001-a01](principles/001-a01.md) - A01'
-    ]),
-    '.xdrs/my-compiled/adrs/principles/001-a01.md': validCompiledPolicy(),
-  });
-
-  const result = lintWorkspace(workspaceRoot);
-
-  expect(result.errors.join('\n')).not.toContain('019');
+  expect(result.errors.join('\n')).toContain('"Source re-sync period days: N" bullet with a positive integer');
+  expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
 });
 
-test('reports compiled scope meta-policy missing ## Sources section', () => {
-  const workspaceRoot = createWorkspace('compiled-meta-missing-sources', {
+test('reports invalid Source storage value in ## Sync Settings', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-invalid-storage-value', {
     '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
     '.xdrs/my-compiled/index.md': compiledScopeIndex(),
     '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
@@ -2755,9 +3612,18 @@ test('reports compiled scope meta-policy missing ## Sources section', () => {
       '',
       'MUST use configured sources.',
       '',
+      '## Sources',
+      '',
+      '- [web] example-standards: https://example.com/standards',
+      '',
       '## Selectors',
       '',
       '- Include requirement sections.',
+      '',
+      '## Sync Settings',
+      '',
+      '- Source re-sync period days: 30',
+      '- Source storage: permanent',
       ''
     ].join('\n'),
     '.xdrs/my-compiled/adrs/principles/002-a01.md': [
@@ -2793,12 +3659,12 @@ test('reports compiled scope meta-policy missing ## Sources section', () => {
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('"## Sources" section');
+  expect(result.errors.join('\n')).toContain('"Source storage" value must be "temporary" if present');
   expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
 });
 
-test('reports compiled scope meta-policy missing ## Selectors section', () => {
-  const workspaceRoot = createWorkspace('compiled-meta-missing-selectors', {
+test('reports compiled scope ## Sources bullet missing NAME slug', () => {
+  const workspaceRoot = createWorkspace('compiled-meta-sources-missing-name', {
     '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
     '.xdrs/my-compiled/index.md': compiledScopeIndex(),
     '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
@@ -2832,6 +3698,14 @@ test('reports compiled scope meta-policy missing ## Selectors section', () => {
       '## Sources',
       '',
       '- [web] https://example.com/standards',
+      '',
+      '## Selectors',
+      '',
+      '- Include requirement sections.',
+      '',
+      '## Sync Settings',
+      '',
+      '- Source re-sync period days: 30',
       ''
     ].join('\n'),
     '.xdrs/my-compiled/adrs/principles/002-a01.md': [
@@ -2867,7 +3741,7 @@ test('reports compiled scope meta-policy missing ## Selectors section', () => {
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('"## Selectors" section');
+  expect(result.errors.join('\n')).toContain('must include a NAME slug');
   expect(result.errors.join('\n')).toContain('019.07-compilation-meta-policy');
 });
 
@@ -2931,6 +3805,88 @@ test('skips ## Source check for local meta-policy files in compiled scope', () =
   expect(result.errors.join('\n')).not.toContain('Compiled policy must include a "## Source" section');
 });
 
+test('does not report orphan asset errors for files under .assets/sources in a compiled scope', () => {
+  const workspaceRoot = createWorkspace('compiled-sources-orphan-exempt', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-core](principles/001-core.md) - meta-policy'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-core.md': validCompiledMetaPolicy(),
+    '.xdrs/my-compiled/adrs/principles/.assets/sources/example-standards/source.md': [
+      '# Source',
+      '',
+      'last-fetch-timestamp: 2026-01-01 10:00:00',
+      'last-compilation-timestamp: 2026-01-01 10:05:00',
+      ''
+    ].join('\n'),
+    '.xdrs/my-compiled/adrs/principles/.assets/sources/example-standards/raw-standard.md': '# Raw fetched content\n',
+    '.xdrs/my-compiled/adrs/principles/.assets/unused.png': Buffer.alloc(0),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).not.toContain('raw-standard.md');
+  expect(result.errors.join('\n')).not.toContain('019.12-source-storage-tracking');
+  expect(result.errors.join('\n')).toContain('Orphan asset file');
+  expect(result.errors.join('\n')).toContain('unused.png');
+});
+
+test('reports missing source.md tracking file in a compiled source snapshot folder', () => {
+  const workspaceRoot = createWorkspace('compiled-sources-missing-tracking-file', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-core](principles/001-core.md) - meta-policy'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-core.md': validCompiledMetaPolicy(),
+    '.xdrs/my-compiled/adrs/principles/.assets/sources/example-standards/raw-standard.md': '# Raw fetched content\n',
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('must contain a "source.md" tracking file');
+  expect(result.errors.join('\n')).toContain('019.12-source-storage-tracking');
+});
+
+test('reports malformed source.md tracking file content', () => {
+  const workspaceRoot = createWorkspace('compiled-sources-malformed-tracking-file', {
+    '.xdrs/index.md': rootIndex(['[my-compiled](my-compiled/index.md)']),
+    '.xdrs/my-compiled/index.md': compiledScopeIndex(),
+    '.xdrs/my-compiled/adrs/index.md': compiledAdrIndex([
+      '- [001-core](principles/001-core.md) - meta-policy'
+    ]),
+    '.xdrs/my-compiled/adrs/principles/001-core.md': validCompiledMetaPolicy(),
+    '.xdrs/my-compiled/adrs/principles/.assets/sources/example-standards/source.md': [
+      'last-fetch-timestamp: not-a-date',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('must start with a "# Source" heading');
+  expect(result.errors.join('\n')).toContain('must include a "last-compilation-timestamp:" line');
+  expect(result.errors.join('\n')).toContain('must match "YYYY-MM-DD HH:MM:SS"');
+  expect(result.errors.join('\n')).toContain('019.12-source-storage-tracking');
+});
+
+test('reports document linking directly into a compiled-source snapshot path', () => {
+  const workspaceRoot = createWorkspace('source-snapshot-link-forbidden', {
+    '.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('See [raw source](.assets/sources/example-standards/raw-standard.md).'),
+    '.xdrs/_local/adrs/principles/.assets/sources/example-standards/raw-standard.md': '# Raw fetched content\n',
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('must not link directly to compiled-source snapshot content');
+  expect(result.errors.join('\n')).toContain('019.14-source-not-authoritative');
+});
+
 test('enforces compiled scope checks when scope-type is comma-separated with compiled', () => {
   const workspaceRoot = createWorkspace('compiled-multi-type-checks', {
     '.xdrs/index.md': rootIndex(['[my-compiled-scope](my-compiled-scope/index.md)']),
@@ -3226,6 +4182,71 @@ function skillDocument(body) {
   ].join('\n');
 }
 
+function fullSkillDocument(overrides = {}) {
+  const {
+    omitInputs = false,
+    omitInputsOptional = false,
+    swapInputsOrder = false,
+    inputsRequired = '- A file path to scan',
+    inputsOptional = '- None',
+    runtimeRequirements = null,
+    omitOutputs = false,
+    omitOutputsChanges = false,
+    outputsContents = '- A list of broken links',
+    outputsChanges = '- None',
+    omitHaltConditions = false,
+    haltConditions = '- None',
+    userInteraction = null,
+  } = overrides;
+
+  const lines = ['---', 'name: check-links', 'description: Test skill document', '---', '', '## Overview', '', 'Test skill overview.', ''];
+
+  if (!omitInputs) {
+    lines.push('### Inputs', '');
+    if (omitInputsOptional) {
+      lines.push('#### Required', inputsRequired, '');
+    } else if (swapInputsOrder) {
+      lines.push('#### Optional', inputsOptional, '', '#### Required', inputsRequired, '');
+    } else {
+      lines.push('#### Required', inputsRequired, '', '#### Optional', inputsOptional, '');
+    }
+  }
+
+  if (!omitOutputs) {
+    lines.push('### Outputs', '');
+    if (omitOutputsChanges) {
+      lines.push('#### Contents', outputsContents, '');
+    } else {
+      lines.push('#### Contents', outputsContents, '', '#### Changes', outputsChanges, '');
+    }
+  }
+
+  if (!omitHaltConditions) {
+    lines.push('### Halt Conditions', '', haltConditions, '');
+  }
+
+  if (userInteraction !== null) {
+    lines.push('### User Interaction', '', userInteraction, '');
+  }
+
+  if (runtimeRequirements !== null) {
+    lines.push('### Runtime Requirements', '', runtimeRequirements, '');
+  }
+
+  lines.push('## Instructions', '', 'Test instructions.', '');
+
+  lines.push(
+    '## Anti-Patterns',
+    '',
+    '- **Mistake:** First mistake. **Why it happens:** Reason. **Instead:** Fix.',
+    '- **Mistake:** Second mistake. **Why it happens:** Reason. **Instead:** Fix.',
+    '- **Mistake:** Third mistake. **Why it happens:** Reason. **Instead:** Fix.',
+    ''
+  );
+
+  return lines.join('\n');
+}
+
 test('accepts a custom root folder name as the XDRS root when it contains index.md', () => {
   const doc = [
     '---',
@@ -3716,8 +4737,8 @@ test('reports research exceeding 5000 word limit', () => {
   expect(result.errors.join('\n')).toContain('Research exceeds maximum word count of 5000');
 });
 
-test('reports SKILL.md exceeding 6500 word limit', () => {
-  const longBody = ('word '.repeat(6501)).trimEnd();
+test('reports SKILL.md exceeding 7000 word limit', () => {
+  const longBody = ('word '.repeat(7001)).trimEnd();
   const workspaceRoot = createWorkspace('skill-too-many-words', {
     '.xdrs/index.md': rootIndex(),
     '.xdrs/_local/adrs/index.md': localAdrIndex([
@@ -3728,7 +4749,7 @@ test('reports SKILL.md exceeding 6500 word limit', () => {
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('Skill exceeds maximum word count of 6500');
+  expect(result.errors.join('\n')).toContain('Skill exceeds maximum word count of 7000');
 });
 
 // ─── Files outside .assets in skill package root ────────────────────────────