@saulwade/swl-ses 2.2.3 → 2.2.4

package/scripts/publicar.js

@@ -16,7 +16,6 @@ const { execFileSync, spawnSync } = require('child_process');
 const fs   = require('fs');
 const path = require('path');
 const os   = require('os');
-const readline = require('readline');
 
 const {
   NOMBRE_GITHUB_MIRROR: GITHUB_NAME,
@@ -131,84 +130,35 @@ function esErrorOTP(stderr) {
 }
 
 /**
- * Solicita una OTP al usuario via stdin (readline síncrono).
- * Retorna la OTP normalizada (string de 6 dígitos) o null si:
- *   - stdin no es TTY (CI, pipe) → no se puede pedir interactivamente
- *   - el usuario cancela con Ctrl+C / línea vacía
- *   - el formato no es válido (no 6 dígitos)
- *
- * El caller debe manejar el caso null como "fallback a guía manual".
- */
-function solicitarOTPInteractiva() {
-  if (!process.stdin.isTTY) {
-    process.stderr.write('[publicar] stdin no es TTY: no se puede pedir OTP interactivamente.\n');
-    process.stderr.write('[publicar] Configurar NPM_CONFIG_OTP=<otp> antes de invocar este script.\n');
-    return null;
-  }
-
-  const rl = readline.createInterface({ input: process.stdin, output: process.stderr });
-  // questionSync vía promesa NO funciona aquí — el script es síncrono.
-  // Usamos un truco: readline es async-only, así que dejamos al usuario
-  // el manejo bloqueante leyendo línea por línea con readSync vía read-int.
-  // Pero readline no expone modo síncrono. Alternativa: leer de stdin con
-  // fd 0 + readSync. En Windows con TTY funciona; en Linux idem.
-  try {
-    process.stderr.write('\nNPM requiere OTP (2FA). Ingresa el código de 6 dígitos de tu autenticador: ');
-    const otp = leerLineaStdinSync().trim();
-    rl.close();
-    if (!/^\d{6}$/.test(otp)) {
-      process.stderr.write(`\n[publicar] OTP inválida: se esperan 6 dígitos, se recibió: "${otp.slice(0, 20)}"\n`);
-      return null;
-    }
-    return otp;
-  } catch (err) {
-    rl.close();
-    process.stderr.write(`\n[publicar] error leyendo OTP: ${String(err.message).slice(0, 120)}\n`);
-    return null;
-  }
-}
-
-/**
- * Lectura síncrona de una línea de stdin. Necesario porque readline solo
- * expone API asíncrona, pero todo el flujo de publicar.js es síncrono.
- * Lee byte por byte hasta encontrar newline o EOF.
+ * Ejecuta npm con stdio HEREDADO (interactivo): npm escribe directo a la
+ * terminal y lee de stdin, de modo que su flujo de 2FA NATIVO funciona —
+ * prompt de OTP TOTP O auth web/passkey ("abrir URL y seleccionar la llave"),
+ * según el método 2FA de la cuenta. Retorna el exit status sin lanzar.
+ * (A1: soporte passkey/WebAuthn, no solo TOTP.)
  */
-function leerLineaStdinSync() {
-  const BUFFER_SIZE = 256;
-  const buf = Buffer.alloc(BUFFER_SIZE);
-  let line = '';
-  while (true) {
-    let bytesRead;
-    try {
-      bytesRead = fs.readSync(0, buf, 0, BUFFER_SIZE, null);
-    } catch (err) {
-      // EAGAIN en stdin no-bloqueante: poco común en TTY, pero defensivo.
-      if (err.code === 'EAGAIN') continue;
-      throw err;
-    }
-    if (bytesRead === 0) break; // EOF
-    const chunk = buf.slice(0, bytesRead).toString('utf-8');
-    const nlIdx = chunk.indexOf('\n');
-    if (nlIdx >= 0) {
-      line += chunk.slice(0, nlIdx);
-      break;
-    }
-    line += chunk;
+function npmExecInherit(args, opts = {}) {
+  const merged = { cwd: ROOT, timeout: 600_000, env: process.env, ...opts, stdio: 'inherit' };
+  let res;
+  if (NPM_CLI_JS) {
+    res = spawnSync(process.execPath, [NPM_CLI_JS, ...args], merged);
+  } else {
+    const bin = process.platform === 'win32' ? 'npm.cmd' : 'npm';
+    res = spawnSync(bin, args, merged);
   }
-  return line.replace(/\r$/, ''); // Windows envía \r\n
+  return res.status;
 }
 
 /**
  * Ejecuta un `npm publish` con soporte automático de OTP (2FA).
  *
  * Flujo:
- *   1. Si NPM_CONFIG_OTP está definida en el entorno, se usa directamente
- *      (npm la lee automáticamente — no hay que pasarla como flag).
- *   2. Si el publish falla con EOTP/one-time password, se intenta solicitar
- *      la OTP interactivamente y reintentar con --ignore-scripts (los
- *      scripts pre-publish ya pasaron en el primer intento).
- *   3. Si no se puede obtener OTP (no TTY o usuario cancela), reporta el
- *      error y retorna false.
+ *   1. Si NPM_CONFIG_OTP está definida en el entorno (o hay token Automation),
+ *      se usa directamente y el intento 1 publica sin interacción.
+ *   2. Si el publish falla con EOTP/one-time password y stdin es TTY, se
+ *      reintenta con stdio HEREDADO para que npm conduzca su 2FA NATIVO —
+ *      OTP TOTP O passkey/WebAuthn (navegador), según la cuenta — con
+ *      --ignore-scripts (los pre-publish ya pasaron en el intento 1).
+ *   3. Si stdin no es TTY (CI sin OTP), reporta el error y retorna false.
  *
  * Retorna: { ok: boolean, stderr: string }
  *
@@ -226,24 +176,23 @@ function ejecutarPublishConOTP(args, opts = {}) {
     return { ok: false, stderr: intento1.stderr };
   }
 
-  // Es EOTP. Intentar solicitar OTP interactivamente.
+  // Es EOTP — 2FA requerida. npm soporta varios métodos: OTP TOTP (6 dígitos)
+  // O passkey/WebAuthn vía navegador ("abrir URL y seleccionar la llave").
+  // Forzar un prompt de TOTP rompe a los usuarios con passkey (A1). En su lugar
+  // se reintenta con stdio HEREDADO para que npm conduzca su 2FA NATIVO —
+  // funcione cual sea el método configurado en la cuenta.
   process.stderr.write('\n[publicar] npm rechazó el publish con EOTP (2FA requerida).\n');
-  const otp = solicitarOTPInteractiva();
-  if (!otp) {
-    process.stderr.write('[publicar] No se obtuvo OTP. Aborta. Reintentar con:\n');
-    process.stderr.write(`  NPM_CONFIG_OTP=<otp> node scripts/publicar.js ${process.argv.slice(2).join(' ')}\n`);
+  if (!process.stdin.isTTY) {
+    process.stderr.write('[publicar] stdin no es TTY: no se puede completar 2FA interactivo.\n');
+    process.stderr.write('[publicar] Opciones: exportar NPM_CONFIG_OTP=<otp>, o usar un token Automation (omite 2FA).\n');
     return { ok: false, stderr: intento1.stderr };
   }
-
-  // Reintento con OTP via env + --ignore-scripts (los pre-publish ya pasaron).
-  process.stderr.write('\n[publicar] Reintentando publish con OTP recibida...\n');
-  const envConOTP = { ...(opts.env || process.env), NPM_CONFIG_OTP: otp };
+  // Reintento interactivo: --ignore-scripts porque los pre-publish ya pasaron en
+  // el intento 1 (el prepack limpió el árbol; no se re-ejecuta test:release).
+  process.stderr.write('\n[publicar] Reintentando con el 2FA interactivo nativo de npm (OTP o passkey)...\n');
   const argsConIgnore = args.includes('--ignore-scripts') ? args : [...args, '--ignore-scripts'];
-  const intento2 = npmSpawnCaptureStderr(argsConIgnore, { ...opts, env: envConOTP });
-  if (intento2.status === 0) {
-    return { ok: true, stderr: intento2.stderr };
-  }
-  return { ok: false, stderr: intento2.stderr };
+  const status = npmExecInherit(argsConIgnore, opts);
+  return { ok: status === 0, stderr: status === 0 ? '' : 'fallo en reintento interactivo de 2FA' };
 }
 
 function leerPkg() {