Support for Arbitrary Module Identifiers syntax (fixes #52) [publish]

CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## Unreleased
+## 0.4.13
 
 - Support for `react-redux` connect (`export default connect(mapStateToProps, mapDispatchToProps)(MyComponent)`) (fixes #51)
+- Support for [Arbitrary Module Identifiers](https://devblogs.microsoft.com/typescript/announcing-typescript-5-6/#support-for-arbitrary-module-identifiers) syntax (fixes #52)
 
 ## 0.4.12
 

eslint.config.js

@@ -0,0 +1,3 @@
+import baseConfig from "@arnaud-barre/eslint-config";
+
+export default baseConfig;

package.json

@@ -1,30 +1,30 @@
 {
   "name": "eslint-plugin-react-refresh",
-  "version": "0.4.12",
+  "version": "0.4.13",
   "type": "module",
   "license": "MIT",
   "scripts": {
     "build": "scripts/bundle.ts",
     "test": "bun test",
-    "lint": "eslint ./ --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
+    "lint": "bun --bun eslint . --max-warnings 0",
     "prettier": "bun prettier-ci --write",
-    "prettier-ci": "prettier --ignore-path=.gitignore --check '**/*.{ts,json,md,yml}'",
+    "prettier-ci": "prettier --ignore-path=.gitignore --check '**/*.{js,ts,json,md,yml}'",
     "ci": "tsc && bun lint && bun prettier-ci && bun test && bun run build"
   },
   "prettier": {},
   "peerDependencies": {
     "eslint": ">=7"
   },
   "devDependencies": {
-    "@arnaud-barre/eslint-config": "^4.1.1",
+    "@arnaud-barre/eslint-config": "^5.0.1",
     "@arnaud-barre/tnode": "^0.19.2",
-    "@types/eslint": "^8.44.8",
-    "@types/node": "^20.10.2",
-    "@typescript-eslint/parser": "^7.18.0",
-    "@typescript-eslint/utils": "^7.18.0",
-    "bun-types": "^1.1.27",
-    "eslint": "^8.57.0",
+    "@types/eslint": "^9.6.1",
+    "@types/node": "^20.16.13",
+    "@typescript-eslint/parser": "^8.10.0",
+    "@typescript-eslint/utils": "^8.10.0",
+    "bun-types": "^1.1.31",
+    "eslint": "^9.13.0",
     "prettier": "3.0.3",
-    "typescript": "~5.4"
+    "typescript": "~5.6"
   }
 }

src/only-export-components.test.ts

@@ -1,16 +1,10 @@
 #!/usr/bin/env tnode
 import { test } from "bun:test";
+import parser from "@typescript-eslint/parser";
 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: 2022,
-    ecmaFeatures: { jsx: true },
-  },
-});
+const ruleTester = new RuleTester({ languageOptions: { parser } });
 
 const valid = [
   {
@@ -274,6 +268,11 @@ const invalid = [
     options: [{ allowExportNames: ["loader", "meta"] }],
     errorId: "namedExport",
   },
+  {
+    name: "Export with arbitrary module identifier",
+    code: 'const Foo = () => {}; export { Foo as "🍌"}',
+    errorId: "localComponents",
+  },
 ];
 
 const it = (name: string, cases: Parameters<typeof ruleTester.run>[2]) => {

src/only-export-components.ts

@@ -82,7 +82,10 @@ export const onlyExportComponents: TSESLint.RuleModule<
         let mayHaveReactExport = false;
         let reactIsInScope = false;
         const localComponents: TSESTree.Identifier[] = [];
-        const nonComponentExports: TSESTree.BindingName[] = [];
+        const nonComponentExports: (
+          | TSESTree.BindingName
+          | TSESTree.StringLiteral
+        )[] = [];
 
         const handleLocalIdentifier = (
           identifierNode: TSESTree.BindingName,
@@ -94,7 +97,7 @@ export const onlyExportComponents: TSESLint.RuleModule<
         };
 
         const handleExportIdentifier = (
-          identifierNode: TSESTree.BindingName,
+          identifierNode: TSESTree.BindingName | TSESTree.StringLiteral,
           isFunction?: boolean,
           init?: TSESTree.Expression | null,
         ) => {
@@ -232,7 +235,8 @@ export const onlyExportComponents: TSESLint.RuleModule<
             if (node.declaration) handleExportDeclaration(node.declaration);
             for (const specifier of node.specifiers) {
               handleExportIdentifier(
-                specifier.exported.name === "default"
+                specifier.exported.type === "Identifier" &&
+                  specifier.exported.name === "default"
                   ? specifier.local
                   : specifier.exported,
               );