sdd-gl 0.6.1 → 0.6.2

package/README.es.md

@@ -3,7 +3,7 @@
 ![SDD-GL Portada](assets/sdd-gl-cover.png)
 
 [![npm](https://img.shields.io/npm/v/sdd-gl)](https://www.npmjs.com/package/sdd-gl)
-[![Version](https://img.shields.io/badge/version-0.6.1-blue)](CHANGELOG.md)
+[![Version](https://img.shields.io/badge/version-0.6.2-blue)](CHANGELOG.md)
 [![License](https://img.shields.io/badge/license-MIT-green)](LICENSE)
 [![MCP Protocol](https://img.shields.io/badge/MCP-Protocol%20Ready-purple)](mcp/sdd-gl-mcp-spec.md)
 [![Claude Code](https://img.shields.io/badge/Claude%20Code-plugin-orange)](https://docs.claude.ai/code)

package/README.md

@@ -3,7 +3,7 @@
 ![SDD-GL Cover](assets/sdd-gl-cover.png)
 
 [![npm](https://img.shields.io/npm/v/sdd-gl)](https://www.npmjs.com/package/sdd-gl)
-[![Version](https://img.shields.io/badge/version-0.6.1-blue)](CHANGELOG.md)
+[![Version](https://img.shields.io/badge/version-0.6.2-blue)](CHANGELOG.md)
 [![License](https://img.shields.io/badge/license-MIT-green)](LICENSE)
 [![MCP Protocol](https://img.shields.io/badge/MCP-Protocol%20Ready-purple)](mcp/sdd-gl-mcp-spec.md)
 [![Claude Code](https://img.shields.io/badge/Claude%20Code-plugin-orange)](https://docs.claude.ai/code)

package/bin/sdd.js

@@ -32,7 +32,7 @@ const ROOT = rootFlag ? path.resolve(rootFlag) : process.cwd();
 
 process.env.SDD_PROJECT_ROOT = ROOT;
 
-const { handleToolCall, TOOLS } = require('../mcp/server.js');
+const { handleToolCall, TOOLS, isContractText, looksLikeContract } = require('../mcp/server.js');
 
 // CLI usage is real usage of the methodology: it is counted and attributed to the
 // contract type it serves (feature/fix) when a contract is involved.
@@ -588,17 +588,28 @@ function commandDoctor() {
 
   const unreadable = [];
   const stalled = [];
+  const ignored = [];
+  let contracts = 0;
 
   if (fs.existsSync(contractsDir)) {
     for (const name of fs.readdirSync(contractsDir).filter(entry => entry.toLowerCase().endsWith('.md'))) {
       const text = fs.readFileSync(path.join(contractsDir, name), 'utf8');
-      if (!/^#\s*ID:/m.test(text) || !/^#\s*Status:/m.test(text)) unreadable.push(name);
+
+      // `contracts/` may also hold a README or notes: only files with a contract
+      // header are counted, and only a broken contract header is a failure.
+      if (!isContractText(text)) {
+        if (looksLikeContract(text)) unreadable.push(name);
+        else ignored.push(name);
+        continue;
+      }
+
+      contracts += 1;
       if (/^\s*#.*\|\s*⏳\s*$/m.test(text)) stalled.push(name);
     }
   }
 
   if (unreadable.length > 0) add('contratos', 'FAIL', `encabezado ilegible: ${unreadable.join(', ')}`);
-  else add('contratos', 'OK', 'todos los encabezados son legibles');
+  else add('contratos', 'OK', `${contracts} contrato(s) legibles${ignored.length ? ` · ignorados: ${ignored.join(', ')}` : ''}`);
 
   if (stalled.length > 0) add('en curso', 'WARN', `ítems en ⏳ (reanudar): ${stalled.join(', ')}`);
   else add('en curso', 'OK', 'sin ítems interrumpidos');
@@ -1320,6 +1331,7 @@ function commandReview() {
 
   for (const file of fs.readdirSync(contractsDir).filter(name => name.toLowerCase().endsWith('.md'))) {
     const text = fs.readFileSync(path.join(contractsDir, file), 'utf8');
+    if (!isContractText(text)) continue;
     const id = (text.match(/^#\s*ID:\s*(\S+)\s*$/m) || [])[1] || file.replace(/\.md$/, '');
     const status = (text.match(/^#\s*Status:\s*(\w+)\s*$/m) || [])[1] || 'UNKNOWN';
     const criteria = readCriteria(text);

package/mcp/server.js

@@ -327,12 +327,30 @@ function inferContractType(contractId, text, fileName) {
   return 'feature';
 }
 
+// A contract is recognized by its header, not by its extension: `contracts/` may
+// also hold a README or free-form notes that must be ignored.
+function isContractText(text) {
+  return /^#\s*ID:/m.test(text) && /^#\s*Status:/m.test(text);
+}
+
+// Files that *look* like contracts but cannot be read as one are real errors.
+function looksLikeContract(text) {
+  return /^#\s*CONTRACT:/m.test(text);
+}
+
 function listContractFiles() {
   if (!fs.existsSync(CONTRACTS_DIR)) return [];
   return fs
     .readdirSync(CONTRACTS_DIR, { withFileTypes: true })
     .filter(entry => entry.isFile() && entry.name.toLowerCase().endsWith('.md'))
-    .map(entry => entry.name);
+    .map(entry => entry.name)
+    .filter(name => {
+      try {
+        return isContractText(fs.readFileSync(path.join(CONTRACTS_DIR, name), 'utf8'));
+      } catch (error) {
+        return false;
+      }
+    });
 }
 
 // Finds a contract by ID regardless of the file naming convention
@@ -1483,7 +1501,7 @@ ${evidence}
 
     case 'sdd_get_status': {
       if (!fs.existsSync(CONTRACTS_DIR)) return { total_contracts: 0, contracts: [] };
-      const files = fs.readdirSync(CONTRACTS_DIR).filter(f => f.endsWith('.md'));
+      const files = listContractFiles();
 
       const summary = files.map(file => {
         const content = fs.readFileSync(path.join(CONTRACTS_DIR, file), 'utf8');
@@ -1517,7 +1535,7 @@ ${evidence}
       const response = { ...metrics, source };
       if (warning) response.warning = warning;
       if (fs.existsSync(CONTRACTS_DIR)) {
-        response.total_contracts = fs.readdirSync(CONTRACTS_DIR).filter(f => f.endsWith('.md')).length;
+        response.total_contracts = listContractFiles().length;
       } else {
         response.total_contracts = 0;
       }
@@ -1527,7 +1545,7 @@ ${evidence}
     case 'sdd_audit_traceability': {
       // BR-005: real coverage. Nothing here is a fixed value.
       const contractFiles = fs.existsSync(CONTRACTS_DIR)
-        ? fs.readdirSync(CONTRACTS_DIR).filter(file => file.endsWith('.md'))
+        ? listContractFiles()
         : [];
 
       const specs = [];
@@ -1665,7 +1683,7 @@ function main() {
   });
 }
 
-module.exports = { TOOLS, handleToolCall, computeNextContractId, reconcileContract, computeCriteriaHashes };
+module.exports = { TOOLS, handleToolCall, computeNextContractId, reconcileContract, computeCriteriaHashes, isContractText, looksLikeContract };
 
 if (require.main === module) {
   main();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sdd-gl",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Model Context Protocol (MCP) Server and CLI for Spec-Driven Development (SDD-GL)",
   "main": "mcp/server.js",
   "bin": {
@@ -35,7 +35,7 @@
     "dashboard": "node mcp/dashboard-gen.js",
     "dashboard:public": "node mcp/dashboard-gen.js --public",
     "fixture": "node test-sandbox/fixtures/demo-project.js",
-    "test": "node test-sandbox/id-generation.test.js && node test-sandbox/step-loop.test.js && node test-sandbox/mcp-core.test.js && node test-sandbox/neutral-format.test.js && node test-sandbox/atomicity.test.js && node test-sandbox/metrics.test.js && node test-sandbox/usage.test.js && node test-sandbox/fix-0001-docs.test.js && node test-sandbox/ci-workflow.test.js && node test-sandbox/interop.test.js && node test-sandbox/cli.test.js && node test-sandbox/fixture-project.test.js && node test-sandbox/split.test.js && node test-sandbox/reconcile.test.js && node test-sandbox/review.test.js && node test-sandbox/publish.test.js && node test-sandbox/multilayer-verifier.test.js && node test-sandbox/escrow.test.js && node test-sandbox/mcp-client-e2e.js",
+    "test": "node test-sandbox/id-generation.test.js && node test-sandbox/step-loop.test.js && node test-sandbox/mcp-core.test.js && node test-sandbox/neutral-format.test.js && node test-sandbox/atomicity.test.js && node test-sandbox/metrics.test.js && node test-sandbox/usage.test.js && node test-sandbox/fix-0001-docs.test.js && node test-sandbox/ci-workflow.test.js && node test-sandbox/interop.test.js && node test-sandbox/cli.test.js && node test-sandbox/fixture-project.test.js && node test-sandbox/split.test.js && node test-sandbox/reconcile.test.js && node test-sandbox/review.test.js && node test-sandbox/publish.test.js && node test-sandbox/doctor.test.js && node test-sandbox/orchestrator-docs.test.js && node test-sandbox/multilayer-verifier.test.js && node test-sandbox/escrow.test.js && node test-sandbox/mcp-client-e2e.js",
     "prepublishOnly": "npm test && node scripts/release.js --check",
     "release:check": "node scripts/release.js --check",
     "release:prepare": "node scripts/release.js"