Skip to content

Switch package to restricted - #3

Merged
HardlyDifficult merged 1 commit into
mainfrom
restricted
Jul 8, 2025
Merged

Switch package to restricted#3
HardlyDifficult merged 1 commit into
mainfrom
restricted

Conversation

@HardlyDifficult

Copy link
Copy Markdown
Collaborator

No description provided.

@HardlyDifficult
HardlyDifficult enabled auto-merge (squash) July 8, 2025 18:55
@HardlyDifficult
HardlyDifficult merged commit f1ba13f into main Jul 8, 2025
1 check passed
@HardlyDifficult
HardlyDifficult deleted the restricted branch July 8, 2025 18:56
cursor Bot pushed a commit that referenced this pull request Aug 5, 2026
…#3)

isTimeoutError treated any axios error with code ETIMEDOUT the same as
our own configured socket-inactivity timeout, making it non-retryable.
Empirically (axios 1.18.1, the version pinned in package.json), the
Node http adapter's createTimeoutError() always uses ECONNABORTED for
our own configured timeout, since we never set
transitional.clarifyTimeoutError (its default is false). A raw
ETIMEDOUT only ever comes from Node's own connect-phase failure (e.g.
'connect ETIMEDOUT <ip>:<port>'), a distinct and genuinely transient
condition that should remain retryable, matching pre-PR behavior for
that failure mode.

isTimeoutError now checks for ECONNABORTED only. Adds a regression test
simulating a connect-phase ETIMEDOUT (code: 'ETIMEDOUT', syscall:
'connect') and asserting it is retried like other transient network
failures.

Co-authored-by: HardlyDifficult <hardlydiff@gmail.com>
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