@open-agent-toolkit/cli 0.2.54 → 0.2.55

package/assets/skills/recon/scripts/lib/validated-run.mjs

@@ -14,18 +14,12 @@ function deepFreeze(value) {
   return Object.freeze(value);
 }
 
-function normalizeTopology(topology) {
-  return {
-    requiredLanes: clone(topology.requiredLanes),
-    stages: [...topology.stageByLane.entries()]
-      .map(([laneId, stage]) => ({ laneId, stage: clone(stage) }))
-      .sort((left, right) => left.laneId.localeCompare(right.laneId)),
-    completeArtifactIdsByMode: Object.fromEntries(
-      [...topology.completeArtifactIdsByMode.entries()]
-        .sort(([left], [right]) => left.localeCompare(right))
-        .map(([mode, ids]) => [mode, [...ids].sort()]),
-    ),
-  };
+function normalizePasses(passes) {
+  return Object.fromEntries(
+    [...passes.entries()]
+      .sort(([left], [right]) => left.localeCompare(right))
+      .map(([mode, ids]) => [mode, [...ids].sort()]),
+  );
 }
 
 export function createValidatedRun({
@@ -36,9 +30,9 @@ export function createValidatedRun({
   ledger,
   artifactsById,
   exactEvidence,
-  topology,
+  passes,
   achievedProfile,
-  receiptedReviewIds,
+  assuranceReviewIds,
   reconciliationContext,
 }) {
   const run = {
@@ -59,9 +53,9 @@ export function createValidatedRun({
       }))
       .sort((left, right) => left.id.localeCompare(right.id)),
     exactEvidenceIds: [...exactEvidence].sort(),
-    topology: normalizeTopology(topology),
+    passes: normalizePasses(passes),
     achievedProfile,
-    receiptedReviewIds: [...receiptedReviewIds].sort(),
+    assuranceReviewIds: [...assuranceReviewIds].sort(),
     terminalReconciliation: reconciliationContext?.reconciliation
       ? clone(reconciliationContext.reconciliation)
       : null,

package/assets/skills/recon/scripts/render-packet.mjs

@@ -49,7 +49,7 @@ function bulletLines(values, empty = 'None.') {
 }
 
 export function renderPacketDocument(validatedRun) {
-  const { manifest, ledger, topology } = assertValidatedRun(validatedRun);
+  const { manifest, ledger } = assertValidatedRun(validatedRun);
   const evidenceById = new Map(
     ledger.evidence.map((evidence) => [evidence.id, evidence]),
   );
@@ -63,9 +63,6 @@ export function renderPacketDocument(validatedRun) {
       claim.status === 'unsupported' ||
       (claim.challenges ?? []).length > 0,
   );
-  const failedStages = topology.stages
-    .map(({ stage }) => stage)
-    .filter((stage) => stage.status !== 'complete');
   const omittedGaps = manifest.gaps.filter((gap) =>
     /PASS_(?:FAILED|OMITTED)/.test(gap.code),
   );
@@ -141,15 +138,11 @@ export function renderPacketDocument(validatedRun) {
     '',
     '## Failed or Omitted Passes',
     '',
-    ...bulletLines([
-      ...failedStages.map(
-        (stage) =>
-          `**${escapeInline(stage.mode)}:** ${escapeInline(stage.status)}${stage.message ? ` — ${escapeInline(stage.message)}` : ''}`,
-      ),
-      ...omittedGaps.map(
+    ...bulletLines(
+      omittedGaps.map(
         (gap) => `**${escapeInline(gap.code)}:** ${escapeInline(gap.message)}`,
       ),
-    ]),
+    ),
     '',
     '## Provenance',
     '',
@@ -320,15 +313,9 @@ export async function renderValidatedPacket(
       achievedProfile: run.achievedProfile,
       claimCounts: claimCounts(ledger),
       gapCount: manifest.gaps.length,
-      failedOrOmittedPasses: [
-        ...run.topology.stages
-          .map(({ stage }) => stage)
-          .filter((stage) => stage.status !== 'complete')
-          .map((stage) => stage.mode),
-        ...manifest.gaps
-          .filter((gap) => /PASS_(?:FAILED|OMITTED)/.test(gap.code))
-          .map((gap) => gap.code),
-      ],
+      failedOrOmittedPasses: manifest.gaps
+        .filter((gap) => /PASS_(?:FAILED|OMITTED)/.test(gap.code))
+        .map((gap) => ({ code: gap.code, message: gap.message })),
       digest,
     };
     await assertCanonicalPacketBytes(run);