jorgex-stack 1.1.1 → 1.1.2

package/dist/cli.js

@@ -1657,6 +1657,22 @@ var PLAYWRIGHT_CLI = {
   version: "0.1.17",
   browserInstallAction: "install-browser"
 };
+var PNPM_GLOBAL_BIN_REMEDY = "Ejecuta 'pnpm setup', abre una terminal nueva y reintenta. El stack no modifica PNPM_HOME ni PATH.";
+function resolvePnpmFailureRemedy(reason) {
+  switch (reason) {
+    case "pnpm-unavailable":
+      return "Instala pnpm o a\xF1\xE1delo a PATH antes de reintentar.";
+    case "pnpm-command":
+      return "No se pudo ejecutar pnpm. Revisa su instalaci\xF3n, PATH y permisos antes de reintentar.";
+    case "pnpm-global-bin":
+      return PNPM_GLOBAL_BIN_REMEDY;
+    case "action-failed":
+      return null;
+  }
+}
+function hasNonzeroProcessStatus(error) {
+  return typeof error === "object" && error !== null && "status" in error && typeof error.status === "number" && error.status !== 0;
+}
 function parsePlaywrightCliVersion(output) {
   if (output === null) return null;
   return /^(?:playwright-cli\s+)?(\d+\.\d+\.\d+)$/i.exec(output.trim())?.[1] ?? null;
@@ -1711,15 +1727,24 @@ function planPlaywrightCliCommand(action, pnpmBin) {
   }
 }
 function executePlaywrightToolAction(action, pnpmBin = resolvePnpmBin()) {
-  if (pnpmBin === null) return false;
+  if (pnpmBin === null) return { ok: false, reason: "pnpm-unavailable" };
+  if (action !== "install-browser") {
+    const preflight = planDetectedBinCommand(pnpmBin, ["bin", "--global"]);
+    if (preflight === null) return { ok: false, reason: "pnpm-command" };
+    try {
+      execFileSync2(preflight.command, preflight.args, { stdio: "inherit" });
+    } catch (error) {
+      return { ok: false, reason: hasNonzeroProcessStatus(error) ? "pnpm-global-bin" : "pnpm-command" };
+    }
+  }
   const command = planPlaywrightCliCommand(action, pnpmBin);
   const invocation = planDetectedBinCommand(command.command, command.args);
-  if (invocation === null) return false;
+  if (invocation === null) return { ok: false, reason: "pnpm-command" };
   try {
     execFileSync2(invocation.command, invocation.args, { stdio: "inherit" });
-    return true;
+    return { ok: true };
   } catch {
-    return false;
+    return { ok: false, reason: "action-failed" };
   }
 }
 
