@tsrx/solid 0.0.17 → 0.0.18

package/package.json

@@ -3,7 +3,7 @@
   "description": "Solid compiler built on @tsrx/core",
   "license": "MIT",
   "author": "Dominic Gannaway",
-  "version": "0.0.17",
+  "version": "0.0.18",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -22,7 +22,7 @@
   "dependencies": {
     "esrap": "^2.1.0",
     "zimmerframe": "^1.1.2",
-    "@tsrx/core": "0.0.17"
+    "@tsrx/core": "0.0.18"
   },
   "peerDependencies": {
     "solid-js": ">=1.8 || >=2.0.0-beta"

package/src/transform.js

@@ -3,6 +3,9 @@
 
 import {
 	createJsxTransform,
+	mergeDuplicateRefs,
+	toJsxAttribute,
+	validateAtMostOneRefAttribute,
 	setLocation,
 	applyLazyTransforms as apply_lazy_transforms,
 	collectLazyBindingsFromComponent as collect_lazy_bindings_from_component,
@@ -71,6 +74,10 @@ const solid_platform = {
 	jsx: {
 		rewriteClassAttr: false,
 		acceptedTsxKinds: ['solid'],
+		// Solid's runtime accepts an array of refs natively, so multiple
+		// `ref` attributes collapse to `ref={[a, b, ...]}` rather than
+		// going through a `mergeRefs` helper.
+		multiRefStrategy: 'array',
 	},
 	validation: {
 		requireUseServerForAwait: true,
@@ -106,8 +113,12 @@ const solid_platform = {
 		componentToFunction: (component, ctx) =>
 			component_to_function_declaration(component, /** @type {any} */ (ctx)),
 		injectImports: (program, ctx) => inject_solid_imports(program, /** @type {any} */ (ctx)),
-		transformElementAttributes: (attrs, ctx, element) =>
-			transform_element_attributes(attrs, is_composite_element(element), /** @type {any} */ (ctx)),
+		// `transformElementAttributes` is intentionally omitted: the
+		// `transformElement` hook below short-circuits core's element walker
+		// before `to_jsx_element` runs, so the dispatch path that would call
+		// `transformElementAttributes` is never reached for Solid. Attribute
+		// lowering happens in Solid's local `transform_element_attributes`,
+		// which `to_jsx_element` and `create_dynamic_jsx_element` call directly.
 		transformElement: (inner, ctx, raw_children) =>
 			to_jsx_element(/** @type {any} */ (inner), /** @type {any} */ (ctx), raw_children),
 	},
@@ -1175,44 +1186,6 @@ function create_element_children(children, transform_context) {
 	return children.map((/** @type {any} */ child) => to_jsx_child(child, transform_context));
 }
 
-/**
- * Attribute transform. Unlike React, Solid uses the native `class` attribute
- * (not `className`). `RefAttribute` and `SpreadAttribute` nodes are handled
- * at the element level by {@link transform_element_attributes} so this
- * function only sees plain attributes.
- *
- * @param {any} attr
- * @returns {any}
- */
-function to_jsx_attribute(attr) {
-	if (!attr) return attr;
-	if (attr.type === 'JSXAttribute' || attr.type === 'JSXSpreadAttribute') return attr;
-
-	const attr_name = attr.name;
-	const name =
-		attr_name && attr_name.type === 'Identifier' ? identifier_to_jsx_name(attr_name) : attr_name;
-
-	let value = attr.value;
-	if (value) {
-		if (value.type === 'Literal' && typeof value.value === 'string') {
-			// Keep string literal as attribute string.
-		} else if (value.type !== 'JSXExpressionContainer') {
-			value = to_jsx_expression_container(value);
-		}
-	}
-
-	return set_loc(
-		/** @type {any} */ ({
-			type: 'JSXAttribute',
-			name,
-			value: value || null,
-			shorthand: false,
-			metadata: { path: [] },
-		}),
-		attr,
-	);
-}
-
 /**
  * Detect whether an `Element` node represents a composite component (tag
  * name starts with an uppercase letter, or is a member expression like
@@ -1254,26 +1227,17 @@ function has_text_content_attribute(attributes) {
 }
 
 /**
- * Transform a list of raw attributes into JSX attributes, lifting
- * `{ref expr}` handling to the element level.
+ * Transform a list of raw attributes into JSX attributes.
  *
- * `{ref expr}` compiles to `ref={expr}` on both DOM elements and composite
- * components. On DOM elements, Solid's JSX transform takes over: if `expr`
- * is a mutable `let`-declared identifier it assigns the element to the
- * variable; if `expr` is a function (or other callable) it invokes it
- * with the element. On composite components, `ref` is passed through as a
- * regular prop; the receiving child can consume it explicitly as
- * `props.ref` or spread `{...props}` onto a DOM element, where Solid's
- * spread runtime automatically applies the `ref` entry. Solid's merge
- * proxies drop Symbol keys, so the Symbol-based forwarding used by
- * Ripple doesn't port; the Solid target relies on its native `ref` prop
- * support instead.
- *
- * Multiple `{ref ...}` attributes on the same element are collected into
- * a single `ref={[a, b, ...]}` array so every callback fires. Solid's
- * ref/spread runtime (`applyRef`) already iterates array refs, so this
- * works on both DOM elements and composite components (when the child
- * spreads `props` or forwards `props.ref`).
+ * Per-attribute conversion (RefAttribute → `ref={expr}`, SpreadAttribute →
+ * `{...expr}`, plain Attribute → JSXAttribute, JSXAttribute pass-through)
+ * is delegated to `@tsrx/core`'s shared {@link toJsxAttribute}. The list
+ * is then run through {@link mergeDuplicateRefs} so multiple ref attributes
+ * on the same element — whether from `{ref expr}` or TSX-style `ref={expr}` —
+ * collapse to a single `ref={[a, b, ...]}` array (the strategy chosen by
+ * Solid's `multiRefStrategy: 'array'`). Solid's runtime iterates array refs
+ * natively, so this works on both DOM elements and composite components
+ * (when the child spreads `props` or forwards `props.ref`).
  *
  * @param {any[]} raw_attrs
  * @param {boolean} is_composite
@@ -1282,66 +1246,15 @@ function has_text_content_attribute(attributes) {
  */
 function transform_element_attributes(raw_attrs, is_composite, transform_context) {
 	void is_composite;
-	void transform_context;
+	validateAtMostOneRefAttribute(raw_attrs);
 	/** @type {any[]} */
 	const result = [];
-	/** @type {any[]} */
-	const ref_attrs = [];
 
 	for (const attr of raw_attrs) {
 		if (!attr) continue;
-		if (attr.type === 'RefAttribute') {
-			ref_attrs.push(attr);
-			continue;
-		}
-		if (attr.type === 'SpreadAttribute') {
-			result.push(
-				set_loc(
-					/** @type {any} */ ({
-						type: 'JSXSpreadAttribute',
-						argument: attr.argument,
-					}),
-					attr,
-				),
-			);
-			continue;
-		}
-		result.push(to_jsx_attribute(attr));
-	}
-
-	if (ref_attrs.length === 1) {
-		result.push(build_ref_attribute(ref_attrs[0].argument, ref_attrs[0]));
-	} else if (ref_attrs.length > 1) {
-		const array_expr = /** @type {any} */ ({
-			type: 'ArrayExpression',
-			elements: ref_attrs.map((attr) => attr.argument),
-			metadata: { path: [] },
-		});
-		result.push(build_ref_attribute(array_expr, ref_attrs[0]));
+		result.push(toJsxAttribute(attr, /** @type {any} */ (transform_context)));
 	}
-
-	return result;
-}
-
-/**
- * Build a `ref={expr}` JSX attribute, passing the expression through
- * unchanged so Solid's JSX transform can apply its normal ref semantics.
- *
- * @param {any} argument
- * @param {any} source_node
- * @returns {any}
- */
-function build_ref_attribute(argument, source_node) {
-	return set_loc(
-		/** @type {any} */ ({
-			type: 'JSXAttribute',
-			name: { type: 'JSXIdentifier', name: 'ref', metadata: { path: [] } },
-			value: to_jsx_expression_container(argument),
-			shorthand: false,
-			metadata: { path: [] },
-		}),
-		source_node,
-	);
+	return mergeDuplicateRefs(result, /** @type {any} */ (transform_context));
 }
 
 /**