Confidence Score: 5/5

[Medium impact] The changes appear safe to merge.

_{Reviews (4) · Last reviewed commit: "test(manufacturing): check visualizer ro..." · Reviewed by Greptile}

Verdict: Approve

Reproduced on develop: plan for 10, BOM with 10% process loss, WO completed with 9 produced / 1 lost. Visualizer shows 90% and 1 pending. With this PR, 100% and 0 pending, in both console and browser. A replacement WO for the lost unit also stays at 100%. The visualizer is develop-only, so no backport.

Pushed:

• 91bb5da: reuse get_produced_and_pending_qty from the Production Plan Summary report instead of copying its unreplaced-loss logic twice. Row completion in JS now comes from pending_qty, so the extra unreplaced_loss_qty field is gone.
• 6c70d10: the finished-good row still said "In progress" at 100%. It now uses pending_qty for the "Completed" label.
• de675b8, b71ce74: plan completion is now worked out from all of the plan's Work Orders, not just the ones the user can read. Otherwise a user who can't read some or all Work Orders saw 0%, 90% or 110% for a fully done plan.
• 90fd5b0, 0b9c328: trimmed the test to one row-level case. The replacement cases are already covered by the report tests.

@Mergifyio queue

Merge Queue Status

• ✅ Entered queue — 2026-10-21 12:35 UTC · Rule: default · triggered by @mihir-kandoi with the @mergifyio queue command
• ❌ Checks failed (attempt 1/2) — 2026-10-21 12:46 UTC · on draft merge queue: checking #61734 + #61773 + #61765 + #61737 together on develop (f01723b) #61787
  • ❌ Python Unit Tests (3) (job log)
• ❌ Checks failed (attempt 2/2) — 2026-10-21 13:00 UTC · on draft merge queue: checking #61734 + #61773 + #61765 + #61737 together on develop (f01723b) #61788
  • ❌ Python Unit Tests (3) (job log)
• ✂️ Bisecting to identify the failing PR (round 1/2)
• ❌ Checks failed (attempt 1/2) — 2026-10-21 13:16 UTC · on draft merge queue: checking #61734 + #61773 together on develop (f01723b) #61789
  • ❌ Python Unit Tests (3) (job log)
• ❌ Checks failed (attempt 2/2) — 2026-10-21 13:34 UTC · on draft merge queue: checking #61734 + #61773 together on develop (f01723b) #61793
  • ❌ Python Unit Tests (3) (job log)
• ✂️ Bisecting to identify the failing PR (round 2/2)
• ✅ Checks passed · on draft merge queue: checking #61773 + #61765 + #61737 together on develop (f01723b) #61798
• ✅ Merged — 2026-10-21 14:35 UTC · at a450b5833c63592b40fa660760b4694e427c9df7 · merge

This pull request spent 1 hour 59 minutes 46 seconds in the queue, including 1 hour 15 minutes 39 seconds running CI.