Added knip config and action for unused files/exports

.github/workflows/knip.yml

@@ -0,0 +1,50 @@
+name: Knip check
+
+on:
+  pull_request:
+    types: [opened, synchronize]
+    branches-ignore: [staging, production]
+    paths:
+      - '**.js'
+      - '**.jsx'
+      - '**.ts'
+      - '**.tsx'
+      - '**.mjs'
+      - '**.cjs'
+      - 'knip.json'
+      - 'package.json'
+      - 'package-lock.json'
+      - 'patches/**'
+      - 'tsconfig.json'
+
+concurrency:
+  group: ${{ github.ref == 'refs/heads/main' && format('{0}-{1}', github.ref, github.sha) || github.ref }}-knip
+  cancel-in-progress: true
+
+jobs:
+  knip-compare:
+    name: Compare knip issues against main
+    runs-on: blacksmith-2vcpu-ubuntu-2404
+    steps:
+      - name: Checkout PR
+        uses: useblacksmith/checkout@c9796daa2a4bdebdab5bd16be2c09a70cd4e1121 # v1
+
+      - name: Setup Node
+        uses: ./.github/actions/composite/setupNode
+
+      - name: Run knip on PR
+        run: KNIP=true npx knip --reporter json --no-exit-code > /tmp/knip-pr.json
+
+      - name: Checkout main
+        run: |
+          git fetch origin main --no-tags --depth=1
+          git checkout origin/main
+
+      - name: Run knip on main
+        run: KNIP=true npx knip --reporter json --no-exit-code > /tmp/knip-main.json
+
+      - name: Restore PR workspace
+        run: git checkout ${{ github.sha }}
+
+      - name: Compare reports
+        run: npx ts-node ./scripts/compareKnipReports.ts --mainPath=/tmp/knip-main.json --prPath=/tmp/knip-pr.json

babel.config.js

@@ -163,18 +163,22 @@ if (process.env.CAPTURE_METRICS === 'true') {
 }
 
 module.exports = (api) => {
-    console.debug('babel.config.js');
-    console.debug('  - api.version:', api.version);
-    console.debug('  - api.env:', api.env());
-    console.debug('  - process.env.NODE_ENV:', process.env.NODE_ENV);
-    console.debug('  - process.env.BABEL_ENV:', process.env.BABEL_ENV);
+    if (!process.env.KNIP) {
+        console.debug('babel.config.js');
+        console.debug('  - api.version:', api.version);
+        console.debug('  - api.env:', api.env());
+        console.debug('  - process.env.NODE_ENV:', process.env.NODE_ENV);
+        console.debug('  - process.env.BABEL_ENV:', process.env.BABEL_ENV);
+    }
 
     // For `react-native` (iOS/Android) caller will be "metro"
     // For `webpack` (Web) caller will be "@babel-loader"
     // For jest, it will be babel-jest
     // For `storybook` there won't be any config at all so we must give default argument of an empty object
     const runningIn = api.caller((args = {}) => args.name);
-    console.debug('  - running in: ', runningIn);
+    if (!process.env.KNIP) {
+        console.debug('  - running in: ', runningIn);
+    }
 
     return ['metro', 'babel-jest'].includes(runningIn) ? metro : webpack;
 };

cspell.json

@@ -976,7 +976,8 @@
         "مثال",
         "NitroFetch",
         "Prefetch",
-        "Prefetcher"
+        "Prefetcher",
+        "knip"
     ],
     "ignorePaths": [
         ".gitignore",

knip.json

@@ -0,0 +1,67 @@
+{
+    "entry": [
+        "index.js",
+        "wdyr.ts",
+        "src/App.tsx",
+        "src/HybridAppHandler.tsx",
+        "scripts/**/*.{js,ts}",
+        "web/proxy.ts",
+        "config/webpack/**/*.{js,mjs,ts}",
+        ".github/scripts/**/*.ts",
+        ".github/actions/javascript/**/*.ts",
+        ".storybook/**/*.{js,ts,tsx}",
+        "metro.config.js",
+        "eslint.changed.config.mjs",
+        "react-native.config.js",
+        "rock.config.mjs"
+    ],
+    "project": [
+        "src/**/*.{js,jsx,ts,tsx}",
+        "tests/**/*.{js,jsx,ts,tsx}",
+        "__mocks__/**/*.{js,jsx,ts,tsx}",
+        "web/**/*.{js,jsx,ts,tsx}",
+        "config/**/*.{js,mjs,ts,tsx}",
+        "scripts/**/*.{js,ts}",
+        "jest/**/*.{js,ts}",
+        ".storybook/**/*.{js,ts,tsx}",
+        ".github/actions/javascript/**/*.ts"
+    ],
+    "ignore": [".github/actions/**/index.js", "tests/perf-test/**", "web/snippets/gib.js"],
+    "ignoreDependencies": [
+        "@expensify/react-native-hybrid-app",
+        "group-ib-fp",
+        "react-native-image-size",
+        "react-native-picker-select",
+        "lodash",
+        "@babel/plugin-proposal-private-methods",
+        "@babel/plugin-proposal-private-property-in-object",
+        "babel-plugin-module-resolver",
+        "babel-plugin-transform-remove-console",
+        "@fullstory/babel-plugin-react-native",
+        "eslint-config-airbnb-typescript",
+        "eslint-config-prettier",
+        "eslint-plugin-storybook",
+        "@dword-design/eslint-plugin-import-alias",
+        "tsconfig-paths",
+        "@vercel/ncc",
+        "shellcheck",
+        "patch-package",
+        "diff-so-fancy"
+    ],
+    "ignoreBinaries": ["metro-symbolicate", "mkcert"],
+    "eslint": {
+        "config": ["config/eslint/eslint.config.mjs", "eslint.changed.config.mjs"]
+    },
+    "webpack": {
+        "config": ["config/webpack/webpack.common.ts"]
+    },
+    "babel": {
+        "config": ["babel.config.js"]
+    },
+    "jest": {
+        "config": ["jest.config.js"]
+    },
+    "storybook": {
+        "config": [".storybook/main.ts"]
+    }
+}