Skip to content

fix: validate dynamic reply channel addresses - #1244

Closed
be-student wants to merge 1 commit into
asyncapi:next-major-specfrom
be-student:codex/876-reply-channel-address
Closed

be-student wants to merge 1 commit into
asyncapi:next-major-specfrom
be-student:codex/876-reply-channel-address

Conversation

@be-student

Copy link
Copy Markdown

Closes #876.

Adds a v3 core Spectral rule for dynamic reply addresses. When reply.address is present, the referenced channel must leave address null or undefined. The validator follows local component-reference chains and reports the error at the reply channel reference.

Validation:

  • npm run lint
  • npm run build:esm
  • npm run test:unit -- --runInBand on Node 20: 2,446 tests passed

Implemented and tested with Codex.

@changeset-bot

changeset-bot Bot commented Sep 6, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5a88483

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.

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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

sonarqubecloud Bot commented Sep 6, 2026

Copy link
Copy Markdown

@be-student be-student closed this Sep 12, 2026
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