Skip to content

Commit ae25f9f

Browse files
authored
Merge pull request #1602 from finos/bump-typescript-to-6
chore: bump typescript to 6.0.3 and typescript-eslint to 8.61.1
2 parents 7eeab39 + 7f31c97 commit ae25f9f

10 files changed

Lines changed: 406 additions & 436 deletions

File tree

experimental/li-cli/package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

experimental/li-cli/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"ts-node": "^10.9.2",
3030
"tsc-alias": "^1.8.16",
3131
"tslib": "^2.8.1",
32-
"typescript": "^5.9.3"
32+
"typescript": "^6.0.3"
3333
}
3434
}

package-lock.json

Lines changed: 373 additions & 421 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,8 @@
202202
"tailwindcss": "^4.2.2",
203203
"ts-node": "^10.9.2",
204204
"tsx": "^4.21.0",
205-
"typescript": "^5.9.3",
206-
"typescript-eslint": "^8.57.2",
205+
"typescript": "^6.0.3",
206+
"typescript-eslint": "^8.61.1",
207207
"vite": "^8.0.14",
208208
"vite-tsconfig-paths": "^5.1.4",
209209
"vitest": "^3.2.7"

src/config/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,7 @@ const handleConfigUpdate = async (newConfig: Configuration) => {
524524
const validatedConfig = Convert.toGitProxyConfig(JSON.stringify(newConfig));
525525

526526
// 2. Get proxy module dynamically to avoid circular dependency
527-
const proxy = (await import('../proxy')) as any;
527+
const proxy = (await import('../proxy/index.js')) as any;
528528

529529
// 3. Stop existing services
530530
await proxy.stop();
@@ -541,7 +541,7 @@ const handleConfigUpdate = async (newConfig: Configuration) => {
541541
handleErrorAndLog(error, 'Failed to apply new configuration');
542542
// Attempt to restart with previous config
543543
try {
544-
const proxy = (await import('../proxy')) as any;
544+
const proxy = (await import('../proxy/index.js')) as any;
545545
await proxy.start();
546546
} catch (startError: unknown) {
547547
handleErrorAndLog(startError, 'Failed to restart services');

src/service/passport/local.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ const isKnownDefaultCredentialAttempt = (username: string, password: string): bo
4848

4949
// Dynamic import to always get the current db module instance
5050
// This is necessary for test environments where modules may be reset
51-
const getDb = () => import('../../db');
51+
const getDb = () => import('../../db/index.js');
5252

5353
export const configure = async (passport: PassportStatic): Promise<PassportStatic> => {
5454
passport.use(

src/service/passport/oidc.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@
1717
import * as db from '../../db';
1818
import { PassportStatic } from 'passport';
1919
import { getAuthMethods } from '../../config';
20-
import { type UserInfoResponse } from 'openid-client';
20+
import type { UserInfoResponse } from 'openid-client';
21+
import type { Request } from 'express';
2122
import { handleErrorAndLog } from '../../utils/errors';
2223

2324
export const type = 'openidconnect';
2425

2526
export const configure = async (passport: PassportStatic): Promise<PassportStatic> => {
2627
// Use dynamic imports to avoid ESM/CommonJS issues
2728
const { discovery, fetchUserInfo } = await import('openid-client');
28-
// @ts-expect-error - throws error due to missing type definitions
2929
const { Strategy } = await import('openid-client/passport');
3030

3131
const authMethods = getAuthMethods();

src/types/globals.d.ts

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
/**
2+
* Copyright 2026 GitProxy Contributors
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
declare module '*.css';

test/fixtures/test-package/package-lock.json

Lines changed: 5 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tsconfig.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,11 @@
55
"allowJs": true,
66
"checkJs": false,
77
"jsx": "react-jsx",
8-
"moduleResolution": "Node",
8+
"moduleResolution": "nodenext",
99
"strict": true,
1010
"skipLibCheck": true,
1111
"isolatedModules": false,
12-
"module": "CommonJS",
12+
"module": "NodeNext",
1313
"esModuleInterop": true,
1414
"allowSyntheticDefaultImports": true,
1515
"resolveJsonModule": true,
@@ -20,7 +20,6 @@
2020
"rootDir": "./src",
2121
"noEmit": false,
2222
"types": ["node"],
23-
"baseUrl": ".",
2423
"paths": {
2524
"@primer/react/experimental": ["./node_modules/@primer/react/dist/experimental/index"]
2625
}

0 commit comments

Comments
 (0)