chore(SRI-NA): pin jest/ts-jest/typescript; relax service handler typing to fix build

Author: TRK95

package.json

@@ -3,7 +3,13 @@
   "version": "1.0.3",
   "license": "Apache-2.0",
   "description": "Scheme-based implementations of Source, written in Typescript",
-  "keywords": ["Scheme", "interpreter", "compiler", "Source", "SICP"],
+  "keywords": [
+    "Scheme",
+    "interpreter",
+    "compiler",
+    "Source",
+    "SICP"
+  ],
   "author": {
     "name": "Source Academy",
     "url": "https://github.com/source-academy/"
@@ -42,7 +48,7 @@
     "@rollup/plugin-commonjs": "^28.0.3",
     "@rollup/plugin-node-resolve": "^16.0.1",
     "@rollup/plugin-typescript": "^12.1.2",
-    "@types/jest": "^30.0.0",
+    "@types/jest": "^29.5.12",
     "@types/node": "^22.15.30",
     "@typescript-eslint/eslint-plugin": "^6.4.0",
     "babel-jest": "^30.0.2",
@@ -53,13 +59,13 @@
     "eslint-plugin-n": "^15.0.0 || ^16.0.0 ",
     "eslint-plugin-promise": "^6.6.0",
     "husky": "^9.1.7",
-    "jest": "^30.0.4",
+    "jest": "^29.7.0",
     "jest-util": "^30.0.5",
     "prettier": "^3.5.3",
     "rollup": "^4.38.0",
     "source-map": "^0.7.4",
-    "ts-jest": "^29.4.0",
+    "ts-jest": "^29.1.2",
     "tslib": "^2.8.1",
-    "typescript": "*"
+    "typescript": "^5.4.5"
   }
 }

src/conductor/host/BasicHostPlugin.ts

@@ -39,50 +39,43 @@ export abstract class BasicHostPlugin implements IHostPlugin {
 
   private __chunkCount: number = 0;
 
-  // @ts-expect-error TODO: figure proper way to typecheck this
   private readonly __serviceHandlers = new Map<
     ServiceMessageType,
-    (message: IServiceMessage) => void
+    (message: any) => void
   >([
     [
       ServiceMessageType.HELLO,
-      function helloServiceHandler(
-        this: BasicHostPlugin,
-        message: HelloServiceMessage
-      ) {
-        if (message.data.version < Constant.PROTOCOL_MIN_VERSION) {
+      (message: any) => {
+        const hello = message as HelloServiceMessage;
+        if (hello.data.version < Constant.PROTOCOL_MIN_VERSION) {
           this.__serviceChannel.send(
             new AbortServiceMessage(Constant.PROTOCOL_MIN_VERSION)
           );
           console.error(
-            `Runner's protocol version (${message.data.version}) must be at least ${Constant.PROTOCOL_MIN_VERSION}`
+            `Runner's protocol version (${hello.data.version}) must be at least ${Constant.PROTOCOL_MIN_VERSION}`
           );
         } else {
           console.log(
-            `Runner is using protocol version ${message.data.version}`
+            `Runner is using protocol version ${hello.data.version}`
           );
         }
       },
     ],
     [
       ServiceMessageType.ABORT,
-      function abortServiceHandler(
-        this: BasicHostPlugin,
-        message: AbortServiceMessage
-      ) {
+      (message: any) => {
+        const abort = message as AbortServiceMessage;
         console.error(
-          `Runner expects at least protocol version ${message.data.minVersion}, but we are on version ${Constant.PROTOCOL_VERSION}`
+          `Runner expects at least protocol version ${abort.data.minVersion}, but we are on version ${Constant.PROTOCOL_VERSION}`
         );
         this.__conduit.terminate();
       },
     ],
     [
       ServiceMessageType.PLUGIN,
-      function pluginServiceHandler(
-        this: BasicHostPlugin,
-        message: PluginServiceMessage
-      ) {
-        const pluginName = message.data;
+      (message: any) => {
+        const pluginMsg = message as PluginServiceMessage;
+        const pluginName = pluginMsg.data;
         this.requestLoadPlugin(pluginName);
       },
     ],

src/conductor/runner/RunnerPlugin.ts

@@ -57,48 +57,41 @@ export class RunnerPlugin implements IRunnerPlugin {
   private readonly __errorChannel: IChannel<IErrorMessage>;
   private readonly __statusChannel: IChannel<IStatusMessage>;
 
-  // @ts-expect-error TODO: figure proper way to typecheck this
   private readonly __serviceHandlers = new Map<
     ServiceMessageType,
-    (message: IServiceMessage) => void
+    (message: any) => void
   >([
     [
       ServiceMessageType.HELLO,
-      function helloServiceHandler(
-        this: RunnerPlugin,
-        message: HelloServiceMessage
-      ) {
-        if (message.data.version < Constant.PROTOCOL_MIN_VERSION) {
+      (message: any) => {
+        const hello = message as HelloServiceMessage;
+        if (hello.data.version < Constant.PROTOCOL_MIN_VERSION) {
           this.__serviceChannel.send(
             new AbortServiceMessage(Constant.PROTOCOL_MIN_VERSION)
           );
           console.error(
-            `Host's protocol version (${message.data.version}) must be at least ${Constant.PROTOCOL_MIN_VERSION}`
+            `Host's protocol version (${hello.data.version}) must be at least ${Constant.PROTOCOL_MIN_VERSION}`
           );
         } else {
-          console.log(`Host is using protocol version ${message.data.version}`);
+          console.log(`Host is using protocol version ${hello.data.version}`);
         }
       },
     ],
     [
       ServiceMessageType.ABORT,
-      function abortServiceHandler(
-        this: RunnerPlugin,
-        message: AbortServiceMessage
-      ) {
+      (message: any) => {
+        const abort = message as AbortServiceMessage;
         console.error(
-          `Host expects at least protocol version ${message.data.minVersion}, but we are on version ${Constant.PROTOCOL_VERSION}`
+          `Host expects at least protocol version ${abort.data.minVersion}, but we are on version ${Constant.PROTOCOL_VERSION}`
         );
         this.__conduit.terminate();
       },
     ],
     [
       ServiceMessageType.ENTRY,
-      function entryServiceHandler(
-        this: RunnerPlugin,
-        message: EntryServiceMessage
-      ) {
-        this.__evaluator.startEvaluator(message.data);
+      (message: any) => {
+        const entry = message as EntryServiceMessage;
+        this.__evaluator.startEvaluator(entry.data);
       },
     ],
   ]);