xdrs-core 0.28.1 → 0.28.2

package/.xdrs/_core/adrs/principles/skills/002-write-policy/SKILL.md

@@ -174,7 +174,7 @@ If any check fails, revise and re-run this phase before proceeding.
 
 1. Run the CLI lint utility from the repository root:
    ```
-   npx -y xdrs-core lint .
+   npx -y xdrs-core@latest lint
    ```
 2. Fix all reported errors before considering the task complete.
 3. Review warnings; fix straightforward ones and note intentional deviations explicitly.

package/.xdrs/_core/adrs/principles/skills/003-write-skill/SKILL.md

@@ -125,6 +125,14 @@ If any check fails, revise before continuing.
    ```
 3. Evaluate whether the scope index at `.xdrs/[scope]/index.md` should be updated to reflect the new skill. If the scope index does not exist, create it following article standards and the scope index rules in `_core-adr-policy-001`.
 
+### Phase 7: Verify with Lint
+
+1. Run the CLI lint utility from the repository root:
+   ```
+   npx -y xdrs-core@latest lint
+   ```
+2. Fix all reported errors before considering the task complete.
+
 ### Constraints
 
 - MUST follow the agentskills SKILL.md format from `003-skill-standards` exactly.

package/.xdrs/_core/adrs/principles/skills/004-write-article/SKILL.md

@@ -110,6 +110,14 @@ Rules to apply while drafting:
    section.
 4. Evaluate whether the scope index at `.xdrs/[scope]/index.md` should be updated to reflect the new article. If the scope index does not exist, create it following article standards and the scope index rules in `_core-adr-policy-001`.
 
+### Phase 7: Verify with Lint
+
+1. Run the CLI lint utility from the repository root:
+   ```
+   npx -y xdrs-core@latest lint
+   ```
+2. Fix all reported errors before considering the task complete.
+
 ## Examples
 
 **Input:** "Write an article about how skills work in this project."

package/.xdrs/_core/adrs/principles/skills/005-write-research/SKILL.md

@@ -247,6 +247,14 @@ If any check fails, revise before continuing.
 3. Add back-references from the related Policy, article, or skill when the relationship is important for discovery.
 4. Evaluate whether the scope index at `.xdrs/[scope]/index.md` should be updated to reflect the new research. If the scope index does not exist, create it following article standards and the scope index rules in `_core-adr-policy-001`.
 
+### Phase 12: Verify with Lint
+
+1. Run the CLI lint utility from the repository root:
+   ```
+   npx -y xdrs-core@latest lint
+   ```
+2. Fix all reported errors before considering the task complete.
+
 ## Examples
 
 **Input**: "Create research comparing package distribution options for our Policy scopes"

package/.xdrs/_core/adrs/principles/skills/006-write-plan/SKILL.md

@@ -130,7 +130,7 @@ Rules to apply while drafting:
 
 1. Run the CLI lint utility from the repository root:
    ```
-   npx -y xdrs-core lint .
+   npx -y xdrs-core@latest lint
    ```
 2. Fix all reported errors before considering the task complete.
 

package/.xdrs/_core/adrs/principles/skills/007-write-presentation/SKILL.md

@@ -126,7 +126,7 @@ Rules:
 
 1. Run the CLI lint utility from the repository root:
    ```
-   npx -y xdrs-core lint .
+   npx -y xdrs-core@latest lint
    ```
 2. Fix all reported errors before considering the task complete.
 

package/.xdrs/index.md

@@ -19,4 +19,6 @@ Defines how XDRS framework work
 
 ### _local (reserved)
 
+_local scope is the default scope for new xdrs and might override other scope decisions. These decisions are local and are not supposed to be shared in other contexts.
+
 Read _local scope index at `_local/index.md` when it exists.

package/lib/lint.js

@@ -791,12 +791,6 @@ function lintOrphanAssets(assetsDir, documentPaths, xdrsRoot, errors) {
     return;
   }
 
-  if (assetTree.directories.length > 0 && assetTree.files.length <= 10) {
-    for (const directoryPath of assetTree.directories) {
-      errors.push(`.assets directory must be flat unless it already contains more than 10 files: ${toDisplayPath(directoryPath)}`);
-    }
-  }
-
   const repoRoot = path.dirname(xdrsRoot);
   const referencedAssets = new Set();
 
@@ -844,9 +838,10 @@ function lintSlideFile(filePath, documentPaths, xdrsRoot, errors) {
   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)}`);
