@reventlessdev/reventless-aws 3.0.0-alpha.335 → 3.0.0-alpha.337

package/CHANGELOG.md

@@ -3,6 +3,51 @@
 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.337 (2026-09-04)
+
+* feat(spec)!: one optional encoding on the wire, with no annotation ([320f91d](https://github.com/ReventlessDev/reventless-core/commit/320f91daa8bd90812a6e82069e7a1cb473041930))
+
+### BREAKING CHANGES
+
+* the two encodings cannot read each other. Stored
+pluginDefinition / pluginStructure payloads and already-deployed plugins must
+go — wipe the platform scope (SEED_RESET_SCOPE=platform), quiesce, and
+redeploy the fleet from one commit. Domain plugin data is untouched.
+
+Two guards had to learn the new shape, both of which defined "optional" as
+has.null and so mistook an omitted key for something to invent:
+
+- Message.fillMissingDefaults reached `return undefined` only below two arms
+  that fire first — an enum's first const, and an object member filled with
+  zeros. Absent option<record> therefore healed to a zero-filled record, not
+  None: on the real schema, dcbEventLog became Some({name: "", eventTopicArn:
+  ""}), which manageSubscriptions would have read as a peer to subscribe to.
+  One line, mirroring the has.null guard, above both arms.
+- PluginDefinitionScalars' walker reported 16 optional fields as newly-added
+  bare required scalars. With both encodings understood, the golden list is
+  unchanged.
+
+The frozen lifecycle corpus holds five real payloads in the old encoding.
+Null-valued keys were stripped mechanically — the rewrite round-trips each
+file unchanged before editing, and a key-by-key diff shows null removals and
+nothing else. The README records it beside the account-id redaction and says
+why it is not a regeneration: no fixture was rebuilt from ReScript types, so
+every generation in its table is still pinned.
+
+check:graphql is unchanged, as expected: SchemaType.fromSury collapses Null
+and Undefined to the same Nullable, so the emitted schema never distinguished
+them.
+
+
+
+# 3.0.0-alpha.336 (2026-09-04)
+
+**Note:** Version bump only for package @reventlessdev/reventless-aws
+
+
+
+
+
 # 3.0.0-alpha.335 (2026-09-02)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-aws",
-  "version": "3.0.0-alpha.335",
+  "version": "3.0.0-alpha.337",
   "description": "AWS adapters for Reventless",
   "license": "Apache-2.0",
   "bin": {
@@ -15,18 +15,18 @@
     "@aws-sdk/s3-request-presigner": "3.970.0",
     "sury": "11.0.0-rc.2",
     "uuid": "^13.0.0",
-    "@reventlessdev/rescript-aws-sdk": "3.0.0-alpha.14",
+    "@reventlessdev/rescript-aws-sdk": "3.0.0-alpha.15",
+    "@reventlessdev/rescript-node": "2.0.0-alpha.9",
     "@reventlessdev/rescript-jest": "1.0.0-alpha.10",
-    "@reventlessdev/rescript-node": "2.0.0-alpha.8",
-    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.7",
     "@reventlessdev/rescript-effect": "0.1.0-alpha.32",
+    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.8",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.19",
     "@reventlessdev/rescript-uuid": "2.0.0-alpha.0",
-    "@reventlessdev/reventless-core": "3.0.0-alpha.254",
-    "@reventlessdev/reventless-infra": "3.0.0-alpha.155",
-    "@reventlessdev/reventless-interop": "3.0.0-alpha.34",
-    "@reventlessdev/reventless-postgres": "3.0.0-alpha.118",
-    "@reventlessdev/reventless-spec": "3.0.0-alpha.127"
+    "@reventlessdev/reventless-core": "3.0.0-alpha.256",
+    "@reventlessdev/reventless-infra": "3.0.0-alpha.157",
+    "@reventlessdev/reventless-interop": "3.0.0-alpha.35",
+    "@reventlessdev/reventless-postgres": "3.0.0-alpha.120",
+    "@reventlessdev/reventless-spec": "3.0.0-alpha.129"
   },
   "devDependencies": {
     "rescript": "12.3.0",

package/src/adapter/Runtime/EventCollectorEntryPoint_Ops.res

@@ -62,11 +62,11 @@ external scanByTableName: (
 let exnMessage = (exn: exn): string =>
   exn->JsExn.fromException->Option.flatMap(JsExn.message)->Option.getOr("unknown error")
 
-// ── js_nullable normalization ───────────────────────────────────────────────
-// pluginDefinition option fields use @s.matches(js_nullable): when the record
-// comes from a plain JSON.parse (not a sury parse), None is represented as
-// `null` — which ReScript's option (undefined-based) would misread as Some.
-// Normalize through Nullable so pattern matches see a real option.
+// ── Absent-field normalization ──────────────────────────────────────────────
+// pluginDefinition's optionals omit the key, so a plain JSON.parse (not a sury
+// parse) yields `undefined` and ReScript reads it as None unaided. Kept because
+// the same records also arrive from stores and peers written when an absent
+// field was `null`, which ReScript's option would misread as Some.
 
 external optionAsNullable: option<'a> => Nullable.t<'a> = "%identity"
 let jsOption = (v: option<'a>): option<'a> => v->optionAsNullable->Nullable.toOption
@@ -237,10 +237,9 @@ let parseHandlerConfig = (rawJson: string): handlerConfig => {
 
 // ── Plugin-RM row projection ────────────────────────────────────────────────
 // The subset of the Plugin read-model state the subscription manager needs.
-// Deliberately sidesteps sury: the state schema marks many fields
-// `@s.matches(js_nullable …)`, but DDB drops undefined attributes on write, so
-// the unmarshalled row arrives with those keys *missing* and sury's strict
-// parser rejects them. Field types reuse the spec's definition records — the
+// Deliberately sidesteps sury: DDB drops undefined attributes on write, so the
+// unmarshalled row can arrive with keys sury's strict parser still expects — the
+// offload fields among them, whose codec keeps its `null` arm. Field types reuse the spec's definition records — the
 // projection is a strict subset of Reventless.Plugin.pluginDefinition.
 
 type pluginProjection = {
@@ -325,8 +324,8 @@ let projectPluginRow = (row: JSON.t): option<pluginProjection> =>
 
 // A pluginDefinition viewed as a projection (the manage logic reads only the
 // shared subset; `status` is never read for the acting plugin). Array/option
-// fields stay js_nullable-normalized: a definition arriving over the wire, rather
-// than from loadPluginDefinition's decode, still carries null for an absent field.
+// fields stay normalized: a definition arriving over the wire, rather than from
+// loadPluginDefinition's decode, may still carry null for an absent field.
 let projectionOfDefinition = (d: Reventless.Plugin.pluginDefinition): pluginProjection => {
   id: d.id,
   status: "",