@vanillagreen/pi-qol 1.6.0 → 1.7.0

package/README.md

@@ -15,7 +15,7 @@ Quality-of-life extension for Pi: compact statusline, multiline input, session n
 - `/search` browses previous sessions with snippet previews; the configured shortcut opens it instantly.
 - `/context` shows a Claude-style context-window breakdown.
 - `/handoff <goal>` drafts a focused prompt for a new session.
-- `/schedule 20m <message>` sends a delayed prompt without invoking the model until the timer fires.
+- `/schedule 20m <message>` or `/schedule 1h45m <message>` sends a delayed prompt without invoking the model until the timer fires.
 - Optional rate-limit auto-resume sends a configurable continuation after reset.
 - Permission gate prompts before risky `bash` commands. Default match: `rm -Rf`.
 - Notifications for ready, questions, blocked states, and task completion.
@@ -53,11 +53,11 @@ Restart Pi after installation.
 | `/search [query]` | Open previous-session search. |
 | `/search:refresh` | Refresh the session search cache. |
 | `/handoff <goal>` | Draft a handoff prompt for a new session. |
-| `/schedule <delay> <message>` | Send a user message after a timer without invoking the model now. Example: `/schedule 20m retry the previous request`. |
+| `/schedule <delay> <message>` | Send a user message after a timer without invoking the model now. Example: `/schedule 1h45m retry the previous request`. |
 
 Arguments support autocomplete.
 
-`/schedule` accepts `ms`, `s`, `m`, `h`, and `d` units; bare numbers mean minutes. Pending prompts render above the statusline like steering/follow-up previews until they are sent or cancelled. Manage pending prompts with `/schedule list` and `/schedule cancel <id|all>`. Schedules are stored in the Pi session and re-armed on reload/resume; if Pi is not running at the due time, an overdue prompt sends when that session is next loaded.
+`/schedule` accepts `ms`, `s`, `m`, `h`, and `d` units; bare numbers mean minutes. Compact composite durations are accepted in largest-to-smallest order, like `1h45m`, `45m10s`, or `1h45m30s`. Pending prompts render above the statusline like steering/follow-up previews until they are sent or cancelled. Manage pending prompts with `/schedule list` and `/schedule cancel <id|all>`. Schedules are stored in the Pi session and re-armed on reload/resume; if Pi is not running at the due time, an overdue prompt sends when that session is next loaded.
 
 ## Settings
 
@@ -138,7 +138,7 @@ Master toggle: **Enable notifications**.
 
 Triggers (notify when): ready, direction needed, question popups, all tasks complete, critical/blocked.
 
-Channels: terminal bell, native terminal notifications (OSC 777/99 or Windows toast), tmux `display-message`, tmux window marking, OSC passthrough, and an optional in-Pi UI notice.
+Channels: terminal bell, **Mute bell sound**, native terminal notifications (OSC 777/99 or Windows toast), tmux `display-message`, tmux window marking, OSC passthrough, and an optional in-Pi UI notice.
 
 Tuning: cooldown seconds, title, ready message, body length, tmux durations.
 
@@ -146,9 +146,10 @@ Notes:
 
 - **Terminal notification protocol** picks between OSC 99 (Kitty) and OSC 777 automatically.
 - **Bell when tmux window active** is off so you don't get bells while looking at Pi.
+- **Mute bell sound** keeps notification routing enabled but suppresses QOL-emitted BEL bytes and uses ST terminators for OSC 777/99 where supported. Terminals or operating systems may still play their own sound for native notifications outside QOL control.
 - **tmux native via client TTY** sends OSC notifications to attached tmux clients so notifications still appear when the Pi window is inactive.
 
-Use `/qol notify-test` to verify your terminal/tmux setup.
+Use `/qol notify-test` to verify your terminal/tmux setup, including silent behavior with **Mute bell sound** enabled.
 
 ### Permission gate
 
