@lark-apaas/miaoda-cli 0.1.33-alpha.59a1bf2 → 0.1.33-alpha.dc7b6a0

package/dist/cli/handlers/app/migrate.js

@@ -11,6 +11,7 @@ const node_path_1 = __importDefault(require("node:path"));
 const index_1 = require("../../../config/migrate-configs/index");
 const index_2 = require("../../../services/app/init/index");
 const migrate_rule_1 = require("../../../utils/migrate-rule");
+const file_ops_1 = require("../../../utils/file-ops");
 const spark_meta_1 = require("../../../utils/spark-meta");
 const error_1 = require("../../../utils/error");
 const output_1 = require("../../../utils/output");
@@ -103,24 +104,31 @@ async function handleAppMigrate(opts) {
     // 沙箱环境下先停掉 dev server（预览）—— 保活由 code-server 管，`pkill dev.js` 会被立刻
     // 重新拉起，拿不到"停住"的状态。dev server 活着时 vite 持续往 node_modules/.vite/deps 写
     // 文件，下面 rm node_modules 走到顶层 rmdir 时目录又非空，直接 ENOTEMPTY 失败（历史故障）。
+    // stop 返回成功即代表 dev 进程已确认退出（服务端等到 exit 事件才返回），不需要再 pkill。
     //   - 只在沙箱内做（本地用户自己管 dev 进程，不该被 CLI 动）
     //   - 软失败：接口不存在（老版本 code-server 返 404）/ 连不上都不阻断，降级回老路径
     //     （带重试的 rm + 结尾 pkill 让保活拉起）
-    //   - ⚠️ stop 成功后必须恢复：下面整段包在 try/finally 里，任何异常路径都会走 startPreview
+    //   - ⚠️ 恢复条件用 mayBeSuspended 而不是 ok：服务端在抛错前就置了暂停位，stop 失败
+    //     （500 / 超时）时暂停位可能已生效，不恢复就等于永久关掉保活。只有 404 能确定无副作用。
     let previewStopped = false;
+    let previewMayBeSuspended = false;
     let previewStopError;
     if ((0, env_1.isSandboxEnv)()) {
         (0, logger_1.log)('migrate', '停止 dev server(预览)以便安全重装依赖...');
         const stopped = await (0, preview_control_1.stopPreview)({ logPrefix: 'migrate' });
         previewStopped = stopped.ok;
+        previewMayBeSuspended = stopped.mayBeSuspended;
         if (!stopped.ok) {
             previewStopError = stopped.error;
-            (0, logger_1.log)('migrate', `⚠ 停止预览失败 (continuing, 降级为不停 dev 直接重装): ${stopped.error ?? ''}`);
+            const tail = previewMayBeSuspended ? '结束时仍会尝试恢复' : '降级为不停 dev 直接重装';
+            (0, logger_1.log)('migrate', `⚠ 停止预览失败 (continuing, ${tail}): ${stopped.error ?? ''}`);
         }
     }
     const followLatest = config.followLatestPackages ?? [];
     let installError;
     let nodeModulesCleanError;
+    let nodeModulesPurgeStrategy;
+    let nodeModulesTrashLeftover;
     let previewRestarted = false;
     let previewRestartError;
     try {
@@ -128,24 +136,24 @@ async function handleAppMigrate(opts) {
         // 全套）+ rule 已经把 package-lock.json 删了。如果 node_modules 留着，npm install 时旧
         // 包还在 node_modules 顶层 hoist 树里，可能跟新 package.json spec 撞 peer 冲突
         // （比如旧 coding-preset-vite-react@1.0.8 peer vite@^8 跟新 vite@^7 撞）。
-        //   - maxRetries：force 只吞 ENOENT，不管 ENOTEMPTY/EBUSY；这类错误 node 只在
-        //     maxRetries>0 时才重试。停预览失败降级时靠它兜住残余写入。
-        //   - 软失败：删不干净也继续跑 npm install 让 npm 自己 reconcile —— 半迁移态
+        //   - 用 purgeDirectory（rename 主档）而不是 fs.rmSync：dev server 没停成功时（老
+        //     code-server 没有 preview/stop 路由）vite 还在往 node_modules/.vite/deps 写，rm 走到
+        //     顶层 rmdir 必然 ENOTEMPTY，而 `maxRetries` 不 re-walk、救不了（沙箱实测数据见
+        //     purgeDirectory 注释）。同分区 rename 是原子的，一次就把旧树摘掉。
+        //   - 软失败：摘不掉也继续跑 npm install 让 npm 自己 reconcile —— 半迁移态
         //     （package-lock 已删 + 新 deps 没装）比"清理不彻底"糟得多。
         const nodeModulesPath = node_path_1.default.join(targetDir, 'node_modules');
         if (node_fs_1.default.existsSync(nodeModulesPath)) {
             (0, logger_1.log)('migrate', '清理 node_modules（dependencies 集合已变）...');
-            try {
-                node_fs_1.default.rmSync(nodeModulesPath, {
-                    recursive: true,
-                    force: true,
-                    maxRetries: 5,
-                    retryDelay: 300,
-                });
+            const purged = (0, file_ops_1.purgeDirectory)(nodeModulesPath);
+            nodeModulesPurgeStrategy = purged.strategy;
+            nodeModulesTrashLeftover = purged.trashLeftover;
+            if (!purged.purged) {
+                nodeModulesCleanError = purged.error;
+                (0, logger_1.log)('migrate', `⚠ 清理 node_modules 失败 (continuing): ${nodeModulesCleanError ?? ''}`);
             }
-            catch (err) {
-                nodeModulesCleanError = err instanceof Error ? err.message : String(err);
-                (0, logger_1.log)('migrate', `⚠ 清理 node_modules 失败 (continuing): ${nodeModulesCleanError}`);
+            else if (purged.trashLeftover !== undefined) {
+                (0, logger_1.log)('migrate', `⚠ node_modules 已移走但残留目录未删净: ${purged.trashLeftover}（不影响 install，需手动清）`);
             }
         }
         // 跑 npm install 物化新依赖集合 + 把 followLatestPackages 钉 latest
@@ -200,12 +208,14 @@ async function handleAppMigrate(opts) {
         }
     }
     finally {
-        // 恢复 dev server —— 只要上面 stop 成功过就必须调，且必须在 finally 里：
+        // 恢复 dev server —— 只要暂停位可能已生效就必须调，且必须在 finally 里：
         // rule 应用 / install / 清 cache 任何一步抛异常都不能把保活留在关闭状态，否则用户沙箱
         // 的预览再也起不来（比迁移失败严重得多）。
         // 无条件恢复（不看 installError）：dev 进程已经被停了，留着不起来只会更糟；新 scripts/dev.js
         // + 新依赖也正是靠这次 start 生效。
-        if (previewStopped) {
+        // 用 previewMayBeSuspended 而非 previewStopped：stop 超时/500 时服务端暂停位已经置上了。
+        // 多余的 start 无害（服务端未暂停且进程在跑时 no-op）。
+        if (previewMayBeSuspended) {
             (0, logger_1.log)('migrate', '恢复 dev server(预览)...');
             const started = await (0, preview_control_1.startPreview)({ logPrefix: 'migrate' });
             previewRestarted = started.ok;
@@ -215,15 +225,17 @@ async function handleAppMigrate(opts) {
             }
         }
     }
-    // 降级路径：preview stop 没成功（老版本 code-server / 接口连不上），dev 进程还是老的，
-    // 按历史行为 pkill 掉 dev.js 主进程，靠平台 supervisor 自动重新 exec dev.sh → dev.js，
+    // 降级路径：preview 接口不存在（老版本 code-server 返 404，确定没进暂停态），dev 进程还是
+    // 老的，按历史行为 pkill 掉 dev.js 主进程，靠平台 supervisor 自动重新 exec dev.sh → dev.js，
     // 让新 deps + 新 scripts 生效。
     //   - 只在沙箱内做(isSandboxEnv;本地环境用户进程混杂,主动 pkill 风险大)
+    //   - previewMayBeSuspended 时不做 —— 新 code-server 上暂停位管着重建，pkill 无意义；
+    //     进程该由上面的 startPreview 拉起
     //   - install 挂了不做 —— 新依赖不全, 杀旧 dev 后新 dev 起不来反而更糟,
     //     留着旧进程让用户先处理 installError
     //   - 软失败:pkill 无匹配进程退出 1,catch 吞掉
     let devRestarted = previewRestarted;
-    if (!previewStopped && installError === undefined && (0, env_1.isSandboxEnv)()) {
+    if (!previewMayBeSuspended && installError === undefined && (0, env_1.isSandboxEnv)()) {
         (0, logger_1.log)('migrate', '沙箱环境,重启 dev process(平台 supervisor 会自动拉起)...');
         try {
             (0, node_child_process_1.execFileSync)('pkill', ['-f', 'node.*scripts/dev\\.js'], { stdio: 'ignore' });
@@ -248,27 +260,34 @@ async function handleAppMigrate(opts) {
             followLatestPackages: followLatest,
             installError,
             nodeModulesCleanError,
+            nodeModulesPurgeStrategy,
+            nodeModulesTrashLeftover,
             devRestarted,
             previewStopped,
             previewStopError,
+            previewMayBeSuspended,
             previewRestarted,
             previewRestartError,
-            nextActions: (0, env_1.isSandboxEnv)()
-                ? [
-                    'git status / git diff 评估改动并 commit',
-                    'miaoda skills sync 同步到新 stack 的 agent skills',
-                    // 预览恢复失败时置顶一条人工兜底动作 —— 保活已被关掉且不会自愈
-                    ...(previewRestartError === undefined
-                        ? []
-                        : [
-                            `⚠ dev server(预览)未恢复成功，需手动 curl -X POST ${(0, preview_control_1.resolveCodeServerBaseUrl)()}/api/v1/preview/start 恢复保活`,
-                        ]),
-                ]
-                : [
-                    'git status / git diff 评估改动并 commit',
-                    '重启 dev 进程让新 scripts/dev.js + 新依赖生效:重跑 npm run dev(killOrphansByPort 会自动踢掉旧进程)',
-                    'miaoda skills sync 同步到新 stack 的 agent skills',
-                ],
+            nextActions: [
+                'git status / git diff 评估改动并 commit',
+                ...((0, env_1.isSandboxEnv)()
+                    ? []
+                    : [
+                        '重启 dev 进程让新 scripts/dev.js + 新依赖生效:重跑 npm run dev(killOrphansByPort 会自动踢掉旧进程)',
+                    ]),
+                'miaoda skills sync 同步到新 stack 的 agent skills',
+                // 预览恢复失败时补一条人工兜底动作 —— 暂停位还挂着，预览不会自己起来。
+                // 兜底自愈：暂停位只存 code-server 内存，重启 code-server 也能恢复。
+                ...(previewRestartError === undefined
+                    ? []
+                    : [
+                        `⚠ dev server(预览)未恢复成功，需手动 curl -X POST ${(0, preview_control_1.resolveCodeServerBaseUrl)()}/api/v1/preview/start 恢复保活（暂停状态只存内存，重启 code-server 亦可自愈）`,
+                    ]),
+                // node_modules 已 rename 摘掉但 trash 没删净：不影响 install，但会出现在 git status 里
+                ...(nodeModulesTrashLeftover === undefined
+                    ? []
+                    : [`⚠ 残留目录未清: rm -rf ${nodeModulesTrashLeftover}`]),
+            ],
         },
     });
 }

package/dist/utils/file-ops.js

@@ -8,6 +8,7 @@ exports.removeLineFromContent = removeLineFromContent;
 exports.unlockDirForWrite = unlockDirForWrite;
 exports.lockDirReadonly = lockDirReadonly;
 exports.forceRemove = forceRemove;
+exports.purgeDirectory = purgeDirectory;
 const node_child_process_1 = require("node:child_process");
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
@@ -124,3 +125,85 @@ function forceRemove(target) {
     }
     node_fs_1.default.rmSync(target, { recursive: true, force: true });
 }
+/** {@link purgeDirectory} 移走目录时用的临时名前缀（相对父目录）。 */
+const PURGE_TRASH_PREFIX = '.miaoda-purge-';
+/**
+ * 删除一个**可能正在被其它进程写入**的目录（典型场景：dev server 活着时清 node_modules）。
+ *
+ * 主档是 `rename` 而不是 `rm`：同分区 rename 是原子的，不受目录内容变化影响，一次就把整棵
+ * 旧树从原路径摘掉；随后再删被移走的 trash（此时原路径已经腾空，删失败也不阻断调用方）。
+ *
+ * ⚠️ 不要改回 `fs.rmSync(..., { maxRetries })`。在沙箱内实测（Node v22）过四种组合：
+ *
+ * ```text
+ *   持续写入 + maxRetries:0            → ENOTEMPTY（241ms）
+ *   持续写入 + maxRetries:5/300ms      → ENOTEMPTY（4836ms，白等 5 轮）
+ *   0.5s 短写入 + maxRetries:5/300ms   → ENOTEMPTY（4747ms）   ← 关键反例
+ *   0.5s 短写入 + 外层重试整个 rmSync  → 成功（717ms）
+ *   持续写入 + shell rm -rf            → 失败
+ *   持续写入 + rename                  → 成功（122ms）
+ * ```
+ *
+ * 原因：`maxRetries` 只重试失败的那一次 `rmdir`，**不会重新 walk 目录**。删除过程中新写进来的
+ * 文件永远不会被 unlink，于是 rmdir 一直 ENOTEMPTY，重试纯属白等。只有重新走一遍（外层重试）
+ * 或干脆 rename 摘掉才有用。
+ *
+ * rename 失败（跨设备 EXDEV、父目录不可写 EPERM 等）时退回“外层重试整个 rmSync”。
+ *
+ * 注意：rename 之后原路径可能被仍在运行的写入方立刻重建（vite 会 `mkdir -p` 回
+ * `node_modules/.vite/deps`）。这不算失败 —— 旧依赖树已经不在原路径了，后续 `npm install`
+ * 能在干净状态上物化新依赖集合。
+ */
+function purgeDirectory(target, opts = {}) {
+    if (!node_fs_1.default.existsSync(target))
+        return { purged: true, strategy: 'absent' };
+    const parent = node_path_1.default.dirname(target);
+    const trash = node_path_1.default.join(parent, `${PURGE_TRASH_PREFIX}${node_path_1.default.basename(target)}-${String(process.pid)}`);
+    try {
+        node_fs_1.default.renameSync(target, trash);
+    }
+    catch (renameErr) {
+        (0, logger_1.debug)(`file-ops: rename ${target} failed: ${renameErr.message}`);
+        return rmSyncWithOuterRetry(target, opts);
+    }
+    try {
+        node_fs_1.default.rmSync(trash, { recursive: true, force: true });
+    }
+    catch (rmErr) {
+        (0, logger_1.debug)(`file-ops: rm trash ${trash} failed: ${rmErr.message}`);
+        return { purged: true, strategy: 'rename', trashLeftover: trash };
+    }
+    return { purged: true, strategy: 'rename' };
+}
+/**
+ * 重试整个 `fs.rmSync`（每次重新 walk 目录），而不是依赖 `maxRetries`（它不 re-walk，见
+ * {@link purgeDirectory} 的实测数据）。
+ */
+function rmSyncWithOuterRetry(target, opts) {
+    const attempts = opts.attempts ?? 4;
+    const delayMs = opts.retryDelayMs ?? 300;
+    let lastErr;
+    for (let i = 1; i <= attempts; i++) {
+        try {
+            node_fs_1.default.rmSync(target, { recursive: true, force: true });
+            return { purged: true, strategy: 'rmSync' };
+        }
+        catch (err) {
+            lastErr = err;
+            if (i < attempts)
+                sleepSync(delayMs);
+        }
+    }
+    return {
+        purged: false,
+        strategy: 'rmSync',
+        error: lastErr instanceof Error ? lastErr.message : String(lastErr),
+    };
+}
+/** 同步等待。这条链路整体是同步的（migrate 的清理阶段），不引入 async 传染。 */
+function sleepSync(ms) {
+    const end = Date.now() + ms;
+    while (Date.now() < end) {
+        // busy wait —— 只在 rename 失败的兜底路径上跑，最多 3 次 300ms
+    }
+}

package/dist/utils/preview-control.js

@@ -20,15 +20,31 @@ const logger_1 = require("../utils/logger");
  * vite 会持续往 `node_modules/.vite/deps` 写文件，`fs.rmSync` 自底向上删到顶层 `rmdir` 时目录
  * 又变非空，直接 `ENOTEMPTY` 失败。先 stop 再删才是干净的做法。
  *
- * ⚠️ 调用契约：**stop 成功后必须保证 start 被调到**（调用方放 `finally`）。保活被关掉后不会
- * 自愈，漏掉 start 会让用户沙箱的预览再也起不来 —— 比迁移失败严重得多。
+ * 服务端语义（ide-vscode `src/vs/platform/force/node/preview/previewService.ts`）：
+ *   - stop 先置暂停位（`_suspended`），再等在途创建落地、shutdown 并等进程真正退出。
+ *     **返回成功即代表 dev 进程已死，调用方不需要自己 pkill。**
+ *   - 等退出有 10s 超时，超时按失败处理（HTTP 500）。所以本地超时必须 > 10s，否则会抢在
+ *     服务端给出结论前 abort（见 STOP_TIMEOUT_MS）。
+ *   - ⚠️ 暂停位在 `_doStop()` 第一行就置上，**在抛错之前**。所以 stop 失败 ≠ 没进暂停态：
+ *     除了 404（路由不存在，必然无副作用）以外的任何失败，都必须当成"可能已暂停"去恢复，
+ *     否则保活被永久关掉。见 {@link PreviewControlResult.mayBeSuspended}。
+ *   - 暂停位只存内存，不持久化：code-server 重启会丢状态并重新拉起预览（兜底自愈路径）。
+ *   - start 在已运行时是 no-op；设计态沙箱（miaoda_design）本就不启动预览，start 也是 no-op。
+ *
+ * ⚠️ 调用契约：**只要 stop 可能已经生效就必须保证 start 被调到**（调用方放 `finally`，并按
+ * `mayBeSuspended` 判断）。漏掉 start 会让用户沙箱的预览再也起不来 —— 比迁移失败严重得多。
  *
  * 全部 API 都是软失败语义：不抛异常，失败信息放在返回值的 `error` 上，由调用方决定降级路径
  * （老版本 code-server 没有这两个接口，会返回 404）。
  */
 const DEFAULT_CODE_SERVER_BASE_URL = 'http://localhost:8080/code-server';
 const DEFAULT_TIMEOUT_MS = 10_000;
-/** start 是恢复动作，值得多试几次；stop 失败可以直接降级，不重试 */
+/**
+ * stop 的本地超时。必须大于服务端等进程退出的 10s（STOP_TIMEOUT）—— 否则我们会先 abort，
+ * 拿到一个"网络超时"，而服务端那边照样置着暂停位并继续走完流程。
+ */
+const STOP_TIMEOUT_MS = 20_000;
+/** start 是恢复动作，值得多试几次；stop 失败由调用方分类处理，不在这里重试 */
 const START_ATTEMPTS = 3;
 const START_RETRY_DELAY_MS = 500;
 /** code-server 本地基址，去掉尾部斜杠。`MIAODA_CODE_SERVER_BASE_URL` > 默认 localhost:8080。 */
@@ -37,18 +53,32 @@ function resolveCodeServerBaseUrl() {
     const base = fromEnv !== undefined && fromEnv !== '' ? fromEnv : DEFAULT_CODE_SERVER_BASE_URL;
     return base.replace(/\/+$/, '');
 }
-/** 停止预览（含保活）。软失败：不抛异常。 */
+/**
+ * 停止预览（dev 进程 + 保活）。返回 `ok` 时进程已确认退出，无需再 pkill。
+ *
+ * 软失败：不抛异常。失败时务必看 `mayBeSuspended` —— 为 true 就必须调 {@link startPreview} 恢复。
+ */
 async function stopPreview(opts = {}) {
-    return postPreview('stop', 1, opts);
+    const res = await postPreview('stop', 1, {
+        ...opts,
+        timeoutMs: opts.timeoutMs ?? STOP_TIMEOUT_MS,
+    });
+    // 404 = 路由不存在（老 code-server），确定没有副作用；其余失败保守认为暂停位可能已置上
+    return { ...res, mayBeSuspended: res.ok || res.status !== 404 };
 }
-/** 启动预览（恢复保活）。软失败：不抛异常；内部带重试，因为这是恢复动作。 */
+/** 启动预览（解除暂停 + 拉起进程）。软失败：不抛异常；内部带重试，因为这是恢复动作。 */
 async function startPreview(opts = {}) {
     return postPreview('start', START_ATTEMPTS, opts);
 }
 async function postPreview(action, attempts, opts) {
     const url = `${resolveCodeServerBaseUrl()}/api/v1/preview/${action}`;
     const logPrefix = opts.logPrefix ?? 'preview';
-    let last = { ok: false, url, error: 'no attempt made' };
+    let last = {
+        ok: false,
+        url,
+        error: 'no attempt made',
+        mayBeSuspended: false,
+    };
     for (let i = 1; i <= attempts; i++) {
         last = await postOnce(url, opts.timeoutMs ?? DEFAULT_TIMEOUT_MS);
         if (last.ok)
@@ -77,15 +107,17 @@ async function postOnce(url, timeoutMs) {
                 url,
                 status: res.status,
                 error: `POST ${url} → ${String(res.status)} ${res.statusText}${detail}`,
+                mayBeSuspended: false,
             };
         }
-        return { ok: true, url, status: res.status };
+        return { ok: true, url, status: res.status, mayBeSuspended: false };
     }
     catch (err) {
         return {
             ok: false,
             url,
             error: `POST ${url} failed: ${err instanceof Error ? err.message : String(err)}`,
+            mayBeSuspended: false,
         };
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lark-apaas/miaoda-cli",
-  "version": "0.1.33-alpha.59a1bf2",
+  "version": "0.1.33-alpha.dc7b6a0",
   "description": "Miaoda 平台命令行工具，面向 Agent 调用",
   "type": "commonjs",
   "bin": {