Merge pull request #835 from secretkeylabs/chore/merge-develop-to-pub…

…lic-1724411378

merge-develop-to-public

.github/workflows/build-rc.yml

@@ -31,6 +31,7 @@ jobs:
         run: npm ci
       - name: Test
         run: |
+          npm run knip
           npx eslint .
           npx tsc --noEmit
           npm test

.github/workflows/build.yml

@@ -28,6 +28,7 @@ jobs:
         run: npm ci
       - name: Test
         run: |
+          npm run knip
           npx eslint .
           npx tsc --noEmit
           npm test
@@ -47,7 +48,7 @@ jobs:
         run: xvfb-run --auto-servernum --server-args="-screen 0 360x360x24" npm run e2etest:smoketest --reporter=html
       - name: Upload Playwright report
         if: always()
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: playwright-report
           path: playwright-report/

.github/workflows/playwright.yml

@@ -25,6 +25,7 @@ jobs:
         run: npm ci
       - name: Test
         run: |
+          npm run knip
           npx eslint .
           npx tsc --noEmit
           npm test

knip.jsonc

@@ -0,0 +1,25 @@
+{
+  "$schema": "https://unpkg.com/knip@5/schema.json",
+  "entry": [
+    "src/background/background.ts",
+    "src/content-scripts/content-script.ts",
+    "src/inpage/index.ts",
+    "src/pages/Options/index.tsx",
+    "src/pages/Popup/index.tsx",
+    "webpack/utils/build.js",
+    "webpack/utils/devServer.js"
+  ],
+  "project": ["src/**/*.{ts,tsx}", "webpack/**/*.js"],
+  "webpack": {
+    "config": ["webpack/webpack.config.js"]
+  },
+  "ignoreDependencies": [
+    // Used by webpack to polyfill references to Node.js's `path` module, yet
+    // not explicitly imported anywhere in the project.
+    "path",
+
+    // Used by the extension, yet not explicitly imported anywhere in the
+    // project.
+    "@types/chrome"
+  ]
+}