upgrade to TS 6.0 initial attempt

Perryvw · Perryvw · commit 590e0713248d · 2026-03-29T19:46:42.000+02:00
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -31,7 +31,7 @@
         "lint:prettier": "prettier --check . || (echo 'Run `npm run fix:prettier` to fix it.' && exit 1)",
         "lint:eslint": "eslint .",
         "fix:prettier": "prettier --write .",
-        "check:language-extensions": "tsc --strict language-extensions/index.d.ts",
+        "check:language-extensions": "tsc --strict language-extensions/index.d.ts --ignoreConfig",
         "preversion": "npm run build && npm test",
         "postversion": "git push && git push --tags"
     },
@@ -42,20 +42,20 @@
         "node": ">=16.10.0"
     },
     "peerDependencies": {
-        "typescript": "5.9.3"
+        "typescript": "6.0.2"
     },
     "dependencies": {
         "@typescript-to-lua/language-extensions": "1.19.0",
         "enhanced-resolve": "5.8.2",
-        "picomatch": "^2.3.1",
+        "picomatch": "^2.3.2",
         "resolve": "^1.15.1",
         "source-map": "^0.7.3"
     },
     "devDependencies": {
         "@types/fs-extra": "^8.1.0",
         "@types/glob": "^7.2.0",
         "@types/jest": "^27.5.2",
-        "@types/node": "^22.10.0",
+        "@types/node": "^22.19.15",
         "@types/picomatch": "^2.3.0",
         "@types/resolve": "1.14.0",
         "eslint": "^9.22.0",
@@ -69,7 +69,7 @@
         "prettier": "^2.8.8",
         "ts-jest": "^29.2.5",
         "ts-node": "^10.9.2",
-        "typescript": "5.9.3",
+        "typescript": "6.0.2",
         "typescript-eslint": "^8.46.3"
     }
 }
