@estebanforge/pi-ask-antigravity 1.2.7 → 1.2.8

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.2.8] - 2026-09-25
+
+### Fixed
+
+- **Exit 0 with no output is a failure, not a silent success.** Same bug as the bridge fix: a headless `agy -p` run auto-denies a permission-gated tool call (the command gate in plan mode bites hardest since plan runs never receive the skip flag), prints the reason only to stderr, exits 0, and the execute path fell through to success, returning just the conversation footer. An empty answer now returns a loud failure note carrying the stderr plus the remedy (allow-list the command under `permissions.allow`, or rerun outside plan mode), and a new `details.empty` marker flips the TUI result row to its error glyph instead of a green checkmark. Three new tests pin both stderr shapes and the glyph flip through the fake-binary stub; HOME is pointed at an empty tmpdir during them so the bridge-deferral scan finds nothing and the factory registers.
+
 ## [1.2.7] - 2026-09-25
 
 ### Fixed

package/extensions/index.ts

@@ -762,6 +762,9 @@ interface AgyDetails {
 	exitCode: number;
 	aborted: boolean;
 	timedOut: boolean;
+	// Exit 0 with no answer on stdout (the empty-output failure branch).
+	// renderResult flips to the error glyph on it.
+	empty?: boolean;
 	durationMs: number;
 	stderr: string;
 }
@@ -1027,7 +1030,8 @@ export default async function (pi: ExtensionAPI) {
 			}
 
 			const body = result.content[0]?.type === "text" ? result.content[0].text : "";
-			const errored = d?.exitCode !== 0 || !!d?.aborted || !!d?.timedOut;
+			const errored =
+		d?.exitCode !== 0 || !!d?.aborted || !!d?.timedOut || !!d?.empty;
 
 			let text = errored
 				? theme.fg("error", "✗ AskAntigravity error")
@@ -1439,6 +1443,27 @@ export default async function (pi: ExtensionAPI) {
 					};
 				}
 
+				// Exit 0 with nothing on stdout is still a failure for the
+				// caller: headless agy auto-denies a permission-gated tool call
+				// (e.g. the command gate in plan mode), prints the reason only to
+				// stderr, and ends cleanly. Falling through to the success path
+				// here returned just the conversation footer, which read as an
+				// empty success (silent-failure bug found 2026-09-25).
+				if (!text) {
+					details.empty = true;
+					const note = [
+						"agy exited cleanly but produced no output.",
+						details.stderr.trim() ? `stderr: ${details.stderr.trim()}` : null,
+						"Common cause: a tool call needed a permission that headless mode cannot prompt for (typically the command gate in plan mode), so it was auto-denied. Allow-list it under permissions.allow in ~/.gemini/antigravity-cli/settings.json, or rerun outside plan mode with skipPermissions.",
+					]
+						.filter(Boolean)
+						.join(" ");
+					return {
+						content: [{ type: "text", text: note }],
+						details,
+					};
+				}
+
 				// Success. Clear the last partial status line (claude-bridge
 				// idiom) so the running-tail preview doesn't linger under the final
 				// answer, then append a conversation footer so the orchestrating model

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@estebanforge/pi-ask-antigravity",
-  "version": "1.2.7",
+  "version": "1.2.8",
   "description": "Pi tool that delegates a self-contained sub-task to Google Antigravity's agy CLI (the CLI for Gemini). Friendly model aliases (flash/pro/gemini), configurable default model + thinking, and the AskAntigravity delegation tool.",
   "keywords": [
     "pi-package",