@cnbcool/cnb-api-generate 2.18.18 → 2.18.20

package/client/lib/format-params.ts

@@ -6,6 +6,7 @@ import { restoreOriginalKeys } from '../utils/restore-original-keys';
 import { matchArrayObjectField } from '../utils/match-array-object-field';
 import { isNestedObjectSchema } from '../utils/is-nested-object-schema';
 import { isMapField, getMapValueType } from '../utils/is-map-field';
+import { normalizeAssetPath } from '../utils/normalize-asset-path';
 
 /**
  * 获取 tool 的参数定义（用于自动分发 --key value 到 path/query）
@@ -16,6 +17,14 @@ export function getToolParamDefs(moduleName: string, toolName: string) {
   return toolHelp.help?.parameter || {};
 }
 
+/**
+ * 取 tool 的接口路径模板（如 `/{repo}/-/imgs/issues/{img_path}`），
+ * 供 img_path / file_path 归一化判断「接口路径已自带哪些前缀」。
+ */
+function getToolPath(moduleName: string, toolName: string): string | undefined {
+  return helpData.modulesHelp?.[moduleName]?.[toolName]?.path;
+}
+
 /**
  * 将 body 字段值按 schema 类型写入 target。
  *   - array 值：原样保留
@@ -162,6 +171,7 @@ export function formatParams(
     params.module as string,
     params.tool as string,
   );
+  const toolPath = getToolPath(params.module as string, params.tool as string);
 
   if (paramDefs) {
     const pathDef = paramDefs.path || {};
@@ -278,7 +288,13 @@ export function formatParams(
       if (pathDef[key]) {
         // 归入 path
         if (!formatted.path) formatted.path = {};
-        formatted.path[key] = value;
+        // img_path / file_path 支持直接粘贴 URL 尾段（可带 issues|pulls 前缀），
+        // 按接口路径已自带的前缀剥离，避免拼出重复段导致 404；
+        // 不带前缀的接口（Assets 分类）需要完整尾段，保持原样。
+        formatted.path[key] =
+          key === 'img_path' || key === 'file_path'
+            ? normalizeAssetPath(value as string, toolPath)
+            : value;
         // path 与 query 同名时，query 参数通过 q- 前缀传入（见下方 q- 分支）；
         // 但若调用方直接用 --<name> 传了值（旧习惯/兼容），且 queryDef 里也有同名项，
         // 则同时回填到 query，保证 URL 查询参数也被带上。

package/client/utils/normalize-asset-path.ts

@@ -0,0 +1,54 @@
+/**
+ * 归一化图片 / 附件的路径参数（img_path / file_path）。
+ *
+ * 背景：正文里的下载地址形如
+ *   .../-/imgs/issues/<数字 ID>/<key>/<文件名>
+ *   .../-/files/pulls/<数字 ID>/<key>/<文件名>
+ *
+ * 用户更自然的做法是直接复制整段 URL 尾段（含 `issues|pulls/` 前缀）粘贴。
+ * 但接口路径本身已带前缀的接口（如 `/{repo}/-/imgs/issues/{img_path}`）
+ * 若再原样拼接，就会拼出 `.../-/imgs/issues/issues/...` 而 404。
+ *
+ * 因此按「接口路径已自带哪些前缀」剥离：
+ * - `imgs|files` 段：只要接口路径含该段就剥（含 Assets 的 `/{repo}/-/imgs/{imgPath}`）
+ * - `issues|pulls` 段：仅当接口路径确实含该段才剥
+ *
+ * 例（toolPath = `/{repo}/-/imgs/issues/{img_path}`）：
+ *   `issues/66/abc/x.png`       → `66/abc/x.png`
+ *   `imgs/issues/66/abc/x.png`  → `66/abc/x.png`
+ *   `66/abc/x.png`              → `66/abc/x.png`
+ * 例（toolPath = `/{repo}/-/imgs/{imgPath}`）：
+ *   `issues/66/abc/x.png`      → 原样（未剥 scope）
+ *   `imgs/issues/66/abc/x.png` → `issues/66/abc/x.png`（剥掉 kind）
+ */
+export function normalizeAssetPath(value: string, toolPath?: string): string {
+  if (typeof value !== 'string') return value;
+
+  const path = toolPath || '';
+  let stripped = value;
+
+  /** 若 stripped 以 `<prefix>/` 开头则去掉该段，否则原样返回（大小写不敏感）。 */
+  const stripPrefix = (prefix: string) =>
+    stripped.toLowerCase().startsWith(`${prefix.toLowerCase()}/`)
+      ? stripped.slice(prefix.length + 1)
+      : stripped;
+
+  // 按「接口路径自带的前缀」逐段剥；顺序固定为 imgs|files → issues|pulls，
+  // 与完整 URL 尾段的出现顺序一致，兼容「只带一段」与「两段都带」两种写法。
+  const lowerPath = path.toLowerCase();
+
+  if (lowerPath.includes('/imgs/')) stripped = stripPrefix('imgs');
+  else if (lowerPath.includes('/files/')) stripped = stripPrefix('files');
+
+  // issues|pulls 段必须确实出现在接口路径里才剥，避免把 Assets 接口要的
+  // 完整尾段（`issues/...`）误剥。
+  if (lowerPath.includes('/imgs/issues') || lowerPath.includes('/files/issues')) {
+    stripped = stripPrefix('issues');
+  } else if (lowerPath.includes('/imgs/pulls') || lowerPath.includes('/files/pulls')) {
+    stripped = stripPrefix('pulls');
+  }
+
+  // 全部剥掉会得到空串（如用户误传 `issues/`），此时保留原值，交给服务端报错更直观。
+  if (stripped.length === 0) return value;
+  return stripped;
+}

