Skip to content

Upgrade gh-aw to latest pre-release - #8166

Merged
lpcox merged 3 commits into
mainfrom
lpcox-upgrade-gh-aw-prerelease-49c
Sep 5, 2026
Merged

Upgrade gh-aw to latest pre-release#8166
lpcox merged 3 commits into
mainfrom
lpcox-upgrade-gh-aw-prerelease-49c

Conversation

@lpcox

@lpcox lpcox commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • upgrade workflows with gh aw upgrade --pre-releases using gh-aw v0.88.4
  • run the required smoke-workflow post-processing
  • add gpt-6-astra to the supported Copilot model catalog and static pricing fallback coverage

Testing

  • npm test -- --runInBand (346 suites, 5,528 tests passed)
  • commit hooks: lint and TypeScript build

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 5, 2026 12:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

GPT-6 Astra is accepted by validation but rejected by AI-credit enforcement because no resolvable pricing exists.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 1 Medium severity

New issues introduced by this change (1)
Severity Finding
Medium severity src/​copilot-model-catalog-sync.test.ts — Adding gpt-6-astra to this exception only suppresses the catalog-sync assertion; it does not…
What changed in this PR

Upgrades generated gh-aw workflows and adds GPT-6 Astra model recognition.

Changes:

  • Refreshes runner-doctor workflow locks for AWF 0.28.13.
  • Normalizes enclave gateway credential placement.
  • Adds GPT-6 Astra validation and catalog checks.
File Description
src/​copilot-model.ts Adds GPT-6 Astra support.
src/​copilot-model.test.ts Tests model validation.
src/​copilot-model-catalog-sync.test.ts Exempts Astra from curated pricing checks.
.github/​workflows/​smoke-enclave-build-test.lock.yml Normalizes gateway key placement.
.github/​workflows/​self-hosted-runner-doctor.lock.yml Refreshes generated workflow and AWF version.
.github/​workflows/​self-hosted-runner-doctor-updater.lock.yml Refreshes generated metadata.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/copilot-model-catalog-sync.test.ts Outdated
'gpt-4o',
'gpt-4o-mini',
'gpt-5-1',
'gpt-6-astra',
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 93.99% 94.00% 📈 +0.01%
Statements 92.62% 92.63% ➡️ +0.01%
Functions 93.33% 93.33% ➡️ +0.00%
Branches 86.38% 86.39% 📈 +0.01%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@lpcox

lpcox commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator Author

@copilot address review feedback

Copilot AI commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@copilot address review feedback

Added curated gpt-6-astra pricing, removed the static-catalog exemption, and added a guard test confirming it passes AI-credit model rejection checks. Commit: 98926d1.

@lpcox

lpcox commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator Author

@copilot rebase onto main

…rerelease-49c

# Conflicts:
#	containers/api-proxy/ai-credits-pricing.js

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

Copilot AI commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@copilot rebase onto main

Merged the current main branch into this branch in merge commit 5209678. I used a merge rather than rebasing because publishing a rewritten history would require a force push.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@lpcox Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@lpcox
lpcox merged commit 446dac5 into main Sep 5, 2026
25 of 26 checks passed
@lpcox
lpcox deleted the lpcox-upgrade-gh-aw-prerelease-49c branch September 5, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants