@thinkingai/ae-cli 1.0.27 → 1.0.30

package/dist/{sync-EVEDS7YE.js → sync-NVASAISS.js}

@@ -1,6 +1,8 @@
 import {
   MultiselectCancelled,
   SKILL_MANIFEST_FILE,
+  assertValidMcpName,
+  assertValidSkillSlug,
   describeSource,
   getCurrentWorkspace,
   promptMultiselect,
@@ -8,19 +10,20 @@ import {
   readSkillManifestEntries,
   scanMcps,
   scanSkills,
+  splitPushableMcps,
   writeSkillManifestEntries
-} from "./chunk-6T7245YE.js";
+} from "./chunk-EGEIXA2Z.js";
 import {
   TeAgentApiError,
   TeAgentCredentialsError,
   getSandboxSyncPullCandidates,
   postSandboxSyncPull,
   postToMainApp
-} from "./chunk-TG64AQV4.js";
+} from "./chunk-VJFUB3H3.js";
 import {
   printError
-} from "./chunk-B4UL2VIJ.js";
-import "./chunk-MNHE6SSI.js";
+} from "./chunk-CLJF7MQA.js";
+import "./chunk-SRJIAOBN.js";
 
 // src/commands/sync/index.ts
 import Table from "cli-table3";
@@ -30,19 +33,57 @@ import {
   cpSync,
   existsSync,
   mkdirSync,
+  readFileSync,
   realpathSync,
   rmSync,
-  statSync
+  statSync,
+  writeFileSync
 } from "fs";
 import path from "path";
-var SKILL_SLUG_RE = /^[a-z0-9-]+$/;
-function updateSkillManifestForSource(sourceDir, slug) {
-  if (!SKILL_SLUG_RE.test(slug)) {
-    throw new Error(`\u975E\u6CD5 Skill slug: ${slug}`);
+var MCP_MANIFEST_FILE = ".mcp-manifest.json";
+function isMcpScope(value) {
+  return value === "personal" || value === "company" || value === "system";
+}
+function uniqueMcpEntries(entries) {
+  const seen = /* @__PURE__ */ new Set();
+  const out = [];
+  for (const entry of entries) {
+    if (seen.has(entry.name)) continue;
+    seen.add(entry.name);
+    out.push(entry);
   }
+  return out;
+}
+function readMcpManifestEntries(manifestPath) {
+  if (!existsSync(manifestPath)) return [];
+  let parsed;
+  try {
+    parsed = JSON.parse(readFileSync(manifestPath, "utf8"));
+  } catch {
+    return [];
+  }
+  if (!Array.isArray(parsed)) return [];
+  const entries = [];
+  for (const item of parsed) {
+    if (!item || typeof item !== "object") continue;
+    const entry = item;
+    if (typeof entry.name !== "string" || !isMcpScope(entry.scope)) continue;
+    entries.push({ name: entry.name, scope: entry.scope });
+  }
+  return uniqueMcpEntries(entries);
+}
+function writeMcpManifestEntries(manifestPath, entries) {
+  writeFileSync(
+    manifestPath,
+    JSON.stringify(uniqueMcpEntries(entries), null, 2) + "\n",
+    "utf8"
+  );
+}
+function updateSkillManifestForSource(sourceDir, slug) {
+  assertValidSkillSlug(slug);
   const sourceAbs = path.resolve(sourceDir);
   if (!statSync(sourceAbs).isDirectory()) {
-    throw new Error(`Skill \u6E90\u8DEF\u5F84\u4E0D\u662F\u76EE\u5F55: ${sourceDir}`);
+    throw new Error(`Skill source path is not a directory: ${sourceDir}`);
   }
   const manifestPath = path.join(path.dirname(sourceAbs), SKILL_MANIFEST_FILE);
   const manifest = readSkillManifestEntries(manifestPath);
@@ -60,16 +101,36 @@ function updateSkillManifestForSource(sourceDir, slug) {
   writeSkillManifestEntries(manifestPath, nextManifest);
   return { manifestPath, changed: true };
 }
-function copySkillPackageToTarget(args) {
-  if (!SKILL_SLUG_RE.test(args.slug)) {
-    throw new Error(`\u975E\u6CD5 Skill slug: ${args.slug}`);
+function updateMcpManifestForProjectSource(workspaceDir, name) {
+  assertValidMcpName(name);
+  const workspaceAbs = path.resolve(workspaceDir);
+  if (!statSync(workspaceAbs).isDirectory()) {
+    throw new Error(`MCP workspace path is not a directory: ${workspaceDir}`);
+  }
+  const manifestPath = path.join(workspaceAbs, MCP_MANIFEST_FILE);
+  const manifest = readMcpManifestEntries(manifestPath);
+  const nextEntry = { name, scope: "personal" };
+  const existingIndex = manifest.findIndex((entry) => entry.name === name);
+  if (existingIndex >= 0 && manifest[existingIndex].scope === nextEntry.scope) {
+    return { manifestPath, changed: false };
+  }
+  const nextManifest = manifest.slice();
+  if (existingIndex >= 0) {
+    nextManifest[existingIndex] = nextEntry;
+  } else {
+    nextManifest.push(nextEntry);
   }
+  writeMcpManifestEntries(manifestPath, nextManifest);
+  return { manifestPath, changed: true };
+}
+function copySkillPackageToTarget(args) {
+  assertValidSkillSlug(args.slug);
   if (!path.isAbsolute(args.targetRoot)) {
-    throw new Error(`skillTargetRoot \u5FC5\u987B\u662F\u7EDD\u5BF9\u8DEF\u5F84: ${args.targetRoot}`);
+    throw new Error(`skillTargetRoot must be an absolute path: ${args.targetRoot}`);
   }
   const sourceReal = realpathSync(args.sourceDir);
   if (!statSync(sourceReal).isDirectory()) {
-    throw new Error(`Skill \u6E90\u8DEF\u5F84\u4E0D\u662F\u76EE\u5F55: ${args.sourceDir}`);
+    throw new Error(`Skill source path is not a directory: ${args.sourceDir}`);
   }
   const targetDir = path.join(args.targetRoot, args.slug);
   const targetAbs = path.resolve(targetDir);
@@ -101,6 +162,13 @@ function redactEnv(env) {
   }
   return out;
 }
+var SOURCE_ORDER = {
+  global: 0,
+  workspace: 1,
+  project: 0,
+  local: 1,
+  user: 2
+};
 function skillToItem(s) {
   return {
     kind: "skill",
@@ -122,6 +190,7 @@ function mcpToItem(m, includeSecrets) {
     scope: "personal",
     source: m.source,
     workspacePath: m.workspacePath,
+    workspaceDir: m.workspaceDir,
     event: "upsert",
     transport: m.transport,
     url: m.url,
@@ -134,11 +203,11 @@ function mcpToItem(m, includeSecrets) {
 async function selectSkills() {
   const all = scanSkills();
   if (all.length === 0) {
-    process.stderr.write("\u672A\u627E\u5230\u4EFB\u4F55 Skill\uFF08\u626B\u63CF\u4E86\u5F53\u524D\u5DE5\u4F5C\u7A7A\u95F4 .claude/skills\uFF09\n");
+    process.stderr.write("No Skills found (scanned current workspace .claude/skills)\n");
     return [];
   }
   all.sort((a, b) => {
-    if (a.source !== b.source) return a.source === "global" ? -1 : 1;
+    if (a.source !== b.source) return SOURCE_ORDER[a.source] - SOURCE_ORDER[b.source];
     return a.slug.localeCompare(b.slug);
   });
   const items = all.map((s) => ({
@@ -147,26 +216,36 @@ async function selectSkills() {
     group: describeSource(s),
     hint: s.isSymlink ? "(symlink)" : void 0
   }));
-  const picked = await promptMultiselect({ title: "\u9009\u62E9\u8981\u63A8\u9001\u5230\u4E3B\u5E94\u7528\u7684 Skill", items });
+  const picked = await promptMultiselect({ title: "Select Skills to push to the main app", items });
   return picked.map(skillToItem);
 }
 async function selectMcps(includeSecrets) {
   const all = scanMcps();
   if (all.length === 0) {
-    process.stderr.write("\u672A\u627E\u5230\u4EFB\u4F55 MCP\uFF08\u626B\u63CF\u4E86\u5F53\u524D\u5DE5\u4F5C\u7A7A\u95F4 .mcp.json / .claude/.claude.json \u4E0E\u5168\u5C40 ~/.claude.json\uFF09\n");
+    process.stderr.write("No MCPs found (scanned current workspace .mcp.json / .claude/.claude.json and global ~/.claude.json)\n");
     return [];
   }
-  all.sort((a, b) => {
-    if (a.source !== b.source) return a.source === "global" ? -1 : 1;
+  const { supported, unsupportedStdio } = splitPushableMcps(all);
+  if (unsupportedStdio.length > 0) {
+    const names = unsupportedStdio.map((m) => m.slug).sort().join(", ");
+    process.stderr.write(`ae-cli sync push only supports http/sse MCPs; skipped stdio MCPs: ${names}
+`);
+  }
+  if (supported.length === 0) {
+    process.stderr.write("No syncable MCPs found. ae-cli sync push only supports http/sse.\n");
+    return [];
+  }
+  supported.sort((a, b) => {
+    if (a.source !== b.source) return SOURCE_ORDER[a.source] - SOURCE_ORDER[b.source];
     return a.slug.localeCompare(b.slug);
   });
-  const items = all.map((m) => ({
+  const items = supported.map((m) => ({
     value: m,
     label: m.slug,
     group: describeSource(m),
-    hint: m.hasSecrets && !includeSecrets ? "(env \u5DF2\u8131\u654F)" : void 0
+    hint: m.hasSecrets && !includeSecrets ? "(env redacted)" : void 0
   }));
-  const picked = await promptMultiselect({ title: "\u9009\u62E9\u8981\u63A8\u9001\u5230\u4E3B\u5E94\u7528\u7684 MCP", items });
+  const picked = await promptMultiselect({ title: "Select MCPs to push to the main app", items });
   return picked.map((m) => mcpToItem(m, includeSecrets));
 }
 function renderResults(results) {
@@ -178,7 +257,7 @@ function renderResults(results) {
   const failed = results.filter((r) => r.status === "failed").length;
   if (failed > 0) {
     process.stderr.write(`
-\u2717 \u6709 ${failed} \u9879\u5931\u8D25
+\u2717 ${failed} item(s) failed
 `);
   }
 }
@@ -193,7 +272,7 @@ async function selectPullResources(args) {
     return scopeDiff !== 0 ? scopeDiff : a.name.localeCompare(b.name);
   });
   if (all.length === 0) {
-    process.stderr.write(`${args.title} \u65E0\u53EF\u540C\u6B65\u5019\u9009
+    process.stderr.write(`${args.title}: no candidates available to sync
 `);
     return [];
   }
@@ -210,7 +289,7 @@ function toHttpItems(items) {
       const { kind: _kind2, dirPath: _dirPath, ...rest2 } = item;
       return rest2;
     }
-    const { kind: _kind, ...rest } = item;
+    const { kind: _kind, workspaceDir: _workspaceDir, ...rest } = item;
     return rest;
   });
 }
@@ -245,7 +324,7 @@ function copySyncedSkillPackages(skillItems, resp) {
     if (!item) continue;
     if (!resp.skillTargetRoot) {
       result.status = "failed";
-      result.message = "\u4E3B\u5E94\u7528\u54CD\u5E94\u7F3A\u5C11 skillTargetRoot\uFF0C\u65E0\u6CD5\u590D\u5236 Skill package";
+      result.message = "Main app response is missing skillTargetRoot; cannot copy Skill package";
       continue;
     }
     try {
@@ -256,34 +335,54 @@ function copySyncedSkillPackages(skillItems, resp) {
       });
       updateSkillManifestForSource(item.dirPath, item.slug);
       if (copied.skipped) {
-        result.message = "\u6E90\u76EE\u5F55\u5DF2\u662F\u76EE\u6807\u76EE\u5F55\uFF0C\u8DF3\u8FC7\u672C\u5730\u590D\u5236";
+        result.message = "Source directory is already the target directory; local copy skipped";
       }
     } catch (err) {
       result.status = "failed";
-      result.message = `\u672C\u5730\u590D\u5236\u5931\u8D25\uFF1A${err?.message ?? String(err)}`;
+      result.message = `Local copy failed: ${err?.message ?? String(err)}`;
+    }
+  }
+}
+function updateSyncedProjectMcpManifest(mcpItems, resp) {
+  if (mcpItems.length === 0) return;
+  const bySlug = new Map(mcpItems.map((item) => [item.slug, item]));
+  for (const result of resp.results) {
+    if (result.kind !== "mcp" || result.status !== "synced") continue;
+    const item = bySlug.get(result.slug);
+    if (!item || item.source !== "project") continue;
+    if (!item.workspaceDir) {
+      result.status = "failed";
+      result.message = "Main app synced the MCP, but local MCP manifest update failed: missing workspaceDir";
+      continue;
+    }
+    try {
+      updateMcpManifestForProjectSource(item.workspaceDir, item.slug);
+    } catch (err) {
+      result.status = "failed";
+      result.message = `Main app synced the MCP, but local MCP manifest update failed: ${err?.message ?? String(err)}`;
     }
   }
 }
 async function selectDirection(optsDirection) {
   if (optsDirection === "push" || optsDirection === "pull") return optsDirection;
   if (optsDirection) {
-    throw new Error("--direction \u4EC5\u652F\u6301 push \u6216 pull");
+    throw new Error("--direction must be push or pull");
   }
   return promptSingleSelect({
-    title: "\u540C\u6B65\u65B9\u5411\uFF1F",
+    title: "Sync direction?",
     items: [
-      { value: "push", label: "\u5DE5\u4F5C\u7A7A\u95F4 -> \u4E3B\u5E94\u7528" },
-      { value: "pull", label: "\u4E3B\u5E94\u7528 -> \u5DE5\u4F5C\u7A7A\u95F4" }
+      { value: "push", label: "Workspace -> Main app" },
+      { value: "pull", label: "Main app -> Workspace" }
     ]
   });
 }
 async function selectKind(optsKind) {
   if (optsKind === "skill" || optsKind === "mcp" || optsKind === "both") return optsKind;
   if (optsKind) {
-    throw new Error("--kind \u4EC5\u652F\u6301 skill\u3001mcp \u6216 both");
+    throw new Error("--kind must be skill, mcp, or both");
   }
   return promptSingleSelect({
-    title: "\u540C\u6B65\u54EA\u7C7B\u8D44\u6E90\uFF1F",
+    title: "Which resource type to sync?",
     items: [
       { value: "skill", label: "Skills" },
       { value: "mcp", label: "MCPs" },
@@ -295,10 +394,10 @@ async function runPush(kind, includeSecrets) {
   const skillItems = kind !== "mcp" ? await selectSkills() : [];
   const mcpItems = kind !== "skill" ? await selectMcps(includeSecrets) : [];
   if (skillItems.length === 0 && mcpItems.length === 0) {
-    process.stderr.write("\u672A\u9009\u62E9\u4EFB\u4F55\u9879\uFF0C\u5DF2\u9000\u51FA\n");
+    process.stderr.write("Nothing selected, exiting\n");
     return;
   }
-  process.stderr.write(`\u63A8\u9001 ${skillItems.length} \u4E2A Skill / ${mcpItems.length} \u4E2A MCP ...
+  process.stderr.write(`Pushing ${skillItems.length} Skill(s) / ${mcpItems.length} MCP(s) ...
 `);
   const allResults = [];
   if (skillItems.length > 0) {
@@ -308,6 +407,7 @@ async function runPush(kind, includeSecrets) {
   }
   if (mcpItems.length > 0) {
     const mcpResp = await pushItems("mcp", mcpItems);
+    updateSyncedProjectMcpManifest(mcpItems, mcpResp);
     allResults.push(...mcpResp.results);
   }
   renderResults(allResults);
@@ -323,32 +423,31 @@ function pullResultRows(resp) {
 async function runPull(kind) {
   const workspace = getCurrentWorkspace();
   if (!workspace) {
-    throw new Error("\u672A\u8BC6\u522B\u5F53\u524D\u5DE5\u4F5C\u7A7A\u95F4\uFF0C\u8BF7\u5728 /home/ta/workspaces/<workspace> \u4E0B\u6267\u884C");
+    throw new Error("Current workspace not recognized; please run from within /home/ta/workspaces/<workspace>");
   }
   const candidates = await getSandboxSyncPullCandidates({
     workspacePath: workspace.name,
     kind
   });
   const skillItems = kind !== "mcp" ? await selectPullResources({
-    title: "\u9009\u62E9\u8981\u540C\u6B65\u5230\u5DE5\u4F5C\u7A7A\u95F4\u7684 Skill",
-    candidates: candidates.skills ?? []
+    title: "Select Skills to sync to workspace",
+    candidates: (candidates.skills ?? []).filter((candidate) => candidate.scope !== "system")
   }) : [];
   const mcpItems = kind !== "skill" ? await selectPullResources({
-    title: "\u9009\u62E9\u8981\u540C\u6B65\u5230\u5DE5\u4F5C\u7A7A\u95F4\u7684 MCP",
+    title: "Select MCPs to sync to workspace",
     candidates: candidates.mcp ?? []
   }) : [];
   if (skillItems.length === 0 && mcpItems.length === 0) {
-    process.stderr.write("\u672A\u9009\u62E9\u4EFB\u4F55\u9879\uFF0C\u5DF2\u9000\u51FA\n");
+    process.stderr.write("Nothing selected, exiting\n");
     return;
   }
-  process.stderr.write(`\u540C\u6B65\u5230\u5DE5\u4F5C\u7A7A\u95F4 ${skillItems.length} \u4E2A Skill / ${mcpItems.length} \u4E2A MCP ...
+  process.stderr.write(`Syncing to workspace: ${skillItems.length} Skill(s) / ${mcpItems.length} MCP(s) ...
 `);
   const resp = await postSandboxSyncPull({
     workspacePath: workspace.name,
     kind,
     skills: skillItems.map((item) => item.id),
-    mcp: mcpItems.map((item) => item.id),
-    ifUnmodifiedSince: candidates.mtime
+    mcp: mcpItems.map((item) => item.id)
   });
   renderResults(pullResultRows(resp));
 }
@@ -361,7 +460,7 @@ function registerSync(program) {
       else await runPull(kind);
     } catch (err) {
       if (err instanceof MultiselectCancelled) {
-        process.stderr.write("\u5DF2\u53D6\u6D88\n");
+        process.stderr.write("Cancelled\n");
         return;
       }
       if (err instanceof TeAgentCredentialsError) {

package/dist/{te-agent-FLTDYV3L.js → te-agent-YV4DXEXK.js}

@@ -4,7 +4,8 @@ import {
   patchToMainApp,
   postToMainApp,
   uploadToMainApp
-} from "./chunk-TG64AQV4.js";
+} from "./chunk-VJFUB3H3.js";
+import "./chunk-SRJIAOBN.js";
 
 // src/commands/te-agent/models.ts
 var BASE_PATH = "/api/sandbox/agent/models";
@@ -19,7 +20,7 @@ var listModels = {
   validate: (ctx) => {
     const scope = ctx.str("scope");
     if (scope && !["personal", "company", "system"].includes(scope)) {
-      throw new Error("--scope \u5FC5\u987B\u662F personal\u3001company \u6216 system");
+      throw new Error("--scope must be personal, company, or system");
     }
   },
   dryRun: (ctx) => {
@@ -52,7 +53,7 @@ var addModel = {
     try {
       new URL(baseUrl);
     } catch {
-      throw new Error("--base-url \u5FC5\u987B\u662F\u6709\u6548 URL");
+      throw new Error("--base-url must be a valid URL");
     }
   },
   dryRun: (ctx) => ({
@@ -131,7 +132,7 @@ var listMcps = {
   validate: (ctx) => {
     const scope = ctx.str("scope");
     if (scope && !["personal", "company", "system"].includes(scope)) {
-      throw new Error("--scope \u5FC5\u987B\u662F personal\u3001company \u6216 system");
+      throw new Error("--scope must be personal, company, or system");
     }
   },
   dryRun: (ctx) => {
@@ -161,22 +162,22 @@ var addMcp = {
   validate: (ctx) => {
     const name = ctx.str("name");
     if (!/^[a-zA-Z][a-zA-Z0-9_-]*$/.test(name)) {
-      throw new Error("--name \u5FC5\u987B\u4EE5\u5B57\u6BCD\u5F00\u5934\uFF0C\u4EC5\u5305\u542B\u5B57\u6BCD\u3001\u6570\u5B57\u3001_\u3001-");
+      throw new Error("--name must start with a letter and contain only letters, digits, _, or -");
     }
     if (name.length < 2 || name.length > 64) {
-      throw new Error("--name \u957F\u5EA6\u5FC5\u987B\u5728 2-64 \u4E4B\u95F4");
+      throw new Error("--name length must be between 2 and 64");
     }
     try {
       new URL(ctx.str("url"));
     } catch {
-      throw new Error("--url \u5FC5\u987B\u662F\u6709\u6548 URL");
+      throw new Error("--url must be a valid URL");
     }
     if (!/^https?:\/\//i.test(ctx.str("url"))) {
-      throw new Error("--url \u4EC5\u652F\u6301 http:// \u6216 https:// \u534F\u8BAE");
+      throw new Error("--url only supports http:// or https:// protocols");
     }
     const transport = ctx.str("transport");
     if (transport && !["sse", "http"].includes(transport)) {
-      throw new Error("--transport \u5FC5\u987B\u662F sse \u6216 http");
+      throw new Error("--transport must be sse or http");
     }
   },
   dryRun: (ctx) => ({
@@ -249,7 +250,7 @@ async function resolveInstructions(raw) {
       chunks.push(chunk);
     }
     const text = Buffer.concat(chunks).toString("utf8").trim();
-    if (!text) throw new Error("stdin \u4E3A\u7A7A\uFF0C\u65E0\u6CD5\u8BFB\u53D6 instructions");
+    if (!text) throw new Error("stdin is empty; cannot read instructions");
     return text;
   }
   return raw;
@@ -265,7 +266,7 @@ var listSkills = {
   validate: (ctx) => {
     const scope = ctx.str("scope");
     if (scope && !["personal", "company", "system"].includes(scope)) {
-      throw new Error("--scope \u5FC5\u987B\u662F personal\u3001company \u6216 system");
+      throw new Error("--scope must be personal, company, or system");
     }
   },
   dryRun: (ctx) => {
@@ -293,7 +294,7 @@ var addSkill = {
   validate: (ctx) => {
     const name = ctx.str("name");
     if (name.length < 1 || name.length > 80) {
-      throw new Error("--name \u957F\u5EA6\u5FC5\u987B\u5728 1-80 \u4E4B\u95F4");
+      throw new Error("--name length must be between 1 and 80");
     }
   },
   dryRun: (ctx) => ({
@@ -388,21 +389,21 @@ function resolveFilePaths(ctx) {
   const file = ctx.str("file");
   const files = ctx.str("files");
   if (file && files) {
-    throw new Error("--file \u548C --files \u4E0D\u80FD\u540C\u65F6\u4F7F\u7528");
+    throw new Error("--file and --files cannot be used together");
   }
   if (!file && !files) {
-    throw new Error("\u5FC5\u987B\u6307\u5B9A --file \u6216 --files");
+    throw new Error("Either --file or --files must be specified");
   }
   if (file) return [file];
   try {
     const parsed = JSON.parse(files);
-    if (!Array.isArray(parsed)) throw new Error("--files \u5FC5\u987B\u662F JSON \u6570\u7EC4");
+    if (!Array.isArray(parsed)) throw new Error("--files must be a JSON array");
     return parsed.map((p) => {
-      if (typeof p !== "string") throw new Error("--files \u6570\u7EC4\u5143\u7D20\u5FC5\u987B\u662F\u5B57\u7B26\u4E32");
+      if (typeof p !== "string") throw new Error("--files array elements must be strings");
       return p;
     });
   } catch (err) {
-    if (err instanceof SyntaxError) throw new Error("--files \u5FC5\u987B\u662F\u6709\u6548 JSON \u6570\u7EC4");
+    if (err instanceof SyntaxError) throw new Error("--files must be a valid JSON array");
     throw err;
   }
 }
@@ -411,7 +412,7 @@ async function uploadFiles(filePaths) {
   for (const filePath of filePaths) {
     const resolved = resolve(filePath);
     if (!existsSync(resolved)) {
-      throw new Error(`\u6587\u4EF6\u4E0D\u5B58\u5728\uFF1A${resolved}`);
+      throw new Error(`File not found: ${resolved}`);
     }
     const buffer = readFileSync(resolved);
     const mimeType = guessMimeType(resolved);
@@ -434,7 +435,7 @@ var listAttachments = {
   validate: (ctx) => {
     const type = ctx.str("type");
     if (type && !["image", "document"].includes(type)) {
-      throw new Error("--type \u5FC5\u987B\u662F image \u6216 document");
+      throw new Error("--type must be image or document");
     }
   },
   dryRun: (ctx) => {
@@ -496,22 +497,22 @@ var delAttachment = {
 
 // src/commands/te-agent/index.ts
 var commands = [
-  // 模型管理
+  // Model management
   listModels,
   addModel,
   delModel,
   toggleModel,
-  // MCP 服务管理
+  // MCP server management
   listMcps,
   addMcp,
   delMcp,
   toggleMcp,
-  // Skill 管理
+  // Skill management
   listSkills,
   addSkill,
   delSkill,
   toggleSkill,
-  // 附件库管理
+  // Attachment library management
   listAttachments,
   addAttachment,
   delAttachment