@reventlessdev/reventless-aws 3.0.0-alpha.422 → 3.0.0-alpha.423

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 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.423 (2026-11-05)
+
+### Bug Fixes
+
+* **aws:** a line logged outside its invocation's fiber still names the plugin ([8e6f553](https://github.com/ReventlessDev/reventless-core/commit/8e6f553cc8ac2e439862bf1f3636ac86a4a22137))
+* **aws:** an extension's lines name its extension point once ([b448c59](https://github.com/ReventlessDev/reventless-core/commit/b448c5976feafd7d49ae8c04a67dcfbb72e973b5))
+
+
 # 3.0.0-alpha.422 (2026-11-05)
 
 **Note:** Version bump only for package @reventlessdev/reventless-aws

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-aws",
-  "version": "3.0.0-alpha.422",
+  "version": "3.0.0-alpha.423",
   "description": "AWS adapters for Reventless",
   "license": "Apache-2.0",
   "bin": {
@@ -24,17 +24,17 @@
     "yaml": "^2.8.3",
     "@reventlessdev/rescript-aws-sdk": "3.0.0-alpha.39",
     "@reventlessdev/rescript-effect": "1.0.0-alpha.1",
-    "@reventlessdev/rescript-node": "2.0.0-alpha.26",
-    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.39",
     "@reventlessdev/rescript-jest": "1.0.0-alpha.13",
-    "@reventlessdev/rescript-pulumi-pulumi": "3.0.0-alpha.0",
+    "@reventlessdev/rescript-pulumi-aws": "3.0.0-alpha.39",
+    "@reventlessdev/rescript-node": "2.0.0-alpha.26",
     "@reventlessdev/rescript-uuid": "2.0.0-alpha.1",
+    "@reventlessdev/rescript-pulumi-pulumi": "3.0.0-alpha.0",
     "@reventlessdev/rescript-web": "1.0.0-alpha.8",
+    "@reventlessdev/reventless-core": "3.0.0-alpha.330",
     "@reventlessdev/reventless-infra": "3.0.0-alpha.223",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.42",
     "@reventlessdev/reventless-spec": "3.0.0-alpha.194",
-    "@reventlessdev/reventless-postgres": "3.0.0-alpha.193",
-    "@reventlessdev/reventless-core": "3.0.0-alpha.329"
+    "@reventlessdev/reventless-postgres": "3.0.0-alpha.194"
   },
   "devDependencies": {
     "rescript": "12.3.1",

package/src/adapter/Runtime/EventCollectorEntryPoint.mjs

@@ -218,7 +218,7 @@ async function buildHandler() {
     };
     const transformedMapping = extensionMappingMake(fullMapping);
     const mappingsModule = {
-      name: ext.name,
+      name: Ops.mappingsName(ext),
       moduleUrl: ext.mappingsModule,
       mappings: [transformedMapping],
     };

package/src/adapter/Runtime/EventCollectorEntryPoint_Ops.res

@@ -561,6 +561,16 @@ let extensionPublishDicts = (names: queueUrlNames, ext: extensionConfig): extens
   {publishToAggregates: resolve(ext.aggregateNames, names.publishToAggregates)}
 }
 
+/** The name an extension's mappings carry, as at deploy: its plugin's. The config names the
+    extension component, `<extension point>.<plugin>`, and `Extension_Operations` prefixes the
+    extension point itself. */
+let mappingsName = (ext: extensionConfig): string => {
+  let prefix = ext.extensionPointName ++ "."
+  ext.name->String.startsWith(prefix)
+    ? ext.name->String.slice(~start=prefix->String.length, ~end=ext.name->String.length)
+    : ext.name
+}
+
 let warnSkippedExtension = (ext: extensionConfig) =>
   logWarn(
     `extension '${ext.name}' missing module specifier(s); skipping. specModule=${ext.specModule}, mappingsModule=${ext.mappingsModule}, delegateModule=${ext.delegateModule}`,

package/src/adapter/Runtime/EventCollectorEntryPoint_Ops.res.mjs

@@ -445,6 +445,15 @@ function extensionPublishDicts(names, ext) {
   };
 }
 
