prefer-export-from: Fix TypeScript compatibility

rules/prefer-export-from.js

@@ -26,6 +26,10 @@ const getSpecifierName = node => {
 	}
 };
 
+const isTypeExportKind = node => node.exportKind === 'type' || node.parent.exportKind === 'type';
+
+const isTypeImportKind = node => node.importKind === 'type' || node.parent.importKind === 'type';
+
 function * removeSpecifier(node, fixer, sourceCode) {
 	const {parent} = node;
 	const {specifiers} = parent;
@@ -108,7 +112,16 @@ function getFixFunction({
 	const importDeclaration = imported.declaration;
 	const sourceNode = importDeclaration.source;
 	const sourceValue = sourceNode.value;
-	const exportDeclaration = exportDeclarations.find(({source}) => source.value === sourceValue);
+	const isTypeSpecifier = imported.isType || exported.isType;
+	let exportDeclaration;
+	if (isTypeSpecifier) {
+		// Prefer put type specifier into a type export declaration, so we don't need add `type ` before specifier
+		exportDeclaration = exportDeclarations.find(({source, exportKind}) => source.value === sourceValue && exportKind === 'type');
+	}
+
+	if (!exportDeclaration) {
+		exportDeclaration = exportDeclarations.find(({source, exportKind}) => source.value === sourceValue && exportKind !== 'type');
+	}
 
 	/** @param {import('eslint').Rule.RuleFixer} fixer */
 	return function * (fixer) {
@@ -118,24 +131,28 @@ function getFixFunction({
 				`\nexport * as ${exported.text} ${getSourceAndAssertionsText(importDeclaration, sourceCode)}`,
 			);
 		} else {
-			const specifier = exported.name === imported.name
+			let specifierText = exported.name === imported.name
 				? exported.text
 				: `${imported.text} as ${exported.text}`;
 
 			if (exportDeclaration) {
 				const lastSpecifier = exportDeclaration.specifiers[exportDeclaration.specifiers.length - 1];
 
+				if (isTypeSpecifier && exportDeclaration.exportKind !== 'type') {
+					specifierText = `type ${specifierText}`;
+				}
+
 				// `export {} from 'foo';`
 				if (lastSpecifier) {
-					yield fixer.insertTextAfter(lastSpecifier, `, ${specifier}`);
+					yield fixer.insertTextAfter(lastSpecifier, `, ${specifierText}`);
 				} else {
 					const openingBraceToken = sourceCode.getFirstToken(exportDeclaration, isOpeningBraceToken);
-					yield fixer.insertTextAfter(openingBraceToken, specifier);
+					yield fixer.insertTextAfter(openingBraceToken, specifierText);
 				}
 			} else {
 				yield fixer.insertTextAfter(
 					program,
-					`\nexport {${specifier}} ${getSourceAndAssertionsText(importDeclaration, sourceCode)}`,
+					`\nexport ${isTypeSpecifier ? 'type ' : ''}{${specifierText}} ${getSourceAndAssertionsText(importDeclaration, sourceCode)}`,
 				);
 			}
 		}
@@ -156,13 +173,15 @@ function getExported(identifier, context, sourceCode) {
 				node: parent,
 				name: DEFAULT_SPECIFIER_NAME,
 				text: 'default',
+				isType: isTypeExportKind(parent),
 			};
 
 		case 'ExportSpecifier':
 			return {
 				node: parent,
 				name: getSpecifierName(parent.exported),
 				text: sourceCode.getText(parent.exported),
+				isType: isTypeExportKind(parent),
 			};
 
 		case 'VariableDeclarator': {
@@ -212,6 +231,7 @@ function getImported(variable, sourceCode) {
 		node: specifier,
 		declaration: specifier.parent,
 		variable,
+		isType: isTypeImportKind(specifier),
 	};
 
 	switch (specifier.type) {

test/prefer-export-from.mjs

@@ -347,8 +347,201 @@ test.typescript({
 			type AceEditor = import("ace-builds").Ace.Editor;
 			export {AceEditor};
 		`,
+		'export type { bar, foo } from "foo";',
+	],
+	invalid: [
+		{
+			code: outdent`
+				import { foo } from "foo";
+				export { foo };
+				export type { bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type { bar } from "foo";
+				export {foo} from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import { foo } from "foo";
+				export { foo };
+				export { type bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { type bar, foo } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import { foo } from 'foo';
+				export { foo };
+				export type { bar } from "foo";
+				export { baz } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type { bar } from "foo";
+				export { baz, foo } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import { foo } from 'foo';
+				export { foo };
+				export { type bar } from "foo";
+				export { baz } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { type bar, foo } from "foo";
+				export { baz } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from "foo";
+				export type { foo };
+				export type { bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type { bar, foo } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import { foo } from 'foo';
+				export { foo };
+				export { baz } from "foo";
+				export { type bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { baz, foo } from "foo";
+				export { type bar } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from "foo";
+				export type { foo };
+				export { type bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { type bar, type foo } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from 'foo';
+				export type { foo };
+				export type { bar } from "foo";
+				export { baz } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type { bar, foo } from "foo";
+				export { baz } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from 'foo';
+				export type { foo };
+				export { baz } from "foo";
+				export type { bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { baz } from "foo";
+				export type { bar, foo } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from 'foo';
+				export type { foo };
+				export { type bar } from "foo";
+				export { baz } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { type bar, type foo } from "foo";
+				export { baz } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from 'foo';
+				export type { foo };
+				export { baz } from "foo";
+				export { type bar } from "foo";
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export { baz, type foo } from "foo";
+				export { type bar } from "foo";
+			`,
+		},
+		{
+			code: outdent`
+				import { type foo } from 'foo';
+				export type { foo };
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type {foo} from 'foo';
+			`,
+		},
+		{
+			code: outdent`
+				import { foo } from 'foo';
+				export type { foo };
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type {foo} from 'foo';
+			`,
+		},
+		{
+			code: outdent`
+				import type { foo } from 'foo';
+				export { type foo };
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type {foo} from 'foo';
+			`,
+		},
+		{
+			code: outdent`
+				import type foo from "foo";
+				export default foo
+			`,
+			errors: 1,
+			output: outdent`
+				\n
+				export type {default} from "foo";
+			`,
+		},
 	],
-	invalid: [],
 });
 
 // `ignoreUsedVariables`