@tsrx/core 0.0.7 → 0.0.8

package/package.json

@@ -3,7 +3,7 @@
   "description": "Core compiler infrastructure for TSRX syntax",
   "license": "MIT",
   "author": "Dominic Gannaway",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "type": "module",
   "repository": {
     "type": "git",

package/src/plugin.js

@@ -1431,7 +1431,8 @@ export function TSRXPlugin(config) {
 			/**
 			 * Override jsx_parseElement to intercept expression-level JSX.
 			 * This is called by acorn-jsx's parseExprAtom when it encounters <
-			 * in expression position. Only <tsx> and <tsx:*> are allowed.
+			 * in expression position. Bare fragments are treated as shorthand
+			 * for <tsx>...</tsx>; other tags must still use <tsx> or <tsx:*>.
 			 * @type {Parse.Parser['jsx_parseElement']}
 			 */
 			jsx_parseElement() {
@@ -1444,6 +1445,7 @@ export function TSRXPlugin(config) {
 				// Check if the element being parsed IS a <tsx> or <tsx:*> tag
 				// Current token is jsxTagStart, this.end is position after '<'
 				const tag_name_start = this.end;
+				const is_fragment_tag = this.input.charCodeAt(tag_name_start) === 62;
 				const char_after_tsx = this.input.charCodeAt(tag_name_start + 3);
 				const is_tsx_tag =
 					this.input.startsWith('tsx', tag_name_start) &&
@@ -1456,8 +1458,9 @@ export function TSRXPlugin(config) {
 						char_after_tsx === 13 || // carriage return
 						char_after_tsx === 58); // : (tsx:react)
 
-				if (is_tsx_tag) {
-					// Use Ripple's parseElement to create a Tsx/TsxCompat node
+				if (is_fragment_tag || is_tsx_tag) {
+					// Use Ripple's parseElement to create a Tsx/TsxCompat node.
+					// Bare fragments (<></>) are shorthand for <tsx>...</tsx>.
 					this.next();
 					return /** @type {import('estree-jsx').JSXElement} */ (
 						/** @type {unknown} */ (this.parseElement())
@@ -1524,6 +1527,8 @@ export function TSRXPlugin(config) {
 							`TSX elements cannot be self-closing. '<tsx />' must have a closing tag '</tsx>'.`,
 						);
 					}
+				} else if (is_fragment) {
+					/** @type {AST.Tsx} */ (element).type = 'Tsx';
 				} else {
 					element.type = 'Element';
 				}
@@ -1575,6 +1580,26 @@ export function TSRXPlugin(config) {
 					this.enterScope(0);
 					this.parseTemplateBody(/** @type {AST.Element} */ (element).children);
 					this.exitScope();
+
+					if (element.type === 'Tsx') {
+						this.#path.pop();
+
+						if (!element.unclosed) {
+							const raise_error = () => {
+								this.raise(this.start, `Expected closing tag '</>'`);
+							};
+
+							this.next();
+							if (this.value !== '/') {
+								raise_error();
+							}
+							this.next();
+							if (this.type !== tstt.jsxTagEnd) {
+								raise_error();
+							}
+							this.next();
+						}
+					}
 				} else {
 					if (/** @type {ESTreeJSX.JSXIdentifier} */ (open.name).name === 'script') {
 						let content = '';
@@ -1875,7 +1900,11 @@ export function TSRXPlugin(config) {
 							return;
 						}
 
-						if (this.input.slice(this.pos, this.pos + 4) === '/tsx') {
+						if (!inside_tsx.openingElement.name) {
+							if (this.input.slice(this.pos, this.pos + 2) === '/>') {
+								return;
+							}
+						} else if (this.input.slice(this.pos, this.pos + 4) === '/tsx') {
 							const after = this.input.charCodeAt(this.pos + 4);
 							// Make sure it's </tsx> and not </tsx:...>
 							if (after === 62 /* > */) {
@@ -2046,7 +2075,7 @@ export function TSRXPlugin(config) {
 									? closingElement.name.namespace.name + ':' + closingElement.name.name.name
 									: this.getElementName(closingElement.name);
 						} else if (currentElement.type === 'Tsx') {
-							openingTagName = 'tsx';
+							openingTagName = currentElement.openingElement.name ? 'tsx' : null;
 							closingTagName =
 								closingElement.name?.type === 'JSXNamespacedName'
 									? closingElement.name.namespace.name + ':' + closingElement.name.name.name
@@ -2081,7 +2110,9 @@ export function TSRXPlugin(config) {
 										elem.type === 'TsxCompat'
 											? 'tsx:' + elem.kind
 											: elem.type === 'Tsx'
-												? 'tsx'
+												? elem.openingElement.name
+													? 'tsx'
+													: null
 												: elem.id
 													? this.getElementName(elem.id)
 													: null;

package/src/transform/segments.js

@@ -1618,9 +1618,18 @@ export function convert_source_map_to_mappings(
 				node.type === 'TSTypeParameterDeclaration'
 			) {
 				if (node.loc) {
-					mappings.push(
-						get_mapping_from_node(node, src_to_gen_map, gen_line_offsets, mapping_data_verify_only),
+					// Generic spans can be emitted by downstream transforms with sparse source-map
+					// coverage around the angle-bracket delimiters. Skip missing whole-node mappings
+					// instead of crashing Volar, and rely on child type-node mappings instead.
+					const mapping = maybe_get_mapping_from_node(
+						node,
+						src_to_gen_map,
+						gen_line_offsets,
+						mapping_data_verify_only,
 					);
+					if (!(mapping instanceof Error)) {
+						mappings.push(mapping);
+					}
 				}
 				// Generic type parameters - visit to collect type variable names
 				if (node.params) {

package/types/parse.d.ts

@@ -1652,7 +1652,6 @@ export namespace Parse {
 			startLoc?: AST.Position,
 		): ESTreeJSX.JSXOpeningElement;
 		// it could also be ESTreeJSX.JSXOpeningFragment
-		// but not in our case since we don't use fragments
 
 		/**
 		 * Parse JSX closing element at position