Skip to content

fix(renovate): resolve invalid config blocking Renovate PRs#6

Merged
alexlovelltroy merged 1 commit into
mainfrom
fix/renovate-config-issue-5
Jul 2, 2026
Merged

fix(renovate): resolve invalid config blocking Renovate PRs#6
alexlovelltroy merged 1 commit into
mainfrom
fix/renovate-config-issue-5

Conversation

@mdklein

@mdklein mdklein commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes the two Renovate config validation errors reported in #5, which were causing Renovate to halt all PRs.

1. Invalid regExp (customManagers)

The second custom manager used a negative lookahead (?!ghcr\.io/openchami/). Renovate's regex managers run on Go's RE2 engine, which does not support lookahead assertions, so the pattern was rejected.

The second manager was byte-for-byte identical to the first except for that lookahead (same datasource, same versioning template), so both are collapsed into a single manager matching any annotated image. The packageRules already scope behaviour per-registry, so no coverage is lost.

2. packageRules combined matchUpdateTypes + ignoreUnstable

Renovate forbids these together in one rule. The rule's job — skip pre-release tags — is handled entirely by ignoreUnstable: true, so the redundant matchUpdateTypes is dropped.

Bonus cleanup

Modernized deprecated matchPackagePrefixes / matchPackagePatterns to matchPackageNames: ["ghcr.io/openchami/**"] (Renovate was already auto-migrating these), preventing future config warnings.

Validation

INFO: Validating .github/renovate.json
INFO: Config validated successfully

Closes #5

🤖 Generated with Claude Code

Renovate stopped opening PRs due to two config errors (issue #5):

- customManagers regex used a negative lookahead
  `(?!ghcr\.io/openchami/)`, which Go's RE2 engine (used by Renovate)
  does not support. The second manager was otherwise identical to the
  first, so collapse both into one manager matching any annotated image;
  packageRules already scope behaviour per registry.
- packageRules cannot combine `matchUpdateTypes` and `ignoreUnstable`.
  Drop the redundant `matchUpdateTypes` — `ignoreUnstable` alone does
  the intended job of skipping pre-release tags.

Also modernize deprecated `matchPackagePrefixes`/`matchPackagePatterns`
to `matchPackageNames` glob form to avoid future config warnings.

Validated with `renovate-config-validator`.

Closes #5

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mdklein
mdklein requested a review from alexlovelltroy July 1, 2026 21:12
@alexlovelltroy
alexlovelltroy merged commit 536811c into main Jul 2, 2026
4 of 5 checks passed
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.

Action Required: Fix Renovate Configuration

2 participants