@@ -1863,7 +1888,9 @@ function resolvePlaywrightToolPlan(consent) {
 async function runPlaywrightToolPlan(plan, deps) {
   for (const action of plan.actions) {
     try {
-      if (!await deps.run(action)) return { ok: false, failedAction: action };
+      const result = await deps.run(action);
+      if (result === false) return { ok: false, failedAction: action };
+      if (result !== true && !result.ok) return { ok: false, failedAction: action, reason: result.reason };
     } catch {
       return { ok: false, failedAction: action };
     }
@@ -2117,7 +2144,8 @@ async function runInstall(opts) {
         persistEnabled: (enabled) => savePlaywrightCliPreference(playwrightCliPreferenceFile(), enabled)
       });
       if (!result.ok) {
-        const reason = result.failedAction === "install" ? "no se pudo instalar el paquete global" : result.failedAction === "install-browser" ? "no se pudo descargar el navegador" : "se instalaron los componentes, pero no se pudo guardar la preferencia";
+        const pnpmRemedy = result.reason === void 0 ? null : resolvePnpmFailureRemedy(result.reason);
+        const reason = result.reason === "pnpm-global-bin" ? `la configuraci\xF3n global de pnpm no est\xE1 lista. ${pnpmRemedy}` : pnpmRemedy ?? (result.failedAction === "install" ? "no se pudo instalar el paquete global" : result.failedAction === "install-browser" ? "no se pudo descargar el navegador" : "se instalaron los componentes, pero no se pudo guardar la preferencia");
         p.log.error(`Playwright CLI: ${reason}; la preferencia no se ha marcado como habilitada. Ejecuta 'jorgex-stack install --playwright' para reintentar.`);
         exitCode = 1;
       } else {
@@ -2291,17 +2319,22 @@ async function runUninstall(opts) {
   } else if (playwrightPlan.actions.length > 0) {
     if (opts.dryRun) {
       p2.log.info("Playwright CLI: se retirar\xEDa solo el paquete global; los datos y navegadores se conservan.");
-    } else if (executePlaywrightToolAction2("remove")) {
-      try {
-        savePlaywrightCliPreference(playwrightCliPreferenceFile(), false);
-        p2.log.success("Playwright CLI: paquete global retirado; los datos y navegadores se conservan.");
-      } catch (error) {
-        p2.log.error(`Playwright CLI: paquete global retirado, pero no se pudo guardar la preferencia (${error instanceof Error ? error.message : String(error)}). Corrige la preferencia antes de reintentar.`);
+    } else {
+      const removal = executePlaywrightToolAction2("remove");
+      if (removal.ok) {
+        try {
+          savePlaywrightCliPreference(playwrightCliPreferenceFile(), false);
+          p2.log.success("Playwright CLI: paquete global retirado; los datos y navegadores se conservan.");
+        } catch (error) {
+          p2.log.error(`Playwright CLI: paquete global retirado, pero no se pudo guardar la preferencia (${error instanceof Error ? error.message : String(error)}). Corrige la preferencia antes de reintentar.`);
+          exitCode = 1;
+        }
+      } else {
+        const pnpmRemedy = resolvePnpmFailureRemedy(removal.reason);
+        const recovery = pnpmRemedy === null ? " Revisa el error de pnpm anterior y ejecuta 'jorgex-stack uninstall --remove-playwright' para reintentar." : ` ${pnpmRemedy} Despu\xE9s, ejecuta 'jorgex-stack uninstall --remove-playwright' para reintentar.`;
+        p2.log.error(`Playwright CLI: no se pudo retirar el paquete global; los datos y la preferencia se conservan.${recovery}`);
         exitCode = 1;
       }
-    } else {
-      p2.log.error("Playwright CLI: no se pudo retirar el paquete global; los datos y la preferencia se conservan.");
-      exitCode = 1;
     }
   } else {
     p2.log.info("Playwright CLI: paquete global y datos del navegador conservados (usa --remove-playwright para retirar solo el paquete).");
@@ -3404,15 +3437,18 @@ ${diff}`);
     if (p4.isCancel(confirmPlaywright) || !confirmPlaywright) {
       p4.log.info("Playwright CLI: actualizaci\xF3n omitida.");
     } else {
-      const packageUpdated = executePlaywrightToolAction2("update");
-      const browserInstalled = packageUpdated && executePlaywrightToolAction2("install-browser");
-      if (packageUpdated && browserInstalled) {
+      const packageResult = executePlaywrightToolAction2("update");
+      const browserResult = packageResult.ok ? executePlaywrightToolAction2("install-browser") : null;
+      if (packageResult.ok && browserResult?.ok) {
         p4.log.success("Playwright CLI actualizado al pin aprobado.");
         appliedUpdates = true;
         updated.push("playwright-cli");
       } else {
-        const failedStep = packageUpdated ? "descargar el navegador" : "actualizar el paquete global";
-        p4.log.error(`Playwright CLI: no se pudo ${failedStep}. Ejecuta 'jorgex-stack install --playwright' para reintentar el paquete y el navegador.`);
+        const failedResult = packageResult.ok ? browserResult : packageResult;
+        const pnpmRemedy = failedResult && !failedResult.ok ? resolvePnpmFailureRemedy(failedResult.reason) : null;
+        const recovery = pnpmRemedy === null ? "Ejecuta 'jorgex-stack install --playwright' para reintentar el paquete y el navegador." : `${pnpmRemedy} Despu\xE9s, ejecuta 'jorgex-stack update' para reintentar.`;
+        const failedStep = packageResult.ok ? "descargar el navegador" : "actualizar el paquete global";
+        p4.log.error(`Playwright CLI: no se pudo ${failedStep}. ${recovery}`);
         exitCode = 1;
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jorgex-stack",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Harness multi-agente portable: instala la config JorgeX (agentes, skills, hooks, Engram, MCPs) en Claude Code, Codex CLI y OpenCode",
   "type": "module",
   "license": "MIT",