Skip to content

fix: use scoped package name for spectral npx invocation#22

Merged
slusset merged 1 commit intomainfrom
fix/spectral-cli-npx-resolution
Mar 7, 2026
Merged

fix: use scoped package name for spectral npx invocation#22
slusset merged 1 commit intomainfrom
fix/spectral-cli-npx-resolution

Conversation

@slusset
Copy link
Owner

@slusset slusset commented Mar 7, 2026

Summary

  • npx spectral lint in the idd-check action resolves to the wrong npm package (spectral@0.0.0) instead of @stoplight/spectral-cli, causing OpenAPI linting to fail for downstream consumers
  • Changed to npx @stoplight/spectral-cli lint which correctly resolves the scoped package already installed on the preceding line

Test plan

  • Consumer repo running idd-check action no longer gets sh: 1: spectral: not found
  • OpenAPI linting step runs successfully when .spectral.yaml and api.yaml are present

🤖 Generated with Claude Code

`npx spectral` resolves to the wrong npm package (spectral@0.0.0)
instead of @stoplight/spectral-cli, causing idd-check to fail for
downstream consumers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions
Copy link

github-actions bot commented Mar 7, 2026

IDD Compliance Review

Check Status Errors Warnings
Traceability 0 0
Front-Matter 0 0
Capability Scope 0 0
Fixtures 0 0
Models 0 3
Journey Maps 0 0
OpenAPI Lint 0
Gherkin Lint 0
✅ Traceability

Info

  • Found 1 persona file(s)
  • Checked 1 journey file(s)
  • Checked 2 story file(s)
  • Checked 1 feature file(s)
  • Checked 2 API operation(s) across 1 contract file(s)
  • Checked 1 fixture file(s)
  • Checked 1 journey map(s)
  • Checked 1 model file(s)
✅ Front-Matter

Info

  • Checked 9 spec file(s): 9 passed, 0 with issues
✅ Capability Scope

Info

  • Found 1 capability file(s): trade-show-signup
  • Scope coverage: 5/5 files covered
✅ Fixtures

Info

  • specs/fixtures/mobile-signup.fixture.yaml: valid
  • Validated 1 fixture file(s)
✅ Models

Warnings

  • specs/models/account.model.yaml: Entity should have a description
  • specs/models/account.model.yaml: Entity should define identity field
  • specs/models/account.model.yaml: Rule "account-activates-on-signup" should have a description

Info

  • Validated 1 model file(s)
✅ Journey Maps

Info

  • specs/journey-maps/trade-show-signup.journey-map.yaml: valid
  • Validated 1 journey map file(s)
✅ OpenAPI Lint
No results with a severity of 'error' found!

Total: 0 error(s), 3 warning(s)

Automated by IDD Toolkit

@slusset slusset merged commit a8ba370 into main Mar 7, 2026
1 check passed
@slusset slusset deleted the fix/spectral-cli-npx-resolution branch March 7, 2026 21:27
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