xdrs-core 0.54.0 → 0.55.0

package/.xdrs/_core/adrs/principles/003-skill-standards.md

@@ -184,7 +184,6 @@ Rules:
 **Script standards**
 
 When a skill includes `scripts/`:
-- Each script MUST declare its runtime/interpreter at the top (shebang or equivalent header comment).
 - SHOULD use the language already used by the consuming project when known, rather than mandating one language repo-wide.
 - MUST avoid non-essential external dependencies, or document install steps in `SKILL.md` when unavoidable.
 - SHOULD support both human-readable and machine-readable (JSON) output when producing analysis or report output.

package/lib/lint.js

@@ -50,7 +50,6 @@ const SKILL_OPTIONAL_RUNTIME_SECTION = '### Runtime Requirements';
 const SKILL_OPTIONAL_USER_INTERACTION_SECTION = '### User Interaction';
 const SKILL_ANTI_PATTERNS_MIN_ENTRIES = 3;
 const SKILL_BUNDLING_MAKEFILE_TARGETS = ['build', 'test', 'clean'];
-const SCRIPT_HEADER_RE = /^(#!|#|\/\/|\/\*|<!--|--|;)/;
 
 // Subject number block ranges per _core-adr-policy-017
 const SUBJECT_NUMBER_RANGES = {
@@ -1264,10 +1263,6 @@ function lintSkillsDirectory(xdrsRoot, scopeName, skillsPath, errors, externalSc
         }
       }
 
-      if (packageEntry.name === 'scripts') {
-        lintSkillScriptHeaders(packageEntryPath, errors);
-      }
-
       if (SKILL_PACKAGE_OPTIONAL_DIRS.has(packageEntry.name)) {
         continue;
       }
@@ -1637,24 +1632,6 @@ function lintSkillAntiPatternsCount(content, filePath, errors) {
   }
 }
 
-/**
- * Validates that each file inside a skill's scripts/ directory declares its runtime or
- * interpreter at the top (shebang or header comment) per _core-adr-policy-003.
- */
-function lintSkillScriptHeaders(scriptsPath, errors) {
-  const entries = safeReadDir(scriptsPath, errors, `read scripts directory ${toDisplayPath(scriptsPath)}`);
-  for (const entry of entries) {
-    if (!entry.isFile()) {
-      continue;
-    }
-    const scriptPath = path.join(scriptsPath, entry.name);
-    const content = fs.readFileSync(scriptPath, 'utf8');
-    if (!SCRIPT_HEADER_RE.test(firstNonEmptyLine(content))) {
-      errors.push(`Script must declare its runtime/interpreter at the top (shebang or header comment): ${toDisplayPath(scriptPath)} [_core-adr-policy-003]`);
-    }
-  }
-}
-
 /**
  * Validates an optional skill bundling Makefile per _core-adr-policy-021: it must expose
  * build/test/clean targets, and its dist/ output must be covered by a .gitignore.

package/lib/lint.test.js

@@ -866,7 +866,7 @@ test('passes when skill bundling Makefile dist output is covered by a local .git
   expect(result.errors.join('\n')).not.toContain('is not covered by any .gitignore');
 });
 
-test('reports scripts/ file missing a shebang or header comment', () => {
+test('does not require a shebang or header comment in scripts/ files', () => {
   const workspaceRoot = createWorkspace('skill-script-missing-header', {
     '.xdrs/index.md': rootIndex(),
     '.xdrs/_local/adrs/index.md': localAdrIndex([
@@ -881,26 +881,7 @@ test('reports scripts/ file missing a shebang or header comment', () => {
 
   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');
+  expect(result.errors.join('\n')).not.toContain('scripts/run.sh');
 });
 
 test('accepts skill package names containing digits, including as a prefix', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xdrs-core",
-  "version": "0.54.0",
+  "version": "0.55.0",
   "description": "XDRS (eXtensible Decision Record System): 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",