@roku-sdk/solid 2.18.0 → 2.19.0

package/EXPORTS.md

@@ -192,6 +192,7 @@ These are SDK-owned symbols — not re-exports from `solid-js`. They power the R
 |---|---|---|
 | `registerTagAlias` | Function | `registerTagAlias(tag, resolve)` — create the SceneGraph node type `resolve()` returns whenever `tag` is created. Used by `@roku-sdk/runtime` and `@roku-sdk/compat` to choose between a firmware node and its shim; apps do not normally call it. |
 | `hasTagAlias` | Function | `hasTagAlias(tag)` — whether `tag` already has an alias, so a default can be registered without replacing one installed earlier. |
+| `registerSetPropertyObserver` | Function | `registerSetPropertyObserver(observer)` — invoke `observer(node, name, value)` after every `tmTransaction.setProperty`. Used by `__DEV__` diagnostics in `@roku-sdk/components` (Effect gradient validation); apps do not normally call it. Empty-list fast path when nothing is registered. |
 
 ### Renderer Runtime Hooks
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@roku-sdk/solid",
-  "version": "2.18.0",
+  "version": "2.19.0",
   "description": "Roku SceneGraph modules",
   "type": "module",
   "main": "src/index.ts",
@@ -28,13 +28,13 @@
     "node": ">=24.0.0"
   },
   "dependencies": {
-    "solid-js": "^1.9.10",
-    "@roku-sdk/common": "2.18.0",
-    "@roku-sdk/interfaces": "2.18.0"
+    "solid-js": "^1.9.17",
+    "@roku-sdk/common": "2.19.0",
+    "@roku-sdk/interfaces": "2.19.0"
   },
   "devDependencies": {
     "typescript": "^5.9.3",
-    "@roku-sdk/dev-tools": "2.18.0"
+    "@roku-sdk/dev-tools": "2.19.0"
   },
   "author": "Roku Inc",
   "license": "MIT",

package/src/solid.ts

@@ -173,6 +173,43 @@ type TmTransaction = {
     preCommit(): void;
 };
 
+/** Called after `tmTransaction.setProperty` applies a property (including event handlers). */
+export type SetPropertyObserver = (node: DomNode, name: string, value: TmValue) => void;
+
+const setPropertyObservers: SetPropertyObserver[] = [];
+
+/**
+ * Register a callback invoked after every successful `tmTransaction.setProperty`.
+ * Observers are skipped entirely when none are registered (empty-list fast path).
+ *
+ * Intended for `__DEV__` diagnostics that need to see intrinsic JSX props (e.g. Effect
+ * gradient validation). Production builds that never register pay only an array-length check.
+ *
+ * Returns an unsubscribe function.
+ */
+export const registerSetPropertyObserver = (observer: SetPropertyObserver): (() => void) => {
+    setPropertyObservers.push(observer);
+    return () => {
+        const index = setPropertyObservers.indexOf(observer);
+        if (index !== -1) {
+            setPropertyObservers.splice(index, 1);
+        }
+    };
+};
+
+const notifySetPropertyObservers = (node: DomNode, name: string, value: TmValue): void => {
+    if (setPropertyObservers.length === 0) return;
+    for (const observer of setPropertyObservers) {
+        // Observers are dev-only diagnostics; one that throws must not abort the
+        // transaction's setProperty call for every other observer and consumer.
+        try {
+            observer(node, name, value);
+        } catch (error) {
+            console.error("[solid] setProperty observer threw:", error);
+        }
+    }
+};
+
 export interface Renderer<NodeType> {
     render(code: () => NodeType, node: NodeType): () => void;
     effect<T>(fn: (prev?: T) => T, init?: T): void;
@@ -456,6 +493,7 @@ export const tmTransaction: TmTransaction = {
                 observerCallbacks.set(nodeRef, callbacks);
             }
             callbacks.set(propRef, value as () => void);
+            notifySetPropertyObservers(node, name, value);
             return;
         } else {
             switch (typeof value) {
@@ -499,6 +537,7 @@ export const tmTransaction: TmTransaction = {
         if (typeof prev === "object" && isDomNode(prev as DomNode)) {
             nodesToRemove.add(prev as DomNode);
         }
+        notifySetPropertyObservers(node, name, value);
     },
     insertNode: (parent: DomNode, node: DomNode, anchor?: DomNode): void => {
         // Do nothing if the node is anchor itself