@reventlessdev/reventless-aws 3.0.0-alpha.205 → 3.0.0-alpha.207

package/CHANGELOG.md

@@ -3,6 +3,21 @@
 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.207 (2026-07-13)
+
+### Bug Fixes
+
+* **admin:** materialise ApiSchemaPush SideEffect Source at runtime ([57a61d9](https://github.com/ReventlessDev/reventless-core/commit/57a61d98fa11dfe3de188cc299c89abdeea41457))
+
+
+# 3.0.0-alpha.206 (2026-07-13)
+
+**Note:** Version bump only for package @reventlessdev/reventless-aws
+
+
+
+
+
 # 3.0.0-alpha.205 (2026-07-13)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-aws",
-  "version": "3.0.0-alpha.205",
+  "version": "3.0.0-alpha.207",
   "description": "AWS adapters for Reventless",
   "license": "Apache-2.0",
   "dependencies": {
@@ -8,16 +8,16 @@
     "@aws-sdk/client-cloudfront": "3.970.0",
     "sury": "11.0.0-alpha.4",
     "uuid": "^13.0.0",
-    "@reventlessdev/rescript-aws-sdk": "2.2.0-alpha.21",
     "@reventlessdev/rescript-effect": "0.1.0-alpha.27",
-    "@reventlessdev/rescript-jest": "1.0.0-alpha.7",
+    "@reventlessdev/rescript-aws-sdk": "2.2.0-alpha.21",
     "@reventlessdev/rescript-pulumi-aws": "2.4.0-alpha.50",
+    "@reventlessdev/rescript-jest": "1.0.0-alpha.7",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.15",
+    "@reventlessdev/reventless-core": "3.0.0-alpha.164",
     "@reventlessdev/rescript-uuid": "1.1.0-alpha.15",
     "@reventlessdev/reventless-infra": "3.0.0-alpha.98",
-    "@reventlessdev/reventless-core": "3.0.0-alpha.163",
-    "@reventlessdev/reventless-postgres": "3.0.0-alpha.27",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.26",
+    "@reventlessdev/reventless-postgres": "3.0.0-alpha.28",
     "@reventlessdev/reventless-spec": "3.0.0-alpha.76"
   },
   "devDependencies": {

package/src/Platform.res.mjs

@@ -1287,9 +1287,9 @@ function MakeWithConfig(Config) {
     ]).apply(param => {
       AdminApiSchemaPushHandler.make("AdminApiSchemaPush", [{
           Source: {
-            name: ApiFragmentRegistrySpec$ReventlessCore.name,
+            name: ApiSchemaPush$ReventlessAws.Source.name,
             Id: Id$Reventless.$$String,
-            eventSchema: ApiFragmentRegistrySpec$ReventlessCore.eventSchema
+            eventSchema: ApiSchemaPush$ReventlessAws.Source.eventSchema
           },
           moduleUrl: ApiSchemaPush$ReventlessAws.moduleUrl,
           execute: ApiSchemaPush$ReventlessAws.execute
@@ -2707,9 +2707,9 @@ function Make($star) {
     ]).apply(param => {
       AdminApiSchemaPushHandler.make("AdminApiSchemaPush", [{
           Source: {
-            name: ApiFragmentRegistrySpec$ReventlessCore.name,
+            name: ApiSchemaPush$ReventlessAws.Source.name,
             Id: Id$Reventless.$$String,
-            eventSchema: ApiFragmentRegistrySpec$ReventlessCore.eventSchema
+            eventSchema: ApiSchemaPush$ReventlessAws.Source.eventSchema
           },
           moduleUrl: ApiSchemaPush$ReventlessAws.moduleUrl,
           execute: ApiSchemaPush$ReventlessAws.execute

package/src/adapter/Api/ApiSchemaPush.res

@@ -14,7 +14,16 @@
 // is loaded via dynamic import so no @aws-sdk / provider code is statically captured
 // beyond what the SideEffectHandler Lambda already bundles (runtime-purity — see
 // reference_pulumi_leaks_into_lambda_runtime_graph).
-module Source = ReventlessCore.ApiFragmentRegistrySpec
+// `include` (not a bare `module Source = …` alias) is deliberate. SideEffectHandler_Callback
+// reads Source.{name,eventSchema,Id} reflectively off this module's compiled export at cold
+// start, but ApiSchemaPush only uses Source at the TYPE level (Source.event, Source.Id.t,
+// Source.fragmentSnapshotEntry) — a bare module alias is dead-shaken by ReScript to
+// `let Source;` (undefined), crashing the handler with "Cannot read properties of undefined
+// (reading 'eventSchema')". `include` materialises the module's runtime values into the export
+// while keeping full type transparency for the pattern match below.
+module Source = {
+  include ReventlessCore.ApiFragmentRegistrySpec
+}
 
 let moduleUrl: string = %raw(`import.meta.url`)
 

package/src/adapter/Api/ApiSchemaPush.res.mjs

@@ -1,5 +1,8 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
+import * as ApiFragmentRegistrySpec$ReventlessCore from "@reventlessdev/reventless-core/src/admin/ApiFragmentRegistry/ApiFragmentRegistrySpec.res.mjs";
+
+let Source = ApiFragmentRegistrySpec$ReventlessCore;
 
 let moduleUrl = import.meta.url;
 
@@ -26,8 +29,6 @@ async function execute(_id, _meta, event, _queryEngine) {
   return await pushApiSchema(jsSnapshot);
 }
 
-let Source;
-
 export {
   Source,
   moduleUrl,

package/tests/ApiSchemaPushTest.res

@@ -0,0 +1,24 @@
+// Regression guard for the ApiSchemaPush SideEffect "Source erasure" deploy-only bug.
+//
+// SideEffectHandler_Callback.Make reads Source.{name,eventSchema,Id} reflectively off
+// the imported ApiSchemaPush module at Lambda cold start. ApiSchemaPush only uses Source
+// at the TYPE level, so a bare `module Source = …` alias gets dead-shaken by ReScript to
+// `let Source;` (undefined) — crashing the handler with "Cannot read properties of
+// undefined (reading 'eventSchema')" and hanging the deploy waiter. The `include` in
+// ApiSchemaPush.res materialises Source's runtime values. These assertions dereference
+// exactly the fields the runtime reads, so a regression (alias reintroduced) fails here
+// instead of only on a live deploy.
+
+open JestGlobals
+
+describe("ApiSchemaPush SideEffect Source (runtime materialisation)", () => {
+  testSync("Source.name is the registry aggregate name", () => {
+    expect(ApiSchemaPush.Source.name)->toBe("ApiFragmentRegistry")
+  })
+
+  testSync("Source.eventSchema is a live schema — extractAllVariantNames succeeds", () => {
+    // Mirrors SideEffectHandler_Callback.Make: throws if Source is erased to undefined.
+    let tags = Reventless.DcbTag.extractAllVariantNames(ApiSchemaPush.Source.eventSchema)
+    expect(tags)->toContain("ApiSchemaComputed")
+  })
+})

package/tests/ApiSchemaPushTest.res.mjs

@@ -0,0 +1,16 @@
+// Generated by ReScript, PLEASE EDIT WITH CARE
+
+import * as DcbTag$Reventless from "@reventlessdev/reventless-spec/src/components/DcbTag.res.mjs";
+import * as ApiSchemaPush$ReventlessAws from "../src/adapter/Api/ApiSchemaPush.res.mjs";
+
+globalThis.describe("ApiSchemaPush SideEffect Source (runtime materialisation)", () => {
+  globalThis.test("Source.name is the registry aggregate name", () => {
+    globalThis.expect(ApiSchemaPush$ReventlessAws.Source.name).toBe("ApiFragmentRegistry");
+  });
+  globalThis.test("Source.eventSchema is a live schema — extractAllVariantNames succeeds", () => {
+    let tags = DcbTag$Reventless.extractAllVariantNames(ApiSchemaPush$ReventlessAws.Source.eventSchema);
+    globalThis.expect(tags).toContain("ApiSchemaComputed");
+  });
+});
+
+/*  Not a pure module */