diff --git a/src/transpilation/resolve.ts b/src/transpilation/resolve.ts
@@ -27,7 +27,7 @@ interface ResolutionResult {
 }
 
 class ResolutionContext {
-    private noResolvePaths: picomatch.Matcher[];
+    private noResolvePaths: RegExp[];
 
     public diagnostics: ts.Diagnostic[] = [];
     public resolvedFiles = new Map<string, ProcessedFile>();
@@ -39,7 +39,7 @@ class ResolutionContext {
         private readonly plugins: Plugin[]
     ) {
         const unique = [...new Set(options.noResolvePaths)];
-        const matchers = unique.map(x => picomatch(x));
+        const matchers = unique.map(x => picomatch.makeRe(x));
         this.noResolvePaths = matchers;
     }
 
@@ -73,7 +73,7 @@ class ResolutionContext {
             return;
         }
 
-        if (this.noResolvePaths.find(isMatch => isMatch(required.requirePath))) {
+        if (this.noResolvePaths.find(isMatch => isMatch.test(required.requirePath))) {
             if (this.options.tstlVerbose) {
                 console.log(
                     `Skipping module resolution of ${required.requirePath} as it is in the tsconfig noResolvePaths.`
diff --git a/src/utils.ts b/src/utils.ts
@@ -77,7 +77,7 @@ export function cast<TOriginal, TCast extends TOriginal>(
 }
 
 export function assert(value: any, message?: string | Error): asserts value {
-    nativeAssert(value, message);
+    nativeAssert.ok(value, message);
 }
 
 export function assertNever(_value: never): never {
diff --git a/test/setup.ts b/test/setup.ts
@@ -13,7 +13,7 @@ declare global {
 
 expect.extend({
     toHaveDiagnostics(diagnostics: ts.Diagnostic[], expected?: number[]): jest.CustomMatcherResult {
-        assert(Array.isArray(diagnostics));
+        assert.ok(Array.isArray(diagnostics));
         // @ts-ignore
         const matcherHint = this.utils.matcherHint("toHaveDiagnostics", undefined, "", this);
 
diff --git a/test/transpile/module-resolution.spec.ts b/test/transpile/module-resolution.spec.ts
@@ -598,7 +598,7 @@ test("module resolution uses baseURL to resolve imported files", () => {
                 return { baz = function() return "baz" end }
             `
         )
-        .setOptions({ baseUrl: "./myproject/mydeps" })
+        .setOptions({ baseUrl: "./myproject/mydeps", ignoreDeprecations: "6.0" })
         .expectToEqual({
             fooResult: "foo",
             barResult: "bar",
diff --git a/test/unit/optionalChaining.spec.ts b/test/unit/optionalChaining.spec.ts
@@ -342,7 +342,6 @@ describe("optional chaining function calls", () => {
         `
             .setOptions({
                 strict,
-                target: ScriptTarget.ES5,
             })
             .expectToMatchJsResult();
     });
diff --git a/test/util.ts b/test/util.ts
@@ -51,7 +51,7 @@ function getLuaBindingsForVersion(target: tstl.LuaTarget): { lauxlib: LauxLib; l
 }
 
 export function assert(value: any, message?: string | Error): asserts value {
-    nativeAssert(value, message);
+    nativeAssert.ok(value, message);
 }
 
 export const formatCode = (...values: unknown[]) => values.map(e => stringify(e)).join(", ");
@@ -92,17 +92,15 @@ export function expectEachVersionExceptJit<T>(
     };
 }
 
-const memoize: MethodDecorator = (_target, _propertyKey, descriptor) => {
-    const originalFunction = descriptor.value as any;
+const memoize = (originalFunction: any) => {
     const memoized = new WeakMap();
-    descriptor.value = function (this: any, ...args: any[]): any {
+    return function (this: any, ...args: any[]): any {
         if (!memoized.has(this)) {
             memoized.set(this, originalFunction.apply(this, args));
         }
 
         return memoized.get(this);
-    } as any;
-    return descriptor;
+    };
 };
 
 export class ExecutionError extends Error {
@@ -158,12 +156,13 @@ export abstract class TestBuilder {
     }
 
     protected options: tstl.CompilerOptions = {
+        strict: false,
         luaTarget: tstl.LuaTarget.Lua55,
         noHeader: true,
         skipLibCheck: true,
         target: ts.ScriptTarget.ES2017,
         lib: ["lib.esnext.d.ts"],
-        moduleResolution: ts.ModuleResolutionKind.Node10,
+        moduleResolution: ts.ModuleResolutionKind.Bundler,
         resolveJsonModule: true,
         sourceMap: true,
     };
diff --git a/tsconfig.json b/tsconfig.json
@@ -4,7 +4,6 @@
         "lib": ["es2019"],
         "types": ["node"],
         "module": "commonjs",
-        "experimentalDecorators": true,
 
         "rootDir": "src",
         "outDir": "dist",

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ export function cast<TOriginal, TCast extends TOriginal>(`
`77`	`77`	`}`
`78`	`78`
`79`	`79`	`export function assert(value: any, message?: string \| Error): asserts value {`
`80`		`- nativeAssert(value, message);`
	`80`	`+ nativeAssert.ok(value, message);`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`export function assertNever(_value: never): never {`
Original file line number	Diff line number	Diff line change
`@@ -598,7 +598,7 @@ test("module resolution uses baseURL to resolve imported files", () => {`
`598`	`598`	`return { baz = function() return "baz" end }`
`599`	`599`	`
`600`	`600`	`)`
`601`		`- .setOptions({ baseUrl: "./myproject/mydeps" })`
	`601`	`+ .setOptions({ baseUrl: "./myproject/mydeps", ignoreDeprecations: "6.0" })`
`602`	`602`	`.expectToEqual({`
`603`	`603`	`fooResult: "foo",`
`604`	`604`	`barResult: "bar",`
Original file line number	Diff line number	Diff line change
`@@ -342,7 +342,6 @@ describe("optional chaining function calls", () => {`
`342`	`342`	`
`343`	`343`	`.setOptions({`
`344`	`344`	`strict,`
`345`		`- target: ScriptTarget.ES5,`
`346`	`345`	`})`
`347`	`346`	`.expectToMatchJsResult();`
`348`	`347`	`});`