fix: postprocess claude-token-optimizer lock file to use local awf build#3300
Conversation
There was a problem hiding this comment.
Pull request overview
Fixes the Daily Claude Token Optimization Advisor workflow which was failing because its lock file referenced a non-existent AWF release v0.25.29. The fix re-runs the standard postprocess script to align this lock file with the rest of the repo's workflows.
Changes:
- Replace broken
install_awf_binary.sh v0.25.29step withnpm ci/npm run build/ localawfshim - Remove stale
sparse-checkoutblock from activation job checkout - Add
--session-state-dirflag and inline the Copilot session-state copy step
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/claude-token-optimizer.lock.yml | Postprocessed to use local awf build, drop sparse-checkout, add session-state dir, inline session-state copy |
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
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@copilot update the .md file to use local build and recompile the lock.yml |
Done. Added the |
Smoke Test Results
Result: 2/3 PASS
|
🔬 Smoke Test Results
Overall: PASS 🟢 Author:
|
Smoke Test: Copilot BYOK (Offline) Mode
Running in BYOK offline mode ( Overall: PARTIAL — BYOK inference ✅, but pre-computed test data was not injected (template variables unexpanded) and GitHub MCP returned 401.
|
|
Smoke Codex: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
|
Smoke test: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Smoke Test: GitHub Actions Services Connectivity
Overall: FAIL
|
The
Daily Claude Token Optimization Advisorworkflow was failing at theInstall AWF binarystep because the compiled lock file referencedv0.25.29, which was never released (releases jump fromv0.25.20→v0.25.40).Root cause
claude-token-optimizer.lock.ymlcontained:This is the only lock file in the repo that still had this pattern — every other workflow was already using local build steps. The post-processing script was not run when this lock file was originally compiled.
Fix
Ran
npx ts-node scripts/ci/postprocess-smoke-workflows.ts, which applied the standard replacements toclaude-token-optimizer.lock.yml:Install awf dependencies/Build awf/Install awf binary (local)(mirrors all other lock files)sparse-checkoutblock from the activation job checkout step--session-state-dirflag to the AWF invocationCopy Copilot session statestep (replaces the helper script call)