Skip to content

Fix convert_gateway_config_codex_test.sh missing environment variables - #9802

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-smoke-codex-workflow
Jan 13, 2026
Merged

Fix convert_gateway_config_codex_test.sh missing environment variables#9802
pelikhan merged 2 commits into
mainfrom
copilot/fix-smoke-codex-workflow

Conversation

CopilotAI commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

The convert_gateway_config_codex_test.sh test was failing because it didn't set the required MCP_GATEWAY_DOMAIN and MCP_GATEWAY_PORT environment variables that convert_gateway_config_codex.sh expects.

Changes

  • Add required environment variables at test start:
    export MCP_GATEWAY_DOMAIN="host.docker.internal"export MCP_GATEWAY_PORT="80"
  • Create output directory /tmp/gh-aw/mcp-config before running tests
  • Add URL format verification test (confirms domain:port substitution works)
  • Add error handling tests for missing MCP_GATEWAY_DOMAIN and MCP_GATEWAY_PORT

This aligns the test with how the Copilot and Claude converter tests work—all three converters require the same environment variables.

Original prompt

Fix the smoke-codex workflow https://github.com/githubnext/gh-aw/actions/runs/20947017640

pay attention to convert_gateway_config_codex.sh and you should learn how smoke-copilot and smoke-claude are working.


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

…variables
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
CopilotAI changed the title [WIP] Fix smoke-codex workflow issuesFix convert_gateway_config_codex_test.sh missing environment variablesJan 13, 2026
CopilotAI requested a review from MossakaJanuary 13, 2026 08:04
@github-actions

github-actionsBot commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions

github-actionsBot commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actionsBot commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

🎬 THE ENDSmoke ClaudeMISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions

github-actionsBot commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test Results

Last 2 Merged PRs:

  • feat: adds create_view operation to update_project
  • style: apply formatting and linting fixes to update_project implementation

Test Results:

  • ✅ GitHub MCP - Retrieved merged PRs
  • ✅ Bash Tool - File created and verified
  • ✅ Playwright - Navigated to GitHub, title contains "GitHub"
  • ❌ Serena Go - Not available in this workflow

Overall Status: PASS (3/4 core tests)

cc @pelikhan

AI generated by Smoke Copilot

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test Results - Claude Engine

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP (list PRs)
  • ✅ Serena Go (config retrieved)
  • ✅ Playwright (navigated to github.com, title verified)
  • ✅ Tavily Web Search (returned 10 results)
  • ✅ File Write (/tmp/gh-aw/agent/smoke-test-claude-20954252602.txt)
  • ✅ Bash Tool (file verified)

Status: PASS

AI generated by Smoke Claude

@pelikhan
pelikhan marked this pull request as ready for review January 13, 2026 11:15
@pelikhan
pelikhan merged commit 673aca0 into mainJan 13, 2026
36 checks passed
@pelikhan
pelikhan deleted the copilot/fix-smoke-codex-workflow branch January 13, 2026 11:15
Sign up for freeto 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

@pelikhan@Mossaka