@sprlab/wccompiler 0.16.2 → 0.16.3

package/lib/codegen.js

@@ -57,6 +57,23 @@ function slotPropRef(source, signalNames, computedNames, propNames) {
   return `'${source}'`;
 }
 
+/**
+ * Wrap an expression in parentheses if it contains a ternary operator.
+ * This prevents operator precedence issues when combining with ?? (nullish coalescing).
+ *
+ * @param {string} expr - Expression to potentially wrap
+ * @returns {string} - Wrapped expression if it contains ternary, otherwise unchanged
+ */
+function wrapTernaryExpr(expr) {
+  const trimmed = expr.trim();
+  // Check if expression contains ternary operator (? followed by :)
+  // Simple heuristic: if it has both ? and :, wrap it
+  if (trimmed.includes('?') && trimmed.includes(':')) {
+    return `(${trimmed})`;
+  }
+  return trimmed;
+}
+
 /**
  * Transform an expression by rewriting signal/computed variable references
  * to use `this._x()` / `this._c_x()` syntax for auto-unwrapping.
@@ -444,7 +461,7 @@ function generateItemSetup(lines, forBlock, itemVar, indexVar, propNames, signal
       lines.push(`${indent}  ${nodeRef}.textContent = ${b.name} ?? '';`);
     } else {
       const expr = transformForExpr(b.name, itemVar, indexVar, propNames, signalNamesSet, computedNamesSet);
-      lines.push(`${indent}  __effect(() => { ${nodeRef}.textContent = ${expr} ?? ''; });`);
+      lines.push(`${indent}  __effect(() => { ${nodeRef}.textContent = ${wrapTernaryExpr(expr)} ?? ''; });`);
     }
   }
 
@@ -650,7 +667,7 @@ function generateItemSetup(lines, forBlock, itemVar, indexVar, propNames, signal
             lines.push(`${indent}      ${nodeRef}.textContent = ${b.name} ?? '';`);
           } else {
             const expr = transformForExpr(b.name, itemVar, indexVar, propNames, signalNamesSet, computedNamesSet);
-            lines.push(`${indent}      __effect(() => { ${nodeRef}.textContent = ${expr} ?? ''; });`);
+            lines.push(`${indent}      __effect(() => { ${nodeRef}.textContent = ${wrapTernaryExpr(expr)} ?? ''; });`);
           }
         }
 
@@ -779,7 +796,7 @@ function generateNestedItemSetup(lines, innerFor, outerItemVar, outerIndexVar, i
       lines.push(`${indent}${nodeRef}.textContent = ${b.name} ?? '';`);
     } else {
       const expr = transformNested(b.name);
-      lines.push(`${indent}__effect(() => { ${nodeRef}.textContent = ${expr} ?? ''; });`);
+      lines.push(`${indent}__effect(() => { ${nodeRef}.textContent = ${wrapTernaryExpr(expr)} ?? ''; });`);
     }
   }
 
@@ -1328,7 +1345,7 @@ export function generateComponent(parseResult, options = {}) {
         ref = transformExpr(exprWithParens, signalNames, computedNames, propsObjectName, propNames, emitsObjectName, constantNames, methodNames, modelVarMap);
       }
       lines.push('    this.__disposers.push(__effect(() => {');
-      lines.push(`      this.${b.varName}.textContent = ${ref} ?? '';`);
+      lines.push(`      this.${b.varName}.textContent = ${wrapTernaryExpr(ref)} ?? '';`);
       lines.push('    }));');
     }
   }
@@ -2016,7 +2033,7 @@ export function generateComponent(parseResult, options = {}) {
           } else {
             ref = transformExpr(b.name, signalNames, computedNames, propsObjectName, propNames, emitsObjectName, constantNames, methodNames, modelVarMap);
           }
-          lines.push(`      __effect(() => { ${b.varName}.textContent = ${ref} ?? ''; });`);
+          lines.push(`      __effect(() => { ${b.varName}.textContent = ${wrapTernaryExpr(ref)} ?? ''; });`);
         }
       }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sprlab/wccompiler",
-  "version": "0.16.2",
+  "version": "0.16.3",
   "description": "Zero-runtime compiler that transforms .wcc single-file components into native web components with signals-based reactivity",
   "type": "module",
   "exports": {