wkgeo-cli 0.2.2 → 0.2.4

package/lib/commands/billing.mjs

@@ -16,7 +16,7 @@
 import { withTrailingSlash } from '../config.mjs';
 import { UsageError, CliError, EXIT } from '../errors.mjs';
 import { formatMoney, formatTime } from '../format.mjs';
-import { openUrlForUser, browserPasteText, browserOpenedText, browserFailedHintLines } from '../login.mjs';
+import { openUrlForUser, browserPasteText, browserOpenedText, browserFailedHintLines, detectNonInteractive, nonInteractiveBrowserHintLines } from '../login.mjs';
 import { renderQrOptional } from '../qr.mjs';
 
 /**
@@ -266,13 +266,12 @@ export const walletRechargeCommand = {
     'amount-cents': { type: 'string', required: true, desc: '充值金额，单位**分**（最低 990 = ¥9.90；**不设上限**）' },
     'no-browser': { type: 'boolean', desc: '不自动打开浏览器（自动化/无桌面环境用；地址仍会打印并放在 pay_url 里）' },
     /**
-     * ★ 老板最终拍板走**传统路线**：默认**总是尝试打开**系统浏览器（人在终端与 AI 场景都一样），
-     *   所以本开关现在**与默认行为等价**。保留它只为两件事：
-     *     1. **显式语义**（脚本里写"我就是要弹"比依赖默认值更清楚）；
-     *     2. **兼容**既有脚本（不许删）。
-     * ⚠️ `--no-browser` 优先（关掉就是关掉，见 login.mjs 的 resolveBrowserIntent）。
+     * ★ 2026-10-21（老板反馈：CI 里弹了浏览器）：默认**只在人类终端**自动打开；
+     *   非交互/CI 环境（stdout 非 TTY / CI 环境变量 / `WKONGAI_CLI_NO_BROWSER=1`）**一律不打开**。
+     *   本开关保留只为显式语义与兼容（⚠️ 它压不过非交互判定）。
+     *   `--no-browser` 优先（关掉就是关掉，见 login.mjs 的 resolveBrowserIntent）。
      */
-    'open-browser': { type: 'boolean', desc: '强制尝试打开系统浏览器（⚠️ 现在与默认行为等价，保留只为显式语义与兼容）' },
+    'open-browser': { type: 'boolean', desc: '强制尝试打开系统浏览器（⚠️ 仅在人类终端生效；CI/非交互环境仍不打开）' },
     'dry-run': { type: 'boolean', desc: '只打印将要做什么，不真的创建订单' },
   },
   async run(ctx) {
@@ -349,7 +348,8 @@ export const walletRechargeCommand = {
     /**
      * 自动打开收银台。
      *
-     * ⚠️ 复用 `login.mjs` 的 `openUrlForUser`（多策略 + 结果归一化 + ★「默认总是尝试打开」闸门），
+     * ⚠️ 复用 `login.mjs` 的 `openUrlForUser`（多策略 + 结果归一化 + 唯一闸门：
+     *    人类终端默认打开，CI/非交互与 `--no-browser` 不打开），
      *    **不是**在这里再写一份 spawn —— 那几个坑（cmd start 的引号、URL 里的 `&`
      *    被 shell 吃掉、error 事件没挂）在那边已经踩过并修过，复制一份必然会再踩。
      * ⚠️ 打开失败**绝不允许**让命令失败：地址已经完整打印（人类可读）并放在
@@ -364,15 +364,20 @@ export const walletRechargeCommand = {
           spawnImpl: ctx.spawnImpl,
           platform: ctx.platform,
           verbose: !!(ctx.config && ctx.config.verbose),
-          /** ★ 与 login 同源的四个事实（见 login.mjs 的 resolveBrowserIntent） */
+          /** ★ 与 login 同源的多个事实（见 login.mjs 的 resolveBrowserIntent / detectNonInteractive） */
           isTTY: ctx.isTTY === true,
           json: out.json === true,
           forceOpen: flags.openBrowser === true,
           noBrowser: flags.noBrowser === true,
+          /** CI / 非交互判定（`CI` / `GITHUB_ACTIONS`… / `WKONGAI_CLI_NO_BROWSER=1`） */
+          env: ctx.env || {},
         },
         log: (m) => out.info(m),
       });
-      data.paste_text = browserPasteText(payUrl, { kind: 'pay' });
+      data.paste_text = browserPasteText(payUrl, {
+        kind: 'pay',
+        nonInteractive: data.open_browser.skipped_reason === 'non_interactive',
+      });
     }
 
     if (!out.json) {
@@ -386,6 +391,13 @@ export const walletRechargeCommand = {
         out.line(`  ${payUrl}`);
         if (flags.noBrowser === true) {
           out.info('  （已按 --no-browser 跳过自动打开）请把上面这一整行复制到浏览器地址栏打开。');
+        } else if (data.open_browser.skipped_reason === 'non_interactive') {
+          /**
+           * ★ 2026-10-21：非交互/CI 里**不弹**，给出可照做的下一步（收银台链接转给用户）。
+           *   绝不能走下面的"打开失败"提示 —— 我们**根本没试**，报失败会误导排障。
+           */
+          const reason = detectNonInteractive({ isTTY: ctx.isTTY === true, env: ctx.env || {} }).reason;
+          for (const line of nonInteractiveBrowserHintLines({ kind: 'pay', reason, cmdName: 'wkgeo', url: payUrl })) out.info(line);
         } else if (data.open_browser.opened) {
           // ★ 成功：明说"已经打开了"（老板最终拍板走传统路线：弹系统浏览器）
           out.line(browserOpenedText({ kind: 'pay' }));
@@ -450,6 +462,10 @@ export const walletRechargeCommand = {
        */
       if (flags.noBrowser === true) {
         out.notice('（已按 --no-browser 跳过自动打开）请手动复制 pay_url 到浏览器地址栏打开。');
+      } else if (data.open_browser.skipped_reason === 'non_interactive') {
+        // ★ 非交互/CI：没弹 ≠ 失败。把"链接交给用户"说清楚（stdout 仍然只有 JSON）。
+        const reason = detectNonInteractive({ isTTY: ctx.isTTY === true, env: ctx.env || {} }).reason;
+        for (const line of nonInteractiveBrowserHintLines({ kind: 'pay', reason, cmdName: 'wkgeo', url: payUrl })) out.notice(line);
       } else if (data.open_browser.opened) {
         out.notice(browserOpenedText({ kind: 'pay' }));
       } else {
@@ -584,11 +600,13 @@ export const walletRechargeLinkCommand = {
           spawnImpl: ctx.spawnImpl,
           platform: ctx.platform,
           verbose: !!(ctx.config && ctx.config.verbose),
-          /** 与 login / wallet recharge 同源的四个事实（见 resolveBrowserIntent） */
+          /** 与 login / wallet recharge 同源的多个事实（见 resolveBrowserIntent / detectNonInteractive） */
           isTTY: ctx.isTTY === true,
           json: out.json === true,
           forceOpen: true,
           noBrowser: flags.noBrowser === true,
+          /** ★ CI / 非交互判定：CI 里即使写了 --open-browser 也不弹（见 login.mjs） */
+          env: ctx.env || {},
         },
         log: (m) => out.info(m),
       });
@@ -607,6 +625,10 @@ export const walletRechargeLinkCommand = {
       out.info('  （不需要登录会员中心；本命令不会代替他付款，也不会模拟支付成功。）');
       if (flags.noBrowser === true) {
         out.info('  （已按 --no-browser 跳过自动打开）');
+      } else if (data.open_browser.skipped_reason === 'non_interactive') {
+        // ★ 非交互/CI：明确说"没弹"，并让用户把链接发给有浏览器的人（不是打开失败）
+        const reason = detectNonInteractive({ isTTY: ctx.isTTY === true, env: ctx.env || {} }).reason;
+        for (const line of nonInteractiveBrowserHintLines({ kind: 'pay', reason, cmdName: 'wkgeo', url: payUrl })) out.info(line);
       } else if (data.open_browser.attempted === false) {
         out.info('  （默认不自动打开浏览器；需要在这台机器上弹出，请加 --open-browser）');
       } else if (data.open_browser.opened) {
@@ -626,6 +648,9 @@ export const walletRechargeLinkCommand = {
         out.notice(
           '（默认不自动打开浏览器；需要在这台机器上弹出请加 --open-browser。把上面的链接发给用户即可。）',
         );
+      } else if (data.open_browser.skipped_reason === 'non_interactive') {
+        const reason = detectNonInteractive({ isTTY: ctx.isTTY === true, env: ctx.env || {} }).reason;
+        for (const line of nonInteractiveBrowserHintLines({ kind: 'pay', reason, cmdName: 'wkgeo', url: payUrl })) out.notice(line);
       } else if (data.open_browser.opened) {
         out.notice(browserOpenedText({ kind: 'pay' }));
       } else {

package/lib/device.mjs

@@ -52,7 +52,7 @@ import { writeCredentials, credentialsFromToken } from './credentials.mjs';
 import { withTrailingSlash } from './config.mjs';
 import { canonicalOauthUrl } from './oauth-urls.mjs';
 import { CliError, EXIT, loginHint } from './errors.mjs';
-import { defaultOpenBrowser, resolveBrowserIntent, browserOpenedText, browserFailedHintLines } from './login.mjs';
+import { defaultOpenBrowser, resolveBrowserIntent, browserOpenedText, browserFailedHintLines, detectNonInteractive, nonInteractiveBrowserHintLines } from './login.mjs';
 
 /** RFC 8628 规定的 urn */
 export const DEVICE_GRANT_TYPE = 'urn:ietf:params:oauth:grant-type:device_code';
@@ -266,15 +266,18 @@ export function announceDeviceCode({ config, log = () => {}, deps = {}, device,
   /**
    * ★ 走**同一个闸门**（`resolveBrowserIntent`，与 login 回调模式、钱包充值完全同一份逻辑）。
    *
-   * ★ 老板最终拍板走**传统路线**：**默认总是尝试打开**系统浏览器（设备码模式同样如此，
-   *   人在终端与 AI 场景都一样）；唯一的"不打开"开关是 `--no-browser`。
-   *   （历史：2026-10-01 曾改成"只有人在终端才弹"，那版已按老板最终决定回退。）
+   * ★ 2026-10-21（老板反馈：CI 里弹了浏览器）：**人在终端默认打开**；
+   *   非交互/CI（stdout 非 TTY / CI 环境变量 / `WKONGAI_CLI_NO_BROWSER=1`）与 `--no-browser`
+   *   都不打开。设备码模式尤其适合非交互环境（不需要任何本地回调），所以这里只要
+   *   把链接/短码交给用户即可 —— 绝不能因为"没弹浏览器"中断流程。
+   *   （历史：2026-10-01 曾改成"只有人在终端才弹"；2026-10-02 又默认总弹；本次是第三版。）
    */
   const browserIntent = resolveBrowserIntent({
     isTTY: deps.isTTY === true,
     json: deps.json === true,
     forceOpen: deps.forceBrowser === true,
     noBrowser: deps.openBrowser === false || deps.noBrowser === true,
+    env: deps.env && typeof deps.env === 'object' ? deps.env : {},
   });
   if (browserIntent.allow) {
     // 传入函数时用注入的（测试/自定义）；否则用**真·打开浏览器**
@@ -297,6 +300,23 @@ export function announceDeviceCode({ config, log = () => {}, deps = {}, device,
   } else if (browserIntent.skipped_reason === 'no_browser') {
     // 用户主动关掉的，不该报"失败"
     log('（已按 --no-browser 跳过自动打开）');
+  } else if (browserIntent.skipped_reason === 'non_interactive') {
+    /**
+     * ★ 非交互（CI / 管道 / AI 子进程）：**不弹**，但把"下一步"说清楚。
+     *   设备码模式在 CI 里本来就能用（只要用户在自己的浏览器里确认），
+     *   所以这里只提示"把地址/短码交给用户"，**不建议**再改模式。
+     */
+    const reason = detectNonInteractive({
+      isTTY: deps.isTTY === true,
+      env: deps.env && typeof deps.env === 'object' ? deps.env : {},
+    }).reason;
+    for (const line of nonInteractiveBrowserHintLines({
+      kind: 'auth',
+      reason,
+      cmdName: config.cmdName || 'wkgeo',
+    })) {
+      log(line);
+    }
   }
   return pasteText;
 }