feat(validator-speclynx): expose OpenAPI namespaces via toolbox#210
feat(validator-speclynx): expose OpenAPI namespaces via toolbox#210
Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughAdds OpenAPI 3.0 and 3.1 ApiDOM namespace modules to the SpecLynx backend toolbox, updates plugin documentation to list those dependencies, and removes several unused apidom-related dependencies from the package.json. No runtime logic changes beyond wiring the namespaces into the toolbox. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the SpecLynx-based OpenAPI validator backend to expose the OpenAPI 3.0 and 3.1 ApiDOM namespace packages via the plugin toolbox deps, and trims the embedded Node resources package.json dependencies accordingly.
Changes:
- Add
@speclynx/apidom-ns-openapi-3-0and@speclynx/apidom-ns-openapi-3-1to the toolboxdepsmap in the SpecLynx CLI resource. - Update plugin documentation/comments (JS fixtures, built-in plugins, sample plugin, and Python backend docstring) to list the newly available dependencies.
- Remove unused SpecLynx packages from the embedded resources
package.jsonand reflect that inpackage-lock.json.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/jentic-openapi-validator-speclynx/tests/fixtures/plugins/version-validator.mjs | Updates fixture plugin doc comment to include the new namespace deps. |
| packages/jentic-openapi-validator-speclynx/src/jentic/apitools/openapi/validator/backends/speclynx/resources/speclynx.mjs | Imports OpenAPI namespace packages and exposes them through toolbox.deps. |
| packages/jentic-openapi-validator-speclynx/src/jentic/apitools/openapi/validator/backends/speclynx/resources/plugins/openapi-document.mjs | Updates plugin doc comment to reflect new deps available in toolbox. |
| packages/jentic-openapi-validator-speclynx/src/jentic/apitools/openapi/validator/backends/speclynx/resources/plugins/example-plugin.mjs.sample | Updates sample plugin documentation of deps contents. |
| packages/jentic-openapi-validator-speclynx/src/jentic/apitools/openapi/validator/backends/speclynx/resources/package.json | Removes unused SpecLynx dependencies; keeps required ones incl. OpenAPI namespaces. |
| packages/jentic-openapi-validator-speclynx/src/jentic/apitools/openapi/validator/backends/speclynx/init.py | Updates Python backend docstring describing plugin toolbox deps. |
| package-lock.json | Removes lock entries corresponding to removed embedded-resource dependencies. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by CodeRabbit
New Features
Chores
Documentation