@reventlessdev/reventless-local 3.0.0-alpha.185 → 3.0.0-alpha.186

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.186 (2026-08-01)
+
+### Bug Fixes
+
+* **core:** drain the inbound audit log per request, not wholesale ([e18d4ec](https://github.com/ReventlessDev/reventless-core/commit/e18d4ec4b7ffb38a9e2bd3594d39a07e2ee91851))
+
+
 # 3.0.0-alpha.185 (2026-08-01)
 
 **Note:** Version bump only for package @reventlessdev/reventless-local

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-local",
-  "version": "3.0.0-alpha.185",
+  "version": "3.0.0-alpha.186",
   "description": "Local platform for Reventless (in-memory or SQLite backend, for development and testing without AWS)",
   "license": "Apache-2.0",
   "bin": {
@@ -35,17 +35,17 @@
     "sury": "11.0.0-alpha.4",
     "ws": "^8.18.0",
     "@reventlessdev/rescript-effect": "0.1.0-alpha.32",
-    "@reventlessdev/rescript-mcp-sdk": "1.0.0-alpha.21",
     "@reventlessdev/rescript-jest": "1.0.0-alpha.10",
     "@reventlessdev/rescript-graphql-yoga": "1.0.0-alpha.26",
+    "@reventlessdev/rescript-mcp-sdk": "1.0.0-alpha.21",
     "@reventlessdev/rescript-node": "2.0.0-alpha.0",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.18",
-    "@reventlessdev/reventless-core": "3.0.0-alpha.197",
-    "@reventlessdev/reventless-gwt": "1.0.0-alpha.144",
+    "@reventlessdev/reventless-core": "3.0.0-alpha.198",
+    "@reventlessdev/reventless-graphql-server": "1.0.0-alpha.46",
+    "@reventlessdev/reventless-gwt": "1.0.0-alpha.145",
     "@reventlessdev/reventless-infra": "3.0.0-alpha.115",
-    "@reventlessdev/reventless-spec": "3.0.0-alpha.90",
-    "@reventlessdev/reventless-graphql-server": "1.0.0-alpha.45",
-    "@reventlessdev/reventless-postgres": "3.0.0-alpha.61"
+    "@reventlessdev/reventless-postgres": "3.0.0-alpha.62",
+    "@reventlessdev/reventless-spec": "3.0.0-alpha.90"
   },
   "devDependencies": {
     "rescript": "12.3.0",

package/tests/components/inboundtranslationslice/InboundTranslationSliceCallbackTest.res

@@ -125,6 +125,60 @@ describe("InboundTranslationSlice Callback", () => {
     })
   })
 
+  // `auditLog` hands a row to whoever persists it and must not keep it: the dict
+  // outlives the request (a warm Lambda container serves many), so a drain that
+  // re-read the whole dict would rewrite every row the container had ever seen on
+  // every request — and, since the write overwrites by row id, would resurrect
+  // rows deleted from the table in between. Persisting is per-request and takes.
+  describe("draining the audit log", () => {
+    testPromise("taking a row leaves the log empty for the next request", async () => {
+      let mockPublish: ReventlessInfra.CommandTopic.publishJsons = async _cmds => ()
+      let input = status =>
+        {"paymentId": "pay-1", "orderId": "ord-1", "status": status}->Obj.magic
+
+      let first = await Callback.receive(mockPublish, input("completed"))
+      let firstId = first->ReventlessCore.InboundTranslationSlice_Callback.requestIdOf
+      expect(
+        Callback.auditLog
+        ->ReventlessCore.InboundTranslationSlice_Callback.takeAuditRow(firstId)
+        ->Option.isSome,
+      )->toBe(true)
+      expect(Callback.auditLog->Dict.toArray->Array.length)->toBe(0)
+
+      // A second request drains only its own row — the first one is gone, so it
+      // cannot be written a second time.
+      let second = await Callback.receive(mockPublish, input("completed"))
+      let secondId = second->ReventlessCore.InboundTranslationSlice_Callback.requestIdOf
+      expect(secondId == firstId)->toBe(false)
+      expect(Callback.auditLog->Dict.toArray->Array.length)->toBe(1)
+      expect(
+        Callback.auditLog
+        ->ReventlessCore.InboundTranslationSlice_Callback.takeAuditRow(firstId)
+        ->Option.isSome,
+      )->toBe(false)
+      expect(
+        Callback.auditLog
+        ->ReventlessCore.InboundTranslationSlice_Callback.takeAuditRow(secondId)
+        ->Option.isSome,
+      )->toBe(true)
+    })
+
+    testPromise("a rejected request's row is drained the same way", async () => {
+      let mockPublish: ReventlessInfra.CommandTopic.publishJsons = async _cmds => ()
+      let result = await Callback.receive(
+        mockPublish,
+        {"paymentId": "pay-1", "orderId": "ord-1", "status": "pending"}->Obj.magic,
+      )
+      let id = result->ReventlessCore.InboundTranslationSlice_Callback.requestIdOf
+      switch Callback.auditLog->ReventlessCore.InboundTranslationSlice_Callback.takeAuditRow(id) {
+      | Some(row) =>
+        expect(row.status)->toBe(ReventlessCore.InboundTranslationSlice_Callback.Failure)
+      | None => expect(true)->toBe(false)
+      }
+      expect(Callback.auditLog->Dict.toArray->Array.length)->toBe(0)
+    })
+  })
+
   describe("multi-command", () => {
     testPromise("translate returning multiple pairs publishes all commands", async () => {
       // Use a spec that returns multiple commands

package/tests/components/inboundtranslationslice/InboundTranslationSliceCallbackTest.res.mjs

@@ -122,6 +122,45 @@ globalThis.describe("InboundTranslationSlice Callback", () => {
       globalThis.expect(match[1].status).toBe("Failure");
     });
   });
+  globalThis.describe("draining the audit log", () => {
+    globalThis.test("taking a row leaves the log empty for the next request", async () => {
+      let mockPublish = async _cmds => {};
+      let first = await Callback.receive(mockPublish, {
+        paymentId: "pay-1",
+        orderId: "ord-1",
+        status: "completed"
+      });
+      let firstId = InboundTranslationSlice_Callback$ReventlessCore.requestIdOf(first);
+      globalThis.expect(Stdlib_Option.isSome(InboundTranslationSlice_Callback$ReventlessCore.takeAuditRow(Callback.auditLog, firstId))).toBe(true);
+      globalThis.expect(Object.entries(Callback.auditLog).length).toBe(0);
+      let second = await Callback.receive(mockPublish, {
+        paymentId: "pay-1",
+        orderId: "ord-1",
+        status: "completed"
+      });
+      let secondId = InboundTranslationSlice_Callback$ReventlessCore.requestIdOf(second);
+      globalThis.expect(secondId === firstId).toBe(false);
+      globalThis.expect(Object.entries(Callback.auditLog).length).toBe(1);
+      globalThis.expect(Stdlib_Option.isSome(InboundTranslationSlice_Callback$ReventlessCore.takeAuditRow(Callback.auditLog, firstId))).toBe(false);
+      globalThis.expect(Stdlib_Option.isSome(InboundTranslationSlice_Callback$ReventlessCore.takeAuditRow(Callback.auditLog, secondId))).toBe(true);
+    });
+    globalThis.test("a rejected request's row is drained the same way", async () => {
+      let mockPublish = async _cmds => {};
+      let result = await Callback.receive(mockPublish, {
+        paymentId: "pay-1",
+        orderId: "ord-1",
+        status: "pending"
+      });
+      let id = InboundTranslationSlice_Callback$ReventlessCore.requestIdOf(result);
+      let row = InboundTranslationSlice_Callback$ReventlessCore.takeAuditRow(Callback.auditLog, id);
+      if (row !== undefined) {
+        globalThis.expect(row.status).toBe("Failure");
+      } else {
+        globalThis.expect(true).toBe(false);
+      }
+      globalThis.expect(Object.entries(Callback.auditLog).length).toBe(0);
+    });
+  });
   globalThis.describe("multi-command", () => {
     globalThis.test("translate returning multiple pairs publishes all commands", async () => {
       let moduleUrl = import.meta.url;