refacil-sdd-ai 4.5.7 → 4.5.8

package/lib/commands/sdd.js

@@ -62,6 +62,47 @@ function validateChangeName(name) {
   return { valid: true };
 }
 
+function resolveExistingChangeName(projectRoot, inputName) {
+  if (!inputName || typeof inputName !== 'string') {
+    return { ok: false, reason: 'El nombre del cambio no puede estar vacío.' };
+  }
+
+  const normalizedInput = inputName.trim();
+  const lowerInput = normalizedInput.toLowerCase();
+  const changesDir = path.join(projectRoot, 'refacil-sdd', 'changes');
+
+  // Keep backward-compatible behavior when directory doesn't exist yet.
+  if (!fs.existsSync(changesDir)) {
+    return { ok: true, name: lowerInput };
+  }
+
+  const entries = fs.readdirSync(changesDir, { withFileTypes: true })
+    .filter((e) => e.isDirectory() && e.name !== 'archive')
+    .map((e) => e.name);
+
+  if (entries.includes(normalizedInput)) {
+    return { ok: true, name: normalizedInput };
+  }
+
+  if (entries.includes(lowerInput)) {
+    return { ok: true, name: lowerInput };
+  }
+
+  const ciMatches = entries.filter((n) => n.toLowerCase() === lowerInput);
+  if (ciMatches.length === 1) {
+    return { ok: true, name: ciMatches[0] };
+  }
+
+  if (ciMatches.length > 1) {
+    return {
+      ok: false,
+      reason: `Nombre de cambio ambiguo: '${inputName}'. Coincidencias: ${ciMatches.join(', ')}`,
+    };
+  }
+
+  return { ok: true, name: lowerInput };
+}
+
 function autoMigrateOpenspec(root) {
   const oldDir = path.join(root, 'openspec');
   const newDir = path.join(root, 'refacil-sdd');
@@ -180,7 +221,15 @@ function cmdNewChange(argv, projectRoot) {
 
 function cmdArchive(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
+
+  autoMigrateOpenspec(projectRoot);
+  const resolved = resolveExistingChangeName(projectRoot, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
 
   const validation = validateChangeName(name);
   if (!validation.valid) {
@@ -188,8 +237,6 @@ function cmdArchive(argv, projectRoot) {
     process.exit(1);
   }
 
-  autoMigrateOpenspec(projectRoot);
-
   const sourceDir = path.join(projectRoot, 'refacil-sdd', 'changes', name);
   if (!fs.existsSync(sourceDir)) {
     console.error(`No existe el cambio '${name}' en refacil-sdd/changes/${name}/`);
@@ -222,14 +269,21 @@ function cmdArchive(argv, projectRoot) {
 
 function cmdSetMemory(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd set-memory <nombre-cambio> [--last-step <value>] [--stack-detected <value>] [--touched-files <csv>] [--commands-run <value>] [--criteria-run <csv>]');
     process.exit(1);
   }
 
   const root = projectRoot;
+  autoMigrateOpenspec(root);
+  const resolved = resolveExistingChangeName(root, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
 
   // Guard: ensure the change directory exists before any file operation
   const changeDir = path.join(root, 'refacil-sdd', 'changes', name);
@@ -274,15 +328,22 @@ function cmdSetMemory(argv, projectRoot) {
 
 function cmdGetMemory(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
   const wantJson = args.json === true;
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd get-memory <nombre-cambio> [--json]');
     process.exit(1);
   }
 
   const root = projectRoot;
+  autoMigrateOpenspec(root);
+  const resolved = resolveExistingChangeName(root, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
   const memoryPath = path.join(root, 'refacil-sdd', 'changes', name, 'memory.yaml');
 
   if (!fs.existsSync(memoryPath)) {
@@ -309,14 +370,21 @@ function cmdGetMemory(argv, projectRoot) {
 
 function cmdSetReviewFails(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd set-review-fails <nombre-cambio> --files <csv>');
     process.exit(1);
   }
 
   const root = projectRoot;
+  autoMigrateOpenspec(root);
+  const resolved = resolveExistingChangeName(root, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
   const changeDir = path.join(root, 'refacil-sdd', 'changes', name);
   if (!fs.existsSync(changeDir)) {
     console.error(`No existe el cambio '${name}' en refacil-sdd/changes/${name}/`);
@@ -334,14 +402,21 @@ function cmdSetReviewFails(argv, projectRoot) {
 
 function cmdClearReviewFails(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd clear-review-fails <nombre-cambio>');
     process.exit(1);
   }
 
   const root = projectRoot;
+  autoMigrateOpenspec(root);
+  const resolved = resolveExistingChangeName(root, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
   const reviewFailsPath = path.join(root, 'refacil-sdd', 'changes', name, '.review-last-fails.json');
 
   if (fs.existsSync(reviewFailsPath)) {
@@ -392,15 +467,21 @@ function cmdList(argv, projectRoot) {
 
 function cmdStatus(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
   const wantJson = args.json === true;
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd status <nombre-cambio> [--json]');
     process.exit(1);
   }
 
   autoMigrateOpenspec(projectRoot);
+  const resolved = resolveExistingChangeName(projectRoot, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
 
   const changeDir = path.join(projectRoot, 'refacil-sdd', 'changes', name);
   if (!fs.existsSync(changeDir)) {
@@ -481,9 +562,9 @@ function cmdStatus(argv, projectRoot) {
 
 function cmdMarkReviewed(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd mark-reviewed <nombre-cambio> --verdict <verdict> --summary "<resumen>" [--fail-count N] [--preexisting-count N] [--blockers]');
     process.exit(1);
   }
@@ -499,6 +580,12 @@ function cmdMarkReviewed(argv, projectRoot) {
   }
 
   autoMigrateOpenspec(projectRoot);
+  const resolved = resolveExistingChangeName(projectRoot, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
 
   const changeDir = path.join(projectRoot, 'refacil-sdd', 'changes', name);
   if (!fs.existsSync(changeDir)) {
@@ -522,9 +609,9 @@ function cmdMarkReviewed(argv, projectRoot) {
 
 function cmdTasksUpdate(argv, projectRoot) {
   const args = parseArgs(argv);
-  const name = args._positional[0];
+  const rawName = args._positional[0];
 
-  if (!name) {
+  if (!rawName) {
     console.error('Uso: refacil-sdd-ai sdd tasks-update <nombre-cambio> --task N --done');
     process.exit(1);
   }
@@ -541,6 +628,12 @@ function cmdTasksUpdate(argv, projectRoot) {
   }
 
   autoMigrateOpenspec(projectRoot);
+  const resolved = resolveExistingChangeName(projectRoot, rawName);
+  if (!resolved.ok) {
+    console.error(resolved.reason);
+    process.exit(1);
+  }
+  const name = resolved.name;
 
   const tasksFile = path.join(projectRoot, 'refacil-sdd', 'changes', name, 'tasks.md');
   if (!fs.existsSync(tasksFile)) {
@@ -661,4 +754,4 @@ function handleSdd(sub, argv, projectRoot) {
   }
 }
 
-module.exports = { handleSdd, parseArgs, autoMigrateOpenspec, validateChangeName, findProjectRoot };
+module.exports = { handleSdd, parseArgs, autoMigrateOpenspec, validateChangeName, resolveExistingChangeName, findProjectRoot };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "refacil-sdd-ai",
-  "version": "4.5.7",
+  "version": "4.5.8",
   "description": "SDD-AI: Specification-Driven Development with AI — development methodology using AI with Claude Code, Cursor and OpenCode",
   "bin": {
     "refacil-sdd-ai": "./bin/cli.js"

package/skills/archive/SKILL.md

@@ -22,7 +22,7 @@ Verify the change is truly complete:
 
 2. **Tests pass**: Resolve and run the test command according to `refacil-prereqs/METHODOLOGY-CONTRACT.md`. If there are failing tests, inform and ask if they want to continue.
 
-3. **No pending files**: Run `git status` and verify if there are uncommitted changes related to the feature. If there are, suggest committing before archiving.
+3. **Working tree scope hygiene**: Run `git status` and check whether there are files unrelated to the current change scope. It is expected to have uncommitted changes in this step. If unrelated files are detected, warn the user and ask whether to continue archiving anyway. Do not suggest commit in this step; commit/push decisions are handled in `refacil:up-code`.
 
 4. **Review approved (blocking)**: Verify that the `.review-passed` file exists in the change folder (`refacil-sdd/changes/[change-name]/.review-passed`) following **`METHODOLOGY-CONTRACT.md` §8** (dotfile; do not conclude by listings without dotfiles). If it does NOT exist, **stop the archiving** and inform the user:
    ```
@@ -126,6 +126,8 @@ Bug fixes only contain `summary.md` (and optionally `.review-passed`). The CLI `
 
 The spec and review evidence are written **before** running the CLI archive command, while the artifacts are still at their original paths. The CLI only moves the folder — it never syncs specs.
 
+`refacil-sdd-ai sdd archive` normalizes the provided `changeName` to lowercase before validation/path resolution. Prefer lowercase names for consistency across commands and records.
+
 1. **Sync spec to `refacil-sdd/specs/` (before archiving)**:
    - Read `refacil-sdd/changes/<changeName>/specs.md` (and all `.md` under `specs/` if that subfolder exists).
    - Determine the spec folder name: use `<changeName>` unless a more descriptive name is clearly better.