@reventlessdev/reventless-core 3.0.0-alpha.146 → 3.0.0-alpha.147

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# 3.0.0-alpha.147 (2026-07-08)
+
+### Bug Fixes
+
+* **reventless-core:** composite DCB slices can read back their own events ([4604a91](https://github.com/ReventlessDev/reventless-core/commit/4604a9159fb8bf59b2191ad69fee6613c7f75cd9))
+
+
 # 3.0.0-alpha.146 (2026-07-08)
 
 ### Features

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-core",
-  "version": "3.0.0-alpha.146",
+  "version": "3.0.0-alpha.147",
   "description": "Core package for Reventless framework",
   "license": "Apache-2.0",
   "jest": {
@@ -28,17 +28,17 @@
   "dependencies": {
     "sury": "11.0.0-alpha.4",
     "uuid": "^13.0.0",
-    "@reventlessdev/rescript-hash-object": "1.2.0-alpha.10",
-    "@reventlessdev/rescript-fast-csv": "1.2.0-alpha.11",
     "@reventlessdev/rescript-effect": "0.1.0-alpha.25",
+    "@reventlessdev/rescript-hash-object": "1.2.0-alpha.10",
     "@reventlessdev/rescript-jest": "1.0.0-alpha.6",
     "@reventlessdev/rescript-node-streams": "1.1.0-alpha.10",
+    "@reventlessdev/rescript-fast-csv": "1.2.0-alpha.11",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.14",
-    "@reventlessdev/rescript-uuid": "1.1.0-alpha.14",
+    "@reventlessdev/rescript-ssh2": "1.1.0-alpha.11",
     "@reventlessdev/reventless-infra": "3.0.0-alpha.91",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.24",
-    "@reventlessdev/reventless-spec": "3.0.0-alpha.69",
-    "@reventlessdev/rescript-ssh2": "1.1.0-alpha.11"
+    "@reventlessdev/rescript-uuid": "1.1.0-alpha.14",
+    "@reventlessdev/reventless-spec": "3.0.0-alpha.69"
   },
   "devDependencies": {
     "rescript": "^12.3.0",

package/src/components/StateChangeSlice/StateChangeSlice_Callback.res

@@ -61,8 +61,7 @@ module Make = (
     | Some(keys) => baseTags->Array.filter(t => keys->Array.includes(t.key))
     | None => baseTags
     }
-    let tags =
-      indexedTags->Array.concat([{Reventless.DcbTag.key: "originatorSlice", value: Spec.name}])
+    let tags = indexedTags
     // Inherit service from the triggering command — the DcbEventLog publish path
     // overrides service to `<name>DcbEventLog` for routing so EventCollector
     // subscriptions still match.
@@ -83,12 +82,8 @@ module Make = (
   // tag value(s) instead of logging no id.
   let readEventId = (tags: array<Reventless.DcbTag.tag>): option<string> => {
     let ownTagValues = () => {
-      // Skip the `originatorSlice` metadata tag that `encodeEvent` appends to
-      // every stored event — it's the producing slice's name, not an entity id.
       let vals =
-        tags
-        ->Array.filter((t: Reventless.DcbTag.tag) => t.key != "originatorSlice")
-        ->Array.reduce([], (acc, t: Reventless.DcbTag.tag) =>
+        tags->Array.reduce([], (acc, t: Reventless.DcbTag.tag) =>
           acc->Array.includes(t.value) ? acc : acc->Array.concat([t.value])
         )
       vals->Array.length == 0 ? None : Some(vals->Array.join(","))

package/src/components/StateChangeSlice/StateChangeSlice_Callback.res.mjs

@@ -29,7 +29,7 @@ function Make(Spec) {
       ]]);
     let readEventId = tags => {
       let ownTagValues = () => {
-        let vals = Stdlib_Array.reduce(tags.filter(t => t.key !== "originatorSlice"), [], (acc, t) => {
+        let vals = Stdlib_Array.reduce(tags, [], (acc, t) => {
           if (acc.includes(t.value)) {
             return acc;
           } else {
@@ -188,15 +188,11 @@ function Make(Spec) {
                 let baseTags = DcbTag$Reventless.extractTagsExpanded(Spec.eventSchema, e);
                 let keys = tagKeysByEventType[eventType];
                 let indexedTags = keys !== undefined ? baseTags.filter(t => keys.includes(t.key)) : baseTags;
-                let tags = indexedTags.concat([{
-                    key: "originatorSlice",
-                    value: Spec.name
-                  }]);
                 let meta = Message$ReventlessCore.deriveMeta(parentMeta, undefined);
                 return {
                   eventType: eventType,
                   data: match[1],
-                  tags: tags,
+                  tags: indexedTags,
                   meta: meta
                 };
               });

package/src/plugin/component/Plugin_SubscriptionSchema.res

@@ -61,8 +61,7 @@ let sourceCFields = (
 // Pushed via AppSync Events API. Admin / observability only.
 //
 // Returns (subscriptionFields, supportingTypes) — the supporting type is the
-// {DisplayName}EventLogEvent type that carries position, eventType, payload, and
-// the optional originatorSlice tag injected by StateChangeSlice_Callback.
+// {DisplayName}EventLogEvent type that carries position, eventType, and payload.
 
 let sourceAFieldsAndTypes = (
   ~eventLogEntries: array<eventLogSchemaEntry>,
@@ -76,7 +75,7 @@ let sourceAFieldsAndTypes = (
       seen->Set.add(n)
       fields->Array.push(`  on${n}EventLog_eventAppended: ${n}EventLogEvent`)
       types->Array.push(
-        `type ${n}EventLogEvent {\n  position: String!\n  eventType: String!\n  payload: AWSJSON!\n  originatorSlice: String\n}`,
+        `type ${n}EventLogEvent {\n  position: String!\n  eventType: String!\n  payload: AWSJSON!\n}`,
       )
     }
   })

package/src/plugin/component/Plugin_SubscriptionSchema.res.mjs

@@ -41,7 +41,7 @@ function sourceAFieldsAndTypes(eventLogEntries) {
     if (!seen.has(n)) {
       seen.add(n);
       fields.push(`  on` + n + `EventLog_eventAppended: ` + n + `EventLogEvent`);
-      types.push(`type ` + n + `EventLogEvent {\n  position: String!\n  eventType: String!\n  payload: AWSJSON!\n  originatorSlice: String\n}`);
+      types.push(`type ` + n + `EventLogEvent {\n  position: String!\n  eventType: String!\n  payload: AWSJSON!\n}`);
       return;
     }
   });

package/tests/dcb/DcbStateChangeSliceTest.res

@@ -81,7 +81,7 @@ describe("StateChangeSlice_Callback:", () => {
       ))->toEqual(([Ok("ref-1")], 1, 1))
     })
 
-    testPromise("stored event has correct tags", async () => {
+    testPromise("stored event carries only its entity tags (no framework provenance tag)", async () => {
       let _ = await TestHandler.handleCommands(
         testDcbEventLog,
         Stream.fromIterable([
@@ -90,10 +90,11 @@ describe("StateChangeSlice_Callback:", () => {
       )->Effect.runPromise
       let storedEvent = mock.getEvents()->Array.getUnsafe(0)
 
-      expect(storedEvent.tags)->toEqual([
-        {Reventless.DcbTag.key: "itemId", value: "item-1"},
-        {Reventless.DcbTag.key: "originatorSlice", value: "TestStateChangeSlice"},
-      ])
+      // No `originatorSlice` (or any other) metadata tag is smuggled into the tag
+      // set — a stored event's tags are exactly its DCB entity tags, so the
+      // composite read/write keys stay aligned. See
+      // docs/analysis/dcb-event-provenance-and-metadata.md.
+      expect(storedEvent.tags)->toEqual([{Reventless.DcbTag.key: "itemId", value: "item-1"}])
     })
   })
 

package/tests/dcb/DcbStateChangeSliceTest.res.mjs

@@ -201,23 +201,17 @@ globalThis.describe("StateChangeSlice_Callback:", () => {
         1
       ]);
     });
-    globalThis.test("stored event has correct tags", async () => {
+    globalThis.test("stored event carries only its entity tags (no framework provenance tag)", async () => {
       await Effect.runPromise(TestHandler.handleCommands(undefined, undefined, testDcbEventLog, Stream.fromIterable([makeTopicItem("ref-1", {
           TAG: "CreateItem",
           itemId: "item-1",
           name: "Test"
         })])));
       let storedEvent = mock.getEvents()[0];
-      globalThis.expect(storedEvent.tags).toEqual([
-        {
+      globalThis.expect(storedEvent.tags).toEqual([{
           key: "itemId",
           value: "item-1"
-        },
-        {
-          key: "originatorSlice",
-          value: "TestStateChangeSlice"
-        }
-      ]);
+        }]);
     });
   });
   globalThis.describe("handleCommands - decide returns Ok([])", () => {