Skip to content

chore: update @redocly/ajv dependency and temporary pin it#2722

Closed
AlbinaBlazhko17 wants to merge 2 commits intomainfrom
chore/pin-redocly-ajv-version
Closed

chore: update @redocly/ajv dependency and temporary pin it#2722
AlbinaBlazhko17 wants to merge 2 commits intomainfrom
chore/pin-redocly-ajv-version

Conversation

@AlbinaBlazhko17
Copy link
Copy Markdown
Contributor

@AlbinaBlazhko17 AlbinaBlazhko17 commented Apr 6, 2026

What/Why/How?

Hotfix:

  • After release of new ajv v8.18.2 the smoke tests started to fail
  • Temporary pin the dep, will be unpinned in this PR #2689

Reference

Testing

Screenshots (optional)

Check yourself

  • This PR follows the contributing guide
  • All new/updated code is covered by tests
  • Core code changed? - Tested with other Redocly products (internal contributions only)
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update has been considered

Security

  • The security impact of the change has been considered
  • Code follows company security practices and guidelines

Note

Low Risk
Low risk dependency maintenance change, but it affects schema validation behavior by locking both @redocly/ajv and the ajv alias to a specific patch version.

Overview
Updates @redocly/openapi-core and @redocly/respect-core to pin @redocly/ajv (and the ajv npm alias) to 8.18.1 instead of allowing ^8.18.0 ranges, and updates the lockfile accordingly.

Adds a changeset to publish patch releases for both packages reflecting this temporary dependency pin.

Reviewed by Cursor Bugbot for commit e5b19b6. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 6, 2026

🦋 Changeset detected

Latest commit: e5b19b6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@redocly/respect-core Patch
@redocly/openapi-core Patch
@redocly/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 79.92% (🎯 79%) 6685 / 8364
🔵 Statements 79.35% (🎯 79%) 6916 / 8715
🔵 Functions 83.12% (🎯 82%) 1355 / 1630
🔵 Branches 71.61% (🎯 71%) 4546 / 6348
File CoverageNo changed files found.
Generated in workflow #9308 for commit e5b19b6 by the Vitest Coverage Report Action

@AlbinaBlazhko17 AlbinaBlazhko17 changed the title chore: update @redocly/ajv dependency and temporary pin it hotfix: update @redocly/ajv dependency and temporary pin it Apr 6, 2026
@AlbinaBlazhko17 AlbinaBlazhko17 changed the title hotfix: update @redocly/ajv dependency and temporary pin it chore: update @redocly/ajv dependency and temporary pin it Apr 6, 2026
@AlbinaBlazhko17 AlbinaBlazhko17 marked this pull request as ready for review April 6, 2026 16:13
@AlbinaBlazhko17 AlbinaBlazhko17 requested review from a team as code owners April 6, 2026 16:13
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e5b19b6. Configure here.

"license": "MIT",
"dependencies": {
"@redocly/ajv": "^8.18.0",
"@redocly/ajv": "8.18.1",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CLI package ajv dependency not updated to 8.18.1

Medium Severity

The @redocly/cli package still has "ajv": "npm:@redocly/ajv@8.18.0" while packages/core and packages/respect-core were both updated to npm:@redocly/ajv@8.18.1. This creates a version inconsistency across the monorepo — the cli package resolves its own ajv to 8.18.0 (confirmed in the lockfile) while its transitive dependencies openapi-core and respect-core use 8.18.1. This was likely an oversight when applying the version update and pin.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e5b19b6. Configure here.

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.

1 participant