dsh-github-flow 0.2.0 → 0.2.1

package/lib/api/routes-actions.js

@@ -1,4 +1,137 @@
+/**
+ * 动作类路由：缓存强制刷新 + 右侧栏快捷操作（create_pr/create_issue/open_browser/open_terminal）。
+ * 共同特征：POST 触发、读少写多、无缓存语义，与概览类路由的演进方向不同。
+ */
+import { spawn } from 'node:child_process';
 import { isSafeExternalUrl, sendJson } from './route-helpers.js';
+/**
+ * 构造「在新终端窗口运行命令」的平台命令（execFile 数组形态，不经 shell）。
+ *
+ * Issue #2 修复要点：
+ * - Windows：cmd.exe（显式 .exe，规避 GUI 父进程 PATH 解析差异）+ start 的窗口标题
+ *   占位 "DSH GitHub"——start 的首个带引号参数会被解析为标题而非程序，占位后
+ *   powershell.exe 才能被稳定识别为目标程序；
+ * - macOS：osascript 驱动 Terminal.app，命令经引号转义拼入 AppleScript；
+ * - Linux：发行版终端二进制与参数形态各异（gnome-terminal 用 `--`、konsole 用
+ *   `-e`、xfce4-terminal 用 `-x`），单一硬编码在主流桌面大概率不存在——由
+ *   detectLinuxTerminal 按探测结果返回对应形态。
+ */
+export function buildTerminalCommand(command) {
+    if (process.platform === 'win32') {
+        return {
+            bin: 'cmd.exe',
+            // -NoProfile 是 Windows 兼容基线：powershell.exe 是全量自带的 5.1，其默认执行
+            // 策略常为 Restricted——加载用户 profile（如 conda init 写入的）会就地报
+            // UnauthorizedAccess 红错（实测：winget 安装仍正常，但启动噪音吓人）。
+            // 白名单命令（gh/winget/brew/apt）均不依赖用户 profile 环境；pwsh 7 是个人
+            // 安装不作依赖，-NoProfile 在 5.1/7 语义一致，一并消除该类报错并加速启动。
+            args: ['/c', 'start', 'DSH GitHub', 'powershell.exe', '-NoExit', '-NoProfile', '-Command', command],
+        };
+    }
+    if (process.platform === 'darwin') {
+        // AppleScript 字符串内的双引号以 \" 转义；白名单命令虽无引号，仍统一转义防未来扩展踩坑
+        const escaped = command.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
+        return {
+            bin: 'osascript',
+            args: [
+                '-e', `tell application "Terminal" to do script "${escaped}"`,
+                '-e', 'tell application "Terminal" to activate',
+            ],
+        };
+    }
+    if (process.platform === 'linux') {
+        const terminal = detectLinuxTerminal();
+        if (!terminal)
+            return null;
+        // 各终端「执行命令并保持窗口」的参数形态不同：
+        // - gnome-terminal -- <sh> -c：新版 -e 已弃用，-- 后直接跟命令行
+        // - konsole/xfce4-terminal/xterm：-e/-x 执行，sh -c + exec 保交互语义
+        //   （命令结束后 exec sh 落回交互 shell，窗口不闪退）
+        const shArgs = ['sh', '-c', `${command}; exec sh`];
+        switch (terminal) {
+            case 'gnome-terminal': return { bin: 'gnome-terminal', args: ['--', ...shArgs] };
+            case 'konsole': return { bin: 'konsole', args: ['-e', ...shArgs] };
+            case 'xfce4-terminal': return { bin: 'xfce4-terminal', args: ['-x', ...shArgs] };
+            case 'x-terminal-emulator': return { bin: 'x-terminal-emulator', args: ['-e', ...shArgs] };
+            case 'xterm': return { bin: 'xterm', args: ['-e', ...shArgs] };
+            default: return null;
+        }
+    }
+    return null;
+}
+/** Linux 终端探测结果缓存（进程级）：which 探测有子进程成本，会话内只做一次 */
+let cachedLinuxTerminal;
+/**
+ * 探测当前 Linux 桌面可用的终端模拟器（按桌面占有率排序）。
+ * 返回 null 表示全部不可用——调用方据此走失败分支而非硬编码赌一个。
+ */
+function detectLinuxTerminal() {
+    if (cachedLinuxTerminal !== undefined)
+        return cachedLinuxTerminal;
+    const candidates = ['gnome-terminal', 'konsole', 'xfce4-terminal', 'x-terminal-emulator', 'xterm'];
+    try {
+        const { execFileSync } = require('node:child_process');
+        for (const name of candidates) {
+            try {
+                execFileSync('which', [name], { stdio: 'ignore', timeout: 2_000 });
+                cachedLinuxTerminal = name;
+                return name;
+            }
+            catch { /* 该候选不存在，试下一个 */ }
+        }
+    }
+    catch { /* which 本身不可用（极端环境），视为无可用终端 */ }
+    cachedLinuxTerminal = null;
+    return null;
+}
+/** 测试辅助：重置 Linux 终端探测缓存（便于单测注入不同探测环境） */
+export function resetLinuxTerminalCache() {
+    cachedLinuxTerminal = undefined;
+}
+/**
+ * 执行终端唤起并等待真实结果（Issue #2 核心修复：不再先斩后奏报假成功）。
+ * start/osascript 场景下命令随宿主立即退出，close 即代表唤起动作用的成败；
+ * err 与 stderr 原样上抛给调用方决定 HTTP 语义。
+ */
+function launchTerminalAndWait(command) {
+    return new Promise((resolve) => {
+        const built = buildTerminalCommand(command);
+        if (!built) {
+            resolve({ ok: false, error: '未找到可用的系统终端（Linux 下请先安装 gnome-terminal / konsole 等任一终端模拟器）' });
+            return;
+        }
+        // stdio 必须忽略：start 唤起的交互式窗口会继承 execFile 的 stdout/stderr 管道句柄，
+        // -NoExit 窗口存活期间管道永不关闭——回调要等满 10s 超时甚至不触发（实测复现：
+        // 唤起成功的窗口让 HTTP 响应挂起 10s+）。交互终端本就不该捕获输出；stdio ignore
+        // 后等待语义回归 cmd.exe 自身退出（start 创建窗口后立即返回，亚秒级）。
+        // 改用原生 spawn：execFile 的类型与实现都强制 stdout 管道捕获，而交互终端唤起
+        // 场景需要的恰恰是「不捕获」——spawn 的 stdio:'ignore' 是官方支持的干净形态。
+        const child = spawn(built.bin, built.args, {
+            stdio: 'ignore',
+            detached: false,
+        });
+        // 10s 兜底超时：正常路径 cmd.exe start 毫秒级返回；超时视为唤起故障并如实回传
+        let timedOut = false;
+        const killTimer = setTimeout(() => { timedOut = true; child.kill(); }, 10_000);
+        child.on('error', (err) => {
+            clearTimeout(killTimer);
+            resolve({ ok: false, error: err.message });
+        });
+        child.on('close', (code) => {
+            clearTimeout(killTimer);
+            if (code === 0) {
+                resolve({ ok: true });
+            }
+            else if (timedOut) {
+                // kill 触发的 close 是 code=null——如实归因为超时而非"退出码 null"
+                resolve({ ok: false, error: '终端启动命令 10 秒未返回（可能被安全软件拦截）' });
+            }
+            else {
+                resolve({ ok: false, error: `终端启动命令退出码 ${code}` });
+            }
+        });
+    });
+}
 export function registerActionRoutes(deps, executor, cacheMap, globalCache) {
     const { bindRoute } = deps;
     // 1. 强制刷新缓存
@@ -80,26 +213,17 @@ export function registerActionRoutes(deps, executor, cacheMap, globalCache) {
                             if (!allowedCommands.includes(command)) {
                                 throw new Error(`不支持唤起该指令: ${command}`);
                             }
-                            const { execFile } = await import('node:child_process');
-                            let bin = 'cmd';
-                            let args = [];
-                            if (process.platform === 'win32') {
-                                bin = 'cmd';
-                                args = ['/c', 'start', 'powershell', '-NoExit', '-Command', command];
-                            }
-                            else if (process.platform === 'darwin') {
-                                bin = 'osascript';
-                                args = ['-e', `tell application "Terminal" to do script "${command}"`, '-e', 'tell application "Terminal" to activate'];
-                            }
-                            else {
-                                bin = 'x-terminal-emulator';
-                                args = ['-e', command];
+                            // Issue #2 修复：等待真实唤起结果再回传——旧实现 execFile 异步回调吞错、
+                            // 路由先行返回 ok:true，导致终端弹不弹都提示成功。
+                            const launch = await launchTerminalAndWait(command);
+                            if (!launch.ok) {
+                                sendJson(res, 500, {
+                                    ok: false,
+                                    message: `唤起终端失败：${launch.error}。请手动打开系统终端执行：${command}`,
+                                });
+                                return;
                             }
-                            execFile(bin, args, (err) => {
-                                if (err)
-                                    console.warn('[dsh-github-flow] 唤起终端失败:', err?.message || err);
-                            });
-                            sendJson(res, 200, { ok: true, message: '已请求唤起系统终端' });
+                            sendJson(res, 200, { ok: true, message: '已唤起系统终端' });
                             break;
                         }
                         default:

package/lib/api/routes-lifecycle.d.ts

@@ -1,5 +1,5 @@
 /**
- * 生命周期类路由：版本检查 / 一键升级 / 升级状态轮询 / 认证轻量探针。
+ * 生命周期类路由：版本检查 / 一键升级 / 升级状态轮询 / 认证轻量探针 / 更新源设置。
  * 共同特征：服务插件自身状态而非 GitHub 业务数据，故障语义独立于概览链路。
  */
 import type { GhExecutor } from '../executor.js';

package/lib/api/routes-lifecycle.js

@@ -1,4 +1,5 @@
-import { sendJson } from './route-helpers.js';
+import { loadUpdateSettings, saveUpdateSettings, isValidUpdateMode, describeUpdateMode } from '../update-settings.js';
+import { sendJson, readJsonBody } from './route-helpers.js';
 export function registerLifecycleRoutes(deps, executor, checker, upgrader) {
     const { bindRoute } = deps;
     // 1. 闲时静默检查更新端点（前端在 gh 业务数据完全拉取就绪后，由 idle 延时静默触发）
@@ -31,8 +32,12 @@ export function registerLifecycleRoutes(deps, executor, checker, upgrader) {
                 return;
             }
             try {
+                // 一键升级是用户显式动作，不受 checkUpdate 配置熔断：check() 在禁用时
+                // 会返回 latestVersion 缺失的兜底态，此时回退 'latest' 让 pnpm 自行解析
+                // 最新版（与 UpgradeManager 自身的 targetVersion 可选语义一致）。
                 const info = await checker.check(false);
-                const result = await upgrader.executeUpgrade(info.latestVersion);
+                const target = info.latestVersion || 'latest';
+                const result = await upgrader.executeUpgrade(target);
                 sendJson(res, result.ok ? 200 : 500, result);
             }
             catch (err) {
@@ -52,6 +57,52 @@ export function registerLifecycleRoutes(deps, executor, checker, upgrader) {
             sendJson(res, 200, { ok: true, data: upgrader.getStatus() });
         },
     });
+    // 5. 更新源设置：GET 读取（store 权威，缺省 official）/ POST 保存并立即用新源验证
+    bindRoute({
+        kind: 'exact',
+        path: '/api/github/update-settings',
+        handler: async (req, res) => {
+            if (req.method === 'GET') {
+                const mode = loadUpdateSettings();
+                sendJson(res, 200, { ok: true, data: { mode, description: describeUpdateMode(mode) } });
+                return;
+            }
+            if (req.method !== 'POST') {
+                sendJson(res, 405, { ok: false, message: '仅支持 GET / POST 请求' });
+                return;
+            }
+            try {
+                const body = await readJsonBody(req);
+                const mode = body?.mode;
+                if (!isValidUpdateMode(mode)) {
+                    sendJson(res, 400, { ok: false, message: `非法更新源模式: ${String(mode)}（允许 official / mirror / off）` });
+                    return;
+                }
+                // 写盘失败不阻断内存态生效（save 内部已 warn），返回携带 error 提示重启后需重设
+                const saved = saveUpdateSettings(mode);
+                checker.applySettings(mode);
+                // 非 off 模式：立即用新源真探一次，把结果带回前端展示（缓存已被 applySettings 失效）
+                let probe = null;
+                if (mode !== 'off') {
+                    const info = await checker.check(true);
+                    probe = { latestVersion: info.latestVersion, hasUpdate: info.hasUpdate, checkedAt: info.checkedAt };
+                }
+                sendJson(res, 200, {
+                    ok: true,
+                    data: {
+                        mode,
+                        description: describeUpdateMode(mode),
+                        persisted: saved.ok,
+                        persistError: saved.error,
+                        probe,
+                    },
+                });
+            }
+            catch (err) {
+                sendJson(res, 500, { ok: false, message: err.message || '保存更新源设置失败' });
+            }
+        },
+    });
     // 4. 快速认证状态轻量探针 (耗时 < 500ms，专供登录自愈检测与快速恢复)
     bindRoute({
         kind: 'exact',