change runOn to return undefined instead of null. add @babel/parser as dep instead of dev dep

Author: mickmister

packages/springboard/cli/package.json

@@ -34,6 +34,9 @@
     "test:watch": "vitest"
   },
   "dependencies": {
+    "@babel/generator": "^7.28.5",
+    "@babel/parser": "^7.28.5",
+    "@babel/traverse": "^7.28.5",
     "commander": "catalog:",
     "concurrently": "^9.1.0",
     "esbuild": "catalog:",
@@ -50,9 +53,6 @@
   "author": "",
   "license": "ISC",
   "devDependencies": {
-    "@babel/generator": "^7.28.5",
-    "@babel/parser": "^7.28.5",
-    "@babel/traverse": "^7.28.5",
     "@babel/types": "^7.28.5",
     "@types/babel__generator": "^7.27.0",
     "@types/babel__traverse": "^7.28.0"

packages/springboard/cli/src/esbuild_plugins/esbuild_plugin_platform_inject.ts

@@ -12,7 +12,7 @@ import type {SpringboardPlatform} from 'springboard/engine/register';
  *
  * **Responsibilities:**
  * 1. **Platform directive blocks** - Remove `@platform "..."` blocks for non-matching platforms
- * 2. **springboard.runOn()** - Transform to IIFE (if platform matches) or `null` (if doesn't match)
+ * 2. **springboard.runOn()** - Transform to IIFE (if platform matches) or `undefined` (if doesn't match)
  * 3. **Server state removal** - Remove `createServerStates()` variable declarations in client builds
  * 4. **Server action stripping** - Strip bodies from `createServerActions()` in client builds
  *
@@ -25,7 +25,7 @@ import type {SpringboardPlatform} from 'springboard/engine/register';
  * | `cf-workers` | `'cf-workers'`, `'server'` |
  * | `web`  | `'web'`, `'browser'`, `'client'`, `'user-agent'` |
  * | `tauri` | `'tauri'`, `'browser'`, `'client'`, `'user-agent'` |
- * | `react-native-web` | `'react-native-web'`, `'browser'`, `'client'` |
+ * | `react-native-webview` | `'react-native-webview'`, `'browser'`, `'client'` |
  * | `react-native` | `'react-native'`, `'user-agent'` |
  *
  * @see packages/springboard/core/engine/register.ts - TypeScript type definitions
@@ -46,7 +46,7 @@ export const esbuildPluginPlatformInject = (
                 return false;
             case 'web':
             case 'tauri':
-            case 'react-native-web':
+            case 'react-native-webview':
             case 'react-native':
                 // Client platforms - strip server states/actions
                 return true;
@@ -64,7 +64,7 @@ export const esbuildPluginPlatformInject = (
                 let source = await fs.promises.readFile(args.path, 'utf8');
 
                 // Early return if file doesn't need any transformations
-                const hasPlatformAnnotations = /@platform "(node|cf-workers|web|tauri|react-native|react-native-web|server|browser|client|user-agent)"/.test(source);
+                const hasPlatformAnnotations = /@platform "(node|cf-workers|web|tauri|react-native|react-native-webview|server|browser|client|user-agent)"/.test(source);
                 // Detect both old and new API patterns for server calls
                 const hasServerCalls = /createServer(State|States|Action|Actions)/.test(source) ||
                                        /\.server\.createServer(States|Actions)/.test(source);
@@ -95,8 +95,8 @@ export const esbuildPluginPlatformInject = (
                                    targetPlatform === 'browser' ||
                                    targetPlatform === 'client' ||
                                    targetPlatform === 'user-agent';
-                        case 'react-native-web':
-                            return targetPlatform === 'react-native-web' ||
+                        case 'react-native-webview':
+                            return targetPlatform === 'react-native-webview' ||
                                    targetPlatform === 'browser' ||
                                    targetPlatform === 'client';
                         case 'react-native':
@@ -170,9 +170,9 @@ export const esbuildPluginPlatformInject = (
                                                            targetPlatform === 'browser' ||
                                                            targetPlatform === 'client' ||
                                                            targetPlatform === 'user-agent';
-                                                case 'react-native-web':
-                                                    // react-native-web build accepts: 'react-native-web', 'browser' and context 'client'
-                                                    return targetPlatform === 'react-native-web' ||
+                                                case 'react-native-webview':
+                                                    // react-native-webview build accepts: 'react-native-webview', 'browser' and context 'client'
+                                                    return targetPlatform === 'react-native-webview' ||
                                                            targetPlatform === 'browser' ||
                                                            targetPlatform === 'client';
                                                 case 'react-native':
@@ -194,9 +194,10 @@ export const esbuildPluginPlatformInject = (
                                                 arguments: [],
                                             } as any);
                                         } else {
-                                            // Replace with null
+                                            // Replace with undefined
                                             path.replaceWith({
-                                                type: 'NullLiteral',
+                                                type: 'Identifier',
+                                                name: 'undefined',
                                             } as any);
                                         }
                                     }

packages/springboard/core/engine/register.ts

@@ -40,7 +40,7 @@ export type SpringboardRegistry = {
      * | `web`  | `'web'`, `'browser'`, `'client'`, `'user-agent'` |
      * | `tauri` | `'tauri'`, `'browser'`, `'client'`, `'user-agent'` |
      * | `browser` | `'browser'`, `'web'`, `'tauri'`, `'client'`, `'user-agent'` (meta-target) |
-     * | `react-native-web` | `'react-native-web'`, `'browser'`, `'client'` |
+     * | `react-native-webview` | `'react-native-webview'`, `'browser'`, `'client'` |
      * | `react-native` | `'react-native'`, `'user-agent'` |
      *
      * **Async Support:** Callbacks can be sync or async. Use `await` if the callback returns a Promise:
@@ -87,7 +87,7 @@ export type SpringboardRegistry = {
     runOn: <T>(
         platform: SpringboardPlatform | SpringboardPlatformContext,
         callback: () => T
-    ) => T | null;
+    ) => T | undefined;
 
     /**
      * Check if the current runtime matches a platform or context at runtime.
@@ -107,7 +107,7 @@ export type SpringboardRegistry = {
      * | `cf-workers` | `'cf-workers'`, `'server'` |
      * | `web`  | `'web'`, `'browser'`, `'client'`, `'user-agent'` |
      * | `tauri` | `'tauri'`, `'browser'`, `'client'`, `'user-agent'` |
-     * | `react-native-webview` | `'browser'`, `'client'`
+     * | `react-native-webviewview` | `'browser'`, `'client'`
      * | `react-native` | `'react-native'`, `'user-agent'` |
      *
      * **Implementation:** Transformed by platform macros in build plugin to return
@@ -177,35 +177,22 @@ export const getRegisteredSplashScreen = (): React.ComponentType | null => {
 };
 
 /**
- * Runtime stub for `springboard.runOn()` - FOR DEVELOPMENT/FALLBACK ONLY.
- *
- * **Expected Behavior:**
- * This function should NEVER be called in production builds. The esbuild plugin
- * (`packages/springboard/cli/src/esbuild_plugins/esbuild_plugin_platform_inject.ts`)
- * transforms all `runOn` calls at compile time:
+ * Runtime stub for `springboard.runOn()`
  *
  * - **Platform matches:** `runOn('node', cb)` → `cb()` (immediate execution)
- * - **Platform doesn't match:** `runOn('browser', cb)` → `null` (removed)
- *
- * **This stub:**
- * - Exists for TypeScript type checking and IDE autocomplete
- * - Provides fallback behavior in non-transformed environments (development, tests)
- * - Simply executes the callback immediately (simulating "platform matches" behavior)
+ * - **Platform doesn't match:** `runOn('browser', cb)` → `undefined` (removed)
  *
  * **Platform parameter:**
  * Accepts either platform names (`'node'`, `'browser'`, etc.) or contexts (`'server'`, `'client'`, etc.).
- * The esbuild plugin (lines 97-118 of esbuild_plugin_platform_inject.ts) handles the platform
+ * The esbuild plugin (esbuild_plugin_platform_inject.ts) handles the platform
  * matching logic based on the build target using a switch statement.
  *
  * @internal
  */
 const runOn = <T>(
-    platform: SpringboardPlatform | SpringboardPlatformContext,
+    _platform: SpringboardPlatform | SpringboardPlatformContext,
     callback: () => T
-): T | null => {
-    // Development/test fallback: execute callback immediately
-    // In production, this code is replaced by the esbuild plugin transformation
-    void platform; // Unused in runtime stub
+): T | undefined => {
     return callback();
 };
 
@@ -224,7 +211,7 @@ const SPRINGBOARD_PLATFORMS = {
     WEB: 'web',
     TAURI: 'tauri',
     REACT_NATIVE: 'react-native',
-    REACT_NATIVE_WEBVIEW: 'react-native-web',
+    REACT_NATIVE_WEBVIEW: 'react-native-webview',
 } as const;
 
 export type SpringboardPlatform = typeof SPRINGBOARD_PLATFORMS[keyof typeof SPRINGBOARD_PLATFORMS];
@@ -244,7 +231,7 @@ export type SpringboardPlatform = typeof SPRINGBOARD_PLATFORMS[keyof typeof SPRI
  * | `web`  | `'web'`, `'browser'`, `'client'`, `'user-agent'` |
  * | `tauri` | `'tauri'`, `'browser'`, `'client'`, `'user-agent'` |
  * | `browser` | `'browser'`, `'web'`, `'tauri'`, `'client'`, `'user-agent'` (meta-target) |
- * | `react-native-web` | `'react-native-web'`, `'browser'`, `'client'` |
+ * | `react-native-webview` | `'react-native-webview'`, `'browser'`, `'client'` |
  * | `react-native` | `'react-native'`, `'user-agent'` |
  */
 const isPlatform = (platform: SpringboardPlatform | SpringboardPlatformContext): boolean => {