Skip to content

fix: add resolved-document validation rules for v3 channel/server references#1124

Open
Shyam-Raghuwanshi wants to merge 1 commit into
asyncapi:masterfrom
Shyam-Raghuwanshi:fix/spectral-rules-resolved-references-924
Open

fix: add resolved-document validation rules for v3 channel/server references#1124
Shyam-Raghuwanshi wants to merge 1 commit into
asyncapi:masterfrom
Shyam-Raghuwanshi:fix/spectral-rules-resolved-references-924

Conversation

@Shyam-Raghuwanshi
Copy link
Copy Markdown

Fixes #924

Desc

  • Added new 'asyncapi3-required-operation-channel-unambiguity-resolved' rule
    that runs on the resolved document to verify operation channels exist
    in root channels by comparing object references
  • Added new 'asyncapi3-required-channel-servers-unambiguity-resolved' rule
    that runs on the resolved document to verify channel servers exist
    in root servers by comparing object references

These new rules complement the existing unresolved rules to catch both:

  1. Same-file invalid references (caught by pattern matching)
  2. Cross-file invalid references (caught by object identity check)

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 25, 2026

⚠️ No Changeset found

Latest commit: 93270f8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@github-actions
Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions Bot added the stale label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some Spectral Rules for spec v3 should run after resolving references

1 participant