agent-inspect 6.31.18 → 6.31.20

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;
@@ -8180,12 +8182,21 @@ function countOperationAttempts(members) {
   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)];
-    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;
+    }
+    unnumbered += 1;
+  }
+  if (numbered.size > 0) {
+    return numbered.size + unnumbered;
   }
-  return unidentified;
+  return unnumbered;
 }
 function classifyRetryAttempt(event, priorOrdered, allMembers) {
   const workflow = workflowFor(event);
@@ -8475,11 +8486,23 @@ function consumerStartMs(event) {
   return parseComparableMs(event.startedAt) ?? parseComparableMs(event.timestamp);
 }
 function resultAvailableMs(event) {
-  const ended = parseComparableMs(event.endedAt);
-  if (ended !== void 0) return ended;
   const started = parseComparableMs(event.startedAt) ?? parseComparableMs(event.timestamp);
-  if (started !== void 0 && event.durationMs !== void 0 && Number.isFinite(event.durationMs)) {
-    return started + event.durationMs;
+  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);
 }
@@ -8505,6 +8528,13 @@ function errorCodeOf(event) {
     const code = nested.code;
     if (typeof code === "string" && code.trim() !== "") return code.trim();
   }
+  const meta2 = attrs.metadata;
+  if (meta2 && typeof meta2 === "object") {
+    for (const key of ["errorCode", "code"]) {
+      const value = meta2[key];
+      if (typeof value === "string" && value.trim() !== "") return value.trim();
+    }
+  }
   return void 0;
 }
 function argumentDigestOf(event) {
@@ -14132,6 +14162,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 = [];
@@ -14163,6 +14257,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}`);
@@ -14200,6 +14295,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"]);
@@ -16864,7 +16960,7 @@ var init_src = __esm({
 });
 
 // package.json
-var version = "6.31.18";
+var version = "6.31.20";
 
 // packages/cli/src/list.ts
 init_advanced();