sinwan-compiler 0.2.7 → 0.2.9

package/dist/{chunk-edjqytm9.js → chunk-6dnsvgyf.js}

@@ -43,8 +43,17 @@ function isBuiltinReactiveProp(componentName, attrName) {
     return false;
   return entry.reactiveProps.has(attrName);
 }
-function isBuiltinControlFlowComponent(componentName) {
-  return !!componentName && BUILTIN_REACTIVE_PROPS.has(componentName);
+function isReactiveContainerPassThrough(expr, scope) {
+  let node = expr;
+  while (t.isTSNonNullExpression(node)) {
+    node = node.expression;
+  }
+  if (!t.isIdentifier(node))
+    return false;
+  const binding = scope.bindings.get(node.name);
+  if (!binding)
+    return false;
+  return binding.kind === "mutable" || binding.kind === "prop";
 }
 function trackReactiveImports2(ast) {
   const names = {
@@ -291,6 +300,35 @@ function analyzeObjectLiteralSpread(spread, fullScope, calleeReactive) {
 function isMemberLike(node) {
   return t.isMemberExpression(node) || t.isOptionalMemberExpression(node);
 }
+function isFunctionLike(node) {
+  return t.isArrowFunctionExpression(node) || t.isFunctionExpression(node);
+}
+function isCallLike(node) {
+  return t.isCallExpression(node) || t.isOptionalCallExpression(node);
+}
+var COMPILER_BINDING_CALLEES = new Set([
+  "_$bindText",
+  "_$bindAttr",
+  "_$bindStyle",
+  "_$bindClass",
+  "_$unwrap"
+]);
+function isCompilerBindingCall(node) {
+  const callee = node.callee;
+  return t.isIdentifier(callee) && COMPILER_BINDING_CALLEES.has(callee.name);
+}
+function visitCallCalleeAndArgs(node, visitExpr) {
+  if (isCompilerBindingCall(node))
+    return;
+  visitExpr(node.callee, true);
+  for (const arg of node.arguments) {
+    if (t.isSpreadElement(arg)) {
+      visitExpr(arg.argument, true);
+      continue;
+    }
+    visitExpr(arg, true);
+  }
+}
 function getMemberExpressionRootAndPath(node) {
   const path = [];
   let current = node;
@@ -428,11 +466,23 @@ function containsReactiveValue2(expr, scope) {
 }
 function containsReactiveRead(expr, scope) {
   let found = false;
-  function visit(node) {
+  function visit(node, enterFunction) {
     if (found)
       return;
     if (!node || typeof node !== "object")
       return;
+    if (isFunctionLike(node)) {
+      if (!enterFunction)
+        return;
+      for (const param of node.params) {
+        visit(param, false);
+      }
+      visit(node.body, false);
+      return;
+    }
+    if (t.isJSXElement(node) || t.isJSXFragment(node)) {
+      return;
+    }
     if (isMemberLike(node) || t.isTSNonNullExpression(node) || t.isIdentifier(node)) {
       if (isReactiveRead(node, scope)) {
         found = true;
@@ -443,7 +493,8 @@ function containsReactiveRead(expr, scope) {
       found = true;
       return;
     }
-    if (t.isArrowFunctionExpression(node) || t.isFunctionExpression(node)) {
+    if (isCallLike(node)) {
+      visitCallCalleeAndArgs(node, visit);
       return;
     }
     for (const key of Object.keys(node)) {
@@ -453,14 +504,14 @@ function containsReactiveRead(expr, scope) {
       if (Array.isArray(value)) {
         for (const item of value) {
           if (item && typeof item === "object")
-            visit(item);
+            visit(item, false);
         }
       } else if (value && typeof value === "object") {
-        visit(value);
+        visit(value, false);
       }
     }
   }
-  visit(expr);
+  visit(expr, false);
   return found;
 }
 function getComponentExpressionInfo(exprPath) {
@@ -552,11 +603,21 @@ function transformPropMemberAccess(expr, scope) {
       parent.object = resolveRoot(rootName);
     }
   }
-  function visit(node) {
+  function visit(node, enterFunction) {
     if (!node || typeof node !== "object")
       return;
-    if (t.isArrowFunctionExpression(node) || t.isFunctionExpression(node))
+    if (isFunctionLike(node)) {
+      if (!enterFunction)
+        return;
+      for (const param of node.params) {
+        visit(param, false);
+      }
+      visit(node.body, false);
       return;
+    }
+    if (t.isJSXElement(node) || t.isJSXFragment(node)) {
+      return;
+    }
     if (isMemberLike(node)) {
       const rootPath = getMemberExpressionRootAndPath(node);
       if (rootPath) {
@@ -569,6 +630,10 @@ function transformPropMemberAccess(expr, scope) {
       }
       return;
     }
+    if (isCallLike(node)) {
+      visitCallCalleeAndArgs(node, visit);
+      return;
+    }
     for (const key of Object.keys(node)) {
       if (key === "loc" || key === "start" || key === "end" || key === "leadingComments" || key === "trailingComments")
         continue;
@@ -576,14 +641,14 @@ function transformPropMemberAccess(expr, scope) {
       if (Array.isArray(value)) {
         for (const item of value) {
           if (item && typeof item === "object")
-            visit(item);
+            visit(item, false);
         }
       } else if (value && typeof value === "object") {
-        visit(value);
+        visit(value, false);
       }
     }
   }
-  visit(expr);
+  visit(expr, false);
   return changed;
 }
 function wrapExpression(expr) {
@@ -611,7 +676,8 @@ function createBindingDescriptor(expr, exprPath, options) {
       wrapExpression(expr)
     ]);
   }
-  if (t.isIdentifier(expr, { name: "children" })) {
+  const compInfo = getComponentExpressionInfo(exprPath);
+  if (t.isIdentifier(expr, { name: "children" }) || compInfo.isComponent && compInfo.attributeName === null) {
     return wrapExpression(expr);
   }
   return t.callExpression(t.identifier("_$bindText"), [wrapExpression(expr)]);
@@ -1050,15 +1116,10 @@ function wrapReactiveExpressions2(ast, options = {}) {
         JSXExpressionContainer(exprPath) {
           const expr = exprPath.node.expression;
           const compInfo = getComponentExpressionInfo(exprPath);
-          if (compInfo.isComponent) {
-            if (compInfo.attributeName !== null) {
-              if (!compInfo.componentName || !isReactiveComponentProp(compInfo.componentName, compInfo.attributeName, componentNames, reactiveProps, importedReactiveProps)) {
-                return;
-              }
-            } else {
-              if (!isBuiltinControlFlowComponent(compInfo.componentName)) {
-                return;
-              }
+          if (compInfo.isComponent && compInfo.attributeName !== null) {
+            const knownProp = !!compInfo.componentName && isReactiveComponentProp(compInfo.componentName, compInfo.attributeName, componentNames, reactiveProps, importedReactiveProps);
+            if (!knownProp && (!shouldWrap(expr, scope) || isReactiveContainerPassThrough(expr, scope))) {
+              return;
             }
           }
           if (shouldWrap(expr, scope)) {

package/dist/cli.js

@@ -3,7 +3,7 @@
 import {
   runAnalyzeCli2,
   runCliIfMain2
-} from "./chunk-edjqytm9.js";
+} from "./chunk-6dnsvgyf.js";
 export {
   runAnalyzeCli2 as runAnalyzeCli,
   runCliIfMain2 as runCliIfMain

package/dist/index.js

@@ -17,7 +17,7 @@ import {
   propagateReactiveProps2,
   wrapReactiveExpressions2,
   runAnalyzeCli2
-} from "./chunk-edjqytm9.js";
+} from "./chunk-6dnsvgyf.js";
 
 // src/transform.ts
 import { parse } from "@babel/parser";

package/dist/transform.d.ts

@@ -19,9 +19,9 @@ export interface TransformOptions {
     /**
      * Resolves an import source (e.g. "./Child") to an absolute file path,
      * relative to the file being transformed. Used to look up reactive-prop
-     * metadata for components imported from other modules so reactive values
-     * passed to them are wrapped at the call site. When omitted, cross-module
-     * component call sites fall back to the conservative (non-wrapping) behavior.
+     * metadata for components imported from other modules. Derived reads
+     * (`title={n.value}`) wrap even without metadata; bare signal/store
+     * identifiers stay unwrapped.
      */
     resolveImport?: (source: string, fromFile: string) => string | null;
 }

package/docs/README.md

@@ -19,6 +19,8 @@ autoWrapComponents()     plain `export function App()` → cc(App)
     │
     ▼
 wrapReactiveExpressions()   {count.value} → {() => count.value}
+                            {arr.map(x => n.value)} → {() => arr.map(...)}
+                            also wraps those reads as children/attrs of <Label>
     │
     ▼
 hoist templates             <div class="card">…</div> → _$tmpl_N

package/docs/plugins.md

@@ -51,6 +51,7 @@ The Vite plugin sets `resolve.dedupe: ["sinwan"]` for the same duplicate-runtime
 | Option | Default | Role |
 | ------ | ------- | ---- |
 | `hoist` | `true` | Template hoisting |
+| `dev` | Bun: `NODE_ENV !== "production"` at `sinwan()`; Vite: `config.mode !== "production"` | Warn when hoisting is skipped. Pass `false` for library/`Bun.build` production bundles. |
 | `explicitBindings` | `false` | `_$bind*` helpers instead of bare `() =>` |
 | `analyze` | unset | Path to CLI JSON (production) |
 | `cache` | `true` | Incremental analyzer in dev. `false` turns it off. An object sets `root`, `tsConfigPath`, `bunfigPath`, `workspaces`, `cachePath` |

package/docs/transform.md

@@ -22,6 +22,8 @@ The compiler turns the child into a lazy getter:
 <p>{() => count.value}</p>
 ```
 
+The same wrap applies to **user-component children**. `<Label>{checked.value ? "On" : "Off"}</Label>` becomes `{() => checked.value ? "On" : "Off"}` so a one-shot `cc()` parent does not snapshot the string.
+
 The runtime runs that function inside an effect and updates the text when `count` changes.
 
 ## Automatic `cc()` wrapping
@@ -70,12 +72,14 @@ Reads the compiler wraps:
 - `useState` getters: `count()`
 - `useFetch` signal values: `f.data.value` (not the shell `f` or `f.data`)
 - Calls to a **local** function whose body reads one of the above
+- Reads **inside call callbacks** that run while the JSX expression is evaluated: `.map` / `.filter` / `Array.from` callbacks, IIFEs, and default parameters of those callbacks. `<For each={keys.map((k) => groups[query.value])}>` becomes `each={() => keys.map(...)}`. Nested JSX inside those callbacks is a separate wrap site (`items.map((i) => <span>{count.value}</span>)` wraps the inner text, not the whole map).
 
 Not wrapped:
 
 - Literals and plain identifiers
 - Event-handler functions (`onClick={fn}` or `onClick={() => …}`)
 - Nested function *values* passed as props (`title={() => s.value}` stays a render prop)
+- Functions stored as object/array values (`{ onClick: () => s.value }`) or returned from a factory (`const make = () => () => s.value`)
 - `useFetch` methods such as `abort` / `execute`
 - Static values the [analyzer](analyzer.md) proved never change
 
@@ -83,9 +87,12 @@ Not wrapped:
 
 **Native elements** (`div`, `p`, `button`, …): wrap reactive children and attributes.
 
-**User components** (`<Child title={…} />`): wrap a prop only if that prop is reactive for `Child` (local call graph, analyzer metadata, or a conservative fallback for exported components). Children forwarded to a user component stay unwrapped so the child can read `children` itself.
+**User components** (`<Label>`, `<Child title={…} />`):
+
+- **Children:** wrap reactive reads the same way as DOM text (`{count.value}`, `{checked.value ? "On" : "Off"}`) so `cc()` parents do not snapshot the value. Use a getter (`() => …`), never `_$bindText`, because the child may render nodes as well as strings. Render-prop functions (`{(v) => …}`) stay untouched.
+- **Props:** wrap a prop if it is known-reactive for that component (built-in registry, local call graph, analyzer metadata). For *unknown* imported components, still wrap **derived reads** (`htmlFor={id.value}`, `disabled={off.value}`) and leave **container pass-through** alone (`checked={checked}`, `user={user}`) so the child receives the signal or store proxy.
 
-**Built-in control-flow** components wrap specific props, and they *do* wrap reactive expression children (those children render directly):
+**Built-in control-flow** components wrap specific props, and they wrap reactive expression children (those children render directly):
 
 | Component | Reactive props |
 | --------- | -------------- |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sinwan-compiler",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "description": "Shared compiler core for Sinwan plugins",
   "type": "module",
   "main": "./dist/index.js",