@d5render/cli 0.1.16 → 0.1.22

package/.skills/review/SKILL.md

@@ -6,6 +6,5 @@ description: When the task is code-review, please follow this document to procee
 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. 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.
+4. the merge request is only for change reference and is not the main objective of the review; therefore, it does not require in-depth analysis.
+5. pay attention to the issue tracking description in the context, and try to use relevant tools to obtain more context.

package/.skills/review/version

@@ -1 +1 @@
-0.0.1
+0.0.2

package/bin/copilot.js

@@ -18918,10 +18918,6 @@ const getCommits = () => {
 		text: `latest commit`
 	}] };
 };
-const canIReview = ({ B }) => ({ content: [{
-	type: "text",
-	text: B.startsWith("Merge") ? "no" : "yes"
-}] });
 const getMergeInfomation = () => {
 	const { CI_MERGE_REQUEST_DESCRIPTION, CI_MERGE_REQUEST_TITLE } = envUsed;
 	if (CI_MERGE_REQUEST_DESCRIPTION || CI_MERGE_REQUEST_TITLE) return { content: [{
@@ -18937,22 +18933,10 @@ const getMergeInfomation = () => {
 	};
 };
 
-//#endregion
-//#region packages/gitlab/schema.ts
-const canireview = "can-i-review";
-const canireviewSchema = {
-	hash: string().describe("commit hash"),
-	B: string().describe("the commit directions logged by `git log --format=%B -n 1 <hash>`")
-};
-
 //#endregion
 //#region packages/gitlab/index.ts
 function install$2(server$1) {
 	server$1.registerTool(getHash, { description: noMandatory }, getCommits);
-	server$1.registerTool(canireview, {
-		inputSchema: canireviewSchema,
-		description: "Determine whether the commit with the given hash needs to be reviewed based on its commit message."
-	}, canIReview);
 	server$1.registerTool("getMergeDescription", {
 		title: "merge pipeline description",
 		description: "When review task from `merge pipeline`, get the merge request description."
@@ -19043,9 +19027,9 @@ const reportSchema = {
 	summary: array(string().describe(one)).optional().describe("Regarding the description of the current review, please summarize the main logic of the submission as concisely as possible."),
 	issues: array(object({
 		severity: string().optional().describe("Bug severity levels, such as CRITICAL, HIGH, MEDIUM, LOW."),
-		commitAuthor: string().optional().describe("The commit author associated with the issue"),
+		commitTitle: string().describe("The commit title associated with the issue"),
+		commitAuthor: string().describe("The commit latest author associated with the issue"),
 		commitSha: string().optional().describe("The commit SHA associated with the issue"),
-		commitTitle: string().optional().describe("The commit title associated with the issue"),
 		title: string().optional().describe("A short title for the issue"),
 		details: array(string()).optional().describe("Details about the issue"),
 		suggestions: array(string()).optional().describe("Improvement suggestion"),
@@ -19062,6 +19046,29 @@ const reportSchema = {
 	})).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/sendding.ts
+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/report.ts
 const lastComment = { value: {} };
@@ -19102,23 +19109,6 @@ function distReports(input) {
 	const mergeURL = pipelineMerge();
 	const linkBase = visitCommit();
 	const commitsURL = commits();
-	for (const issue$1 of issues) {
-		const { file: file$1 = "NON", commitSha = "", severity = "NON", commitSha: NCommitSha = "NON" } = issue$1;
-		const fileList = fileMap.get(file$1) || [], issueList = severityMap.get(severity) || [];
-		fileList.push(issue$1);
-		fileMap.set(file$1, fileList);
-		issueList.push(issue$1);
-		severityMap.set(severity, issueList);
-		if (commitSha) commitsSet.add(commitSha);
-		commitsMap.set(NCommitSha, issue$1);
-		distCommitComments(issue$1);
-	}
-	let gitlabReportMessage = `## 📖 ${title}\n`;
-	if (summary && summary.length > 0) gitlabReportMessage += `> ${summary.join("  \n> ")}\n\n`;
-	if (risks.length > 0) gitlabReportMessage += "\n**📋 主要风险**\n - " + risks.join("\n - ");
-	gitlabReportMessage += `\n### 🐛 问题列表`;
-	const cms = [...commitsSet].map((v) => `[${v.slice(0, 8)}](${linkBase}/${v})`);
-	if (cms.length > 0) gitlabReportMessage += `\n-----\n> 含问题的commits：${cms.join(",")}\n\n`;
 	function distCommitComments({ file: file$1, commitSha, severity = "NON", line = "", details = [], suggestions: suggestions$1 = [], codeExample = "" }) {
 		if (commitSha && file$1) {
 			const lines = line.split(",").map((line$1) => {
@@ -19134,14 +19124,33 @@ function distReports(input) {
 			});
 		}
 	}
-	for (const [file$1, fileIssues] of fileMap) {
-		gitlabReportMessage += `\n-----\n\n#### 文件： ${file$1}`;
-		for (const issue$1 of fileIssues) {
-			const { severity = "NON", line = "NON", title: issueTitle = "NON", details = [], suggestions: issueSuggestions = [], codeExample = "" } = issue$1;
-			gitlabReportMessage += `\n\n**${severity}** | ${line} | ${issueTitle}`;
-			gitlabReportMessage += `\n- 详情：\n   - ${details.join("\n   - ")}`;
-			gitlabReportMessage += `\n- 建议：\n   - ${issueSuggestions.join("\n   - ")}`;
-			if (codeExample) gitlabReportMessage += `\n${toCode(codeExample)}\n`;
+	for (const issue$1 of issues) {
+		const { file: file$1 = "NON", commitSha = "", severity = "NON", commitSha: NCommitSha = "NON" } = issue$1;
+		const fileList = fileMap.get(file$1) || [], issueList = severityMap.get(severity) || [];
+		fileList.push(issue$1);
+		fileMap.set(file$1, fileList);
+		issueList.push(issue$1);
+		severityMap.set(severity, issueList);
+		if (commitSha) commitsSet.add(commitSha);
+		commitsMap.set(NCommitSha, issue$1);
+		distCommitComments(issue$1);
+	}
+	let gitlabReportMessage = `## 📖 ${title}\n`;
+	if (summary && summary.length > 0) gitlabReportMessage += `> ${summary.join("  \n> ")}\n\n`;
+	if (risks.length > 0) gitlabReportMessage += "\n**📋 主要风险**\n - " + risks.join("\n - ");
+	if (fileMap.size > 0 || commitsSet.size > 0) {
+		gitlabReportMessage += `\n### 🐛 问题列表`;
+		const cms = [...commitsSet].map((v) => `[${v.slice(0, 8)}](${linkBase}/${v})`);
+		if (cms.length > 0) gitlabReportMessage += `\n-----\n> 含问题的commits：${cms.join(",")}\n\n`;
+		for (const [file$1, fileIssues] of fileMap) {
+			gitlabReportMessage += `\n-----\n\n#### 文件： ${file$1}`;
+			for (const issue$1 of fileIssues) {
+				const { severity = "NON", line = "NON", title: issueTitle = "NON", details = [], suggestions: issueSuggestions = [], codeExample = "" } = issue$1;
+				gitlabReportMessage += `\n\n**${severity}** | ${line} | ${issueTitle}`;
+				gitlabReportMessage += `\n- 详情：\n   - ${details.join("\n   - ")}`;
+				gitlabReportMessage += `\n- 建议：\n   - ${issueSuggestions.join("\n   - ")}`;
+				if (codeExample) gitlabReportMessage += `\n${toCode(codeExample)}\n`;
+			}
 		}
 	}
 	if (suggestions.length > 0) {
@@ -19243,26 +19252,6 @@ 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

package/bin/d5cli

@@ -1,13 +1,13 @@
 #!/usr/bin/env node
 import { execSync, spawn } from "node:child_process";
-import { copyFileSync, existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
+import { copyFileSync, existsSync, mkdirSync, readFileSync, readdirSync, rmSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { argv, env, exit, platform } from "node:process";
 import { fileURLToPath } from "node:url";
 
 //#region package.json
 var name$1 = "@d5render/cli";
-var version = "0.1.16";
+var version = "0.1.22";
 
 //#endregion
 //#region copilot/server/config.ts
@@ -60,6 +60,29 @@ function buildEnv() {
 	return envJson$1;
 }
 
+//#endregion
+//#region packages/message/sendding.ts
+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 copilot/bin/install.ts
 function install() {
@@ -142,10 +165,13 @@ const commits = () => {
 
 //#endregion
 //#region copilot/bin/utils.ts
-const root = env.HOME ?? env.HOMEPATH ?? env.LOCALAPPDATA;
+const HOME = env.HOME ?? env.HOMEPATH;
+const root = HOME ?? env.LOCALAPPDATA;
 if (!root) throw new Error("cannot find home directory");
 const cwd = join(dirname(fileURLToPath(import.meta.url)), "../");
 function deploy() {
+	const config = join(HOME, ".copilot/config.json");
+	if (existsSync(config)) rmSync(config);
 	const dir = join(root, ".copilot/skills/codereview");
 	if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
 	const skillRoot = join(cwd, ".skills/review");
@@ -194,31 +220,42 @@ function need() {
 
 //#endregion
 //#region copilot/bin/index.ts
-const prompt = `Load skills, then call the mcp tool '${name}-${getHash}' to load code-review commits, if the task encounters an error, throw that.
+try {
+	main();
+} catch (error) {
+	sendding("CRITICAL", "未知错误，请自行检查日志");
+	throw error;
+}
+function main() {
+	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();
-	const copilot = spawn("node", [
-		findCopilopt(),
-		...tools,
-		"--stream",
-		"off",
-		"-p",
-		prompt
-	], {
-		cwd: env.CI_PROJECT_DIR,
-		stdio: [
-			"inherit",
-			"pipe",
-			"pipe"
-		],
-		...platform === "win32" && { windowsHide: true }
-	});
-	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("重复提交，进程跳过");
+	deploy();
+	if (need()) {
+		install();
+		const copilot = spawn("node", [
+			findCopilopt(),
+			...tools,
+			"--stream",
+			"off",
+			"-p",
+			prompt
+		], {
+			cwd: env.CI_PROJECT_DIR,
+			stdio: [
+				"inherit",
+				"pipe",
+				"pipe"
+			],
+			...platform === "win32" && { windowsHide: true }
+		});
+		copilot.stdout.on("data", (chunk) => console.log(String(chunk)));
+		copilot.stderr.on("data", (chunk) => console.error(String(chunk)));
+		copilot.on("close", (code) => {
+			sendding("CRITICAL", "Copilot 运行过程发生错误，请自行检查日志");
+			exit(code);
+		});
+	} else console.log("重复提交，进程跳过");
+}
 function findCopilopt() {
 	let copilot = execSync("npm list @github/copilot -g -p").toString().trim();
 	if (!copilot) {

package/package.json

@@ -4,7 +4,7 @@
   "license": "MIT",
   "author": "jasirou",
   "main": "./bin/d5cli",
-  "version": "0.1.16",
+  "version": "0.1.22",
   "devDependencies": {
     "@modelcontextprotocol/sdk": "^1.25.1",
     "@types/node": "^25.0.3",
@@ -27,7 +27,7 @@
   },
   "scripts": {
     "build:copilot": "node ./setup.js build",
-    "setup": "node ./setup.js",
-    "release": "node ./setup.js publish"
+    "release": "node ./setup.js publish",
+    "setup": "node ./setup.js"
   }
 }