refactor: вынести подготовку прогона из _collect_one - #50
Conversation
axisrow
left a comment
There was a problem hiding this comment.
Changes requested. The fresh-run preparation regresses manifest provenance: source_url is captured before the requested phrase is entered, so this PR is not ready until it is captured after _set_phrase (and region setup) or refreshed before the first manifest write. No other high-confidence findings.
axisrow
left a comment
There was a problem hiding this comment.
Changes requested. The URL regression is fixed, but fresh-run updated_at still does not follow the manifest timestamp contract. No other high-confidence findings.
axisrow
left a comment
There was a problem hiding this comment.
Approved. The current head fixes the fresh-run URL provenance issue and now timestamps the post-setup manifest write with updated_at=datetime.now(UTC). I found no remaining high-confidence correctness, security, or error-handling issues in the diff.
Summary
_prepare_run_PreparedRunstate to_collect_oneCloses #44
Tests
ruff check .pytest -q(190 passed)No tests changed.