Skip to content

fix: resolve npx double package install failure - #848

Open
Andris28 wants to merge 7 commits into
mainfrom
fa/ruleset-is-not-object
Open

fix: resolve npx double package install failure#848
Andris28 wants to merge 7 commits into
mainfrom
fa/ruleset-is-not-object

Conversation

@Andris28

@Andris28 Andris28 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

PR summary

In this commit we fix an issue where installing the
validator with npx would cause ruleset is not an object
failure, due to spectral core version mismatch.

PR Checklist

General checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Dependencies have been updated as needed
  • .secrets.baseline has been updated as needed
  • npm run update-utilities has been run if any files in packages/utilities/src have been updated

Checklist for adding a new validation rule:

  • Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
  • If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
  • Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
  • Added tests for new rule (packages/ruleset/test/*.test.js)
  • Added docs for new rule (docs/ibm-cloud-rules.md)
  • Added scoring rubric entry for new rule (packages/validator/src/scoring-tool/rubric.js)

Andris28 and others added 7 commits August 7, 2026 12:30
Signed-off-by: András Felleg <afelleg@gmail.com>
Signed-off-by: András Felleg <afelleg@gmail.com>
Signed-off-by: András Felleg <afelleg@gmail.com>
Signed-off-by: András Felleg <afelleg@gmail.com>
Signed-off-by: András Felleg <afelleg@gmail.com>
Signed-off-by: András Felleg <afelleg@gmail.com>
@Andris28 Andris28 changed the title build: add npx install test to build fix: resolve npx double package install failure Aug 10, 2026
@Andris28
Andris28 requested review from diatrcz and pyrooka August 10, 2026 08:44
@Andris28
Andris28 marked this pull request as ready for review August 10, 2026 08:44
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