fix: reported error location at correct node for parameters by kanoru3101 · Pull Request #2626 · Redocly/redocly-cli

kanoru3101 · 2026-03-02T14:09:05Z

What/Why/How?

For parameters referenced via $ref, the error location now points at the $ref usage instead of the component definition.

Reference

#1241

Testing

openapi: 3.1.0
servers:
  - url: https://test-example.com
info:
  title: Repro issue
  version: 0.0.1
  license:
    name: MIT
    url: https://opensource.org/licenses/MIT
security: []
paths:
  /users/data:
    post:
      summary: Query user data
      operationId: queryUserData
      parameters:
        - $ref: "#/components/parameters/header_contentType"
        - $ref: "#/components/parameters/path_userId"
      responses:
        "200":
          description: OK
        "400":
          description: Bad Request

components:
  parameters:
    header_contentType:
      name: content-type
      in: header
      schema:
        type: string
    path_userId:
      name: user-id
      in: path
      required: true
      schema:
        type: string

Screenshots (optional)

Check yourself

Code changed? - Tested with Redoc/Realm/Reunite (internal)
All new/updated code is covered by tests
New package installed? - Tested in different environments (browser/node)
Documentation update considered

Security

The security impact of the change has been considered
Code follows company security practices and guidelines

changeset-bot · 2026-03-02T14:09:31Z

🦋 Changeset detected

Latest commit: 289c664

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@redocly/openapi-core	Patch
@redocly/cli	Patch
@redocly/respect-core	Patch

Not sure what this means? Click here to learn what changesets are.

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

github-actions · 2026-03-02T14:10:35Z

📦 A new experimental 🧪 version v0.0.0-snapshot.1772460631 of Redocly CLI has been published for testing.

Install with NPM:

npm install @redocly/cli@0.0.0-snapshot.1772460631
# or
npm install @redocly/openapi-core@0.0.0-snapshot.1772460631
# or
npm install @redocly/respect-core@0.0.0-snapshot.1772460631

⚠️ Note: This is a development build and may contain unstable features.

github-actions · 2026-03-02T14:11:18Z

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	79.15% (🎯 79%)	6381 / 8061
🔵	Statements	78.6% (🎯 78%)	6593 / 8387
🔵	Functions	82.4% (🎯 82%)	1307 / 1586
🔵	Branches	71.01% (🎯 70%)	4344 / 6117

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
packages/core/src/rules/common/path-params-defined.ts	96.55%	93.75%	92.3%	98.21%	118, 131

Generated in workflow #8911 for commit 289c664 by the Vitest Coverage Report Action

github-actions · 2026-03-02T14:11:31Z

CLI Version	Mean Time ± Std Dev (s)	Relative Performance (Lower is Faster)
cli-latest	3.669s ± 0.089s	▓ 1.02x
cli-next	3.591s ± 0.054s	▓ 1.00x (Fastest)

.changeset/sad-mice-tease.md

packages/core/src/rules/common/__tests__/path-params-defined.test.ts

packages/core/src/rules/common/path-params-defined.ts

tatomyr

Please also add an e2e test with transitive $refs.

Co-authored-by: Andrew Tatomyr <andrew.tatomyr@redocly.com>

…cli into fix/resolve-ref-in-path

tatomyr

Almost there. Please also update the PR description and the test name.

.changeset/sad-mice-tease.md

packages/core/src/rules/common/path-params-defined.ts

.changeset/sad-mice-tease.md

tatomyr

Please adjust the changeset according to this and feel free to merge.

fix: resolve path where is error in path

2199c27

kanoru3101 changed the title ~~fix: resolve path where is error in path~~ fix: reported error location at correct node for parameters Mar 2, 2026

kanoru3101 added the snapshot Create experimental release PR label Mar 2, 2026

kanoru3101 self-assigned this Mar 2, 2026

kanoru3101 added 4 commits March 2, 2026 17:10

Merge remote-tracking branch 'origin/main' into fix/resolve-ref-in-path

ab673b6

fix: updated message

0c1c52c

Merge remote-tracking branch 'origin/main' into fix/resolve-ref-in-path

b0d4db2

fix: update test

1674cbe

kanoru3101 marked this pull request as ready for review March 2, 2026 16:57

kanoru3101 requested review from a team as code owners March 2, 2026 16:57