@stalfh233/omc-cli 0.5.1 → 0.5.2

package/dist/lanes/model.js

@@ -14,6 +14,7 @@ import { writeTool } from "../write.js";
 import { registerPackageTool } from "../package-tool.js";
 import { defineTool, ToolInputError, ToolRefusalError } from "../tool-types.js";
 import { wireCall, wireRows, WireError } from "../wire.js";
+import { readModelRecord } from "../model-record.js";
 import { MODEL_CREATE_BODY_DEFAULTS, MODEL_LANE_CONTRACT } from "./model-contract.js";
 const MODEL_LANE = MODEL_LANE_CONTRACT.lane;
 const MODEL_TOOLS = MODEL_LANE_CONTRACT.tools;
@@ -48,7 +49,7 @@ function modelRollbackSpec(toolName) {
         target: (args) => `model:${String(args.schemaCode ?? args.code ?? "")}`,
         read: async (session, args) => {
             const resolved = await resolveModelTreeNode(session, String(args.schemaCode ?? args.code));
-            const record = (await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: String(args.schemaCode ?? args.code) } })).data;
+            const record = await readModelRecord(session, String(args.schemaCode ?? args.code));
             if (!resolved.node && !record)
                 return null;
             return { node: resolved.node ?? null, record: record ?? null };
@@ -386,7 +387,7 @@ export function registerModelLane(tools) {
             // bizmodels/get_all is keyed by the query, so a non-null object IS the
             // model (an absent one answers a business errcode — wireCall throws and
             // we fall through to create).
-            const record = (await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: args.code } })).data;
+            const record = await readModelRecord(session, args.code);
             if (record !== null && typeof record === "object") {
                 throw new ToolRefusalError({
                     status: "refused",
@@ -461,7 +462,7 @@ export function registerModelLane(tools) {
     async function modelDeleteDryRun(session, args) {
         const record = await (async () => {
             try {
-                return (await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: args.schemaCode } })).data ?? null;
+                return readModelRecord(session, args.schemaCode);
             }
             catch (error) {
                 if (error instanceof WireError && error.kind === "business")
@@ -510,17 +511,7 @@ export function registerModelLane(tools) {
     writeTool(tools, "model.delete", MODEL_LANE, MODEL_TOOLS["model.delete"].scope, MODEL_TOOLS["model.delete"].description, { schemaCode: z.string().min(1), applicationCode: z.string().optional() }, async (session, args) => {
         // authoritative model record (bizmodels/get_all); the app-tree node id
         // may lag behind and is cleaned best-effort.
-        const readRecord = async () => {
-            try {
-                const result = await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: args.schemaCode } });
-                return result.data ?? null;
-            }
-            catch (error) {
-                if (error instanceof WireError && error.kind === "business")
-                    return null;
-                throw error;
-            }
-        };
+        const readRecord = () => readModelRecord(session, args.schemaCode);
         const record = await readRecord();
         // Ticket 68: resolve the app-tree node across applications (the tree row
         // id IS the AppFunction id physically_delete needs). Never fall back to
@@ -673,17 +664,7 @@ export function registerModelLane(tools) {
         // every existing model in that single request after prechecking each
         // target's inbound references — a referenced model refuses the whole
         // batch with per-target evidence and ZERO side effects.
-        const readRecord = async (schemaCode) => {
-            try {
-                const result = await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode } });
-                return result.data ?? null;
-            }
-            catch (error) {
-                if (error instanceof WireError && error.kind === "business")
-                    return null;
-                throw error;
-            }
-        };
+        const readRecord = (schemaCode) => readModelRecord(session, schemaCode);
         const applicationCode = await resolveApplicationCode(session, args.applicationCode).catch(() => "");
         const targets = [...new Set(args.schemaCodes)];
         const alreadyAbsent = [];
@@ -885,8 +866,10 @@ export function registerModelLane(tools) {
         // NOT migrated by this endpoint (probe evidence).
         const after = await resolveModelTreeNode(session, args.schemaCode);
         const nodeAfter = after.node;
-        const record = await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: args.schemaCode } });
-        const recordName = (record.data ?? {}).name ?? null;
+        // Informational only: the app-tree node is the authoritative face for a
+        // rename, so an unreadable record face must not fail the whole call.
+        const record = await readModelRecord(session, args.schemaCode).catch(() => null);
+        const recordName = record?.name ?? null;
         const nodeName = typeof nodeAfter?.name === "string" ? nodeAfter.name : null;
         return {
             performed: { errcode: performed.errcode },

package/dist/model-purge.js

@@ -27,6 +27,7 @@ import { CLI_EXIT_CODES, classifyResult, envelope, refusalText, requestId } from
 import { writeEnvelope } from "./cli-output.js";
 import { ENDPOINTS } from "./endpoints.js";
 import { wireCall, wireRows } from "./wire.js";
+import { readModelRecord } from "./model-record.js";
 import { SessionManager } from "./session-manager.js";
 import { ConfigStore, defaultPaths } from "./config-store.js";
 async function sessionFor(environment) {
@@ -43,8 +44,7 @@ export async function planPurge(options, injected) {
     const session = injected ?? (await sessionFor(options.environment));
     let modelExists = true;
     try {
-        const record = (await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode: options.schemaCode } })).data;
-        modelExists = record !== null && record !== undefined;
+        modelExists = (await readModelRecord(session, options.schemaCode)) !== null;
     }
     catch {
         modelExists = false;

package/dist/model-record.js

@@ -0,0 +1,40 @@
+import { ENDPOINTS } from "./endpoints.js";
+import { wireCall, wireRows, WireError } from "./wire.js";
+/**
+ * Authoritative biz-model record, across platform versions.
+ *
+ * 8.6 serves `/api/api/app/bizmodels/get_all` (BizSchemaController resolves
+ * getBizSchemaBySchemaCode and answers 10011 on a miss). 8.0/8.1 deployments
+ * answer **HTTP 404** for that path (field report 2026-10-30, 8.1.108 /
+ * 上海电建) and expose the same rows through `/bizmodels/list_schema`, filtered
+ * by `schemaCode`. Pinning the 8.6 path made model.*, rule.* and the purge
+ * precheck die with a raw 404 before any of their own logic ran.
+ *
+ * Reads the primary face first; only a 404 (endpoint absent) falls back, so a
+ * genuine 401/500/transport failure still surfaces as itself. Returns null when
+ * the model is not there (business "not found" on either face).
+ */
+export async function readModelRecord(session, schemaCode) {
+    try {
+        const result = await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode } });
+        return result.data ?? null;
+    }
+    catch (error) {
+        if (error instanceof WireError && error.kind === "business")
+            return null;
+        const endpointAbsent = error instanceof WireError && error.kind === "http-status" && error.diagnostic?.httpStatus === 404;
+        if (!endpointAbsent)
+            throw error;
+    }
+    try {
+        const rows = wireRows((await wireCall(session, { path: ENDPOINTS.modelList, query: { schemaCode } })).data);
+        const match = rows.find((row) => String(row.schemaCode ?? row.code ?? "") === schemaCode);
+        return match ?? null;
+    }
+    catch (error) {
+        if (error instanceof WireError && error.kind === "business")
+            return null;
+        throw error;
+    }
+}
+//# sourceMappingURL=model-record.js.map

