siesa-agents 2.1.91 → 2.1.92

package/bmad/config.toml

@@ -0,0 +1,69 @@
+# ─────────────────────────────────────────────────────────────────
+# Installer-managed. Regenerated on every install — treat as read-only.
+#
+# Direct edits to this file will be overwritten on the next install.
+# To change an install answer durably, re-run the installer (your prior
+# answers are remembered as defaults). To pin a value regardless of
+# install answers, or to add custom agents / override descriptors, use:
+#   _bmad/custom/config.toml       (team, committed)
+#   _bmad/custom/config.user.toml  (personal, gitignored)
+# Those files are never touched by the installer.
+# ─────────────────────────────────────────────────────────────────
+
+[core]
+document_output_language = "English"
+output_folder = "{project-root}/_bmad-output"
+
+[modules.bmm]
+project_name = "Siesa-Agents"
+planning_artifacts = "{project-root}/_bmad-output/planning-artifacts"
+implementation_artifacts = "{project-root}/_bmad-output/implementation-artifacts"
+project_knowledge = "{project-root}/docs"
+
+[agents.bmad-agent-analyst]
+module = "bmm"
+team = "software-development"
+name = "Mary"
+title = "Business Analyst"
+icon = "📊"
+description = "Channels Porter's strategic rigor and Minto's Pyramid Principle, grounds every finding in verifiable evidence, represents every stakeholder voice. Speaks like a treasure hunter narrating the find: thrilled by every clue, precise once the pattern emerges."
+
+[agents.bmad-agent-tech-writer]
+module = "bmm"
+team = "software-development"
+name = "Paige"
+title = "Technical Writer"
+icon = "📚"
+description = "Master of CommonMark, DITA, and OpenAPI; turns complex concepts into accessible structured docs, favors diagrams over walls of text, every word earning its place. Speaks like the patient teacher you wish you'd had, using analogies that make complex things feel simple."
+
+[agents.bmad-agent-pm]
+module = "bmm"
+team = "software-development"
+name = "John"
+title = "Product Manager"
+icon = "📋"
+description = "Drives Jobs-to-be-Done over template filling, user value first, technical feasibility is a constraint not the driver. Speaks like a detective interrogating a cold case: short questions, sharper follow-ups, every 'why?' tightening the net."
+
+[agents.bmad-agent-ux-designer]
+module = "bmm"
+team = "software-development"
+name = "Sally"
+title = "UX Designer"
+icon = "🎨"
+description = "Balances empathy with edge-case rigor, starts simple and evolves through feedback, every decision serves a genuine user need. Speaks like a filmmaker pitching the scene before the code exists, painting user stories that make you feel the problem."
+
+[agents.bmad-agent-architect]
+module = "bmm"
+team = "software-development"
+name = "Winston"
+title = "System Architect"
+icon = "🏗️"
+description = "Favors boring technology for stability, developer productivity as architecture, ties every decision to business value. Speaks like a seasoned engineer at the whiteboard: measured, always laying out trade-offs rather than verdicts."
+
+[agents.bmad-agent-dev]
+module = "bmm"
+team = "software-development"
+name = "Amelia"
+title = "Senior Software Engineer"
+icon = "💻"
+description = "Test-first discipline (red, green, refactor), 100% pass before review, no fluff all precision. Speaks like a terminal prompt: exact file paths, AC IDs, and commit-message brevity — every statement citable."

package/bmad/config.user.toml

@@ -0,0 +1,17 @@
+# ─────────────────────────────────────────────────────────────────
+# Installer-managed. Regenerated on every install — treat as read-only.
+# Holds install answers scoped to YOU personally.
+#
+# Direct edits to this file will be overwritten on the next install.
+# To change an answer durably, re-run the installer (your prior answers
+# are remembered as defaults). For pinned overrides or custom sections
+# the installer does not know about, use _bmad/custom/config.user.toml
+# — it is never touched by the installer.
+# ─────────────────────────────────────────────────────────────────
+
+[core]
+user_name = "Siesa-Team"
+communication_language = "Spanish"
+
+[modules.bmm]
+user_skill_level = "intermediate"

package/bmad/core/config.yaml

