modelfunction-cli 0.4.0 → 0.4.1

package/lib/js-tools.js

@@ -115,7 +115,11 @@ function toolFromObject(node, ctx, preset = {}) {
     node.props.has("isEnabled") ||
     node.props.has("searchHint");
   if (!description && Object.keys(schema.properties ?? {}).length === 0 && !toolish) return undefined;
-  return { name, description, parameters: schema };
+  // Claude-style tools build the description in `async description()` or a
+  // prompt template; lint must not call that "missing".
+  const descNode = node.props.get("description");
+  const runtimeDescription = !description && (descNode?.t === "unknown" || node.props.has("prompt"));
+  return { name, description, parameters: schema, ...(runtimeDescription ? { descriptionSource: "runtime" } : {}) };
 }
 
 function isSchema(value) {
@@ -601,7 +605,14 @@ function parseObject(src, start, depth) {
       const skipped = skipToValueEnd(src, i);
       if (skipped === null) {
       return null; }
-      props.set(key, { t: "unknown" });
+      // `async description() {...}` — record the method's real name so lint can
+      // tell "description generated at runtime" apart from "no description".
+      let methodName = key;
+      if ((key === "async" || key === "get" || key === "set") && /[A-Za-z_$]/.test(src[i] ?? "")) {
+        const inner = readIdent(src, i);
+        if (inner && src[skipTrivia(src, inner.end)] === "(") methodName = inner.name;
+      }
+      props.set(methodName, { t: "unknown" });
       i = skipped;
       if (src[i] === ",") {
         i += 1;

package/lib/lint.js

@@ -91,7 +91,11 @@ export function lintSuite(suite) {
     const description = (tool.description ?? "").trim();
     const template = `建议：\`${tool.name}：当用户需要____时使用。输入____，返回____。不适用于____。\``;
 
-    if (!description) {
+    if (tool.descriptionSource === "runtime") {
+      // The description exists at runtime but is assembled in code, so static
+      // review cannot judge its wording. Not a defect.
+      add("info", "desc-runtime", tool.name, "description 由代码在运行时生成，静态不可评；请确认运行时输出包含触发时机与边界");
+    } else if (!description) {
       add("error", "desc-missing", tool.name, "工具没有 description，模型无法判断何时调用", { suggestion: template });
     } else {
       if (description.length < 20) {
@@ -107,8 +111,10 @@ export function lintSuite(suite) {
       }
     }
 
+    // Naming style is an ecosystem convention, not a model-facing defect:
+    // Claude-style PascalCase agents work fine on their own. Info only.
     if (!/^[a-z][a-z0-9_]*$/.test(tool.name)) {
-      add("warn", "name-style", tool.name, "工具名建议用小写 snake_case");
+      add("info", "name-style", tool.name, "工具名不是小写 snake_case；对外分发（MCP/插件市场）时建议统一");
     }
     if (GENERIC_NAMES.has(tool.name)) {
       add("warn", "name-generic", tool.name, "工具名过于通用，容易和其它工具混淆", {

package/lib/scan.js

@@ -348,7 +348,8 @@ export function buildSuiteDraft(report) {
     tools: report.tools.map((tool) => ({
       name: tool.name,
       description: tool.description,
-      parameters: tool.parameters
+      parameters: tool.parameters,
+      ...(tool.descriptionSource ? { descriptionSource: tool.descriptionSource } : {})
     })),
     cases: []
   };

package/lib/suite.js

@@ -54,7 +54,8 @@ async function normalizeTool(entry, baseDir) {
     description: entry.description ?? "",
     parameters: entry.parameters ?? entry.input ?? EMPTY_SCHEMA,
     source: null,
-    sideEffects: normalizeSideEffects(entry.sideEffects ?? entry.side_effects)
+    sideEffects: normalizeSideEffects(entry.sideEffects ?? entry.side_effects),
+    ...(entry.descriptionSource === "runtime" ? { descriptionSource: "runtime" } : {})
   };
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "modelfunction-cli",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "type": "module",
   "license": "MIT",
   "description": "mf — function-calling accuracy scorecard for AI agents",
@@ -25,7 +25,7 @@
     "LICENSE"
   ],
   "scripts": {
-    "test": "node --test \"test/**/*.test.js\"",
+    "test": "node --test --test-concurrency=1 \"test/**/*.test.js\"",
     "eval": "node mf.js eval",
     "eval:mock": "node mf.js eval --models mock --show-cases",
     "pack:check": "npm pack --dry-run",