package/package.json

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

package/quick-commands.json

@@ -12,7 +12,9 @@
     { "shortName": "upload-file",         "realTool": "post-issue-comment-file-asset-upload-url", "description": "上传 issue 评论文件",  "upload": true, "kind": "file",  "hideInSkill": true, "dataTip": "--file 文件路径" },
     { "shortName": "upload-image",        "realTool": "post-issue-comment-image-asset-upload-url","description": "上传 issue 评论图片",  "upload": true, "kind": "image", "hideInSkill": true, "dataTip": "--file 图片路径" },
     { "shortName": "create-upload-file",  "realTool": "post-asset-group",                         "description": "创建 issue 时上传文件", "upload": true, "kind": "file",  "hideInSkill": true, "dataTip": "--file 文件路径" },
-    { "shortName": "create-upload-image", "realTool": "post-asset-group",                         "description": "创建 issue 时上传图片", "upload": true, "kind": "image", "hideInSkill": true, "dataTip": "--file 图片路径" }
+    { "shortName": "create-upload-image", "realTool": "post-asset-group",                         "description": "创建 issue 时上传图片", "upload": true, "kind": "image", "hideInSkill": true, "dataTip": "--file 图片路径" },
+    { "shortName": "get-imgs",            "realTool": "get-issue-imgs",                           "description": "获取 issue 图片",       "repoOnly": true, "dataTip": "--img-path 图片路径" },
+    { "shortName": "get-files",           "realTool": "get-issue-files",                          "description": "获取 issue 附件",       "repoOnly": true, "dataTip": "--file-path 附件路径" }
   ],
   "pulls": [
     { "shortName": "get",            "realTool": "get-pull",                  "description": "获取详情" },
@@ -29,7 +31,9 @@
     { "shortName": "upload-file",    "realTool": "upload-files",              "description": "上传文件",      "repoOnly": true, "upload": true, "kind": "file",  "hideInSkill": true, "dataTip": "--file 文件路径" },
     { "shortName": "upload-image",   "realTool": "upload-imgs",               "description": "上传图片",      "repoOnly": true, "upload": true, "kind": "image", "hideInSkill": true, "dataTip": "--file 图片路径" },
     { "shortName": "get-ci-logs",    "realTool": "__get-ci-logs__",           "description": "获取 CI 失败日志", "repoOnly": true, "custom": true, "dataTip": "--sn 构建号（可选）" },
-    { "shortName": "get-ci-timing",  "realTool": "__get-ci-timing__",         "description": "分析 CI 耗时瓶颈", "repoOnly": true, "custom": true, "dataTip": "--sn 构建号（可选）" }
+    { "shortName": "get-ci-timing",  "realTool": "__get-ci-timing__",         "description": "分析 CI 耗时瓶颈", "repoOnly": true, "custom": true, "dataTip": "--sn 构建号（可选）" },
+    { "shortName": "get-imgs",       "realTool": "get-pr-imgs",               "description": "获取 PR 图片",     "repoOnly": true, "dataTip": "--img-path 图片路径" },
+    { "shortName": "get-files",      "realTool": "get-pr-files",              "description": "获取 PR 附件",     "repoOnly": true, "dataTip": "--file-path 附件路径" }
   ],
   "skills": [
     { "shortName": "list", "realTool": "__skills-list__", "description": "列出本地 skills", "custom": true, "dataTip": "-p -a codebuddy --json" }

package/skills-template/SKILL.md

@@ -30,7 +30,7 @@ description: CNB 平台交互命令，支持代码仓库、Issue、PR、CI、制
 - 图片 `.../-/imgs/<issues|pulls>/<数字 ID>/<key>/<文件名>`
 - 附件 `.../-/files/<issues|pulls>/<数字 ID>/<key>/<文件名>`
 
-`图片路径/附件路径`=`<数字 ID>/<key>/<文件名>`，即 `参数` 取 `<issues|pulls>/` 之后的**完整尾段**。
+`图片路径/附件路径`=`<issues|pulls>/<数字 ID>/<key>/<文件名>`，即 `参数` 取 `<issues|pulls>/` 开头的**完整尾段**。
 
 ## 常用链接