@@ -0,0 +1,9 @@
+# CORE Module Configuration
+# Generated by BMAD installer
+# Version: 6.5.0
+# Date: 2026-04-28T20:53:00.884Z
+
+user_name: Siesa-Team
+communication_language: Spanish
+document_output_language: English
+output_folder: "{project-root}/_bmad-output"

package/bmad/core/module-help.csv

@@ -0,0 +1,13 @@
+module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs
+Core,_meta,,,,,,,,,false,https://docs.bmad-method.org/llms.txt,
+Core,bmad-brainstorming,Brainstorming,BSP,Use early in ideation or when stuck generating ideas.,,anytime,,,false,{output_folder}/brainstorming,brainstorming session
+Core,bmad-party-mode,Party Mode,PM,Orchestrate multi-agent discussions when you need multiple perspectives or want agents to collaborate.,,anytime,,,false,,
+Core,bmad-help,BMad Help,BH,,,anytime,,,false,,
+Core,bmad-index-docs,Index Docs,ID,Use when LLM needs to understand available docs without loading everything.,,anytime,,,false,,
+Core,bmad-shard-doc,Shard Document,SD,Use when doc becomes too large (>500 lines) to manage effectively.,[path],anytime,,,false,,
+Core,bmad-editorial-review-prose,Editorial Review - Prose,EP,Use after drafting to polish written content.,[path],anytime,,,false,report located with target document,three-column markdown table with suggested fixes
+Core,bmad-editorial-review-structure,Editorial Review - Structure,ES,Use when doc produced from multiple subprocesses or needs structural improvement.,[path],anytime,,,false,report located with target document,
+Core,bmad-review-adversarial-general,Adversarial Review,AR,"Use for quality assurance or before finalizing deliverables. Code Review in other modules runs this automatically, but also useful for document reviews.",[path],anytime,,,false,,
+Core,bmad-review-edge-case-hunter,Edge Case Hunter Review,ECH,Use alongside adversarial review for orthogonal coverage — method-driven not attitude-driven.,[path],anytime,,,false,,
+Core,bmad-distillator,Distillator,DG,Use when you need token-efficient distillates that preserve all information for downstream LLM consumption.,[path],anytime,,,false,adjacent to source document or specified output_path,distillate markdown file(s)
+Core,bmad-customize,BMad Customize,BC,"Use when you want to change how an agent or workflow behaves — add persistent facts, swap templates, insert activation hooks, or customize menus. Scans what's customizable, picks the right scope (agent vs workflow), writes the override to _bmad/custom/, and verifies the merge. No TOML hand-authoring required.",,anytime,,,false,{project-root}/_bmad/custom,TOML override files

package/bmad/custom/config.toml

@@ -0,0 +1,7 @@
+# Team / enterprise overrides for _bmad/config.toml.
+# Committed to the repo — applies to every developer on the project.
+# Tables deep-merge over base config; keyed entries merge by key.
+# Example: override an agent descriptor, or add a new agent.
+#
+# [agents.bmad-agent-pm]
+# description = "Prefers short, bulleted PRDs over narrative drafts."

package/bmad/scripts/resolve_config.py

