agent-inspect 6.17.1 → 6.17.3

package/packages/cli/dist/index.cjs

@@ -2765,7 +2765,12 @@ function countErrorNodes(nodes) {
   return nodes.filter((entry) => entry.node.event.status === "error").length;
 }
 function slowestNode(nodes) {
-  return nodes.filter((entry) => entry.node.event.durationMs !== void 0).sort((a, b) => {
+  return nodes.filter(
+    (entry) => entry.node.event.durationMs !== void 0 && // The RUN boundary spans the whole run, so its duration always dominates
+    // and shadows every real step. Rank actual work nodes only, matching the
+    // stats slowest-step ranking, which never counts the run envelope.
+    entry.node.event.kind !== "RUN"
+  ).sort((a, b) => {
     const delta = (b.node.event.durationMs ?? 0) - (a.node.event.durationMs ?? 0);
     return delta !== 0 ? delta : a.index - b.index;
   })[0];
@@ -7036,6 +7041,18 @@ function stringAttr(event, keys) {
   }
   return void 0;
 }
+function metadataStringAttr(event, keys) {
+  const metadata = event.attributes?.metadata;
+  if (typeof metadata !== "object" || metadata === null || Array.isArray(metadata)) {
+    return void 0;
+  }
+  const record = metadata;
+  for (const key of keys) {
+    const value = record[key];
+    if (typeof value === "string" && value.trim() !== "") return value;
+  }
+  return void 0;
+}
 function numericAttr(event, keys) {
   for (const key of keys) {
     const value = event.attributes?.[key];
@@ -7094,13 +7111,16 @@ function semanticEvents(context) {
   return context.logicalEvents ?? context.events;
 }
 function llmModel(event) {
-  return stringAttr(event, ["model", "modelId", "responseModelId", "modelName", "model_name"]) ?? stripPrefix(event.name, ["llm:", "generation:", "transcription:", "speech:"]);
+  const keys = ["model", "modelId", "responseModelId", "modelName", "model_name"];
+  return stringAttr(event, keys) ?? metadataStringAttr(event, keys) ?? stripPrefix(event.name, ["llm:", "generation:", "transcription:", "speech:"]);
 }
 function llmProvider(event) {
-  return stringAttr(event, ["provider", "providerName", "provider_name"]);
+  const keys = ["provider", "providerName", "provider_name"];
+  return stringAttr(event, keys) ?? metadataStringAttr(event, keys);
 }
 function llmFinishReason(event) {
-  return stringAttr(event, ["finishReason", "rawFinishReason", "finish_reason"]);
+  const keys = ["finishReason", "rawFinishReason", "finish_reason"];
+  return stringAttr(event, keys) ?? metadataStringAttr(event, keys);
 }
 function retryCount(event) {
   return numericAttr(event, ["retryCount", "retryAttempt", "retry_attempt", "attempt"]);
@@ -12486,7 +12506,7 @@ var init_src = __esm({
 });
 
 // package.json
-var version = "6.17.1";
+var version = "6.17.3";
 
 // packages/cli/src/list.ts
 init_advanced();
@@ -18347,6 +18367,58 @@ function mergeSafetyExtensions(result, read, options) {
 }
 
 // packages/cli/src/check.ts
+var DEFAULT_SELECT = ["run.status"];
+function uniqueSelectIds(ids) {
+  const seen = /* @__PURE__ */ new Set();
+  const out = [];
+  for (const id of ids) {
+    const trimmed = id.trim();
+    if (trimmed === "" || seen.has(trimmed)) continue;
+    seen.add(trimmed);
+    out.push(trimmed);
+  }
+  return out;
+}
+function cliShorthandSelectIds(options) {
+  const ids = [];
+  if (options.failOnObservation !== void 0 && options.failOnObservation.trim() !== "") {
+    ids.push("outcome.status");
+  }
+  if ((options.requiredTool?.length ?? 0) > 0 || (options.forbiddenTool?.length ?? 0) > 0) {
+    ids.push("tool.usage");
+  }
+  if ((options.allowedModel?.length ?? 0) > 0 || options.maxTotalTokens !== void 0) {
+    ids.push("llm.usage");
+  }
+  if (options.maxDurationMs !== void 0) {
+    ids.push("run.duration");
+  }
+  if (options.maxStepDuration !== void 0) {
+    ids.push("run.maxStepDuration");
+  }
+  if (options.detectStalls === true) {
+    ids.push("run.stall");
+  }
+  return ids;
+}
+function unionCheckSelect(input3) {
+  const explicit = uniqueSelectIds([
+    ...input3.presetSelect ?? [],
+    ...input3.explicitRules ?? [],
+    ...input3.configSelect ?? []
+  ]);
+  if (explicit.length === 0) {
+    return uniqueSelectIds([
+      ...DEFAULT_SELECT,
+      ...(input3.constructedRuleIds ?? []).filter((id) => id !== "run.status")
+    ]);
+  }
+  const constructed = new Set(input3.constructedRuleIds ?? []);
+  const shorthand = (input3.shorthandIds ?? []).filter(
+    (id) => constructed.size === 0 || constructed.has(id)
+  );
+  return uniqueSelectIds([...explicit, ...shorthand]);
+}
 function resolvePreset(preset, context = {}) {
   if (preset === void 0 || preset.trim() === "") return void 0;
   const name = preset.trim().toLowerCase();
@@ -18395,7 +18467,6 @@ function resolvePreset(preset, context = {}) {
     select
   };
 }
-var DEFAULT_SELECT = ["run.status"];
 var CONFIG_EXTENSIONS2 = /* @__PURE__ */ new Set([".json", ".js", ".mjs", ".cjs"]);
 var TS_CONFIG_EXTENSIONS2 = /* @__PURE__ */ new Set([".ts", ".mts", ".cts"]);
 function diagnostic5(code, message, severity = "error") {
@@ -18487,12 +18558,11 @@ function applyResolvedPreset(config, options, resolved) {
     config: { checks: checks2 },
     options: {
       ...options,
-      ...resolved.requireCompleted ? { requireCompleted: true } : {},
-      rule: [...resolved.select, ...options.rule ?? []]
+      ...resolved.requireCompleted ? { requireCompleted: true } : {}
     }
   };
 }
-function buildRules(config, options) {
+function buildRules(config, options, presetSelect = []) {
   const diagnostics = [];
   const checks2 = normalizeConfig(config);
   const run = checks2.run ?? {};
@@ -18582,21 +18652,15 @@ function buildRules(config, options) {
   if (safety.maxStringLength !== void 0 || safety.maxArrayLength !== void 0 || safety.maxObjectKeys !== void 0 || safety.maxSerializedBytes !== void 0) {
     rules.push(createSafetyOversizedAttributeRule(safety));
   }
-  const select = [
-    ...asStringArray2(checks2.select) ?? [],
-    ...options.rule ?? []
-  ];
-  if (select.length === 0) {
-    const auto = new Set(DEFAULT_SELECT);
-    for (const rule of rules) {
-      if (rule.id !== "run.status") auto.add(rule.id);
-    }
-    return {
-      rules,
-      select: [...auto],
-      diagnostics
-    };
-  }
+  const constructedRuleIds = rules.map((rule) => rule.id);
+  const shorthandIds = cliShorthandSelectIds(options);
+  const select = unionCheckSelect({
+    presetSelect,
+    explicitRules: options.rule ?? [],
+    configSelect: asStringArray2(checks2.select) ?? [],
+    shorthandIds,
+    constructedRuleIds
+  });
   return {
     rules,
     select,
@@ -18725,7 +18789,7 @@ async function checkCommand(target, options = {}, stdin = process.stdin) {
       config = applied.config;
       effectiveOptions = applied.options;
     }
-    const built = buildRules(config, effectiveOptions);
+    const built = buildRules(config, effectiveOptions, resolved?.select ?? []);
     if (built.diagnostics.some((item) => item.severity === "error")) {
       result = errorResult2("AI_CHECK_INVALID_CONFIG", "Invalid check configuration.");
       result.diagnostics = [...built.diagnostics];