Skip to content

[bug] E4-5: retryWithJitter has no default shouldRetry β€” auth failures retriedΒ #1430

@OneStepAt4time

Description

@OneStepAt4time

Finding

ID: ERR-4 | Severity: 🟠 MEDIUM

retryWithJitter has no default shouldRetry. If a caller omits it, every error retries β€” including auth failures and validation errors.

Fix Required

Add a default shouldRetry that calls error-categories.shouldRetry(categorize(err)). Document and lint enforcement.

Files: src/retry.ts, src/error-categories.ts

Acceptance Criteria

A retry of a 400 validation error stops after the first attempt with the default policy.

Milestone

M-E4: Reliability & Robustness

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2bugSomething isn't workingpriority: mediumreadyTriaged and ready for an agent to pick up

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions