@d5render/cli 0.1.12 → 0.1.16

package/.skills/review/SKILL.md

@@ -1,9 +1,11 @@
 ---
 name: code-review
-description: code review suggestions
+description: When the task is code-review, please follow this document to proceed the work.
 ---
 
 1. read file [review.instructions.md](./review.instructions.md).
 2. find all Markdown resumes within the project and their basic understanding of the project.
-3. based on the understanding of review to supplemented.
-4. run...
+3. integrate your understanding of code-review to process task.
+4. pay attention to the issue tracking description in the context, and try to use relevant tools to obtain more context.
+5. output your plan for the task，and when task run, output the intermediate process of the task.
+6. output all the context read, especially the introductory document.

package/.skills/review/version

@@ -1 +1 @@
-0.0.0
+0.0.1

package/bin/copilot.js

@@ -18805,7 +18805,7 @@ var StdioServerTransport = class {
 //#region copilot/server/config.ts
 const name = "d5_mcp_review_builtin";
 const report = "report";
-const getHash = "getHash";
+const getHash = "hash";
 const noMandatory = "dont't call under non-mandatory conditions";
 const file = "./copilot.js";
 const serveFile = join(dirname(fileURLToPath(import.meta.url)), file);
@@ -18836,6 +18836,7 @@ const tools = [
 		args: [serveFile, `--customizenv=${JSON.stringify(envUsed)}`],
 		tools: ["*"]
 	} } }),
+	"--allow-all-paths",
 	"--allow-all-tools",
 	"--deny-tool",
 	"write",
@@ -19056,33 +19057,11 @@ const reportSchema = {
 	}).describe("Related JIRA issues")).optional().describe("List of related JIRA issues"),
 	risks: array(string().describe(one).describe("A short summary of the risk")).optional().describe("After summarizing all the bugs, identify the main risks that could potentially impact the business."),
 	suggestions: array(object({
-		details: array(string().describe(one)).describe("Improvement suggestion"),
+		details: string().describe("Suggestions with the same meaning should be stated in one sentence; please do not increase the array length."),
 		example: object(code).optional().describe("Pass in when you feel it is necessary to provide code.")
-	})).optional().describe("After summarizing all the issues, other code snippets that deserve correction.")
+	})).optional().describe("After gaining a comprehensive understanding of the files involved in the commits, propose suggestions for code corrections beyond the diff.")
 };
 
-//#endregion
-//#region packages/message/dingding.ts
-function send(title, text) {
-	if (!envUsed.DINGTALK_WEBHOOK) return Promise.reject(/* @__PURE__ */ new Error("non DINGTALK_WEBHOOK"));
-	const urls = envUsed.DINGTALK_WEBHOOK.split(",");
-	return Promise.all(urls.map((url) => fetch(url, {
-		method: "POST",
-		headers: { "Content-Type": "application/json" },
-		body: JSON.stringify({
-			msgtype: "markdown",
-			markdown: {
-				title,
-				text
-			},
-			at: {
-				atMobiles: ["17856104313"],
-				isAtAll: false
-			}
-		})
-	})));
-}
-
 //#endregion
 //#region packages/message/report.ts
 const lastComment = { value: {} };
@@ -19169,14 +19148,14 @@ function distReports(input) {
 		gitlabReportMessage += "\n\n### 📈 其他建议\n\n-----\n";
 		let withNoFile = "", withFile = "";
 		for (const suggestion of suggestions) {
-			const { details = [], example } = suggestion;
+			const { details = "", example } = suggestion;
 			const { file: file$1 = "", line = "", codeExample = "" } = example || {};
 			if (file$1) {
 				withFile += `\n-----\n\n#### 文件： ${file$1}`;
 				if (line) withFile += `\n- 行数：${line}`;
-				withFile += `\n- 详情：\n   - ${details.join("\n   - ")}`;
+				withFile += `\n- 详情：\n   - ${details}`;
 				if (codeExample) withFile += `\n${toCode(codeExample)}\n`;
-			} else withNoFile += `\n   - ${details.join("\n   - ")}`;
+			} else withNoFile += `\n   - ${details}`;
 		}
 		gitlabReportMessage += withNoFile + withFile;
 	}
