chore: replace depcheck with knip

knip.config.ts

@@ -0,0 +1,78 @@
+import type { KnipConfig } from 'knip';
+
+const config: KnipConfig = {
+  ignoreBinaries: [
+    // These are installed, but don't have valid package.json bin fields for knip to detect them
+    'stylelint'
+  ],
+  workspaces: {
+    '.': {
+      entry: [
+        'scripts/**/*'
+      ],
+      ignoreDependencies: [
+        // Workspace packages
+        'eslint-config-jwp',
+      ],
+      ignoreBinaries: [
+        // false positives from yarn scripts in github actions
+        'build',
+        'global',
+        'start:test',
+        'codecept:*'
+      ],
+    },
+    'packages/ui-react': {
+      ignoreDependencies: [
+        'sass' // Used in css
+      ]
+    },
+    'platforms/web': {
+      ignoreDependencies: [
+        '@codeceptjs/allure-legacy',
+        '@codeceptjs/configure', // Used in e2e tests
+        '@babel/plugin-proposal-decorators', // Used to build with decorators for ioc resolution
+        '@babel/core', // Required peer dependency for babel plugins
+        '@types/luxon', // Used in tests
+        'babel-plugin-transform-typescript-metadata', // Used to build with decorators for ioc resolution
+        'eslint-plugin-codeceptjs', // Used by apps/web/test-e2e/.eslintrc.cjs
+        'luxon', // Used in tests
+        'playwright', // Used in test configs
+        'sharp', // Requirement for @vite-pwa/assets-generator
+        'tsconfig-paths', // Used for e2e test setup
+        'virtual:pwa-register', // Service Worker code is injected at build time,
+      ]
+    },
+    'configs/eslint-config-jwp': {
+      entry: [
+        '*.*'
+      ],
+      ignoreDependencies: [
+        // Dynamically loaded in the eslint config
+        '@typescript-eslint/eslint-plugin',
+        '@typescript-eslint/parser',
+        'eslint-plugin-import',
+        'eslint-plugin-react',
+        'eslint-plugin-react-hooks'
+      ]
+    },
+    'configs/postcss-config-jwp': {
+      ignoreDependencies: [
+        // Dynamically loaded in the postcss config
+        'postcss-scss'
+      ]
+    },
+    'configs/stylelint-config-jwp': {
+      ignoreDependencies: [
+        // Dynamically loaded in the stylelint config
+        'stylelint',
+        'stylelint-order',
+        'stylelint-config-recommended-scss',
+        'stylelint-declaration-strict-value',
+        'stylelint-scss'
+      ]
+    }
+  }
+};
+
+export default config;

package.json

@@ -16,7 +16,7 @@
   ],
   "scripts": {
     "commit-msg": "commitlint --edit $1",
-    "depcheck": "depcheck && yarn workspaces run depcheck",
+    "depcheck": "knip --dependencies",
     "format": "run-s -c format:*",
     "format:eslint": "eslint \"{**/*,*}.{js,ts,jsx,tsx}\" --fix",
     "format:prettier": "prettier --write \"{**/*,*}.{js,ts,jsx,tsx}\"",
@@ -39,10 +39,10 @@
     "@commitlint/config-conventional": "^12.1.1",
     "@types/node": "^18.15.3",
     "csv-parse": "^5.4.0",
-    "depcheck": "^1.4.3",
     "eslint": "^8.57.0",
     "husky": "^6.0.0",
     "i18next-parser": "^8.0.0",
+    "knip": "^5.0.3",
     "lint-staged": "^15.1.0",
     "npm-run-all": "^4.1.5",
     "prettier": "^2.8.8",

packages/common/package.json

@@ -2,7 +2,9 @@
   "name": "@jwp/ott-common",
   "version": "4.30.0",
   "private": true,
-  "main": "./src",
+  "exports": {
+    "./*": "./src/**/*"
+  },
   "scripts": {
     "lint:ts": "tsc --pretty --noEmit -p ./",
     "test": "TZ=UTC LC_ALL=en_US.UTF-8 vitest run",

packages/hooks-react/package.json

@@ -23,6 +23,7 @@
   "devDependencies": {
     "@testing-library/react": "^14.0.0",
     "@types/jwplayer": "^8.2.13",
+    "jsdom": "^22.1.0",
     "vi-fetch": "^0.8.0",
     "vitest": "^1.3.1"
   },

packages/ui-react/package.json

@@ -2,13 +2,15 @@
   "name": "@jwp/ott-ui-react",
   "version": "4.30.0",
   "private": true,
-  "main": "./src",
   "scripts": {
     "lint:ts": "tsc --pretty --noEmit -p ./",
     "test": "TZ=UTC LC_ALL=en_US.UTF-8 vitest run",
     "test-update": "TZ=UTC LC_ALL=en_US.UTF-8 vitest run --update",
     "test-watch": "TZ=UTC LC_ALL=en_US.UTF-8 vitest"
   },
+  "exports": {
+    "./*": "./src/**/*"
+  },
   "dependencies": {
     "@adyen/adyen-web": "^5.42.1",
     "@inplayer-org/inplayer.js": "^3.13.24",
@@ -41,6 +43,7 @@
     "@types/payment": "^2.1.4",
     "@types/react-infinite-scroller": "^1.2.3",
     "@vitejs/plugin-react": "^4.0.4",
+    "jsdom": "^22.1.0",
     "sass": "^1.49.10",
     "typescript-plugin-css-modules": "^5.0.2",
     "vi-fetch": "^0.8.0",

platforms/web/package.json

@@ -67,6 +67,7 @@
     "sass": "^1.49.10",
     "sharp": "^0.33.2",
     "stylelint": "^15.11.0",
+    "stylelint-config-jwp": "*",
     "ts-node": "^10.9.1",
     "tsconfig-paths": "^4.1.0",
     "vi-fetch": "^0.8.0",