@saykit/transform-jsx 0.6.0 → 0.6.1

package/dist/parser.mjs

@@ -1,4 +1,4 @@
-import { AUTO_INCREMENT_IDENTIFIER, ArgumentMessage, ChoiceMessage, CompositeMessage, ElementMessage, LiteralMessage } from "@saykit/config/features/messages";
+import { AUTO_INCREMENT_IDENTIFIER, ArgumentMessage, ChoiceMessage, CompositeMessage, ElementMessage, LiteralMessage, validateBranchIdentifier } from "@saykit/config/features/messages";
 import { unwrapPlaceholder } from "@saykit/transform-js/parser";
 import * as t from "@babel/types";
 //#region src/parser.ts
@@ -16,9 +16,7 @@ function parseJSXContainerElement(element) {
 	const [accessor] = processed;
 	const children = element.children.reduce((p, c, i, a) => {
 		if (t.isJSXText(c)) {
-			let text = c.value.replace(/\s+/g, " ");
-			if (i === 0) text = text.trimStart();
-			if (i === a.length - 1) text = text.trimEnd();
+			const text = normalizeJSXText(c.value, i === 0, i === a.length - 1);
 			if (text) p.push(new LiteralMessage(text));
 		}
 		if (t.isJSXElement(c)) p.push(parseJSXElement(c, true));
@@ -77,6 +75,7 @@ function parseJSXOpeningElement(element) {
 			}
 			return b;
 		}, []);
+		for (const branch of branches) validateBranchIdentifier(kind, branch.identifier);
 		const initialiser = findAttributeValueIfExpressionOrStringLiteral(element.attributes, "_");
 		if (!initialiser) return null;
 		const [identifier, selector] = unwrapPlaceholder(initialiser);
@@ -88,6 +87,31 @@ function parseJSXOpeningElement(element) {
 	}
 	return null;
 }
+/**
+* Punctuation that never takes a space in front of it. A formatter wrapping
+* long JSX regularly strands these at the start of a line, on their own or
+* ahead of the rest of a clause.
+*/
+const CLOSING_PUNCTUATION = /^[.,;:!?)\]}»”’]/;
+/**
+* Collapse the whitespace in a JSX text child the way the source reads.
+*
+* A line break between a word and its neighbour is only how a formatter wrapped
+* a long line, so it still means a space. Dropping it would run the words
+* together — and because ids are content hashes, silently orphan every existing
+* translation for the message.
+*
+* The exception is a line that begins with punctuation, which belongs tight
+* against whatever precedes it. Only an implicit break is treated this way: a
+* space the author wrote on the same line is deliberate and always survives.
+*/
+function normalizeJSXText(value, isFirst, isLast) {
+	let text = value.replace(/\s+/g, " ");
+	if (isFirst) text = text.trimStart();
+	else if (text.startsWith(" ") && /^[^\S\n]*\n/.test(value) && CLOSING_PUNCTUATION.test(text.slice(1))) text = text.slice(1);
+	if (isLast) text = text.trimEnd();
+	return text;
+}
 function processJSXOpeningElement(element) {
 	if (t.isJSXIdentifier(element.name) && element.name.name === "Say") return [element.name, null];
 	if (t.isJSXMemberExpression(element.name) && t.isJSXIdentifier(element.name.object) && element.name.object.name === "Say" && t.isJSXIdentifier(element.name.property)) return [element.name.object, element.name.property.name.toLowerCase()];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saykit/transform-jsx",
-  "version": "0.6.0",
+  "version": "0.6.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.6.0",
-    "@saykit/transform-js": "^0.6.0"
+    "@saykit/config": "^0.6.1",
+    "@saykit/transform-js": "^0.6.1"
   },
   "devDependencies": {
     "@babel/core": "^7.29.7",