fix: do not override bash-mode or empty-editor Escape

Match Pi's built-in onEscape contract more closely: only arm double-Esc
clear for non-empty non-bash drafts while idle, leave whitespace/empty,
`!` bash mode, streaming, autocomplete, and abort paths to the app.

Author: Spikeysanju

extensions/double-esc-clear.ts

@@ -2,10 +2,16 @@
  * Double Escape clears a non-empty editor draft.
  *
  * Built-in double-Escape only acts on an *empty* editor (tree / fork / none via
- * `doubleEscapeAction`). When the editor has text, Escape is a no-op while idle.
- * This extension makes Escape → Escape (within 500ms) clear the draft instead.
+ * `doubleEscapeAction`). When the editor has normal draft text, Escape is a
+ * no-op while idle. This extension makes Escape → Escape (within 500ms) clear
+ * that draft instead.
  *
- * Streaming / bash abort paths are left to the app.
+ * Must not steal these built-in Escape paths:
+ * - streaming abort
+ * - bash-running abort
+ * - bash-mode (`!…`) single-Esc clear/exit
+ * - autocomplete cancel
+ * - empty / whitespace-only double-Esc (`doubleEscapeAction`)
  */
 import {
 	CustomEditor,
@@ -16,6 +22,15 @@ import { matchesKey } from "@earendil-works/pi-tui";
 
 const DOUBLE_ESC_MS = 500;
 
+/** Draft Pi would leave alone on Escape (not empty, not `!` bash mode). */
+function isClearableDraft(text: string): boolean {
+	const trimmed = text.trim();
+	if (trimmed.length === 0) return false;
+	// Bash mode is exited with a single Escape by the app.
+	if (text.trimStart().startsWith("!")) return false;
+	return true;
+}
+
 export default function (pi: ExtensionAPI) {
 	pi.on("session_start", (_event, ctx) => {
 		if (ctx.mode !== "tui") return;
@@ -37,7 +52,6 @@ export default function (pi: ExtensionAPI) {
 			let lastEscapeTime = 0;
 
 			editor.handleInput = (data: string) => {
-				// Match configured interrupt binding if available, else raw Escape.
 				const isEscape =
 					typeof keybindings.matches === "function"
 						? keybindings.matches(data, "app.interrupt")
@@ -46,16 +60,22 @@ export default function (pi: ExtensionAPI) {
 				if (
 					isEscape &&
 					!editor.isShowingAutocomplete() &&
+					// Streaming / non-idle abort stays with the app.
 					ctx.isIdle() &&
-					editor.getText().length > 0
+					isClearableDraft(editor.getText())
 				) {
 					const now = Date.now();
 					if (now - lastEscapeTime < DOUBLE_ESC_MS) {
+						// Second Esc on a plain draft → clear. Built-in onEscape is a
+						// no-op for this case, so we do not fall through.
 						editor.setText("");
 						lastEscapeTime = 0;
-					} else {
-						lastEscapeTime = now;
+						return;
 					}
+					// First Esc: arm timer, then fall through so bash-running /
+					// other app Escape handlers still run when applicable.
+					lastEscapeTime = now;
+					originalHandleInput(data);
 					return;
 				}