@reventlessdev/reventless-core 3.0.0-alpha.178 → 3.0.0-alpha.180

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 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.180 (2026-07-24)
+
+### Features
+
+* **core:** Monitoring seam announces owning plugin/platform to backends ([79dd49a](https://github.com/ReventlessDev/reventless-core/commit/79dd49ae29feb95ed88c836cd8827c8e68978889))
+
+
+# 3.0.0-alpha.179 (2026-07-23)
+
+### Bug Fixes
+
+* **aws:** round-trip the DynamoDB Scan cursor in full-list connections ([0e21008](https://github.com/ReventlessDev/reventless-core/commit/0e2100860767a153a5c2f50a88fd6b501787c7f4))
+
+
 # 3.0.0-alpha.178 (2026-07-22)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-core",
-  "version": "3.0.0-alpha.178",
+  "version": "3.0.0-alpha.180",
   "description": "Core package for Reventless framework",
   "license": "Apache-2.0",
   "jest": {
@@ -29,16 +29,16 @@
     "sury": "11.0.0-alpha.4",
     "uuid": "^13.0.0",
     "@reventlessdev/rescript-effect": "0.1.0-alpha.30",
+    "@reventlessdev/rescript-jest": "1.0.0-alpha.9",
     "@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-ssh2": "1.1.0-alpha.14",
     "@reventlessdev/rescript-pulumi-pulumi": "2.3.0-alpha.17",
-    "@reventlessdev/rescript-uuid": "1.1.0-alpha.17",
+    "@reventlessdev/rescript-ssh2": "1.1.0-alpha.14",
     "@reventlessdev/reventless-interop": "3.0.0-alpha.29",
     "@reventlessdev/reventless-infra": "3.0.0-alpha.104",
-    "@reventlessdev/reventless-spec": "3.0.0-alpha.80"
+    "@reventlessdev/reventless-spec": "3.0.0-alpha.80",
+    "@reventlessdev/rescript-uuid": "1.1.0-alpha.17"
   },
   "devDependencies": {
     "rescript": "12.3.0",

package/src/adapter/Monitoring/Monitoring.res

@@ -58,18 +58,26 @@ FunctionName or a PrometheusRule deployment selector, but unusable as a logical
 resource name). Because `onProvisioned` fires during resource creation inside the
 deploy program, a Pulumi-based backend can create monitoring resources in the
 same stack, parented naturally.
+
+`~plugin` / `~platform` are the OWNING plugin and platform of the unit, taken from the ambient
+`ResourceAttribution` context the plugin builder publishes during construction (the same source
+`AWS_Tags` uses). Both are `None` for substrate provisioned outside any plugin construct — the
+correct answer for platform-scope resources. A backend that names or routes its monitoring resources
+per plugin needs this: `~name` is only the component, and two plugins can own like-named components.
 */
 module type Backend = {
   let onProvisioned: (
     ~kind: unitKind,
     ~name: string,
     ~component: ReventlessInfra.Adapter.resource,
+    ~plugin: option<string>,
+    ~platform: option<string>,
   ) => unit
 }
 
 /** Default backend: does nothing. Active unless an extension calls `use`. */
 module Noop: Backend = {
-  let onProvisioned = (~kind as _, ~name as _, ~component as _) => ()
+  let onProvisioned = (~kind as _, ~name as _, ~component as _, ~plugin as _, ~platform as _) => ()
 }
 
 let backend: ref<module(Backend)> = ref(module(Noop: Backend))
@@ -92,5 +100,9 @@ let notify = (
   ~component: ReventlessInfra.Adapter.resource,
 ) => {
   module B = unpack(backend.contents)
-  B.onProvisioned(~kind, ~name, ~component)
+  // The owning plugin/platform come from the ambient context the plugin builder publishes
+  // (ResourceAttribution.enter) — populated exactly when this fires, so the call sites need not
+  // thread the names. `None` outside a plugin construct (platform substrate).
+  let {ResourceAttribution.plugin: plugin, platform} = ResourceAttribution.current.contents
+  B.onProvisioned(~kind, ~name, ~component, ~plugin, ~platform)
 }

package/src/adapter/Monitoring/Monitoring.res.mjs

@@ -1,7 +1,8 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
+import * as ResourceAttribution$ReventlessCore from "../../ResourceAttribution.res.mjs";
 
-function onProvisioned(param, param$1, param$2) {
+function onProvisioned(param, param$1, param$2, param$3, param$4) {
   
 }
 
@@ -19,7 +20,8 @@ function use(b) {
 
 function notify(kind, name, component) {
   let B = backend.contents;
-  B.onProvisioned(kind, name, component);
+  let match = ResourceAttribution$ReventlessCore.current.contents;
+  B.onProvisioned(kind, name, component, match.plugin, match.platform);
 }
 
 export {

package/src/components/Api/GraphQL_FragmentGenerator.res

@@ -251,7 +251,7 @@ let validateScanSortAlignment = (
         None
       } else {
         Some(
-          `Read model "${readModelName}": @scanSort field "${field}" is not the sort key of any table or GSI. Sort requests for this field will be evaluated as a JS-runtime per-page sort over a full Scan — expensive in production. Add it as the sort key of an index, or accept the per-page-sort caveat.`,
+          `Read model "${readModelName}": @scanSort field "${field}" is not the sort key of any table or GSI. Sort requests Scan the whole table and sort per-page, so results past the first page are NOT globally ordered — and the Scan is expensive in production. Promote the field to an index sort key for correct, cheap ordering, or accept per-page ordering.`,
         )
       }
     )

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

@@ -221,7 +221,7 @@ function validateScanSortAlignment(schema, readModelName, knownSortFields) {
       if (knownSortFields.includes(field)) {
         return;
       } else {
-        return `Read model "` + readModelName + `": @scanSort field "` + field + `" is not the sort key of any table or GSI. Sort requests for this field will be evaluated as a JS-runtime per-page sort over a full Scan — expensive in production. Add it as the sort key of an index, or accept the per-page-sort caveat.`;
+        return `Read model "` + readModelName + `": @scanSort field "` + field + `" is not the sort key of any table or GSI. Sort requests Scan the whole table and sort per-page, so results past the first page are NOT globally ordered — and the Scan is expensive in production. Promote the field to an index sort key for correct, cheap ordering, or accept per-page ordering.`;
       }
     });
   } else {

package/tests/adapter/MonitoringTest.res

@@ -15,7 +15,8 @@ describe("Monitoring", () => {
   testSync("use registers a backend that receives every notify, role and name", () => {
     let recorded: array<(Monitoring.unitKind, string)> = []
     module Recorder: Monitoring.Backend = {
-      let onProvisioned = (~kind, ~name, ~component as _) => recorded->Array.push((kind, name))
+      let onProvisioned = (~kind, ~name, ~component as _, ~plugin as _, ~platform as _) =>
+        recorded->Array.push((kind, name))
     }
     Monitoring.use(module(Recorder: Monitoring.Backend))
 
@@ -38,7 +39,8 @@ describe("Monitoring", () => {
   testSync("notify forwards the static name and component resource to the backend", () => {
     let seen: ref<option<(string, ReventlessInfra.Adapter.resource)>> = ref(None)
     module Capture: Monitoring.Backend = {
-      let onProvisioned = (~kind as _, ~name, ~component) => seen := Some((name, component))
+      let onProvisioned = (~kind as _, ~name, ~component, ~plugin as _, ~platform as _) =>
+        seen := Some((name, component))
     }
     Monitoring.use(module(Capture: Monitoring.Backend))
 
@@ -50,4 +52,32 @@ describe("Monitoring", () => {
 
     Monitoring.use(module(Monitoring.Noop))
   })
+
+  testSync("notify delivers the ambient plugin/platform inside a construct scope, None outside", () => {
+    let seen: array<(option<string>, option<string>)> = []
+    module OwnerCapture: Monitoring.Backend = {
+      let onProvisioned = (~kind as _, ~name as _, ~component as _, ~plugin, ~platform) =>
+        seen->Array.push((plugin, platform))
+    }
+    Monitoring.use(module(OwnerCapture: Monitoring.Backend))
+
+    // Outside any plugin construct: platform substrate → both None.
+    Monitoring.notify(~kind=CommandHandler, ~name="ApiRouter", ~component=stubResource)
+
+    // Inside a construct scope (what Plugin_Builder.construct establishes): both delivered.
+    let prev = ResourceAttribution.enter(~platform="online-shop", ~plugin="Ordering")
+    Monitoring.notify(~kind=CommandHandler, ~name="AllAggregatesCmdHandler", ~component=stubResource)
+    ResourceAttribution.restore(prev)
+
+    // After restore: back to None.
+    Monitoring.notify(~kind=Projection, ~name="AllStateViewSlices", ~component=stubResource)
+
+    expect(seen)->toEqual([
+      (None, None),
+      (Some("Ordering"), Some("online-shop")),
+      (None, None),
+    ])
+
+    Monitoring.use(module(Monitoring.Noop))
+  })
 })