Skip to content

[9.4] [Zod Helper][OAS Docs] Fix OAS docs generation for routes using buildRouteValidationWithZod (#263354)#264675

Merged
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/pr-263354
Apr 21, 2026
Merged

[9.4] [Zod Helper][OAS Docs] Fix OAS docs generation for routes using buildRouteValidationWithZod (#263354)#264675
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/pr-263354

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.4:

Questions ?

Please refer to the Backport tool documentation

…RouteValidationWithZod (elastic#263354)

## Summary

When working on <elastic#262554> we have
discovered both path and query parameters missing from generated OAS
docs, despite being defined in the route itself. I have tracked the
issue down to `buildRouteValidationWithZod` function obscuring the Zod
schema.

This PR aims to fix this issue by attaching the original, source schema
to the output of `buildRouteValidationWithZod` and adding code to try
and unwrap it in the `kbn-router-to-openapispec` code. This has no
influence on existing code, other than correctly handling the routes
with `buildRouteValidationWithZod`.

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit f50036c)
@kibanamachine kibanamachine force-pushed the backport/9.4/pr-263354 branch from 8f1c333 to 36f46f1 Compare April 21, 2026 12:01
@elasticmachine
Copy link
Copy Markdown
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #23 / Endpoint plugin @ess @serverless @skipInServerlessMKI Endpoint policy response api GET /api/endpoint/policy_response "before all" hook for "should return one policy response for an id"
  • [job] [logs] FTR Configs #23 / Endpoint plugin @ess @serverless @skipInServerlessMKI Endpoint policy response api GET /api/endpoint/policy_response "before all" hook for "should return one policy response for an id"

cc @kubasobon

@kibanamachine kibanamachine merged commit 9431ca2 into elastic:9.4 Apr 21, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants