Skip to content

screenshot-uploads fixing#1773

Merged
pelikhan merged 2 commits intomainfrom
screenshots-mcp
Oct 16, 2025
Merged

screenshot-uploads fixing#1773
pelikhan merged 2 commits intomainfrom
screenshots-mcp

Conversation

@pelikhan
Copy link
Copy Markdown
Collaborator

Fixing screenshot uploads

…arameter

- Updated the replyError function in various workflow files to eliminate the unused 'data' parameter.
- Adjusted error handling to simplify the response structure while maintaining functionality.
- Ensured consistent formatting and error messaging across all affected workflows.
@pelikhan
Copy link
Copy Markdown
Collaborator Author

@copilot it seems that patching generation from an action on a branch fails. Investigate https://github.com/githubnext/gh-aw/actions/runs/18545886694/job/52863628682#step:42:100 and fix.

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 16, 2025

@pelikhan I've opened a new pull request, #1774, to work on those changes. Once the pull request is ready, I'll request review from you.

@pelikhan
Copy link
Copy Markdown
Collaborator Author

@copilot the GITHUB_AW_ASSETS_BRANCH environment variable need to be configured such that the safe-outputs MCP server has access to it. Update support in claude/codex/copilot agentic engines.

❌ safe_outputs::upload_asset(path: /tmp/gh-aw/screenshots/custom-safe-outpu...) ~28t
MCP error -32603: GITHUB_AW_ASSETS_BRANCH not set

https://github.com/githubnext/gh-aw/actions/runs/18545886694

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 16, 2025

@pelikhan I've opened a new pull request, #1775, to work on those changes. Once the pull request is ready, I'll request review from you.

@pelikhan
Copy link
Copy Markdown
Collaborator Author

@copilot merge main, format, lint, fix tests

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 16, 2025

@pelikhan I've opened a new pull request, #1776, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI mentioned this pull request Oct 16, 2025
6 tasks
@pelikhan pelikhan marked this pull request as ready for review October 16, 2025 01:58
@pelikhan pelikhan merged commit 787e28b into main Oct 16, 2025
12 of 16 checks passed
@pelikhan pelikhan deleted the screenshots-mcp branch October 16, 2025 01:58
@github-actions
Copy link
Copy Markdown
Contributor

Agentic Changeset Generator triggered by this pull request

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