@reckona/mreact-compiler 0.0.131 → 0.0.134

package/dist/emit-server.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"emit-server.d.ts","sourceRoot":"","sources":["../src/emit-server.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAwD,QAAQ,EAAE,MAAM,SAAS,CAAC;AAC9F,OAAO,KAAK,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAmBrE,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,aAAa,EAAE,CAAC;CAC1B;AAED,MAAM,WAAW,iBAAiB;IAChC,iBAAiB,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACpC,MAAM,CAAC,EAAE,mBAAmB,GAAG,SAAS,CAAC;IACzC,eAAe,CAAC,EAAE,OAAO,CAAC;CAC3B;AASD,wBAAgB,UAAU,CAAC,EAAE,EAAE,QAAQ,EAAE,OAAO,GAAE,iBAAsB,GAAG,UAAU,CA6GpF"}
+{"version":3,"file":"emit-server.d.ts","sourceRoot":"","sources":["../src/emit-server.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAwD,QAAQ,EAAE,MAAM,SAAS,CAAC;AAC9F,OAAO,KAAK,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,YAAY,CAAC;AAoBrE,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,aAAa,EAAE,CAAC;CAC1B;AAED,MAAM,WAAW,iBAAiB;IAChC,iBAAiB,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACpC,MAAM,CAAC,EAAE,mBAAmB,GAAG,SAAS,CAAC;IACzC,eAAe,CAAC,EAAE,OAAO,CAAC;CAC3B;AASD,wBAAgB,UAAU,CAAC,EAAE,EAAE,QAAQ,EAAE,OAAO,GAAE,iBAAsB,GAAG,UAAU,CA6GpF"}

package/dist/emit-server.js

@@ -1,7 +1,7 @@
 import { emitEscapeHtmlHelper } from "./emit-escape-helper.js";
 import { createCodeBuilder } from "./emit-code-builder.js";
 import { escapeHtmlAttribute as escapeHtml } from "@reckona/mreact-shared/html-escape";
-import { htmlAttributeName, isDangerousHtmlAttribute, isStaticUrlValueUnsafe, isUrlAttribute, isVoidHtmlElement, parseStaticStyleObjectLiteral, parseStyleLiteralValue, simpleSideEffectFreeExpression, } from "./emit-server-shared.js";
+import { htmlAttributeName, isBooleanishStringAttribute, isDangerousHtmlAttribute, isStaticUrlValueUnsafe, isUrlAttribute, isVoidHtmlElement, parseStaticStyleObjectLiteral, parseStyleLiteralValue, simpleSideEffectFreeExpression, } from "./emit-server-shared.js";
 import { emitOxcCompatObjectChildren, oxcServerStringReactNodeRenderHelperPlaceholder, } from "./oxc-runtime-emit.js";
 // Module-local handle to the URL-safety helper name for the current emit
 // call. Used by deeply-nested attribute emitters to avoid threading the
@@ -597,6 +597,7 @@ function readFormValueAttributeCode(attr) {
     return attr.kind === "static-attr" ? stringLiteral(attr.value) : `(${attr.code})`;
 }
 function emitDynamicAttributeExpression(name, code, escapeHelperName) {
+    const booleanishString = isBooleanishStringAttribute(name);
     if (isUrlAttribute(name)) {
         // Run the value through the inline URL safety helper. The helper
         // returns the value when safe and `undefined` when the attribute
@@ -610,9 +611,13 @@ function emitDynamicAttributeExpression(name, code, escapeHelperName) {
         // Safe because `simpleSideEffectFreeExpression` only matches expressions
         // whose evaluation has no observable side effects (identifier read,
         // member chain, literal, this).
-        return `(${inlineExpr} == null || ${inlineExpr} === false ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(${inlineExpr} === true ? "" : ${inlineExpr}) + ${stringLiteral('"')})`;
+        return booleanishString
+            ? `(${inlineExpr} == null ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(${inlineExpr}) + ${stringLiteral('"')})`
+            : `(${inlineExpr} == null || ${inlineExpr} === false ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(${inlineExpr} === true ? "" : ${inlineExpr}) + ${stringLiteral('"')})`;
     }
-    return `(() => { const _value = (${code}); return _value == null || _value === false ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(_value === true ? "" : _value) + ${stringLiteral('"')}; })()`;
+    return booleanishString
+        ? `(() => { const _value = (${code}); return _value == null ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(_value) + ${stringLiteral('"')}; })()`
+        : `(() => { const _value = (${code}); return _value == null || _value === false ? "" : ${stringLiteral(` ${name}="`)} + ${escapeHelperName}(_value === true ? "" : _value) + ${stringLiteral('"')}; })()`;
 }
 function emitDynamicStyleAttributeExpression(code, escapeHelperName, escapeBatchHelperName) {
     const staticStyleExpression = emitStaticStyleObjectAttributeExpression(code, escapeHelperName);
@@ -893,7 +898,12 @@ function emitClientBoundaryHelper(name) {
         `  const _props = props ?? {};`,
         `  const _nonSerializable = ${propsHelperName}(_props);`,
         `  const _nonSerializableAttr = _nonSerializable ? ' data-mreact-client-boundary-nonserializable="true"' : "";`,
-        `  const _json = (JSON.stringify(_props) ?? "{}").replaceAll("<", "\\\\u003c");`,
+        `  const _json = (JSON.stringify(_props) ?? "{}")`,
+        `    .replaceAll("&", "\\\\u0026")`,
+        `    .replaceAll("<", "\\\\u003c")`,
+        `    .replaceAll(">", "\\\\u003e")`,
+        `    .replaceAll("\\u2028", "\\\\u2028")`,
+        `    .replaceAll("\\u2029", "\\\\u2029");`,
         `  return \`<template data-mreact-client-boundary="\${_escapedName}"\${_nonSerializableAttr}></template>\${childrenHtml}<script type="application/json" data-mreact-client-boundary-props="\${_escapedName}">\${_json}</script>\`;`,
         `}`,
     ].join("\n");
@@ -958,11 +968,13 @@ function emitSpreadAttributesHelper(name, escapeHelperName, urlSafeHelperName) {
         `  let _out = "";`,
         `  for (const _rawName of Object.keys(props)) {`,
         `    let _value = props[_rawName];`,
-        `    if (_value == null || _value === false) continue;`,
+        `    if (_value == null) continue;`,
         `    if (_rawName === "key" || _rawName === "ref" || _rawName === "children") continue;`,
         `    if (/^on[A-Za-z]/.test(_rawName)) continue;`,
         `    let _name = tagName === "input" && _rawName === "defaultValue" ? "value" : tagName === "input" && _rawName === "defaultChecked" ? "checked" : (${name}$aliases[_rawName] ?? _rawName);`,
         `    if (!/^[A-Za-z_:][A-Za-z0-9:_.-]*$/.test(_name)) continue;`,
+        `    const _booleanish = _name.startsWith("aria-") || _name.startsWith("data-") || _name === "contenteditable" || _name === "draggable" || _name === "spellcheck";`,
+        `    if (_value === false && !_booleanish) continue;`,
         `    if (_name === "style") {`,
         `      const _style = ${name}$style(_value);`,
         `      if (_style !== "") _out += " style=\\"" + ${escapeHelperName}(_style) + "\\"";`,
@@ -978,7 +990,7 @@ function emitSpreadAttributesHelper(name, escapeHelperName, urlSafeHelperName) {
         `      _value = ${urlSafeHelperName}(_name, _value === true ? "" : _value);`,
         `      if (_value === undefined) continue;`,
         `    }`,
-        `    _out += " " + _name + "=\\"" + ${escapeHelperName}(_value === true ? "" : _value) + "\\"";`,
+        `    _out += " " + _name + "=\\"" + ${escapeHelperName}(_value === true && !_booleanish ? "" : _value) + "\\"";`,
         `  }`,
         `  return _out;`,
         `}`,