sinwan-compiler 0.2.7 → 0.2.8

package/dist/{chunk-edjqytm9.js → chunk-v4zrz7j7.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 = {
@@ -611,7 +620,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 +1060,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-v4zrz7j7.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-v4zrz7j7.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,7 @@ autoWrapComponents()     plain `export function App()` → cc(App)
     │
     ▼
 wrapReactiveExpressions()   {count.value} → {() => count.value}
+                            also wraps those reads as children/attrs of <Label>
     │
     ▼
 hoist templates             <div class="card">…</div> → _$tmpl_N

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
@@ -83,9 +85,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.8",
   "description": "Shared compiler core for Sinwan plugins",
   "type": "module",
   "main": "./dist/index.js",