Tool calls are run in the background and don't block the display thread.

[chabicht]

No description provided.

[chabicht]

@copilot please review.

[Copilot]

@copilot please review.

Reviewed. The overall approach is solid — using Eclipse Jobs + CompletableFuture for background tool execution, generation-based stale-callback detection, and cooperative IProgressMonitor cancellation throughout the chain are all good patterns.

Issues I found and fixed in commit 5e906fe:

1. Indentation — isCurrentChatSession's continuation && line was aligned at column 2 instead of the method body indent level.
2. Extra blank line — spurious double blank line before abortChat().
3. Cancel state inconsistency — reexecuteToolBatch only set tool_execution_state to "completed" when !report.isCanceled(), leaving the message in "queued" state after cancellation. Now it always sets "completed" so the re-execute button remains accessible.
4. Pre-existing indentation in executeBatchesSequentially's inner loop — the batchCallsExecuted/batchCallsFailed block and the for-loop body were at mismatched indent levels, made more confusing by the added cancel checks. Corrected to consistent indentation.

Also added .m2/ to .gitignore and removed the accidentally tracked Maven cache in commit b291241.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• www.eclipse.org
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.14/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.14/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.14 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.14/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/code-intelligence/code-intelligence org.codehaus.plexus.classworlds.launcher.Launcher -B --settings ./.m2/settings.xml -Dmaven.repo.local=./.m2/repository -q package tycho-surefire:test (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)