Skip to content

fix(deps): drop @stoplight/spectral-rulesets pin in validation/ - #405

Merged
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/deps-spectral-rulesets-unpin
Aug 5, 2026
Merged

fix(deps): drop @stoplight/spectral-rulesets pin in validation/#405
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/deps-spectral-rulesets-unpin

Conversation

@hdamker

@hdamker hdamker commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

bug

What this PR does / why we need it:

Drops the @stoplight/spectral-rulesets pin (1.22.2, added in #329) from validation/package.json overrides. Upstream shipped the null-guard fix (stoplightio/spectral#2960) in 1.22.7, so the pin is no longer needed; removing it lets the lockfile resolve and hoist a single 1.22.7 copy (previously two nested 1.22.2 copies under spectral-cli and spectral-ruleset-bundler).

Which issue(s) this PR fixes:

Fixes #330

Special notes for reviewers:

Dependency-only (validation/package.json + lockfile). Verified: npm ci --ignore-scripts clean, npm audit 0 vulnerabilities, full validation/ unit suite (1218/1218) passes including the existing test_spectral_does_not_crash_on_null_value regression test from #328/#329.

Changelog input

 release-note
Remove the @stoplight/spectral-rulesets 1.22.2 pin now that upstream has shipped the null-guard fix (1.22.7)

Additional documentation

This section can be blank.

docs

Upstream shipped the null-guard fix (stoplightio/spectral#2960) in
1.22.7, so the 1.22.2 pin from camaraproject#329 is no longer needed. Lockfile now
resolves and hoists a single 1.22.7 copy.
@hdamker
hdamker requested review from Kevsy and rartych as code owners August 5, 2026 09:34
@hdamker
hdamker merged commit efe16ce into camaraproject:main Aug 5, 2026
8 checks passed
@hdamker hdamker added this to the v0.9.0 milestone Aug 6, 2026
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.

Remove @stoplight/spectral-rulesets pin once upstream null-crash fix ships

1 participant