@sprlab/wccompiler 0.5.0 → 0.5.1

package/lib/codegen.js

@@ -530,9 +530,21 @@ export function generateComponent(parseResult, options = {}) {
 
   const lines = [];
 
+  // ── 0. Source comment ──
+  if (options.sourceFile) {
+    lines.push(`// Generated from: ${options.sourceFile} (wcCompiler)`);
+  }
+
   // ── 1. Reactive runtime (shared import or inline) ──
   if (options.runtimeImportPath) {
-    lines.push(`import { __signal, __computed, __effect, __batch } from '${options.runtimeImportPath}';`);
+    // Tree-shake: only import what this component actually uses
+    const usedRuntime = new Set(['__signal']); // always need __signal
+    if (computeds.length > 0) usedRuntime.add('__computed');
+    if (effects.length > 0 || bindings.length > 0 || showBindings.length > 0 || modelBindings.length > 0 || attrBindings.length > 0 || ifBlocks.length > 0 || forBlocks.length > 0 || watchers.length > 0 || childComponents.length > 0 || slots.some(s => s.slotProps.length > 0)) usedRuntime.add('__effect');
+    // __batch is available but only needed if user code calls it explicitly — always include for safety
+    usedRuntime.add('__batch');
+    const imports = [...usedRuntime].join(', ');
+    lines.push(`import { ${imports} } from '${options.runtimeImportPath}';`);
   } else {
     lines.push(reactiveRuntime.trim());
   }
@@ -546,12 +558,16 @@ export function generateComponent(parseResult, options = {}) {
     lines.push('');
   }
 
-  // ── 2. CSS injection (scoped CSS into document.head, always) ──
+  // ── 2. CSS injection (scoped, deduplicated via id guard) ──
   if (style) {
     const scoped = scopeCSS(style, tagName);
-    lines.push(`const __css_${className} = document.createElement('style');`);
-    lines.push(`__css_${className}.textContent = \`${scoped}\`;`);
-    lines.push(`document.head.appendChild(__css_${className});`);
+    const cssId = `__css_${className}`;
+    lines.push(`if (!document.getElementById('${cssId}')) {`);
+    lines.push(`  const ${cssId} = document.createElement('style');`);
+    lines.push(`  ${cssId}.id = '${cssId}';`);
+    lines.push(`  ${cssId}.textContent = \`${scoped}\`;`);
+    lines.push(`  document.head.appendChild(${cssId});`);
+    lines.push('}');
     lines.push('');
   }
 
@@ -724,8 +740,12 @@ export function generateComponent(parseResult, options = {}) {
   lines.push('  }');
   lines.push('');
 
-  // connectedCallback
+  // connectedCallback (idempotent — safe for re-mount)
   lines.push('  connectedCallback() {');
+  lines.push('    if (this.__connected) return;');
+  lines.push('    this.__connected = true;');
+  lines.push('    this.__ac = new AbortController();');
+  lines.push('');
 
   // Binding effects — one __effect per binding
   for (const b of bindings) {
@@ -846,10 +866,10 @@ export function generateComponent(parseResult, options = {}) {
     }
   }
 
-  // Event listeners
+  // Event listeners (with AbortController signal for cleanup)
   for (const e of events) {
     const handlerExpr = generateEventHandler(e.handler, signalNames, computedNames, propsObjectName, propNames, emitsObjectName, constantNames);
-    lines.push(`    this.${e.varName}.addEventListener('${e.event}', ${handlerExpr});`);
+    lines.push(`    this.${e.varName}.addEventListener('${e.event}', ${handlerExpr}, { signal: this.__ac.signal });`);
   }
 
   // Show effects — one __effect per ShowBinding
@@ -880,17 +900,17 @@ export function generateComponent(parseResult, options = {}) {
     }
   }
 
-  // Model event listeners — DOM → signal (one addEventListener per ModelBinding)
+  // Model event listeners — DOM → signal (with AbortController signal)
   for (const mb of modelBindings) {
     if (mb.prop === 'checked' && mb.radioValue === null) {
       // Checkbox: read e.target.checked
-      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(e.target.checked); });`);
+      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(e.target.checked); }, { signal: this.__ac.signal });`);
     } else if (mb.coerce) {
       // Number input: wrap in Number()
-      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(Number(e.target.value)); });`);
+      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(Number(e.target.value)); }, { signal: this.__ac.signal });`);
     } else {
       // All others: read e.target.value
-      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(e.target.value); });`);
+      lines.push(`    this.${mb.varName}.addEventListener('${mb.event}', (e) => { this._${mb.signal}(e.target.value); }, { signal: this.__ac.signal });`);
     }
   }
 
@@ -1078,9 +1098,11 @@ export function generateComponent(parseResult, options = {}) {
   lines.push('  }');
   lines.push('');
 
-  // disconnectedCallback (only when destroy hooks exist)
+  // disconnectedCallback (cleanup: abort listeners + user hooks)
+  lines.push('  disconnectedCallback() {');
+  lines.push('    this.__connected = false;');
+  lines.push('    this.__ac.abort();');
   if (onDestroyHooks.length > 0) {
-    lines.push('  disconnectedCallback() {');
     for (const hook of onDestroyHooks) {
       const body = transformMethodBody(hook.body, signalNames, computedNames, propsObjectName, propNames, emitsObjectName, refVarNames, constantNames);
       if (hook.async) {
@@ -1097,9 +1119,9 @@ export function generateComponent(parseResult, options = {}) {
         }
       }
     }
-    lines.push('  }');
-    lines.push('');
   }
+  lines.push('  }');
+  lines.push('');
 
   // attributeChangedCallback (if props exist)
   if (propDefs.length > 0) {

package/lib/compiler.js

@@ -323,7 +323,8 @@ async function compileSFC(filePath, config) {
   parseResult.processedTemplate = rootEl.innerHTML;
 
   // 20. Generate component
-  return generateComponent(parseResult, config);
+  const genOptions = { ...config, sourceFile: fileName };
+  return generateComponent(parseResult, genOptions);
 }
 
 /**

package/package.json

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