Expose onRequest in RequestOptions for SSE request interception#3392
Expose onRequest in RequestOptions for SSE request interception#3392
Conversation
|
|
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
|
@copilot update snapshots and examples |
Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
Updated all snapshots and examples. Changes include:
Commit: 06f773f Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot CI checks are failing, figure out why and fix them |
Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
Fixed the CI failures. The issue was that test snapshots for client-nuxt and client-ofetch had Removed All CI checks now pass:
Commit: 0bc7a8b Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3392 +/- ##
=======================================
Coverage 39.59% 39.59%
=======================================
Files 473 473
Lines 17080 17080
Branches 5230 5224 -6
=======================================
Hits 6763 6763
Misses 8266 8266
Partials 2051 2051
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Plan: Add
onRequesttoRequestOptionsTypeonRequestto the Pick statement in each RequestOptions interfaceFiles Updated:
/packages/openapi-ts/src/plugins/@hey-api/client-fetch/bundle/types.ts/packages/openapi-ts/src/plugins/@hey-api/client-axios/bundle/types.ts/packages/openapi-ts/src/plugins/@hey-api/client-ky/bundle/types.ts/packages/openapi-ts/src/plugins/@hey-api/client-next/bundle/types.ts/packages/openapi-ts/src/plugins/@hey-api/client-angular/bundle/types.ts/packages/openapi-ts/src/plugins/@hey-api/client-nuxt/bundle/types.ts- Already hasonRequestfrom FetchOptions/packages/openapi-ts/src/plugins/@hey-api/client-ofetch/bundle/types.ts- Already hasonRequestfrom OfetchOptionsExamples and Snapshots Updated:
onRequestexposedonRequestin the Pick statementonRequestto avoid conflicts)Summary:
Added
onRequestto theRequestOptionsinterface's Pick statement for 5 client bundles (fetch, axios, ky, next, angular). The client-nuxt and client-ofetch bundles were excluded from havingonRequestin the Pick statement because they already have it in their Config interfaces. All examples and test snapshots have been updated accordingly.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.