@tsrx/prettier-plugin 0.3.99 → 0.3.101

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tsrx/prettier-plugin",
-  "version": "0.3.99",
+  "version": "0.3.101",
   "description": "Ripple plugin for Prettier",
   "type": "module",
   "module": "src/index.js",
@@ -27,7 +27,7 @@
     "prettier": "^3.8.4"
   },
   "dependencies": {
-    "@tsrx/core": "0.1.41"
+    "@tsrx/core": "0.1.43"
   },
   "files": [
     "src/"

package/src/index.js

@@ -2048,6 +2048,7 @@ function printRippleNode(node, path, options, print, args) {
 
 		case 'TSModuleDeclaration': {
 			nodeContent = [
+				node.declare ? 'declare ' : '',
 				node.metadata?.module_keyword ?? 'module',
 				' ',
 				path.call(print, 'id'),
@@ -2598,16 +2599,17 @@ function printExportNamedDeclaration(node, path, options, print) {
 		return parts;
 	} else if (node.specifiers && node.specifiers.length > 0) {
 		const specifiers = node.specifiers.map((spec) => {
+			const typePrefix = spec.exportKind === 'type' ? 'type ' : '';
 			const exportedName = /** @type {AST.Identifier} */ (spec.exported).name;
 			const localName = /** @type {AST.Identifier} */ (spec.local).name;
 			if (exportedName === localName) {
-				return localName;
+				return typePrefix + localName;
 			} else {
-				return localName + ' as ' + exportedName;
+				return typePrefix + localName + ' as ' + exportedName;
 			}
 		});
 
-		const parts = ['export { '];
+		const parts = [node.exportKind === 'type' ? 'export type { ' : 'export { '];
 		for (let i = 0; i < specifiers.length; i++) {
 			if (i > 0) parts.push(', ');
 			parts.push(specifiers[i]);
@@ -4696,7 +4698,9 @@ function printJSXSwitchCase(node, path, options, print, index) {
 		if (!child || child.type === 'EmptyStatement') {
 			continue;
 		}
-		printedConsequents.push(path.call(print, 'cases', index, 'consequent', i));
+		printedConsequents.push(
+			path.call((casePath) => casePath.call(print, 'consequent', i), 'cases', index),
+		);
 	}
 
 	const bodyDoc =

package/src/index.test.js

@@ -1507,6 +1507,36 @@ import { Something, type Props, track } from 'ripple';`;
 			expect(result).toBeWithNewline(expected);
 		});
 
+		// Regressed in 0.3.97: the printer dropped the `type` keyword from
+		// type-only re-exports and inline export specifiers, turning them into
+		// runtime re-exports of bindings that only exist as types (a module-load
+		// failure once compiled).
+		it('should keep the type keyword on export type statements', async () => {
+			const input = `export type { Config } from './types.js';
+export { type Extra, realValue } from './mixed.js';`;
+			const expected = `export type { Config } from './types.js';
+export { type Extra, realValue } from './mixed.js';`;
+			const result = await format(input, { singleQuote: true });
+			expect(result).toBeWithNewline(expected);
+		});
+
+		// Regressed in 0.3.97: `declare module 'name' { … }` lost its `declare`
+		// keyword, leaving invalid `module 'name' { … }` output.
+		it('should keep the declare keyword on ambient module declarations', async () => {
+			const input = `declare module 'some-module' {
+  interface Thing {
+    x: number;
+  }
+}`;
+			const expected = `declare module 'some-module' {
+  interface Thing {
+    x: number;
+  }
+}`;
+			const result = await format(input, { singleQuote: true });
+			expect(result).toBeWithNewline(expected);
+		});
+
 		it('should format long import statements correctly', async () => {
 			const input = `import { flushSync, track, effect, bindValue, bindChecked, bindGroup, bindClientWidth, bindClientHeight, bindOffsetWidth, bindOffsetHeight, bindContentRect, bindContentBoxSize, bindBorderBoxSize, bindDevicePixelContentBoxSize, bindInnerHTML, bindInnerText, bindTextContent, bindNode } from 'ripple';`;
 			const expected = `import {