+function mappingsName(ext) {
+  let prefix = ext.extensionPointName + ".";
+  if (ext.name.startsWith(prefix)) {
+    return ext.name.slice(prefix.length, ext.name.length);
+  } else {
+    return ext.name;
+  }
+}
+
 function warnSkippedExtension(ext) {
   HandlerFactoryHelpersMjs.log.warn(`extension '` + ext.name + `' missing module specifier(s); skipping. specModule=` + ext.specModule + `, mappingsModule=` + ext.mappingsModule + `, delegateModule=` + ext.delegateModule, {
     comp: "EventCollectorEntryPoint"
@@ -592,6 +601,7 @@ export {
   makePublishJsons,
   buildPublishToAggregates,
   extensionPublishDicts,
+  mappingsName,
   warnSkippedExtension,
   assetPath,
   loadPluginDefinition,

package/src/adapter/Runtime/HandlerFactoryHelpers.mjs

@@ -6,6 +6,7 @@ import {
   tag as requestContextTag,
   fromOptions as makeRequestContext,
 } from "@reventlessdev/reventless-core/src/RequestContext.res.mjs";
+import { currentPluginName } from "@reventlessdev/reventless-spec/src/LogPrefix.res.mjs";
 import { DynamoDBClient } from "@aws-sdk/client-dynamodb";
 import { DynamoDBDocumentClient, ScanCommand } from "@aws-sdk/lib-dynamodb";
 
@@ -120,6 +121,19 @@ export function runEffect(effect, opts = {}) {
   // Only annotate a redelivery. A constant "1" on every line is noise, and its
   // absence is what makes `filter retryCount > 1` a usable query.
   if (retryCount > 1) e = e.pipe(Effect.annotateLogs("retryCount", String(retryCount)));
+  // A line run on a fiber of its own (`->Effect.runSync`) misses these annotations, and the
+  // component→plugin registry is empty in a Lambda: it names its plugin from the ambient one,
+  // set for the invocation and restored after it.
+  if (resolvedPlugin) {
+    const inner = e;
+    e = Effect.suspend(() => {
+      const before = currentPluginName.contents;
+      currentPluginName.contents = resolvedPlugin;
+      return inner.pipe(
+        Effect.ensuring(Effect.sync(() => { currentPluginName.contents = before; })),
+      );
+    });
+  }
   return e
     .pipe(
       Effect.provideService(

package/tests/EventCollectorEntryPoint_OpsTest.res

@@ -226,3 +226,23 @@ describe("EventCollectorEntryPoint_Ops.parseHandlerConfig", () => {
     expect((c.extensions->Array.getUnsafe(0)).extensionPointName)->toBe("Ordering.Orders")
   })
 })
+
+describe("EventCollectorEntryPoint_Ops.mappingsName", () => {
+  let ext = (~name): EventCollectorEntryPoint_Ops.extensionConfig => {
+    name,
+    specModule: "",
+    mappingsModule: "",
+    delegateModule: "",
+    extensionPointName: "Ordering.Orders",
+    aggregateNames: [],
+  }
+  // The component is `Extension(<point>.<name>)`: the point is not named twice.
+  testSync("is the plugin's name, as at deploy", () =>
+    expect(EventCollectorEntryPoint_Ops.mappingsName(ext(~name="Ordering.Orders.Catalog")))->toBe(
+      "Catalog",
+    )
+  )
+  testSync("keeps a name that does not start with its extension point", () =>
+    expect(EventCollectorEntryPoint_Ops.mappingsName(ext(~name="Catalog")))->toBe("Catalog")
+  )
+})

package/tests/EventCollectorEntryPoint_OpsTest.res.mjs

@@ -315,6 +315,29 @@ globalThis.describe("EventCollectorEntryPoint_Ops.parseHandlerConfig", () => {
   });
 });
 
+globalThis.describe("EventCollectorEntryPoint_Ops.mappingsName", () => {
+  globalThis.test("is the plugin's name, as at deploy", () => {
+    globalThis.expect(EventCollectorEntryPoint_Ops$ReventlessAws.mappingsName({
+      name: "Ordering.Orders.Catalog",
+      specModule: "",
+      mappingsModule: "",
+      delegateModule: "",
+      extensionPointName: "Ordering.Orders",
+      aggregateNames: []
+    })).toBe("Catalog");
+  });
+  globalThis.test("keeps a name that does not start with its extension point", () => {
+    globalThis.expect(EventCollectorEntryPoint_Ops$ReventlessAws.mappingsName({
+      name: "Catalog",
+      specModule: "",
+      mappingsModule: "",
+      delegateModule: "",
+      extensionPointName: "Ordering.Orders",
+      aggregateNames: []
+    })).toBe("Catalog");
+  });
+});
+
 let project = EventCollectorEntryPoint_Ops$ReventlessAws.projectPluginRow;
 
 export {

package/tests/HandlerFactoryDispatchTest.res

@@ -200,6 +200,23 @@ describe("entry-point dispatch boundary", () => {
       },
     )
 
+    // A line run on a fiber of its own (`->Effect.runSync`) misses the invocation's annotations,
+    // and a Lambda's component→plugin registry is empty: the ambient plugin names it.
+    test(
+      "a line logged on a fiber of its own names the invocation's plugin",
+      async () => {
+        let before = Reventless.LogPrefix.currentPluginName.contents
+        let seen = await runEffect(
+          Effect.sync(
+            () => Reventless.LogPrefix.resolvePlugin(~comp="StateChangeSlice(PlaceOrder)", ()),
+          ),
+          {correlationId: "c", comp: "StateChangeSlice(PlaceOrder)", plugin: "Ordering"},
+        )
+        expect(seen)->Expect.toEqual(Some("Ordering"))
+        expect(Reventless.LogPrefix.currentPluginName.contents)->Expect.toEqual(before)
+      },
+    )
+
     // A constant "1" on every line is noise; its absence is what makes
     // `filter retryCount > 1` a usable CloudWatch query.
     test(

package/tests/HandlerFactoryDispatchTest.res.mjs

@@ -3,6 +3,7 @@
 import * as Effect from "@reventlessdev/rescript-effect/src/Effect.res.mjs";
 import * as Effect$1 from "effect/Effect";
 import * as Identity$Reventless from "@reventlessdev/reventless-spec/src/types/Identity.res.mjs";
+import * as LogPrefix$Reventless from "@reventlessdev/reventless-spec/src/LogPrefix.res.mjs";
 import * as RequestContext$ReventlessCore from "@reventlessdev/reventless-core/src/RequestContext.res.mjs";
 import * as HandlerFactoryHelpersMjs from "@reventlessdev/reventless-aws/src/adapter/Runtime/HandlerFactoryHelpers.mjs";
 
@@ -112,6 +113,16 @@ globalThis.describe("entry-point dispatch boundary", () => {
       globalThis.expect(annotated$1["correlationId"]).toEqual("corr-1");
       globalThis.expect(annotated$1["requestId"]).toEqual("req-abc");
     });
+    globalThis.test("a line logged on a fiber of its own names the invocation's plugin", async () => {
+      let before = LogPrefix$Reventless.currentPluginName.contents;
+      let seen = await HandlerFactoryHelpersMjs.runEffect(Effect$1.sync(() => LogPrefix$Reventless.resolvePlugin("StateChangeSlice(PlaceOrder)", undefined)), {
+        correlationId: "c",
+        comp: "StateChangeSlice(PlaceOrder)",
+        plugin: "Ordering"
+      });
+      globalThis.expect(seen).toEqual("Ordering");
+      globalThis.expect(LogPrefix$Reventless.currentPluginName.contents).toEqual(before);
+    });
     globalThis.test("annotates retryCount only on a redelivery", async () => {
       let first = await HandlerFactoryHelpersMjs.runEffect(Effect$1.logAnnotations, {
         correlationId: "c",