siesa-agents 2.1.91 → 2.1.93

package/siesa-agents/bmm/workflows/4-implementation/code-review/steps/step-07-commit-push.md

@@ -98,18 +98,30 @@ To securely persist the verified and approved code changes to the repository fol
 
   2. **Construct Commit Message**:
      <action>
-     Per Section 3 of `{gitFlowGuide}`, use Conventional Commits format:
-     - For features: `feat: {description}`
-     - For fixes: `fix: {description}`
-
-     Derive type from story context and construct message.
+     Build the message STRICTLY from Section 3 (template) and Section 6 (deterministic
+     derivation) of `{gitFlowGuide}`. Do NOT redefine or invent the format here — the
+     guide is the single source of truth.
+
+     1. Parse `{{story_key}}` into `NNN`, `E`, `HH` per Section 6.
+     2. Resolve `<feature_scope>` per Section 6.1: FIRST look up `FEATURE_CODE_JIRA` in the
+        epic shard (then the PRD shard). If it is a real synced Jira issue key (NOT
+        `PENDING:...`) → use it as the scope. Otherwise fall back to `feat-<NNN3>`.
+     3. Fill the template:
+        `<tipo>(<feature_scope>): <descripción>` + `Epic:` and `Story:` trailers.
+     4. Take epic/story titles literally from the epic/story files; if absent, omit the
+        `— título` (never invent one).
+     5. Run the Section 6 validation checklist. If any item fails, STOP and report the
+        missing field instead of committing.
+
+     Store the result as `{{commit_message}}` (subject + trailers).
      </action>
 
   3. **User Confirmation**:
      <output>
      READY TO COMMIT:
      - Branch: {{current_branch}}
-     - Message: "feat: implementation for story {{story_key}}"
+     - Message:
+     {{commit_message}}
 
      Do you want to proceed with the commit and push?
      </output>
@@ -129,7 +141,7 @@ To securely persist the verified and approved code changes to the repository fol
      </logic>
 
   4. **Commit Changes**:
-     <action>Run command `git commit -m "feat: implementation for story {{story_key}}"`</action>
+     <action>Commit using `{{commit_message}}` (one `-m` for the subject and one `-m` per trailer line, per the example in Section 3.2 of `{gitFlowGuide}`).</action>
      <check if="Commit failed (no changes?)">
         <output>ℹ️ No changes to commit.</output>
      </check>

package/siesa-agents/scripts/phases/phase1.js

@@ -11,6 +11,35 @@ function run(command, cwd) {
   return execSync(command, { cwd, stdio: 'pipe' }).toString().trim();
 }
 