@@ -19240,8 +19219,8 @@ function distReports(input) {
 		})
 	}).then((res) => res.json()).then((res) => {
 		lastComment.value = res;
-	})] : [() => Promise.reject(/* @__PURE__ */ new Error("⚠️ 当前为 push 事件，无合并请求，无法在 GitLab 中发布报告。报告内容已生成但未发布。"))];
-	dingdingReport.report = DINGTALK_WEBHOOK ? [() => send(title, dingdingReportMessage).then((response) => {
+	})] : [() => Promise.reject(/* @__PURE__ */ new Error("⚠️ 报告内容已生成但未发布。"))];
+	dingdingReport.report = DINGTALK_WEBHOOK ? [() => sendding(title, dingdingReportMessage).then((response) => {
 		if (!response.ok) throw new Error(response.statusText);
 		return response.json();
 	}).then((res) => {
@@ -19258,12 +19237,32 @@ async function runReport(input) {
 		error: error$1.message || "未知错误"
 	})));
 	const error = results.find((v) => v.error);
-	return error ? error.error : "✅ GitLab MCP Report 执行完成，无问题";
+	return error ? `error: ${error.title}\n${error.error}` : "✅ GitLab MCP Report 执行完成，无问题";
 }
 function toCode(code$1) {
 	const ct = code$1.trim();
 	return ct.startsWith("```") && ct.endsWith("```") ? code$1 : `\`\`\`\n${code$1}\n\`\`\``;
 }
+async function sendding(title, text) {
+	if (!envUsed.DINGTALK_WEBHOOK) throw new Error("non DINGTALK_WEBHOOK");
+	let res = new Response();
+	for (const url of envUsed.DINGTALK_WEBHOOK.split(",")) res = await fetch(url, {
+		method: "POST",
+		headers: { "Content-Type": "application/json" },
+		body: JSON.stringify({
+			msgtype: "markdown",
+			markdown: {
+				title,
+				text
+			},
+			at: {
+				atMobiles: ["17856104313"],
+				isAtAll: false
+			}
+		})
+	});
+	return res;
+}
 
 //#endregion
 //#region packages/message/index.ts
@@ -19271,12 +19270,10 @@ function install(server$1) {
 	server$1.registerTool(report, {
 		description: noMandatory,
 		inputSchema: reportSchema
-	}, async (input) => {
-		return { content: [{
-			type: "text",
-			text: await runReport(distReports(input))
-		}] };
-	});
+	}, async (input) => ({ content: [{
+		type: "text",
+		text: await runReport(distReports(input))
+	}] }));
 }
 
 //#endregion

package/bin/d5cli

@@ -7,13 +7,13 @@ import { fileURLToPath } from "node:url";
 
 //#region package.json
 var name$1 = "@d5render/cli";
-var version = "0.1.12";
+var version = "0.1.16";
 
 //#endregion
 //#region copilot/server/config.ts
 const name = "d5_mcp_review_builtin";
 const report = "report";
-const getHash = "getHash";
+const getHash = "hash";
 const file = "./copilot.js";
 const serveFile = join(dirname(fileURLToPath(import.meta.url)), file);
 const envJson = buildEnv();
@@ -43,6 +43,7 @@ const tools = [
 		args: [serveFile, `--customizenv=${JSON.stringify(envUsed)}`],
 		tools: ["*"]
 	} } }),
+	"--allow-all-paths",
 	"--allow-all-tools",
 	"--deny-tool",
 	"write",
