[WIP] Fix failed jobs in Smoke Goose workflow - #50392
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
/smoke-goose |
|
🪿 Smoke Goose Goose delivered. |
|
@copilot recompile and review this error: |
|
@copilot the harness driver should fetch the /reflect endpoint and determine the available endpoints. |
There was a problem hiding this comment.
🟡 Not ready to approve
The static endpoint regresses OpenAI/Codex routing in the shared engine.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Updates Goose CLI endpoint configuration to restore Copilot token telemetry.
Changes:
- Replaces ignored desktop-only variables with Goose CLI variables.
- Regenerates the Smoke Goose workflow.
File summaries
| File | Description |
|---|---|
.github/workflows/shared/goose.md |
Configures the Goose OpenAI endpoint. |
.github/workflows/smoke-goose.lock.yml |
Regenerates the compiled workflow. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Balanced
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
| # variables are Goose desktop-only settings and are ignored by the CLI, so | ||
| # OPENAI_HOST must point at the AWF Copilot api-proxy sidecar; otherwise Goose | ||
| # sends the Copilot token to the OpenAI proxy port and is rejected with 401. | ||
| OPENAI_HOST: http://172.30.0.30:10002 |
|
@copilot Please address the latest review feedback on this PR, refresh the branch if GitHub allows it, and then run the Open review feedback to close out (newest first):
Please also recompile the workflow markdown after updating
|
|
@copilot recompile and review this error: Warning: Failed to start extension 'safeoutputs' (failed to initialize MCP client: Send message error Transport [rmcp::transport::worker::WorkerTransport<rmcp::transport::streamable_http_client::StreamableHttpClientWorkerreqwest::async_impl::client::Client>] error: Client error: error sending request for url (http://localhost:8080/mcp/safeoutputs), when send initialize request), continuing without it ____) 20260804_1 · /home/runner/work/gh-aw/gh-aw Harness should use /reflect to resolve the paths to the llms apis |
|
🎉 This pull request is included in a new release. Release: |
check_token_telemetryfailure in Smoke Goose run 30944874071GOOSE_PROVIDER__HOST/GOOSE_PROVIDER__API_KEY/GOOSE_PROVIDER__TYPEare Goose desktop (Electron) settings and are ignored by the Goose CLI. The Rust OpenAI provider resolves its endpoint fromOPENAI_HOST(highest priority) thenOPENAI_BASE_URL, so Goose sent the Copilot token to the OpenAI proxy port 10000 and got401 Unauthorized→ no LLM calls → emptytoken-usage.jsonl→check_token_telemetryfailedGOOSE_PROVIDER__*vars withOPENAI_HOST(AWF Copilot api-proxy at172.30.0.30:10002) andOPENAI_BASE_PATHin.github/workflows/shared/goose.mdmake recompile)make agent-report-progress— lint clean, lock files in sync, tests pass