+// Extrae el detalle real de un error de execSync. Con stdio:'pipe', git escribe el
+// motivo (identidad faltante, "nothing to commit", etc.) en stdout/stderr, NO en
+// error.message — que solo dice "Command failed: <cmd>". Sin esto el usuario ve un
+// fallo sin causa. Preferimos stderr y caemos a stdout.
+function gitErr(error) {
+  const err = (error.stderr ? error.stderr.toString() : '').trim();
+  const out = (error.stdout ? error.stdout.toString() : '').trim();
+  return [err, out].filter(Boolean).join('\n') || error.message;
+}
+
+// En una máquina recién configurada (git init sin ~/.gitconfig) `git commit` falla
+// con "unable to auto-detect email address". Si no hay identidad, seteamos una local
+// de respaldo para que el flujo no se caiga, y avisamos para que el usuario ponga la suya.
+function ensureGitIdentity(projectRoot) {
+  const has = (key) => {
+    try { return run(`git config ${key}`, projectRoot).length > 0; }
+    catch { return false; }
+  };
+  if (has('user.email') && has('user.name')) return;
+  console.warn('⚠️  Git no tiene identidad configurada; usando una de respaldo local.');
+  console.warn('   Configura la tuya: git config user.name "Tu Nombre" && git config user.email tu@correo');
+  try {
+    if (!has('user.name')) run('git config user.name "Siesa Agents"', projectRoot);
+    if (!has('user.email')) run('git config user.email "agents@siesa.com"', projectRoot);
+  } catch (e) {
+    console.warn('   No se pudo setear identidad de respaldo:', gitErr(e));
+  }
+}
+
 function createDiscoveryBranch(projectRoot) {
   console.log(`\n🔍 Phase 1 - Creando rama "${BRANCH_NAME}"...`);
 
@@ -78,19 +107,28 @@ function commitAll(comentario, projectRoot) {
 
   console.log('\n📦 Agregando todos los archivos al stage...');
   try {
-    run('git add .', projectRoot);
-    console.log('✓ git add . ejecutado correctamente.');
+    run('git add -A', projectRoot);
+    console.log('✓ git add ejecutado correctamente.');
   } catch (error) {
-    console.error('❌ Error al ejecutar git add .:', error.message);
+    console.error('❌ Error al ejecutar git add:\n' + gitErr(error));
     process.exit(1);
   }
 
+  ensureGitIdentity(projectRoot);
+
   console.log(`\n💾 Creando commit: "${comentario}"...`);
   try {
     run(`git commit -m "${comentario.replace(/"/g, '\\"')}"`, projectRoot);
     console.log('✅ Commit creado correctamente.');
   } catch (error) {
-    console.error('❌ Error al crear el commit:', error.message);
+    const detail = gitErr(error);
+    // "nothing to commit, working tree clean" no es un error del flujo: no había
+    // cambios que commitear, así que continuamos sin abortar.
+    if (/working tree clean|nada para (confirmar|hacer commit)/i.test(detail)) {
+      console.log('ℹ️  No había cambios nuevos para commitear; continuando.');
+      return;
+    }
+    console.error('❌ Error al crear el commit:\n' + detail);
     process.exit(1);
   }
 }

package/siesa-agents/scripts/phases/phase2.js

@@ -11,6 +11,35 @@ function run(command, cwd) {
   return execSync(command, { cwd, stdio: 'pipe' }).toString().trim();
 }
 
+// Extrae el detalle real de un error de execSync. Con stdio:'pipe', git escribe el
+// motivo (identidad faltante, "nothing to commit", etc.) en stdout/stderr, NO en
+// error.message — que solo dice "Command failed: <cmd>". Sin esto el usuario ve un
+// fallo sin causa. Preferimos stderr y caemos a stdout.
+function gitErr(error) {
+  const err = (error.stderr ? error.stderr.toString() : '').trim();
+  const out = (error.stdout ? error.stdout.toString() : '').trim();
+  return [err, out].filter(Boolean).join('\n') || error.message;
+}
+
+// En una máquina recién configurada (git init sin ~/.gitconfig) `git commit` falla
+// con "unable to auto-detect email address". Si no hay identidad, seteamos una local
+// de respaldo para que el flujo no se caiga, y avisamos para que el usuario ponga la suya.
+function ensureGitIdentity(projectRoot) {
+  const has = (key) => {
+    try { return run(`git config ${key}`, projectRoot).length > 0; }
+    catch { return false; }
+  };
+  if (has('user.email') && has('user.name')) return;
+  console.warn('⚠️  Git no tiene identidad configurada; usando una de respaldo local.');
+  console.warn('   Configura la tuya: git config user.name "Tu Nombre" && git config user.email tu@correo');
+  try {
+    if (!has('user.name')) run('git config user.name "Siesa Agents"', projectRoot);
+    if (!has('user.email')) run('git config user.email "agents@siesa.com"', projectRoot);
+  } catch (e) {
+    console.warn('   No se pudo setear identidad de respaldo:', gitErr(e));
+  }
+}
+
 function createPlanningBranch(projectRoot) {
   console.log(`\n📋 Phase 2 - Creando rama "${BRANCH_NAME}"...`);
 
@@ -78,19 +107,28 @@ function commitAll(comentario, projectRoot) {
 
   console.log('\n📦 Agregando todos los archivos al stage...');
   try {
-    run('git add .', projectRoot);
-    console.log('✓ git add . ejecutado correctamente.');
+    run('git add -A', projectRoot);
+    console.log('✓ git add ejecutado correctamente.');
   } catch (error) {
-    console.error('❌ Error al ejecutar git add .:', error.message);
+    console.error('❌ Error al ejecutar git add:\n' + gitErr(error));
     process.exit(1);
   }
 
+  ensureGitIdentity(projectRoot);
+
   console.log(`\n💾 Creando commit: "${comentario}"...`);
   try {
     run(`git commit -m "${comentario.replace(/"/g, '\\"')}"`, projectRoot);
     console.log('✅ Commit creado correctamente.');
   } catch (error) {
-    console.error('❌ Error al crear el commit:', error.message);
+    const detail = gitErr(error);
+    // "nothing to commit, working tree clean" no es un error del flujo: no había
+    // cambios que commitear, así que continuamos sin abortar.
+    if (/working tree clean|nada para (confirmar|hacer commit)/i.test(detail)) {
+      console.log('ℹ️  No había cambios nuevos para commitear; continuando.');
+      return;
+    }
+    console.error('❌ Error al crear el commit:\n' + detail);
     process.exit(1);
   }
 }

package/siesa-agents/scripts/phases/phase3.js

@@ -11,6 +11,35 @@ function run(command, cwd) {
   return execSync(command, { cwd, stdio: 'pipe' }).toString().trim();
 }
 
+// Extrae el detalle real de un error de execSync. Con stdio:'pipe', git escribe el
+// motivo (identidad faltante, "nothing to commit", etc.) en stdout/stderr, NO en
+// error.message — que solo dice "Command failed: <cmd>". Sin esto el usuario ve un
+// fallo sin causa. Preferimos stderr y caemos a stdout.
+function gitErr(error) {
+  const err = (error.stderr ? error.stderr.toString() : '').trim();
+  const out = (error.stdout ? error.stdout.toString() : '').trim();
+  return [err, out].filter(Boolean).join('\n') || error.message;
+}
+
+// En una máquina recién configurada (git init sin ~/.gitconfig) `git commit` falla
+// con "unable to auto-detect email address". Si no hay identidad, seteamos una local
+// de respaldo para que el flujo no se caiga, y avisamos para que el usuario ponga la suya.
+function ensureGitIdentity(projectRoot) {
+  const has = (key) => {
+    try { return run(`git config ${key}`, projectRoot).length > 0; }
+    catch { return false; }
+  };
+  if (has('user.email') && has('user.name')) return;
+  console.warn('⚠️  Git no tiene identidad configurada; usando una de respaldo local.');
+  console.warn('   Configura la tuya: git config user.name "Tu Nombre" && git config user.email tu@correo');
+  try {
+    if (!has('user.name')) run('git config user.name "Siesa Agents"', projectRoot);
+    if (!has('user.email')) run('git config user.email "agents@siesa.com"', projectRoot);
+  } catch (e) {
+    console.warn('   No se pudo setear identidad de respaldo:', gitErr(e));
+  }
+}
+
 function createSolutioningBranch(projectRoot) {
   console.log(`\n🛠️ Phase 3 - Creando rama "${BRANCH_NAME}"...`);
 
@@ -78,19 +107,28 @@ function commitAll(comentario, projectRoot) {
 
   console.log('\n📦 Agregando todos los archivos al stage...');
   try {
-    run('git add .', projectRoot);
-    console.log('✓ git add . ejecutado correctamente.');
+    run('git add -A', projectRoot);
+    console.log('✓ git add ejecutado correctamente.');
   } catch (error) {
-    console.error('❌ Error al ejecutar git add .:', error.message);
+    console.error('❌ Error al ejecutar git add:\n' + gitErr(error));
     process.exit(1);
   }
 
+  ensureGitIdentity(projectRoot);
+
   console.log(`\n💾 Creando commit: "${comentario}"...`);
   try {
     run(`git commit -m "${comentario.replace(/"/g, '\\"')}"`, projectRoot);
     console.log('✅ Commit creado correctamente.');
   } catch (error) {
-    console.error('❌ Error al crear el commit:', error.message);
+    const detail = gitErr(error);
+    // "nothing to commit, working tree clean" no es un error del flujo: no había
+    // cambios que commitear, así que continuamos sin abortar.
+    if (/working tree clean|nada para (confirmar|hacer commit)/i.test(detail)) {
+      console.log('ℹ️  No había cambios nuevos para commitear; continuando.');
+      return;
+    }
+    console.error('❌ Error al crear el commit:\n' + detail);
     process.exit(1);
   }
 }