Skip to content

Fix drift counts from runner group inventory#90

Merged
jmcte merged 3 commits intomainfrom
codex/drift-runner-group-runners
Apr 26, 2026
Merged

Fix drift counts from runner group inventory#90
jmcte merged 3 commits intomainfrom
codex/drift-runner-group-runners

Conversation

@jmcte
Copy link
Copy Markdown
Contributor

@jmcte jmcte commented Apr 25, 2026

Summary

  • count drift inventory from each runner group runners endpoint instead of relying on org-wide runner_group_id fields
  • record the current selected public runner-group repository access, including apw-cli
  • keep drift output aligned with GitHub's current org runner API behavior

Verification

  • pnpm exec vitest run test/drift-detect.test.ts test/cli.test.ts test/github.test.ts test/config.test.ts
  • pnpm validate-config -- --config config/pools.yaml --env .env
  • pnpm validate-github -- --config config/pools.yaml --env .env
  • GITHUB_PAT= pnpm drift-detect -- --config config/pools.yaml --threshold 0 now reports synology-private actual=4 and synology-public actual=2

@jmcte jmcte enabled auto-merge (squash) April 25, 2026 16:49
@jmcte jmcte requested a review from daedalus-omt April 25, 2026 20:24
Copy link
Copy Markdown
Contributor

@pheidon pheidon left a comment

Choose a reason for hiding this comment

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

Pheidon pass: reviewed diff and green checks look good. Approving.

@pheidon
Copy link
Copy Markdown
Contributor

pheidon commented Apr 26, 2026

@daedalus-omt quick nudge from the Pheidon pass: this PR is approved on my side and auto-merge is enabled. If you still need to review, please do; if the request is stale, please clear it so this can land cleanly.

@pheidon pheidon removed the request for review from daedalus-omt April 26, 2026 12:00
@jmcte jmcte merged commit 6fd3118 into main Apr 26, 2026
9 of 10 checks passed
@jmcte jmcte deleted the codex/drift-runner-group-runners branch April 26, 2026 12:10
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.

2 participants