Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a --harden option to the lint command to close down schemas #222

Open
jviotti opened this issue Feb 6, 2025 · 2 comments
Open

Add a --harden option to the lint command to close down schemas #222

jviotti opened this issue Feb 6, 2025 · 2 comments

Comments

@jviotti
Copy link
Member

jviotti commented Feb 6, 2025

Like enforcing additionalProperties: false, etc

@cbum-dev
Copy link

Hi @jviotti can I work on this issue?

@jviotti
Copy link
Member Author

jviotti commented Feb 27, 2025

You can, but keep in mind that before even working on the CLI, we need the specific linting rules that would be enabled, which should be implemented here: https://github.com/sourcemeta/core/tree/main/src/extension/alterschema. And to do that, we first need to determine what the hardened rules even are...

I guess just forcing additionalProperties: false if additionalProperties is not used (unless unevaluatedProperties in 2019-09 and 2020-12) is a good one. I would expect a new harden folder in https://github.com/sourcemeta/core/tree/main/src/extension/alterschema.

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

No branches or pull requests

2 participants