Skip to content

Commit c9fab68

Browse files
committed
fix(config-builder): fix broken TOC link and simplify mergeAbortSignals early-abort
- Fix README TOC: change #how-mergesignals-works to #how-mergeabortsignals-works to match the actual heading anchor - Simplify mergeAbortSignals: return primarySignal directly when it's already aborted instead of creating a new AbortController
1 parent e737436 commit c9fab68

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

src/api/providers/config-builder/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ A generic, SDK-agnostic request configuration builder that provides a fluent API
1515
- [Deep Dive: Abort Signal Handling](#deep-dive-abort-signal-handling)
1616
- [Why Abort Signals Matter](#why-abort-signals-matter)
1717
- [How `addAbortSignal` Works](#how-addabortsignal-works)
18-
- [How `mergeAbortSignals` Works](#how-mergesignals-works)
18+
- [How `mergeAbortSignals` Works](#how-mergeabortsignals-works)
1919
- [Generic Design](#generic-design)
2020
- [Multi-SDK Usage Examples](#multi-sdk-usage-examples)
2121
- [Extending for Your SDK](#extending-for-your-sdk)

src/api/providers/config-builder/request-config-builder.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,7 @@ export class RequestConfigBuilder<TOptions extends Record<string, any> = Record<
124124
const controller = new AbortController()
125125

126126
if (primarySignal.aborted) {
127-
controller.abort()
128-
return controller.signal
127+
return primarySignal
129128
}
130129

131130
primarySignal.addEventListener("abort", () => controller.abort(), { once: true })

0 commit comments

Comments
 (0)