Merge pull request #94 from fortanix/feature/gh-93-vite-dts-issues

Fix vite-plugin-dts issues + add installation test
fortanix · Jan 7, 2025 · 0880553 · 0880553
2 parents 84674f6 + 87fa3a1
commit 0880553
Show file tree

Hide file tree

Showing 24 changed files with 6,981 additions and 198 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -30,3 +30,4 @@ jobs:
     #- run: npm run build --if-present
     - run: npm run verify verify:source
     - run: npm test
+    - run: cd tests/installation && npm ci && npm test
diff --git a/biome.jsonc b/biome.jsonc
@@ -24,6 +24,7 @@
   "linter": {
     "enabled": true,
     "include": ["app/**/*", "src/**/*", "tests/**/*"],
+    "ignore": ["tests/installation/**/*"],
     "rules": {
       "recommended": true,
       "complexity": {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -68,7 +68,7 @@
     "glob": "^11.0.0",
     "vite": "^5.4.11",
     "@vitejs/plugin-react": "^4.3.4",
-    "vite-plugin-dts": "^4.3.0",
+    "vite-plugin-dts": "^4.4.0",
     "vite-plugin-lib-inject-css": "^2.1.1",
     "vite-plugin-svg-icons": "^2.0.1",
     "typescript": "^5.7.2",

diff --git a/package.json.js b/package.json.js
@@ -110,7 +110,7 @@ const packageConfig = {
     // Build
     'vite': '^5.4.11', // Cannot use vite v6, blocked by https://github.com/privatenumber/vite-css-modules/issues/13
     '@vitejs/plugin-react': '^4.3.4',
-    'vite-plugin-dts': '^4.3.0',
+    'vite-plugin-dts': '^4.4.0',
     'vite-plugin-lib-inject-css': '^2.1.1',
     'vite-plugin-svg-icons': '^2.0.1',
 

diff --git a/scripts/verify.ts b/scripts/verify.ts
@@ -5,7 +5,6 @@ import * as pathUtil from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { type PathLike, createReadStream } from 'node:fs';
 import * as fs from 'node:fs/promises';
-import { exec } from 'node:child_process';
 import { AsyncLocalStorage } from 'node:async_hooks';
 
 

diff --git a/tests/installation/.gitignore b/tests/installation/.gitignore
@@ -0,0 +1,24 @@
+# Logs
+logs
+*.log
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+pnpm-debug.log*
+lerna-debug.log*
+
+node_modules
+dist
+dist-ssr
+*.local
+
+# Editor directories and files
+.vscode/*
+!.vscode/extensions.json
+.idea
+.DS_Store
+*.suo
+*.ntvs*
+*.njsproj
+*.sln
+*.sw?
diff --git a/tests/installation/eslint.config.js b/tests/installation/eslint.config.js
@@ -0,0 +1,28 @@
+import js from '@eslint/js'
+import globals from 'globals'
+import reactHooks from 'eslint-plugin-react-hooks'
+import reactRefresh from 'eslint-plugin-react-refresh'
+import tseslint from 'typescript-eslint'
+
+export default tseslint.config(
+  { ignores: ['dist'] },
+  {
+    extends: [js.configs.recommended, ...tseslint.configs.recommended],
+    files: ['**/*.{ts,tsx}'],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+    plugins: {
+      'react-hooks': reactHooks,
+      'react-refresh': reactRefresh,
+    },
+    rules: {
+      ...reactHooks.configs.recommended.rules,
+      'react-refresh/only-export-components': [
+        'warn',
+        { allowConstantExport: true },
+      ],
+    },
+  },
+)
diff --git a/tests/installation/index.html b/tests/installation/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>Fortanix Baklava</title>
+  </head>
+  <body>
+    <div id="root"></div>
+    <script type="module" src="/src/main.tsx"></script>
+  </body>
+</html>