Skip to content

fix(client): remove caller abort listener after successful fetch#1835

Open
MukundaKatta wants to merge 1 commit intoopenai:masterfrom
MukundaKatta:codex/openai-node-clean-abort-listener
Open

fix(client): remove caller abort listener after successful fetch#1835
MukundaKatta wants to merge 1 commit intoopenai:masterfrom
MukundaKatta:codex/openai-node-clean-abort-listener

Conversation

@MukundaKatta
Copy link
Copy Markdown

Summary

  • remove the forwarded abort listener from the caller signal once fetchWithTimeout completes
  • avoid leaving an orphaned listener attached to successful requests
  • add regression coverage for listener cleanup on the success path

Testing

  • npm test -- --runInBand tests/index.test.ts --testNamePattern="custom signal|removes the caller abort listener"
  • npx prettier --check src/client.ts tests/index.test.ts

@MukundaKatta MukundaKatta requested a review from a team as a code owner April 21, 2026 06:08
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