From 9ca899ec1d229a6c8c9b0eca3859f4b2c7b41487 Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 14 Jul 2026 13:55:47 +0000 Subject: [PATCH] chore(js-client): upgrade to TypeScript 6 Bump typescript ^5.9.3 -> ^6.0.3 with a proper tsconfig migration so the workspace compiles cleanly under TS 6 without ignoreDeprecations: - Remove the deprecated `baseUrl` and rebase the `@leancodepl/pipe` path map to a relative specifier (TS resolves `paths` without `baseUrl` since 4.1). - Migrate `moduleResolution` from the deprecated `"node"` to `"bundler"` (the pipe lib is bundled via rollup/swc), matching js_corelibrary's org-preferred setting. `module` stays `ESNext`. Verified on Node 24.18: nx lint / build / test are green (the live integration test only 403s on the sandbox egress block) and tsc reports no TS5101 deprecation errors, with no ignoreDeprecations escape hatch. Stacked on chore/js-client-npm-dependencies-bump (#92). Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_012ET2dL5CiUkuFR1ZGiUrUH --- js-client/package-lock.json | 36 ++++++++++++++++++------------------ js-client/package.json | 2 +- js-client/tsconfig.base.json | 5 ++--- 3 files changed, 21 insertions(+), 22 deletions(-) diff --git a/js-client/package-lock.json b/js-client/package-lock.json index 7590505..7c70d77 100644 --- a/js-client/package-lock.json +++ b/js-client/package-lock.json @@ -48,7 +48,7 @@ "rollup": "^4.62.2", "ts-jest": "^29.4.11", "ts-node": "10.9.2", - "typescript": "^5.9.3", + "typescript": "^6.0.3", "typescript-eslint": "8.64.0", "verdaccio": "6.7.4" } @@ -5031,20 +5031,6 @@ } } }, - "node_modules/@nx/eslint/node_modules/typescript": { - "version": "6.0.3", - "resolved": "https://registry.npmjs.org/typescript/-/typescript-6.0.3.tgz", - "integrity": "sha512-y2TvuxSZPDyQakkFRPZHKFm+KKVqIisdg9/CZwm9ftvKXLP8NRWj38/ODjNbr43SsoXqNuAisEf1GdCxqWcdBw==", - "dev": true, - "license": "Apache-2.0", - "bin": { - "tsc": "bin/tsc", - "tsserver": "bin/tsserver" - }, - "engines": { - "node": ">=14.17" - } - }, "node_modules/@nx/jest": { "version": "23.1.0", "resolved": "https://registry.npmjs.org/@nx/jest/-/jest-23.1.0.tgz", @@ -18253,6 +18239,20 @@ "node": ">=6" } }, + "node_modules/lerna/node_modules/typescript": { + "version": "5.9.3", + "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.9.3.tgz", + "integrity": "sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw==", + "dev": true, + "license": "Apache-2.0", + "bin": { + "tsc": "bin/tsc", + "tsserver": "bin/tsserver" + }, + "engines": { + "node": ">=14.17" + } + }, "node_modules/lerna/node_modules/wrap-ansi": { "version": "7.0.0", "resolved": "https://registry.npmjs.org/wrap-ansi/-/wrap-ansi-7.0.0.tgz", @@ -24240,9 +24240,9 @@ "license": "MIT" }, "node_modules/typescript": { - "version": "5.9.3", - "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.9.3.tgz", - "integrity": "sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw==", + "version": "6.0.3", + "resolved": "https://registry.npmjs.org/typescript/-/typescript-6.0.3.tgz", + "integrity": "sha512-y2TvuxSZPDyQakkFRPZHKFm+KKVqIisdg9/CZwm9ftvKXLP8NRWj38/ODjNbr43SsoXqNuAisEf1GdCxqWcdBw==", "dev": true, "license": "Apache-2.0", "bin": { diff --git a/js-client/package.json b/js-client/package.json index 471984f..9810aef 100644 --- a/js-client/package.json +++ b/js-client/package.json @@ -40,7 +40,7 @@ "rollup": "^4.62.2", "ts-jest": "^29.4.11", "ts-node": "10.9.2", - "typescript": "^5.9.3", + "typescript": "^6.0.3", "typescript-eslint": "8.64.0", "verdaccio": "6.7.4" }, diff --git a/js-client/tsconfig.base.json b/js-client/tsconfig.base.json index 979490d..1c51d53 100644 --- a/js-client/tsconfig.base.json +++ b/js-client/tsconfig.base.json @@ -4,7 +4,7 @@ "rootDir": ".", "sourceMap": true, "declaration": false, - "moduleResolution": "node", + "moduleResolution": "bundler", "emitDecoratorMetadata": true, "experimentalDecorators": true, "importHelpers": true, @@ -13,9 +13,8 @@ "lib": ["ES2022", "dom"], "skipLibCheck": true, "skipDefaultLibCheck": true, - "baseUrl": ".", "paths": { - "@leancodepl/pipe": ["packages/pipe/src/index.ts"] + "@leancodepl/pipe": ["./packages/pipe/src/index.ts"] } } }