Skip to content

Conversation

@vadyvas
Copy link

@vadyvas vadyvas commented Oct 28, 2025

What issue does this pull request resolve?

  • Update ajv to the latest version.

What changes did you make?

  • sync with origin ajv
  • rename build action to build & test
  • Change publish flow from NPM_TOKEN to trusted publishing

Is there anything that requires more attention while reviewing?

epoberezkin and others added 30 commits October 28, 2025 11:40
There are already console logs in the parseAndLog function itself. parseAndLog(json) and parseAndLog(invalidJson) will just print 2 undefined in plus
* update the documentation of addKeyword args

* add the

* corrections

Co-authored-by: Evgeny Poberezkin <[email protected]>
…or#2072)

* add Non-standard JTD keywords warn

* correction

* correction

Co-authored-by: Evgeny Poberezkin <[email protected]>
Bumps [husky](https://github.com/typicode/husky) from 7.0.4 to 8.0.2.
- [Release notes](https://github.com/typicode/husky/releases)
- [Commits](typicode/husky@v7.0.4...v8.0.2)

---
updated-dependencies:
- dependency-name: husky
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
…r#2079)

nullable was enforced for optional parameters, but not forbidden for
required parameters. This tests and enforces the latter case.

fixes ajv-validator#2030

Co-authored-by: Evgeny Poberezkin <[email protected]>
* Clarify validation errors and fix link for useDefaults.

* Make it clear the value, not the property, may be missing.

* remove correction

Co-authored-by: Evgeny Poberezkin <[email protected]>
…jv-validator#2142)

Bumps [@rollup/plugin-commonjs](https://github.com/rollup/plugins/tree/HEAD/packages/commonjs) from 21.1.0 to 23.0.2.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/commonjs/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/commonjs-v23.0.2/packages/commonjs)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-commonjs"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
Bumps [fast-uri](https://github.com/fastify/fast-uri) from 1.0.1 to 2.1.0.
- [Release notes](https://github.com/fastify/fast-uri/releases)
- [Commits](fastify/fast-uri@v1.0.1...v2.1.0)

---
updated-dependencies:
- dependency-name: fast-uri
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Connor Smith <[email protected]>
Co-authored-by: Evgeny Poberezkin <[email protected]>
I made a mistake with one of the code snippets in the documentation. The 'regExp' attribute belongs to the code options, not the ajv options directly.

Co-authored-by: Evgeny Poberezkin <[email protected]>
….1 (ajv-validator#2143)

Bumps [@rollup/plugin-node-resolve](https://github.com/rollup/plugins/tree/HEAD/packages/node-resolve) from 13.3.0 to 15.0.1.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/node-resolve/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/node-resolve-v15.0.1/packages/node-resolve)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-node-resolve"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
Bumps [mocha](https://github.com/mochajs/mocha) from 9.2.2 to 10.0.0.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](mochajs/mocha@v9.2.2...v10.0.0)

---
updated-dependencies:
- dependency-name: mocha
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
…r#2019)

Bumps [lint-staged](https://github.com/okonet/lint-staged) from 12.5.0 to 13.0.3.
- [Release notes](https://github.com/okonet/lint-staged/releases)
- [Commits](lint-staged/lint-staged@v12.5.0...v13.0.3)

---
updated-dependencies:
- dependency-name: lint-staged
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
…closes ajv-validator#1840 (ajv-validator#2020)

* expost Validation Error - fix to ajv-validator#1926

* fixed prettier issues

Co-authored-by: Evgeny Poberezkin <[email protected]>
…lidator#2159)

Bumps [@rollup/plugin-json](https://github.com/rollup/plugins/tree/HEAD/packages/json) from 4.1.0 to 5.0.1.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/json/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/url-v5.0.1/packages/json)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-json"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#2001, fixes ajv-validator#2171, fixes ajv-validator#2181 (ajv-validator#2190)

* fixes ajv-validator#2001

fixes ajv-validator#2001

* reduce diff

* prettier

* JTD only optional properties test

* Test case

* simplify, avoid run time code changes when possible

* fix test

* style

* style

* prettier

Co-authored-by: Anton Piliugin <[email protected]>
Co-authored-by: Anton Piliugin <[email protected]>
…ajv-validator#2192)

* Add failing test for integer subschema narrowing

* Add number to includesType check for context types

* narrow number to integer correctly

* fix lint errors

Co-authored-by: Jacob Ley <[email protected]>
…jv-validator#2160)

Bumps [@rollup/plugin-typescript](https://github.com/rollup/plugins/tree/HEAD/packages/typescript) from 8.5.0 to 9.0.2.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/typescript/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/typescript-v9.0.2/packages/typescript)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-typescript"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
…lidator#2183)

Bumps [@rollup/plugin-json](https://github.com/rollup/plugins/tree/HEAD/packages/json) from 5.0.2 to 6.0.0.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/json/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/url-v6.0.0/packages/json)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-json"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Poberezkin <[email protected]>
…jv-validator#2193)

Bumps [@rollup/plugin-typescript](https://github.com/rollup/plugins/tree/HEAD/packages/typescript) from 9.0.2 to 10.0.1.
- [Release notes](https://github.com/rollup/plugins/releases)
- [Changelog](https://github.com/rollup/plugins/blob/master/packages/typescript/CHANGELOG.md)
- [Commits](https://github.com/rollup/plugins/commits/typescript-v10.0.1/packages/typescript)

---
updated-dependencies:
- dependency-name: "@rollup/plugin-typescript"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jasoniangreen and others added 11 commits October 28, 2025 11:55
As it's name is so different and not just `addKeywords`, there has been some confusion as to it's behaviour.
The use of "not more" with the reverse ordering of the sentence made
this difficult to parse. After reversing the order and removing
negation, the surrounding paragraphs were updated to match the style.

Co-authored-by: Jason Ian Green <[email protected]>
"there is many" -> "there are many"

Co-authored-by: Jason Ian Green <[email protected]>
* test: add encoded ref test

* bump fast-uri

* remove .only
…ator#2487)

* fix: ajv-validator#2482 Infinity and NaN serialise to null

* feat: add safeNumbers option

* include test for default behavior

* change option name to specialNumbers

* refactor to be more dry

* pr feedback

* remove string option
@vadyvas vadyvas marked this pull request as ready for review October 28, 2025 10:58
@vadyvas
Copy link
Author

vadyvas commented Oct 28, 2025

Ran the test suite locally for CLI and across the monorepo for all workspaces that depend on @redocly/cli, @redocly/ajv, and @redocly/openapi-core. Tests executed without errors on my side

@vadyvas
Copy link
Author

vadyvas commented Oct 28, 2025

@RomanHotsiy could you please configure npm Trusted Publisher for this repo

RomanHotsiy
RomanHotsiy previously approved these changes Nov 3, 2025
"@types/require-from-string": "^1.2.3",
"@typescript-eslint/eslint-plugin": "^7.3.1",
"@typescript-eslint/parser": "^7.3.1",
"ajv-formats": "^3.0.1",
Copy link

Choose a reason for hiding this comment

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

Make sure to update the corresponding package in Redocly CLI and further when updating the fork there.

@tatomyr tatomyr changed the title Chore/sync with origin chore: sync with origin Nov 4, 2025
@vadyvas vadyvas merged commit 7109c71 into master Nov 6, 2025
7 checks passed
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.