Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Reduces the Smoke Claude agentic workflow’s token spend by switching to a lighter Claude model, lowering the turn cap, and narrowing the GitHub MCP tool surface.
Changes:
- Set Claude engine model to
claude-haiku-4-5and reducedmax-turnsfrom12to8. - Narrowed GitHub MCP toolsets from
[repos, pull_requests]to[pull_requests]. - Minor formatting tweak in the workflow’s markdown instructions.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/smoke-claude.md | Adjusts Claude engine config and GitHub toolset scope to reduce per-run token usage. |
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: 1
| engine: | ||
| id: claude | ||
| max-turns: 12 | ||
| model: claude-haiku-4-5 | ||
| max-turns: 8 |
There was a problem hiding this comment.
These updates are in the .md source, but the workflow that actually runs is .github/workflows/smoke-claude.lock.yml (which is currently still generated with --max-turns 12 and the old GitHub tool allowlist). Please re-run gh aw compile and commit the regenerated smoke-claude.lock.yml so the model/max-turns/toolset changes take effect and the lock freshness check doesn’t fail.
This comment has been minimized.
This comment has been minimized.
|
@copilot address this review feedback #2065 (review) |
This comment has been minimized.
This comment has been minimized.
Addressed in commit 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:
|
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
Smoke Test: Copilot BYOK (Offline) Mode
Running in BYOK offline mode ( Overall: PASS CC
|
🔥 Smoke Test Results
Overall: PASS Author:
|
|
Smoke Test Results ✅ GitHub MCP: Last 2 merged PRs retrieved
✅ Playwright: github.com page title verified Status: PASS
|
Smoke Test: GitHub Actions Services Connectivity
All checks passed. (
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
The Smoke Claude workflow was over-provisioned for its task profile, with cost dominated by large tool schema/cache writes and a higher-capability model than required. This PR reduces per-run token cost by switching to a lighter model and tightening execution/tool scope.
Engine cost controls
claude-haiku-4-5max-turnsfrom12to8GitHub MCP scope reduction
[repos, pull_requests]to[pull_requests]to reduce loaded tool schema surfaceWorkflow source updated
.github/workflows/smoke-claude.mdWarning
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:
https://api.github.com/graphql/usr/bin/gh gh repo view github/gh-aw --json name,defaultBranchRef(http block)https://api.github.com/repos/actions/github-script/git/ref/tags/v9/usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v9 --jq .object.sha(http block)https://api.github.com/repos/github/gh-aw-actions/git/ref/tags/v0.68.3/usr/bin/gh gh api /repos/github/gh-aw-actions/git/ref/tags/v0.68.3 --jq .object.sha(http block)If you need me to access, download, or install something from one of these locations, you can either: