Ignore export type * (fixes #12)

Author: ArnaudBarre

CHANGELOG.md

@@ -1,12 +1,16 @@
 # Changelog
 
+## Unreleased
+
+- Ignore `export type *` (fixes #12)
+
 ## 0.4.0
 
 ### Add `allowConstantExport` option (fixes #8)
 
-This option allow tp don't warn when a constant (string, number, boolean, templateLiteral) is exported aside one or more components.
+This option allow to don't warn when a constant (string, number, boolean, templateLiteral) is exported aside one or more components.
 
-This should be enabled if the fast refresh implementation correctly handles this case (HMR when the constant doesn't change, propagate update to importers when the constant changes.). Vite supports it, PR welcome if you notice other integrations works well.
+This should be enabled if the fast refresh implementation correctly handles this case (HMR when the constant doesn't change, propagate update to importers when the constant changes). Vite supports it, PR welcome if you notice other integrations works well.
 
 ### Allow all-uppercase function exports (fixes #11)
 

bun.lockb

@@ -6,7 +6,6 @@
     "build": "scripts/bundle.ts",
     "prettier": "bun prettier-ci --write",
     "prettier-ci": "prettier --ignore-path=.gitignore --check '**/*.{ts,json,md,yml}'",
-    "test": "src/tests.ts",
     "ci": "tsc && bun prettier-ci && bun test && bun run build"
   },
   "prettier": {
@@ -19,6 +18,7 @@
     "@nabla/tnode": "^0.9.0",
     "@types/eslint": "^8.37.0",
     "@types/node": "^18.16.3",
+    "@typescript-eslint/parser": "^5.59.2",
     "@typescript-eslint/utils": "^5.59.1",
     "bun-types": "^0.5.8",
     "eslint": "^8.39.0",

package.json

@@ -4,9 +4,10 @@ import { RuleTester } from "eslint";
 import { onlyExportComponents } from "./only-export-components.ts";
 
 const ruleTester = new RuleTester({
+  parser: require.resolve("@typescript-eslint/parser"),
   parserOptions: {
     sourceType: "module",
-    ecmaVersion: 2018,
+    ecmaVersion: 2022,
     ecmaFeatures: { jsx: true },
   },
 });
@@ -76,6 +77,11 @@ const valid = [
     name: "Direct export default AF",
     code: "export default function foo () {};",
   },
+  {
+    name: "export type *",
+    code: "export type * from './module';",
+    filename: "Test.tsx",
+  },
   {
     name: "Mixed export in JS without checkJS",
     code: "export const foo = () => {}; export const Bar = () => {};",

src/only-export-components.test.ts

@@ -132,6 +132,7 @@ export const onlyExportComponents: TSESLint.RuleModule<
 
         for (const node of program.body) {
           if (node.type === "ExportAllDeclaration") {
+            if (node.exportKind === "type") continue;
             hasExports = true;
             context.report({ messageId: "exportAll", node });
           } else if (node.type === "ExportDefaultDeclaration") {