fix: enable path parsing to account for root level paths

Redocly / redocly-cli

⚒️ Redocly CLI makes OpenAPI easy. Lint/validate to any standard, generate beautiful docs, and more.

https://redocly.com/docs/cli/

MIT License

912 stars 139 forks source link

fix: enable path parsing to account for root level paths #1638

Open LukeHagar opened 2 months ago

LukeHagar commented 2 months ago

What/Why/How?

Fixes #1634

Reference

Testing

Screenshots (optional)

Check yourself

[ ] Code changed? - Tested with redoc/reference-docs/workflows (internal)
[ ] All new/updated code is covered with tests
[ ] New package installed? - Tested in different environments (browser/node)

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: 9b7f50f9a9421b519646aeedcdde44186cc4b8f2

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

lornajane commented 2 months ago

Thanks for the pull request @LukeHagar , we'll take a look.

tatomyr commented 2 months ago

@LukeHagar do you mind adding a simple e2e test for the case? (The corresponding folder is __tests__/split).

LukeHagar commented 2 months ago

I'd be happy too, I realize this solution would also collide with any API endpoints that are actually /root so I'll solve for that as well.