@rippling/rippling-sdk 0.2.0-alpha.37 → 0.2.0-alpha.38

package/src/lib/sdui/render.ts

@@ -1,5 +1,5 @@
-import type { SpecNode } from './components';
 import type { ActionBinding } from './actions';
+import type { SpecNode } from './components';
 
 export interface FlatElement {
   type: string;
@@ -31,10 +31,15 @@ export function render(stateProxy: { $initialValues: Record<string, unknown> },
       localStates.push(registration);
     }
 
-    const id = `${node.type.toLowerCase()}_${counter++}`;
-    const childIds = node.children.map(flattenNode).filter((id): id is string => id !== null);
+    counter += 1;
+    const id = `${node.type.toLowerCase()}_${counter}`;
+    const childIds = node.children.map(flattenNode).filter((childId): childId is string => childId !== null);
 
-    const element: FlatElement = { type: node.type, props: node.props, children: childIds };
+    const element: FlatElement = {
+      type: node.type,
+      props: node.props,
+      children: childIds,
+    };
     if (node.visible !== undefined) element.visible = node.visible;
     if (node.on) element.on = node.on;
     if (node.watch) element.watch = node.watch;
@@ -44,7 +49,10 @@ export function render(stateProxy: { $initialValues: Record<string, unknown> },
     return id;
   }
 
-  const rootId = flattenNode(rootElement)!;
+  const rootId = flattenNode(rootElement);
+  if (rootId === null) {
+    throw new Error('render(): root element produced no id');
+  }
 
   // Merge global state + local states from component() calls
   const state: Record<string, unknown> = { ...stateProxy.$initialValues };

package/src/lib/sdui/state.ts

@@ -7,7 +7,11 @@ export type BindStateRef = { $bindState: string };
 // This allows deep access regardless of the initial value (even null)
 type DeepStateProxy = {
   readonly [key: string]: DeepStateProxy;
-} & { readonly $state: string; readonly $bind: BindStateRef; readonly $path: string };
+} & {
+  readonly $state: string;
+  readonly $bind: BindStateRef;
+  readonly $path: string;
+};
 
 export type StateProxy<T> = {
   [K in keyof T]: T[K] extends Record<string, unknown> ? StateProxy<T[K]> & DeepStateProxy : DeepStateProxy;
@@ -28,7 +32,9 @@ export function extractPath(ref: unknown): string {
     try {
       const p = (ref as any).$path;
       if (typeof p === 'string') return p;
-    } catch {}
+    } catch {
+      // proxy access threw; fall through to the error below
+    }
   }
   throw new Error('Expected a state proxy or { $state } ref');
 }

package/src/version.ts

@@ -1 +1 @@
-export const VERSION = '0.2.0-alpha.37'; // x-release-please-version
+export const VERSION = '0.2.0-alpha.38'; // x-release-please-version

package/version.d.mts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.2.0-alpha.37";
+export declare const VERSION = "0.2.0-alpha.38";
 //# sourceMappingURL=version.d.mts.map

package/version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.2.0-alpha.37";
+export declare const VERSION = "0.2.0-alpha.38";
 //# sourceMappingURL=version.d.ts.map

package/version.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.VERSION = void 0;
-exports.VERSION = '0.2.0-alpha.37'; // x-release-please-version
+exports.VERSION = '0.2.0-alpha.38'; // x-release-please-version
 //# sourceMappingURL=version.js.map

package/version.mjs

@@ -1,2 +1,2 @@
-export const VERSION = '0.2.0-alpha.37'; // x-release-please-version
+export const VERSION = '0.2.0-alpha.38'; // x-release-please-version
 //# sourceMappingURL=version.mjs.map