[Mikola Lysenko (mikolalysenko)]

Problem

test-release (2) failed in the full-scope run 39745825228 (job 119714465566). That run made ci-ok red for #1663's head. The failing test was cargo_hosted_refuses_a_lockless_project_with_other_dependencies, and the message gave no cause:

panicked at crates/socket-patch-cli/tests/e2e_redirect_cargo_shapes.rs:977:9:
cannot lock cfg-if@1.0.4:
<the generated Cargo.lock, with cfg-if 1.0.5>

The project has no lockfile, so generate-lockfile resolves the newest cfg-if (1.0.5). pin_patched_versions then runs cargo update -p cfg-if@1.0.5 --precise 1.0.4. That command returned non-zero, but its stderr was dropped. The pin works fine locally, and the full suite passes (33/33). So the likely cause is a transient index fetch in the fresh CARGO_HOME, the same blip class as #1459 and #1654. The log can't confirm it.

The same suite runs in coverage, which gates ci-ok and the merge queue. There, one such blip during fixture setup evicts the queue entry.

Root cause

Fixture setup already soft-skips when cargo can't reach crates.io: a failed generate-lockfile and a failed baseline cargo build both go through cargo_e2e_matrix::skip. The --precise pin is the same kind of network-dependent setup step, but it hard-asserted and dropped cargo's error.

Fix

pin_patched_versions now returns Err carrying each failed cargo update's stderr, and run_shape passes that to cargo_e2e_matrix::skip:

• Optional legs (coverage, test-release) skip, with the cause printed.
• The required lock-v* legs (SOCKET_PATCH_CARGO_E2E_REQUIRED=1, ci.yml:2397-2410, which run this suite) still fail on it. The panic now includes cargo's error.

This complements #1654, which widens CARGO_NET_RETRY under CI. That change makes blips rarer; this one makes a remaining blip skip on optional legs and makes any failure readable. The two PRs touch different files.

Proof

• cargo test -p socket-patch-cli --test e2e_redirect_cargo_shapes: 33 passed.
• A cargo shim on PATH that fails only update with a simulated download error:
  • Optional mode: SKIP e2e_redirect_cargo_shapes (lockless-other-dependencies): cannot lock cfg-if@1.0.4 (crates.io unreachable?): \cargo update -p cfg-if@1.0.5`: error: failed to download ... (simulated blip)`, and the test passes as a skip.
  • SOCKET_PATCH_CARGO_E2E_REQUIRED=1: the test panics with the same text plus (and SOCKET_PATCH_CARGO_E2E_REQUIRED is set, so this leg must not skip).
• rustfmt was run on the touched file. cargo clippy -p socket-patch-cli --test e2e_redirect_cargo_shapes -- -D warnings is clean.

No tests were removed or moved. Every shape still runs everywhere it ran before.

https://claude.ai/code/session_01WKXDFSSDxDgdAGiUGLKWQB

---

Generated by Claude Code

[Mikola Lysenko (mikolalysenko)]

bugbot run

---

Generated by Claude Code

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 07c2698. Configure here.}

[Mikola Lysenko (mikolalysenko)]

Labeled Ready for review at 07c26982666de3666ff43691d712c2fec343482f.

• CI: 35/35 green, 19 skipped (ci-ok success). No main-wide failures.
• Bugbot: reviewed 07c2698, no findings.
• Reviewer note: one test file only (e2e_redirect_cargo_shapes.rs). A failed cargo update --precise during fixture setup now skips on optional legs with cargo's stderr; required lock-v* legs still fail on it.
• Slack not announced: no Slack send tool was available in this run.

---

Generated by Claude Code

[Mikola Lysenko (mikolalysenko)]

Final review brief

What it does. Test-only change. In the cargo "shapes" e2e suite, a failed cargo update --precise during fixture setup used to hard-assert and throw away cargo's stderr. Now it returns the error with cargo's stderr and goes through the existing cargo_e2e_matrix::skip. Optional legs (coverage, test-release) skip with the cause printed. Required lock-v* legs still fail, and their failure now says why.

Risk: low. One test file. No product code changes. It reuses the skip path that generate-lockfile and the baseline build already take, so required legs keep full coverage.

Look here

• crates/socket-patch-cli/tests/e2e_redirect_cargo_shapes.rs#L466-L469:`` the new skip on a pin error, the same as the neighbouring call sites at L457 and L479.
• crates/socket-patch-cli/tests/e2e_redirect_cargo_shapes.rs#L955:``` pin_patched_versions now returns `Result`.
• crates/socket-patch-cli/tests/e2e_redirect_cargo_shapes.rs#L981-L992:```` stderr from each failed attempt is collected and returned only when no candidate pinned.

Verified

• Read the full diff and the description, which matches the code.
• cargo_e2e_matrix::skip is used the same way (let _ =) at the other two setup-skip sites.
• stderr() is the file's existing helper (L172).
• .any() still short-circuits on the first successful pin.
• Scope is one file. No debug leftovers. CHANGELOG.md is untouched.
• CI on 07c26982f1c4: 35/35 green (19 skipped by design). Bugbot: success.
• Not re-run locally: the suite needs crates.io. The author's shim proof is in the description.

Changes I made: none.

Open questions: none blocking. One nit: when several candidates fail, their stderr strings are joined without a separator. This affects only the readability of a rare message.

Auto-merge is armed, so approving sends this straight to the merge queue.

---

Generated by Claude Code