fix(coprocessor): do not update is_completed on unallowed handles

[antoniupop]

No description provided.

[mergify]

🧪 CI Insights

Here's what we observed from your CI run for c1421d9.

🟢 All jobs passed!

But CI Insights is watching 👀

[Copilot]

Pull request overview

This PR fixes an issue where the is_completed flag was being incorrectly set for unallowed computation handles. The fix ensures that only successfully computed handles (which are allowed) have their is_completed status updated, and sets is_completed = true immediately for unallowed computations when they are inserted into the database.

Key changes:

• Modified handle tracking logic to only mark successfully computed handles as complete
• Added is_completed column to the computation insertion SQL with logic to set it based on is_allowed status
• Wrapped database operations in conditional blocks to avoid unnecessary queries when there's no data to process

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
coprocessor/fhevm-engine/tfhe-worker/src/tfhe_worker.rs	Changed handles_to_update initialization from including intermediate and unneeded handles to only including successfully computed results; added conditional guards around database operations
coprocessor/fhevm-engine/host-listener/src/database/tfhe_event_propagate.rs	Added is_completed column to computation INSERT statement, setting it to !log.is_allowed to immediately mark unallowed computations as complete

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[antoniupop]

No longer relevant