@sanity/ailf 3.1.0 → 3.1.1

package/dist/orchestration/steps/calculate-scores-step.js

@@ -143,16 +143,18 @@ export class CalculateScoresStep {
         // The full responseOutput lives in the GCS artifact; PublishReportStep
         // later strips it from the inline Content Lake document when this
         // upload succeeds.
+        //
+        // The emits flow through `ctx.artifactWriter`, which the composition
+        // root wraps in `AccumulatingArtifactWriter`. That's where the
+        // authoritative merged ref is built; `FinalizeRunStep` reads it
+        // straight from the accumulator, so producer-side registration on
+        // `state.artifactRefs` would only clobber the accumulator's full set
+        // with a partial single-entry ref.
+        //
         // W0050 — ctx.artifactWriter is always present; no guard needed.
         const testResults = tryReadTestResults(ctx.config.rootDir);
         if (testResults?.length) {
-            const artifactRef = await uploadTestOutputs(ctx.artifactWriter, ctx.runId, testResults, ctx.config.mode);
-            if (artifactRef) {
-                state.artifactRefs = {
-                    ...state.artifactRefs,
-                    testOutputs: artifactRef,
-                };
-            }
+            await uploadTestOutputs(ctx.artifactWriter, ctx.runId, testResults, ctx.config.mode);
         }
         const criticalSuffix = belowCritical.length > 0
             ? ` (${belowCritical.length} area(s) below critical threshold: ${belowCritical.join(", ")})`

package/dist/orchestration/steps/finalize-run-step.js

@@ -86,11 +86,12 @@ export class FinalizeRunStep {
         });
         // W0051 revisit: the composition-root wraps `ctx.artifactWriter` in
         // `AccumulatingArtifactWriter`, which keeps a map of every ref any
-        // producer emitted this run. Merge that into `state.artifactRefs` so
-        // the manifest reflects the FULL set — not just the subset producers
-        // happened to register manually. When the writer is a NoOp / plain
-        // decorator without accumulation, `aggregated` stays empty and the
-        // manifest falls back to the producer-side registration.
+        // producer emitted this run — the authoritative FULL set, merged by
+        // entry key. `state.artifactRefs` is a producer-side fallback for
+        // writers that don't accumulate (NoOp / plain decorators). The
+        // accumulator wins per type when both exist, because producer-side
+        // registrations tend to capture only the last ref from a parallel
+        // batch and would otherwise clobber the merged entries list.
         //
         // W0058: `findAccumulator` unwraps `InstrumentedArtifactWriter` too
         // so the manifest stays fully populated when `AILF_UPLOAD_METRICS=1`.
@@ -99,8 +100,8 @@ export class FinalizeRunStep {
             ? accumulator.getAccumulatedArtifactRefs()
             : {};
         const artifacts = {
-            ...aggregated,
             ...(state.artifactRefs ?? {}),
+            ...aggregated,
         };
         const manifest = {
             version: 1,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/ailf",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -52,8 +52,8 @@
     "@types/node": "^22.13.1",
     "tsx": "^4.19.2",
     "typescript": "^5.7.3",
-    "@sanity/ailf-shared": "0.1.0",
-    "@sanity/ailf-core": "0.1.0"
+    "@sanity/ailf-core": "0.1.0",
+    "@sanity/ailf-shared": "0.1.0"
   },
   "scripts": {
     "build": "tsc && tsx scripts/bundle-workspace-deps.ts",