Upgrades dependencies to latest + changes required to support latest versions (#265)

* Upgrades dependencies to latest + changes required to support latest versions.

* Updates the build to use Node 22 to install and build, while testing for each supported nodejs version

* - Removes the automatic linting on test and adds it as a specific test in the cicd

* Updates nodejs versions supported.

* Removes reference to yarn in `package.json`

* adds important dependency that must be fixed in yarn.lock

* Updates dependencies

* Edits the yarn.lock file to update the dependency version for istanbul-lib-report. Else, it takes an old one and things break.

* sets an alpha version to avoid publishing issues

Author: etiennenoel

.eslintrc.yml

@@ -14,7 +14,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [8.x, 10.x, 12.x, 14.x, 16.x, 18.x, 20.x, 22.x]
+        node-version: [18.x, 20.x, 22.x]
 
     steps:
       - uses: actions/checkout@v4
@@ -28,10 +28,12 @@ jobs:
         run: |
           npm -g i yarn
 
-      - name: npm install, build, and test
+      - name: yarn install, yarn build, and yarn test
         run: |
           yarn
           yarn build
           yarn test
         env:
           CI: true
+
+

.github/workflows/build.yml

@@ -0,0 +1,65 @@
+import { defineConfig } from "eslint/config";
+import typescriptEslint from "@typescript-eslint/eslint-plugin";
+import prettier from "eslint-plugin-prettier";
+import globals from "globals";
+import tsParser from "@typescript-eslint/parser";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import js from "@eslint/js";
+import { FlatCompat } from "@eslint/eslintrc";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const compat = new FlatCompat({
+    baseDirectory: __dirname,
+    recommendedConfig: js.configs.recommended,
+    allConfig: js.configs.all
+});
+
+export default defineConfig([{
+    files: ["./src/**/*.js", "./src/**/*.jsx", "./src/**/*.ts", "./src/**/*.tsx"],
+    extends: compat.extends(
+        "plugin:@typescript-eslint/recommended",
+        "plugin:prettier/recommended",
+    ),
+
+    plugins: {
+        "@typescript-eslint": typescriptEslint,
+        prettier,
+    },
+
+    languageOptions: {
+        globals: {
+            ...globals.browser,
+            ...globals.commonjs,
+            Atomics: "readonly",
+            SharedArrayBuffer: "readonly",
+        },
+
+        parser: tsParser,
+        ecmaVersion: 2018,
+        sourceType: "script",
+    },
+
+    rules: {
+        "@typescript-eslint/explicit-function-return-type": ["warn", {
+            allowExpressions: true,
+        }],
+
+        "@typescript-eslint/explicit-member-accessibility": "off",
+        "@typescript-eslint/ban-types": "off",
+        "@typescript-eslint/no-angle-bracket-type-assertion": "off",
+        "@typescript-eslint/no-explicit-any": "off",
+        "@typescript-eslint/no-non-null-assertion": "off",
+        "@typescript-eslint/no-parameter-properties": "off",
+        "@typescript-eslint/prefer-interface": "off",
+        "@typescript-eslint/indent": "off",
+    },
+}, {
+    files: ["**/*.test.ts", "**/*.spec.ts"],
+
+    rules: {
+        "@typescript-eslint/no-empty-function": "off",
+        "@typescript-eslint/interface-name-prefix": "off",
+    },
+}]);

eslint.config.mjs

@@ -1,23 +1,25 @@
 {
   "name": "tsyringe",
-  "version": "4.9.1",
+  "version": "5.0.0-alpha.1",
   "description": "Lightweight dependency injection container for JavaScript/TypeScript",
   "main": "dist/cjs/index.js",
   "module": "./dist/esm5/index.js",
   "es2015": "./dist/esm2015/index.js",
   "typings": "./dist/typings/index.d.ts",
   "scripts": {
-    "build": "yarn clean && yarn build:cjs && yarn build:es5 && yarn build:es2015 && yarn build:types",
+    "build": "yarn clean && yarn build:cjs && yarn build:es5 && yarn build:es2015 && yarn build:es2020 && yarn build:esnext && yarn build:types",
     "build:cjs": "tsc",
     "build:es5": "tsc -p ./typescript/tsconfig.esm5.json",
     "build:es2015": "tsc -p ./typescript/tsconfig.esm2015.json",
+    "build:es2020": "tsc -p ./typescript/tsconfig.esm2020.json",
+    "build:esnext": "tsc -p ./typescript/tsconfig.esmnext.json",
     "build:types": "tsc -p ./typescript/tsconfig.types.json",
     "clean": "rimraf ./dist",
     "test": "yarn lint && jest --config test/jest.config.js",
     "test:inspect": "yarn lint && node --inspect-brk ./node_modules/jest/bin/jest.js --runInBand --config test/jest.config.js",
     "test:coverage": "jest --config test/jest.config.js --coverage",
-    "lint": "eslint --ext \".js,.jsx,.ts,.tsx\" \"./src\"",
-    "lint:fix": "eslint --fix --ext \".js,.jsx,.ts,.tsx\" \"./src\""
+    "lint": "eslint",
+    "lint:fix": "eslint --fix"
   },
   "repository": {
     "type": "git",
@@ -33,7 +35,7 @@
     "typescript"
   ],
   "engines": {
-    "node": ">= 6.0.0"
+    "node": ">= 18.0.0"
   },
   "author": "Steven Hobson-Campbell",
   "license": "MIT",
@@ -42,22 +44,23 @@
   },
   "homepage": "https://github.com/Microsoft/tsyringe#readme",
   "dependencies": {
-    "tslib": "^1.9.3"
+    "tslib": "^2.8.1"
   },
   "devDependencies": {
-    "@types/jest": "^24.0.21",
-    "@types/node": "^8.10.16",
-    "@typescript-eslint/eslint-plugin": "^2.6.0",
-    "@typescript-eslint/parser": "^2.6.0",
-    "eslint": "^6.6.0",
-    "eslint-config-prettier": "^6.5.0",
-    "eslint-plugin-prettier": "^3.1.1",
-    "husky": "^3.0.0",
-    "jest": "^24.7.1",
-    "prettier": "1.18.2",
-    "reflect-metadata": "^0.1.12",
-    "rimraf": "^3.0.0",
-    "ts-jest": "^24.0.2",
-    "typescript": "^3.1.6"
+    "@types/istanbul-lib-report": "^3.0.3",
+    "@types/jest": "^29.5.14",
+    "@types/node": "^22.14.0",
+    "@typescript-eslint/eslint-plugin": "^8.29.0",
+    "@typescript-eslint/parser": "^8.29.0",
+    "eslint": "^9.24.0",
+    "eslint-config-prettier": "^10.1.1",
+    "eslint-plugin-prettier": "^5.2.6",
+    "husky": "^9.1.7",
+    "jest": "^29.7.0",
+    "prettier": "^3.5.3",
+    "reflect-metadata": "^0.2.2",
+    "rimraf": "^5.0.10",
+    "ts-jest": "^29.3.1",
+    "typescript": "^5.8.3"
   }
 }

package.json

@@ -36,7 +36,10 @@ test("@autoInjectable injects parameters beyond those specified manually", () =>
   class FooBar {}
   @autoInjectable()
   class Foo {
-    constructor(public myFooBar: FooBar, public myBar?: Bar) {}
+    constructor(
+      public myFooBar: FooBar,
+      public myBar?: Bar
+    ) {}
   }
 
   const myFooBar = new FooBar();

src/__tests__/auto-injectable.test.ts

@@ -1,12 +1,11 @@
-/* eslint-disable @typescript-eslint/interface-name-prefix, @typescript-eslint/no-empty-interface */
-
 import {instance as globalContainer} from "../dependency-container";
 
 afterEach(() => {
   globalContainer.reset();
 });
 
 test("child container resolves even when parent doesn't have registration", () => {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   interface IFoo {}
   class Foo implements IFoo {}
 
@@ -19,6 +18,7 @@ test("child container resolves even when parent doesn't have registration", () =
 });
 
 test("child container resolves using parent's registration when child container doesn't have registration", () => {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   interface IFoo {}
   class Foo implements IFoo {}
 
@@ -31,6 +31,7 @@ test("child container resolves using parent's registration when child container
 });
 
 test("child container resolves all even when parent doesn't have registration", () => {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   interface IFoo {}
   class Foo implements IFoo {}
 
@@ -45,6 +46,7 @@ test("child container resolves all even when parent doesn't have registration",
 });
 
 test("child container resolves all using parent's registration when child container doesn't have registration", () => {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   interface IFoo {}
   class Foo implements IFoo {}
 

src/__tests__/child-container.test.ts

@@ -10,6 +10,7 @@ describe("Disposable", () => {
 
     it("returns false when dispose method takes too many args", () => {
       const specialDisposable = {
+        // eslint-disable-next-line @typescript-eslint/no-unused-vars
         dispose(_: any) {}
       };
 

src/__tests__/disposable.test.ts

@@ -21,7 +21,10 @@ test("Error message composition", () => {
 
   @injectable()
   class A {
-    constructor(public d: Ok, public b: B) {}
+    constructor(
+      public d: Ok,
+      public b: B
+    ) {}
   }
 
   expect(() => {

src/__tests__/errors.test.ts

@@ -1,5 +1,3 @@
-/* eslint-disable @typescript-eslint/interface-name-prefix */
-
 import {inject, injectable, registry, singleton} from "../decorators";
 import {
   instanceCachingFactory,
@@ -558,7 +556,11 @@ test("dependencies of an @singleton can be resolved", () => {
 test("passes through the given params", () => {
   @injectable()
   class MyViewModel {
-    constructor(public a: any, public b: any, public c: any) {}
+    constructor(
+      public a: any,
+      public b: any,
+      public c: any
+    ) {}
   }
 
   const a = {};
@@ -936,7 +938,7 @@ describe("dispose", () => {
   class Baz implements Disposable {
     disposed = false;
     async dispose(): Promise<void> {
-      return new Promise(resolve => {
+      return new Promise((resolve) => {
         process.nextTick(() => {
           this.disposed = true;
           resolve();

src/__tests__/global-container.test.ts

@@ -15,7 +15,7 @@ test("DelayedConstructor delays creation until first usage", () => {
   }
   const delayedConstructor = delay(() => Foo);
   expect(delayedConstructor).toBeInstanceOf(DelayedConstructor);
-  const foo: Foo = delayedConstructor.createProxy(Target => new Target());
+  const foo: Foo = delayedConstructor.createProxy((Target) => new Target());
   expect(created).toBe(false);
   expect(foo).toBeInstanceOf(Foo);
   expect(created).toBe(true);