Enable LLM gateway support for Codex engine#15574
Closed
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
Contributor
|
🧪 Smoke Temporary ID is now testing temporary ID functionality... |
Contributor
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
Contributor
|
🧪 Smoke Project is now testing project operations... |
Contributor
|
❌ Changeset Generator failed. Please review the logs for details. |
Contributor
|
🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation... |
This was referenced Feb 14, 2026
Contributor
|
✅ Smoke Temporary ID completed successfully. Temporary ID validation passed. |
Contributor
|
✅ Smoke Project completed successfully. All project operations validated. |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Agent Container Tool Check ✅
Result: 12/12 tools available ✅ All required development tools are present and functional in the agent container environment.
|
Copilot
AI
changed the title
[WIP] Add supportsLLMGateway enumeration for Codex
Enable LLM gateway support for Codex engine
Feb 14, 2026
Contributor
Smoke Test Results: PARTIAL PASS
|
Contributor
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Contributor
|
@copilot merge main |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Merged main branch successfully in commit 9c60b95. Resolved conflict in smoke-codex.lock.yml by keeping the pinned Playwright MCP version (0.0.64) and recompiled all workflows. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Contributor
|
@copilot merge main and recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Merged main and recompiled workflows in commit 1e16709. Resolved conflicts by accepting main's version of lock files and copilot-client files, then recompiled all 150 workflows successfully. The merge includes copilot-client bundling updates, discussion label support, and other recent changes. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request merges several updates from the main branch into a feature branch, including infrastructure improvements and new functionality. The primary changes involve: (1) pinning the Playwright MCP Docker image to a specific version for stability, (2) introducing a new TypeScript copilot-client for programmatic Copilot SDK integration, (3) adding discussion label support via GraphQL mutations, and (4) ensuring all workflow job outputs are consistently set.
Changes:
- Pins Playwright MCP Docker image to version 0.0.64 across all workflows and Go code for version stability
- Adds new copilot-client TypeScript package with comprehensive build system, testing infrastructure, and CI workflow
- Implements discussion label application using GraphQL API (fetchLabelIds and applyLabelsToDiscussion functions)
- Updates test expectations to verify all job outputs are set in every code path
Reviewed changes
Copilot reviewed 71 out of 73 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/workflow/*.go | Adds version pinning for Playwright MCP Docker image using constants.DefaultPlaywrightMCPVersion |
| copilot-client/* | New TypeScript client for Copilot SDK with ESM bundling, testing, and documentation |
| actions/setup/js/create_discussion.cjs | Adds GraphQL-based label support for discussions with sanitization and validation |
| actions/setup/js/collect_ndjson_output* | Ensures all outputs are set in error handlers and early returns |
| Makefile | Adds copilot-client build targets and integrates into test-all and recompile workflows |
| DEVGUIDE.md, CONTRIBUTING.md | Documents copilot-client setup, usage, and testing procedures |
| .github/workflows/*.lock.yml | Recompiles all workflow files with Playwright version pin and API proxy flag updates |
| .github/workflows/copilot-sdk-ci.yml | New CI workflow for testing copilot-client with headless Copilot CLI server |
| .github/workflows/smoke-copilot.md | Adds discussion creation test requirement with labels and expiration |
Files not reviewed (1)
- copilot-client/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.