[saiyam-jain-dev]

What and why

In BrowserControl._hashed(), computing the size and sha256 digest for a completed download could fail if the downloaded file was removed, moved, or inaccessible. Previously, an exception during hashing skipped download.update(ended_at=...), leaving the completed download without an ended_at timestamp.

This change:

• Wraps hashing in a try/finally block so ended_at is always set once a download has completed, even if computing size or digest raises an error.
• Adds a unit test in test_website_browser.py asserting that ended_at is recorded when download file hashing fails.

How it was tested

• Ran pytest on packages/agentenv-protocol/tests/test_website_browser.py for control download record tests.
• Ran ruff check on control.py and test_website_browser.py.
• Ran check_plugin_api.py against origin/main (no plugin surface changes).

Confidence Score: 5/5

The PR appears safe to merge.

Summary

Completed downloads now keep ended_at when reading the file for its size or digest fails.

• Completed downloads now get an end time when hashing fails.

_{Reviews (3) · Last reviewed commit: "Merge branch 'main' into fix/browser-dow..." · Reviewed by Greptile}

[andrewpark-scaleai]

LGTM. I ran test_website_browser.py (56 passed), and the new test fails against main. The finally still re-raises, so _finished keeps logging the hashing error, and Download already has size/sha256 as optional.

Nit: the Download docstring in shapes.py says a completed download comes with its size and sha256. That's no longer guaranteed, so it's worth a word there.

[saiyam-jain-dev]

Updated the Download docstring in shapes.py to clarify that size and sha256 are included when available.