Skip to content

ADR-0002 step-aware retry: withRetry exists and is tested but never wired into the pipeline #50

Description

@Alimedhat000

Summary

withRetry (packages/worker/src/infrastructure/retry.ts) implements per-step retry classification per ADR-0002 and is unit-tested, but no production code calls it.

Evidence

  • Only references to withRetry: infrastructure/retry.ts itself and test/unit/infrastructure/retry.test.ts.
  • packages/worker/src/deployments/pipeline.ts has no retry logic; the only inline retry is the port-conflict loop in strategies/dockerfile.ts:23.

Scope

Wire withRetry into the pipeline steps per ADR-0002:

  • git clone — retry on timeout, fail fast on auth errors (exit 128).
  • npm install / build — retry on network blips, fail fast on 404 / non-zero build.

Or, if the classification isn't wanted, soften the ADR / resume wording instead.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestneeds-triageMaintainer needs to evaluate the issue

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions