Bump Claude/Codex/Pi/Playwright defaults and regenerate compiled workflow artifacts#42690
Bump Claude/Codex/Pi/Playwright defaults and regenerate compiled workflow artifacts#42690Copilot wants to merge 2 commits into
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges... |
|
✅ PR Code Quality Reviewer completed the code quality review. |
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. Test Quality Sentinel skipped. |
|
🚀 Smoke Gemini MISSION COMPLETE! Gemini has spoken. ✨ |
|
✅ All tools validated successfully! Agent Container Smoke Test confirms agent container is ready. |
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅ |
|
🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation... |
|
🚀 Smoke Pi MISSION COMPLETE! Pi delivered. 🥧 |
|
✅ Design Decision Gate 🏗️ completed the design decision gate check. No ADR enforcement needed: PR #42690 does not have the 'implementation' label and has only 24 new lines of code in business logic directories (threshold: 100). |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🚀 Smoke Antigravity MISSION COMPLETE! Antigravity has spoken. ✨ |
|
📰 BREAKING: Smoke Copilot - AOAI (apikey) is now investigating this pull request. Sources say the story is developing... |
|
📰 BREAKING: Smoke Copilot - AOAI (Entra) is now investigating this pull request. Sources say the story is developing... |
There was a problem hiding this comment.
Pull request overview
This pull request updates the pinned default versions for several agent runtimes/tools in gh aw (via pkg/constants/version_constants.go) and regenerates compiled workflow lockfiles plus WASM golden fixtures so generated outputs reflect the new defaults.
Changes:
- Bumped default versions for Claude Code, Codex, Pi, Playwright MCP, and Playwright CLI in
pkg/constants/version_constants.go. - Recompiled workflow
.lock.ymlartifacts to reflect the updated install commands /GH_AW_INFO_VERSIONmetadata. - Updated WASM golden fixtures that assert against compiled output containing these version strings.
Show a summary per file
| File | Description |
|---|---|
| pkg/constants/version_constants.go | Bumps default pinned versions for agent runtimes/tools. |
| pkg/workflow/testdata/TestWasmGolden_CompileFixtures/playwright-cli-mode.golden | Updates compiled output fixture for Playwright CLI install version. |
| pkg/workflow/testdata/TestWasmGolden_AllEngines/claude.golden | Updates compiled output fixture for Claude Code version metadata/install. |
| pkg/workflow/testdata/TestWasmGolden_AllEngines/codex.golden | Updates compiled output fixture for Codex version metadata/install. |
| pkg/workflow/testdata/TestWasmGolden_AllEngines/pi.golden | Updates compiled output fixture for Pi version metadata/install. |
| .github/workflows/ai-moderator.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/auto-triage-issues.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/avenger.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/blog-auditor.lock.yml | Regenerated lockfile with updated Claude Code + Playwright CLI defaults. |
| .github/workflows/chaos-pr-bundle-fuzzer.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/changeset.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/cli-version-checker.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/codex-github-remote-mcp-test.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/commit-changes-analyzer.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/daily-awf-spec-compiler-surfacing.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/daily-choice-test.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/daily-function-namer.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/daily-hippo-learn.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/daily-model-inventory.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/daily-model-resolution.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/daily-multi-device-docs-tester.lock.yml | Regenerated lockfile with updated Pi + Playwright CLI defaults. |
| .github/workflows/daily-reliability-review.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/daily-safe-outputs-conformance.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/daily-team-evolution-insights.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/dev.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/docs-noob-tester.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/duplicate-code-detector.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/eslint-monster.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/github-mcp-tools-report.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/go-pattern-detector.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/grumpy-reviewer.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/hippo-embed.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/hourly-ci-cleaner.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/issue-monster.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/lint-monster.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/schema-consistency-checker.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/schema-feature-coverage.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/slide-deck-maintainer.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/smoke-agent-all-none.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/smoke-agent-public-approved.lock.yml | Regenerated lockfile with updated Claude Code default version metadata/install. |
| .github/workflows/smoke-call-workflow.lock.yml | Regenerated lockfile with updated Codex default version metadata/install. |
| .github/workflows/smoke-copilot-arm.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/smoke-copilot-aoai-apikey.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/smoke-copilot-aoai-entra.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/smoke-copilot.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/spec-enforcer.lock.yml | Regenerated lockfile with updated Pi default version metadata/install. |
| .github/workflows/visual-regression-checker.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
| .github/workflows/weekly-editors-health-check.lock.yml | Regenerated lockfile with updated Playwright CLI default install. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 109/109 changed files
- Comments generated: 0
- Review effort level: Low
Agent Container Tool Check
Result: 12/12 tools available ✅ Overall Status: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "awmgmcpg"See Network Configuration for more information.
|
Smoke test
Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "proxy.golang.org"See Network Configuration for more information.
|
Smoke Test Results✅ GitHub MCP Testing Overall Status: 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.
|
There was a problem hiding this comment.
Skills-Based Review 🧠
Applied /codebase-design — this is a clean, well-structured routine version bump. All changes are consistent and regenerated artifacts align with the updated defaults. Approving with two minor observations.
📋 Key Themes & Highlights
Positive Highlights
- ✅ All five version bumps (
DefaultClaudeCodeVersion,DefaultCodexVersion,DefaultPiVersion,DefaultPlaywrightMCPVersion,DefaultPlaywrightCLIVersion) are applied consistently throughoutversion_constants.go, all lock files, and golden fixtures. - ✅ Good discipline in regenerating golden WASM fixtures alongside lock files to keep snapshot tests in sync.
- ✅ Compact, symmetric diff (1078 additions / 1078 deletions) confirms no unintended collateral changes.
- ✅ Lock file metadata (
engine_versions,GH_AW_INFO_VERSION,GH_AW_INFO_AGENT_VERSION, and thenpm installcommand) all updated in lock-step for each engine.
Observations
Claude Code skips 2.1.196: The bump goes 2.1.195 → 2.1.197. If this is intentional (e.g., 2.1.196 had a known regression), a brief note in the PR description helps future maintainers understand why that version was skipped.
Missing upgrade checklist comments: DefaultCopilotVersion has a detailed comment documenting known failure modes to verify on each bump (MCP tool loading, /models with PAT auth). The Claude Code, Codex, Pi, and Playwright constants have no equivalent guidance. Extending that pattern to other runtime constants would make future bumps safer — per /codebase-design: policy lives next to the thing it governs.
🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · 48.8 AIC · ⌖ 7.09 AIC · ⊞ 6.6K
Comment /matt to run again
There was a problem hiding this comment.
Approved. Clean mechanical version bump with complete artifact regeneration.
Review findings
What was reviewed
-
pkg/constants/version_constants.go— 5 one-line version constant bumps (ClaudeCode 2.1.195→2.1.197, Codex 0.142.4→0.142.5, Pi 0.80.2→0.80.3, PlaywrightMCP 0.0.76→0.0.77, PlaywrightCLI 0.1.14→0.1.15). No logic changes, no new/removed constants, types and comments unchanged. -
104 lock files — all auto-generated substitutions only;
frontmatter_hashandbody_hashmetadata correctly unchanged (source markdown unaltered);engine_versionsfields in metadata correctly updated. -
4 golden fixtures —
claude.golden,codex.golden,pi.golden,playwright-cli-mode.goldenall properly aligned with new defaults.
Completeness check
A grep for old version strings (2.1.195, 0.142.4, 0.80.2, 0.0.76, 0.1.14) across all .go, .yml, and .golden files returns empty — no stragglers.
Notable but non-blocking
Claude Code skips 2.1.196 (goes 2.1.195→2.1.197). This is a valid npm publish pattern (failed/yanked release) and is correctly described in the PR body. Not a concern.
🔎 Code quality review by PR Code Quality Reviewer · 65 AIC · ⌖ 6.91 AIC · ⊞ 1.6K
Comment /review to run again
There was a problem hiding this comment.
Version Bump Review: Claude/Codex/Pi/Playwright defaults
Summary: Straightforward version constant bump with full lock file and golden fixture regeneration.
Verified:
pkg/constants/version_constants.go: all 5 constants updated correctly- Claude:
2.1.195→2.1.197 - Codex:
0.142.4→0.142.5 - Pi:
0.80.2→0.80.3 - PlaywrightMCP:
0.0.76→0.0.77 - PlaywrightCLI:
0.1.14→0.1.15
- Claude:
- 105
.lock.ymlfiles recompiled —GH_AW_INFO_VERSION,GH_AW_INFO_AGENT_VERSION,Install ... CLIsteps all consistent with the new constants TestWasmGolden_AllEngines/{claude,codex,pi}.goldenandTestWasmGolden_CompileFixtures/playwright-cli-mode.goldenupdated correctly
Pre-existing divergence (not introduced by this PR)
pkg/workflow/testdata/wasm_golden/WasmBinary/playwright-cli-mode.golden still shows @playwright/cli@0.1.13 — two versions behind. This divergence predates this PR (it wasn't changed here) and the file is only exercised by the Node.js WASM binary test (test-wasm-golden.mjs). Not a blocker for this PR.
No security, correctness, or reliability concerns found. ✅
🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · 104.4 AIC · ⌖ 6.78 AIC · ⊞ 4.9K
|
Smoke Test Results: 1:✅ 2:❌ 3:❌ 4:❌ 5:❌ 6:❌ 7:❌ 8:❌ 9:❌ 10:❌ 11:❌ 12:❌ 13:❌ 14:❌ 15:❌
|
Comment MemoryNote This comment is managed by comment memory.It stores persistent context for this thread in the code block at the top of this comment. Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
There was a problem hiding this comment.
Smoke bot looked. Tiny notes left.
Warning
Firewall blocked 6 domains
The following domains were blocked by the firewall during workflow execution:
accounts.google.comandroid.clients.google.comclients2.google.comcontentautofill.googleapis.comsafebrowsingohttpgateway.googleapis.comwww.google.com
To allow these domains, add them to the
network.allowedlist in your workflow frontmatter:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
📰 BREAKING: Report filed by Smoke Copilot · 327.4 AIC · ⌖ 15.6 AIC · ⊞ 19.2K
Comment /smoke-copilot to run again
Add label smoke to run again
This updates the pinned default versions for five agent runtimes/tools (Claude Code, Codex, Pi, Playwright MCP, Playwright CLI) in
pkg/constants/version_constants.go. Generated artifacts were recompiled so workflow outputs and golden fixtures stay aligned with the new defaults.Version constants
pkg/constants/version_constants.go:DefaultClaudeCodeVersion:2.1.195→2.1.197DefaultCodexVersion:0.142.4→0.142.5DefaultPiVersion:0.80.2→0.80.3DefaultPlaywrightMCPVersion:0.0.76→0.0.77DefaultPlaywrightCLIVersion:0.1.14→0.1.15Compiled workflow lockfiles
.github/workflows/*.lock.ymlso emitted install commands andGH_AW_INFO_VERSIONmetadata reflect the new defaults.WASM golden fixture alignment
pkg/workflow/testdata/TestWasmGolden_*where expected compiled output includes these version strings.