Skip to content

Convert safe-outputs MCP server to HTTP transport #11120

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
Copilot wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Convert safe-outputs MCP server to HTTP transport #11120

Copilot wants to merge 8 commits into from
+5,497 −4,785

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026 •
edited
Loading

Migrates safe-outputs from stdio to HTTP transport, matching the safe-inputs pattern. The server now starts as a background HTTP service before the agent job, eliminating container overhead and improving connection reliability.

Changes

HTTP Server Implementation

  • Created safe_outputs_mcp_server_http.cjs with stateful HTTP transport
  • Entry point updated to spawn HTTP server on port 3001
  • Shell script start_safe_outputs_server.sh handles server lifecycle

Workflow Generation

  • Added safe-outputs-config step: generates API key + port
  • Added safe-outputs-start step: launches HTTP server before agent
  • MCP configuration now renders HTTP transport instead of stdio: 
    safeoutputs:
  type: http
  url: http://host.docker.internal:$GH_AW_SAFE_OUTPUTS_PORT
  headers:
    Authorization: $GH_AW_SAFE_OUTPUTS_API_KEY

Removed

  • Stdio configuration fields: container, entrypoint, entrypointArgs, mounts
  • Environment variable passing through containerized stdio

Environment Variables

  • GH_AW_SAFE_OUTPUTS_PORT and GH_AW_SAFE_OUTPUTS_API_KEY sourced from step outputs
  • Added to MCP gateway environment for URL template resolution

All engine implementations (Copilot, Claude, Codex, Custom) updated to pass workflowData for host resolution (localhost vs host.docker.internal based on agent configuration).

Original prompt

Convert the safe-outputs mcp server to an HTML transport (see safe-inputs) and update the generated steps to start the safe-outputs server before the agent. Then update the generated mcp server configuration to use the http server. See how the safe-inputs is implemented in the compiler and javascript.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@pelikhan
Convert safe-outputs to HTTP transport - Phase 1-4 complete
a7343da 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan mentioned this pull request Jan 22, 2026
Open
Copilot AI and others added 2 commits January 22, 2026 00:03
@pelikhan
Fix test expectations for HTTP transport
86315e3 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
Fix nil pointer dereference in safe-outputs env check
eb0862b 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Convert safe-outputs mcp server to HTML transport Convert safe-outputs MCP server to HTTP transport Jan 22, 2026
Copilot AI requested a review from pelikhan January 22, 2026 00:14
@pelikhan pelikhan added the smoke label Jan 22, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited by agentic-workflows-dev bot
Loading

💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@pelikhan pelikhan added smoke and removed smoke labels Jan 22, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited by agentic-workflows-dev bot
Loading

💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@pelikhan pelikhan added smoke and removed smoke labels Jan 22, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited by agentic-workflows-dev bot
Loading

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@pelikhan pelikhan added smoke and removed smoke labels Jan 22, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited
Loading

🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026 •
edited by agentic-workflows-dev bot
Loading

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

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

Reviewers

@pelikhan pelikhan Awaiting requested review from pelikhan

Assignees

@pelikhan pelikhan

Copilot code review Copilot

Labels

smoke

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pelikhan