kcode-pi 0.1.31 → 0.1.35

package/src/rules/checker.ts

@@ -80,10 +80,10 @@ function checkLifecycleMisuse(lines: string[]): CheckResult[] {
 	for (const range of initializeRanges) {
 		forEachLineInRange(lines, range, (line, index) => {
 			if (/addItemClickListener\s*\(/.test(line)) {
-				results.push(makeResult(index, line, "addItemClickListener", "lifecycle", "error", "P0：initialize 中注册监听器，可能导致监听生命周期错误；请移到 registerListener", "p0-initialize-listener"));
+				results.push(makeResult(index, line, "addItemClickListener", "lifecycle", "error", "P0：initialize 中注册监听器，可能导致监听生命周期错误；必须移到 registerListener", "p0-initialize-listener"));
 			}
 			if (/getView\s*\(\)\s*\.\s*(setVisible|setEnable|updateView|showConfirm|showTipNotification)\s*\(/.test(line)) {
-				results.push(makeResult(index, line, "getView", "lifecycle", "error", "P0：initialize 中操作 UI 控件，控件状态可能不生效；请移到 afterBindData 等合适阶段", "p0-initialize-ui"));
+				results.push(makeResult(index, line, "getView", "lifecycle", "error", "P0：initialize 中操作 UI 控件，控件状态可能不生效；必须移到 afterBindData 等合适阶段", "p0-initialize-ui"));
 			}
 		});
 	}
@@ -91,7 +91,7 @@ function checkLifecycleMisuse(lines: string[]): CheckResult[] {
 	for (const range of [...beforeBindRanges, ...afterBindRanges]) {
 		forEachLineInRange(lines, range, (line, index) => {
 			if (/(getModel\s*\(\)\s*\.\s*)?setValue\s*\(/.test(line)) {
-				results.push(makeResult(index, line, "setValue", "lifecycle", "error", "P0：数据绑定阶段修改模型数据，可能破坏绑定流程；默认值应放到 afterCreateNewData 等阶段", "p0-binddata-setvalue"));
+				results.push(makeResult(index, line, "setValue", "lifecycle", "error", "P0：数据绑定阶段修改模型数据，可能破坏绑定流程；默认值必须放到 afterCreateNewData 等阶段", "p0-binddata-setvalue"));
 			}
 		});
 	}
@@ -99,7 +99,7 @@ function checkLifecycleMisuse(lines: string[]): CheckResult[] {
 	for (const range of afterOperationRanges) {
 		forEachLineInRange(lines, range, (line, index) => {
 			if (/getOperationResult\s*\(/.test(line)) {
-				results.push(makeResult(index, line, "getOperationResult", "lifecycle", "error", "P0：afterExecuteOperationTransaction 的参数不应调用 getOperationResult，请确认事件参数类型", "p0-after-operation-result"));
+				results.push(makeResult(index, line, "getOperationResult", "lifecycle", "error", "P0：afterExecuteOperationTransaction 的参数禁止调用 getOperationResult；必须确认事件参数类型", "p0-after-operation-result"));
 			}
 		});
 	}
@@ -114,7 +114,7 @@ function checkTransactionMisuse(lines: string[]): CheckResult[] {
 	for (const range of ranges) {
 		forEachLineInRange(lines, range, (line, index) => {
 			if (/SaveServiceHelper\s*\.\s*(save|update)\s*\(/.test(line)) {
-				results.push(makeResult(index, line, "SaveServiceHelper", "transaction", "error", "P0：操作事务钩子中独立保存会破坏事务一致性；应直接修改平台传入的数据实体", "p0-transaction-save"));
+				results.push(makeResult(index, line, "SaveServiceHelper", "transaction", "error", "P0：操作事务钩子中独立保存会破坏事务一致性；必须直接修改平台传入的数据实体", "p0-transaction-save"));
 			}
 		});
 	}
@@ -139,10 +139,10 @@ function checkResourceHandling(lines: string[]): CheckResult[] {
 		const line = lines[i];
 		if (isCommentLine(line)) continue;
 		if (/\.\s*getDynamicObject\s*\([^)]*\)\s*\.\s*get(?:String|Long|Int|Integer|Double|Date|PkValue|DynamicObject|DynamicObjectCollection)\s*\(/.test(line)) {
-			results.push(makeResult(i, line, "getDynamicObject", "resource", "warning", "P1：嵌套 DynamicObject 直接取值缺少空值保护，引用字段为空时可能 NPE；先取对象并判空或用安全取值工具", "p1-dynamicobject-chain-null-risk"));
+			results.push(makeResult(i, line, "getDynamicObject", "resource", "warning", "P1：嵌套 DynamicObject 直接取值缺少空值保护，引用字段为空时可能 NPE；取对象并判空后再取值，或使用安全取值工具", "p1-dynamicobject-chain-null-risk"));
 		}
 		if (/row\s*\.\s*get(?:BigDecimal|String|Long|Int|Integer|Date)\s*\(/.test(line) && !nearbyLineHas(lines, i, -2, /\brow\b\s*(?:!=|==)\s*null|Optional\.ofNullable\s*\(\s*row\s*\)|Objects\.nonNull\s*\(\s*row\s*\)/)) {
-			results.push(makeResult(i, line, "row", "resource", "warning", "P1：DataSet.Row 取值可能返回 null，参与运算或解包前应判空或给默认值", "p1-dataset-row-null-risk"));
+			results.push(makeResult(i, line, "row", "resource", "warning", "P1：DataSet.Row 取值可能返回 null，参与运算或解包前必须判空或给默认值", "p1-dataset-row-null-risk"));
 		}
 	}
 
@@ -157,19 +157,19 @@ function checkSecurityPatterns(lines: string[]): CheckResult[] {
 		if (isCommentLine(line)) continue;
 
 		if (/\bStatement\b/.test(line) && !/\bPreparedStatement\b/.test(line)) {
-			results.push(makeResult(i, line, "Statement", "security", "error", "P0：使用 Statement 存在 SQL 注入风险；请改用参数化查询或 QFilter", "p0-raw-statement"));
+			results.push(makeResult(i, line, "Statement", "security", "error", "P0：使用 Statement 存在 SQL 注入风险；必须改用参数化查询或 QFilter", "p0-raw-statement"));
 		}
 
 		if (/"\s*(SELECT|UPDATE|DELETE|INSERT)\b[^"]*"\s*\+|\+\s*"\s*(WHERE|AND|OR|SET)\b/i.test(line)) {
-			results.push(makeResult(i, line, "+", "security", "error", "P0：SQL 字符串拼接存在注入风险；请改用参数化查询或 QFilter", "p0-sql-concat"));
+			results.push(makeResult(i, line, "+", "security", "error", "P0：SQL 字符串拼接存在注入风险；必须改用参数化查询或 QFilter", "p0-sql-concat"));
 		}
 
 		if (/<!DOCTYPE|<!ENTITY|\bSYSTEM\b/.test(line)) {
-			results.push(makeResult(i, line, "XML", "security", "error", "P0：XML 外部实体相关内容可能导致 XXE 风险，请禁用外部实体", "p0-xxe"));
+			results.push(makeResult(i, line, "XML", "security", "error", "P0：XML 外部实体相关内容可能导致 XXE 风险；必须禁用外部实体", "p0-xxe"));
 		}
 
 		if (/(password|passwd|secret|token|ak|sk)\s*=\s*"[^"]{4,}"/i.test(line)) {
-			results.push(makeResult(i, line, "=", "security", "error", "P0：疑似敏感信息硬编码，请改为安全配置或密文存储", "p0-secret-hardcode"));
+			results.push(makeResult(i, line, "=", "security", "error", "P0：疑似敏感信息硬编码；必须改为安全配置或密文存储", "p0-secret-hardcode"));
 		}
 
 		if (/\bcache\s*\.\s*put\s*\(/i.test(line) || /\bCacheFactory\b/.test(line)) {
@@ -197,7 +197,7 @@ function checkThreadingPatterns(lines: string[]): CheckResult[] {
 		const line = lines[i];
 		if (isCommentLine(line)) continue;
 		if (/new\s+Thread\s*\(|Executors\s*\.\s*new\w+/.test(line)) {
-			results.push(makeResult(i, line, "Thread", "threading", "error", "P0：使用 JDK 原生线程会绕过平台线程管理；请使用苍穹 ThreadPools", "p0-native-thread"));
+			results.push(makeResult(i, line, "Thread", "threading", "error", "P0：使用 JDK 原生线程会绕过平台线程管理；必须使用苍穹 ThreadPools", "p0-native-thread"));
 		}
 	}
 
@@ -217,15 +217,15 @@ function checkUiPerformance(lines: string[]): CheckResult[] {
 		if (/endInit\s*\(/.test(line)) endInitCount++;
 
 		if (isInsideLoop(lines, i) && /updateView\s*\(/.test(line)) {
-			results.push(makeResult(i, line, "updateView", "performance", "warning", "P1：循环内刷新视图会造成界面卡顿；应在循环外统一刷新", "p1-loop-update-view"));
+			results.push(makeResult(i, line, "updateView", "performance", "warning", "P1：循环内刷新视图会造成界面卡顿；必须在循环外统一刷新", "p1-loop-update-view"));
 		}
 
 		if (isInsideLoop(lines, i) && /getFieldIndex\s*\(/.test(line)) {
-			results.push(makeResult(i, line, "getFieldIndex", "performance", "warning", "P1：循环内重复获取 FieldIndex 会造成性能损耗；应在循环外缓存", "p1-loop-field-index"));
+			results.push(makeResult(i, line, "getFieldIndex", "performance", "warning", "P1：循环内重复获取 FieldIndex 会造成性能损耗；必须在循环外缓存", "p1-loop-field-index"));
 		}
 
 		if (/SerializationUtils\s*\.\s*toJsonString\s*\(/.test(line) && /(view|model|event|page|args|this|getView\s*\(|getModel\s*\(|\be\b)/i.test(line)) {
-			results.push(makeResult(i, line, "SerializationUtils", "performance", "warning", "P1：不要序列化页面、模型或事件大对象做日志，容易造成 CPU 和内存压力；只记录关键字段", "p1-heavy-json-serialization"));
+			results.push(makeResult(i, line, "SerializationUtils", "performance", "warning", "P1：禁止序列化页面、模型或事件大对象做日志，容易造成 CPU 和内存压力；只记录关键字段", "p1-heavy-json-serialization"));
 		}
 
 		const entryUiMatch = line.match(/\bgetView\s*\(\)\s*\.\s*(setEnable|setVisible)\s*\(/);
@@ -235,7 +235,7 @@ function checkUiPerformance(lines: string[]): CheckResult[] {
 			if (args.length >= 2 && !looksLikeRowIndexedViewCall(args)) {
 				const fieldArgs = args.slice(1).join(",");
 				if (/(entry|entries|detail|row|qty|price|amount|material|item)/i.test(fieldArgs)) {
-					results.push(makeResult(i, line, entryUiMatch[1], "performance", "warning", "P1：疑似对分录字段使用单头 setEnable/setVisible；分录字段应使用带 rowIndex 的重载", "p1-entry-field-without-row-index"));
+					results.push(makeResult(i, line, entryUiMatch[1], "performance", "warning", "P1：疑似对分录字段使用单头 setEnable/setVisible；分录字段必须使用带 rowIndex 的重载", "p1-entry-field-without-row-index"));
 				}
 			}
 		}
@@ -262,7 +262,7 @@ function checkViewInteractionOrder(lines: string[]): CheckResult[] {
 			for (let j = i + 1; j < Math.min(lines.length, i + 25); j++) {
 				if (/^\s*}\s*$/.test(lines[j])) break;
 				if (!isCommentLine(lines[j]) && /(getModel\s*\(\)\s*\.\s*)?setValue\s*\(/.test(lines[j])) {
-					results.push(makeResult(i, line, "setDataChanged", "lifecycle", "warning", "P1：setDataChanged(false) 后仍继续 setValue，脏标记会被重新置回；应放到所有 setValue 之后", "p1-set-data-changed-before-setvalue"));
+					results.push(makeResult(i, line, "setDataChanged", "lifecycle", "warning", "P1：setDataChanged(false) 后仍继续 setValue，脏标记会被重新置回；必须放到所有 setValue 之后", "p1-set-data-changed-before-setvalue"));
 					break;
 				}
 			}
@@ -280,7 +280,7 @@ function checkViewInteractionOrder(lines: string[]): CheckResult[] {
 			for (let j = i + 1; j < Math.min(lines.length, i + 10); j++) {
 				if (/^\s*}\s*$/.test(lines[j])) break;
 				if (!isCommentLine(lines[j]) && /returnDataToParent\s*\(/.test(lines[j])) {
-					results.push(makeResult(i, line, "close", "lifecycle", "error", "P0：先 close 再 returnDataToParent 会导致返回数据不执行；应先 returnDataToParent 再 close", "p0-close-before-return-data"));
+					results.push(makeResult(i, line, "close", "lifecycle", "error", "P0：close 早于 returnDataToParent 会导致返回数据不执行；必须执行 returnDataToParent 后再 close", "p0-close-before-return-data"));
 					break;
 				}
 			}
@@ -298,11 +298,11 @@ function checkLoggingAndDiagnostics(lines: string[]): CheckResult[] {
 		if (isCommentLine(line)) continue;
 
 		if (/System\s*\.\s*out\s*\.\s*println\s*\(/.test(line)) {
-			results.push(makeResult(i, line, "System.out.println", "exception", "warning", "P1：生产代码不应使用 System.out.println，请使用平台日志", "p1-system-out"));
+			results.push(makeResult(i, line, "System.out.println", "exception", "warning", "P1：生产代码禁止使用 System.out.println；必须使用平台日志", "p1-system-out"));
 		}
 
 		if (/\.printStackTrace\s*\(/.test(line)) {
-			results.push(makeResult(i, line, "printStackTrace", "exception", "warning", "P2：不要直接 printStackTrace，请使用 logger.error 并传入异常对象", "p2-print-stack-trace"));
+			results.push(makeResult(i, line, "printStackTrace", "exception", "warning", "P2：禁止直接 printStackTrace；必须使用 logger.error 并传入异常对象", "p2-print-stack-trace"));
 		}
 	}
 
@@ -326,7 +326,7 @@ function checkMagicValues(lines: string[]): CheckResult[] {
 					column: match.index ?? 0,
 					type: "magic_value",
 					severity: "warning",
-					message: `检测到硬编码数字 ${value}，建议使用常量或配置`,
+					message: `检测到硬编码数字 ${value}，必须改为常量或配置`,
 					rule: "magic-number",
 				});
 			}
@@ -350,7 +350,7 @@ function checkMagicValues(lines: string[]): CheckResult[] {
 					column: match.index ?? 0,
 					type: "magic_value",
 					severity: "warning",
-					message: `检测到可能的业务常量 "${value}"，建议定义为常量`,
+					message: `检测到可能的业务常量 "${value}"，必须定义为常量`,
 					rule: "magic-string",
 				});
 			}
@@ -378,7 +378,7 @@ function checkNaming(lines: string[], language: CheckLanguage): CheckResult[] {
 						column: line.indexOf(className),
 						type: "naming",
 						severity: "error",
-						message: `类名 "${className}" 应使用 PascalCase 命名`,
+						message: `类名 "${className}" 必须使用 PascalCase 命名`,
 						rule: "class-naming",
 					});
 				}
@@ -401,7 +401,7 @@ function checkNaming(lines: string[], language: CheckLanguage): CheckResult[] {
 						column: line.indexOf(methodName),
 						type: "naming",
 						severity: "warning",
-						message: `方法名 "${methodName}" 应使用 camelCase 命名`,
+						message: `方法名 "${methodName}" 必须使用 camelCase 命名`,
 						rule: "method-naming",
 					});
 				}
@@ -416,7 +416,7 @@ function checkNaming(lines: string[], language: CheckLanguage): CheckResult[] {
 						column: line.indexOf(constantName),
 						type: "naming",
 						severity: "warning",
-						message: `常量 "${constantName}" 应使用 UPPER_SNAKE_CASE 命名`,
+						message: `常量 "${constantName}" 必须使用 UPPER_SNAKE_CASE 命名`,
 						rule: "constant-naming",
 					});
 				}
@@ -519,7 +519,7 @@ function checkExceptionHandling(lines: string[]): CheckResult[] {
 				column: line.indexOf("catch"),
 				type: "exception",
 				severity: "error",
-				message: "空的 catch 块，应记录日志或处理异常",
+				message: "空的 catch 块，必须记录日志或处理异常",
 				rule: "empty-catch",
 			});
 		}
@@ -539,7 +539,7 @@ export function formatCheckResults(results: CheckResult[]): string {
 		`金蝶代码检查发现 ${results.length} 个问题：`,
 		`错误：${errors.length}`,
 		`警告：${warnings.length}`,
-		`提示：${infos.length}`,
+		`信息：${infos.length}`,
 		"",
 	];
 

package/src/tools/build-debug.ts

@@ -20,14 +20,14 @@ export interface DebugFinding {
 
 export function planBuild(cwd: string, profile: ProductProfile, target?: string): BuildPlan {
 	if (profile.product === "unknown") {
-		throw new Error("产品画像未知。运行 kd_build 前请先设置 product。");
+		throw new Error("产品画像未知。运行 kd_build 前必须设置 product。");
 	}
 
 	if (profile.platform === "cosmic") return planJavaBuild(cwd, profile, target);
 	if (profile.platform === "enterprise-csharp") return planCsharpBuild(cwd, profile, target);
 	if (profile.platform === "enterprise-python") {
 		throw new Error(
-			"企业版 Python 插件通常没有本地构建步骤。请通过 BOS 注册和功能测试验证，并在 VERIFY.md 中记录脚本路径、插件类型、FormId、字段/实体标识和测试数据。",
+			"企业版 Python 插件通常没有本地构建步骤。LLM 不能替代 BOS 注册和功能测试；必须要求用户提供验证结果，并在 VERIFY.md 记录脚本路径、插件类型、FormId、字段/实体标识、测试数据和用户侧验证证据。",
 		);
 	}
 	throw new Error(`未找到 ${profile.product}/${profile.platform}/${profile.techStack} 的构建策略。`);
@@ -105,7 +105,7 @@ export function analyzeDebugText(text: string): DebugFinding[] {
 		severity: "error",
 		rule: "build-error",
 		message: "检测到构建或编译错误。",
-		nextStep: "使用产品对应的构建输出。Cosmic Java 需查证 SDK 签名；企业版 C# 需查证命名空间和引用。",
+		nextStep: "使用产品对应的构建输出。Cosmic Java 必须查证 SDK 签名；企业版 C# 必须查证命名空间和引用。",
 	});
 
 	if (findings.length === 0) {
@@ -114,7 +114,7 @@ export function analyzeDebugText(text: string): DebugFinding[] {
 			rule: "no-known-pattern",
 			message: "未匹配到已知金蝶调试模式。",
 			evidence: "",
-			nextStep: "请提供更完整的日志、堆栈、产品/版本、目标单据/表单和最近代码改动。",
+			nextStep: "补充完整日志、堆栈、产品/版本、目标单据/表单和最近代码改动。",
 		});
 	}
 
@@ -138,7 +138,7 @@ export function formatDebugFindings(findings: DebugFinding[]): string {
 
 export function readDebugInput(cwd: string, text?: string, path?: string): { source: string; text: string } {
 	if (text) return { source: "inline", text };
-	if (!path) throw new Error("kd_debug 需要提供 text 或 path。");
+	if (!path) throw new Error("kd_debug 参数要求：提供 text 或 path。");
 
 	const fullPath = resolveWorkspacePath(cwd, path);
 	return { source: path, text: readFileSync(fullPath, "utf8") };

package/src/tools/sdk-signature.ts

@@ -46,7 +46,7 @@ export async function inspectSdkSignature(cwd: string, params: SdkSignatureParam
 			query,
 			exitCode: 2,
 			stdout: "",
-			stderr: "kd_sdk_signature 需要提供 query 或 className。method 只是在已匹配类/类型内部过滤。",
+			stderr: "kd_sdk_signature 参数要求：提供 query 或 className。method 仅在已匹配类/类型内部过滤。",
 			sources: [],
 		};
 	}
@@ -64,7 +64,7 @@ export function formatSdkSignatureResult(result: SdkSignatureResult): string {
 		result.stdout.trim() ? `\nSTDOUT:\n${result.stdout.trim()}` : undefined,
 		result.stderr.trim() ? `\nSTDERR:\n${result.stderr.trim()}` : undefined,
 		"",
-		"只有退出码为 0 时，才能把该结果作为本地 SDK 证据。失败时，应先使用构建输出或项目 SDK 配置查证，再相信随包知识库。",
+		"证据规则：只有退出码为 0 时，该结果才能作为本地 SDK 证据。失败时必须使用构建输出或项目 SDK 配置查证；禁止用随包知识库替代。",
 	]
 		.filter(Boolean)
 		.join("\n");
@@ -79,7 +79,7 @@ async function inspectJavaSignature(cwd: string, params: SdkSignatureParams, que
 			query,
 			exitCode: 2,
 			stdout: "",
-			stderr: "当前项目未找到 jar 文件。请先构建/复制依赖，或传入 path=<sdk/lib 目录>。",
+			stderr: "当前项目未找到 jar 文件。执行构建/复制依赖，或传入 path=<sdk/lib 目录>。",
 			sources: [],
 		};
 	}
@@ -136,7 +136,7 @@ async function inspectCsharpSignature(cwd: string, params: SdkSignatureParams, q
 			query,
 			exitCode: 2,
 			stdout: "",
-			stderr: "当前项目未找到 dll 文件。请先构建/还原引用，或传入 path=<sdk/bin 目录>。",
+			stderr: "当前项目未找到 dll 文件。执行构建/还原引用，或传入 path=<sdk/bin 目录>。",
 			sources: [],
 		};
 	}