@sprlab/wccompiler 0.16.2 → 0.16.4

package/lib/codegen.js

@@ -57,6 +57,47 @@ function slotPropRef(source, signalNames, computedNames, propNames) {
   return `'${source}'`;
 }
 
+/**
+ * Wrap an expression in parentheses if it contains operators that could have
+ * precedence issues when combined with ?? (nullish coalescing).
+ *
+ * This prevents bugs like: this._count() || 'No items' ?? ''
+ * which JavaScript interprets as: this._count() || ('No items' ?? '')
+ *
+ * Operators that need wrapping:
+ * - Ternary: ? :
+ * - Logical OR: ||
+ * - Logical AND: &&
+ * - Nullish coalescing: ?? (nested)
+ *
+ * @param {string} expr - Expression to potentially wrap
+ * @returns {string} - Wrapped expression if it contains risky operators, otherwise unchanged
+ */
+function wrapTernaryExpr(expr) {
+  const trimmed = expr.trim();
+  
+  // Check for operators that have lower precedence than ?? or can cause ambiguity
+  // Ternary operator (? :)
+  const hasTernary = trimmed.includes('?') && trimmed.includes(':');
+  
+  // Logical OR (||)
+  const hasLogicalOr = trimmed.includes('||');
+  
+  // Logical AND (&&)
+  const hasLogicalAnd = trimmed.includes('&&');
+  
+  // Nested nullish coalescing (??)
+  // If expression contains ??, wrap it to avoid conflict with the trailing ?? ''
+  const hasNullish = trimmed.includes('??');
+  
+  // Wrap if any risky operator is found
+  if (hasTernary || hasLogicalOr || hasLogicalAnd || hasNullish) {
+    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 +485,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 +691,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 +820,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 +1369,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 +2057,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.4",
   "description": "Zero-runtime compiler that transforms .wcc single-file components into native web components with signals-based reactivity",
   "type": "module",
   "exports": {