chore: finalize config, fix all non-prod knip issues

Author: gagik

knip.ts

@@ -1,6 +1,12 @@
 import type { KnipConfig } from "knip";
 
 const config: KnipConfig = {
+  rules: {
+    // Disable checking for unused exports, unused exported types, and duplicate dependencies.
+    exports: "off",
+    types: "off",
+    duplicates: "off",
+  },
   // Ignore patterns for files that should not be analyzed
   ignore: [
     // Build outputs
@@ -12,135 +18,151 @@ const config: KnipConfig = {
     // Generated files
     "**/tmp/**",
     "**/.sbom/**",
-    // Test fixtures and data
-    "**/test/fixtures/**",
     "**/test/data/**",
-    // Docker and scripts that are executed directly
-    "scripts/docker/**",
-    // Async rewriter3 has its own build system (Rust/WASM)
-    "packages/async-rewriter3/**",
-    // Configuration files
+    // Configuration files (from depcheck ignore-patterns)
     ".evergreen/**",
     "config/**",
     "configs/**",
+    "**/.eslintrc.js",
   ],
 
-  // Ignore dependencies that are used in build processes or special contexts
-  ignoreDependencies: [
-    // Used in build scripts and webpack configs
-    "terser-webpack-plugin",
-    "webpack-bundle-analyzer",
-    "webpack-cli",
-    "webpack-merge",
-    // Used in special build contexts
-    "mongodb-runner",
-    "mongodb-crypt-library-dummy",
-    // Used in scripts
-    "mongodb-sbom-tools",
-    "@pkgjs/nv",
-    // Used for husky git hooks
-    "husky",
-    // CLI tool
-    "depcheck",
-  ],
-
-  // Ignore binaries that are used in scripts
-  ignoreBinaries: ["depalign", "monorepo-where", "precommit"],
-
-  // Global entry files at root level
-  entry: ["scripts/**/*.{js,ts,mjs}", "testing/**/*.ts"],
-
-  // Project files to analyze at root level
-  project: [
-    "scripts/**/*.{js,ts,mjs}",
-    "testing/**/*.ts",
-    "configs/**/*.{js,ts}",
-  ],
+  // Received from @mongodb-js/sbom-tools
+  ignoreBinaries: ["mongodb-sbom-tools"],
 
   // Workspace-specific configurations
   workspaces: {
+    // Root package (monorepo root, no entry file needed)
     ".": {
-      // Root workspace configuration
-      entry: ["scripts/**/*.{js,ts,mjs}", "testing/**/*.ts"],
+      entry: ["scripts/*.ts"],
+      project: [],
+      ignoreBinaries: [
+        // Lerna is listed as an optional dependency.
+        "lerna",
+      ],
     },
 
     // Config packages
     "configs/eslint-config-mongosh": {
-      entry: ["index.js", "utils.js"],
+      entry: ["utils.js"],
     },
 
     "configs/tsconfig-mongosh": {
       entry: ["tsconfig.common.json"],
     },
 
-    // Main packages
-    "packages/*": {
-      entry: ["src/index.ts", "src/index.tsx", "lib/index.js", "bin/**/*.js"],
-      project: ["src/**/*.{ts,tsx}", "bin/**/*.js"],
-    },
-
     // Special cases for packages with different entry points
     "packages/cli-repl": {
-      entry: ["bin/mongosh.js", "src/run.ts", "dist/add-module-mapping.js"],
-      project: ["src/**/*.ts", "bin/**/*.js"],
+      project: ["src/**/*.ts", "bin/**/*.js", "test/**/*.ts"],
+      ignoreDependencies: [
+        // Eagerly loaded startup snapshot dependencies
+        "@mongodb-js/saslprep",
+        "socks",
+        "emphasize",
+        "ipv6-normalize",
+        "bindings",
+        "system-ca",
+        // Used for monkey-patching our s390x fix
+        "@tootallnate/quickjs-emscripten",
+      ],
     },
 
     "packages/shell-api": {
       entry: ["src/api.ts"],
-      project: ["src/**/*.ts"],
     },
 
-    "packages/build": {
-      entry: [
-        "src/evergreen-release.ts",
-        "src/release.ts",
-        "src/update-cta.ts",
-      ],
-      project: ["src/**/*.ts"],
+    "packages/mongosh": {
+      project: ["bin/**/*.js"],
     },
 
     "packages/e2e-tests": {
-      entry: ["test/**/*.ts"],
-      project: ["test/**/*.ts"],
+      entry: ["test/**/*", "test/fixtures/**/*"],
+      ignoreDependencies: [
+        // Used indirectly
+        "@mongosh/cli-repl",
+      ],
+    },
+
+    "scripts/docker": {
+      ignoreDependencies: [
+        // Used by build.sh script.
+        "mongodb-crypt-library-version",
+      ],
+    },
+
+    "packages/service-provider-node-driver": {
+      ignoreDependencies: [
+        // Used for MONGODB-AWS auth
+        // See: https://github.com/mongodb-js/mongosh/pull/1149
+        // See: https://jira.mongodb.org/browse/NODE-5005
+        "aws4",
+      ],
+    },
+
+    "packages/node-runtime-worker-thread": {
+      ignoreDependencies: [
+        // Used in worker thread context
+        "system-ca",
+      ],
+      ignoreFiles: [
+        // Used in package.json
+        "tests/register-worker.js",
+      ],
     },
 
     "packages/browser-repl": {
-      entry: ["src/index.tsx"],
+      entry: ["config/*.js"],
       project: ["src/**/*.{ts,tsx}"],
+      ignoreDependencies: [
+        "@wojtekmaj/enzyme-adapter-react-17",
+        "enzyme",
+        // Karma test runner plugins
+        "karma-chrome-launcher",
+        "karma-mocha",
+        "karma-mocha-reporter",
+        "karma-typescript",
+        // Resolved as `<depname>/` so depcheck doesn't see it being used
+        "buffer",
+        "util",
+      ],
     },
 
     "packages/java-shell": {
-      entry: [
-        "src/main/kotlin/**/*.kt",
-        "src/test/kotlin/**/*.kt",
-        "gen-doc/**/*.js",
-        "gen-kotlin/**/*.js",
-      ],
-      project: [
-        "src/main/kotlin/**/*.kt",
-        "gen-doc/**/*.js",
-        "gen-kotlin/**/*.js",
+      entry: ["src/test/js/run-tests.ts"],
+      project: ["src/main/js/**/*"],
+      ignoreDependencies: [
+        // Used in webpack and build scripts
+        "bson",
+        "tr46",
+        "assert",
+        "buffer",
+        "util",
       ],
     },
 
     "packages/connectivity-tests": {
-      entry: ["index.js"],
-      project: ["**/*.js"],
+      entry: ["scripts/disable-dns-srv.js"],
+      // This package only contains bash test scripts
+      ignoreDependencies: [
+        // Used by test scripts
+        "mongosh",
+      ],
     },
 
-    testing: {
-      entry: [
-        "eventually.ts",
-        "fake-kms.ts",
-        "integration-testing-hooks.ts",
-        "disable-dns-srv.js",
+    "packages/async-rewriter2": {
+      entry: ["test/fixtures/**/*"],
+      ignoreFiles: [
+        // Used by make-runtime-support.js
+        "src/runtime-support.nocov.js",
       ],
-      project: ["**/*.{ts,js}"],
     },
 
-    "scripts/docker": {
-      entry: ["**/*.js"],
-      project: ["**/*.js"],
+    "packages/snippet-manager": {
+      entry: ["test/fixtures/**/*"],
+    },
+
+    testing: {
+      entry: ["src/**/*.ts"],
+      project: ["src/**/*.ts"],
     },
   },
 

package-lock.json

@@ -1,7 +1,6 @@
 {
   "name": "mongosh",
   "description": "The MongoDB Shell",
-  "main": "index.js",
   "bin": {
     "mongosh": "packages/cli-repl/bin/mongosh.js"
   },
@@ -15,9 +14,7 @@
     "check-ci": "npm run check --workspaces --if-present",
     "depalign": "depalign",
     "predepcheck": "npm run depalign",
-    "depcheck": "npm run depcheck --workspaces --if-present",
-    "knip": "knip",
-    "knip:summary": "knip --reporter summary",
+    "depcheck": "knip",
     "knip:production": "knip --production --strict",
     "lint": "npm run lint --workspaces --if-present",
     "test": "rimraf .nyc_output && npm run test-coverage --workspaces --if-present && npm run report-coverage",
@@ -101,31 +98,24 @@
     "node": ">=20.19.3"
   },
   "devDependencies": {
-    "@babel/compat-data": "^7.26.8",
     "@mongodb-js/monorepo-tools": "^1.1.10",
     "@pkgjs/nv": "^0.2.2",
     "@types/chai": "^5.2.3",
     "@types/mocha": "^5.2.7",
     "@types/node": "^22.15.30",
-    "@types/rimraf": "^3.0.0",
     "@types/semver": "^7.3.4",
     "@types/sinon-chai": "^4.0.0",
-    "@types/which": "^1.3.2",
     "chai": "^6.2.1",
     "cross-env": "^6.0.3",
-    "depcheck": "^1.4.7",
-    "duplexpair": "^1.0.2",
     "find-up": "^5.0.0",
     "glob": "^10.3.12",
     "husky": "^9.0.11",
+    "knip": "^5.69.1",
     "mocha": "^10.2.0",
-    "mongodb": "^6.19.0",
     "mongodb-runner": "^6.0.0",
-    "node-gyp": "^11.5.0",
     "nyc": "^15.1.0",
     "pkg-up": "^3.1.0",
     "rimraf": "^3.0.2",
-    "semver": "^7.6.3",
     "sinon": "^7.5.0",
     "sinon-chai": "^4.0.1",
     "terser-webpack-plugin": "^5.3.11",
@@ -136,9 +126,7 @@
     "webpack": "^5.99.9",
     "webpack-bundle-analyzer": "^4.7.0",
     "webpack-cli": "^6.0.1",
-    "which": "^2.0.2",
-    "yaml": "^1.10.0",
-    "knip": "^5.69.1"
+    "yaml": "^1.10.0"
   },
   "optionalDependencies": {
     "lerna": "^8.1.8"

package.json

@@ -28,7 +28,8 @@
     "path-browserify": "^1.0.1",
     "stream-browserify": "^3.0.0",
     "util": "^0.12.5",
-    "webpack-merge": "^5.8.0"
+    "webpack-merge": "^5.8.0",
+    "@mongosh/testing": "0.0.0-dev.0"
   },
   "license": "SSPL",
   "publishConfig": {