Skip to content

Fix safe-outputs server startup by copying tools.json to expected location#11129

Merged
pelikhan merged 3 commits intocopilot/convert-safe-outputs-to-html-transportfrom
copilot/update-server-sh-file-location
Jan 22, 2026
Merged

Fix safe-outputs server startup by copying tools.json to expected location#11129
pelikhan merged 3 commits intocopilot/convert-safe-outputs-to-html-transportfrom
copilot/update-server-sh-file-location

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

The safe-outputs MCP server failed to start because tools.json was not present at /opt/gh-aw/safeoutputs/tools.json where the server expects it.

Changes

  • actions/setup/setup.sh: Copy safe_outputs_tools.jsontools.json in safeoutputs directory, create empty config.json if missing
  • actions/setup/sh/start_safe_outputs_server.sh: Validate tools.json existence, make config.json optional (server uses defaults when absent)

The server's safe_outputs_tools_loader.cjs reads from GH_AW_SAFE_OUTPUTS_TOOLS_PATH (set to /opt/gh-aw/safeoutputs/tools.json) but setup was not copying the source file to this location.

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/21230742993/job/61088494029#step:19:1
review the reference workflow-run error and update server.sh to use the new file location.


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

Copilot AI and others added 2 commits January 22, 2026 00:24
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Copy safe_outputs_tools.json to /opt/gh-aw/safeoutputs/tools.json
- Create empty config.json in /opt/gh-aw/safeoutputs/ if it doesn't exist
- Update start_safe_outputs_server.sh to require tools.json
- Make config.json optional with informational message

Fixes workflow run error where tools.json was not found at expected location

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update server.sh to use new file location Fix safe-outputs server startup by copying tools.json to expected location Jan 22, 2026
Copilot AI requested a review from pelikhan January 22, 2026 00:38
@pelikhan pelikhan marked this pull request as ready for review January 22, 2026 00:40
@pelikhan pelikhan merged commit 8b8dd95 into copilot/convert-safe-outputs-to-html-transport Jan 22, 2026
@pelikhan pelikhan deleted the copilot/update-server-sh-file-location branch January 22, 2026 00:41
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