@@ -192,6 +193,8 @@ For autonomous/Flightdeck-style runs the agent never goes idle, so idle compacti
 
 When the budget guard fires it injects a sentinel into the compaction request so the QOL bounded handler always runs — chunked summarizer + handoff artifact — even if **Custom compaction summaries** is off. Manual compactions (`/tree`, idle compaction, user-triggered) still only use the QOL handler when **Custom compaction summaries** is on; otherwise they fall through to Pi's default compaction with no handoff artifact and no chunking. If you want every compaction to use the QOL bounded path, turn **Custom compaction summaries** on.
 
+While budget-guard compaction is running, QOL keeps a persistent status line above the prompt (and in the normal status footer when the compact statusline is disabled). After Pi prints the compacted-summary block, the line changes to `QOL budget guard finalizing compaction…` until `ctx.compact()` reports completion, so long reload/finalization gaps do not look frozen.
+
 Recommended values for autonomous/Flightdeck runs:
 
 - Keep **Long-session budget guard** on. Lower **Budget guard percent** to `75` if your provider buffers are tight.

package/extensions/qol/budget-guard-runtime.ts

@@ -17,6 +17,7 @@ export interface GuardDispatchInput {
 	trigger: BudgetTrigger | undefined;
 	compact: ((options: GuardCompactOptions) => void) | undefined;
 	notify: (message: string, level: GuardLevel) => void;
+	onStatus?: (message: string | undefined) => void;
 	staleCtx?: () => boolean;
 }
 
@@ -70,6 +71,7 @@ export class BudgetGuardDriver {
 			return { kind: "no-compact-fn" };
 		}
 		input.notify(`QOL budget guard starting compaction: ${trigger.reason}`, "info");
+		input.onStatus?.(`QOL budget guard compacting session: ${trigger.reason}`);
 		this.inFlight = true;
 		this.lastKey = trigger.key;
 		const instructions = `${QOL_BUDGET_GUARD_SENTINEL} QOL budget guard triggered at agent_end because ${trigger.reason}. Bound the summary input, preserve current task state, decisions, files, blockers, and next steps.`;
@@ -78,6 +80,7 @@ export class BudgetGuardDriver {
 				customInstructions: instructions,
 				onComplete: () => {
 					this.inFlight = false;
+					input.onStatus?.(undefined);
 					input.notify("QOL budget guard compaction completed.", "info");
 				},
 				onError: (error: Error) => {
@@ -85,6 +88,7 @@ export class BudgetGuardDriver {
 					// Allow the next agent_end to retry rather than poisoning the
 					// crossing key after a transient failure.
 					this.lastKey = undefined;
+					input.onStatus?.(undefined);
 					input.notify(`QOL budget guard compaction failed: ${error.message}`, "error");
 				},
 			});
@@ -92,6 +96,7 @@ export class BudgetGuardDriver {
 		} catch (error) {
 			this.inFlight = false;
 			this.lastKey = undefined;
+			input.onStatus?.(undefined);
 			const message = error instanceof Error ? error.message : String(error);
 			input.notify(`QOL budget guard compaction failed to start: ${message}`, "error");
 			return { kind: "dispatch-threw", error: message, reason: trigger.reason };

package/extensions/qol/notifications.ts

@@ -52,6 +52,19 @@ function tmuxPassthrough(sequence: string): string {
 	return `\x1bPtmux;${sequence.replace(/\x1b/g, "\x1b\x1b")}\x1b\\`;
 }
 
+export function terminalBellSequence(muteBellSound: boolean): string | undefined {
+	return muteBellSound ? undefined : "\x07";
+}
+
+export function osc777NotificationSequence(title: string, body: string, muteBellSound = false): string {
+	const terminator = muteBellSound ? "\x1b\\" : "\x07";
+	return `\x1b]777;notify;${title};${body}${terminator}`;
+}
+
+function notificationBellMuted(cwd?: string): boolean {
+	return settingBoolean("notification.muteBellSound", false, cwd);
+}
+
 function sourcePaneTty(): string | undefined {
 	try {
 		if (!process.env.TMUX_PANE) return undefined;
@@ -127,21 +140,22 @@ function writeTerminalSequence(sequence: string, cwd?: string): void {
 	if (process.env.TMUX && settingBoolean("notification.tmuxNativeClientTty", true, cwd)) {
 		// Inactive tmux windows do not forward arbitrary OSC output to the terminal.
 		// Send native terminal notifications straight to attached tmux client TTYs,
-		// while BEL still goes through the source pane so tmux marks the right tab.
+		// while the explicit terminal bell still goes through the source pane when unmuted.
 		if (writeRawToPaths(tmuxClientTtys(), sequence)) return;
 	}
 	const output = process.env.TMUX && settingBoolean("notification.tmuxPassthrough", true, cwd) ? tmuxPassthrough(sequence) : sequence;
 	writeToTerminal(output);
 }
 
-function writeTerminalBell(): void {
+function writeTerminalBell(cwd?: string): void {
 	// Match Claude-style hooks: resolve the source pane TTY and write raw BEL there.
 	// This lets tmux set window_bell_flag for the correct source window.
-	writeToTerminal("\x07");
+	const sequence = terminalBellSequence(notificationBellMuted(cwd));
+	if (sequence) writeToTerminal(sequence);
 }
 
 function notifyOSC777(title: string, body: string, cwd?: string): void {
-	writeTerminalSequence(`\x1b]777;notify;${title};${body}\x07`, cwd);
+	writeTerminalSequence(osc777NotificationSequence(title, body, notificationBellMuted(cwd)), cwd);
 }
 
 function notifyOSC99(title: string, body: string, cwd?: string): void {
@@ -242,7 +256,7 @@ export function sendQolNotification(ctx: ExtensionContext | undefined, kind: Qol
 	const title = sanitizeNotificationPart(settingString("notification.title", DEFAULT_NOTIFICATION_TITLE, cwd), 80) || DEFAULT_NOTIFICATION_TITLE;
 	const text = sanitizeNotificationPart(body, Math.max(40, Math.floor(settingNumber("notification.bodyMaxChars", DEFAULT_NOTIFICATION_BODY_MAX_CHARS, cwd))));
 	const tmuxWindowActive = sourceTmuxWindowActive();
-	if (settingBoolean("notification.bell", true, cwd) && (!tmuxWindowActive || settingBoolean("notification.bellWhenActive", false, cwd))) writeTerminalBell();
+	if (settingBoolean("notification.bell", true, cwd) && (!tmuxWindowActive || settingBoolean("notification.bellWhenActive", false, cwd))) writeTerminalBell(cwd);
 	if (settingBoolean("notification.native", true, cwd)) notifyNativeTerminal(title, text, cwd);
 	if (!tmuxWindowActive) markTmuxWindow(cwd);
 	if (settingBoolean("notification.tmux", false, cwd)) notifyTmux(title, text, cwd);

package/extensions/qol/schedule.ts

@@ -56,6 +56,7 @@ const SCHEDULE_COMPLETIONS: AutocompleteItem[] = [
 	{ value: "list", label: "list", description: "Show pending scheduled messages" },
 	{ value: "cancel ", label: "cancel <id|all>", description: "Cancel one scheduled message or all pending messages" },
 	{ value: "20m ", label: "20m <message>", description: "Send a message after 20 minutes" },
+	{ value: "1h30m ", label: "1h30m <message>", description: "Send a message after 1 hour 30 minutes" },
 	{ value: "1h ", label: "1h <message>", description: "Send a message after 1 hour" },
 ];
 
@@ -66,25 +67,47 @@ export function getScheduleArgumentCompletions(prefix: string): AutocompleteItem
 }
 
 export function parseDurationMs(input: string): number | undefined {
-	const match = input.trim().toLowerCase().match(/^(\d+(?:\.\d+)?)(ms|s|sec|secs|second|seconds|m|min|mins|minute|minutes|h|hr|hrs|hour|hours|d|day|days)?$/);
-	if (!match) return undefined;
-	const amount = Number(match[1]);
-	if (!Number.isFinite(amount) || amount <= 0) return undefined;
-	const unit = match[2] ?? "m";
-	const multiplier = unit === "ms"
-		? 1
-		: ["s", "sec", "secs", "second", "seconds"].includes(unit)
-			? 1000
-			: ["h", "hr", "hrs", "hour", "hours"].includes(unit)
-				? 60 * 60 * 1000
-				: ["d", "day", "days"].includes(unit)
-					? 24 * 60 * 60 * 1000
-					: 60 * 1000;
-	const delayMs = Math.round(amount * multiplier);
+	const text = input.trim().toLowerCase();
+	if (!text) return undefined;
+	if (/^\d+(?:\.\d+)?$/.test(text)) return clampScheduleDelay(Number(text) * 60 * 1000);
+
+	let totalMs = 0;
+	let matched = false;
+	let previousRank = Number.POSITIVE_INFINITY;
+	const seenUnits = new Set<string>();
+	const re = /(\d+(?:\.\d+)?)(ms|seconds?|secs?|s|minutes?|mins?|min|m|hours?|hrs?|hr|h|days?|d)/gy;
+
+	while (re.lastIndex < text.length) {
+		const match = re.exec(text);
+		if (!match) return undefined;
+		const amount = Number(match[1]);
+		const unit = scheduleUnit(match[2]);
+		if (!Number.isFinite(amount) || amount < 0 || !unit) return undefined;
+		if (seenUnits.has(unit.name) || unit.rank >= previousRank) return undefined;
+		seenUnits.add(unit.name);
+		previousRank = unit.rank;
+		totalMs += amount * unit.multiplier;
+		matched = true;
+	}
+
+	const delayMs = Math.round(totalMs);
+	return matched ? clampScheduleDelay(delayMs) : undefined;
+}
+
+function clampScheduleDelay(delayMs: number): number | undefined {
 	if (!Number.isFinite(delayMs) || delayMs <= 0 || delayMs > MAX_SCHEDULE_DELAY_MS) return undefined;
 	return delayMs;
 }
 
+function scheduleUnit(unit: string): { multiplier: number; name: string; rank: number } | undefined {
+	if (unit === "ms") return { multiplier: 1, name: "ms", rank: 0 };
+	if (["s", "sec", "secs", "second", "seconds"].includes(unit)) return { multiplier: 1000, name: "s", rank: 1 };
+	if (["m", "min", "mins", "minute", "minutes"].includes(unit)) return { multiplier: 60 * 1000, name: "m", rank: 2 };
+	if (["h", "hr", "hrs", "hour", "hours"].includes(unit)) return { multiplier: 60 * 60 * 1000, name: "h", rank: 3 };
+	if (["d", "day", "days"].includes(unit)) return { multiplier: 24 * 60 * 60 * 1000, name: "d", rank: 4 };
+	return undefined;
+}
+
 export function parseScheduleCommandArgs(args: string): ParsedScheduleCommand {
 	const trimmed = args.trim();
 	if (!trimmed) return { kind: "list" };
@@ -131,7 +154,8 @@ function usageText(): string {
 	return [
 		"Usage: /schedule <duration> <message>",
 		"Examples: /schedule 20m retry the previous request",
-		"          /schedule 1h retry after the rate limit reset",
+		"          /schedule 1h30m retry after the rate limit reset",
+		"Durations: 20, 20m, 90s, 500ms, 1h45m30s (bare numbers mean minutes)",
 		"Manage: /schedule list, /schedule cancel <id|all>",
 	].join("\n");
 }

package/extensions/qol/status-message.ts

@@ -39,7 +39,7 @@ export function statusMessage(ctx: ExtensionContext): string {
 		`Custom compaction: ${settingBoolean("compaction.customEnabled", false, ctx.cwd) ? `enabled (${settingString("compaction.model", DEFAULT_COMPACTION_MODEL, ctx.cwd)}, ${compactionProfile(ctx.cwd)})` : "disabled (Pi default)"}`,
 		`Idle compaction: ${settingBoolean("compaction.idleEnabled", false, ctx.cwd) ? `enabled after ${Math.max(1, Math.floor(settingNumber("compaction.idleTimeoutSeconds", DEFAULT_IDLE_COMPACTION_SECONDS, ctx.cwd)))}s idle` : "disabled"}`,
 		`Branch summary override: ${settingBoolean("compaction.branchSummaryEnabled", false, ctx.cwd) ? "enabled" : "disabled"}`,
-		`Notifications: ${settingBoolean("notification.enabled", true, ctx.cwd) ? `enabled (bell=${settingBoolean("notification.bell", true, ctx.cwd)}, native=${settingBoolean("notification.native", true, ctx.cwd)}, tmuxClientTty=${settingBoolean("notification.tmuxNativeClientTty", true, ctx.cwd)}, tmuxMessage=${settingBoolean("notification.tmux", false, ctx.cwd)})` : "disabled"}`,
+		`Notifications: ${settingBoolean("notification.enabled", true, ctx.cwd) ? `enabled (bell=${settingBoolean("notification.bell", true, ctx.cwd)}, muteBell=${settingBoolean("notification.muteBellSound", false, ctx.cwd)}, native=${settingBoolean("notification.native", true, ctx.cwd)}, tmuxClientTty=${settingBoolean("notification.tmuxNativeClientTty", true, ctx.cwd)}, tmuxMessage=${settingBoolean("notification.tmux", false, ctx.cwd)})` : "disabled"}`,
 		`Permission gate: ${settingBoolean("permissionGate.enabled", false, ctx.cwd) ? `enabled (${permissionGateCommands(ctx.cwd).join(", ") || "none configured"}; preview ${boundedSettingNumber("permissionGate.previewLines", DEFAULT_PERMISSION_GATE_PREVIEW_LINES, 4, 40, ctx.cwd)} lines/${boundedSettingNumber("permissionGate.previewChars", DEFAULT_PERMISSION_GATE_PREVIEW_CHARS, 200, 5000, ctx.cwd)} chars)` : "disabled"}`,
 		`Thinking timer: ${settingBoolean("thinkingTimer.enabled", true, ctx.cwd) ? "enabled" : "disabled"}`,
 		"If shift+enter still submits, configure your terminal/tmux to send a distinct shift+enter sequence or use the fallback key.",

package/extensions/qol.ts

@@ -1,8 +1,9 @@
 import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext } from "@earendil-works/pi-coding-agent";
-import type { TUI } from "@earendil-works/pi-tui";
+import { truncateToWidth, type TUI } from "@earendil-works/pi-tui";
 import { criticalInfo, lastAssistantTextFromAgentEnd, needsDirection, taskStats } from "./qol/agent-end.js";
 import { getQuestionService, readCavemanBridge, type QuestionOpenedEventLike } from "./qol/bridges.js";
 import { BudgetGuardDriver } from "./qol/budget-guard-runtime.js";
+import { ansiGreen } from "./qol/ansi.js";
 import {
 	budgetGuardTrigger,
 	compactionTriggerReason,
@@ -262,9 +263,19 @@ export default function qol(pi: ExtensionAPI): void {
 	};
 
 	const budgetGuardDriver = new BudgetGuardDriver();
+	let budgetGuardStatus: string | undefined;
+
+	const setBudgetGuardStatus = (ctx: ExtensionContext, message: string | undefined) => {
+		budgetGuardStatus = message;
+		if (ctx.hasUI) {
+			ctx.ui.setStatus("qol-budget-guard", message ? ctx.ui.theme.fg("accent", message) : undefined);
+			requestRender();
+		}
+	};
 
 	const resetBudgetGuard = () => {
 		budgetGuardDriver.reset();
+		budgetGuardStatus = undefined;
 	};
 
 	const maybeFireBudgetGuard = (ctx: ExtensionContext) => {
@@ -286,6 +297,7 @@ export default function qol(pi: ExtensionAPI): void {
 		budgetGuardDriver.dispatch({
 			compact: typeof ctx.compact === "function" ? ctx.compact.bind(ctx) : undefined,
 			notify: notifySafely,
+			onStatus: (message) => setBudgetGuardStatus(ctx, message),
 			trigger,
 		});
 	};
@@ -584,6 +596,7 @@ export default function qol(pi: ExtensionAPI): void {
 							render(width: number): string[] {
 								const lines = rateLimitAutoResumeController.enabled(ctx) ? rateLimitAutoResumeController.renderPreviewLines(width) : [];
 								if (scheduleEnabled) lines.push(...scheduleController.renderPreviewLines(width));
+								if (budgetGuardStatus) lines.push(truncateToWidth(ansiGreen(`┃ ${budgetGuardStatus}`), width, ""));
 								if (showStatusline) lines.push(renderStatusLine(width, ctx, gitState ?? makeFallbackGitState(ctx.cwd), pi, theme));
 								return lines;
 							},
@@ -635,10 +648,12 @@ export default function qol(pi: ExtensionAPI): void {
 		questionUnsubscribe?.();
 		questionUnsubscribe = undefined;
 		currentCtx = undefined;
+		budgetGuardStatus = undefined;
 		const host = globalThis as unknown as Record<PropertyKey, unknown>;
 		if (host[QOL_NOTIFICATION_SERVICE_SYMBOL] === notificationService) delete host[QOL_NOTIFICATION_SERVICE_SYMBOL];
 		ctx.ui.setStatus(STATUS_KEY, undefined);
 		ctx.ui.setStatus(SESSION_SEARCH_STATUS_KEY, undefined);
+		ctx.ui.setStatus("qol-budget-guard", undefined);
 		restorePendingQueueThemePatch(ctx);
 		resetStatuslineUi(ctx);
 		ctx.ui.setEditorComponent(undefined);
@@ -721,6 +736,7 @@ export default function qol(pi: ExtensionAPI): void {
 		sendQolNotification(ctx, "ready", settingString("notification.readyMessage", "Ready for input", ctx.cwd), "info", "ready");
 	});
 	pi.on("session_compact", (_event, ctx) => {
+		if (budgetGuardStatus) setBudgetGuardStatus(ctx, "QOL budget guard finalizing compaction…");
 		// After a successful compaction usage drops below the budget, so reset the
 		// crossing key so the next threshold crossing re-fires the guard.
 		budgetGuardDriver.noteSessionCompacted();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vanillagreen/pi-qol",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "description": "Pi quality-of-life extension: compact statusline/π prompt, reliable multiline input, styled pasted-image chips, session naming/search/context import, scheduled prompts, handoff, permission prompts, notifications, custom compaction, and a collapsed-thinking timer.",
   "license": "MIT",
   "keywords": [
@@ -582,6 +582,15 @@
           "category": "Notifications Channels",
           "apply": "live"
         },
+        {
+          "key": "notification.muteBellSound",
+          "label": "Mute bell sound",
+          "description": "Keep notifications enabled while suppressing QOL-emitted BEL bytes; OSC terminal notifications use ST terminators where supported.",
+          "type": "boolean",
+          "default": false,
+          "category": "Notifications Channels",
+          "apply": "live"
+        },
         {
           "key": "notification.bellWhenActive",
           "label": "Bell when tmux window active",