Skip to content

fix(ci): exclude test fixtures from Renovate scanning - #2457

Open
fengmk2 wants to merge 2 commits into
mainfrom
fix/renovate-fixture-crash
Open

fix(ci): exclude test fixtures from Renovate scanning#2457
fengmk2 wants to merge 2 commits into
mainfrom
fix/renovate-fixture-crash

Conversation

@fengmk2

@fengmk2 fengmk2 commented Aug 14, 2026

Copy link
Copy Markdown
Member

Renovate aborts the whole repository scan with ERR_PNPM_INVALID_SELECTOR when it parses the multi-level pnpm override selector (vite-plugin-svgr>foo>vite) in the migration_monorepo_pnpm_overrides_dependency_selector snapshot fixture (error log). The selector is intentional vp migrate test data, but @pnpm/parse-overrides cannot parse two > levels. Every run since mid-June failed, so the update PRs #1780, #1782, and #1831 went stale.

Fix: add the snapshot and unit-test fixture trees to ignorePaths. Renovate applies ignorePaths before extraction, so fixture files never reach the parser on any Renovate version. Fixture dependencies also stop receiving bump PRs, which would break recorded snapshots.

Verified with renovate-config-validator and a local dry run (npx renovate --platform=local --dry-run=extract): the npm manager drops from 726 files to 11 real package files, no fixture file is extracted, and the cargo and github-actions managers are unchanged (crates/vp_cli_snapshots/Cargo.toml stays managed).

Closes #2451

@netlify

netlify Bot commented Aug 14, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 2544a9d
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a801a919a64010008e0167e

@fengmk2 fengmk2 self-assigned this Aug 14, 2026
@fengmk2
fengmk2 requested a review from wan9chi August 14, 2026 14:37
Renovate aborts the whole repository scan with ERR_PNPM_INVALID_SELECTOR
on the multi-level pnpm override selector in the
migration_monorepo_pnpm_overrides_dependency_selector fixture, so no
dependency update PRs have been created since mid-June. Ignore the
snapshot and unit-test fixture trees; fixture deps are test data.

Closes #2451

Claude-Session: https://claude.ai/code/session_01QGJxVVp6EsxhuxfBjKfvDy
@fengmk2
fengmk2 force-pushed the fix/renovate-fixture-crash branch from e975b18 to 86ec881 Compare August 14, 2026 14:38
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.

Renovate & Dependencies update broken

1 participant