@cnbcool/cnb-api-generate 2.17.4 → 2.18.1

package/client/lib/execute-action.ts

@@ -5,6 +5,7 @@ import { diagnoseBuild } from './build-diagnostics';
 import { analyzeBuildTiming } from './build-timing';
 import { formatParams } from './format-params';
 import { formatOutput } from './format-output';
+import { applyExitCode } from './exit-code';
 import { buildToolParams } from './build-tool-params';
 import { skillsList } from './skills-list';
 // @ts-ignore
@@ -103,6 +104,8 @@ export async function executeAction(
       data = await handleUpload(shortcut, opts.file, toolFunction, pathParams);
     }
 
+    // 统一退出码策略：非 2xx 置非零退出码（见 lib/exit-code.ts）
+    applyExitCode(data);
     console.log(formatOutput(data, isVerbose, !isVerbose, `${shortcut.module}/${shortcut.tool}`));
     return;
   }
@@ -148,5 +151,7 @@ export async function executeAction(
   const toolKey = `${formattedParams.module}/${formattedParams.tool}`;
   const isVerbose = !!formattedParams.verbose;
   const isSummary = shortcut ? !isVerbose : false;
+  // 统一退出码策略：所有 API 响应都经此出口，非 2xx 置非零退出码
+  applyExitCode(data);
   console.log(formatOutput(data, isVerbose, isSummary, toolKey));
 }

package/client/lib/exit-code.ts

@@ -0,0 +1,50 @@
+/**
+ * 统一退出码策略（见 cnb/skills/cnb-skill#458）
+ *
+ * 现状：`cnb` 的退出码只区分「命令名拼错」「未登录」两类失败，
+ * HTTP 4xx/5xx 一律 exit 0 —— 失败只体现在 stdout 的 `status:` 字段里。
+ * 对自动化（含 AI Agent）来说「退出码 0」就是成功信号，
+ * `cnb pulls submit-review` 在权限不足 / 编号写错时返回 4xx + exit 0，
+ * 看起来像已生效。
+ *
+ * 策略（本文件只做这一件事）：HTTP 非 2xx 响应置非零退出码。
+ * 用 `process.exitCode` 而非 `process.exit()`：让已排队的 stdout 正常 flush，
+ * 输出内容不被截断，进程在事件循环结束后以非零码退出。
+ */
+
+/** 退出码：与既有失败路径区分，便于脚本定位失败类型 */
+export const EXIT_CODE_HTTP_ERROR = 1;
+
+/**
+ * 从响应对象中解析 HTTP 状态码。
+ * 兼容 responseConverter 可能返回的裸 data（无 status 字段）。
+ */
+export function getResponseStatus(response: any): number | undefined {
+  if (response && typeof response === 'object' && typeof response.status === 'number') {
+    return response.status;
+  }
+  return undefined;
+}
+
+/** 是否为 HTTP 成功响应（2xx） */
+export function isHttpSuccess(status: number | undefined): boolean {
+  return typeof status === 'number' && status >= 200 && status < 300;
+}
+
+/**
+ * 按统一策略处理一次 API 调用结果：
+ * - 非 2xx 响应把退出码置为 1（不抛错，输出照旧）。
+ * - 拿不到 status（converter 返回的裸 data）时不误判失败。
+ *
+ * @param response 响应对象（含 status）
+ * @returns 该响应的 HTTP 状态码（无则 undefined）
+ */
+export function applyExitCode(response: any): number | undefined {
+  const status = getResponseStatus(response);
+
+  if (status !== undefined && !isHttpSuccess(status)) {
+    process.exitCode = EXIT_CODE_HTTP_ERROR;
+  }
+
+  return status;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cnbcool/cnb-api-generate",
-  "version": "2.17.4",
+  "version": "2.18.1",
   "main": "./built/index.js",
   "module": "./src/index.ts",
   "types": "./src/index.ts",

package/skills-template/SKILL.md

@@ -19,29 +19,20 @@ description: CNB 平台交互命令，支持代码仓库、Issue、PR、CI、制
 编号同 `--repo` 一样默认从环境变量识别，无需传递；用 `--repo <组织/仓库>`、`--number <编号>`
 可覆盖，例：`cnb issues get --number 68`。
 
-- `cnb issues get-imgs --img-path 图片路径` — 取正文里的图片
-- `cnb issues get-files --file-path 附件路径` — 取正文里的附件
-
-`--img-path` / `--file-path` 取 URL 的**完整尾段**，首位数字 ID 段不能省，漏掉返回 `404`：
-
-- 原文 `.../-/imgs/issues/2101868753481326592/Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png`
-- 正确 `2101868753481326592/Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png` ✅
-- 漏 ID `Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png` ❌ `404`
-
-鉴权由 `@cnbcool/cnb-cli`（命令 `cnb`）内置，执行命令即带登录态，
-`get-imgs` / `get-files` 无需自己传鉴权头。
-
 ## pull 访问
 
 编号同 `--repo` 一样默认从环境变量识别，无需传递；用 `--repo <组织/仓库>`、`--number <编号>`
 可覆盖，例：`cnb pulls get --repo x/y --number 97`。
 
-- `cnb pulls get-imgs --img-path 图片路径` — 取正文里的图片
-- `cnb pulls get-files --file-path 附件路径` — 取正文里的附件
+## 图片与附件解析
+
+正文里图片、附件的下载地址形如：
+
+- 图片 `.../-/imgs/<issues|pulls>/<数字 ID>/<key>/<文件名>`
+- 附件 `.../-/files/<issues|pulls>/<数字 ID>/<key>/<文件名>`
 
-`--img-path` / `--file-path` 与 issue 同规则：取 URL 的**完整尾段**，首位数字 ID 段不能省，
-漏掉返回 `404`（原文形如 `.../-/imgs/pulls/<数字 ID>/<key>/<file>`）。
-鉴权同样由 `cnb` 命令内置的登录态提供，无需自己传鉴权头。
+参数取 `/imgs/<issues|pulls>/`、`/files/<issues|pulls>/` 之后的**完整尾段**，
+首位的数字 ID 段不能省，漏掉返回 `404`。
 
 ## 常用链接