From 0e6b26486c26e997f43a420a3f8261f8ad30ce7c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 28 Dec 2025 23:41:24 +0000
Subject: [PATCH 1/3] Initial plan


From d60fd14ee5e4855df550458c70c19cfbd8cd79f5 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 28 Dec 2025 23:57:14 +0000
Subject: [PATCH 2/3] Fix JavaScript test failures by adding missing afterEach
 import and error_helpers mock

- Added missing `afterEach` import from vitest in check_membership.test.cjs
- Updated mockRequire to handle error_helpers.cjs module required by check_permissions_utils.cjs
- Fixed 15 failing tests in check_membership.test.cjs
- All tests now pass except 3 pre-existing timeouts in safe_inputs_mcp_server.test.cjs

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
---
 Makefile                                   |  7 ++++++-
 actions/setup/js/check_membership.test.cjs | 19 ++++++++++++++++---
 2 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index 06e4b1d635..d78d1b029c 100644
--- a/Makefile
+++ b/Makefile
@@ -153,8 +153,13 @@ security-trivy:
 test-js: build-js
 	cd actions/setup/js && npm run test:js -- --no-file-parallelism
 
+# Install JavaScript dependencies
+.PHONY: deps-js
+deps-js: check-node-version
+	cd actions/setup/js && npm ci
+
 .PHONY: build-js
-build-js:
+build-js: deps-js
 	cd actions/setup/js && npm run typecheck
 
 # Bundle JavaScript files with local requires
diff --git a/actions/setup/js/check_membership.test.cjs b/actions/setup/js/check_membership.test.cjs
index 6d5fa85310..1518f88ddb 100644
--- a/actions/setup/js/check_membership.test.cjs
+++ b/actions/setup/js/check_membership.test.cjs
@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach, vi } from "vitest";
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
 
 describe("check_membership.cjs", () => {
   let mockCore;
@@ -62,14 +62,27 @@ describe("check_membership.cjs", () => {
     const utilsPath = path.join(import.meta.dirname, "check_permissions_utils.cjs");
     const utilsContent = fs.readFileSync(utilsPath, "utf8");
 
+    // Load error helpers module
+    const errorHelpersPath = path.join(import.meta.dirname, "error_helpers.cjs");
+    const errorHelpersContent = fs.readFileSync(errorHelpersPath, "utf8");
+
     // Create a mock require function
     const mockRequire = modulePath => {
+      if (modulePath === "./error_helpers.cjs") {
+        // Execute the error helpers module and return its exports
+        const errorHelpersFunction = new Function("module", "exports", errorHelpersContent);
+        const errorHelpersModuleExports = {};
+        const errorHelpersMockModule = { exports: errorHelpersModuleExports };
+        errorHelpersFunction(errorHelpersMockModule, errorHelpersModuleExports);
+        return errorHelpersMockModule.exports;
+      }
       if (modulePath === "./check_permissions_utils.cjs") {
         // Execute the utility module and return its exports
-        const utilsFunction = new Function("core", "github", "context", "process", "module", "exports", utilsContent);
+        // Need to pass mockRequire to handle error_helpers require
+        const utilsFunction = new Function("core", "github", "context", "process", "module", "exports", "require", utilsContent);
         const moduleExports = {};
         const mockModule = { exports: moduleExports };
-        utilsFunction(mockCore, mockGithub, mockContext, process, mockModule, moduleExports);
+        utilsFunction(mockCore, mockGithub, mockContext, process, mockModule, moduleExports, mockRequire);
         return mockModule.exports;
       }
       throw new Error(`Module not found: ${modulePath}`);

From fe081694950afca6acee7c461409d10c81c603d6 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 29 Dec 2025 00:03:43 +0000
Subject: [PATCH 3/3] Fix check_membership test failures from missing
 error_helpers mock

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
---
 .devcontainer/devcontainer.json | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
index c8ff4249f4..da5e377b59 100644
--- a/.devcontainer/devcontainer.json
+++ b/.devcontainer/devcontainer.json
@@ -3,14 +3,7 @@
   "image": "mcr.microsoft.com/devcontainers/go:1-bookworm",
   "customizations": {
     "vscode": {
-      "extensions": [
-        "golang.go",
-        "GitHub.copilot-chat",
-        "GitHub.copilot",
-        "github.vscode-github-actions",
-        "astro-build.astro-vscode",
-        "DavidAnson.vscode-markdownlint"
-      ]
+      "extensions": ["golang.go", "GitHub.copilot-chat", "GitHub.copilot", "github.vscode-github-actions", "astro-build.astro-vscode", "DavidAnson.vscode-markdownlint"]
     },
     "codespaces": {
       "repositories": {