Skip to content

Stabilize extended validation checks#111

Merged
jmcte merged 2 commits into
mainfrom
codex/ci-alert-fixes
May 16, 2026
Merged

Stabilize extended validation checks#111
jmcte merged 2 commits into
mainfrom
codex/ci-alert-fixes

Conversation

@jmcte
Copy link
Copy Markdown
Contributor

@jmcte jmcte commented May 16, 2026

Summary

  • isolate the doctor test from inherited GITHUB_PAT so mutation dry-runs are deterministic
  • make scheduled/manual runner drift detection report drift as a warning instead of failing code validation

Verification

  • pnpm install --frozen-lockfile passed after network approval
  • pnpm exec vitest run test/doctor.test.ts passed
  • GITHUB_PAT=secret pnpm exec vitest run test/doctor.test.ts passed
  • pnpm exec vitest run test/workflow.test.ts test/mutation-testing.test.ts passed

CI alert

  • Addresses Extended Validation failure from May 16, 2026: mutation dry-run expected missing GITHUB_PAT, SYNOLOGY_HOST while CI had GITHUB_PAT, plus live drift detection reported under-provisioned runner pools.

@athena-omt athena-omt added area:infra Infrastructure, CI, release, governance, scripts, or repo setup. lane:hephaestus Hephaestus build/repo-ops lane. review:athena Athena review governance requested. risk:medium Medium-risk change; normal care required. state:waiting-checks Waiting for CI/check status to settle. status:needs-review PR is ready for Athena review. labels May 16, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 681b981557

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/extended-validation.yml
pheidon
pheidon previously approved these changes May 16, 2026
@jmcte jmcte enabled auto-merge (squash) May 16, 2026 15:28
@jmcte jmcte merged commit 7197120 into main May 16, 2026
12 checks passed
@jmcte jmcte deleted the codex/ci-alert-fixes branch May 16, 2026 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:infra Infrastructure, CI, release, governance, scripts, or repo setup. lane:hephaestus Hephaestus build/repo-ops lane. review:athena Athena review governance requested. risk:medium Medium-risk change; normal care required. state:waiting-checks Waiting for CI/check status to settle. status:needs-review PR is ready for Athena review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants