@cnbcool/cnb-api-generate 2.17.3 → 2.18.0

package/built/codegen/printer/printer-ts-ast-to-js-file.js

@@ -42,7 +42,6 @@ const babel = __importStar(require("@babel/core"));
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const debug_1 = __importDefault(require("debug"));
-const skills_1 = require("../../constants/skills");
 const logger = (0, debug_1.default)('csg:babel');
 function transformTsToJs(tsCode, filename) {
     return babel.transformSync(tsCode, {
@@ -55,7 +54,7 @@ function transformTsToJs(tsCode, filename) {
             [
                 'transform-define',
                 {
-                    'process.env.CNB_CLI_CMD': process.env.CNB_CLI_CMD || skills_1.CNB_CLI_PROD_CMD,
+                    'process.env.CNB_CLI_CMD': process.env.CNB_CLI_CMD || 'cnb',
                 },
             ],
         ],

package/built/skills-tools/generate-skill-md.js

@@ -8,8 +8,6 @@ const debug_1 = __importDefault(require("debug"));
 const path_1 = __importDefault(require("path"));
 const fs_1 = __importDefault(require("fs"));
 const get_skills_codegen_target_1 = require("../utils/get-skills-codegen-target");
-const get_config_1 = require("../utils/get-config");
-const skills_1 = require("../constants/skills");
 const render_quick_commands_md_1 = require("./render-quick-commands-md");
 const logger = (0, debug_1.default)('csg:skills');
 async function generateSkillMd(
@@ -24,17 +22,13 @@ _requestMap, quickCommandsByModule) {
     const SKILL_MD = path_1.default.join(skillMDTempateDir, 'SKILL.md');
     const targetSkillMD = path_1.default.join(skillMDDir, 'SKILL.md');
     const skillMDContent = fs_1.default.readFileSync(SKILL_MD, 'utf8');
-    const config = (0, get_config_1.getConfig)();
-    const cliCmd = config.skillsDev ? skills_1.CNB_CLI_DEV_CMD : skills_1.CNB_CLI_PROD_CMD;
-    // 仅替换两类占位符：
-    // - <$QUICK_COMMANDS$>：配置驱动的快捷命令清单（标题/使用约定写在模板里即可）
-    // - <$CNB_CLI_CMD$>   ：CLI 命令前缀
-    const quickCommandsMd = (0, render_quick_commands_md_1.renderQuickCommandsMarkdown)(quickCommandsByModule, cliCmd);
+    // 仅替换 <$QUICK_COMMANDS$>：配置驱动的快捷命令清单。
+    // 模板正文、渲染出的快捷命令前缀都写死 `cnb`（skillsDev 只影响产物运行时的 CNB_CLI_CMD），
+    // 不在这里传 CLI 前缀，避免同一份 SKILL.md 出现两种命令写法。
+    const quickCommandsMd = (0, render_quick_commands_md_1.renderQuickCommandsMarkdown)(quickCommandsByModule);
     const total = Object.values(quickCommandsByModule).reduce((n, arr) => n + arr.length, 0);
     logger(`quick commands rendered: ${Object.keys(quickCommandsByModule).length} modules, ${total} commands`);
-    const newSkillMDContent = skillMDContent
-        .replaceAll('<$QUICK_COMMANDS$>', quickCommandsMd)
-        .replaceAll('<$CNB_CLI_CMD$>', cliCmd);
+    const newSkillMDContent = skillMDContent.replaceAll('<$QUICK_COMMANDS$>', quickCommandsMd);
     fs_1.default.writeFileSync(targetSkillMD, newSkillMDContent);
     logger(`generate ${SKILL_MD} to ${targetSkillMD}`);
 }

package/built/skills-tools/render-quick-commands-md.js

@@ -9,10 +9,11 @@ exports.renderQuickCommandsMarkdown = renderQuickCommandsMarkdown;
  *
  * 渲染规则：
  * - 每个模块先输出 `<moduleName>:` 标题行
- * - 每条命令渲染为 ``- `<cliCmd> <module> <shortName>[ <dataTip>]` — <description>`` 的列表项
+ * - 每条命令渲染为 ``- `cnb <module> <shortName>[ <dataTip>]` — <description>`` 的列表项。
+ *   前缀固定为 `cnb`，与模板正文写法一致，不跟随 skillsDev 切换
  * - 没有任何模块/命令时返回空字符串（让模板里的占位符消失）
  */
-function renderQuickCommandsMarkdown(byModule, cliCmd) {
+function renderQuickCommandsMarkdown(byModule) {
     const moduleNames = Object.keys(byModule);
     if (!moduleNames.length)
         return '';
@@ -25,7 +26,7 @@ function renderQuickCommandsMarkdown(byModule, cliCmd) {
         lines.push(`${moduleName}:`);
         for (const cmd of items) {
             const dataPart = cmd.dataTip ? ` ${cmd.dataTip}` : '';
-            const cmdLine = `${cliCmd} ${moduleName} ${cmd.shortName}${dataPart}`;
+            const cmdLine = `cnb ${moduleName} ${cmd.shortName}${dataPart}`;
             lines.push(`- \`${cmdLine}\` — ${cmd.description}`);
         }
         lines.push('');

package/built/utils/get-config.js

@@ -9,7 +9,6 @@ const fs_1 = __importDefault(require("fs"));
 const lodash_1 = __importDefault(require("lodash"));
 const constants_1 = require("../constants");
 const is_skills_1 = require("./is-skills");
-const skills_1 = require("../constants/skills");
 const defaultConfig = {
     printeReduxAction: true,
     requestDependencyMap: {
@@ -61,8 +60,8 @@ function getConfig() {
         : {});
     if (globalConfig && !process.env.CNB_CLI_CMD) {
         process.env.CNB_CLI_CMD = globalConfig.skillsDev
-            ? skills_1.CNB_CLI_DEV_CMD
-            : skills_1.CNB_CLI_PROD_CMD;
+            ? 'node scripts/core/index.js'
+            : 'cnb';
     }
     return globalConfig;
 }

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

package/skills-template/SKILL.md

@@ -11,26 +11,37 @@ description: CNB 平台交互命令，支持代码仓库、Issue、PR、CI、制
 
 ## 使用约定
 
-- **编号自识别**：Issue/PR 编号自动从环境变量识别，无需额外传递。
-- **可显式覆盖**：`--repo <组织/仓库>` 与 `--number <编号>` 均为可选参数。
-  不传时沿用环境变量（即当前仓库的当前 Issue/PR）；传入时覆盖环境变量，
-  可用于跨仓库或跨编号场景，例如 `cnb pulls get --repo x/y --number 97`、
-  `cnb issues get --number 68`。两个参数相互独立，可只传其中一个。
 - **默认摘要省流**：默认精简输出，加 `--verbose` 输出完整数据。
 - **多行用单引号**：bash 参数为多行文本时用单引号，降低命令注入风险。
-- **提及不召唤**：评论中直接 @npc 会召唤 npc；仅提及不召唤时，用反引号包裹 `@npc`。
-  NPC 的触发条件见 `cnb-npc-search`，单独 `@仓库名` 不会触发。
 
-## 取正文里的图片与附件
+## issue 访问
 
-评论正文里的图片、附件 URL，一律用 `get-imgs` / `get-files` 取回，URL 从 `list-comments` 原文取：
+编号同 `--repo` 一样默认从环境变量识别，无需传递；用 `--repo <组织/仓库>`、`--number <编号>`
+可覆盖，例：`cnb issues get --number 68`。
 
-- **参数取 URL 的尾段，且必须含数字 ID 段**：图片取 `/imgs/<issues|pulls>/` 之后的**完整尾段**，
-  附件取 `/files/<issues|pulls>/` 之后的完整尾段；尾段首位的数字 ID 段不能省，漏掉会返回 `404 Resource not found`：
-  - 原文 `.../-/imgs/issues/2101868753481326592/Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png`
-  - 正确 `2101868753481326592/Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png` ✅
-  - 漏 ID `Lma2nfHjnchalMtHZcq4OA/6d7ab033-xxxx.png` ❌ `404`
-- **CLI 内置鉴权**：手写 `Authorization: Bearer` 或 `token` 都不被识别，只会拿到 400
+- `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 附件路径` — 取正文里的附件
+
+`--img-path` / `--file-path` 与 issue 同规则：取 URL 的**完整尾段**，首位数字 ID 段不能省，
+漏掉返回 `404`（原文形如 `.../-/imgs/pulls/<数字 ID>/<key>/<file>`）。
+鉴权同样由 `cnb` 命令内置的登录态提供，无需自己传鉴权头。
 
 ## 常用链接
 
@@ -40,6 +51,6 @@ description: CNB 平台交互命令，支持代码仓库、Issue、PR、CI、制
 
 ## 更多 API
 
-1. `<$CNB_CLI_CMD$> --help` 查看所有模块
-2. `<$CNB_CLI_CMD$> <module> --help` 查看模块下的工具列表
-3. `<$CNB_CLI_CMD$> <module> <tool> --help` 查看工具参数
+1. `cnb --help` 查看所有模块
+2. `cnb <module> --help` 查看模块下的工具列表
+3. `cnb <module> <tool> --help` 查看工具参数

package/built/constants/skills.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.CNB_CLI_PROD_CMD = exports.CNB_CLI_DEV_CMD = void 0;
-exports.CNB_CLI_DEV_CMD = 'node scripts/core/index.js';
-exports.CNB_CLI_PROD_CMD = 'cnb';