Skip to content

[WIP] Fix failing GitHub Actions job 'Integration: Workflow Features'#41153

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-integration-workflow-features
Jun 24, 2026
Merged

[WIP] Fix failing GitHub Actions job 'Integration: Workflow Features'#41153
pelikhan merged 2 commits into
mainfrom
copilot/fix-integration-workflow-features

Conversation

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Fix the failing GitHub Actions job "Integration: Workflow Features"
Analyze the Actions logs, identify the root cause of the failure, and implement a fix.
Check run ID: 83108335417
Job URL: https://github.com/github/gh-aw/actions/runs/28072001997/job/83108335417

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan June 24, 2026 03:13
@pelikhan pelikhan marked this pull request as ready for review June 24, 2026 03:34
Copilot AI review requested due to automatic review settings June 24, 2026 03:34
@pelikhan pelikhan merged commit 433c534 into main Jun 24, 2026
21 checks passed
@pelikhan pelikhan deleted the copilot/fix-integration-workflow-features branch June 24, 2026 03:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Copilot HTTP MCP integration test to expect the correct JSON escaping for ${VAR}-style environment variable references when rendered into the Copilot MCP config output, addressing the failing “Integration: Workflow Features” job.

Changes:

  • Adjusts expected MCP config substrings from \${VAR} to \\${VAR} for HTTP header and env passthrough checks.
  • Aligns integration test expectations with the existing shared MCP HTTP header rendering behavior (and related unit tests).
Show a summary per file
File Description
pkg/workflow/copilot_mcp_http_integration_test.go Updates integration test expectations to match the JSON-escaped env-var reference format emitted by MCP config rendering.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

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.

3 participants