Skip to content

refactor(ci): consolidate config precedence helpers#574

Merged
wesm merged 1 commit intomainfrom
bd-pui-6-ci-config-precedence
Mar 24, 2026
Merged

refactor(ci): consolidate config precedence helpers#574
wesm merged 1 commit intomainfrom
bd-pui-6-ci-config-precedence

Conversation

@mariusvniekerk
Copy link
Collaborator

Summary

  • move CI and GitHub Actions workflow precedence resolution into shared helpers in internal/config
  • update ci and gh-action commands to consume the shared resolvers instead of open-coded precedence ladders
  • add regression coverage for CI precedence edge cases, including workflow-agent fallback order

Verification

  • roborev fix --open --list
  • go fmt ./...
  • go vet ./...
  • go test ./...

Notes

  • bd-pui.6 is closed
  • beads Dolt sync is not configured in this checkout (bd dolt pull fails because no Dolt remote origin exists)

Move CI review and GitHub Actions workflow config resolution through shared helpers in internal/config.

Verification:
- roborev fix --open --list
- go fmt ./...
- go vet ./...
- go test ./...

🤖 Generated with [OpenAI Codex](https://openai.com/codex)
Co-authored-by: OpenAI Codex <noreply@openai.com>
@roborev-ci
Copy link

roborev-ci bot commented Mar 24, 2026

roborev: Combined Review (1c833e7)

Verdict: Approved.

All review agents agree the code is clean, with no issues of medium or higher severity found.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

@wesm wesm merged commit e9c674c into main Mar 24, 2026
8 checks passed
@wesm wesm deleted the bd-pui-6-ci-config-precedence branch March 24, 2026 17:09
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