@@ -0,0 +1,176 @@
+#!/usr/bin/env python3
+"""
+Resolve BMad's central config using four-layer TOML merge.
+
+Reads from four layers (highest priority last):
+  1. {project-root}/_bmad/config.toml              (installer-owned team)
+  2. {project-root}/_bmad/config.user.toml         (installer-owned user)
+  3. {project-root}/_bmad/custom/config.toml       (human-authored team, committed)
+  4. {project-root}/_bmad/custom/config.user.toml  (human-authored user, gitignored)
+
+Outputs merged JSON to stdout. Errors go to stderr.
+
+Requires Python 3.11+ (uses stdlib `tomllib`). No `uv`, no `pip install`,
+no virtualenv — plain `python3` is sufficient.
+
+  python3 resolve_config.py --project-root /abs/path/to/project
+  python3 resolve_config.py --project-root ... --key core
+  python3 resolve_config.py --project-root ... --key agents
+
+Merge rules (same as resolve_customization.py):
+  - Scalars: override wins
+  - Tables: deep merge
+  - Arrays of tables where every item shares `code` or `id`: merge by that key
+  - All other arrays: append
+"""
+
+import argparse
+import json
+import sys
+from pathlib import Path
+
+try:
+    import tomllib
+except ImportError:
+    sys.stderr.write(
+        "error: Python 3.11+ is required (stdlib `tomllib` not found).\n"
+    )
+    sys.exit(3)
+
+
+_MISSING = object()
+_KEYED_MERGE_FIELDS = ("code", "id")
+
+
+def load_toml(file_path: Path, required: bool = False) -> dict:
+    if not file_path.exists():
+        if required:
+            sys.stderr.write(f"error: required config file not found: {file_path}\n")
+            sys.exit(1)
+        return {}
+    try:
+        with file_path.open("rb") as f:
+            parsed = tomllib.load(f)
+        if not isinstance(parsed, dict):
+            return {}
+        return parsed
+    except tomllib.TOMLDecodeError as error:
+        level = "error" if required else "warning"
+        sys.stderr.write(f"{level}: failed to parse {file_path}: {error}\n")
+        if required:
+            sys.exit(1)
+        return {}
+    except OSError as error:
+        level = "error" if required else "warning"
+        sys.stderr.write(f"{level}: failed to read {file_path}: {error}\n")
+        if required:
+            sys.exit(1)
+        return {}
+
+
+def _detect_keyed_merge_field(items):
+    if not items or not all(isinstance(item, dict) for item in items):
+        return None
+    for candidate in _KEYED_MERGE_FIELDS:
+        if all(item.get(candidate) is not None for item in items):
+            return candidate
+    return None
+
+
+def _merge_by_key(base, override, key_name):
+    result = []
+    index_by_key = {}
+    for item in base:
+        if not isinstance(item, dict):
+            continue
+        if item.get(key_name) is not None:
+            index_by_key[item[key_name]] = len(result)
+        result.append(dict(item))
+    for item in override:
+        if not isinstance(item, dict):
+            result.append(item)
+            continue
+        key = item.get(key_name)
+        if key is not None and key in index_by_key:
+            result[index_by_key[key]] = dict(item)
+        else:
+            if key is not None:
+                index_by_key[key] = len(result)
+            result.append(dict(item))
+    return result
+
+
+def _merge_arrays(base, override):
+    base_arr = base if isinstance(base, list) else []
+    override_arr = override if isinstance(override, list) else []
+    keyed_field = _detect_keyed_merge_field(base_arr + override_arr)
+    if keyed_field:
+        return _merge_by_key(base_arr, override_arr, keyed_field)
+    return base_arr + override_arr
+
+
+def deep_merge(base, override):
+    if isinstance(base, dict) and isinstance(override, dict):
+        result = dict(base)
+        for key, over_val in override.items():
+            if key in result:
+                result[key] = deep_merge(result[key], over_val)
+            else:
+                result[key] = over_val
+        return result
+    if isinstance(base, list) and isinstance(override, list):
+        return _merge_arrays(base, override)
+    return override
+
+
+def extract_key(data, dotted_key: str):
+    parts = dotted_key.split(".")
+    current = data
+    for part in parts:
+        if isinstance(current, dict) and part in current:
+            current = current[part]
+        else:
+            return _MISSING
+    return current
+
+
+def main():
+    parser = argparse.ArgumentParser(
+        description="Resolve BMad central config using four-layer TOML merge.",
+    )
+    parser.add_argument(
+        "--project-root", "-p", required=True,
+        help="Absolute path to the project root (contains _bmad/)",
+    )
+    parser.add_argument(
+        "--key", "-k", action="append", default=[],
+        help="Dotted field path to resolve (repeatable). Omit for full dump.",
+    )
+    args = parser.parse_args()
+
+    project_root = Path(args.project_root).resolve()
+    bmad_dir = project_root / "_bmad"
+
+    base_team = load_toml(bmad_dir / "config.toml", required=True)
+    base_user = load_toml(bmad_dir / "config.user.toml")
+    custom_team = load_toml(bmad_dir / "custom" / "config.toml")
+    custom_user = load_toml(bmad_dir / "custom" / "config.user.toml")
+
+    merged = deep_merge(base_team, base_user)
+    merged = deep_merge(merged, custom_team)
+    merged = deep_merge(merged, custom_user)
+
+    if args.key:
+        output = {}
+        for key in args.key:
+            value = extract_key(merged, key)
+            if value is not _MISSING:
+                output[key] = value
+    else:
+        output = merged
+
+    sys.stdout.write(json.dumps(output, indent=2, ensure_ascii=False) + "\n")
+
+
+if __name__ == "__main__":
+    main()

package/bmad/scripts/resolve_customization.py

@@ -0,0 +1,230 @@
+#!/usr/bin/env python3
+"""
+Resolve customization for a BMad skill using three-layer TOML merge.
+
+Reads customization from three layers (highest priority first):
+  1. {project-root}/_bmad/custom/{name}.user.toml  (personal, gitignored)
+  2. {project-root}/_bmad/custom/{name}.toml        (team/org, committed)
+  3. {skill-root}/customize.toml                    (skill defaults)
+
+Skill name is derived from the basename of the skill directory.
+
+Outputs merged JSON to stdout. Errors go to stderr.
+
+Requires Python 3.11+ (uses stdlib `tomllib`). No `uv`, no `pip install`,
+no virtualenv — plain `python3` is sufficient.
+
+  python3 resolve_customization.py --skill /abs/path/to/skill-dir
+  python3 resolve_customization.py --skill ... --key agent
+  python3 resolve_customization.py --skill ... --key agent.menu
+
+Merge rules (purely structural — no field-name special-casing):
+  - Scalars (string, int, bool, float): override wins
+  - Tables: deep merge (recursively apply these rules)
+  - Arrays of tables where every item shares the *same* identifier
+    field (every item has `code`, or every item has `id`):
+    merge by that key (matching keys replace, new keys append)
+  - All other arrays — including arrays where only some items have
+    `code` or `id`, or where items mix the two keys:
+    append (base items followed by override items)
+
+No removal mechanism — overrides cannot delete base items. To suppress
+a default, fork the skill or override the item by code with a no-op
+description/prompt.
+"""
+
+import argparse
+import json
+import sys
+from pathlib import Path
+
+try:
+    import tomllib
+except ImportError:
+    sys.stderr.write(
+        "error: Python 3.11+ is required (stdlib `tomllib` not found).\n"
+        "Install a newer Python or run the resolution manually per the\n"
+        "fallback instructions in the skill's SKILL.md.\n"
+    )
+    sys.exit(3)
+
+
+_MISSING = object()
+_KEYED_MERGE_FIELDS = ("code", "id")
+
+
+def find_project_root(start: Path):
+    current = start.resolve()
+    while True:
+        if (current / "_bmad").exists() or (current / ".git").exists():
+            return current
+        parent = current.parent
+        if parent == current:
+            return None
+        current = parent
+
+
+def load_toml(file_path: Path, required: bool = False) -> dict:
+    if not file_path.exists():
+        if required:
+            sys.stderr.write(f"error: required customization file not found: {file_path}\n")
+            sys.exit(1)
+        return {}
+    try:
+        with file_path.open("rb") as f:
+            parsed = tomllib.load(f)
+        if not isinstance(parsed, dict):
+            if required:
+                sys.stderr.write(f"error: {file_path} did not parse to a table\n")
+                sys.exit(1)
+            return {}
+        return parsed
+    except tomllib.TOMLDecodeError as error:
+        level = "error" if required else "warning"
+        sys.stderr.write(f"{level}: failed to parse {file_path}: {error}\n")
+        if required:
+            sys.exit(1)
+        return {}
+    except OSError as error:
+        level = "error" if required else "warning"
+        sys.stderr.write(f"{level}: failed to read {file_path}: {error}\n")
+        if required:
+            sys.exit(1)
+        return {}
+
+
+def _detect_keyed_merge_field(items):
+    """Return 'code' or 'id' if every table item carries that *same* field.
+
+    All items must share the same identifier (all `code`, or all `id`).
+    Mixed arrays — where some items use `code` and others use `id` —
+    return None and fall through to append semantics. This is intentional:
+    mixing identifier keys within one array is a schema smell, and
+    append-fallback is safer than guessing which key should merge.
+    """
+    if not items or not all(isinstance(item, dict) for item in items):
+        return None
+    for candidate in _KEYED_MERGE_FIELDS:
+        if all(item.get(candidate) is not None for item in items):
+            return candidate
+    return None
+
+
+def _merge_by_key(base, override, key_name):
+    result = []
+    index_by_key = {}
+
+    for item in base:
+        if not isinstance(item, dict):
+            continue
+        if item.get(key_name) is not None:
+            index_by_key[item[key_name]] = len(result)
+        result.append(dict(item))
+
+    for item in override:
+        if not isinstance(item, dict):
+            result.append(item)
+            continue
+        key = item.get(key_name)
+        if key is not None and key in index_by_key:
+            result[index_by_key[key]] = dict(item)
+        else:
+            if key is not None:
+                index_by_key[key] = len(result)
+            result.append(dict(item))
+
+    return result
+
+
+def _merge_arrays(base, override):
+    """Shape-aware array merge. Base + override combined tables may opt into
+    keyed merge if every item has `code` or `id`. Otherwise: append."""
+    base_arr = base if isinstance(base, list) else []
+    override_arr = override if isinstance(override, list) else []
+    keyed_field = _detect_keyed_merge_field(base_arr + override_arr)
+    if keyed_field:
+        return _merge_by_key(base_arr, override_arr, keyed_field)
+    return base_arr + override_arr
+
+
+def deep_merge(base, override):
+    """Recursively merge override into base using structural rules.
+    - Table + table: deep merge
+    - Array + array: shape-aware (keyed merge if all items have code/id, else append)
+    - Anything else: override wins
+    """
+    if isinstance(base, dict) and isinstance(override, dict):
+        result = dict(base)
+        for key, over_val in override.items():
+            if key in result:
+                result[key] = deep_merge(result[key], over_val)
+            else:
+                result[key] = over_val
+        return result
+    if isinstance(base, list) and isinstance(override, list):
+        return _merge_arrays(base, override)
+    return override
+
+
+def extract_key(data, dotted_key: str):
+    parts = dotted_key.split(".")
+    current = data
+    for part in parts:
+        if isinstance(current, dict) and part in current:
+            current = current[part]
+        else:
+            return _MISSING
+    return current
+
+
+def main():
+    parser = argparse.ArgumentParser(
+        description="Resolve customization for a BMad skill using three-layer TOML merge.",
+        add_help=True,
+    )
+    parser.add_argument(
+        "--skill", "-s", required=True,
+        help="Absolute path to the skill directory (must contain customize.toml)",
+    )
+    parser.add_argument(
+        "--key", "-k", action="append", default=[],
+        help="Dotted field path to resolve (repeatable). Omit for full dump.",
+    )
+    args = parser.parse_args()
+
+    skill_dir = Path(args.skill).resolve()
+    skill_name = skill_dir.name
+    defaults_path = skill_dir / "customize.toml"
+
+    defaults = load_toml(defaults_path, required=True)
+
+    # Prefer the project that contains this skill. Only fall back to cwd if
+    # the skill isn't inside a recognizable project tree (unusual but possible
+    # for standalone skills invoked directly). Using cwd first is unsafe when
+    # an ancestor of cwd happens to have a stray _bmad/ from another project.
+    project_root = find_project_root(skill_dir) or find_project_root(Path.cwd())
+
+    team = {}
+    user = {}
+    if project_root:
+        custom_dir = project_root / "_bmad" / "custom"
+        team = load_toml(custom_dir / f"{skill_name}.toml")
+        user = load_toml(custom_dir / f"{skill_name}.user.toml")
+
+    merged = deep_merge(defaults, team)
+    merged = deep_merge(merged, user)
+
+    if args.key:
+        output = {}
+        for key in args.key:
+            value = extract_key(merged, key)
+            if value is not _MISSING:
+                output[key] = value
+    else:
+        output = merged
+
+    sys.stdout.write(json.dumps(output, indent=2, ensure_ascii=False) + "\n")
+
+
+if __name__ == "__main__":
+    main()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "siesa-agents",
-  "version": "2.1.91",
+  "version": "2.1.92",
   "description": "Paquete para instalar y configurar agentes SIESA en tu proyecto",
   "main": "index.js",
   "bin": {

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