@reventlessdev/reventless-core 3.0.0-alpha.181 → 3.0.0-alpha.183

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.183 (2026-07-27)
+
+### Bug Fixes
+
+* **aws:** keep InboundTranslationSlice runtime path out of the Pulumi graph ([9166cff](https://github.com/ReventlessDev/reventless-core/commit/9166cff85939012d12e44835e2eb79a88fd1b66a))
+
+
+# 3.0.0-alpha.182 (2026-07-26)
+
+### Bug Fixes
+
+* **api:** commit regenerated SchemaType output for nullable optional enums ([6d48f53](https://github.com/ReventlessDev/reventless-core/commit/6d48f53c31489b8cb312c4bc63dac3b927145bd4))
+* **api:** preserve nullability for optional enum fields in generated GraphQL ([3110a4a](https://github.com/ReventlessDev/reventless-core/commit/3110a4af032440ba2639519f61f221960ac78fba))
+
+
 # 3.0.0-alpha.181 (2026-07-26)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-core",
-  "version": "3.0.0-alpha.181",
+  "version": "3.0.0-alpha.183",
   "description": "Core package for Reventless framework",
   "license": "Apache-2.0",
   "jest": {
@@ -28,16 +28,16 @@
   "dependencies": {
     "sury": "11.0.0-alpha.4",
     "uuid": "^13.0.0",
+    "@reventlessdev/rescript-effect": "0.1.0-alpha.31",
     "@reventlessdev/rescript-fast-csv": "1.2.0-alpha.14",
-    "@reventlessdev/rescript-hash-object": "1.2.0-alpha.13",
     "@reventlessdev/rescript-jest": "1.0.0-alpha.9",
     "@reventlessdev/rescript-node-streams": "1.1.0-alpha.13",
-    "@reventlessdev/rescript-effect": "0.1.0-alpha.31",
+    "@reventlessdev/rescript-hash-object": "1.2.0-alpha.13",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.17",
-    "@reventlessdev/rescript-ssh2": "1.1.0-alpha.14",
     "@reventlessdev/rescript-uuid": "1.1.0-alpha.17",
-    "@reventlessdev/reventless-infra": "3.0.0-alpha.105",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.29",
+    "@reventlessdev/rescript-ssh2": "1.1.0-alpha.14",
+    "@reventlessdev/reventless-infra": "3.0.0-alpha.105",
     "@reventlessdev/reventless-spec": "3.0.0-alpha.81"
   },
   "devDependencies": {

package/src/components/Api/SchemaType.res

@@ -82,6 +82,7 @@ let rec fromSury = (~parentName: string, ~fieldName: string, schema: S.t<unknown
         | _ => true
         }
       )
+      let isOptional = nonNullVariants->Array.length < anyOf->Array.length
       if nonNullVariants->Array.length == 1 {
         Nullable(
           fromSury(
@@ -102,7 +103,8 @@ let rec fromSury = (~parentName: string, ~fieldName: string, schema: S.t<unknown
             parentName ++
             fieldName->String.charAt(0)->String.toUpperCase ++
             fieldName->String.slice(~start=1, ~end=fieldName->String.length)
-          Enum(enumName, constValues)
+          let enum = Enum(enumName, constValues)
+          isOptional ? Nullable(enum) : enum
         } else {
           Unknown
         }

package/src/components/Api/SchemaType.res.mjs

@@ -83,7 +83,8 @@ function fromSury(parentName, fieldName, schema) {
         };
       }
     case "union" :
-      let nonNullVariants = schema.anyOf.filter(v => {
+      let anyOf = schema.anyOf;
+      let nonNullVariants = anyOf.filter(v => {
         switch (v.type) {
           case "null" :
           case "undefined" :
@@ -92,6 +93,7 @@ function fromSury(parentName, fieldName, schema) {
             return true;
         }
       });
+      let isOptional = nonNullVariants.length < anyOf.length;
       if (nonNullVariants.length === 1) {
         return {
           TAG: "Nullable",
@@ -111,11 +113,19 @@ function fromSury(parentName, fieldName, schema) {
         return "Unknown";
       }
       let enumName = parentName + fieldName.charAt(0).toUpperCase() + fieldName.slice(1, fieldName.length);
-      return {
+      let $$enum = {
         TAG: "Enum",
         _0: enumName,
         _1: constValues
       };
+      if (isOptional) {
+        return {
+          TAG: "Nullable",
+          _0: $$enum
+        };
+      } else {
+        return $$enum;
+      }
     default:
       return "Unknown";
   }

package/src/components/Dcb/Dcb_Builder.res

@@ -720,7 +720,7 @@ module Make = (
                     let result = await receiveFn(args)
                     let response =
                       result
-                      ->InboundTranslationSlice.receiveResultToOutcome
+                      ->InboundTranslationSlice_Callback.receiveResultToOutcome
                       ->CommandTopic.commandOutcomeToJson
                     response->Obj.magic
                   })

package/src/components/Dcb/Dcb_Builder.res.mjs

@@ -23,7 +23,6 @@ import * as ApiNoApiHelpers$ReventlessCore from "../Api/ApiNoApiHelpers.res.mjs"
 import * as Plugin_Structure$ReventlessCore from "../../plugin/component/Plugin_Structure.res.mjs";
 import * as DcbEventLog_Builder$ReventlessCore from "../DcbEventLog/DcbEventLog_Builder.res.mjs";
 import * as CommandTopic_Builder$ReventlessCore from "../CommandTopic/CommandTopic_Builder.res.mjs";
-import * as InboundTranslationSlice$ReventlessCore from "../InboundTranslationSlice/InboundTranslationSlice.res.mjs";
 import * as AutomationSlice_Callback$ReventlessCore from "../AutomationSlice/AutomationSlice_Callback.res.mjs";
 import * as CommandGenerator_Callback$ReventlessCore from "../CommandGenerator/CommandGenerator_Callback.res.mjs";
 import * as InboundTranslationSlice_Callback$ReventlessCore from "../InboundTranslationSlice/InboundTranslationSlice_Callback.res.mjs";
@@ -384,7 +383,7 @@ function Make(DcbEventLogStorage) {
             if (receiveFn !== undefined) {
               return Effect.promise(async () => {
                 let result = await receiveFn(args);
-                return CommandTopic$ReventlessCore.commandOutcomeToJson(InboundTranslationSlice$ReventlessCore.receiveResultToOutcome(result));
+                return CommandTopic$ReventlessCore.commandOutcomeToJson(InboundTranslationSlice_Callback$ReventlessCore.receiveResultToOutcome(result));
               });
             } else {
               return baseHandler(event, ctx);

package/src/components/InboundTranslationSlice/InboundTranslationSlice.res

@@ -5,31 +5,8 @@ type outputs = ReventlessInfra.InboundTranslationSlice.outputs
 type operations = ReventlessInfra.InboundTranslationSlice.operations
 type acceptedResult = ReventlessInfra.InboundTranslationSlice.acceptedResult
 type rejectedResult = ReventlessInfra.InboundTranslationSlice.rejectedResult
-type receiveResult = ReventlessInfra.InboundTranslationSlice.receiveResult
 type component = Component.t<t, outputs, operations>
 
-/**
-Map a `receive` outcome onto the `CommandResult` union the slice's mutation field
-declares, so both surfaces feed `CommandTopic.commandOutcomeToJson` rather than
-hand-rolling a second `__typename` writer.
-
-A translation can fan out across several targets; the mutation response reports
-the first target as `entityId` and the fan-out count as `eventCount`. The
-per-target detail stays queryable through the slice's audit read model. When the
-translation legitimately produced no command, `entityId` is omitted (encoding as
-`null`) rather than reporting a target that does not exist.
-*/
-let receiveResultToOutcome = (result: receiveResult): CommandTopic.commandOutcome =>
-  switch result {
-  | Ok({requestId, targetIds, commandCount}) =>
-    switch targetIds->Array.get(0) {
-    | Some(entityId) => Accepted({msgId: requestId, entityId, eventCount: commandCount})
-    | None => Accepted({msgId: requestId, eventCount: commandCount})
-    }
-  | Error({requestId, error}) =>
-    Rejected({msgId: requestId, errorCode: "TranslationFailed", errorDetail: Some(error)})
-  }
-
 let toResolvedOutputs = (
   outputs: outputs,
 ): Pulumi.Output.t<ReventlessInterop.InboundTranslationSlice.resolvedOutputs> =>

package/src/components/InboundTranslationSlice/InboundTranslationSlice.res.mjs

@@ -3,36 +3,6 @@
 import * as Pulumi from "@pulumi/pulumi";
 import * as Adapter$ReventlessCore from "../../adapter/Adapter.res.mjs";
 
-function receiveResultToOutcome(result) {
-  if (result.TAG === "Ok") {
-    let match = result._0;
-    let commandCount = match.commandCount;
-    let requestId = match.requestId;
-    let entityId = match.targetIds[0];
-    if (entityId !== undefined) {
-      return {
-        TAG: "Accepted",
-        msgId: requestId,
-        entityId: entityId,
-        eventCount: commandCount
-      };
-    } else {
-      return {
-        TAG: "Accepted",
-        msgId: requestId,
-        eventCount: commandCount
-      };
-    }
-  }
-  let match$1 = result._0;
-  return {
-    TAG: "Rejected",
-    msgId: match$1.requestId,
-    errorCode: "TranslationFailed",
-    errorDetail: match$1.error
-  };
-}
-
 function toResolvedOutputs(outputs) {
   return Pulumi.all([
     Adapter$ReventlessCore.resourcesToInterop(outputs.resources),
@@ -49,7 +19,6 @@ let componentType = "InboundTranslationSlice";
 
 export {
   componentType,
-  receiveResultToOutcome,
   toResolvedOutputs,
 }
 /* @pulumi/pulumi Not a pure module */

package/src/components/InboundTranslationSlice/InboundTranslationSlice_Callback.res

@@ -18,6 +18,34 @@ type auditRow = {
   receivedAt: string,
 }
 
+/**
+Map a `receive` outcome onto the `CommandResult` union the slice's mutation field
+declares, so both surfaces feed `CommandTopic.commandOutcomeToJson` rather than
+hand-rolling a second `__typename` writer.
+
+A translation can fan out across several targets; the mutation response reports
+the first target as `entityId` and the fan-out count as `eventCount`. The
+per-target detail stays queryable through the slice's audit read model. When the
+translation legitimately produced no command, `entityId` is omitted (encoding as
+`null`) rather than reporting a target that does not exist.
+
+Lives here rather than in `InboundTranslationSlice.res` because the deployed DCB
+command Lambda calls it at runtime, and the base module imports `@pulumi/pulumi`
+at deploy time — keeping this runtime-pure keeps Pulumi out of the Lambda graph.
+*/
+let receiveResultToOutcome = (
+  result: ReventlessInfra.InboundTranslationSlice.receiveResult,
+): CommandTopic.commandOutcome =>
+  switch result {
+  | Ok({requestId, targetIds, commandCount}) =>
+    switch targetIds->Array.get(0) {
+    | Some(entityId) => Accepted({msgId: requestId, entityId, eventCount: commandCount})
+    | None => Accepted({msgId: requestId, eventCount: commandCount})
+    }
+  | Error({requestId, error}) =>
+    Rejected({msgId: requestId, errorCode: "TranslationFailed", errorDetail: Some(error)})
+  }
+
 module type T = {
   module Spec: Reventless.InboundTranslationSlice.Spec
   module Translation: Reventless.InboundTranslationSlice.Translation with module Spec := Spec

package/src/components/InboundTranslationSlice/InboundTranslationSlice_Callback.res.mjs

@@ -25,6 +25,36 @@ let auditRowSchema = S.schema(s => ({
   receivedAt: s.m(S.string)
 }));
 
+function receiveResultToOutcome(result) {
+  if (result.TAG === "Ok") {
+    let match = result._0;
+    let commandCount = match.commandCount;
+    let requestId = match.requestId;
+    let entityId = match.targetIds[0];
+    if (entityId !== undefined) {
+      return {
+        TAG: "Accepted",
+        msgId: requestId,
+        entityId: entityId,
+        eventCount: commandCount
+      };
+    } else {
+      return {
+        TAG: "Accepted",
+        msgId: requestId,
+        eventCount: commandCount
+      };
+    }
+  }
+  let match$1 = result._0;
+  return {
+    TAG: "Rejected",
+    msgId: match$1.requestId,
+    errorCode: "TranslationFailed",
+    errorDetail: match$1.error
+  };
+}
+
 function Make(Spec) {
   return Translation => {
     let auditLog = {};
@@ -190,6 +220,7 @@ function Make(Spec) {
 export {
   auditStatusSchema,
   auditRowSchema,
+  receiveResultToOutcome,
   Make,
 }
 /*  Not a pure module */