Skip to content

fix(openapi): replace repeated path parameters - #363

Open
pacocartones wants to merge 1 commit into
punkpeye:mainfrom
pacocartones:fix/repeated-openapi-path-parameters
Open

fix(openapi): replace repeated path parameters#363
pacocartones wants to merge 1 commit into
punkpeye:mainfrom
pacocartones:fix/repeated-openapi-path-parameters

Conversation

@pacocartones

Copy link
Copy Markdown
Contributor

Summary

  • replace every occurrence of a repeated OpenAPI path parameter
  • reuse the same encoded value for each occurrence
  • add a regression covering repeated placeholders and reserved characters

The current single replacement leaves later occurrences such as /accounts/{id}/mirrors/{id} unresolved. Using replaceAll preserves the existing encoding behavior while substituting every occurrence.

Validation

  • all 20 requestBuilder tests pass
  • type checking, ESLint and Prettier pass

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