@lark-apaas/miaoda-cli 0.1.24-alpha.7499e8d → 0.1.24-alpha.77e43f5

package/dist/cli/commands/skills/index.js

@@ -26,10 +26,11 @@ function registerSkillsSync(parent) {
         .option('--version <ver>', 'coding-steering 包版本或 dist-tag，缺省 latest')
         .option('--local', '走本地 dev / agent 输出形态：拷到 .agents/skills 平铺 + 创建 .claude/skills 软链。' +
         '不传时退回老形态：拷到 .agent/skills/steering/<stack>/skills/（跟沙箱端 update-skills.sh 对齐）', false)
-        .option('--type <type>', '走 sandbox-skills 分支：miaoda | miaoda-modern | miaoda-design | openclaw。' +
-        '有值时忽略 --version/--local，改从 @lark-apaas/coding-<pkg>-sandbox 拉对应包写入 .agent/skills/')
+        .option('--type <type>', '走 sandbox 分支：miaoda | miaoda-modern | miaoda-design | openclaw。有值时忽略 --local，' +
+        '从 @lark-apaas/coding-<pkg>-sandbox-skills 拉包平铺写入，并额外同步 steering ' +
+        '到 .agent/skills/steering/<stack>/（--version 对后者仍生效）')
         .option('--tgz-key <key>', '配合 --type 使用；老 TCC skillsUpdateTgzKey 值透传，' +
-        '从 skills_versions/all_<X>.tgz 反推出 dist-tag lane-<X>；缺省走 @latest')
+        '从 skills_versions/all_<X>.tgz 反推出 lane 名 X，再过滤出 -alpha.<X> 版本；缺省走 @latest')
         .addHelpText('after', `
 前置要求
   当前目录（或 --dir）已走过 'miaoda app init'（.spark/meta.json 含 stack）
@@ -45,15 +46,28 @@ function registerSkillsSync(parent) {
     .agents/tech.md
     .claude/skills → ../.agents/skills（软链，让 Claude Code 识别同一份）
 
+  传 --type（沙箱链路，两部分都同步）：
+    .agent/skills/<skill>/...                        ← 业务线 skill，平铺
+    .agent/skills/steering/<stack>/skills/<skill>/   ← steering，同上面默认形态
+    （openclaw 的业务线部分落 skills/ 而非 .agent/skills/）
+
 JSON 输出
   {"data": {"stack": "...", "version": "...", "syncedSkills": [...], "techSynced": true|false,
             "claudeSkillsLink": "created|updated|noop|conflict"  // 仅 --local 时}}
 
+  传 --type 时：
+  {"data": {"mode": "sandbox", "type": "...", "packageName": "...", "version": "...",
+            "lane": "...", "syncedTop": [...], "targetDir": "...",
+            "steering": {"status": "synced|skipped|failed", ...}}}
+  steering 失败或无 stack 不影响整体成功——业务线 skill 已同步，不该因增量部分阻断
+
 示例
   $ miaoda skills sync                              # 拉 @latest
   $ miaoda skills sync --local                      # 本地 dev / agent 用
   $ miaoda skills sync --version 0.2.0              # 显式指定版本
   $ miaoda skills sync --dir ./my-app
+  # 沙箱链路（由 feida-ai 下发）：业务线 skill + steering 一次同步到位
+  $ miaoda skills sync --type miaoda-modern --tgz-key 'skills_versions/all_demo-lane.tgz'
 `);
     cmd.action((0, shared_1.withHelp)(cmd, async (rawOpts) => {
         await (0, index_1.handleSkillsSync)({

package/dist/cli/handlers/skills/sync.js

@@ -10,6 +10,57 @@ const sandbox_skills_1 = require("../../../utils/sandbox-skills");
 const spark_meta_1 = require("../../../utils/spark-meta");
 const error_1 = require("../../../utils/error");
 const output_1 = require("../../../utils/output");
+/**
+ * sandbox 分支下补同步 steering（`.agent/skills/steering/<stack>/`）。
+ *
+ * **绝不抛错**，因为它在 sandbox 链路里是「附加项」而非门禁：
+ *   - 无 `.spark/meta.json` / 无 stack → skipped。对齐老 cp-skills.sh 的
+ *     `[ -n "$STACK" ] && [ "$STACK" != "null" ]` 判断（无 stack 就跳过 steering）。
+ *   - 拉包 / 写盘失败 → failed，但仍让整条 sync 算成功。
+ *
+ * 为什么不能抛：feida-ai 侧 TCC 可配 `allowFailure: false`，一抛就阻断 agent 启动。
+ * 平铺业务线 skill 已经同步好了，不该因为 steering 这个增量部分让用户开不了会话。
+ */
+function syncSteeringForSandbox(params) {
+    let stack;
+    try {
+        const meta = (0, spark_meta_1.readSparkMeta)(params.targetDir);
+        if (meta.stack === undefined || meta.stack === '' || meta.stack === 'null') {
+            return { status: 'skipped', reason: '.spark/meta.json missing stack' };
+        }
+        stack = meta.stack;
+    }
+    catch (err) {
+        return { status: 'skipped', reason: `read .spark/meta.json failed: ${err.message}` };
+    }
+    // 版本口径与下方本地分支保持一致：--version > BOE 的 dist-tag dev > latest
+    const isBoe = process.env.FORCE_FRAMEWORK_ENVIRONMENT === 'boe';
+    const effectiveVersion = params.version ?? (isBoe ? 'dev' : undefined);
+    try {
+        const r = (0, coding_steering_1.syncCodingSteering)({
+            stack,
+            targetDir: params.targetDir,
+            version: effectiveVersion,
+            // 必须显式传 mode，不能落到 inferMode()：它靠 env MIAODA_DEP_CACHE_DIR 嗅探，
+            // 而 mode 决定读包内哪个 layer（sandbox→`skills/`，local→`skills_local/`）。
+            // `--type` 分支本身就是沙箱语义，嗅探失手会去读不存在的 skills_local/、
+            // 静默同步 0 个 skill。
+            mode: 'sandbox',
+            // 同理固定 nested：落 .agent/skills/steering/<stack>/，与老链路一致
+            outputLayout: 'nested',
+        });
+        return {
+            status: 'synced',
+            stack,
+            version: r.version,
+            syncedSkills: r.syncedSkills,
+            techSynced: r.techSynced,
+        };
+    }
+    catch (err) {
+        return { status: 'failed', reason: err.message };
+    }
+}
 /**
  * miaoda skills sync [--dir <path>] [--version <ver>]
  *
@@ -27,11 +78,25 @@ const output_1 = require("../../../utils/output");
 async function handleSkillsSync(opts) {
     await Promise.resolve();
     const targetDir = node_path_1.default.resolve(opts.dir ?? process.cwd());
-    // --type 走 sandbox-skills 分支（迁仓后沙箱内 update-skills.sh 调用），跟本地 dev 的
-    // steering 同步逻辑并列——两条路径的目标 / 拉包 / 目录结构完全不同，用 --type 显式路由。
+    // --type 走 sandbox-skills 分支（迁仓后沙箱内由 feida-ai 直接下发调用）。
+    //
+    // 这条分支必须**同时**同步两部分，缺一不可 —— 线上老链路 update-skills.sh 也是两步
+    // （自己 rsync 平铺，再于第 230 行调不带 --type 的 `miaoda skills sync` 写 steering）：
+    //   ① syncSandboxSkills：平铺业务线 skill，rsync 式对齐（保 .keep/.config/steering）
+    //   ② steering：`.agent/skills/steering/<stack>/`，按 .spark/meta.json 的 stack 过滤
+    //
+    // 历史 bug：早期这里 ① 完就 return，steering 既不同步、又被当时的整目录清空删掉，
+    // 沙箱少掉整批技术栈 skill（BOE 实测 scanAllSkills 从 59 掉到 34）。
+    //
+    // `.keep` 语义（update-skills.sh:149）：目标目录挂了 .keep 表示「我手改过，别覆盖」，
+    // ① 会跳过并回报 skippedByKeepFile，此时 ② 也必须跳过 —— 否则 steering 仍被覆写，
+    // 冻结只冻结了一半。
     if (opts.type !== undefined && opts.type !== '') {
         const { type, tgzKey } = (0, sandbox_skills_1.validateSandboxSyncOpts)({ type: opts.type, tgzKey: opts.tgzKey });
         const result = (0, sandbox_skills_1.syncSandboxSkills)({ type, tgzKey, targetDir });
+        const steering = result.skippedByKeepFile
+            ? { status: 'skipped', reason: '.keep file found, sync frozen' }
+            : syncSteeringForSandbox({ targetDir, version: opts.version });
         (0, output_1.emit)({
             data: {
                 mode: 'sandbox',
@@ -40,7 +105,10 @@ async function handleSkillsSync(opts) {
                 version: result.version,
                 lane: result.lane,
                 syncedTop: result.syncedTop,
+                deleted: result.deleted,
+                skippedByKeepFile: result.skippedByKeepFile,
                 targetDir: result.targetDir,
+                steering,
             },
         });
         return;

package/dist/utils/sandbox-skills.js

@@ -44,11 +44,53 @@ function forceRemoveDir(dir) {
     node_fs_1.default.rmSync(dir, { recursive: true, force: true });
 }
 const PACKAGE_BY_TYPE = {
-    miaoda: '@lark-apaas/coding-miaoda-sandbox',
-    'miaoda-modern': '@lark-apaas/coding-miaoda-sandbox',
-    'miaoda-design': '@lark-apaas/coding-miaoda-sandbox',
-    openclaw: '@lark-apaas/coding-openclaw-sandbox',
+    miaoda: '@lark-apaas/coding-miaoda-sandbox-skills',
+    'miaoda-modern': '@lark-apaas/coding-miaoda-sandbox-skills',
+    'miaoda-design': '@lark-apaas/coding-miaoda-sandbox-skills',
+    openclaw: '@lark-apaas/coding-openclaw-sandbox-skills',
 };
+/**
+ * 目标根目录下**永不删除**的控制文件（对齐 `update-skills.sh` 的删除白名单）：
+ *   - `.keep`        沙箱级「冻结 skills」开关，存在即整体跳过同步（见 {@link syncSandboxSkills}）
+ *   - `.config`      沙箱级 tgzKey 覆盖，feida-ai 的 `enableConfigFileOverride` 读它
+ *                    （`getKeepFileTgzKey` → `.agent/skills/.config`），删了会让版本钉死失效
+ *   - `.last-update` 同步时间戳，feida-ai freshness 检查读它；每轮末尾会重写，列进来只是语义完备
+ */
+const PRESERVED_ENTRIES = new Set(['.keep', '.config', '.last-update']);
+/** `.keep` 存在时整体跳过同步的标记文件名，语义同 `update-skills.sh:149`。 */
+const KEEP_FILE = '.keep';
+/**
+ * rsync 式对齐：删掉目标里「源包已不提供」的顶层条目，对齐 `update-skills.sh:196-218`。
+ *
+ * 与整目录清空的区别（后者是本函数取代的旧实现）：
+ *   - 保住 {@link PRESERVED_ENTRIES} 里的控制文件
+ *   - 保住 `steering/` 子树 —— 它由 handler 层的 syncCodingSteering 独占管理
+ *     （`update-skills.sh:207` 平铺型分支同样明确「归 miaoda CLI 管，update.sh 不删不覆盖」）
+ *
+ * 只处理顶层：源包内容本身就是「一个 skill 一个顶层目录」，逐文件比对没有额外收益。
+ */
+function pruneStaleEntries(skillsRoot, sourceTopEntries) {
+    if (!node_fs_1.default.existsSync(skillsRoot))
+        return [];
+    const deleted = [];
+    for (const entry of node_fs_1.default.readdirSync(skillsRoot)) {
+        if (PRESERVED_ENTRIES.has(entry) || entry === 'steering')
+            continue;
+        if (sourceTopEntries.has(entry))
+            continue;
+        const target = node_path_1.default.join(skillsRoot, entry);
+        try {
+            // forceRemoveDir 的 chmod→rm -rf→rmSync 三层兜底对文件同样适用：
+            // 老 update.sh 以 root 跑过、且会 chmod -R 锁只读，直接 rmSync 会 EACCES
+            forceRemoveDir(target);
+            deleted.push(entry);
+        }
+        catch (err) {
+            (0, logger_1.debug)(`sandbox-skills: prune ${entry} failed: ${err.message}`);
+        }
+    }
+    return deleted;
+}
 // 合并包内的业务线子目录（miaoda / miaoda-modern / miaoda-design）；shared 单独处理。
 const BUSINESS_LINE_BY_TYPE = {
     miaoda: 'miaoda',
@@ -88,8 +130,15 @@ function extractLaneFromTgzKey(tgzKey) {
 /**
  * 主流程：按 type 拉包 + sync 到目标目录。
  *
- * sync 策略：**覆盖式写入 + 老文件清理**，对齐老 cp-skills.sh 行为
- * （`rm -rf $TARGET; mkdir -p $TARGET; cp -r ...`）。
+ * sync 策略：**rsync 式对齐**，逐条对应线上真实跑的 `update-skills.sh`（不是
+ * `cp-skills.sh`——那个脚本线上不执行，日志里的 `copied/skipped/deleted` 出自前者）：
+ *   1. 目标目录有 `.keep` → 整体跳过（`update-skills.sh:149`）
+ *   2. 覆盖式写入源包提供的顶层条目
+ *   3. 删掉源包已不提供的顶层条目（{@link pruneStaleEntries}），但保住
+ *      {@link PRESERVED_ENTRIES} 与 `steering/`
+ *
+ * 不用整目录清空：那会删掉 `.config`（沙箱级 tgzKey 覆盖）、`.keep`（冻结开关）
+ * 和 handler 层刚写的 `steering/`，而老链路对这三者都有保护。
  */
 function syncSandboxSkills(opts) {
     const targetDir = node_path_1.default.resolve(opts.targetDir ?? process.cwd());
@@ -137,11 +186,21 @@ function syncSandboxSkills(opts) {
         const skillsRoot = opts.type === 'openclaw'
             ? node_path_1.default.join(targetDir, 'skills')
             : node_path_1.default.join(targetDir, '.agent', 'skills');
-        // 覆盖式：清空目标 skill 根目录（对齐 cp-skills.sh 行为，避免遗留老 skill）
-        // 用 forceRemoveDir 兜底：老沙箱可能残留其他用户写的 stale 文件（例如老 update.sh
-        // 以 root 跑，写下 root owned 的 `.last-update`），fs.rmSync {force:true} 遇到 EACCES
-        // 会 throw。见 forceRemoveDir 注释。
-        forceRemoveDir(skillsRoot);
+        // `.keep` 短路：对齐 update-skills.sh:149-152，沙箱侧放了这个文件即表示
+        // 「我手改过 skills，别覆盖我」，整体跳过同步（含 steering —— 调用方看到
+        // skipped 就不该再往下写）。
+        if (node_fs_1.default.existsSync(node_path_1.default.join(skillsRoot, KEEP_FILE))) {
+            (0, logger_1.debug)(`sandbox-skills: ${KEEP_FILE} found in ${skillsRoot}, skip sync`);
+            return {
+                packageName,
+                version: fetched.version,
+                lane,
+                syncedTop: [],
+                deleted: [],
+                skippedByKeepFile: true,
+                targetDir: skillsRoot,
+            };
+        }
         node_fs_1.default.mkdirSync(skillsRoot, { recursive: true });
         const syncedTop = [];
         if (businessLine !== null) {
@@ -188,6 +247,10 @@ function syncSandboxSkills(opts) {
                 syncedTop.push(entry);
             }
         }
+        // rsync 式收尾：删掉源包已不提供的顶层条目（对齐 update-skills.sh 的 deleted）。
+        // 放在写入之后：syncedTop 就是本轮源包提供的全部顶层条目，直接作为保留集，
+        // 不必再遍历一遍源目录。
+        const deleted = pruneStaleEntries(skillsRoot, new Set(syncedTop));
         // 写 .last-update 时间戳（对齐老 update-skills.sh；供 feida-ai freshness 检查用）
         const lastUpdatePath = node_path_1.default.join(skillsRoot, '.last-update');
         node_fs_1.default.writeFileSync(lastUpdatePath, String(Date.now()));
@@ -195,6 +258,8 @@ function syncSandboxSkills(opts) {
             packageName,
             version: fetched.version,
             lane,
+            deleted,
+            skippedByKeepFile: false,
             syncedTop,
             targetDir: skillsRoot,
         };

package/package.json

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