Merge pull request #322 from inversify/chore/clean-dependencies

Clean dependencies
inversify · Jan 18, 2025 · 724672f · 724672f
2 parents 23ca7fe + 48955a1
commit 724672f
Show file tree

Hide file tree

Showing 21 changed files with 66 additions and 44 deletions.
diff --git a/knip.ts b/knip.ts
@@ -0,0 +1,47 @@
+import { KnipConfig } from "knip";
+
+type RecordValues<T> = T extends Record<any, infer U> ? U : never;
+
+type WorkspaceProjectConfig = RecordValues<Required<KnipConfig["workspaces"]>>;
+
+const defaultWorkspaceProjectConfig: WorkspaceProjectConfig = {
+  entry: [
+    "{index,cli,main}.{js,cjs,mjs,jsx,ts,cts,mts,tsx}",
+    "src/{index,cli,main}.{js,cjs,mjs,jsx,ts,cts,mts,tsx}",
+  ],
+  project: [
+    "**/*.{js,cjs,mjs,jsx,ts,cts,mts,tsx}!",
+    "!jest.config.stryker.mjs",
+    "!**/__mocks__",
+  ],
+};
+
+export default {
+  workspaces: {
+    ".": {
+      entry: ["config/commitlint/commitlint.config.js"],
+      project: [],
+    },
+    "packages/container/libraries/*": defaultWorkspaceProjectConfig,
+    "packages/container/tools/*": defaultWorkspaceProjectConfig,
+    "packages/docs/services/*": defaultWorkspaceProjectConfig,
+    "packages/docs/tools/inversify-code-examples": {
+      entry: ["src/examples/**/*.ts", "src/scripts/generateExamples.mts"],
+      project: defaultWorkspaceProjectConfig.project,
+    },
+    "packages/docs/tools/*": defaultWorkspaceProjectConfig,
+    "packages/foundation/libraries/*": defaultWorkspaceProjectConfig,
+    "packages/foundation/tools/*": defaultWorkspaceProjectConfig,
+    "packages/container/tools/e2e-tests": {
+      entry: [
+        "src/*/parameters/*.ts",
+        "src/*/step-definitions/*.ts",
+        "src/app/hooks/*.ts",
+      ],
+      project: [
+        ...(defaultWorkspaceProjectConfig.project as string[]),
+        "!config/*",
+      ],
+    },
+  },
+} satisfies KnipConfig;
diff --git a/package.json b/package.json
@@ -9,14 +9,15 @@
     "@commitlint/cli": "19.6.1",
     "@commitlint/config-conventional": "19.6.0",
     "@commitlint/prompt-cli": "19.7.0",
-    "@inversifyjs/foundation-prettier-config": "workspace:*",
     "@inversifyjs/foundation-eslint-config": "workspace:*",
     "@inversifyjs/foundation-jest-config": "workspace:*",
+    "@inversifyjs/foundation-prettier-config": "workspace:*",
     "@inversifyjs/foundation-rollup-config": "workspace:*",
     "@inversifyjs/foundation-scripts": "workspace:*",
     "@inversifyjs/foundation-stryker-config": "workspace:*",
     "@inversifyjs/foundation-typescript-config": "workspace:*",
     "husky": "9.1.7",
+    "knip": "5.42.2",
     "lint-staged": "15.4.1",
     "turbo": "2.3.3"
   },
@@ -34,6 +35,7 @@
     "build": "turbo run build",
     "commit": "commit",
     "format": "turbo run format",
+    "knip": "knip",
     "lint": "turbo run lint",
     "prepare": "husky && turbo telemetry disable",
     "prerelease": "pnpm run build",

diff --git a/packages/container/libraries/common/package.json b/packages/container/libraries/common/package.json
@@ -5,22 +5,18 @@
   },
   "description": "InversifyJs common utils",
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@stryker-mutator/core": "8.7.1",
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/container/libraries/common/src/services/models/LazyServiceIdentifier.ts b/packages/container/libraries/common/src/services/models/LazyServiceIdentifier.ts
@@ -1,6 +1,6 @@
 import { ServiceIdentifier } from './ServiceIdentifier';
 
-export const islazyServiceIdentifierSymbol: unique symbol = Symbol.for(
+const islazyServiceIdentifierSymbol: unique symbol = Symbol.for(
   '@inversifyjs/common/islazyServiceIdentifier',
 );
 

diff --git a/packages/container/libraries/container/package.json b/packages/container/libraries/container/package.json
@@ -10,22 +10,18 @@
     "@inversifyjs/reflect-metadata-utils": "workspace:*"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@stryker-mutator/core": "8.7.1",
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/container/libraries/container/src/error/models/InversifyContainerError.ts b/packages/container/libraries/container/src/error/models/InversifyContainerError.ts
@@ -1,6 +1,6 @@
 import { InversifyContainerErrorKind } from './InversifyContainerErrorKind';
 
-export const isAppErrorSymbol: unique symbol = Symbol.for(
+const isAppErrorSymbol: unique symbol = Symbol.for(
   '@inversifyjs/container/InversifyContainerError',
 );
 

diff --git a/packages/container/libraries/core/package.json b/packages/container/libraries/core/package.json
@@ -10,14 +10,11 @@
     "@inversifyjs/reflect-metadata-utils": "workspace:*"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@stryker-mutator/core": "8.7.1",
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
@@ -26,7 +23,6 @@
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/container/libraries/core/src/common/models/OneToManyMapStar.ts b/packages/container/libraries/core/src/common/models/OneToManyMapStar.ts
@@ -2,7 +2,7 @@ import { Cloneable } from './Cloneable';
 
 const NOT_FOUND_INDEX: number = -1;
 
-export type OneToManyMapStartSpec<TRelation extends object> = {
+type OneToManyMapStartSpec<TRelation extends object> = {
   [TKey in keyof TRelation]: {
     isOptional: undefined extends TRelation[TKey] ? true : false;
   };

diff --git a/packages/container/libraries/core/src/error/models/InversifyCoreError.ts b/packages/container/libraries/core/src/error/models/InversifyCoreError.ts
@@ -1,6 +1,6 @@
 import { InversifyCoreErrorKind } from './InversifyCoreErrorKind';
 
-export const isAppErrorSymbol: unique symbol = Symbol.for(
+const isAppErrorSymbol: unique symbol = Symbol.for(
   '@inversifyjs/core/InversifyCoreError',
 );
 

diff --git a/packages/container/libraries/strongly-typed/package.json b/packages/container/libraries/strongly-typed/package.json
@@ -5,19 +5,15 @@
   },
   "description": "InversifyJs strong type definitions",
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/container/tools/e2e-tests/package.json b/packages/container/tools/e2e-tests/package.json
@@ -11,9 +11,6 @@
     "@inversifyjs/core": "workspace:*"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",

diff --git a/packages/container/tools/e2e-tests/src/common/models/InversifyWorld.ts b/packages/container/tools/e2e-tests/src/common/models/InversifyWorld.ts
@@ -3,12 +3,12 @@ import { Container } from '@inversifyjs/container';
 
 import { BindingParameter } from '../../binding/models/BindingParameter';
 
-export interface EntitiesMap {
+interface EntitiesMap {
   bindings: Map<string, BindingParameter>;
   containers: Map<string, Container>;
 }
 
-export interface ContainerRequests {
+interface ContainerRequests {
   get: Map<string, unknown>;
 }
 

diff --git a/packages/docs/services/inversify-site/package.json b/packages/docs/services/inversify-site/package.json
@@ -18,10 +18,7 @@
   "devDependencies": {
     "@docusaurus/module-type-aliases": "3.7.0",
     "@docusaurus/types": "3.7.0",
-    "@eslint/js": "9.18.0",
     "@types/react": "18.3.18",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "eslint-plugin-prettier": "5.2.2",
     "eslint-plugin-simple-import-sort": "12.1.1",

diff --git a/packages/docs/tools/inversify-code-examples/package.json b/packages/docs/tools/inversify-code-examples/package.json
@@ -9,19 +9,15 @@
     "inversify7": "npm:[email protected]"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "glob": "11.0.1",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "reflect-metadata": "0.2.2",
     "rimraf": "6.0.1",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "typescript": "5.7.3"
   },
   "homepage": "https://inversify.io",

diff --git a/packages/foundation/libraries/prototype-utils/package.json b/packages/foundation/libraries/prototype-utils/package.json
@@ -29,22 +29,18 @@
     "@inversifyjs/common": "workspace:*"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@stryker-mutator/core": "8.7.1",
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/foundation/libraries/reflect-metadata-utils/package.json b/packages/foundation/libraries/reflect-metadata-utils/package.json
@@ -31,22 +31,18 @@
     "url": "https://github.com/inversify/monorepo/issues"
   },
   "devDependencies": {
-    "@eslint/js": "9.18.0",
     "@jest/globals": "29.7.0",
     "@stryker-mutator/core": "8.7.1",
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@types/node": "22.10.7",
-    "@typescript-eslint/eslint-plugin": "8.20.0",
-    "@typescript-eslint/parser": "8.20.0",
     "eslint": "9.18.0",
     "jest": "29.7.0",
     "prettier": "3.4.2",
     "rimraf": "6.0.1",
     "rollup": "4.30.1",
     "ts-loader": "9.5.2",
     "ts-jest": "29.2.5",
-    "ts-node": "10.9.2",
     "tslib": "2.8.1",
     "typescript": "5.7.3"
   },

diff --git a/packages/foundation/tools/jest-config/lib/config/getJestJsProjectConfig.js b/packages/foundation/tools/jest-config/lib/config/getJestJsProjectConfig.js
@@ -5,7 +5,7 @@ import getTestMatch from './getTestMatch.js';
  * @param { !string } projectName Jest project's name
  * @param { !Array<string> } testPathIgnorePatterns Expressions to match to ignored file paths by jest
  * @param { ?string } extension Test extension to match
- * @returns { !import("jest").Config } Jest config
+ * @returns { !import("@jest/types").Config.ProjectConfig } Jest config
  */
 function getJestJsProjectConfig(
   projectName,

diff --git a/packages/foundation/tools/jest-config/lib/config/getJestTsProjectConfig.js b/packages/foundation/tools/jest-config/lib/config/getJestTsProjectConfig.js
@@ -5,7 +5,7 @@ import getTestMatch from './getTestMatch.js';
  * @param { !string } projectName Jest project's name
  * @param { !Array<string> } testPathIgnorePatterns Expressions to match to ignored file paths by jest
  * @param { ?string } extension Test extension to match
- * @returns @returns { !import("jest").Config } Jest config
+ * @returns @returns { !import("@jest/types").Config.ProjectConfig } Jest config
  */
 function getJestTsProjectConfig(
   projectName,

diff --git a/packages/foundation/tools/scripts/package.json b/packages/foundation/tools/scripts/package.json
@@ -2,6 +2,9 @@
   "name": "@inversifyjs/foundation-scripts",
   "private": true,
   "description": "Common scripts for monorepo packages",
+  "devDependencies": {
+    "prettier": "3.4.2"
+  },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/inversify/monorepo.git"

diff --git a/packages/foundation/tools/stryker-config/package.json b/packages/foundation/tools/stryker-config/package.json
@@ -5,7 +5,8 @@
   },
   "description": "Stryker config for monorepo packages",
   "devDependencies": {
-    "@stryker-mutator/api": "8.7.1"
+    "@stryker-mutator/api": "8.7.1",
+    "prettier": "3.4.2"
   },
   "exports": "./lib/index.js",
   "homepage": "https://github.com/inversify/monorepo#readme",

diff --git a/packages/foundation/tools/typescript-config/package.json b/packages/foundation/tools/typescript-config/package.json
@@ -2,6 +2,9 @@
   "author": "Roberto Pintos López",
   "name": "@inversifyjs/foundation-typescript-config",
   "description": "Common typescript config for inversify monorepo packages",
+  "devDependencies": {
+    "prettier": "3.4.2"
+  },
   "license": "MIT",
   "bugs": {
     "url": "https://github.com/inversify/monorepo/issues"