agent-inspect 6.31.17 → 6.31.19

package/packages/cli/dist/index.cjs

@@ -3851,7 +3851,7 @@ function extractSessionWorkflowMetadata(record) {
   assignString("conversationId", record.conversationId);
   assignString("groupId", record.groupId);
   assignString("parentGroupId", record.parentGroupId);
-  assignString("retryOf", record.retryOf);
+  assignString("retryOf", record.retryOf ?? record["agent_inspect.retry_of"]);
   assignString("retryReason", record.retryReason);
   assignString("handoffFrom", record.handoffFrom);
   assignString("handoffTo", record.handoffTo);
@@ -3864,8 +3864,8 @@ function extractSessionWorkflowMetadata(record) {
   assignString("toolCallId", record.toolCallId);
   assignString("mcpToolCallId", record.mcpToolCallId);
   assignString("linkedStepId", record.linkedStepId);
-  assignString("operationId", record.operationId);
-  assignString("attemptId", record.attemptId);
+  assignString("operationId", record.operationId ?? record["agent_inspect.operation_id"]);
+  assignString("attemptId", record.attemptId ?? record["agent_inspect.attempt_id"]);
   assignString("fallbackOf", record.fallbackOf);
   assignString("idempotencyKey", record.idempotencyKey);
   assignString("correlationId", record.correlationId);
@@ -3876,7 +3876,9 @@ function extractSessionWorkflowMetadata(record) {
     out.attempt = attempt;
     found = true;
   }
-  const attemptNumber2 = finitePositiveInt(record.attemptNumber);
+  const attemptNumber2 = finitePositiveInt(
+    record.attemptNumber ?? record["agent_inspect.attempt_number"]
+  );
   if (attemptNumber2 !== void 0) {
     out.attemptNumber = attemptNumber2;
     found = true;
@@ -8166,24 +8168,35 @@ function attemptIdOf(event) {
   return typeof value === "string" && value.trim() !== "" ? value : void 0;
 }
 function countOperationAttempts(members) {
-  const attemptIds = new Set(
-    members.map((event) => attemptIdOf(event)).filter((value) => value !== void 0)
-  );
-  if (attemptIds.size > 0) {
-    return attemptIds.size;
+  const identified = /* @__PURE__ */ new Set();
+  let unidentified = 0;
+  for (const event of members) {
+    if (event.kind !== "TOOL" && event.kind !== "LLM") continue;
+    const id = attemptIdOf(event);
+    if (id !== void 0) {
+      identified.add(id);
+      continue;
+    }
+    unidentified += 1;
+  }
+  if (identified.size > 0) {
+    return identified.size + unidentified;
   }
-  const numbers = members.map((event) => attemptNumberOf(event)).filter((value) => value !== void 0).sort((a, b) => a - b);
-  if (numbers.length > 0) {
-    const unique = [...new Set(numbers)];
-    const min = unique[0];
-    const max = unique[unique.length - 1];
-    const contiguous = unique.length === max - min + 1 && unique.every((value, index) => value === min + index);
-    if (contiguous) {
-      return unique.length;
+  const numbered = /* @__PURE__ */ new Set();
+  let unnumbered = 0;
+  for (const event of members) {
+    if (event.kind !== "TOOL" && event.kind !== "LLM") continue;
+    const num2 = attemptNumberOf(event);
+    if (num2 !== void 0) {
+      numbered.add(num2);
+      continue;
     }
-    return unique.length;
+    unnumbered += 1;
   }
-  return members.filter((event) => event.kind === "TOOL" || event.kind === "LLM").length;
+  if (numbered.size > 0) {
+    return numbered.size + unnumbered;
+  }
+  return unnumbered;
 }
 function classifyRetryAttempt(event, priorOrdered, allMembers) {
   const workflow = workflowFor(event);
@@ -8464,8 +8477,42 @@ function eventEvidence3(event) {
 function eventTime2(event) {
   return event.startedAt ?? event.timestamp ?? "";
 }
+function parseComparableMs(value) {
+  if (!value) return void 0;
+  const parsed = Date.parse(value);
+  return Number.isFinite(parsed) ? parsed : void 0;
+}
+function consumerStartMs(event) {
+  return parseComparableMs(event.startedAt) ?? parseComparableMs(event.timestamp);
+}
+function resultAvailableMs(event) {
+  const started = parseComparableMs(event.startedAt) ?? parseComparableMs(event.timestamp);
+  const endedRaw = event.endedAt;
+  const endedPresent = endedRaw !== void 0 && endedRaw !== null && String(endedRaw).trim() !== "";
+  if (endedPresent) {
+    const ended = parseComparableMs(typeof endedRaw === "string" ? endedRaw : String(endedRaw));
+    if (ended === void 0) return void 0;
+    if (started !== void 0 && ended < started) return void 0;
+    return ended;
+  }
+  if (event.durationMs !== void 0) {
+    if (!Number.isFinite(event.durationMs) || event.durationMs < 0) {
+      return void 0;
+    }
+    if (started !== void 0) {
+      return started + event.durationMs;
+    }
+    return void 0;
+  }
+  return parseComparableMs(event.timestamp);
+}
 function sortByTime2(events) {
-  return [...events].sort((a, b) => eventTime2(a).localeCompare(eventTime2(b)));
+  return [...events].sort((a, b) => {
+    const aMs = consumerStartMs(a) ?? 0;
+    const bMs = consumerStartMs(b) ?? 0;
+    if (aMs !== bMs) return aMs - bMs;
+    return eventTime2(a).localeCompare(eventTime2(b));
+  });
 }
 function errorCodeOf(event) {
   if (typeof event.error?.code === "string" && event.error.code.trim() !== "") {
@@ -8537,9 +8584,10 @@ function isUnevaluableWriteCompletion(event) {
   if (event.status === "running" || event.status === "unknown") return true;
   const attrs = event.attributes ?? {};
   if (attrs.timeout === true) return true;
+  if (attrs.unknownCompletion === true) return true;
   if (attrs.completionState === "unknown" || attrs.completionState === "timeout") return true;
   const code = errorCodeOf(event)?.toUpperCase();
-  if (code === "TIMEOUT" || code === "UNKNOWN_COMPLETION" || code === "DEADLINE_EXCEEDED") {
+  if (code && UNEVALUABLE_WRITE_ERROR_CODES.has(code)) {
     return true;
   }
   return false;
@@ -8587,14 +8635,17 @@ function collectExplicitReferences(event) {
   return refs;
 }
 function llmReferencesTool(llm, toolEvent, requireExplicit) {
-  if (!requireExplicit) {
-    return eventTime2(llm) >= eventTime2(toolEvent);
-  }
-  const refs = collectExplicitReferences(llm);
-  if (refs.has(toolEvent.eventId)) return true;
-  const toolCallId = workflowFor2(toolEvent).toolCallId;
-  if (toolCallId && refs.has(toolCallId)) return true;
-  return false;
+  if (llm.runId !== toolEvent.runId) return false;
+  if (requireExplicit) {
+    const refs = collectExplicitReferences(llm);
+    const toolCallId = workflowFor2(toolEvent).toolCallId;
+    const hasRef = refs.has(toolEvent.eventId) || toolCallId !== void 0 && refs.has(toolCallId);
+    if (!hasRef) return false;
+  }
+  const resultMs = resultAvailableMs(toolEvent);
+  const consumerMs = consumerStartMs(llm);
+  if (resultMs === void 0 || consumerMs === void 0) return false;
+  return resultMs <= consumerMs;
 }
 function attemptIdOf2(event) {
   const value = workflowFor2(event).attemptId;
@@ -8906,6 +8957,7 @@ function evaluateRecoveryOperations(events, operations) {
   }
   return findings;
 }
+var UNEVALUABLE_WRITE_ERROR_CODES;
 var init_recovery_operations = __esm({
   "packages/core/src/checks/recovery-operations.ts"() {
     init_canonical_equality();
@@ -8913,6 +8965,12 @@ var init_recovery_operations = __esm({
     init_tool_arguments();
     init_retry_safety();
     init_metadata();
+    UNEVALUABLE_WRITE_ERROR_CODES = /* @__PURE__ */ new Set([
+      "TIMEOUT",
+      "ETIMEDOUT",
+      "UNKNOWN_COMPLETION",
+      "DEADLINE_EXCEEDED"
+    ]);
   }
 });
 
@@ -14097,6 +14155,70 @@ function mapOtlpEvents(value, pathPrefix) {
     unsupportedFields
   };
 }
+function promoteAgentInspectWorkflowIdentity(attributes, pathPrefix) {
+  const warnings = [];
+  const mapping = [
+    ["agent_inspect.operation_id", "operationId", "string"],
+    ["agent_inspect.attempt_id", "attemptId", "string"],
+    ["agent_inspect.retry_of", "retryOf", "string"],
+    ["agent_inspect.attempt_number", "attemptNumber", "int"]
+  ];
+  const metadata = attributes.metadata && typeof attributes.metadata === "object" && !Array.isArray(attributes.metadata) ? { ...attributes.metadata } : {};
+  for (const [dotted, camel, kind] of mapping) {
+    const raw = attributes[dotted];
+    if (raw === void 0 || raw === null) continue;
+    let promoted;
+    if (kind === "int") {
+      const n = typeof raw === "number" ? raw : typeof raw === "string" ? Number(raw) : NaN;
+      if (!Number.isFinite(n) || n < 1) {
+        warnings.push({
+          code: "otlp_identity_invalid",
+          message: `Ignoring invalid ${dotted} identity value.`,
+          severity: "warning",
+          field: `${pathPrefix}.${dotted}`
+        });
+        continue;
+      }
+      promoted = Math.trunc(n);
+    } else if (typeof raw === "string" && raw.trim() !== "") {
+      promoted = raw.trim();
+    } else {
+      warnings.push({
+        code: "otlp_identity_invalid",
+        message: `Ignoring non-string ${dotted} identity value.`,
+        severity: "warning",
+        field: `${pathPrefix}.${dotted}`
+      });
+      continue;
+    }
+    const direct = attributes[camel];
+    const nested = metadata[camel];
+    if (direct !== void 0 && direct !== promoted) {
+      warnings.push({
+        code: "otlp_identity_conflict",
+        message: `Conflicting ${camel}: attributes.${camel} vs ${dotted}; keeping camelCase.`,
+        severity: "warning",
+        field: `${pathPrefix}.${dotted}`
+      });
+      continue;
+    }
+    if (nested !== void 0 && nested !== promoted && direct === void 0) {
+      warnings.push({
+        code: "otlp_identity_conflict",
+        message: `Conflicting ${camel}: attributes.metadata.${camel} vs ${dotted}; keeping nested.`,
+        severity: "warning",
+        field: `${pathPrefix}.${dotted}`
+      });
+      continue;
+    }
+    if (direct === void 0) attributes[camel] = promoted;
+    if (nested === void 0) metadata[camel] = promoted;
+  }
+  if (Object.keys(metadata).length > 0) {
+    attributes.metadata = metadata;
+  }
+  return warnings;
+}
 function mapOtlpSpan(context) {
   const { span, pathPrefix } = context;
   const warnings = [];
@@ -14128,6 +14250,7 @@ function mapOtlpSpan(context) {
   if (context.scopeVersion !== void 0) {
     attributes["scope.version"] = context.scopeVersion;
   }
+  warnings.push(...promoteAgentInspectWorkflowIdentity(attributes, pathPrefix));
   for (const [key, value] of Object.entries(span)) {
     if (OTLP_SPAN_KEYS.has(key)) continue;
     unsupportedFields.push(`${pathPrefix}.${key}`);
@@ -14165,6 +14288,7 @@ function mapOtlpSpan(context) {
   if (events.events !== void 0) {
     attributes["otlp.events"] = events.events;
   }
+  warnings.push(...promoteAgentInspectWorkflowIdentity(attributes, pathPrefix));
   const traceId = readStringField(span, ["traceId"]) ?? "trace-unknown";
   const spanId = readStringField(span, ["spanId"]) ?? "span-unknown";
   const parentSpanId = readStringField(span, ["parentSpanId"]);
@@ -16829,7 +16953,7 @@ var init_src = __esm({
 });
 
 // package.json
-var version = "6.31.17";
+var version = "6.31.19";
 
 // packages/cli/src/list.ts
 init_advanced();
@@ -19480,6 +19604,51 @@ function genAiOperationName(kind) {
       return void 0;
   }
 }
+var STRUCTURAL_IDENTITY_KEYS = [
+  "operationId",
+  "attemptId",
+  "retryOf",
+  "attemptNumber",
+  "attempt"
+];
+function pickBoundedIdentityString(bag, key, maxLen) {
+  if (!bag || typeof bag !== "object") return void 0;
+  const direct = bag[key];
+  if (typeof direct === "string" && direct.trim() !== "") {
+    return direct.trim().slice(0, maxLen);
+  }
+  const nested = bag.metadata && typeof bag.metadata === "object" ? bag.metadata[key] : void 0;
+  if (typeof nested === "string" && nested.trim() !== "") {
+    return nested.trim().slice(0, maxLen);
+  }
+  return void 0;
+}
+function pickBoundedIdentityNumber(bag, key) {
+  if (!bag || typeof bag !== "object") return void 0;
+  const direct = bag[key];
+  if (typeof direct === "number" && Number.isFinite(direct)) return direct;
+  const nested = bag.metadata && typeof bag.metadata === "object" ? bag.metadata[key] : void 0;
+  if (typeof nested === "number" && Number.isFinite(nested)) return nested;
+  return void 0;
+}
+function appendStructuralIdentityAttrs(attrs, meta2, maxLen) {
+  const operationId = pickBoundedIdentityString(meta2, "operationId", maxLen);
+  if (operationId !== void 0) {
+    attrs.push(stringAttr2("agent_inspect.operation_id", operationId));
+  }
+  const attemptId = pickBoundedIdentityString(meta2, "attemptId", maxLen);
+  if (attemptId !== void 0) {
+    attrs.push(stringAttr2("agent_inspect.attempt_id", attemptId));
+  }
+  const retryOf = pickBoundedIdentityString(meta2, "retryOf", maxLen);
+  if (retryOf !== void 0) {
+    attrs.push(stringAttr2("agent_inspect.retry_of", retryOf));
+  }
+  const attemptNumber2 = pickBoundedIdentityNumber(meta2, "attemptNumber") ?? pickBoundedIdentityNumber(meta2, "attempt");
+  if (attemptNumber2 !== void 0) {
+    attrs.push(intAttr("agent_inspect.attempt_number", attemptNumber2));
+  }
+}
 function resolveSpanTimes(ev) {
   const startedAtMs = typeof ev.attributes?.startedAtMs === "number" && Number.isFinite(ev.attributes.startedAtMs) ? ev.attributes.startedAtMs : void 0;
   const endedAtMs = typeof ev.attributes?.endedAtMs === "number" && Number.isFinite(ev.attributes.endedAtMs) ? ev.attributes.endedAtMs : void 0;
@@ -19500,7 +19669,8 @@ function exportOtlpJson(tree, options) {
   const warnings = [
     "OTLP JSON export uses OTel GenAI-aligned attributes where applicable; experimental until verified against specific collectors.",
     "Not OTLP gRPC/protobuf \u2014 JSON mapping only. Generated locally; no network upload.",
-    "Status and span kind use OTLP numeric enums (StatusCode 0/1/2, SpanKind INTERNAL=1)."
+    "Status and span kind use OTLP numeric enums (StatusCode 0/1/2, SpanKind INTERNAL=1).",
+    "When present on source events, operationId/attemptId/retryOf/attemptNumber are emitted as bounded agent_inspect.* identity attributes (not a broad attribute dump)."
   ];
   const traceId = hexFrom2(`trace:${tree.runId}`, 16);
   const includeAttributes = options?.includeAttributes ?? false;
@@ -19548,6 +19718,11 @@ function exportOtlpJson(tree, options) {
       if (typeof inp === "number") attrs.push(intAttr("gen_ai.usage.input_tokens", inp));
       if (typeof outp === "number") attrs.push(intAttr("gen_ai.usage.output_tokens", outp));
     }
+    appendStructuralIdentityAttrs(
+      attrs,
+      meta2 && typeof meta2 === "object" ? meta2 : void 0,
+      maxLen
+    );
     if (includeAttributes && meta2 && typeof meta2 === "object") {
       const skipKeys = /* @__PURE__ */ new Set([
         "tokens",
@@ -19556,7 +19731,8 @@ function exportOtlpJson(tree, options) {
         "responseModel",
         "responseModelId",
         "provider",
-        "originalSourceType"
+        "originalSourceType",
+        ...STRUCTURAL_IDENTITY_KEYS
       ]);
       for (const [k, v] of Object.entries(meta2)) {
         if (skipKeys.has(k)) continue;