Fixed in 00a7b82, the same finding as on f0b4e4a (this review was of 15dd358): the usage goes to stderr after each call and survives a timeout.
Fixed in 00a7b82, the same finding as on f0b4e4a (this review was of 15dd358): diagram_calls > 0 without diagram_cost_usd makes the review cost unknown.
Fixed in 00a7b82: a drawing that cost something but attached no diagram gets the section "Diagrams" with "No diagram was attached." and the cost line. Test: `test_a_drawing_without_a_diagram_name…
Fixed in 00a7b82: the comment of Deps.draw_diagrams names the new contract, prlens.Drawing and prlens.DiagramError with its usage; all callers return a Drawing.
Fixed in 00a7b82: a record with diagram_calls > 0 and no diagram_cost_usd makes the cost of the review unknown, so it counts in unknownCostReviews. Test: Scenario#140.03 in `UsageApiIntegra…
Fixed in 00a7b82: pr_lens.mjs writes the usage to stderr after each call; on a timeout prlens.draw reads the last line from the killed process. Test: `test_a_killed_pr_lens_names_the_usage_of…
Fixed in 00a7b82: the wrapper counts the call before fetch, and a rejected fetch makes the cost unknown. Test: test_a_call_without_an_answer_counts_with_an_unknown_cost.
Fixed in 15dd358: the deliberate paragraph is removed; the test of the release moves to a simpler PR.
Answer to the review of f0b4e4a:
- YELLOW "tests lost or disabled: tests/test_pr_lens_script.py:62": the new test class is skipped only without Node.js. The PR-Agent image of the documented…