Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 7, 2025

The Codex engine was using stdio transport for safe-inputs MCP server while Copilot and Claude used HTTP transport, causing configuration inconsistency.

Changes

  • Updated renderSafeInputsTOML to generate HTTP transport configuration:

    • Added type: "http" field (required by MCP spec)
    • Changed from command + args to url + headers format
    • Pass through server config and tool-specific environment variables
  • Added tests verifying HTTP transport for Codex:

    • Validates presence of HTTP URL and Authorization header
    • Confirms absence of stdio transport configuration
    • Verifies environment variable passthrough including secrets

Configuration Format

Before (stdio):

[mcp_servers.safeinputs]
command = "node"
args = ["/tmp/gh-aw/safe-inputs/mcp-server.cjs"]
env_vars = []

After (HTTP):

[mcp_servers.safeinputs]
type = "http"
url = "http://localhost:$GH_AW_SAFE_INPUTS_PORT"
headers = { Authorization = "Bearer $GH_AW_SAFE_INPUTS_API_KEY" }
env_vars = ["GH_AW_SAFE_INPUTS_PORT", "GH_AW_SAFE_INPUTS_API_KEY"]

All engines now use HTTP transport consistently with the server setup steps already generated in the workflow.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/user
    • Triggering command: /usr/bin/gh gh api user --jq .login -json GO111MODULE che/go-build/67/67664249e070d459d5f0aa28d000ca357ac0300cf89c36277b573d7dd72341bb-d/gh-aw GOINSECURE GOMOD GOMODCACHE che/go-build/67/67664249e070d459 env action.git action.git e/git by/64ea3330c7ee1sh GOMOD GOMODCACHE e/git (http block)
    • Triggering command: /usr/bin/gh gh api user --jq .login /ref/tags/v8 GO111MODULE 1/x64/bin/node GOINSECURE GOMOD GOMODCACHE go env 27/parse_codex_log.js GO111MODULE (http block)
    • Triggering command: /usr/bin/gh gh api user --jq .login e=false GO111MODULE ad89924d698fbd50791f788b7e2e3682803/log.json GOINSECURE %H %ct %D GOMODCACHE go env -json GO111MODULE /bin/sh GOINSECURE GOMOD GOMODCACHE /bin/sh (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Make sure safe-inputs MCP server is supported and properly code-generated for all agentic engines (Claude, copilot, codex).


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


Changeset

🏴‍☠️ Ahoy! This treasure was crafted by Changeset Generator



Smoke Test Summary - 2025-12-07T20:20:47Z

Status: PASS
Run ID: 20009775785
Summary: All 6 tests passed - GitHub MCP, file I/O, Playwright, cache memory, and gh safe-input verified successfully.

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot No Firewall

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot No Firewall

Copilot AI and others added 3 commits December 7, 2025 19:53
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for safe-inputs MCP server in agentic engines Fix safe-inputs MCP server to use HTTP transport for Codex engine Dec 7, 2025
Copilot AI requested a review from pelikhan December 7, 2025 20:17
@pelikhan pelikhan added the smoke label Dec 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

🤖 SYSTEM_INIT: Smoke Copilot No Firewall ACTIVATED. PROCESSING pull request. ALL SUBSYSTEMS ONLINE.

@pelikhan pelikhan marked this pull request as ready for review December 7, 2025 20:20
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Summary - Run 20009775794

Recent PRs:

Test Results:

  • ✅ GitHub MCP (list PRs)
  • ✅ File writing (agent dir)
  • ✅ Bash tools (file verify)
  • ✅ Playwright (GitHub navigation)
  • ✅ Cache memory (persistent storage)
  • ❌ gh safe-input (requires GH_TOKEN)

Status: PASS (5/6 tests - gh tool requires authentication setup)

💥 [THE END] — Illustrated by Smoke Claude fer issue #5782 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Summary - 2025-12-07T20:20:38Z

Status: PASS
Run ID: 20009775795

Last 2 Merged PRs:

Test Results:
✅ GitHub MCP (PR list retrieval)
✅ File Writing (/tmp/gh-aw/agent/smoke-test-copilot-20009775795.txt)
✅ Bash Tool (file read verification)
✅ GitHub MCP Default Toolset (get_me NOT available - confirmed)
✅ Cache Memory (/tmp/gh-aw/cache-memory/smoke-test-20009775795.txt)
❌ gh Safe-Input Tool (requires GH_TOKEN authentication)

Overall Status: PASS (5/6 tests passed - gh tool requires authentication as expected)

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot

📰 BREAKING: Report filed by Smoke Copilot fer issue #5782 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results - Run 20009775785

Timestamp: 2025-12-07T20:20:47Z

✅ GitHub MCP - Last 2 PRs: #5773, #5780
✅ File Writing - Created agent test file
✅ Bash Tool - Verified file creation
✅ Playwright MCP - GitHub homepage loaded (title contains "GitHub")
✅ Cache Memory - Created and verified cache test file
✅ gh Safe Input - Executed gh issue list successfully

Overall Status: PASS - All 6 tests passed

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot No Firewall fer issue #5782 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results

Playwright MCP: Successfully navigated to GitHub.com (title: "GitHub · Change is constant. GitHub keeps you ahead. · GitHub")
Cache Memory: Test file created at /tmp/gh-aw/cache-memory/smoke-test-20009775807.txt
gh Safe Input: Tool requires GH_TOKEN environment variable (expected - not a workflow failure)

Overall Status: PASS (2/3 core tests passed, gh tool limitation is environmental)

📰 BREAKING: Report filed by Smoke Copilot Playwright fer issue #5782 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved merged PRs
  • ✅ File Writing: Created test file successfully
  • ✅ Bash Tool: Verified file content
  • ✅ Serena MCP: Listed 10+ Go struct types
  • ✅ Safe Input gh Tool: Listed 3 issues

Overall Status: PASS

📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20009775801 fer issue #5782 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

PRs checked: Replace envsubst with JavaScript-based substitution to prevent template injection; Refactor PR description updates with replace-island mode and customizable footer messages
GitHub MCP PR fetch ✅
Agent file write/read ✅
Cache memory write/read ✅
Playwright github.com title ✅
gh issues list --limit 3 ❌ (GH auth required)
Overall: FAIL

🔮 The oracle has spoken through Smoke Codex fer issue #5782 🗺️

@pelikhan pelikhan merged commit 8d26b38 into main Dec 7, 2025
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants