@stalfh233/omc-cli 0.4.69 → 0.4.70

package/dist/endpoints.js

@@ -55,6 +55,7 @@ export const ENDPOINTS = {
     businessRuleCreate: "/api/api/app/businessrule/create",
     businessRuleUpdate: "/api/api/app/businessrule/update",
     businessRulePublish: "/api/api/app/businessrule/publish",
+    businessRuleNodeRunMap: "/api/api/app/businessrule/node_run_map",
     businessRuleEnable: "/api/api/app/businessrule/enable",
     businessRuleDelete: "/api/api/app/businessrule/delete",
     dataRuleCreate: "/api/api/app/bizdatarule/create",

package/dist/lanes/rule-lifecycle.js

@@ -3,7 +3,7 @@ import { ENDPOINTS } from "../endpoints.js";
 import { writeTool } from "../write.js";
 import { registerPackageTool } from "../package-tool.js";
 import { writeSummary } from "../human-summary.js";
-import { executeRuleSaveWrite, ruleNodeGraphArg, assertRuleSchemaExists } from "../rule-support.js";
+import { executeRuleSaveWrite, ruleNodeGraphArg, assertRuleSchemaExists, explainRuleSelfCheck } from "../rule-support.js";
 import { ToolRefusalError } from "../tool-types.js";
 import { prepareRuleGraph, validateRuleNodeGraph, PLATFORM_TRIGGER_RULE_CODES } from "./rule-graph.js";
 import { wireCall, wireRows, WireError } from "../wire.js";
@@ -52,7 +52,25 @@ export async function executeRuleEnableWrite(session, args) {
     const full = (await wireCall(session, { path: ENDPOINTS.businessRuleGet, query: { id: String(row.id) } })).data;
     if (!full)
         throw new WireError("business", `rule ${args.ruleCode} 完整模型读回为空（businessrule/get id ${row.id}），拒绝用列表投影发布`, 302024);
-    const published = await wireCall(session, { path: ENDPOINTS.businessRulePublish, method: "POST", body: { ...full, enabled: true, status: "ENABLED" } });
+    let published;
+    try {
+        published = await wireCall(session, { path: ENDPOINTS.businessRulePublish, method: "POST", body: { ...full, enabled: true, status: "ENABLED" } });
+    }
+    catch (error) {
+        // 550018 is the platform's self-check refusal (see explainRuleSelfCheck).
+        if (error instanceof WireError && error.errcode === 550018) {
+            const detail = await explainRuleSelfCheck(session, { ...full, enabled: true, status: "ENABLED" });
+            throw new ToolRefusalError({
+                status: "refused",
+                blocker: "rule-self-check-failed",
+                schemaCode: args.schemaCode,
+                ruleCode: args.ruleCode,
+                ...detail,
+                message: `规则 ${args.ruleCode} 启用时被平台自检拒绝（550018）。${detail.nodeChecks.length > 0 ? `问题节点：${JSON.stringify(detail.nodeChecks).slice(0, 600)}` : "先用 field.list(all:true) 核对引用的字段是否已发布。"}`,
+            });
+        }
+        throw error;
+    }
     let enableErrcode = 0;
     let alreadyEnabled = false;
     try {

package/dist/rule-support.js

@@ -463,6 +463,71 @@ export async function computeCarriedNodeCodes(session, schemaCode, ruleCode, pre
         return undefined; // 取不到存量图 → 维持严格闭集，行为与放宽前一致
     }
 }
+/**
+ * 550018 is the platform's catch-all "rule self-check failed" (ErrorCode
+ * BUSINESS_RULE_PARAM_ERROR from BusinessRuleFacadeImpl.publishBusinessRule when
+ * checkBizRuleIsEnable() is false). The message names nothing, and the real原因
+ * is one of: a referenced property that does not exist, a missing org object, or
+ * a dead cycle. `businessrule/node_run_map` (BusinessRuleController.getNodeRunMaps
+ * → checkBizRuleError) returns exactly that per node, so a refusal can name the
+ * offending node and field instead of asking the caller to open the designer.
+ */
+const RULE_SELF_CHECK_ERRCODE = 550018;
+function ruleSelfCheckMessage(ruleCode, checks) {
+    if (checks.length === 0)
+        return `规则 ${ruleCode} 发布被平台自检拒绝（550018「保存规则失败，规则存在异常」），但 node_run_map 未给出具体节点——常见原因：引用的字段未发布/不存在、组织对象失效、或存在死循环。先用 field.list(all:true) 核对引用的字段是否已发布。`;
+    const detail = checks.map((check) => {
+        const parts = [];
+        const missing = check.missingProperties;
+        if (missing?.length)
+            parts.push(`字段不存在/未发布：${missing.map((item) => `${item.objectName ?? "?"}.${item.propertyName ?? item.propertyCode ?? "?"}`).join("、")}`);
+        if (check.missingOrgObjects)
+            parts.push("组织对象不存在");
+        if (check.deadCycles)
+            parts.push("存在死循环");
+        return `${String(check.nodeCode ?? "?")}${check.nodeName ? `(${String(check.nodeName)})` : ""} ${parts.join("；")}`;
+    });
+    return `规则 ${ruleCode} 发布被平台自检拒绝（550018）：${detail.join(" | ")}。节点内引用了不存在或未发布的字段时，保存能过、发布必拒——先补字段并 field.publish，再重发。`;
+}
+export async function explainRuleSelfCheck(session, model) {
+    try {
+        const response = await wireCall(session, { path: ENDPOINTS.businessRuleNodeRunMap, method: "POST", body: model });
+        const rows = wireRows(response.data);
+        const nodeChecks = rows.flatMap((row) => {
+            // Wire shape (QuoteObjectVO): {objectCode, objectName, quotePropertyList:
+            // [{propertyCode, propertyName, exist, propertyPublished}]} — the property
+            // sits one level deeper than the object.
+            const missingProperties = Array.isArray(row.notExistPropertyObjectList)
+                ? row.notExistPropertyObjectList.flatMap((object) => {
+                    const properties = Array.isArray(object.quotePropertyList) ? object.quotePropertyList : [];
+                    const broken = properties.filter((property) => property.exist === false || property.propertyPublished === false);
+                    const source = broken.length > 0 ? broken : properties;
+                    return source.map((property) => ({
+                        objectName: object.objectName ?? object.objectCode ?? null,
+                        propertyCode: property.propertyCode ?? null,
+                        propertyName: property.propertyName ?? null,
+                        published: property.propertyPublished ?? null,
+                    }));
+                })
+                : [];
+            const missingOrgObjects = Array.isArray(row.notExistOrgObjectList) ? row.notExistOrgObjectList : [];
+            const deadCycles = Array.isArray(row.deadCycleRunMapList) ? row.deadCycleRunMapList : [];
+            if (missingProperties.length === 0 && missingOrgObjects.length === 0 && deadCycles.length === 0)
+                return [];
+            return [{
+                    nodeCode: row.nodeCode ?? null,
+                    nodeName: row.nodeName ?? null,
+                    ...(missingProperties.length > 0 ? { missingProperties } : {}),
+                    ...(missingOrgObjects.length > 0 ? { missingOrgObjects } : {}),
+                    ...(deadCycles.length > 0 ? { deadCycles } : {}),
+                }];
+        });
+        return { nodeChecks };
+    }
+    catch {
+        return { nodeChecks: [] };
+    }
+}
 export async function executeRuleSaveWrite(session, args) {
     // Pre-flight dry lint (ticket 14): a mis-shaped graph must be refused BEFORE
     // any wire call, with structured blockers locating the node/key — never
@@ -594,7 +659,23 @@ export async function executeRuleSaveWrite(session, args) {
             // schedule and a TIMING_TASK rule never gets a quartz job (2026-10-06). Use
             // the authoritative `businessrule/get?id` model (same as rule.get/enable).
             const full = (await wireCall(session, { path: ENDPOINTS.businessRuleGet, query: { id: String(row.id) } })).data;
-            await wireCall(session, { path: ENDPOINTS.businessRulePublish, method: "POST", body: full ?? row });
+            try {
+                await wireCall(session, { path: ENDPOINTS.businessRulePublish, method: "POST", body: full ?? row });
+            }
+            catch (error) {
+                if (error instanceof WireError && error.errcode === RULE_SELF_CHECK_ERRCODE) {
+                    const detail = await explainRuleSelfCheck(session, (full ?? row));
+                    throw new ToolRefusalError({
+                        status: "refused",
+                        blocker: "rule-self-check-failed",
+                        schemaCode: args.schemaCode,
+                        ruleCode: args.ruleCode,
+                        ...detail,
+                        message: ruleSelfCheckMessage(args.ruleCode, detail.nodeChecks),
+                    });
+                }
+                throw error;
+            }
         }
         const effective = await currentRow();
         // Live 8.6.26 (2026-09-17): businessrule/publish mints a version but does

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stalfh233/omc-cli",
-  "version": "0.4.69",
+  "version": "0.4.70",
   "description": "CloudPivot 8.6 configuration capability CLI — governed, fail-closed, coverage-manifest bounded.",
   "type": "module",
   "bin": {
@@ -50,9 +50,9 @@
     "access": "public"
   },
   "dependencies": {
-    "@stalfh233/omc-fake-platform": "0.4.69",
+    "@stalfh233/omc-fake-platform": "0.4.70",
     "commander": "14.0.3",
-    "omc-core": "0.4.69",
+    "omc-core": "0.4.70",
     "proper-lockfile": "4.1.2",
     "write-file-atomic": "6.0.0",
     "zod": "^3.23.8",