+    const frontmatterBody = content.match(/^---\r?\n([\s\S]*?)\r?\n---/)[1];
+    const firstKey = frontmatterBody.trimStart().split(/\r?\n/)[0];
+    if (!/^marp:\s*true$/.test(firstKey)) {
+      errors.push(`Slide frontmatter must include marp: true as the first key: ${toDisplayPath(filePath)}`);
     }
   }
 

package/lib/lint.test.js

@@ -721,8 +721,8 @@ test('reports unexpected directories inside a skill package', () => {
   expect(result.errors.join('\n')).toContain('extras');
 });
 
-test('reports nested directories in .assets when it has 10 files or fewer', () => {
-  const workspaceRoot = createWorkspace('asset-subdir-too-small', {
+test('allows nested directories in .assets', () => {
+  const workspaceRoot = createWorkspace('asset-subdir-allowed', {
     '.xdrs/index.md': rootIndex(),
     '.xdrs/_local/index.md': '# _local Scope Overview\n\nOverview.\n\n[ADRs](adrs/index.md)\n',
     '.xdrs/_local/adrs/index.md': localAdrIndex([
@@ -738,64 +738,19 @@ test('reports nested directories in .assets when it has 10 files or fewer', () =
       '',
       '## Context and Problem Statement',
       '',
-      'See ![img](.assets/used.png).',
+      'See ![img](.assets/grouped/used.png).',
       '',
       '## Decision Outcome',
       '',
       'Test decision outcome.',
       ''
     ].join('\n'),
-    '.xdrs/_local/adrs/principles/.assets/used.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/grouped/extra.png': Buffer.alloc(0),
-  });
-
-  const result = lintWorkspace(workspaceRoot);
-
-  expect(result.errors.join('\n')).toContain('.assets directory must be flat unless it already contains more than 10 files');
-  expect(result.errors.join('\n')).toContain('.assets/grouped');
-});
-
-test('allows nested directories in .assets when it has more than 10 files', () => {
-  const workspaceRoot = createWorkspace('asset-subdir-large-enough', {
-    '.xdrs/index.md': rootIndex(),
-    '.xdrs/_local/index.md': '# _local Scope Overview\n\nOverview.\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': [
-      '---',
-      'name: _local-adr-policy-001-main-decision',
-      'description: Test XDRS element',
-      '---',
-      '',
-      '# _local-adr-policy-001: Main decision',
-      '',
-      '## Context and Problem Statement',
-      '',
-      'See ![img](.assets/used.png).',
-      '',
-      '## Decision Outcome',
-      '',
-      'Test decision outcome.',
-      ''
-    ].join('\n'),
-    '.xdrs/_local/adrs/principles/.assets/used.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/01.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/02.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/03.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/04.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/05.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/06.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/07.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/08.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/09.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/10.png': Buffer.alloc(0),
-    '.xdrs/_local/adrs/principles/.assets/grouped/11.png': Buffer.alloc(0),
+    '.xdrs/_local/adrs/principles/.assets/grouped/used.png': Buffer.alloc(0),
   });
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).not.toContain('.assets directory must be flat unless it already contains more than 10 files');
+  expect(result.errors.join('\n')).not.toContain('.assets directory must be flat');
 });
 
 function teamAdrIndex(entries) {
@@ -1346,7 +1301,32 @@ test('reports slide file missing marp: true in frontmatter', () => {
 
   const result = lintWorkspace(workspaceRoot);
 
-  expect(result.errors.join('\n')).toContain('Slide frontmatter must include marp: true');
+  expect(result.errors.join('\n')).toContain('Slide frontmatter must include marp: true as the first key');
+});
+
+test('reports slide file with marp: true not as the first frontmatter key', () => {
+  const workspaceRoot = createWorkspace('slide-marp-not-first', {
+    '.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',
+      'marp: true',
+      '---',
+      '',
+      '# Slides',
+      '',
+      '[Parent](../001-main.md)',
+      ''
+    ].join('\n'),
+  });
+
+  const result = lintWorkspace(workspaceRoot);
+
+  expect(result.errors.join('\n')).toContain('Slide frontmatter must include marp: true as the first key');
 });
 
 test('reports slide file without any frontmatter', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xdrs-core",
-  "version": "0.28.1",
+  "version": "0.28.2",
   "description": "A framework to structure, compile and distribute Architectural (ADR), Business (BDR), and Engineering (EDR) decision records contents so that AI agents and humans can reliably find and use them with hierarchical scopes and controlled rollout in the format of distributable versioned packages.",
   "repository": {
     "type": "git",
@@ -30,7 +30,7 @@
     "jest": "^29.7.0"
   },
   "dependencies": {
-    "filedist": "^0.34.1",
+    "filedist": "^0.34.2",
     "ignore": "^7.0.5",
     "minimatch": "^10.2.5"
   },