Skip to content

Commit cee4f9c

Browse files
dsymeCopilot
andauthored
fix: restore MCP CLI concept explanation and fix safeoutputs framing (#39689)
* fix: restore MCP CLI concept explanation and fix safeoutputs framing (#39684) - Restore concept explanation to mcp_cli_tools_prompt.md that was stripped by PR #36778: servers are CLI executables on PATH, not MCP tools; include usage example and JSON stdin mode - Fix safe_outputs_prompt.md line 2: 'safeoutputs MCP server tools' → 'safeoutputs CLI tool' to match the CLI framing - Update prompts.go comment to match * fix: use RUNNER_TEMP path for safe_outputs_mcp_server.cjs in gateway config PR #39100 introduced the containerized stdio transport for safeoutputs but hardcoded ${GITHUB_WORKSPACE}/actions/setup/js/safe_outputs_mcp_server.cjs as the entrypoint. This path only exists in the gh-aw repo's own workspace. For consumer repos (e.g. github-automation), GITHUB_WORKSPACE is that repo's checkout which does not contain actions/setup/js/. The gateway container gets MODULE_NOT_FOUND, crashes with EOF on initialize, and registers 0 tools. The agent then spends ~30 minutes trying to discover the tool schema before failing. Fix: use ${RUNNER_TEMP}/gh-aw/safeoutputs/safe_outputs_mcp_server.cjs — setup.sh already copies the file there and that directory is mounted rw. Recompile all 249 lock files. Observed in: https://github.com/github/github-automation/actions/runs/27647012218 * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: update integration tests for RUNNER_TEMP safeoutputs path * commit tweaks * fix: update remaining tests for RUNNER_TEMP safeoutputs path --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 5ad2873 commit cee4f9c

255 files changed

Lines changed: 819 additions & 800 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/ab-testing-advisor.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/ace-editor.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/agent-performance-analyzer.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/agent-persona-explorer.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/agentic-token-audit.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/agentic-token-optimizer.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/agentic-token-trend-audit.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/ai-moderator.lock.yml

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/api-consumption-report.lock.yml

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)