siesa-agents 2.1.91 → 2.1.92

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);
   }
 }