[woutdenolf]

Stack (split from #501, merge in order):

1. Qt test fixes: drop test_owwidget_drop timing dependency, fix Windows crash at exit #506 Qt test fixes: drop test_owwidget_drop timing dependency, fix Windows crash at exit
2. Rename TaskFuture.abort/aborted to request_stop/stop_requested #507 Rename TaskFuture.abort/aborted to request_stop/stop_requested
3. Never start a queued task that is cancelled or stopped; two queues for the process backend #508 Never start a queued task that is cancelled or stopped; two queues for the process backend
4. EwoksExecutor: completion signals after start signals, finished for every execution #509 EwoksExecutor: completion signals after start signals, finished for every execution 👈 this PR
5. Execute subprocess tasks with pebble, add TaskFuture.abort and TaskFuture.kill #501 Execute subprocess tasks with pebble, add TaskFuture.abort and TaskFuture.kill
6. Doc: executor contract, backend diagrams and glossary #510 Doc: executor contract, backend diagrams and glossary

PR summary

• The completion signals are emitted after the start and progress signals.
  • Thread and process backends: the done callback emits an internal _done signal, queued to the executor thread.
• Every submitted execution ends with finished.
  • A cancelled task emits cancelled and finished. The widget no longer stays pending.
• Signal stop_requested → stopRequested, emitted with the completion.
• A failed submit() no longer leaves the executor busy.

AI Disclosure

• AI tools claude used for generate the code

🤖 Generated with Claude Code

[codecov]

Codecov Report

❌ Patch coverage is 97.67442% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/ewoksorange/gui/concurrency/executor.py	97.43%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[payno]

I think the three objects could be more typed (futures.Future[VariableContainer], TaskFuture, and _controllers.TaskController).
But is seems (to be confirmed) that only the TaskFuture is required - _handle_done only uses the task_future if I am correct. That would simplify and aligned things.

[woutdenolf]

Indeed, only task_future is used. _done and _handle_done now take just that.

[payno]

The queue_done is not really helping IMO. This is True when the executor exists. AI suggest defer_completion that seems fair to me.

Suggested change

	queue_done: bool,
	defer_completion: bool,

[woutdenolf]

Renamed to defer_completion.