agent-inspect 6.31.18 → 6.31.19

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 6.31.19
+
+### Patch Changes
+
+- 8df3096: Fix residual recovery attemptNumber undercount and invalid result-availability times; promote OTLP agent_inspect.\* identity into workflow metadata on read; harden packed MCP SDK e2e (429/cancel/overlap/readiness); complete keyless Fix-Owes fixture corpus (Om through Thorin, Bezalel invariants).
+
 ## 6.31.18
 
 ### Patch Changes

package/README.md

@@ -213,7 +213,7 @@ The root package is enough for custom capture, the CLI, checks, and Evidence wor
 
 ## Status and documentation
 
-**Current published baseline:** **6.31.18** · persisted schema `1.0` · Node.js `>=20` · MIT.
+**Current published baseline:** **6.31.19** · persisted schema `1.0` · Node.js `>=20` · MIT.
 
 Legacy v0.1 and v0.2 traces remain readable. Check the npm badge and [changelog](CHANGELOG.md) for the current published version.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-inspect",
-  "version": "6.31.18",
+  "version": "6.31.19",
   "license": "MIT",
   "type": "module",
   "description": "Local evidence debugger and trajectory-test toolkit for TypeScript AI agents — execution trees, TraceContract checks, Evidence v2, and read-only MCP",

package/packages/cli/dist/{chunk-AHARYDFO.mjs → chunk-NALE36AJ.mjs}

@@ -3696,7 +3696,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);
@@ -3709,8 +3709,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);
@@ -3721,7 +3721,9 @@ function extractSessionWorkflowMetadata(record) {
     out.attempt = attempt;
     found = true;
   }
-  const attemptNumber = finitePositiveInt(record.attemptNumber);
+  const attemptNumber = finitePositiveInt(
+    record.attemptNumber ?? record["agent_inspect.attempt_number"]
+  );
   if (attemptNumber !== void 0) {
     out.attemptNumber = attemptNumber;
     found = true;
@@ -5751,12 +5753,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 num = attemptNumberOf(event);
+    if (num !== void 0) {
+      numbered.add(num);
+      continue;
+    }
+    unnumbered += 1;
   }
-  return unidentified;
+  if (numbered.size > 0) {
+    return numbered.size + unnumbered;
+  }
+  return unnumbered;
 }
 function classifyRetryAttempt(event, priorOrdered, allMembers) {
   const workflow = workflowFor(event);
@@ -6040,11 +6051,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);
 }
@@ -13607,6 +13630,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 = [];
@@ -13638,6 +13725,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}`);
@@ -13675,6 +13763,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"]);
@@ -15581,5 +15670,5 @@ function renderGateReport(result, options = {}) {
 }
 
 export { ATTRIBUTION_CONFIDENCES, COHORT_METRIC_IDS, DEFAULT_REDACT_KEYS, DEFAULT_SUITE_ARTIFACTS_DIR, EVIDENCE_FORMAT_VERSION, EVIDENCE_HTML_FILENAME, EVIDENCE_MANIFEST_FILENAME, Redactor, TraceDirectory, TraceReadError, TreeBuilder, aggregateBundleSafeStatus, aggregateSessionCheckResults, analyzeCohort, applyProfileMetadataCaps, assertBundlePathContained, assertEvidenceRelativePath, buildActivitySummary, buildBundleMetadata, buildBundleSummaryMarkdown, buildEvidenceCausalFailureViewHtml, buildEvidenceCiPackage, buildEvidenceCircuitViewHtml, buildEvidenceContractsViewHtml, buildEvidenceDiffViewHtml, buildEvidenceHtmlShell, buildEvidenceManifest, buildEvidenceOutcomesViewHtml, buildEvidenceProvenanceViewHtml, buildEvidenceSafetyViewHtml, buildEvidenceTimelineViewHtml, buildEvidenceToolsLlmViewHtml, buildEvidenceTreeViewHtml, buildLocalExplanation, buildPlaceholderArtifact, buildRunSummary, buildRunTimeline, buildRunWhatSummary, buildSessionIndex, buildTraceStats, buildZipArchive, bundleFailsOnSafety, bundleRunAssetRelativePath, collectTraceSchemaVersions, compactAttributes, createBaselineRegressionRule, createLlmUsageRule, createMaxStepDurationRule, createObservedOutcomeRule, createRequireCompletedRule, createRunDepthRule, createRunDurationRule, createRunStatusRule, createSafetyOversizedAttributeRule, createSafetyRawContentRule, createSafetyRedactionRule, createSafetySecretPatternRule, createStallDetectionRule, createStructureCycleRule, createStructureOrphanRule, createStructureParallelWidthRule, createStructureRelationshipRule, createToolUsageRule, defaultBundleOutputPath, defaultSuiteConfigTemplate, defineTraceContract, diffRuns, diffTraceEvents, enrichSessionRunRecord, escapeHtml, escapeMarkdown, evaluateTraceContract, extractMetadata, extractOutcomesFromTraceEvents, filterMetasBySessionScope, filterTraces, flattenTree, formatDuration2 as formatDuration, formatStepLabel, formatTimestamp, gateHasThresholds, getIndent, getTraceFilePath, inferEvidenceFileRole, isAgentInspectTrace, isAttributionConfidence, isCredentialSensitiveKey, isPersistedInspectEvent, loadSessionRunRecords, loadSuiteConfig, loadTraceMetadataList, manualTraceEventsToComparableRun, nanoid, normalizeBundleOutputPath, openTrace, parseCohortMetricList, parseDuration, parseDurationFilter, parseGateList, parseTraceJsonl, persistedInspectEventsToRunTrees, persistedInspectEventsToTraceEvents, projectLogicalEvents, renderActivitySummaryHuman, renderCohortReport, renderErrorLine, renderGateReport, renderObservedOutcomesHtml, renderObservedOutcomesMarkdown, renderRunDiff, renderRunWhat, renderStepLine, renderSuiteReport, renderTimeline, renderTraceStats, resolveBundleRunIds, resolveRedactionProfile, resolveSuiteTemplate, resolveTraceDir, runGate, runSuite, runTraceChecks, safeString, sanitizeBundleRunId, searchTraces, serializeEvidenceManifest, sha256Hex, stableJson, stringContainsHighConfidenceCredential, summarizeObservedOutcomes, summarizeSemanticParity, traceEventToPersistedInspectEvent, truncateName, truncateStringForProfile, validateEvent, validateSuiteConfig, verifyEvidenceDirectory, zeroKinds };
-//# sourceMappingURL=chunk-AHARYDFO.mjs.map
-//# sourceMappingURL=chunk-AHARYDFO.mjs.map
+//# sourceMappingURL=chunk-NALE36AJ.mjs.map
+//# sourceMappingURL=chunk-NALE36AJ.mjs.map