sinwan-compiler 0.2.5 → 0.2.6

package/README.md

@@ -19,6 +19,8 @@ bun add -d sinwan-compiler
 
 App templates already depend on the Bun or Vite plugin. You only need this package if you call the compiler yourself.
 
+The plugins declare `sinwan-compiler` as `>=0.2.5 <1.0.0`. After you publish a new 0.x compiler, do **not** bump the plugins. Apps pick it up with a fresh install or `bun update sinwan-compiler`. Republish a plugin only if its own API changed.
+
 ## Production analysis
 
 ```bash

package/dist/index.js

@@ -92,6 +92,63 @@ function cleanJSXText(value) {
   }
   return str;
 }
+function htmlAttributeName(attrName) {
+  if (attrName === "defaultValue")
+    return "value";
+  if (attrName === "defaultChecked")
+    return "checked";
+  return attrName;
+}
+function jsxAttrName(attr) {
+  if (attr.type !== "JSXAttribute" || attr.name.type !== "JSXIdentifier") {
+    return;
+  }
+  return attr.name.name;
+}
+function hasAttr(attributes, name) {
+  for (const attr of attributes) {
+    if (jsxAttrName(attr) === name)
+      return true;
+  }
+  return false;
+}
+function attrIsExpression(attributes, name) {
+  for (const attr of attributes) {
+    if (attr.type !== "JSXAttribute" || jsxAttrName(attr) !== name)
+      continue;
+    return attr.value?.type === "JSXExpressionContainer";
+  }
+  return false;
+}
+function elementNeedsJsxEnhancer(node) {
+  const tagName = jsxNameToString(node.openingElement.name);
+  const attrs = node.openingElement.attributes;
+  switch (tagName) {
+    case "select":
+      return hasAttr(attrs, "defaultValue") || hasAttr(attrs, "value");
+    case "textarea":
+      return hasAttr(attrs, "value");
+    case "option":
+      return hasAttr(attrs, "selected");
+    case "form":
+      return attrIsExpression(attrs, "action");
+    case "input":
+    case "button":
+      return attrIsExpression(attrs, "formAction");
+    case "progress":
+      return attrIsExpression(attrs, "value");
+    case "style":
+      return hasAttr(attrs, "href") && hasAttr(attrs, "precedence");
+    case "script":
+      return hasAttr(attrs, "src") && hasAttr(attrs, "async");
+    case "link":
+    case "meta":
+    case "title":
+      return !hasAttr(attrs, "itemProp");
+    default:
+      return false;
+  }
+}
 function extractTemplate(node, filename) {
   const counter = { value: 0 };
   const slots = [];
@@ -175,6 +232,7 @@ function elementToHtml(node, slots, path, filename, counter) {
   }
   const isVoid = VOID_ELEMENTS.has(tagName);
   let html = `<${tagName}`;
+  let textareaDefault = "";
   for (const attr of node.openingElement.attributes) {
     if (attr.type === "JSXSpreadAttribute") {
       throw new Error("Cannot hoist element with spread attributes");
@@ -196,13 +254,14 @@ function elementToHtml(node, slots, path, filename, counter) {
       }
       continue;
     }
+    const htmlName = htmlAttributeName(attrName);
     if (attr.value?.type === "JSXExpressionContainer") {
       const expr = attr.value.expression;
       if (expr.type === "JSXEmptyExpression")
         continue;
       if (attrName === "style" && t.isExpression(expr) && isStaticStyleValue(expr)) {
         const serialized = serializeStyleValue(expr);
-        html += ` ${attrName}="${escapeHtml(serialized)}"`;
+        html += ` ${htmlName}="${escapeHtml(serialized)}"`;
         continue;
       }
       slots.push({
@@ -211,7 +270,7 @@ function elementToHtml(node, slots, path, filename, counter) {
         name: attrName,
         expr
       });
-      html += ` ${attrName}=""`;
+      html += ` ${htmlName}=""`;
       continue;
     }
     if (attr.value?.type === "JSXText" || attr.value?.type === "StringLiteral") {
@@ -219,9 +278,13 @@ function elementToHtml(node, slots, path, filename, counter) {
       if (attrName === "style") {
         warnSuspectedStyleInterpolation(rawValue, filename, attr.value.loc?.start?.line);
       }
-      html += ` ${attrName}="${escapeHtml(rawValue)}"`;
+      if (tagName === "textarea" && attrName === "defaultValue") {
+        textareaDefault = escapeHtml(rawValue);
+        continue;
+      }
+      html += ` ${htmlName}="${escapeHtml(rawValue)}"`;
     } else if (!attr.value) {
-      html += ` ${attrName}`;
+      html += ` ${htmlName}`;
     }
   }
   html += isVoid ? " />" : ">";
@@ -229,6 +292,7 @@ function elementToHtml(node, slots, path, filename, counter) {
     return html;
   const childResult = childrenToHtml(node.children, slots, path, filename, 0, counter);
   html += childResult.html;
+  html += textareaDefault;
   html += `</${tagName}>`;
   return html;
 }
@@ -257,7 +321,7 @@ function childrenToHtml(children, slots, path, filename, startIndex, counter) {
       const childName = child.openingElement.name;
       const childTag = childName.type === "JSXIdentifier" ? childName.name : "";
       const isFirstCharUpper = childTag && childTag.charAt(0) === childTag.charAt(0).toUpperCase();
-      if (isFirstCharUpper) {
+      if (isFirstCharUpper || elementNeedsJsxEnhancer(child)) {
         const slot = nextSlotId(counter);
         slots.push({
           path: [...path, childIndex],
@@ -316,6 +380,8 @@ function transformJSX(code, filename, options = {}) {
       const name = tagName.name;
       if (!name || name[0] !== name[0].toLowerCase())
         return;
+      if (elementNeedsJsxEnhancer(path.node))
+        return;
       try {
         const extracted = extractTemplate(path.node, filename);
         const tmplId = `_$tmpl_${templateCounter++}`;

package/docs/plugins.md

@@ -1,6 +1,8 @@
 # Using the compiler from Bun and Vite
 
-App projects should depend on **`bun-plugin-sinwan`** or **`vite-plugin-sinwan`**. Those packages import `sinwan-compiler` and pass the right options. You do not add `sinwan-compiler` unless you customize the pipeline.
+App projects should depend on **`bun-plugin-sinwan`** or **`vite-plugin-sinwan`**. Those packages import `sinwan-compiler` at runtime (`packages: "external"` in their builds) and pass the right options. You do not add `sinwan-compiler` unless you customize the pipeline.
+
+The plugins accept any compiler `>=0.2.5 <1.0.0`. A new compiler release does not need a matching plugin publish. Existing apps update with `bun update sinwan-compiler`.
 
 ## Bun
 

package/docs/transform.md

@@ -113,7 +113,7 @@ Slots:
 
 Static `style` objects and strings are written into the HTML (`backgroundColor` → `background-color`). Quoted style strings that contain `${…}` log a warning in the compiler: use a template literal, `style={\`…${x}\`}`.
 
-Hoisting is skipped (the JSX is left as JSX) when the root is a component, the tag is a member expression (`Icons.Star` as the root), or a spread is on a native element (`<div {...props}>`). In `dev: true`, skipped hoists log a warning.
+Hoisting is skipped (the JSX is left as JSX) when the root is a component, the tag is a member expression (`Icons.Star` as the root), a spread is on a native element (`<div {...props}>`), or the element needs a JSX enhancer (`<select defaultValue>`, function `action`/`formAction`, head tags without `itemProp`, and the other cases in `enhanced-elements.ts`). Nested enhanced tags become child slots so the native shell can still hoist. In `dev: true`, skipped hoists from extract errors log a warning.
 
 Boolean attributes without a value are emitted as HTML flags: `<button disabled>`.
 

package/package.json

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