[vdusek]

In shared mode, fetch_next_request returned None when get_request couldn't find a request that list_and_lock_head had just locked. It also removed the ID from the local queue head but kept the 3-minute platform lock. Later head listings skip locked requests, so neither this client nor any other consumer could fetch the request until the lock expired.

This showed up as a flake of test_shared_is_finished_false_while_request_in_progress (run). It polled fetch_next_request for 30 s and never got the request. #1105 added that polling, but polling can't get past a 3-minute lock.

Both not-found paths now put the ID back on the local queue head, so the next fetch retries it. If the request never shows up, the lock check in _ensure_lock_window drops it, so it isn't retried forever. Two unit tests cover the paths: the plain cache hit, and a request evicted from the cache.

✍️ Drafted by Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.50%. Comparing base (5fbbf76) to head (c172de8).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1207      +/-   ##
==========================================
+ Coverage   93.09%   93.50%   +0.40%     
==========================================
  Files          54       54              
  Lines        3708     3710       +2     
==========================================
+ Hits         3452     3469      +17     
+ Misses        256      241      -15     

Flag	Coverage Δ
e2e	31.56% <0.00%> (-0.02%)	⬇️
integration	51.56% <33.33%> (-0.03%)	⬇️
unit	90.61% <100.00%> (+0.73%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.