@cnbcool/cnb-api-generate 2.17.4 → 2.18.0

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.0",
   "main": "./built/index.js",
   "module": "./src/index.ts",
   "types": "./src/index.ts",