Skip to content

Conversation

@Katli95
Copy link

@Katli95 Katli95 commented Nov 21, 2025

Changes

Decided to open PR in lieu of issue.

I just changed the .d.ts file manually and extracted the MiddlewareRequestParams into their own interface. It opens up the possibility for using TS module augmentation to add parameters to the params property and use those in the middleware.

My use-case is changing authentication logic based on a variable that isn't passed through to the API being invoked.

How to Review

Look at the file, I guess? 🤗

Also, I would be open to going deeper in a follow-up PR to look into how the client facing types could mirror the custom parameter added, if this is indeed an avenue you're open to exploring.

Checklist

(skipped all since it's just a small TS update and I didn't pull to local)

  • Unit tests updated
  • docs/ updated (if necessary)
  • pnpm run update:examples run (only applicable for openapi-typescript)

@Katli95 Katli95 requested a review from a team as a code owner November 21, 2025 10:00
@Katli95 Katli95 requested a review from htunnicliff November 21, 2025 10:00
@netlify
Copy link

netlify bot commented Nov 21, 2025

👷 Deploy request for openapi-ts pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1671f6f

@changeset-bot
Copy link

changeset-bot bot commented Nov 21, 2025

⚠️ No Changeset found

Latest commit: 1671f6f

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

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