@dazitech/cli 3.1.0 → 3.1.1

package/README.md

@@ -78,7 +78,7 @@ pnpm run publish:cli
 
 ```powershell
 # 全局
-pnpm add -g @dazitech/cli@3.1.0
+pnpm add -g @dazitech/cli@3.1.1
 
 # 确认 PATH（pnpm 全局 bin）
 pnpm bin -g

package/dist/clis/dazi-app.js

@@ -13076,8 +13076,8 @@ var {
   Help
 } = import_index.default;
 
-// cli/shared/src/version.ts
-var DAZI_VERSION = true ? "3.1.0" : readDevVersion();
+// cli/shared/src/version.js
+var DAZI_VERSION = true ? "3.1.1" : readDevVersion();
 
 // cli/dazi-app/src/lib/httpClient.ts
 init_config();

package/dist/clis/dazi-flow.js

@@ -4981,8 +4981,8 @@ function resolveWorkspace(cwd = process.cwd()) {
   };
 }
 
-// cli/shared/src/version.ts
-var DAZI_VERSION = true ? "3.1.0" : readDevVersion();
+// cli/shared/src/version.js
+var DAZI_VERSION = true ? "3.1.1" : readDevVersion();
 
 // cli/dazi-flow/src/commands/flows.ts
 var import_path3 = __toESM(require("path"), 1);

package/dist/clis/dazi-onto.js

@@ -3215,13 +3215,14 @@ function resolveWorkspace(cwd = process.cwd()) {
   };
 }
 
-// cli/shared/src/version.ts
-var DAZI_VERSION = true ? "3.1.0" : readDevVersion();
+// cli/shared/src/version.js
+var DAZI_VERSION = true ? "3.1.1" : readDevVersion();
 
-// cli/shared/src/ontoWorkspaceAudit.ts
+// cli/shared/src/ontoWorkspaceAudit.js
 function ontoItemSubdirForScriptType(scriptType) {
   const t = (scriptType ?? "").toLowerCase();
-  if (t === "ontology_function" || t === "ontology_action") return "functions";
+  if (t === "ontology_function" || t === "ontology_action")
+    return "functions";
   return "setup";
 }
 
@@ -3318,14 +3319,246 @@ function makeSpaceCommand() {
 // cli/dazi-onto/src/commands/function.ts
 var import_path4 = __toESM(require("path"), 1);
 var import_fs3 = __toESM(require("fs"), 1);
+
+// cli/dazi-onto/src/lib/ontologyFunctionLint.ts
+function extractTestArgumentsParams(code) {
+  const block = extractPythonAssignDict(code, "TEST_ARGUMENTS");
+  if (!block) return null;
+  const argsBlock = extractDictValueForKey(block, "arguments");
+  if (!argsBlock) return null;
+  try {
+    return JSON.parse(pythonDictFragmentToJson(argsBlock));
+  } catch {
+    return null;
+  }
+}
+function extractPythonAssignDict(code, varName) {
+  const re = new RegExp(`(?:^|\\n)${varName}\\s*=\\s*\\{`, "m");
+  const m = re.exec(code);
+  if (!m) return null;
+  const start = m.index + m[0].length - 1;
+  return extractBalancedBraces(code, start);
+}
+function extractDictValueForKey(dictStr, key) {
+  const re = new RegExp(`["']${key}["']\\s*:\\s*\\{`);
+  const m = re.exec(dictStr);
+  if (!m) return null;
+  const start = m.index + m[0].length - 1;
+  return extractBalancedBraces(dictStr, start);
+}
+function extractBalancedBraces(text, openIndex) {
+  if (text[openIndex] !== "{") return null;
+  let depth = 0;
+  for (let i = openIndex; i < text.length; i++) {
+    const ch = text[i];
+    if (ch === "{") depth++;
+    else if (ch === "}") {
+      depth--;
+      if (depth === 0) return text.slice(openIndex, i + 1);
+    }
+  }
+  return null;
+}
+function pythonDictFragmentToJson(fragment) {
+  return fragment.replace(/\bTrue\b/g, "true").replace(/\bFalse\b/g, "false").replace(/\bNone\b/g, "null").replace(/,\s*}/g, "}").replace(/,\s*]/g, "]").replace(/'/g, '"');
+}
+function lineOf(code, index) {
+  return code.slice(0, index).split("\n").length;
+}
+function lintOntologyFunctionCode(code, filePath) {
+  const issues = [];
+  const label = filePath ? pathBasename(filePath) : "\u811A\u672C";
+  if (/output\.print_json\s*\(/.test(code)) {
+    issues.push({
+      severity: "error",
+      rule: "no-print-json",
+      message: `${label}: \u7981\u6B62 output.print_json()\uFF1B\u987B return p.function_result(...)`
+    });
+  }
+  if (/def\s+main\s*\(\s*params/.test(code)) {
+    issues.push({
+      severity: "error",
+      rule: "main-no-params",
+      message: `${label}: main() \u4E0D\u5F97\u5E26 params \u5F62\u53C2\uFF1B\u987B def main():`
+    });
+  } else if (!/def\s+main\s*\(\s*\)\s*:/.test(code)) {
+    issues.push({
+      severity: "error",
+      rule: "main-required",
+      message: `${label}: \u7F3A\u5C11 def main(): \u5165\u53E3`
+    });
+  }
+  if (!/function_result\s*\(/.test(code)) {
+    issues.push({
+      severity: "error",
+      rule: "function-result-required",
+      message: `${label}: \u987B\u8C03\u7528 p.function_result(...) \u8FD4\u56DE\u7ED3\u6784\u5316\u7ED3\u679C`
+    });
+  }
+  const frPos = /function_result\s*\(\s*\{/.exec(code);
+  if (frPos) {
+    issues.push({
+      severity: "error",
+      rule: "function-result-kwargs",
+      message: `${label}: function_result \u987B\u7528\u5173\u952E\u5B57\u53C2\u6570 columns=/data=/row_count=\uFF0C\u52FF\u4F20 dict  positional`,
+      line: lineOf(code, frPos.index)
+    });
+  }
+  if (!/return\s+/.test(code)) {
+    issues.push({
+      severity: "warn",
+      rule: "return-missing",
+      message: `${label}: main() \u5EFA\u8BAE return _ontology_fn_body(p)`
+    });
+  }
+  if (!/TEST_ARGUMENTS\s*=/.test(code)) {
+    issues.push({
+      severity: "warn",
+      rule: "test-arguments-missing",
+      message: `${label}: \u5EFA\u8BAE\u5B9A\u4E49 TEST_ARGUMENTS\uFF08function run \u65E0\u53C2\u65F6\u56DE\u9000\uFF09`
+    });
+  } else {
+    const args = extractTestArgumentsParams(code);
+    if (!args || Object.keys(args).length === 0) {
+      issues.push({
+        severity: "warn",
+        rule: "test-arguments-empty",
+        message: `${label}: TEST_ARGUMENTS.arguments \u4E3A\u7A7A\uFF0C\u9996\u6B21 function run \u53EF\u80FD\u5931\u8D25`
+      });
+    }
+  }
+  const emptyDate = /\.get\s*\(\s*["'](?:start_date|end_date)["']\s*,\s*["']['"]\s*\)/g;
+  let dm;
+  while ((dm = emptyDate.exec(code)) !== null) {
+    issues.push({
+      severity: "warn",
+      rule: "empty-date-default",
+      message: `${label}: params.get("start_date/end_date", "") \u7A7A\u4E32\u6613\u5BFC\u81F4 strptime \u5931\u8D25\uFF1B\u5EFA\u8BAE\u975E\u7A7A\u9ED8\u8BA4\u6216\u6821\u9A8C`,
+      line: lineOf(code, dm.index)
+    });
+  }
+  return issues;
+}
+function lintOntologySetupInitCode(code, filePath) {
+  const issues = [];
+  const label = filePath ? pathBasename(filePath) : "\u811A\u672C";
+  const base = label.toLowerCase();
+  if (base.includes("category_mount")) return issues;
+  const isInit = /init/i.test(base) && !base.includes("seed");
+  if (!isInit) return issues;
+  const applyMatch = /\.categories\.apply_registry\s*\(/.exec(code);
+  if (applyMatch) {
+    issues.push({
+      severity: "error",
+      rule: "init-no-apply-registry",
+      message: `${label}: init \u811A\u672C\u7981\u6B62 apply_registry\uFF1B\u8BF7\u5C06 CATEGORY_REGISTRY \u8FC1\u81F3 *_category_mount.py\uFF0C\u6D41\u7A0B\uFF1Ainit \u2192 seed \u2192 \u51FD\u6570 publish \u2192 category_mount`,
+      line: lineOf(code, applyMatch.index)
+    });
+  }
+  if (/^CATEGORY_REGISTRY\s*=/m.test(code) && applyMatch) {
+    issues.push({
+      severity: "warn",
+      rule: "init-category-registry-inline",
+      message: `${label}: CATEGORY_REGISTRY \u5E94\u653E\u5728\u72EC\u7ACB *_category_mount.py\uFF0C\u52FF\u4E0E init \u540C\u6587\u4EF6`
+    });
+  }
+  return issues;
+}
+function lintOntologyScriptCode(code, scriptType, filePath) {
+  if (scriptType === "ontology_function") {
+    return lintOntologyFunctionCode(code, filePath);
+  }
+  if (scriptType === "setup") {
+    return lintOntologySetupInitCode(code, filePath);
+  }
+  return [];
+}
+function formatLintReport(issues) {
+  if (!issues.length) return "";
+  return issues.map((i) => {
+    const tag = i.severity === "error" ? "\u2717" : "\u26A0";
+    const loc = i.line ? ` (\u884C ${i.line})` : "";
+    return `  ${tag} [${i.rule}]${loc} ${i.message}`;
+  }).join("\n");
+}
+function lintHasErrors(issues) {
+  return issues.some((i) => i.severity === "error");
+}
+function pathBasename(p) {
+  return p.replace(/\\/g, "/").split("/").pop() ?? p;
+}
+
+// cli/dazi-onto/src/lib/functionHelpers.ts
+function isParamsEmpty(params) {
+  if (params == null) return true;
+  if (typeof params !== "object") return false;
+  return Object.keys(params).length === 0;
+}
+async function resolveFunctionRunParams(fn, explicitParams, fetchScriptCode3) {
+  if (!isParamsEmpty(explicitParams)) {
+    return { params: explicitParams, source: "explicit" };
+  }
+  const ta = fn.test_arguments;
+  if (ta && typeof ta === "object" && !Array.isArray(ta)) {
+    const args = ta.arguments;
+    if (args && typeof args === "object" && Object.keys(args).length > 0) {
+      return { params: { ...args }, source: "test_arguments" };
+    }
+  }
+  const scriptId = fn.script_id ?? fn.adapter_config?.script_id;
+  if (scriptId) {
+    try {
+      const code = await fetchScriptCode3(scriptId);
+      const fromScript = extractTestArgumentsParams(code);
+      if (fromScript && Object.keys(fromScript).length > 0) {
+        return { params: fromScript, source: "script_test_arguments" };
+      }
+    } catch {
+    }
+  }
+  return { params: explicitParams };
+}
+function resolveFunctionDef(fns, idOrFunctionId) {
+  const key = idOrFunctionId.trim();
+  if (!key) return void 0;
+  const byRowId = fns.find((f) => f.id === key);
+  if (byRowId) return byRowId;
+  return fns.find((f) => f.function_id === key);
+}
+function isFunctionIdAlias(value) {
+  return value.includes(".fn.") || !value.startsWith("ofn_") && value.includes(".");
+}
+function enrichFunctionListItem(fn, spaceId) {
+  const fnId = fn.function_id ?? fn.id;
+  const jsonFile = `functions/test_arguments/${fnId}.json`;
+  const saveCommand = `dazi onto function save-test-arguments ${fn.id} --space ${spaceId} --arguments-json-file <\u9879\u76EE\u8DEF\u5F84>/${jsonFile}`;
+  const saveByAlias = fn.function_id ? `dazi onto function save-test-arguments --function-id ${fn.function_id} --space ${spaceId} --arguments-json-file <\u9879\u76EE\u8DEF\u5F84>/${jsonFile}` : saveCommand;
+  return {
+    ...fn,
+    save_command: saveByAlias
+  };
+}
+
+// cli/dazi-onto/src/commands/function.ts
+async function fetchScriptCode(scriptId) {
+  const detail = await apiRequest(`/api/scripts/${encodeURIComponent(scriptId)}`);
+  return detail.code ?? "";
+}
+async function listFunctionDefs(spaceId) {
+  return apiRequest(`/api/ontology-v2/spaces/${spaceId}/function-defs`);
+}
+function wantsJsonOutput(opts) {
+  return Boolean(opts.json || opts.output === "json");
+}
 function makeFunctionCommand() {
   const cmd = new Command("function").description("\u672C\u4F53\u51FD\u6570\u7BA1\u7406");
-  cmd.command("list").description("\u5217\u51FA\u51FD\u6570\u5B9A\u4E49").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--json", "\u8F93\u51FA JSON").action(async (opts) => {
+  cmd.command("list").description("\u5217\u51FA\u51FD\u6570\u5B9A\u4E49").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--json", "\u8F93\u51FA JSON").option("--output <format>", "\u8F93\u51FA\u683C\u5F0F\uFF08json\uFF0C\u4E0E --json \u7B49\u4EF7\uFF09").action(async (opts) => {
     try {
-      const fns = await apiRequest(`/api/ontology-v2/spaces/${opts.space}/function-defs`);
-      if (opts.json) {
-        console.log(JSON.stringify(fns, null, 2));
-        ok({ functions: fns });
+      const fns = await listFunctionDefs(opts.space);
+      const enriched = fns.map((f) => enrichFunctionListItem(f, opts.space));
+      if (wantsJsonOutput(opts)) {
+        console.log(JSON.stringify(enriched, null, 2));
+        ok({ functions: enriched });
         return;
       }
       if (!fns.length) {
@@ -3333,42 +3566,57 @@ function makeFunctionCommand() {
         ok({ functions: [] });
         return;
       }
-      for (const f of fns) {
+      for (const f of enriched) {
         const fnId = (f.function_id ?? f.id ?? "").padEnd(40);
         const name = (f.display_name ?? "").padEnd(24);
         const adapter = f.adapter ?? "dazi_script";
-        console.log(`  ${fnId} ${name} [${adapter}]`);
+        console.log(`  ${fnId} ${name} [${adapter}]  id=${f.id}`);
       }
-      ok({ functions: fns });
+      ok({ functions: enriched });
     } catch (err) {
       handleError(err);
     }
   });
-  cmd.command("get <functionId>").description("\u67E5\u770B\u51FD\u6570\u8BE6\u60C5").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").action(async (functionId, opts) => {
+  cmd.command("get <functionId>").description("\u67E5\u770B\u51FD\u6570\u8BE6\u60C5\uFF08\u652F\u6301 function_id \u6216 ofn \u884C id\uFF09").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").action(async (functionId, opts) => {
     try {
-      const fns = await apiRequest(`/api/ontology-v2/spaces/${opts.space}/function-defs`);
-      const fn = fns.find((f) => f.function_id === functionId);
+      const fns = await listFunctionDefs(opts.space);
+      const fn = resolveFunctionDef(fns, functionId);
       if (!fn) {
         console.error(`\u672A\u627E\u5230\u51FD\u6570: ${functionId}`);
         process.exit(1);
       }
-      console.log(JSON.stringify(fn, null, 2));
+      console.log(JSON.stringify(enrichFunctionListItem(fn, opts.space), null, 2));
       ok({ function: fn });
     } catch (err) {
       handleError(err);
     }
   });
-  cmd.command("run <functionId>").description("\u6267\u884C\u672C\u4F53\u51FD\u6570").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--params <json>", "\u53C2\u6570 JSON \u5B57\u7B26\u4E32", "{}").option("--dry-run", "\u4EC5\u6821\u9A8C\uFF0C\u4E0D\u5B9E\u9645\u6267\u884C").option("--json", "\u8F93\u51FA JSON").action(async (functionId, opts) => {
+  cmd.command("run <functionId>").description("\u6267\u884C\u672C\u4F53\u51FD\u6570\uFF08\u65E0 --params \u65F6\u56DE\u9000 test_arguments / \u811A\u672C TEST_ARGUMENTS\uFF09").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--params <json>", "\u53C2\u6570 JSON \u5B57\u7B26\u4E32", "{}").option("--dry-run", "\u4EC5\u6821\u9A8C\uFF0C\u4E0D\u5B9E\u9645\u6267\u884C").option("--json", "\u8F93\u51FA JSON").action(async (functionId, opts) => {
     try {
-      const params = JSON.parse(opts.params);
-      const fns = await apiRequest(`/api/ontology-v2/spaces/${opts.space}/function-defs`);
-      const fn = fns.find((f) => f.function_id === functionId);
-      const scriptId = fn?.script_id ?? fn?.adapter_config?.script_id;
+      const explicitParams = JSON.parse(opts.params);
+      const fns = await listFunctionDefs(opts.space);
+      const fn = resolveFunctionDef(fns, functionId);
+      if (!fn) {
+        console.error(`\u672A\u627E\u5230\u51FD\u6570: ${functionId}`);
+        process.exit(1);
+      }
+      const scriptId = fn.script_id ?? fn.adapter_config?.script_id;
       if (!scriptId) {
-        console.error(`\u672A\u627E\u5230\u51FD\u6570\u6216\u672A\u7ED1\u5B9A\u811A\u672C: ${functionId}`);
+        console.error(`\u672A\u627E\u5230\u51FD\u6570\u6216\u672A\u7ED1\u5B9A\u811A\u672C: ${fn.function_id ?? functionId}`);
         process.exit(1);
       }
-      console.log(`${opts.dryRun ? "[dry-run] " : ""}\u6267\u884C\u51FD\u6570: ${functionId} (script: ${scriptId})`);
+      const { params, source } = await resolveFunctionRunParams(
+        fn,
+        explicitParams,
+        fetchScriptCode
+      );
+      if (source === "test_arguments") {
+        console.log("\u2139 \u672A\u4F20 --params\uFF0C\u4F7F\u7528\u5DF2\u4FDD\u5B58 test_arguments.arguments");
+      } else if (source === "script_test_arguments") {
+        console.log("\u2139 \u672A\u4F20 --params\uFF0C\u56DE\u9000\u811A\u672C\u5185 TEST_ARGUMENTS.arguments");
+      }
+      const runFnId = fn.function_id ?? functionId;
+      console.log(`${opts.dryRun ? "[dry-run] " : ""}\u6267\u884C\u51FD\u6570: ${runFnId} (script: ${scriptId})`);
       const result = await apiRequest(
         `/api/scripts/${scriptId}/execute`,
         {
@@ -3376,7 +3624,7 @@ function makeFunctionCommand() {
           body: {
             params,
             dryRun: opts.dryRun ?? false,
-            ontology_function_id: functionId
+            ontology_function_id: runFnId
           }
         }
       );
@@ -3389,12 +3637,12 @@ function makeFunctionCommand() {
           result.logs.forEach((l) => console.log("  " + l));
         }
       }
-      ok({ functionId, result });
+      ok({ functionId: runFnId, paramsSource: source, result });
     } catch (err) {
       handleError(err);
     }
   });
-  cmd.command("publish <file>").description("\u53D1\u5E03\u51FD\u6570\u811A\u672C\u5230\u5E93").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--function-id <id>", "\u7ED1\u5B9A/\u8986\u76D6\u6307\u5B9A\u51FD\u6570 ID").option("--display-name <name>", "\u51FD\u6570\u663E\u793A\u540D").option("--entry <entryPoint>", "\u5165\u53E3\u51FD\u6570\u540D").option("--object-type-id <id>", "\u5BF9\u8C61\u7C7B\u578B ID").option("--dry-run", "\u4EC5\u9884\u68C0\uFF0C\u4E0D\u5199\u5E93").option("--json", "\u8F93\u51FA JSON").action(async (file, opts) => {
+  cmd.command("publish <file>").description("\u53D1\u5E03\u51FD\u6570\u811A\u672C\u5230\u5E93").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--function-id <id>", "\u7ED1\u5B9A/\u8986\u76D6\u6307\u5B9A\u51FD\u6570 ID").option("--display-name <name>", "\u51FD\u6570\u663E\u793A\u540D").option("--entry <entryPoint>", "\u5165\u53E3\u51FD\u6570\u540D").option("--object-type-id <id>", "\u5BF9\u8C61\u7C7B\u578B ID").option("--dry-run", "\u4EC5\u9884\u68C0\uFF0C\u4E0D\u5B9E\u9645\u6267\u884C").option("--json", "\u8F93\u51FA JSON").action(async (file, opts) => {
     try {
       const filePath = import_path4.default.resolve(file);
       if (!import_fs3.default.existsSync(filePath)) {
@@ -3411,9 +3659,8 @@ function makeFunctionCommand() {
         objectTypeId: opts.objectTypeId,
         dryRun: opts.dryRun ?? false
       };
-      const endpoint = opts.dryRun ? `/api/scripts` : `/api/scripts`;
       const result = await apiRequest(
-        endpoint,
+        `/api/scripts`,
         { method: "POST", body }
       );
       if (opts.json) {
@@ -3501,7 +3748,7 @@ function makeFunctionCommand() {
       if (opts.dryRun) params.set("dryRun", "true");
       const result = await apiRequest(
         `/api/ontology-v2/spaces/${opts.space}/function-defs?${params}`,
-        { method: "DELETE" }
+        { method: opts.dryRun ? "GET" : "DELETE", body: opts.dryRun ? void 0 : {} }
       );
       console.log(`${opts.dryRun ? "[dry-run] " : ""}\u5220\u9664 ${result.count} \u4E2A\u51FD\u6570`);
       ok({ count: result.count, dryRun: opts.dryRun });
@@ -3509,20 +3756,33 @@ function makeFunctionCommand() {
       handleError(err);
     }
   });
-  cmd.command("save-test-arguments <functionId>").description("\u4FDD\u5B58\u51FD\u6570\u6D4B\u8BD5\u53C2\u6570").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--params <json>", "\u53C2\u6570 JSON \u5B57\u7B26\u4E32", "{}").option("--arguments-json-file <file>", "\u4ECE JSON \u6587\u4EF6\u8BFB\u53D6\u53C2\u6570").action(async (functionId, opts) => {
+  cmd.command("save-test-arguments [functionDefId]").description("\u4FDD\u5B58\u51FD\u6570\u6D4B\u8BD5\u53C2\u6570\uFF08\u652F\u6301 ofn_xxx \u6216 --function-id domain.fn.name\uFF09").requiredOption("--space <spaceId>", "\u7A7A\u95F4 ID").option("--function-id <id>", "function_id \u522B\u540D\uFF08\u5982 sales.fn.get_summary\uFF09").option("--params <json>", "\u53C2\u6570 JSON \u5B57\u7B26\u4E32", "{}").option("--arguments-json-file <file>", "\u4ECE JSON \u6587\u4EF6\u8BFB\u53D6\u53C2\u6570").action(async (functionDefId, opts) => {
     try {
-      let params;
+      const lookupKey = (opts.functionId ?? functionDefId ?? "").trim();
+      if (!lookupKey) {
+        console.error("\u987B\u63D0\u4F9B ofn_xxx \u6216 --function-id <domain.fn.name>");
+        process.exit(1);
+      }
+      let payload;
       if (opts.argumentsJsonFile) {
-        params = JSON.parse(import_fs3.default.readFileSync(import_path4.default.resolve(opts.argumentsJsonFile), "utf-8"));
+        payload = JSON.parse(import_fs3.default.readFileSync(import_path4.default.resolve(opts.argumentsJsonFile), "utf-8"));
       } else {
-        params = JSON.parse(opts.params);
+        payload = JSON.parse(opts.params);
       }
-      await apiRequest(`/api/ontology-v2/spaces/${opts.space}/function-defs/${functionId}`, {
+      const fns = await listFunctionDefs(opts.space);
+      const fn = resolveFunctionDef(fns, lookupKey);
+      if (!fn) {
+        const hint = isFunctionIdAlias(lookupKey) ? "\u8BF7\u786E\u8BA4\u5DF2 publish --register-function-id" : "\u8BF7\u7528 function list \u67E5\u770B id \u5B57\u6BB5";
+        console.error(`\u672A\u627E\u5230\u51FD\u6570: ${lookupKey}\uFF08${hint}\uFF09`);
+        process.exit(1);
+      }
+      await apiRequest(`/api/ontology-v2/spaces/${opts.space}/function-defs/${fn.id}`, {
         method: "PATCH",
-        body: { test_arguments: params }
+        body: { test_arguments: payload }
       });
-      console.log(`\u2705 \u6D4B\u8BD5\u53C2\u6570\u5DF2\u4FDD\u5B58: ${functionId}`);
-      ok({ functionId, saved: true });
+      const label = fn.function_id ?? fn.id;
+      console.log(`\u2705 \u6D4B\u8BD5\u53C2\u6570\u5DF2\u4FDD\u5B58: ${label} (${fn.id})`);
+      ok({ functionId: fn.function_id, functionDefId: fn.id, saved: true });
     } catch (err) {
       handleError(err);
     }
@@ -3712,7 +3972,7 @@ function makeRuleCommand() {
 // cli/dazi-onto/src/commands/script.ts
 var import_path6 = __toESM(require("path"), 1);
 var import_fs5 = __toESM(require("fs"), 1);
-async function fetchScriptCode(scriptId) {
+async function fetchScriptCode2(scriptId) {
   const detail = await apiRequest(`/api/scripts/${encodeURIComponent(scriptId)}`);
   return detail.code ?? "";
 }
@@ -3762,6 +4022,18 @@ function buildSpacesRelPath(filePath, spaceId, scriptType) {
   const dir = workspaceDirForScriptType(scriptType);
   return `spaces/${spaceId}/editorial/scripts/${dir}/${stem}.py`;
 }
+function runOntologyScriptLocalLint(filePath, scriptType) {
+  const code = import_fs5.default.readFileSync(filePath, "utf-8");
+  const issues = lintOntologyScriptCode(code, scriptType, filePath);
+  if (!issues.length) return;
+  console.log("\u672C\u4F53\u811A\u672C\u9759\u6001\u9884\u68C0\uFF1A");
+  console.log(formatLintReport(issues));
+  if (lintHasErrors(issues)) {
+    console.error("\u2717 \u9884\u68C0\u5931\u8D25\uFF1A\u8BF7\u5148\u4FEE\u590D\u4E0A\u8FF0 error \u9879");
+    process.exit(1);
+  }
+  console.log("\u26A0 \u5B58\u5728 warn \u9879\uFF0C\u53EF\u7EE7\u7EED\u53D1\u5E03");
+}
 function buildPublishFromSpacesBody(filePath, opts) {
   const code = import_fs5.default.readFileSync(filePath, "utf-8");
   const scriptType = inferScriptTypeFromPath(filePath, opts.type);
@@ -3880,7 +4152,7 @@ function makeScriptCommand() {
           continue;
         }
         if (!import_fs5.default.existsSync(outDir)) import_fs5.default.mkdirSync(outDir, { recursive: true });
-        const code = s.code?.length ? s.code : await fetchScriptCode(s.id);
+        const code = s.code?.length ? s.code : await fetchScriptCode2(s.id);
         if (!code.trim()) {
           console.warn(`  \u26A0 \u8DF3\u8FC7 ${s.id}\uFF1A\u65E0\u811A\u672C\u4EE3\u7801`);
           continue;
@@ -3907,6 +4179,10 @@ function makeScriptCommand() {
         console.error(`\u6587\u4EF6\u4E0D\u5B58\u5728: ${filePath}`);
         process.exit(1);
       }
+      const scriptType = inferScriptTypeFromPath(filePath, opts.type);
+      if (opts.dryRun || scriptType === "ontology_function" || scriptType === "setup") {
+        runOntologyScriptLocalLint(filePath, scriptType);
+      }
       const body = buildPublishFromSpacesBody(filePath, opts);
       const endpoint = opts.dryRun ? "/api/scripts/example-files/publish-preview" : "/api/scripts/example-files/publish-from-spaces";
       const result = await apiRequest(endpoint, { method: "POST", body });
@@ -3934,9 +4210,11 @@ function makeScriptCommand() {
         console.error(`\u6587\u4EF6\u4E0D\u5B58\u5728: ${filePath}`);
         process.exit(1);
       }
+      const scriptType = inferScriptTypeFromPath(filePath, opts.type);
+      runOntologyScriptLocalLint(filePath, scriptType);
       const body = buildPublishFromSpacesBody(filePath, { ...opts, registerFunctionId: opts.registerFunctionId });
       const result = await apiRequest("/api/scripts/example-files/publish-preview", { method: "POST", body });
-      console.log("\u2705 \u9884\u68C0\u901A\u8FC7");
+      console.log("\u2705 \u670D\u52A1\u7AEF\u9884\u68C0\u901A\u8FC7");
       if (result && typeof result === "object") {
         console.log(JSON.stringify(result, null, 2));
       }