@@ -193,8 +194,8 @@ function need() {
 
 //#endregion
 //#region copilot/bin/index.ts
-const prompt = `call the mcp tool '${name}-${getHash}' to load review commits, if the task encounters an error, throw that, otherwise, you can decide for yourself which tools to use to enrich context and determine whether a specific commit needs to be reviewed.
-Then, use chinese as default language to organize the report into JSON format and call the mcp tool '${name}-${report}', please ignore error encountered when calling this and report title should not contain words such as MCP or "中文".`;
+const prompt = `Load skills, then call the mcp tool '${name}-${getHash}' to load code-review commits, if the task encounters an error, throw that.
+Otherwise, use chinese as default language to call the mcp tool '${name}-${report}'`;
 deploy();
 if (need()) {
 	install();
@@ -207,22 +208,28 @@ if (need()) {
 		prompt
 	], {
 		cwd: env.CI_PROJECT_DIR,
+		stdio: [
+			"inherit",
+			"pipe",
+			"pipe"
+		],
 		...platform === "win32" && { windowsHide: true }
 	});
-	copilot.stdout.on("data", (data) => console.log(data.toString()));
-	copilot.stderr.on("data", (data) => console.error(data.toString()));
+	copilot.stdout.on("data", (chunk) => console.log(String(chunk)));
+	copilot.stderr.on("data", (chunk) => console.error(String(chunk)));
 	copilot.on("close", (code) => exit(code));
 } else console.log("重复提交，进程跳过");
 function findCopilopt() {
 	let copilot = execSync("npm list @github/copilot -g -p").toString().trim();
-	if (!copilot) try {
-		const pathEnv = env.PATH || env.Path || "";
-		const pathSeparator = platform === "win32" ? ";" : ":";
-		const npm = pathEnv.split(pathSeparator).find((p) => p.includes("npm"));
-		if (npm) copilot = join(npm, "node_modules", "@github", "copilot");
-	} catch {}
+	if (!copilot) {
+		const first = platform === "win32" ? win : linux;
+		const second = platform === "win32" ? linux : win;
+		copilot = first();
+		if (!copilot) copilot = second();
+		if (!copilot) throw new Error("没找到安装的包");
+	}
 	const pkg = join(copilot, "package.json");
-	if (!existsSync(pkg)) throw new Error("non copilot package found");
+	if (!existsSync(pkg)) throw new Error("安装的包找不到正确版本 " + pkg);
 	const copilotPackage = JSON.parse(readFileSync(pkg, "utf8"));
 	const binPath = typeof copilotPackage.bin === "string" ? copilotPackage.bin : copilotPackage.bin?.copilot || copilotPackage.bin?.["@github/copilot"];
 	if (!binPath) throw new Error("non copilot executable found");
@@ -236,5 +243,23 @@ copilot:
  path: ${copilotPath}`);
 	return copilotPath;
 }
+function win() {
+	const pathEnv = env.PATH || env.Path || "";
+	const pathSeparator = platform === "win32" ? ";" : ":";
+	const npm = pathEnv.split(pathSeparator).find((p) => p.includes("npm"));
+	if (npm) return join(npm, "node_modules", "@github", "copilot");
+	return "";
+}
+function linux() {
+	let cached = env.NVM_BIN;
+	if (!cached) {
+		const pathEnv = env.PATH || env.Path || "";
+		const pathSeparator = platform === "win32" ? ";" : ":";
+		const npm = pathEnv.split(pathSeparator).find((p) => p.includes(".nvm"));
+		if (npm) cached = npm;
+	}
+	if (cached) return join(cached, "..", "lib", "node_modules", "@github", "copilot");
+	return "";
+}
 
 //#endregion

package/package.json

@@ -4,7 +4,7 @@
   "license": "MIT",
   "author": "jasirou",
   "main": "./bin/d5cli",
-  "version": "0.1.12",
+  "version": "0.1.16",
   "devDependencies": {
     "@modelcontextprotocol/sdk": "^1.25.1",
     "@types/node": "^25.0.3",