Skip to content

[WIP] Register safe output MCP server as Node.js runtime - #5942

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/register-safe-output-mcp-server
Closed

[WIP] Register safe output MCP server as Node.js runtime#5942
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/register-safe-output-mcp-server

Conversation

Copilot AI commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Fix safe-outputs MCP server runtime mode and bundling

Problem

The safe-outputs MCP server JavaScript is executed as a standalone Node.js process but:

  • Registered as RuntimeModeGitHubScript instead of RuntimeModeNodeJS
  • Has incorrect /// <reference types="@actions/github-script" /> TypeScript reference
  • Being bundled with GitHub Script mode validations instead of Node.js mode

Plan

  • Analyze current state and identify issues
  • Remove incorrect TypeScript reference from safe_outputs_mcp_server.cjs
  • Change registration from RuntimeModeGitHubScript to RuntimeModeNodeJS
  • Add API to request bundled vs unbundled version of scripts
  • Update GetSafeOutputsMCPServerScript to use RuntimeModeNodeJS
  • Verify no mixing of JavaScript from different runtimes
  • Run tests to ensure changes work correctly
  • Run make agent-finish for final validation
Original prompt

The safe output MCP server JavaScript is a bundled JavaScript using node JS runtime.

  • make sure to register safe output MCP server as nodejs runtime
  • make sure JavaScript from different runtimes are not mixed
  • have an api to request a bundle version of a program (vs all files) and use it for the safe output server/

DO NOT CHANGE file locations!


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants