Skip to content

fix: Remove extractVersion from matchUpdateTypes rules#365

Merged
gjtorikian merged 1 commit intomainfrom
devin/1776275998-fix-renovate-config
Apr 15, 2026
Merged

fix: Remove extractVersion from matchUpdateTypes rules#365
gjtorikian merged 1 commit intomainfrom
devin/1776275998-fix-renovate-config

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Summary

Removes extractVersion from two GitHub Actions packageRules entries in renovate.json that also use matchUpdateTypes.

Renovate 43.x does not allow combining matchUpdateTypes and extractVersion in the same package rule, causing a config validation error:

packageRules cannot combine both matchUpdateTypes and extractVersion

This was the root cause of the "Action Required: Fix Renovate Configuration" issues opened by the Renovate bot across all SDK repos. The extractVersion field (^v(?<version>\d+\.\d+\.\d+)$) is no longer needed here — the github-actions manager in modern Renovate natively handles v-prefixed semver tags.

The same fix is being applied to all 8 affected SDK repos.

Review & Testing Checklist for Human

  • Verify that removing extractVersion does not regress GitHub Actions version detection for this repo's workflows (check that Renovate can still detect and propose updates for digest-pinned actions)
  • After merging, trigger a Renovate run (e.g. via Mend Developer Portal) and confirm it no longer reports a config validation error

Notes

The previous extractVersion was originally in a standalone rule without matchUpdateTypes. A recent refactor split the GitHub Actions rules into major/non-major groups but carried extractVersion into both, creating the invalid combination.

Link to Devin session: https://app.devin.ai/sessions/8f92a2c219484c7a8123b098ba22aff0

Renovate 43.x does not allow combining matchUpdateTypes and
extractVersion in the same packageRule. The github-actions manager
natively handles v-prefixed semver tags, so extractVersion is no
longer needed.

Co-Authored-By: garen.torikian <garen.torikian@workos.com>
@devin-ai-integration devin-ai-integration bot requested review from a team as code owners April 15, 2026 18:02
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Original prompt from garen.torikian

›why is this issue happening across various repos? workos/workos-node#1555, workos/workos-dotnet#231

@devin-ai-integration devin-ai-integration bot requested a review from nicknisi April 15, 2026 18:02
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 15, 2026

Greptile Summary

This PR removes extractVersion from two github-actions packageRules entries in renovate.json that also specify matchUpdateTypes. Renovate 43.x rejects rules that combine both fields, causing the "Fix Renovate Configuration" bot issues. The modern github-actions manager handles v-prefixed semver tags natively, so extractVersion is no longer required.

Confidence Score: 5/5

Safe to merge — minimal, targeted config fix with no logic or security implications.

The change removes exactly the two extractVersion fields that Renovate 43.x prohibits when combined with matchUpdateTypes. The remaining config is syntactically valid JSON and logically correct; the github-actions manager handles v-prefixed tags natively so no functionality is lost. No custom rules are violated and there are no code changes.

No files require special attention.

Important Files Changed

Filename Overview
renovate.json Removes extractVersion from two matchManagers: github-actions rules that also use matchUpdateTypes, fixing Renovate 43.x validation error; remaining config is well-formed.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Renovate runs config validation] --> B{Renovate 43.x check}
    B -- "matchUpdateTypes + extractVersion\nin same rule" --> C[❌ Config validation error\n'Fix Renovate Configuration' issue opened]
    B -- "matchUpdateTypes only\n(no extractVersion)" --> D[✅ Config valid]
    D --> E[github-actions manager\nnatively strips 'v' prefix\nfrom semver tags]
    E --> F[PRs proposed for\ngithub-actions updates]
Loading

Reviews (1): Last reviewed commit: "Remove extractVersion from matchUpdateTy..." | Re-trigger Greptile

@devin-ai-integration devin-ai-integration bot changed the title Remove extractVersion from matchUpdateTypes rules fix: Remove extractVersion from matchUpdateTypes rules Apr 15, 2026
@gjtorikian gjtorikian merged commit f4ad142 into main Apr 15, 2026
8 of 9 checks passed
@gjtorikian gjtorikian deleted the devin/1776275998-fix-renovate-config branch April 15, 2026 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant