chore: upgrade TypeScript from 5.6.3 to 6.0.3

- Update typescript to ^6.0.3 and @types/node to ^24.12.2
- Change tsconfig module/moduleResolution from esnext/node to nodenext
- Fix catch clause variables typed as unknown (stricter in TS 6)
- Replace @ts-ignore with @ts-expect-error
- Fix definite assignment assertion for Logger.result
- Fix view-engine type compatibility with express

Author: marcoscaceres

package.json

@@ -53,10 +53,10 @@
     "@types/cors": "^2.8.19",
     "@types/express": "^5.0.6",
     "@types/morgan": "^1.9.10",
-    "@types/node": "^22.9.0",
+    "@types/node": "^24.12.2",
     "@types/split2": "^4.2.3",
     "jasmine": "^6.2.0",
     "simple-git-hooks": "^2.13.1",
-    "typescript": "^5.6.3"
+    "typescript": "^6.0.3"
   }
 }

pnpm-lock.yaml

@@ -24,10 +24,11 @@ export default async function route(req: IRequest, res: Response) {
     }
     res.json({ result });
   } catch (error) {
-    const errorCode = error.message === "INTERNAL_ERROR" ? 500 : 404;
+    const message = error instanceof Error ? error.message : String(error);
+    const errorCode = message === "INTERNAL_ERROR" ? 500 : 404;
     res.status(errorCode);
     res.setHeader("Content-Type", "text/plain");
-    res.send(error.message);
+    res.send(message);
   }
 }
 

routes/caniuse/feature.ts

@@ -5,7 +5,7 @@ import { cache } from "./lib/index.js";
 import { Request, Response } from "express";
 
 const workerFile = path.join(import.meta.dirname, "update.worker.js");
-const taskQueue = new BackgroundTaskQueue<typeof import("./update.worker")>(
+const taskQueue = new BackgroundTaskQueue<typeof import("./update.worker.ts")>(
   workerFile,
   "caniuse_update",
 );

routes/caniuse/update.ts

@@ -14,7 +14,8 @@ export default async function route(_req: Request, res: Response) {
     console.log(`Successfully regenerated docs in ${Date.now() - start}ms.`);
     res.sendStatus(200); // ok
   } catch (error) {
-    const { message = "", statusCode = 500 } = error;
+    const message = error instanceof Error ? error.message : String(error);
+    const statusCode = error instanceof HTTPError ? error.statusCode : 500;
     console.error(`Failed to regenerate docs: ${message.slice(0, 400)}...`);
     res.status(statusCode);
     res.send(message);

routes/docs/update.ts

@@ -37,6 +37,7 @@ export default async function route(req: IRequest, res: Response) {
     res.json(commits);
   } catch (error) {
     res.set("Content-Type", "text/plain");
-    res.status(404).send(error.message);
+    const message = error instanceof Error ? error.message : String(error);
+    res.status(404).send(message);
   }
 }

routes/github/commits.ts

@@ -41,7 +41,7 @@ export default async function route(req: IRequest, res: Response) {
       contributors.push(contributor);
     }
   } catch (err) {
-    if (err.message && err.message.includes("404 Not Found")) {
+    if (err instanceof Error && err.message.includes("404 Not Found")) {
       await cache.set(cacheKey, null);
       return res.sendStatus(404);
     } else {

routes/github/contributors.ts

@@ -17,10 +17,11 @@ export default async function route(req: IRequest, res: Response) {
     res.set("Cache-Control", `max-age=${seconds("30m")}`);
     res.json({ entries });
   } catch (error) {
-    const errorCode = error.message === "INTERNAL_ERROR" ? 500 : 404;
+    const message = error instanceof Error ? error.message : String(error);
+    const errorCode = message === "INTERNAL_ERROR" ? 500 : 404;
     res.status(errorCode);
     res.setHeader("Content-Type", "text/plain");
-    res.send(error.message);
+    res.send(message);
   }
 }
 

routes/github/files.ts

@@ -3,7 +3,7 @@ import { mkdir, readFile } from "node:fs/promises";
 
 import type { Request, Response } from "express";
 
-import { env } from "../../../utils/misc.js";
+import { env, HTTPError } from "../../../utils/misc.js";
 import sh from "../../../utils/sh.js";
 
 export const PKG_DIR = path.join(env("DATA_DIR"), "respec", "package");
@@ -25,7 +25,8 @@ export default async function route(req: Request, res: Response) {
     await pullRelease();
     res.sendStatus(200); // ok
   } catch (error) {
-    const { message = "", statusCode = 500 } = error;
+    const message = error instanceof Error ? error.message : String(error);
+    const statusCode = error instanceof HTTPError ? error.statusCode : 500;
     console.error(`Failed to pull respec release: ${message.slice(0, 400)}...`);
     res.status(statusCode);
     res.send(message);

routes/respec/builds/update.ts

@@ -65,7 +65,8 @@ export default async function route(req: IRequest, res: Response) {
     res.set("Cache-Control", `max-age=${seconds("24h")}`);
     res.json(groupInfo);
   } catch (error) {
-    const { statusCode = 500, message } = error;
+    const statusCode = error instanceof HTTPError ? error.statusCode : 500;
+    const message = error instanceof Error ? error.message : String(error);
     res.set("Content-Type", "text/plain");
     res.status(statusCode).send(message);
   }
@@ -118,8 +119,8 @@ async function fetchGroupInfo(
     var json = (await res.json()) as APIResponse;
   } catch (error) {
     throw new HTTPError(
-      error.statusCode || 500,
-      error.message
+      error instanceof HTTPError ? error.statusCode : 500,
+      error instanceof Error ? error.message : String(error),
     );
   }