package/dist/rule-support.js

@@ -1,6 +1,7 @@
 import { z } from "zod";
 import { wireCall, wireRows, WireError } from "./wire.js";
 import { ENDPOINTS } from "./endpoints.js";
+import { readModelRecord } from "./model-record.js";
 import { ToolRefusalError } from "./tool-types.js";
 import { decodeJsonColumn, JSON_COLUMN_NULL_COALESCE } from "./json-column.js";
 import { RULE_BIZ_RULE_TYPE_DESCRIPTION, RULE_CREATE_BODY, RULE_ROUTES_DESCRIPTION, ruleNodeGraphDescription, schedulerSettingDescription, } from "./lanes/businessrule-contract.js";
@@ -602,17 +603,10 @@ export async function assertRuleSchemaExists(session, schemaCode) {
     // SCHEMA_NOT_FOUNT_ERR (10011 业务模型不存在) on miss — the same facade
     // primitive the rule faces rely on. The fake answers ok+null for an absent
     // schema, so BOTH signals decode as a miss.
-    let missing = false;
-    try {
-        const probe = await wireCall(session, { path: ENDPOINTS.modelGet, query: { schemaCode } });
-        missing = probe.data == null;
-    }
-    catch (error) {
-        if (error instanceof WireError && error.kind === "business" && error.errcode === 10011)
-            missing = true;
-        else
-            throw error;
-    }
+    // Version-tolerant: 8.1 has no `/bizmodels/get_all` (404) and serves the same
+    // rows from `/bizmodels/list_schema`; see model-record.ts.
+    const record = await readModelRecord(session, schemaCode);
+    const missing = record === null;
     if (missing) {
         throw new ToolRefusalError({
             status: "refused",

package/package.json

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