@reventlessdev/reventless-aws 3.0.0-alpha.301 → 3.0.0-alpha.303

package/tests/Util_ShellConfigTest.res.mjs

@@ -1,9 +1,12 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
 import * as JestGlobals from "@reventlessdev/rescript-jest/src/JestGlobals.res.mjs";
+import * as Stdlib_JSON from "@rescript/runtime/lib/es6/Stdlib_JSON.js";
+import * as Stdlib_Array from "@rescript/runtime/lib/es6/Stdlib_Array.js";
 import * as Stdlib_Option from "@rescript/runtime/lib/es6/Stdlib_Option.js";
 import * as Primitive_exceptions from "@rescript/runtime/lib/es6/Primitive_exceptions.js";
 import * as Util_ShellConfig$ReventlessAws from "../src/util/Util_ShellConfig.res.mjs";
+import * as Platform_BakedManifest$ReventlessCore from "@reventlessdev/reventless-core/src/admin/Platform_BakedManifest.res.mjs";
 
 let computed = [
   [
@@ -107,6 +110,103 @@ globalThis.describe("Util_ShellConfig.fields — bakedManifest", () => {
   });
 });
 
+globalThis.describe("Util_ShellConfig.fields — journeys", () => {
+  let withJourneys = journeys => ({
+    components: [{
+        plugin: "Catalog",
+        views: ["Products"],
+        commands: []
+      }],
+    journeys: journeys
+  });
+  let shopper_components = [{
+      plugin: "Catalog",
+      views: ["Products"],
+      commands: []
+    }];
+  let shopper = {
+    group: "Shopper",
+    components: shopper_components
+  };
+  let fulfilment_components = [{
+      plugin: "Ordering",
+      views: ["Orders"],
+      commands: ["ShipOrder"]
+    }];
+  let fulfilment_key = "fulfilment.json";
+  let fulfilment = {
+    group: "Fulfilment",
+    components: fulfilment_components,
+    key: fulfilment_key
+  };
+  globalThis.test("a bake declaring no journeys writes no map", () => {
+    let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, {
+      components: [{
+          plugin: "Catalog",
+          views: ["Products"],
+          commands: []
+        }]
+    }, undefined);
+    globalThis.expect(Stdlib_Option.isNone(out["journeyManifestUrls"])).toBe(true);
+  });
+  globalThis.test("an empty journeys array is the same as none", () => {
+    let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, withJourneys([]), undefined);
+    globalThis.expect(Stdlib_Option.isNone(out["journeyManifestUrls"])).toBe(true);
+  });
+  globalThis.test("keeps manifestUrl as the default journey", () => {
+    let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, withJourneys([shopper]), undefined);
+    globalThis.expect(get(out, "manifestUrl")).toEqual("/component-manifest.json");
+  });
+  globalThis.test("maps each declared group to its own file", () => {
+    let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, withJourneys([
+      shopper,
+      fulfilment
+    ]), undefined);
+    globalThis.expect(get(out, "journeyManifestUrls")).toEqual(Object.fromEntries([
+      [
+        "Shopper",
+        "/component-manifest-shopper.json"
+      ],
+      [
+        "Fulfilment",
+        "/fulfilment.json"
+      ]
+    ]));
+  });
+  globalThis.test("publishes the URL of every key the bake writes, exactly once", () => {
+    let bake = withJourneys([
+      shopper,
+      fulfilment
+    ]);
+    let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, bake, undefined);
+    let published = Stdlib_Option.mapOr(Stdlib_JSON.Decode.string(get(out, "manifestUrl")), [], url => [url]).concat(Stdlib_Array.filterMap(Object.values(Stdlib_Option.getOr(Stdlib_JSON.Decode.object(get(out, "journeyManifestUrls")), {})), Stdlib_JSON.Decode.string));
+    globalThis.expect(published).toEqual(Platform_BakedManifest$ReventlessCore.files(bake).map(param => "/" + param[0]));
+  });
+  globalThis.test("a shellConfig journey map fails the deploy rather than redirecting it", () => {
+    let failure;
+    try {
+      Util_ShellConfig$ReventlessAws.fields(computed, undefined, withJourneys([shopper]), Object.fromEntries([[
+          "journeyManifestUrls",
+          {}
+        ]]));
+      failure = undefined;
+    } catch (raw_message) {
+      let message = Primitive_exceptions.internalToException(raw_message);
+      if (message.RE_EXN_ID === "Failure") {
+        failure = message._1;
+      } else {
+        throw message;
+      }
+    }
+    if (failure !== undefined) {
+      globalThis.expect(failure.includes("journeyManifestUrls")).toBe(true);
+      return;
+    } else {
+      return JestGlobals.fail("a shellConfig key redirecting the computed journey map must fail the deploy");
+    }
+  });
+});
+
 globalThis.describe("Util_ShellConfig.fields — shellConfig passthrough", () => {
   globalThis.test("shell-owned keys land verbatim, under the computed ones", () => {
     let out = Util_ShellConfig$ReventlessAws.fields(computed, undefined, undefined, Object.fromEntries([

package/tests/stateChangeDescriptorParity.mjs

@@ -49,6 +49,59 @@ const CASES = [
     entityKey: "p3",
     state: { id: "p3", blob: "x".repeat(70 * 1024), updatedAt: "2026-05-19T12:00:00Z" },
   },
+  // Retirement: the row is withdrawn, so every implementation must publish the
+  // metadata-only shape — no state, and no sortKeyValue either, even though this
+  // row carries an updatedAt that the same row would publish while live.
+  {
+    name: "retired row degrades to metadata only",
+    changeKind: "Updated",
+    entityKey: "p4",
+    state: { id: "p4", name: "Widget", archived: true, updatedAt: "2026-05-19T12:00:00Z" },
+    retiredField: "archived",
+  },
+  // The control: same field declared, flag false, so nothing changes. Without
+  // this, an implementation that dropped the payload unconditionally whenever a
+  // retiredField was configured would still pass the case above.
+  {
+    name: "live row with a retirement flag carries the row",
+    changeKind: "Updated",
+    entityKey: "p5",
+    state: { id: "p5", name: "Widget", archived: false, updatedAt: "2026-05-19T12:00:00Z" },
+    retiredField: "archived",
+  },
+  // The state form, and the reason it is a set: a lifecycle may end in more than
+  // one way, and both ways withdraw the row identically. Two cases rather than
+  // one so an implementation comparing against only the first member fails.
+  {
+    name: "row in the first retired state degrades to metadata only",
+    changeKind: "Updated",
+    entityKey: "p6",
+    state: { id: "p6", name: "Widget", shelfStatus: "Archived", updatedAt: "2026-05-19T12:00:00Z" },
+    retiredField: "shelfStatus",
+    retiredValues: ["Archived", "Discontinued"],
+  },
+  {
+    name: "row in the second retired state degrades to metadata only",
+    changeKind: "Updated",
+    entityKey: "p7",
+    state: {
+      id: "p7",
+      name: "Widget",
+      shelfStatus: "Discontinued",
+      updatedAt: "2026-05-19T12:00:00Z",
+    },
+    retiredField: "shelfStatus",
+    retiredValues: ["Archived", "Discontinued"],
+  },
+  // The state-form control: a live state on the same declared lifecycle.
+  {
+    name: "row in a live state carries the row",
+    changeKind: "Updated",
+    entityKey: "p8",
+    state: { id: "p8", name: "Widget", shelfStatus: "Listed", updatedAt: "2026-05-19T12:00:00Z" },
+    retiredField: "shelfStatus",
+    retiredValues: ["Archived", "Discontinued"],
+  },
   {
     name: "delete carries no row",
     changeKind: "Removed",
@@ -79,10 +132,17 @@ export async function buildAll() {
     return { ...descriptor, seq: "<seq>" };
   };
 
-  return CASES.map(({ name, changeKind, entityKey, state }) => ({
+  return CASES.map(({ name, changeKind, entityKey, state, retiredField, retiredValues }) => ({
     name,
     local: normalise(
-      local.make(changeKind, entityKey, state === undefined ? undefined : state, local.nextSequence()),
+      local.make(
+        changeKind,
+        entityKey,
+        state === undefined ? undefined : state,
+        local.nextSequence(),
+        retiredField,
+        retiredValues,
+      ),
     ),
     relay: normalise(
       relay.makeDescriptor(
@@ -93,10 +153,19 @@ export async function buildAll() {
         // OldImage, which makeDescriptor then drops.
         state === undefined ? { id: entityKey } : state,
         "49590300000000016818000000",
+        retiredField,
+        retiredValues,
       ),
     ),
     postgres: normalise(
-      postgres.makeDescriptor({ changeKind, entityKey, state, seq: postgres.nextSequence() }),
+      postgres.makeDescriptor({
+        changeKind,
+        entityKey,
+        state,
+        seq: postgres.nextSequence(),
+        retiredField,
+        retiredValues,
+      }),
     ),
   }));
 }