feat: support TS/JSX inside node_modules

ArnaudBarre · ArnaudBarre · commit 4b9b2d5958ad · 2023-03-27T03:23:51.000+02:00
diff --git a/.gitignore b/.gitignore
@@ -2,4 +2,5 @@
 node_modules/
 dist/
 playground-temp/
+test-results/
 .swc/
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Support TS/JSX in node_modules to help the community experiment with it. Note that for now this not supported by TS and errors from these files cannot be silenced if the user is using a stricter configuration than the library author: https://github.com/microsoft/TypeScript/issues/30511. I advise to use it only for internal libraries for now (fixes #53)
+
 ## 3.2.0
 
 - Support HMR for MDX (fixes #52)
diff --git a/playground/ts-lib/__tests__/ts-lib.spec.ts b/playground/ts-lib/__tests__/ts-lib.spec.ts
@@ -0,0 +1,24 @@
+import { expect, test } from "@playwright/test";
+import { setupDevServer, setupBuildAndPreview } from "../../utils.ts";
+
+test("TS lib build", async ({ page }) => {
+  const { testUrl, server } = await setupBuildAndPreview("ts-lib");
+  await page.goto(testUrl);
+  await expect(page.locator("main")).toHaveText("Home page");
+
+  await page.locator("a", { hasText: "About" }).click();
+  await expect(page.locator("main")).toHaveText("About page");
+
+  await server.httpServer.close();
+});
+
+test("TS lib dev", async ({ page }) => {
+  const { testUrl, server } = await setupDevServer("ts-lib");
+  await page.goto(testUrl);
+  await expect(page.locator("main")).toHaveText("Home page");
+
+  await page.locator("a", { hasText: "About" }).click();
+  await expect(page.locator("main")).toHaveText("About page");
+
+  await server.close();
+});
diff --git a/playground/ts-lib/index.html b/playground/ts-lib/index.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <link rel="icon" type="image/svg+xml" href="/vite.svg" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <title>Vite + React + TS lib</title>
+  </head>
+  <body>
+    <div id="root"></div>
+    <script type="module" src="/src/index.tsx"></script>
+  </body>
+</html>
diff --git a/playground/ts-lib/package.json b/playground/ts-lib/package.json
@@ -0,0 +1,22 @@
+{
+  "name": "playground-ts-lib",
+  "type": "module",
+  "private": true,
+  "scripts": {
+    "dev": "vite",
+    "build": "vite build",
+    "preview": "vite preview"
+  },
+  "dependencies": {
+    "@generouted/react-router": "^1.11.7",
+    "generouted": "^1.11.7",
+    "react": "^18.2.0",
+    "react-dom": "^18.2.0",
+    "react-router-dom": "^6.9.0"
+  },
+  "devDependencies": {
+    "@types/react": "^18.0.29",
+    "@types/react-dom": "^18.0.11",
+    "@vitejs/plugin-react-swc": "../../dist"
+  }
+}
diff --git a/playground/ts-lib/public/vite.svg b/playground/ts-lib/public/vite.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 256 257"><defs><linearGradient id="a" x1="-.828%" x2="57.636%" y1="7.652%" y2="78.411%"><stop offset="0%" stop-color="#41D1FF"/><stop offset="100%" stop-color="#BD34FE"/></linearGradient><linearGradient id="b" x1="43.376%" x2="50.316%" y1="2.242%" y2="89.03%"><stop offset="0%" stop-color="#FFEA83"/><stop offset="8.333%" stop-color="#FFDD35"/><stop offset="100%" stop-color="#FFA800"/></linearGradient></defs><path fill="url(#a)" d="M255.153 37.938 134.897 252.976c-2.483 4.44-8.862 4.466-11.382.048L.875 37.958c-2.746-4.814 1.371-10.646 6.827-9.67l120.385 21.517a6.537 6.537 0 0 0 2.322-.004l117.867-21.483c5.438-.991 9.574 4.796 6.877 9.62Z"/><path fill="url(#b)" d="M185.432.063 96.44 17.501a3.268 3.268 0 0 0-2.634 3.014l-5.474 92.456a3.268 3.268 0 0 0 3.997 3.378l24.777-5.718c2.318-.535 4.413 1.507 3.936 3.838l-7.361 36.047c-.495 2.426 1.782 4.5 4.151 3.78l15.304-4.649c2.372-.72 4.652 1.36 4.15 3.788l-11.698 56.621c-.732 3.542 3.979 5.473 5.943 2.437l1.313-2.028 72.516-144.72c1.215-2.423-.88-5.186-3.54-4.672l-25.505 4.922c-2.396.462-4.435-1.77-3.759-4.114l16.646-57.705c.677-2.35-1.37-4.583-3.769-4.113Z"/></svg>
diff --git a/playground/ts-lib/src/index.tsx b/playground/ts-lib/src/index.tsx
@@ -0,0 +1,9 @@
+import { StrictMode } from "react";
+import { createRoot } from "react-dom/client";
+import { Routes } from "generouted/react-router";
+
+createRoot(document.getElementById("root")!).render(
+  <StrictMode>
+    <Routes />
+  </StrictMode>,
+);
diff --git a/playground/ts-lib/src/pages/404.tsx b/playground/ts-lib/src/pages/404.tsx
@@ -0,0 +1,3 @@
+export default function NotFound() {
+  return <h1>404</h1>
+}
diff --git a/playground/ts-lib/src/pages/_app.tsx b/playground/ts-lib/src/pages/_app.tsx
@@ -0,0 +1,16 @@
+import { Link, Outlet } from "react-router-dom";
+
+export default function App() {
+  return (
+    <section style={{ margin: 24 }}>
+      <header style={{ display: "flex", gap: 24 }}>
+        <Link to="/">Home</Link>
+        <Link to="/about">About</Link>
+      </header>
+
+      <main>
+        <Outlet />
+      </main>
+    </section>
+  );
+}
diff --git a/playground/ts-lib/src/pages/about.tsx b/playground/ts-lib/src/pages/about.tsx
@@ -0,0 +1,3 @@
+export default function About() {
+  return <h1>About page</h1>;
+}
diff --git a/playground/ts-lib/src/pages/index.tsx b/playground/ts-lib/src/pages/index.tsx
@@ -0,0 +1,10 @@
+import { Outlet } from "react-router-dom";
+
+export default function Home() {
+  return (
+    <div>
+      <h1>Home page</h1>
+      <Outlet />
+    </div>
+  );
+}
diff --git a/playground/ts-lib/tsconfig.json b/playground/ts-lib/tsconfig.json
@@ -0,0 +1,25 @@
+{
+  "include": ["src", "vite.config.ts"],
+  "compilerOptions": {
+    "module": "ESNext",
+    "lib": ["ESNext", "DOM", "DOM.Iterable"],
+    "target": "ESNext",
+    "jsx": "react-jsx",
+    "types": ["vite/client"],
+    "noEmit": true,
+    "isolatedModules": true,
+    "skipLibCheck": true,
+    "moduleResolution": "bundler",
+    "allowImportingTsExtensions": true,
+    "resolveJsonModule": true,
+    "noUncheckedIndexedAccess": true,
+    "noPropertyAccessFromIndexSignature": true,
+
+    /* Linting */
+    "strict": true,
+    "noUnusedLocals": true,
+    "noUnusedParameters": true,
+    "noFallthroughCasesInSwitch": true,
+    "useUnknownInCatchVariables": true
+  }
+}
diff --git a/playground/ts-lib/vite.config.ts b/playground/ts-lib/vite.config.ts
@@ -0,0 +1,7 @@
+import { defineConfig } from "vite";
+import react from "@vitejs/plugin-react-swc";
+
+export default defineConfig({
+  optimizeDeps: { include: ["react-router-dom"] },
+  plugins: [react()],
+});
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/index.ts b/src/index.ts

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+export default function NotFound() {`
	`2`	`+ return <h1>404</h1>`
	`3`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+export default function About() {`
	`2`	`+ return <h1>About page</h1>;`
	`3`	`+}`