pi-usereq 0.4.0 → 0.6.0

package/src/core/config.ts

@@ -19,6 +19,8 @@ import {
   DEFAULT_PI_NOTIFY_SOUND_MID_CMD,
   DEFAULT_PI_NOTIFY_SOUND_TOGGLE_SHORTCUT,
   normalizePiNotifyCommand,
+  normalizePiNotifyPushoverCredential,
+  normalizePiNotifyPushoverPriority,
   normalizePiNotifyShortcut,
   normalizePiNotifySoundLevel,
 } from "./pi-notify.js";
@@ -50,6 +52,11 @@ export interface UseReqConfig {
   "notify-beep-on-error": boolean;
   "notify-sound": "none" | "low" | "mid" | "high";
   "notify-sound-toggle-shortcut": string;
+  "notify-pushover-global-disable": boolean;
+  "notify-pushover-on-success": boolean;
+  "notify-pushover-user-key": string;
+  "notify-pushover-api-token": string;
+  "notify-pushover-priority": 0 | 1;
   PI_NOTIFY_SOUND_LOW_CMD: string;
   PI_NOTIFY_SOUND_MID_CMD: string;
   PI_NOTIFY_SOUND_HIGH_CMD: string;
@@ -83,10 +90,10 @@ export function getProjectConfigPath(projectBase: string): string {
 
 /**
  * @brief Builds the default project configuration.
- * @details Populates canonical docs/test/source directories, the default startup tool set, and default pi-notify fields while excluding runtime-derived path metadata. Time complexity is O(n) in default tool count. No filesystem side effects occur.
+ * @details Populates canonical docs/test/source directories, the default startup tool set, default pi-notify fields, and default Pushover settings while excluding runtime-derived path metadata. Time complexity is O(n) in default tool count. No filesystem side effects occur.
  * @param[in] projectBase {string} Absolute project root path.
  * @return {UseReqConfig} Fresh default configuration object.
- * @satisfies CTN-001, CTN-012, REQ-066, REQ-146
+ * @satisfies CTN-001, CTN-012, REQ-066, REQ-129, REQ-146, REQ-163
  */
 export function getDefaultConfig(_projectBase: string): UseReqConfig {
   return {
@@ -95,11 +102,16 @@ export function getDefaultConfig(_projectBase: string): UseReqConfig {
     "src-dir": [...DEFAULT_SRC_DIRS],
     "static-check": {},
     "enabled-tools": normalizeEnabledPiUsereqTools(undefined),
-    "notify-beep-on-end": false,
-    "notify-beep-on-esc": false,
-    "notify-beep-on-error": false,
+    "notify-beep-on-end": true,
+    "notify-beep-on-esc": true,
+    "notify-beep-on-error": true,
     "notify-sound": "none",
     "notify-sound-toggle-shortcut": DEFAULT_PI_NOTIFY_SOUND_TOGGLE_SHORTCUT,
+    "notify-pushover-global-disable": false,
+    "notify-pushover-on-success": false,
+    "notify-pushover-user-key": "",
+    "notify-pushover-api-token": "",
+    "notify-pushover-priority": 0,
     PI_NOTIFY_SOUND_LOW_CMD: DEFAULT_PI_NOTIFY_SOUND_LOW_CMD,
     PI_NOTIFY_SOUND_MID_CMD: DEFAULT_PI_NOTIFY_SOUND_MID_CMD,
     PI_NOTIFY_SOUND_HIGH_CMD: DEFAULT_PI_NOTIFY_SOUND_HIGH_CMD,
@@ -108,11 +120,11 @@ export function getDefaultConfig(_projectBase: string): UseReqConfig {
 
 /**
  * @brief Loads and sanitizes the persisted project configuration.
- * @details Returns defaults when the config file does not exist. Otherwise parses JSON, validates directory and static-check field shapes, normalizes enabled tool names and pi-notify fields, and ignores removed or runtime-derived path metadata. Runtime is O(n) in config size. Side effects are limited to filesystem reads.
+ * @details Returns defaults when the config file does not exist. Otherwise parses JSON, validates directory and static-check field shapes, normalizes enabled tool names plus pi-notify and Pushover fields, applies enabled beep defaults for missing flag payloads, and ignores removed or runtime-derived path metadata. Runtime is O(n) in config size. Side effects are limited to filesystem reads.
  * @param[in] projectBase {string} Absolute project root path.
  * @return {UseReqConfig} Sanitized effective configuration.
  * @throws {ReqError} Throws with exit code `11` when the config file contains invalid JSON or a non-object payload.
- * @satisfies CTN-012, REQ-066, REQ-146
+ * @satisfies CTN-012, REQ-066, REQ-129, REQ-146, REQ-163
  */
 export function loadConfig(projectBase: string): UseReqConfig {
   const configPath = getProjectConfigPath(projectBase);
@@ -140,11 +152,16 @@ export function loadConfig(projectBase: string): UseReqConfig {
     ? (data["static-check"] as Record<string, StaticCheckEntry[]>)
     : {};
   const enabledTools = normalizeEnabledPiUsereqTools(data["enabled-tools"]);
-  const notifyBeepOnEnd = data["notify-beep-on-end"] === true;
-  const notifyBeepOnEsc = data["notify-beep-on-esc"] === true;
-  const notifyBeepOnError = data["notify-beep-on-error"] === true;
+  const notifyBeepOnEnd = data["notify-beep-on-end"] !== false;
+  const notifyBeepOnEsc = data["notify-beep-on-esc"] !== false;
+  const notifyBeepOnError = data["notify-beep-on-error"] !== false;
   const notifySound = normalizePiNotifySoundLevel(data["notify-sound"]);
   const notifySoundToggleShortcut = normalizePiNotifyShortcut(data["notify-sound-toggle-shortcut"]);
+  const pushoverGlobalDisable = data["notify-pushover-global-disable"] === true;
+  const pushoverOnSuccess = data["notify-pushover-on-success"] === true;
+  const pushoverUserKey = normalizePiNotifyPushoverCredential(data["notify-pushover-user-key"]);
+  const pushoverApiToken = normalizePiNotifyPushoverCredential(data["notify-pushover-api-token"]);
+  const pushoverPriority = normalizePiNotifyPushoverPriority(data["notify-pushover-priority"]);
   const lowSoundCommand = normalizePiNotifyCommand(data.PI_NOTIFY_SOUND_LOW_CMD, DEFAULT_PI_NOTIFY_SOUND_LOW_CMD);
   const midSoundCommand = normalizePiNotifyCommand(data.PI_NOTIFY_SOUND_MID_CMD, DEFAULT_PI_NOTIFY_SOUND_MID_CMD);
   const highSoundCommand = normalizePiNotifyCommand(data.PI_NOTIFY_SOUND_HIGH_CMD, DEFAULT_PI_NOTIFY_SOUND_HIGH_CMD);
@@ -160,6 +177,11 @@ export function loadConfig(projectBase: string): UseReqConfig {
     "notify-beep-on-error": notifyBeepOnError,
     "notify-sound": notifySound,
     "notify-sound-toggle-shortcut": notifySoundToggleShortcut,
+    "notify-pushover-global-disable": pushoverGlobalDisable,
+    "notify-pushover-on-success": pushoverOnSuccess,
+    "notify-pushover-user-key": pushoverUserKey,
+    "notify-pushover-api-token": pushoverApiToken,
+    "notify-pushover-priority": pushoverPriority,
     PI_NOTIFY_SOUND_LOW_CMD: lowSoundCommand,
     PI_NOTIFY_SOUND_MID_CMD: midSoundCommand,
     PI_NOTIFY_SOUND_HIGH_CMD: highSoundCommand,
@@ -168,10 +190,10 @@ export function loadConfig(projectBase: string): UseReqConfig {
 
 /**
  * @brief Builds the persisted configuration payload that excludes runtime-derived fields.
- * @details Copies only the canonical persisted configuration keys into a fresh object so runtime-derived metadata such as `base-path` and `git-path` can never be written to disk. Runtime is O(n) in config size. No external state is mutated.
+ * @details Copies only the canonical persisted configuration keys into a fresh object so runtime-derived metadata such as `base-path` and `git-path` can never be written to disk while preserving notification and Pushover settings verbatim. Runtime is O(n) in config size. No external state is mutated.
  * @param[in] config {UseReqConfig} Effective configuration object.
  * @return {UseReqConfig} Persistable configuration payload.
- * @satisfies CTN-012, REQ-146
+ * @satisfies CTN-012, REQ-146, REQ-163
  */
 function buildPersistedConfig(config: UseReqConfig): UseReqConfig {
   return {
@@ -194,6 +216,11 @@ function buildPersistedConfig(config: UseReqConfig): UseReqConfig {
     "notify-beep-on-error": config["notify-beep-on-error"],
     "notify-sound": config["notify-sound"],
     "notify-sound-toggle-shortcut": config["notify-sound-toggle-shortcut"],
+    "notify-pushover-global-disable": config["notify-pushover-global-disable"],
+    "notify-pushover-on-success": config["notify-pushover-on-success"],
+    "notify-pushover-user-key": config["notify-pushover-user-key"],
+    "notify-pushover-api-token": config["notify-pushover-api-token"],
+    "notify-pushover-priority": config["notify-pushover-priority"],
     PI_NOTIFY_SOUND_LOW_CMD: config.PI_NOTIFY_SOUND_LOW_CMD,
     PI_NOTIFY_SOUND_MID_CMD: config.PI_NOTIFY_SOUND_MID_CMD,
     PI_NOTIFY_SOUND_HIGH_CMD: config.PI_NOTIFY_SOUND_HIGH_CMD,

package/src/core/extension-status.ts

@@ -8,6 +8,7 @@
  * scheduling through exported controller helpers.
  */
 
+import path from "node:path";
 import type {
   AgentEndEvent,
   ContextUsage,
@@ -15,8 +16,8 @@ import type {
   ThemeColor,
 } from "@mariozechner/pi-coding-agent";
 import type { UseReqConfig } from "./config.js";
-import { formatPiNotifyBeepStatus } from "./pi-notify.js";
-import { formatAbsoluteGitPath, formatBasePathRelativeToGitPath, resolveRuntimeGitPath } from "./runtime-project-paths.js";
+import { normalizePathSlashes } from "./path-context.js";
+import { formatPiNotifyBeepStatus, formatPiNotifyPushoverStatus } from "./pi-notify.js";
 
 /**
  * @brief Enumerates the CLI-supported theme tokens consumed by status rendering.
@@ -66,7 +67,7 @@ interface StatusThemeAdapter {
 interface ContextUsageOverlaySpec {
   backgroundColor: StatusForegroundColor;
   foregroundColor: StatusForegroundColor;
-  text: "CLEAR" | "FULL!";
+  text: "◀ CLEAR ▶ " | " ◀ FULL ▶ ";
 }
 
 /**
@@ -112,28 +113,36 @@ export const PI_USEREQ_STATUS_HOOK_NAMES = [
  */
 export type PiUsereqStatusHookName = (typeof PI_USEREQ_STATUS_HOOK_NAMES)[number];
 
+/**
+ * @brief Describes one prompt request tracked across extension command delivery and runtime execution.
+ * @details Stores the bundled prompt command name plus the raw argument string substituted into `%%ARGS%%` so later successful-run side effects can reconstruct prompt-specific completion notifications. The interface is compile-time only and introduces no runtime cost.
+ */
+export interface PiUsereqPromptRequest {
+  promptName: string;
+  promptArgs: string;
+}
+
 /**
  * @brief Stores the mutable runtime facts displayed by the status bar.
- * @details Persists the latest context-usage snapshot, the active run start
- * timestamp, and the most recent normally completed run duration. Runtime state
- * is mutated in-place by controller helpers. Compile-time only and introduces
- * no runtime cost.
+ * @details Persists the latest context-usage snapshot, the active run start timestamp, the most recent normally completed run duration, the accumulated duration of all normally completed runs, and prompt-request metadata carried from command dispatch into the next runtime execution. Runtime state is mutated in-place by controller helpers. Compile-time only and introduces no runtime cost.
  */
 export interface PiUsereqStatusState {
   contextUsage: ContextUsage | undefined;
   runStartTimeMs: number | undefined;
   lastRunDurationMs: number | undefined;
+  totalRunDurationMs: number | undefined;
+  pendingPromptRequest: PiUsereqPromptRequest | undefined;
+  activePromptRequest: PiUsereqPromptRequest | undefined;
 }
 
 /**
  * @brief Stores the controller state required for event-driven status updates.
  * @details Keeps the mutable status snapshot, the current configuration, the
- * latest extension context used for rendering, the active-tools provider, and
- * the interval handle used for live elapsed-time refreshes. Compile-time only
- * and introduces no runtime cost.
+ * latest extension context used for rendering, and the interval handle used
+ * for live elapsed-time refreshes. Compile-time only and introduces no runtime
+ * cost.
  */
 export interface PiUsereqStatusController {
-  readonly getActiveTools: () => readonly string[];
   config: UseReqConfig | undefined;
   latestContext: ExtensionContext | undefined;
   state: PiUsereqStatusState;
@@ -264,12 +273,12 @@ function refreshContextUsage(
 }
 
 /**
- * @brief Counts the filled cells rendered by the 5-cell context bar.
+ * @brief Counts the filled cells rendered by the 10-cell context bar.
  * @details Uses ceiling semantics for positive percentages so any non-zero
  * usage occupies at least one cell and zero usage occupies none. Runtime is
  * O(1). No external state is mutated.
  * @param[in] contextUsage {ContextUsage | undefined} Normalized context snapshot.
- * @return {number} Filled-cell count in the inclusive range `[0, 5]`.
+ * @return {number} Filled-cell count in the inclusive range `[0, 10]`.
  * @satisfies REQ-122
  */
 function countFilledContextCells(
@@ -279,13 +288,13 @@ function countFilledContextCells(
   if (percent === null || percent === undefined || percent <= 0) {
     return 0;
   }
-  return Math.min(5, Math.ceil((percent * 5) / 100));
+  return Math.min(10, Math.ceil((percent * 10) / 100));
 }
 
 /**
  * @brief Resolves the threshold-specific context-bar overlay when required.
- * @details Returns the empty-state `CLEAR` overlay when normalized context
- * usage is unavailable or non-positive and returns the high-water `FULL!`
+ * @details Returns the empty-state `◀ CLEAR ▶ ` overlay when normalized context
+ * usage is unavailable or non-positive and returns the centered ` ◀ FULL ▶ `
  * overlay with the active theme `error` token when usage exceeds 90 percent.
  * Runtime is O(1). No external state is mutated.
  * @param[in] contextUsage {ContextUsage | undefined} Normalized context snapshot.
@@ -300,14 +309,14 @@ function resolveContextUsageOverlay(
     return {
       backgroundColor: "accent",
       foregroundColor: "warning",
-      text: "CLEAR",
+      text: "◀ CLEAR ▶ ",
     };
   }
   if (percent > 90) {
     return {
       backgroundColor: "warning",
       foregroundColor: "error",
-      text: "FULL!",
+      text: " ◀ FULL ▶ ",
     };
   }
   return undefined;
@@ -335,7 +344,7 @@ function formatContextUsageOverlay(
 }
 
 /**
- * @brief Formats one 5-cell context-usage bar.
+ * @brief Formats one 10-cell context-usage bar.
  * @details Renders threshold-specific overlays for empty and high-water states;
  * otherwise renders filled cells with the theme `warning` token on an
  * accent-derived background and unfilled cells in `dim` on the same background
@@ -343,7 +352,7 @@ function formatContextUsageOverlay(
  * mutated.
  * @param[in] theme {StatusThemeAdapter} Normalized status theme.
  * @param[in] contextUsage {ContextUsage | undefined} Normalized context snapshot.
- * @return {string} Rendered 5-cell bar or overlay.
+ * @return {string} Rendered 10-cell bar or overlay.
  * @satisfies REQ-121, REQ-122, REQ-126, REQ-127, REQ-128
  */
 function formatContextUsageBar(
@@ -355,7 +364,7 @@ function formatContextUsageBar(
     return formatContextUsageOverlay(theme, overlay);
   }
   const filledCells = countFilledContextCells(contextUsage);
-  return Array.from({ length: 5 }, (_value, index) =>
+  return Array.from({ length: 10 }, (_value, index) =>
     index < filledCells ? theme.filledContextCell : theme.emptyContextCell,
   ).join("");
 }
@@ -376,6 +385,44 @@ function formatStatusDuration(durationMs: number): string {
   return `${minutes}:${String(seconds).padStart(2, "0")}`;
 }
 
+/**
+ * @brief Formats one optional completed-duration value.
+ * @details Returns the canonical unset placeholder `--:--` until the supplied
+ * timer receives a normally completed prompt duration, then delegates to
+ * `formatStatusDuration(...)`. Runtime is O(1). No external state is mutated.
+ * @param[in] durationMs {number | undefined} Optional completed-duration value.
+ * @return {string} Rendered duration or unset placeholder.
+ * @satisfies REQ-124
+ */
+function formatCompletedStatusDuration(
+  durationMs: number | undefined,
+): string {
+  return durationMs === undefined ? "--:--" : formatStatusDuration(durationMs);
+}
+
+/**
+ * @brief Formats the consolidated `elapsed` status-bar value.
+ * @details Emits the active prompt segment `⏱︎ <active>`, the latest normally
+ * completed segment `⚑ <last>`, and the accumulated successful-runtime segment
+ * `⌛︎ <total>` with fixed spacing. Runtime is O(1). No external state is
+ * mutated.
+ * @param[in] state {PiUsereqStatusState} Mutable status state snapshot.
+ * @param[in] nowMs {number} Current wall-clock time in milliseconds.
+ * @return {string} Consolidated `elapsed` field value.
+ * @satisfies REQ-123, REQ-124, REQ-125, REQ-159
+ */
+function formatElapsedStatusValue(
+  state: PiUsereqStatusState,
+  nowMs: number,
+): string {
+  const activeText = state.runStartTimeMs === undefined
+    ? "--:--"
+    : formatStatusDuration(nowMs - state.runStartTimeMs);
+  const lastText = formatCompletedStatusDuration(state.lastRunDurationMs);
+  const totalText = formatCompletedStatusDuration(state.totalRunDurationMs);
+  return ` ⏱︎ ${activeText} ⚑ ${lastText} ⌛︎ ${totalText}`;
+}
+
 /**
  * @brief Formats one standard status-bar field.
  * @details Renders the field label in accent color and the value in warning
@@ -431,52 +478,38 @@ function didAgentEndAbort(messages: AgentEndEvent["messages"]): boolean {
 
 /**
  * @brief Builds the full single-line pi-usereq status-bar payload.
- * @details Renders git, base, docs, tests, src, tools, context, elapsed, last, beep, and sound fields in the canonical order with dim bullet separators and threshold-specific context-bar overlays. Runtime is O(s) in configured source-path count plus runtime git probing. No external state is mutated.
- * @param[in] cwd {string} Runtime working directory used for git/base path derivation.
+ * @details Renders base, context, elapsed, beep, sound, and pushover fields in the canonical order with dim bullet separators and threshold-specific context-bar overlays. Runtime is O(1). No external state is mutated.
+ * @param[in] cwd {string} Runtime working directory used for base-path derivation.
  * @param[in] config {UseReqConfig} Effective project configuration.
- * @param[in] activeTools {readonly string[]} Active runtime tool names.
  * @param[in] theme {StatusThemeAdapter} Normalized status theme.
  * @param[in] state {PiUsereqStatusState} Mutable status state snapshot.
  * @param[in] nowMs {number} Current wall-clock time in milliseconds.
  * @return {string} Single-line status-bar text.
- * @satisfies REQ-109, REQ-112, REQ-120, REQ-121, REQ-123, REQ-124, REQ-125, REQ-126, REQ-127, REQ-128, REQ-135, REQ-136, REQ-147, REQ-148, REQ-156
+ * @satisfies REQ-109, REQ-112, REQ-120, REQ-121, REQ-123, REQ-124, REQ-125, REQ-126, REQ-127, REQ-128, REQ-135, REQ-136, REQ-148, REQ-156, REQ-159, REQ-170, REQ-171
  */
 function buildPiUsereqStatusText(
   cwd: string,
   config: UseReqConfig,
-  activeTools: readonly string[],
   theme: StatusThemeAdapter,
   state: PiUsereqStatusState,
   nowMs: number,
 ): string {
-  const gitPath = resolveRuntimeGitPath(cwd);
-  const gitText = formatAbsoluteGitPath(gitPath);
-  const baseText = formatBasePathRelativeToGitPath(cwd, gitPath);
-  const sourcePaths = config["src-dir"].join(",");
-  const elapsedText = state.runStartTimeMs === undefined
-    ? "idle"
-    : formatStatusDuration(nowMs - state.runStartTimeMs);
-  const lastText = state.lastRunDurationMs === undefined
-    ? "N/A"
-    : formatStatusDuration(state.lastRunDurationMs);
+  const baseText = normalizePathSlashes(path.resolve(cwd));
+  const elapsedText = formatElapsedStatusValue(state, nowMs);
   const beepText = formatPiNotifyBeepStatus(config);
   const soundText = config["notify-sound"];
+  const pushoverText = formatPiNotifyPushoverStatus(config);
   return [
-    formatStatusField(theme, "git", gitText),
     formatStatusField(theme, "base", baseText),
-    formatStatusField(theme, "docs", config["docs-dir"]),
-    formatStatusField(theme, "tests", config["tests-dir"]),
-    formatStatusField(theme, "src", sourcePaths),
-    formatStatusField(theme, "tools", String(activeTools.length)),
     formatRenderedStatusField(
       theme,
       "context",
       formatContextUsageBar(theme, state.contextUsage),
     ),
     formatStatusField(theme, "elapsed", elapsedText),
-    formatStatusField(theme, "last", lastText),
     formatStatusField(theme, "beep", beepText),
     formatStatusField(theme, "sound", soundText),
+    formatStatusField(theme, "pushover", pushoverText),
   ].join(theme.separator);
 }
 
@@ -525,24 +558,21 @@ function syncPiUsereqStatusTicker(
 
 /**
  * @brief Creates an empty pi-usereq status controller.
- * @details Initializes the mutable status snapshot, stores the active-tools
- * provider used by render-time tool counting, and starts with no config, no
- * context, and no live ticker. Runtime is O(1). No external state is mutated.
- * @param[in] getActiveTools {() => readonly string[]} Provider for active tools.
+ * @details Initializes the mutable status snapshot, including empty prompt-request tracking, and starts with no config, no context, and no live ticker. Runtime is O(1). No external state is mutated.
  * @return {PiUsereqStatusController} New status controller.
  * @satisfies DES-010
  */
-export function createPiUsereqStatusController(
-  getActiveTools: () => readonly string[],
-): PiUsereqStatusController {
+export function createPiUsereqStatusController(): PiUsereqStatusController {
   return {
-    getActiveTools,
     config: undefined,
     latestContext: undefined,
     state: {
       contextUsage: undefined,
       runStartTimeMs: undefined,
       lastRunDurationMs: undefined,
+      totalRunDurationMs: undefined,
+      pendingPromptRequest: undefined,
+      activePromptRequest: undefined,
     },
     tickHandle: undefined,
   };
@@ -574,7 +604,7 @@ export function setPiUsereqStatusConfig(
  * @param[in,out] controller {PiUsereqStatusController} Mutable status controller.
  * @param[in] ctx {ExtensionContext} Active extension context.
  * @return {void} No return value.
- * @satisfies REQ-120, REQ-121, REQ-123, REQ-124, REQ-125, REQ-126, REQ-127, REQ-128, REQ-135, REQ-136
+ * @satisfies REQ-120, REQ-121, REQ-123, REQ-124, REQ-125, REQ-126, REQ-127, REQ-128, REQ-135, REQ-136, REQ-148, REQ-159, REQ-170, REQ-171
  */
 export function renderPiUsereqStatus(
   controller: PiUsereqStatusController,
@@ -590,7 +620,6 @@ export function renderPiUsereqStatus(
     buildPiUsereqStatusText(
       ctx.cwd,
       controller.config,
-      controller.getActiveTools(),
       theme,
       controller.state,
       Date.now(),
@@ -601,17 +630,17 @@ export function renderPiUsereqStatus(
 /**
  * @brief Updates mutable status state for one intercepted lifecycle hook.
  * @details Refreshes stored context usage on every hook, starts run timing on
- * `agent_start`, captures non-aborted run duration on `agent_end`, clears live
- * timing on shutdown, synchronizes the live ticker, and re-renders the status
- * bar when configuration is available. Runtime is O(n) in `agent_end` message
- * count and otherwise O(1). Side effects include in-memory state mutation,
- * interval scheduling, and footer-status updates.
+ * `agent_start`, promotes pending prompt-request metadata into the active run, captures non-aborted run duration on `agent_end`, accumulates successful runtime into `Σ`, clears live timing on shutdown, synchronizes
+ * the live ticker, and re-renders the status bar when configuration is
+ * available. Runtime is O(n) in `agent_end` message count and otherwise O(1).
+ * Side effects include in-memory state mutation, interval scheduling, and
+ * footer-status updates.
  * @param[in,out] controller {PiUsereqStatusController} Mutable status controller.
  * @param[in] hookName {PiUsereqStatusHookName} Intercepted hook name.
  * @param[in] event {unknown} Hook payload forwarded from the wrapper.
  * @param[in] ctx {ExtensionContext} Active extension context.
  * @return {void} No return value.
- * @satisfies REQ-117, REQ-118, REQ-119, REQ-123, REQ-124, REQ-125
+ * @satisfies REQ-117, REQ-118, REQ-119, REQ-123, REQ-124, REQ-125, REQ-159, REQ-169
  */
 export function updateExtensionStatus(
   controller: PiUsereqStatusController,
@@ -625,18 +654,25 @@ export function updateExtensionStatus(
 
   if (hookName === "agent_start") {
     controller.state.runStartTimeMs = nowMs;
+    controller.state.activePromptRequest = controller.state.pendingPromptRequest;
+    controller.state.pendingPromptRequest = undefined;
   }
 
   if (hookName === "agent_end" && controller.state.runStartTimeMs !== undefined) {
     const durationMs = nowMs - controller.state.runStartTimeMs;
     if (!didAgentEndAbort((event as AgentEndEvent).messages ?? [])) {
       controller.state.lastRunDurationMs = durationMs;
+      controller.state.totalRunDurationMs = controller.state.totalRunDurationMs === undefined
+        ? durationMs
+        : controller.state.totalRunDurationMs + durationMs;
     }
     controller.state.runStartTimeMs = undefined;
   }
 
   if (hookName === "session_shutdown") {
     controller.state.runStartTimeMs = undefined;
+    controller.state.pendingPromptRequest = undefined;
+    controller.state.activePromptRequest = undefined;
   }
 
   syncPiUsereqStatusTicker(controller);

package/src/core/find-payload.ts

@@ -156,10 +156,11 @@ export interface FindToolRequestSection {
 
 /**
  * @brief Describes the summary section of the find payload.
- * @details Exposes aggregate file, match, line, and Doxygen counts as numeric fields plus one stable search-status discriminator. The interface is compile-time only and introduces no runtime cost.
+ * @details Exposes aggregate file, match, line, and Doxygen counts as numeric fields plus one stable search-status discriminator and the normalized validation error when request parsing fails. The interface is compile-time only and introduces no runtime cost.
  */
 export interface FindToolSummarySection {
   search_status: FindSearchStatus;
+  validation_error_message?: string;
   processable_file_count: number;
   matched_file_count: number;
   no_match_file_count: number;
@@ -172,22 +173,20 @@ export interface FindToolSummarySection {
 
 /**
  * @brief Describes the repository section of the find payload.
- * @details Stores the base path, configured source-directory scope, canonical file list, and supported-tag matrix needed to specialize later searches without rereading tool descriptions. The interface is compile-time only and introduces no runtime cost.
+ * @details Stores the base path, configured source-directory scope, and canonical file list used during search while omitting the static supported-tag matrix because that data belongs in tool registration metadata. The interface is compile-time only and introduces no runtime cost.
  */
 export interface FindToolRepositorySection {
   root_directory_path: string;
   source_directory_paths: string[];
   file_count: number;
   file_canonical_paths: string[];
-  supported_tags_by_language: Record<string, string[]>;
 }
 
 /**
  * @brief Describes the full agent-oriented find payload.
- * @details Orders the top-level sections as request, summary, repository, and files so execution metadata can be appended deterministically by the tool wrapper. The interface is compile-time only and introduces no runtime cost.
+ * @details Exposes only aggregate search totals, repository scope, and per-file match records, omitting request echoes and static supported-tag matrices that already belong in registration metadata. The interface is compile-time only and introduces no runtime cost.
  */
 export interface FindToolPayload {
-  request: FindToolRequestSection;
   summary: FindToolSummarySection;
   repository: FindToolRepositorySection;
   files: FindToolFileEntry[];
@@ -262,19 +261,6 @@ function buildLineRange(startLineNumber: number, endLineNumber: number): FindLin
   };
 }
 
-/**
- * @brief Returns the supported-tag matrix ordered for deterministic JSON emission.
- * @details Sorts languages alphabetically and tag arrays lexicographically so downstream agents can reuse the matrix without reparsing human prose. Runtime is O(l * t log t). No side effects occur.
- * @return {Record<string, string[]>} Supported tags keyed by canonical language identifier.
- */
-function buildSupportedTagsByLanguage(): Record<string, string[]> {
-  return Object.fromEntries(
-    Object.entries(LANGUAGE_TAGS)
-      .sort(([left], [right]) => left.localeCompare(right))
-      .map(([language, tagSet]) => [language, [...tagSet].sort()]),
-  );
-}
-
 /**
  * @brief Resolves one stable symbol name from an analyzed element.
  * @details Prefers explicit analyzer name metadata, then falls back to the derived signature or the first source line so every matched construct retains a direct-access identifier. Runtime is O(1). No side effects occur.
@@ -717,9 +703,9 @@ function analyzeFindFile(
 
 /**
  * @brief Builds the full agent-oriented find payload.
- * @details Validates request parameters, analyzes requested files in caller order when the request is valid, preserves skipped and no-match outcomes in structured file entries, computes aggregate numeric totals, and emits a structured supported-tag matrix. Runtime is O(F log F + S + M). Side effects are limited to filesystem reads and optional stderr logging.
+ * @details Validates request parameters, analyzes requested files in caller order when the request is valid, preserves skipped and no-match outcomes in structured file entries, computes aggregate numeric totals, and omits request echoes plus the static supported-tag matrix already encoded in registration metadata. Runtime is O(F log F + S + M). Side effects are limited to filesystem reads and optional stderr logging.
  * @param[in] options {BuildFindToolPayloadOptions} Payload-construction options.
- * @return {FindToolPayload} Structured find payload ordered as request, summary, repository, and files.
+ * @return {FindToolPayload} Structured find payload ordered as summary, repository, and files.
  * @satisfies REQ-089, REQ-090, REQ-091, REQ-092, REQ-093, REQ-094, REQ-096, REQ-098
  */
 export function buildFindToolPayload(options: BuildFindToolPayloadOptions): FindToolPayload {
@@ -835,28 +821,20 @@ export function buildFindToolPayload(options: BuildFindToolPayloadOptions): Find
   const processableFiles = files.filter((file) => file.status !== "skipped");
   const repositoryFileCanonicalPaths = files.map((file) => file.canonical_path);
 
+  void toolName;
+  void scope;
+  void lineNumberMode;
+  void tagFilter;
+  void pattern;
+  void canonicalRequestedPaths;
   return {
-    request: {
-      tool_name: toolName,
-      scope,
-      base_dir_path: absoluteBaseDir,
-      line_number_mode: lineNumberMode,
-      tag_filter_text: tagFilter,
-      tag_filter_values: tagValidation.tagValues,
-      tag_filter_status: tagValidation.status,
-      tag_filter_error_message: tagValidation.errorMessage,
-      name_regex_text: pattern,
-      regex_engine: "javascript-regexp-search",
-      regex_status: regexValidation.status,
-      regex_error_message: regexValidation.errorMessage,
-      source_directory_count: sourceDirectoryPaths.length,
-      source_directory_paths: sourceDirectoryPaths.map((sourceDirectoryPath) => canonicalizeFindPath(sourceDirectoryPath, absoluteBaseDir)),
-      requested_file_count: requestedPaths.length,
-      requested_input_paths: [...requestedPaths],
-      requested_canonical_paths: canonicalRequestedPaths,
-    },
     summary: {
       search_status: searchStatus,
+      validation_error_message: tagValidation.status === "invalid"
+        ? tagValidation.errorMessage
+        : regexValidation.status === "invalid"
+          ? regexValidation.errorMessage
+          : undefined,
       processable_file_count: processableFiles.length,
       matched_file_count: matchedFiles.length,
       no_match_file_count: files.filter((file) => file.status === "no_match").length,
@@ -874,7 +852,6 @@ export function buildFindToolPayload(options: BuildFindToolPayloadOptions): Find
       source_directory_paths: sourceDirectoryPaths.map((sourceDirectoryPath) => canonicalizeFindPath(sourceDirectoryPath, absoluteBaseDir)),
       file_count: repositoryFileCanonicalPaths.length,
       file_canonical_paths: repositoryFileCanonicalPaths,
-      supported_tags_by_language: buildSupportedTagsByLanguage(),
     },
     files,
   };
@@ -889,11 +866,11 @@ export function buildFindToolPayload(options: BuildFindToolPayloadOptions): Find
  */
 export function buildFindToolExecutionStderr(payload: FindToolPayload): string {
   const diagnostics: string[] = [];
-  if (payload.request.tag_filter_status === "invalid") {
-    diagnostics.push(`error: tag_filter: ${payload.request.tag_filter_error_message ?? "invalid tag filter"}`);
+  if (payload.summary.search_status === "invalid_tag_filter") {
+    diagnostics.push(`error: tag_filter: ${payload.summary.validation_error_message ?? "invalid tag filter"}`);
   }
-  if (payload.request.regex_status === "invalid") {
-    diagnostics.push(`error: name_regex: ${payload.request.regex_error_message ?? "invalid regex"}`);
+  if (payload.summary.search_status === "invalid_regex") {
+    diagnostics.push(`error: name_regex: ${payload.summary.validation_error_message ?? "invalid regex"}`);
   }
   payload.files.forEach((file) => {
     if (file.status === "skipped") {