fix(desktop): leave a stopped native file tool's outcome to Stop

[waleedlatif1]

Summary

• A local read or import cancelled by Stop (or sign-out) reported its own error completion. That raced Stop's settlement, so a stopped import ended failed or cancelled depending on which committed first.
• Native file tools now report nothing once cancelled, the same as browser actions and granted-folder (VFS) reads. Stop's own record stands: cancelled, with the stopped-before-start or stopped-while-running message.
• Regression tests: an import stopped mid-upload, and a read stopped while the desktop was reading, report nothing.
• Why the existing tests didn't catch it: they covered success, partial failure and replay, but never a cancelled signal. The live-Sim Electron E2E in improvement(desktop): Electron E2E for desktop tool lifetimes against a live local Sim #8697 surfaced it.

Type of Change

• Bug fix

Testing

• New unit tests failed before the fix and pass after
• Lint, typecheck and the client tool tests pass

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 7, 2026 7:20am UTC

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Changes how a stopped file operation reports its outcome.

The PR appears safe to merge.

What we checked:

• Stop still saves the outcome: abortRun calls requestRunStop, which saves cancelled results for open desktop calls. Both native file tools belong to that group.
• Cancellation still removes the listener: Both returns still run finally, which removes the pagehide listener.

Summary

Native file reads and imports now skip completion reports after cancellation, leaving Stop's saved outcome unchanged.

• Adds regression tests for Stop during an upload and during a native read.
• Keeps page-exit cleanup and upload cleanup unchanged.
• No actionable issues found. Tests were inspected but not run.

Diagram

sequenceDiagram
  participant User
  participant Client
  participant Server
  participant Files as Native file tool
  User->>Client: Stop
  Client->>Files: Abort the turn's signal
  Client->>Server: Request Stop
  Server->>Server: Save cancelled desktop calls
  Files->>Files: Read or upload returns or throws
  Files->>Files: Skip completion report if aborted
  Files->>Files: Remove page-exit listener

Loading

_{Reviews (1) · Last reviewed commit: "fix(desktop): leave a stopped native fil..." · Reviewed by Greptile}