Skip to content

feat: add ajv-keywords as devDependency#5728

Open
miteshashar wants to merge 2 commits into
SchemaStore:masterfrom
miteshashar:add-ajv-keywords-dependency
Open

feat: add ajv-keywords as devDependency#5728
miteshashar wants to merge 2 commits into
SchemaStore:masterfrom
miteshashar:add-ajv-keywords-dependency

Conversation

@miteshashar
Copy link
Copy Markdown
Contributor

@miteshashar miteshashar commented May 24, 2026

Summary

  • Adds ajv-keywords to devDependencies in package.json
  • Wires uniqueItemProperties keyword into the AJV factory in cli.js
  • package-lock.json updated accordingly

Why

ajv-keywords was previously only a transitive peer dep. Schemas using uniqueItemProperties (e.g. claude-code-keybindings.json) fail validation without it being explicitly registered. As a build/test-only tool, it belongs in devDependencies.

Dependents

🤖 Generated with Claude Code

Required for uniqueItemProperties keyword support in schema validation.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the cli.js Changes to the build system / JavaScript. label May 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

ya7010 added a commit to ya7010/schemastore that referenced this pull request May 24, 2026
github-actions Bot pushed a commit that referenced this pull request May 24, 2026
… and examples in JSON schema (#5728) (#5729)

Co-authored-by: ya7010 <ya7010@users.noreply.github.com>
@madskristensen
Copy link
Copy Markdown
Contributor

@hyperupcall thoughts?

@miteshashar
Copy link
Copy Markdown
Contributor Author

@hyperupcall Do chime in when you get the chance!

All other ajv-related packages (ajv, ajv-draft-04, ajv-formats) live in
devDependencies — ajv-keywords belongs there too.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@miteshashar
Copy link
Copy Markdown
Contributor Author

Fixed an anti-pattern: ajv-keywords was incorrectly added to dependencies instead of devDependencies. All other ajv-related packages (ajv, ajv-draft-04, ajv-formats) live in devDependencies — moved ajv-keywords to match.

@miteshashar miteshashar changed the title feat: add ajv-keywords as direct dependency fix: add ajv-keywords as devDependency May 31, 2026
@miteshashar miteshashar changed the title fix: add ajv-keywords as devDependency feat: add ajv-keywords as devDependency May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli.js Changes to the build system / JavaScript.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants