@saykit/transform-jsx 0.9.0 → 0.9.1

package/dist/index.mjs

@@ -5,7 +5,7 @@ import * as parser from "@babel/parser";
 import traverse_ from "@babel/traverse";
 import { assignSequenceIdentifiers } from "@saykit/config/features/messages";
 import { generateSayCallExpression } from "@saykit/transform-js/generator";
-import { isEquivalentPlaceholder, parseExpression } from "@saykit/transform-js/parser";
+import { collectLeadingComments, isEquivalentPlaceholder, parseExpression } from "@saykit/transform-js/parser";
 //#region src/index.ts
 const traverse = traverse_.default || traverse_;
 function createProgram(code, id) {
@@ -37,7 +37,7 @@ function createJsxTransformer() {
 			const messages = [];
 			traverse(program, {
 				Expression(path) {
-					path.node.leadingComments = path.node.leadingComments ?? [];
+					path.node.leadingComments = collectLeadingComments(path);
 					const message = parseExpression(path.node);
 					if (message) {
 						assignSequenceIdentifiers(message, { current: 0 }, isEquivalentPlaceholder);
@@ -46,7 +46,7 @@ function createJsxTransformer() {
 					}
 				},
 				JSXElement(path) {
-					path.node.leadingComments = path.node.leadingComments ?? [];
+					path.node.leadingComments = collectLeadingComments(path);
 					const message = parseJSXElement(path.node);
 					if (message) {
 						assignSequenceIdentifiers(message, { current: 0 }, isEquivalentPlaceholder);
@@ -68,7 +68,6 @@ function createJsxTransformer() {
 			const program = createProgram(code, id);
 			traverse(program, {
 				Expression(path) {
-					path.node.leadingComments = path.node.leadingComments ?? [];
 					const message = parseExpression(path.node);
 					if (message) {
 						assignSequenceIdentifiers(message, { current: 0 }, isEquivalentPlaceholder);
@@ -78,7 +77,6 @@ function createJsxTransformer() {
 					}
 				},
 				JSXElement(path) {
-					path.node.leadingComments = path.node.leadingComments ?? [];
 					const message = parseJSXElement(path.node);
 					if (message) {
 						assignSequenceIdentifiers(message, { current: 0 }, isEquivalentPlaceholder);

package/dist/parser.d.mts

@@ -2,7 +2,12 @@ import { CompositeMessage, ElementMessage } from "@saykit/config/features/messag
 import * as t from "@babel/types";
 //#region src/parser.d.ts
 declare function parseJSXContainerElement(element: t.JSXElement): CompositeMessage | null;
-declare function parseJSXOpeningElement(element: t.JSXOpeningElement): CompositeMessage | null;
+/**
+ * A self-closing `Say` element. Comments are passed in rather than read off the
+ * node, since they were written in front of the element as a whole and Babel
+ * attaches them there.
+ */
+declare function parseJSXOpeningElement(element: t.JSXOpeningElement, comments?: readonly t.Comment[] | null): CompositeMessage | null;
 declare function parseJSXElement(element: t.JSXElement, fallback?: false): CompositeMessage | null;
 declare function parseJSXElement(element: t.JSXElement, fallback: true): CompositeMessage | ElementMessage;
 //#endregion

package/dist/parser.mjs

@@ -1,5 +1,5 @@
 import { AUTO_INCREMENT_IDENTIFIER, ArgumentMessage, ChoiceMessage, CompositeMessage, ElementMessage, LiteralMessage, isArgumentType, validateArgumentStyle, validateBranchIdentifier } from "@saykit/config/features/messages";
-import { unwrapPlaceholder } from "@saykit/transform-js/parser";
+import { getTranslatorComments, unwrapPlaceholder } from "@saykit/transform-js/parser";
 import * as t from "@babel/types";
 //#region src/parser.ts
 const TAG_ATTRIBUTE = "say-tag";
@@ -16,9 +16,9 @@ function parseJSXContainerElement(element) {
 	return new CompositeMessage({
 		id: descriptorId,
 		context: descriptorContext
-	}, [], getReferences(element), children, accessor, whitespace);
+	}, getTranslatorComments(element.leadingComments), getReferences(element), children, accessor, whitespace);
 }
-function parseJSXOpeningElement(element) {
+function parseJSXOpeningElement(element, comments) {
 	if (!element.selfClosing) return null;
 	const processed = processJSXOpeningElement(element);
 	if (!processed) return null;
@@ -28,7 +28,7 @@ function parseJSXOpeningElement(element) {
 	const wrap = (children) => new CompositeMessage({
 		id: descriptorId,
 		context: descriptorContext
-	}, [], getReferences(element), children, accessor);
+	}, getTranslatorComments(comments), getReferences(element), children, accessor);
 	if (typeof kind === "string" && [
 		"select",
 		"ordinal",
@@ -132,7 +132,7 @@ function processJSXOpeningElement(element) {
 	return null;
 }
 function parseJSXElement(element, fallback) {
-	const message = element.openingElement.selfClosing ? parseJSXOpeningElement(element.openingElement) : parseJSXContainerElement(element);
+	const message = element.openingElement.selfClosing ? parseJSXOpeningElement(element.openingElement, element.leadingComments) : parseJSXContainerElement(element);
 	if (message) return message;
 	if (!fallback) return null;
 	const identifier = takeElementTag(element) ?? AUTO_INCREMENT_IDENTIFIER;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saykit/transform-jsx",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "JSX and TSX source transformer for SayKit",
   "keywords": [
     "i18n",
@@ -47,8 +47,8 @@
     "@babel/parser": "^7.29.7",
     "@babel/traverse": "^7.29.7",
     "@babel/types": "^7.29.7",
-    "@saykit/config": "^0.9.0",
-    "@saykit/transform-js": "^0.9.0"
+    "@saykit/config": "^0.9.1",
+    "@saykit/transform-js": "^0.9.1"
   },
   "devDependencies": {
     "@babel/core": "^7.29.7",