Skip to content

fix: Remove extractVersion from matchUpdateTypes rules#623

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

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

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Summary

Removes extractVersion from the two GitHub Actions packageRules entries in renovate.json that also specify matchUpdateTypes. Renovate 43.x does not allow combining these two fields in the same rule, which causes a config validation error and prevents Renovate from running (see #619).

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 across all affected SDK repos (workos-node, workos-dotnet, workos-ruby, workos-php, workos-kotlin, workos-php-laravel, workos-go).

Review & Testing Checklist for Human

  • Confirm that extractVersion is redundant for the github-actions manager in Renovate 43.x (i.e., version extraction from v1.2.3 tags still works without it)
  • After merging, trigger a Renovate run (e.g., via the Mend Developer Portal) and verify it no longer reports config validation errors

Notes

  • The config validation error was: packageRules cannot combine both matchUpdateTypes and extractVersion
  • The schedule is set to "on the 15th day of the month before 12pm" UTC, so a manual trigger or Mend portal run is needed to verify the fix before the next scheduled window

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:01
@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
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 disallows combining these two fields in the same rule, causing a config validation error that prevents Renovate from running at all. The removal is safe because the modern github-actions manager natively handles v-prefixed semver tags without an explicit extraction pattern.

Confidence Score: 5/5

Safe to merge — minimal, targeted config fix with no functional regression.

The change is a two-line removal that directly addresses a documented Renovate 43.x validation error. The github-actions manager handles v-prefixed tags natively, so dropping extractVersion here causes no behavioral change. No P0/P1 findings.

No files require special attention.

Important Files Changed

Filename Overview
renovate.json Removes extractVersion from two github-actions packageRules that also use matchUpdateTypes, fixing a Renovate 43.x config validation error; no functional regression since the github-actions manager natively handles v-prefixed semver tags.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Renovate runs] --> B{Config validation}
    B -- "Before: extractVersion + matchUpdateTypes in same rule" --> C[❌ Validation error\nRenovate aborts]
    B -- "After: extractVersion removed" --> D[✅ Config valid]
    D --> E{Match github-actions packages}
    E --> F["minor/patch/digest/pinDigest\n→ automerge: true\n(group: github-actions-non-major)"]
    E --> G["major\n→ automerge: false\n(group: github-actions-major)"]
    F --> H[Renovate natively strips v-prefix\nfrom tags like v1.2.3]
    G --> H
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 09a2e5d into main Apr 15, 2026
10 of 11 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