pi-usereq 0.4.0 → 0.6.0

package/tests/extension-registration.test.ts

@@ -4,7 +4,13 @@ import { spawnSync } from "node:child_process";
 import fs from "node:fs";
 import path from "node:path";
 import piUsereqExtension from "../src/index.ts";
-import { DEFAULT_DOCS_DIR, getProjectConfigPath } from "../src/core/config.js";
+import {
+  DEFAULT_DOCS_DIR,
+  getDefaultConfig,
+  getProjectConfigPath,
+} from "../src/core/config.js";
+import { formatRuntimePathForDisplay } from "../src/core/path-context.js";
+import { setPiNotifyHttpsRequestForTests } from "../src/core/pi-notify.js";
 import {
   PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES,
   PI_USEREQ_EMBEDDED_TOOL_NAMES,
@@ -284,9 +290,9 @@ function formatFakeThemeBackgroundFromForeground(color: string, text: string): s
 
 /**
  * @brief Builds the expected fake context-bar payload for assertions.
- * @details Resolves the empty-state `CLEAR` overlay for unavailable or
- * non-positive percent, resolves the high-water `FULL!` overlay for percent
- * above 90, and otherwise reconstructs the 5-cell bar. Runtime is O(1). No
+ * @details Resolves the empty-state `◀ CLEAR ▶ ` overlay for unavailable or
+ * non-positive percent, resolves the centered ` ◀ FULL ▶ ` overlay for percent
+ * above 90, and otherwise reconstructs the 10-cell bar. Runtime is O(1). No
  * external state is mutated.
  * @param[in] options {{ filledCells: number; percent?: number | null }} Expected context-bar facts.
  * @return {string} Encoded context-bar string.
@@ -298,16 +304,16 @@ function buildExpectedFakeContextBar(options: {
   if (options.percent === undefined || options.percent === null || options.percent <= 0) {
     return formatFakeThemeBackgroundFromForeground(
       "accent",
-      formatFakeThemeForeground("warning", "CLEAR"),
+      formatFakeThemeForeground("warning", "◀ CLEAR ▶ "),
     );
   }
   if (options.percent > 90) {
     return formatFakeThemeBackgroundFromForeground(
       "warning",
-      formatFakeThemeForeground("error", "FULL!"),
+      formatFakeThemeForeground("error", " ◀ FULL ▶ "),
     );
   }
-  return Array.from({ length: 5 }, (_value, index) =>
+  return Array.from({ length: 10 }, (_value, index) =>
     formatFakeThemeBackgroundFromForeground(
       "accent",
       formatFakeThemeForeground(index < options.filledCells ? "warning" : "dim", "▓"),
@@ -317,23 +323,21 @@ function buildExpectedFakeContextBar(options: {
 
 /**
  * @brief Builds the expected fake pi-usereq status-bar string for assertions.
- * @details Reconstructs the field order, separators, context bar, and timing fields emitted by the extension using deterministic fake theme markers. Runtime is O(s) in source-path count. No external state is mutated.
- * @param[in] options {{ gitPath?: string; basePath?: string; docsDir: string; testsDir: string; srcDir: string[]; toolCount: number; contextFilledCells: number; contextPercent?: number | null; elapsed: string; last: string; beep?: string; sound?: string }} Expected status facts.
+ * @details Reconstructs the field order, separators, context bar, consolidated elapsed field, and Pushover enable field emitted by the extension using deterministic fake theme markers. Runtime is O(1). No external state is mutated.
+ * @param[in] options {{ basePath: string; contextFilledCells: number; contextPercent?: number | null; et: string; beep?: string; sound?: string; pushover?: string }} Expected status facts.
  * @return {string} Encoded status-bar string.
  */
 function buildExpectedFakeStatusText(options: {
-  gitPath?: string;
-  basePath?: string;
-  docsDir: string;
-  testsDir: string;
-  srcDir: string[];
-  toolCount: number;
+  basePath: string;
+  docsDir?: string;
+  testsDir?: string;
+  srcDir?: string[];
   contextFilledCells: number;
   contextPercent?: number | null;
-  elapsed: string;
-  last: string;
+  et: string;
   beep?: string;
   sound?: string;
+  pushover?: string;
 }): string {
   const buildField = (fieldName: string, value: string): string =>
     `${formatFakeThemeForeground("accent", `${fieldName}:`)}${formatFakeThemeForeground("warning", value)}`;
@@ -342,20 +346,27 @@ function buildExpectedFakeStatusText(options: {
     percent: options.contextPercent,
   });
   return [
-    buildField("git", options.gitPath ?? ""),
-    buildField("base", options.basePath ?? "."),
-    buildField("docs", options.docsDir),
-    buildField("tests", options.testsDir),
-    buildField("src", options.srcDir.join(",")),
-    buildField("tools", String(options.toolCount)),
+    buildField("base", options.basePath),
     `${formatFakeThemeForeground("accent", "context:")}${contextBar}`,
-    buildField("elapsed", options.elapsed),
-    buildField("last", options.last),
-    buildField("beep", options.beep ?? "none"),
+    buildField("elapsed", options.et),
+    buildField("beep", options.beep ?? "end,esc,err"),
     buildField("sound", options.sound ?? "none"),
+    buildField("pushover", options.pushover ?? "off"),
   ].join(formatFakeThemeForeground("dim", " • "));
 }
 
+/**
+ * @brief Formats one expected absolute base path for fake status assertions.
+ * @details Resolves the supplied cwd and normalizes path separators to `/` so
+ * fake status comparisons stay stable across operating systems. Runtime is O(p)
+ * in path length. No external state is mutated.
+ * @param[in] cwd {string} Runtime working directory.
+ * @return {string} Slash-normalized absolute base path.
+ */
+function buildExpectedFakeBasePath(cwd: string): string {
+  return path.resolve(cwd).split(path.sep).join("/");
+}
+
 /**
  * @brief Creates a fake extension command context with scripted UI interactions.
  * @details Materializes deterministic `select`, shared settings-menu `custom`, `input`, `notify`, `setStatus`, `setEditorText`, `waitForIdle`, `newSession`, `sessionManager`, `getContextUsage`, theme-color behaviors, and captured custom-menu render output backed by in-memory state so menu-oriented and prompt-command tests can assert side effects without a real UI. Optional `onNewSession` and theme overrides let tests emulate runtime replacement and strict token validation. Runtime is O(1) plus queued interaction count and delegated new-session setup cost. Side effects are limited to in-memory state mutation.
@@ -595,13 +606,13 @@ test("token tools register agent-oriented descriptions and schema details", () =
   assert.ok(filesTokens, "missing files-tokens tool");
   assert.ok(tokens, "missing tokens tool");
 
-  assert.match(filesTokens.description ?? "", /LLM-oriented JSON payload/);
+  assert.match(filesTokens.description ?? "", /token-optimized JSON payload/);
   assert.ok(filesTokens.promptGuidelines?.some((line) => line.includes("Output contract:")));
-  assert.match(String((filesTokens.parameters as { description?: string } | undefined)?.description ?? ""), /line ranges/);
+  assert.match(String((filesTokens.parameters as { description?: string } | undefined)?.description ?? ""), /token metrics/);
 
   assert.match(tokens.description ?? "", /canonical docs/);
-  assert.ok(tokens.promptGuidelines?.some((line) => line.includes("docs_dir_path")));
-  assert.match(String((tokens.parameters as { description?: string } | undefined)?.description ?? ""), /canonical_doc_names/);
+  assert.ok(tokens.promptGuidelines?.some((line) => line.includes("registration metadata")));
+  assert.match(String((tokens.parameters as { description?: string } | undefined)?.description ?? ""), /token-optimized JSON shape/);
 });
 
 test("reference tools register agent-oriented descriptions and schema details", () => {
@@ -613,7 +624,7 @@ test("reference tools register agent-oriented descriptions and schema details",
   assert.ok(filesReferences, "missing files-references tool");
   assert.ok(references, "missing references tool");
 
-  assert.match(filesReferences.description ?? "", /request, summary, repository, files, and execution sections/);
+  assert.match(filesReferences.description ?? "", /summary, repository, files, and execution sections/);
   assert.ok(filesReferences.promptGuidelines?.some((line) => line.includes("Numeric contract:")));
   assert.ok(filesReferences.promptGuidelines?.some((line) => line.includes("Behavior contract:")));
   assert.match(String((filesReferences.parameters as { description?: string } | undefined)?.description ?? ""), /structured Doxygen fields/);
@@ -633,11 +644,11 @@ test("find tools register agent-oriented descriptions and schema details", () =>
   assert.ok(filesFind, "missing files-find tool");
   assert.ok(find, "missing find tool");
 
-  assert.match(filesFind.description ?? "", /LLM-oriented JSON payload/);
+  assert.match(filesFind.description ?? "", /token-optimized JSON payload/);
   assert.ok(filesFind.promptGuidelines?.some((line) => line.includes("Regex rule:")));
   assert.ok(filesFind.promptGuidelines?.some((line) => line.includes("Supported tags [Typescript]:")));
   assert.ok(filesFind.promptGuidelines?.some((line) => line.includes("Supported tags [Python]:")));
-  assert.match(String((filesFind.parameters as { description?: string } | undefined)?.description ?? ""), /supported_tags_by_language/);
+  assert.match(String((filesFind.parameters as { description?: string } | undefined)?.description ?? ""), /Static supported-tag matrices are documented in tool registration metadata/);
 
   assert.match(find.description ?? "", /configured project source directories/);
   assert.ok(find.promptGuidelines?.some((line) => line.includes("source_directory_paths")));
@@ -654,7 +665,7 @@ test("compression tools register agent-oriented descriptions and schema details"
   assert.ok(filesCompress, "missing files-compress tool");
   assert.ok(compress, "missing compress tool");
 
-  assert.match(filesCompress.description ?? "", /request, summary, repository, files, and execution sections/);
+  assert.match(filesCompress.description ?? "", /summary, repository, files, and execution sections/);
   assert.ok(filesCompress.promptGuidelines?.some((line) => line.includes("Line-number behavior:")));
   assert.ok(filesCompress.promptGuidelines?.some((line) => line.includes("Behavior contract:")));
   assert.match(String((filesCompress.parameters as { description?: string } | undefined)?.description ?? ""), /structured compressed lines/);
@@ -665,7 +676,7 @@ test("compression tools register agent-oriented descriptions and schema details"
   assert.match(String((compress.parameters as { description?: string } | undefined)?.description ?? ""), /structured compressed lines/);
 });
 
-test("files-tokens returns structured source facts and separated guidance", async () => {
+test("files-tokens returns token-optimized structured source facts", async () => {
   const { projectBase } = initFixtureRepo({
     fixtures: [],
     docs: {
@@ -682,10 +693,6 @@ test("files-tokens returns structured source facts and separated guidance", asyn
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          base_dir_path: string;
-          requested_input_paths: string[];
-        };
         summary: {
           counted_file_count: number;
           skipped_file_count: number;
@@ -698,14 +705,8 @@ test("files-tokens returns structured source facts and separated guidance", asyn
           line_count: number;
           byte_count: number;
           primary_heading_text?: string;
+          error_message?: string;
         }>;
-        guidance: {
-          source_observations: {
-            skipped_inputs: Array<{ input_path: string; canonical_path: string; reason: string }>;
-          };
-          derived_recommendations: Array<{ kind: string; ordered_paths: string[] }>;
-          actionable_next_steps: Array<{ kind: string; ordered_paths: string[] }>;
-        };
         execution: {
           stderr: string;
         };
@@ -714,8 +715,6 @@ test("files-tokens returns structured source facts and separated guidance", asyn
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.equal(payload.request.base_dir_path, projectBase);
-    assert.deepEqual(payload.request.requested_input_paths, [`${DEFAULT_DOCS_DIR}/REQUIREMENTS.md`, `${DEFAULT_DOCS_DIR}/MISSING.md`]);
     assert.equal(payload.summary.counted_file_count, 1);
     assert.equal(payload.summary.skipped_file_count, 1);
     assert.equal(payload.files[0]?.canonical_path, `${DEFAULT_DOCS_DIR}/REQUIREMENTS.md`);
@@ -723,11 +722,8 @@ test("files-tokens returns structured source facts and separated guidance", asyn
     assert.equal(payload.files[0]?.end_line_number, payload.files[0]?.line_count);
     assert.ok((payload.files[0]?.byte_count ?? 0) > 0);
     assert.equal(payload.files[0]?.primary_heading_text, "Requirements");
-    assert.deepEqual(payload.guidance.source_observations.skipped_inputs, [
-      { input_path: `${DEFAULT_DOCS_DIR}/MISSING.md`, canonical_path: `${DEFAULT_DOCS_DIR}/MISSING.md`, reason: "not found" },
-    ]);
-    assert.equal(payload.guidance.derived_recommendations[0]?.kind, "prioritize_high_token_paths");
-    assert.equal(payload.guidance.actionable_next_steps[0]?.kind, "read_top_token_paths_first");
+    assert.equal(payload.files[1]?.canonical_path, `${DEFAULT_DOCS_DIR}/MISSING.md`);
+    assert.equal(payload.files[1]?.error_message, "not found");
     assert.match(payload.execution.stderr, new RegExp(`skipped: ${DEFAULT_DOCS_DIR.replace("/", "\\/")}/MISSING\\.md: not found`));
   } finally {
     fs.rmSync(projectBase, { recursive: true, force: true });
@@ -760,9 +756,6 @@ export function buildSample(input: string): string {
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          requested_input_paths: string[];
-        };
         summary: {
           analyzed_file_count: number;
           skipped_file_count: number;
@@ -797,7 +790,6 @@ export function buildSample(input: string): string {
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.deepEqual(payload.request.requested_input_paths, ["src/sample.ts", "src/missing.ts"]);
     assert.equal(payload.summary.analyzed_file_count, 1);
     assert.equal(payload.summary.skipped_file_count, 1);
     assert.deepEqual(payload.repository.file_canonical_paths, ["src/sample.ts"]);
@@ -901,10 +893,6 @@ export function buildSample(input: string): string {
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          line_number_mode: string;
-          requested_input_paths: string[];
-        };
         summary: {
           compressed_file_count: number;
           skipped_file_count: number;
@@ -939,8 +927,6 @@ export function buildSample(input: string): string {
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.equal(payload.request.line_number_mode, "enabled");
-    assert.deepEqual(payload.request.requested_input_paths, ["src/sample.ts", "src/missing.ts"]);
     assert.equal(payload.summary.compressed_file_count, 1);
     assert.equal(payload.summary.skipped_file_count, 1);
     assert.equal(payload.summary.total_symbol_count, 1);
@@ -988,10 +974,6 @@ test("compress returns a structured repository-scoped compression payload", asyn
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          scope: string;
-          line_number_mode: string;
-        };
         summary: {
           compressed_file_count: number;
           total_compressed_line_count: number;
@@ -1015,8 +997,6 @@ test("compress returns a structured repository-scoped compression payload", asyn
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.equal(payload.request.scope, "configured-source-directories");
-    assert.equal(payload.request.line_number_mode, "disabled");
     assert.equal(payload.summary.compressed_file_count, 2);
     assert.ok((payload.summary.total_compressed_line_count ?? 0) >= 2);
     assert.deepEqual(payload.repository.source_directory_paths, ["src"]);
@@ -1060,12 +1040,6 @@ export function buildSample(input: string): string {
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          tag_filter_values: string[];
-          tag_filter_status: string;
-          line_number_mode: string;
-          requested_input_paths: string[];
-        };
         summary: {
           search_status: string;
           matched_file_count: number;
@@ -1073,7 +1047,7 @@ export function buildSample(input: string): string {
           total_match_count: number;
         };
         repository: {
-          supported_tags_by_language: Record<string, string[]>;
+          file_canonical_paths: string[];
         };
         files: Array<{
           canonical_path: string;
@@ -1106,15 +1080,11 @@ export function buildSample(input: string): string {
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.deepEqual(payload.request.tag_filter_values, ["FUNCTION"]);
-    assert.equal(payload.request.tag_filter_status, "valid");
-    assert.equal(payload.request.line_number_mode, "enabled");
-    assert.deepEqual(payload.request.requested_input_paths, ["src/sample.ts", "src/missing.ts"]);
     assert.equal(payload.summary.search_status, "matched");
     assert.equal(payload.summary.matched_file_count, 1);
     assert.equal(payload.summary.skipped_file_count, 1);
     assert.equal(payload.summary.total_match_count, 1);
-    assert.ok(payload.repository.supported_tags_by_language.typescript?.includes("FUNCTION"));
+    assert.deepEqual(payload.repository.file_canonical_paths, ["src/sample.ts", "src/missing.ts"]);
     assert.deepEqual(payload.files.map((file) => file.status), ["matched", "skipped"]);
     assert.equal(payload.files[0]?.canonical_path, "src/sample.ts");
     assert.equal(payload.files[0]?.match_count, 1);
@@ -1171,11 +1141,6 @@ export function buildAlpha(): number {
     }) as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: {
-          scope: string;
-          regex_status: string;
-          requested_file_count: number;
-        };
         summary: {
           search_status: string;
           matched_file_count: number;
@@ -1185,7 +1150,6 @@ export function buildAlpha(): number {
         repository: {
           source_directory_paths: string[];
           file_canonical_paths: string[];
-          supported_tags_by_language: Record<string, string[]>;
         };
         files: Array<{
           canonical_path: string;
@@ -1205,16 +1169,12 @@ export function buildAlpha(): number {
 
     const payload = result.details!;
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(payload)));
-    assert.equal(payload.request.scope, "configured-source-directories");
-    assert.equal(payload.request.regex_status, "valid");
-    assert.equal(payload.request.requested_file_count, 2);
     assert.equal(payload.summary.search_status, "matched");
     assert.equal(payload.summary.matched_file_count, 1);
     assert.equal(payload.summary.no_match_file_count, 1);
     assert.equal(payload.summary.total_match_count, 1);
     assert.deepEqual(payload.repository.source_directory_paths, ["src"]);
     assert.deepEqual(payload.repository.file_canonical_paths, ["src/alpha.ts", "src/nested/beta.ts"]);
-    assert.ok(payload.repository.supported_tags_by_language.typescript?.includes("FUNCTION"));
     assert.deepEqual(payload.files.map((file) => file.status), ["matched", "no_match"]);
     assert.equal(payload.files[0]?.matches[0]?.symbol_name, "buildAlpha");
     assert.match(payload.files[0]?.matches[0]?.code_lines[0]?.display_text ?? "", /^export function buildAlpha/);
@@ -1246,31 +1206,28 @@ test("path, static-check, git, docs, and worktree tools return structured JSON p
       content?: Array<{ text?: string }>;
       details?: {
         result: { path_value: string; path_present: boolean };
-        runtime_paths: { git_path: string };
         execution: { code: number };
       };
     };
     assert.deepEqual(JSON.parse(gitPathResult.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(gitPathResult.details)));
-    assert.equal(gitPathResult.details?.result.path_value, gitPathResult.details?.runtime_paths.git_path);
+    assert.equal(gitPathResult.details?.result.path_value, projectBase);
     assert.equal(gitPathResult.details?.result.path_present, true);
     assert.equal(gitPathResult.details?.execution.code, 0);
 
     const basePathResult = await executeRegisteredTool(pi, "get-base-path", projectBase) as {
       details?: {
         result: { path_value: string };
-        runtime_paths: { base_path: string };
         execution: { code: number };
       };
     };
-    assert.equal(basePathResult.details?.result.path_value, basePathResult.details?.runtime_paths.base_path);
+    assert.equal(basePathResult.details?.result.path_value, projectBase);
     assert.equal(basePathResult.details?.execution.code, 0);
 
     const gitCheckResult = await executeRegisteredTool(pi, "git-check", projectBase) as {
-      details?: { result: { status: string; worktree_status: string; head_status: string }; execution: { code: number } };
+      details?: { result: { git_path_present: boolean; status: string }; execution: { code: number } };
     };
+    assert.equal(gitCheckResult.details?.result.git_path_present, true);
     assert.equal(gitCheckResult.details?.result.status, "clean");
-    assert.equal(gitCheckResult.details?.result.worktree_status, "clean");
-    assert.equal(gitCheckResult.details?.result.head_status, "valid");
     assert.equal(gitCheckResult.details?.execution.code, 0);
 
     const docsCheckResult = await executeRegisteredTool(pi, "docs-check", projectBase) as {
@@ -1306,43 +1263,34 @@ test("path, static-check, git, docs, and worktree tools return structured JSON p
 
     const staticCheckResult = await executeRegisteredTool(pi, "static-check", projectBase) as {
       details?: {
-        request: { selection_directory_paths: string[]; excluded_directory_paths: string[] };
         summary: { selected_file_count: number };
         files: Array<{ canonical_path: string; status: string }>;
         execution: { code: number };
       };
     };
-    assert.deepEqual(staticCheckResult.details?.request.selection_directory_paths, ["src", "tests"]);
-    assert.ok(staticCheckResult.details?.request.excluded_directory_paths.includes("tests/fixtures"));
     assert.equal(staticCheckResult.details?.summary.selected_file_count, 1);
     assert.equal(staticCheckResult.details?.files[0]?.canonical_path, "src/check.ts");
     assert.equal(staticCheckResult.details?.files[0]?.status, "selected");
     assert.equal(staticCheckResult.details?.execution.code, 0);
 
     const gitWtNameResult = await executeRegisteredTool(pi, "git-wt-name", projectBase) as {
-      details?: { result: { worktree_name?: string; format_text: string; status: string }; execution: { code: number } };
+      details?: { result: { worktree_name?: string }; execution: { code: number } };
     };
     const wtName = gitWtNameResult.details?.result.worktree_name ?? "";
-    assert.equal(gitWtNameResult.details?.result.status, "generated");
-    assert.equal(gitWtNameResult.details?.result.format_text, "useReq-<project>-<sanitized-branch>-<YYYYMMDDHHMMSS>");
     assert.match(wtName, /^useReq-/);
     assert.equal(gitWtNameResult.details?.execution.code, 0);
 
     const gitWtCreateResult = await executeRegisteredTool(pi, "git-wt-create", projectBase, { wtName }) as {
-      details?: { result: { status: string; worktree_name: string; branch_name: string; worktree_path: string }; execution: { code: number } };
+      details?: { result: { worktree_name: string; worktree_path: string }; execution: { code: number } };
     };
-    assert.equal(gitWtCreateResult.details?.result.status, "created");
     assert.equal(gitWtCreateResult.details?.result.worktree_name, wtName);
-    assert.equal(gitWtCreateResult.details?.result.branch_name, wtName);
     assert.match(gitWtCreateResult.details?.result.worktree_path ?? "", new RegExp(`${wtName}$`));
     assert.equal(gitWtCreateResult.details?.execution.code, 0);
 
     const gitWtDeleteResult = await executeRegisteredTool(pi, "git-wt-delete", projectBase, { wtName }) as {
-      details?: { result: { status: string; worktree_name: string; branch_name: string; worktree_path: string }; execution: { code: number } };
+      details?: { result: { worktree_name: string; worktree_path: string }; execution: { code: number } };
     };
-    assert.equal(gitWtDeleteResult.details?.result.status, "deleted");
     assert.equal(gitWtDeleteResult.details?.result.worktree_name, wtName);
-    assert.equal(gitWtDeleteResult.details?.result.branch_name, wtName);
     assert.match(gitWtDeleteResult.details?.result.worktree_path ?? "", new RegExp(`${wtName}$`));
     assert.equal(gitWtDeleteResult.details?.execution.code, 0);
   } finally {
@@ -1383,7 +1331,9 @@ test("configuration menu omits prompt-delivery controls and persisted config omi
 });
 
 test("configuration menus expose show-config ordering and omit overview or notification-reference rows", async () => {
-  const cwd = createTempDir("pi-usereq-menu-structure-");
+  const cwd = fs.mkdtempSync(
+    path.join(process.env.HOME ?? process.cwd(), "pi-usereq-menu-structure-"),
+  );
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
   const pi = createFakePi();
   piUsereqExtension(pi);
@@ -1393,6 +1343,7 @@ test("configuration menus expose show-config ordering and omit overview or notif
 
   await command!.handler("", ctx);
 
+  const renderedMenu = (ctx.__state.customRenderLines[0] ?? []).join("\n");
   assert.deepEqual(ctx.__state.selectCalls[0]?.items ?? [], [
     "docs-dir",
     "tests-dir",
@@ -1400,10 +1351,11 @@ test("configuration menus expose show-config ordering and omit overview or notif
     "static-check",
     "startup tools",
     "notifications",
-    "Reset defaults",
     "show-config",
+    "Reset defaults",
     "Save and close",
   ]);
+  assert.ok(renderedMenu.includes(formatRuntimePathForDisplay(getProjectConfigPath(cwd))));
   assert.deepEqual(ctx.__state.selectCalls[2]?.items ?? [], [
     "Toggle beep on success",
     "Toggle beep on escape",
@@ -1413,6 +1365,7 @@ test("configuration menus expose show-config ordering and omit overview or notif
     "Notify command (low vol.)",
     "Notify command (mid vol.)",
     "Notify command (high vol.)",
+    "Pushover notifications",
     "Back",
   ]);
 });
@@ -1493,13 +1446,12 @@ test("session_start applies configured pi-usereq startup tools", async () => {
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src", "foobar"],
-      toolCount: 2,
       contextFilledCells: 0,
-      elapsed: "idle",
-      last: "N/A",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
     }),
   );
 });
@@ -1518,14 +1470,13 @@ test("session_start renders the CLEAR overlay when context usage is empty", asyn
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src"],
-      toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
       contextFilledCells: 0,
       contextPercent: 0,
-      elapsed: "idle",
-      last: "N/A",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
     }),
   );
 });
@@ -1586,14 +1537,13 @@ test("context hook refreshes context usage and rounds progress cells upward", as
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src"],
-      toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
-      contextFilledCells: 1,
+      contextFilledCells: 2,
       contextPercent: 19.1,
-      elapsed: "idle",
-      last: "N/A",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
     }),
   );
 });
@@ -1614,14 +1564,13 @@ test("context hook renders the FULL! overlay when usage exceeds ninety percent",
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src"],
-      toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
-      contextFilledCells: 5,
+      contextFilledCells: 10,
       contextPercent: 91,
-      elapsed: "idle",
-      last: "N/A",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
     }),
   );
 });
@@ -1657,19 +1606,18 @@ test("status FULL overlay uses only CLI-supported theme tokens", async () => {
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src"],
-      toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
-      contextFilledCells: 5,
+      contextFilledCells: 10,
       contextPercent: 91,
-      elapsed: "idle",
-      last: "N/A",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
     }),
   );
 });
 
-test("agent timing status updates elapsed and last while preserving aborted last values", async () => {
+test("agent timing status updates et while preserving accumulated completed runtime", async () => {
   const cwd = createTempDir("pi-usereq-timing-status-");
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
 
@@ -1698,13 +1646,12 @@ test("agent timing status updates elapsed and last while preserving aborted last
     assert.equal(
       ctx.__state.statuses.get("pi-usereq"),
       buildExpectedFakeStatusText({
+        basePath: buildExpectedFakeBasePath(cwd),
         docsDir: DEFAULT_DOCS_DIR,
         testsDir: "tests",
         srcDir: ["src"],
-        toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
         contextFilledCells: 0,
-        elapsed: "0:00",
-        last: "N/A",
+        et: " ⏱︎ 0:00 ⚑ --:-- ⌛︎ --:--",
       }),
     );
 
@@ -1713,13 +1660,12 @@ test("agent timing status updates elapsed and last while preserving aborted last
     assert.equal(
       ctx.__state.statuses.get("pi-usereq"),
       buildExpectedFakeStatusText({
+        basePath: buildExpectedFakeBasePath(cwd),
         docsDir: DEFAULT_DOCS_DIR,
         testsDir: "tests",
         srcDir: ["src"],
-        toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
         contextFilledCells: 0,
-        elapsed: "1:01",
-        last: "N/A",
+        et: " ⏱︎ 1:01 ⚑ --:-- ⌛︎ --:--",
       }),
     );
 
@@ -1735,13 +1681,12 @@ test("agent timing status updates elapsed and last while preserving aborted last
     assert.equal(
       ctx.__state.statuses.get("pi-usereq"),
       buildExpectedFakeStatusText({
+        basePath: buildExpectedFakeBasePath(cwd),
         docsDir: DEFAULT_DOCS_DIR,
         testsDir: "tests",
         srcDir: ["src"],
-        toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
         contextFilledCells: 0,
-        elapsed: "idle",
-        last: "1:01",
+        et: " ⏱︎ --:-- ⚑ 1:01 ⌛︎ 1:01",
       }),
     );
 
@@ -1760,13 +1705,12 @@ test("agent timing status updates elapsed and last while preserving aborted last
     assert.equal(
       ctx.__state.statuses.get("pi-usereq"),
       buildExpectedFakeStatusText({
+        basePath: buildExpectedFakeBasePath(cwd),
         docsDir: DEFAULT_DOCS_DIR,
         testsDir: "tests",
         srcDir: ["src"],
-        toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
         contextFilledCells: 0,
-        elapsed: "idle",
-        last: "1:01",
+        et: " ⏱︎ --:-- ⚑ 1:01 ⌛︎ 1:01",
       }),
     );
     assert.ok(clearedHandles.length >= 2);
@@ -1794,6 +1738,14 @@ test("session_start enables default custom tools and default embedded tools only
   assert.equal(activeTools.has("ls"), false);
 });
 
+test("default configuration enables all pi-notify beep outcomes", () => {
+  const config = getDefaultConfig(createTempDir("pi-usereq-default-notify-"));
+
+  assert.equal(config["notify-beep-on-end"], true);
+  assert.equal(config["notify-beep-on-esc"], true);
+  assert.equal(config["notify-beep-on-error"], true);
+});
+
 test("configuration menu can enable embedded builtin tools", async () => {
   const cwd = createTempDir("pi-usereq-menu-embedded-");
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
@@ -1866,9 +1818,9 @@ test("configuration menu can persist pi-notify settings", async () => {
   await command!.handler("", ctx);
 
   const config = JSON.parse(fs.readFileSync(getProjectConfigPath(cwd), "utf8"));
-  assert.equal(config["notify-beep-on-end"], true);
-  assert.equal(config["notify-beep-on-error"], true);
-  assert.equal(config["notify-beep-on-esc"], false);
+  assert.equal(config["notify-beep-on-end"], false);
+  assert.equal(config["notify-beep-on-error"], false);
+  assert.equal(config["notify-beep-on-esc"], true);
   assert.equal(config["notify-sound"], "high");
   assert.equal(config["notify-sound-toggle-shortcut"], "alt+shift+s");
   assert.equal(config.PI_NOTIFY_SOUND_LOW_CMD, "echo low %%INSTALLATION_PATH%%");
@@ -1877,19 +1829,285 @@ test("configuration menu can persist pi-notify settings", async () => {
   assert.equal(
     ctx.__state.statuses.get("pi-usereq"),
     buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
       docsDir: DEFAULT_DOCS_DIR,
       testsDir: "tests",
       srcDir: ["src"],
-      toolCount: pi.getActiveTools().length,
       contextFilledCells: 0,
-      elapsed: "idle",
-      last: "N/A",
-      beep: "end,err",
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
+      beep: "esc",
       sound: "high",
     }),
   );
 });
 
+test("configuration menu can persist pushover settings", async () => {
+  const cwd = createTempDir("pi-usereq-menu-pushover-");
+  fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
+  const pi = createFakePi();
+  piUsereqExtension(pi);
+  const command = pi.commands.get("pi-usereq");
+  assert.ok(command);
+  const ctx = createFakeCtx(cwd, {
+    selects: [
+      "notifications",
+      "Pushover notifications",
+      "Notify on success",
+      "User Key/Delivery Group Key",
+      "Token/API Token Key",
+      "Priority",
+      "1=High Priority",
+      "Back",
+      "Back",
+      "Save and close",
+    ],
+    inputs: [
+      "gzfjjvp1xxmhibqwzh9m7i1zwvf83j",
+      "ah6bf5u2sj63mcvou6qamiabeoubbe",
+    ],
+  });
+
+  await command!.handler("", ctx);
+
+  const config = JSON.parse(fs.readFileSync(getProjectConfigPath(cwd), "utf8"));
+  assert.equal(config["notify-pushover-global-disable"], false);
+  assert.equal(config["notify-pushover-on-success"], true);
+  assert.equal(config["notify-pushover-user-key"], "gzfjjvp1xxmhibqwzh9m7i1zwvf83j");
+  assert.equal(config["notify-pushover-api-token"], "ah6bf5u2sj63mcvou6qamiabeoubbe");
+  assert.equal(config["notify-pushover-priority"], 1);
+  assert.equal(
+    ctx.__state.statuses.get("pi-usereq"),
+    buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
+      docsDir: DEFAULT_DOCS_DIR,
+      testsDir: "tests",
+      srcDir: ["src"],
+      contextFilledCells: 0,
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
+      pushover: "on",
+    }),
+  );
+});
+
+test("successful req prompt completion sends pushover requests and suppresses invalid scenarios", async () => {
+  const cwd = createTempDir("pi-usereq-pushover-run-");
+  fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
+  const configPath = getProjectConfigPath(cwd);
+  const writeConfig = (overrides: Record<string, unknown>) => {
+    const config = {
+      ...getDefaultConfig(cwd),
+      ...overrides,
+    };
+    fs.writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`, "utf8");
+  };
+  const pi = createFakePi();
+  piUsereqExtension(pi);
+  const ctx = createFakeCtx(cwd);
+  const originalDateNow = Date.now;
+  const recordedRequests: Array<{ url: string; options: Record<string, unknown>; body: string }> = [];
+  let nowMs = 0;
+  setPiNotifyHttpsRequestForTests(((url: URL, options: Record<string, unknown>, callback?: (response: any) => void) => {
+    const record = { url: url.toString(), options, body: "" };
+    recordedRequests.push(record);
+    const fakeResponse = {
+      on(_eventName: string, _handler: (...args: any[]) => void) {
+        return fakeResponse;
+      },
+      resume() {
+        return undefined;
+      },
+    };
+    callback?.(fakeResponse);
+    const fakeRequest = {
+      on(_eventName: string, _handler: (...args: any[]) => void) {
+        return fakeRequest;
+      },
+      end(body?: string) {
+        record.body = body ?? "";
+      },
+    };
+    return fakeRequest as unknown as ReturnType<typeof import("node:https").request>;
+  }) as typeof import("node:https").request);
+  Date.now = () => nowMs;
+
+  try {
+    writeConfig({
+      "notify-pushover-on-success": true,
+      "notify-pushover-user-key": "gzfjjvp1xxmhibqwzh9m7i1zwvf83j",
+      "notify-pushover-api-token": "ah6bf5u2sj63mcvou6qamiabeoubbe",
+      "notify-pushover-priority": 1,
+    });
+    await pi.emit("session_start", { reason: "startup" }, ctx);
+    recordedRequests.length = 0;
+    nowMs = 0;
+    await pi.commands.get("req-analyze")!.handler("Inspect src/index.ts", ctx);
+    await pi.emit("agent_start", {}, ctx);
+    nowMs = 5_000;
+    await pi.emit("agent_end", {
+      messages: [
+        {
+          role: "assistant",
+          stopReason: "stop",
+          content: [],
+        },
+      ],
+    }, ctx);
+
+    assert.equal(recordedRequests.length, 1);
+    assert.equal(recordedRequests[0]?.url, "https://api.pushover.net/1/messages.json");
+    const successParams = new URLSearchParams(recordedRequests[0]?.body ?? "");
+    assert.equal(successParams.get("token"), "ah6bf5u2sj63mcvou6qamiabeoubbe");
+    assert.equal(successParams.get("user"), "gzfjjvp1xxmhibqwzh9m7i1zwvf83j");
+    assert.equal(successParams.get("priority"), "1");
+    assert.equal(successParams.get("message"), "Inspect src/index.ts");
+    assert.equal(successParams.get("title"), `analyze @ ${buildExpectedFakeBasePath(cwd)} [0:05]`);
+
+    recordedRequests.length = 0;
+    nowMs = 10_000;
+    await pi.commands.get("req-change")!.handler("Adjust docs", ctx);
+    await pi.emit("agent_start", {}, ctx);
+    nowMs = 14_000;
+    await pi.emit("agent_end", {
+      messages: [
+        {
+          role: "assistant",
+          stopReason: "aborted",
+          content: [],
+        },
+      ],
+    }, ctx);
+    assert.equal(recordedRequests.length, 0);
+
+    recordedRequests.length = 0;
+    nowMs = 20_000;
+    await pi.commands.get("req-create")!.handler("Create docs", ctx);
+    await pi.emit("agent_start", {}, ctx);
+    nowMs = 23_000;
+    await pi.emit("agent_end", {
+      messages: [
+        {
+          role: "assistant",
+          stopReason: "error",
+          content: [],
+        },
+      ],
+    }, ctx);
+    assert.equal(recordedRequests.length, 0);
+
+    writeConfig({
+      "notify-pushover-on-success": true,
+      "notify-pushover-user-key": "gzfjjvp1xxmhibqwzh9m7i1zwvf83j",
+      "notify-pushover-api-token": "",
+      "notify-pushover-priority": 1,
+    });
+    await pi.emit("session_start", { reason: "startup" }, ctx);
+    recordedRequests.length = 0;
+    nowMs = 30_000;
+    await pi.commands.get("req-write")!.handler("Write docs", ctx);
+    await pi.emit("agent_start", {}, ctx);
+    nowMs = 32_000;
+    await pi.emit("agent_end", {
+      messages: [
+        {
+          role: "assistant",
+          stopReason: "stop",
+          content: [],
+        },
+      ],
+    }, ctx);
+    assert.equal(recordedRequests.length, 0);
+  } finally {
+    setPiNotifyHttpsRequestForTests(undefined);
+    Date.now = originalDateNow;
+  }
+});
+
+test("pushover global disable suppresses delivery without clearing enabled state", async () => {
+  const cwd = createTempDir("pi-usereq-pushover-disabled-");
+  fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
+  const pi = createFakePi();
+  piUsereqExtension(pi);
+  const command = pi.commands.get("pi-usereq");
+  assert.ok(command);
+  const ctx = createFakeCtx(cwd, {
+    selects: [
+      "notifications",
+      "Pushover notifications",
+      "Notify on success",
+      "Global disable",
+      "User Key/Delivery Group Key",
+      "Token/API Token Key",
+      "Back",
+      "Back",
+      "Save and close",
+    ],
+    inputs: [
+      "gzfjjvp1xxmhibqwzh9m7i1zwvf83j",
+      "ah6bf5u2sj63mcvou6qamiabeoubbe",
+    ],
+  });
+
+  await command!.handler("", ctx);
+
+  const config = JSON.parse(fs.readFileSync(getProjectConfigPath(cwd), "utf8"));
+  assert.equal(config["notify-pushover-on-success"], true);
+  assert.equal(config["notify-pushover-global-disable"], true);
+  assert.equal(
+    ctx.__state.statuses.get("pi-usereq"),
+    buildExpectedFakeStatusText({
+      basePath: buildExpectedFakeBasePath(cwd),
+      docsDir: DEFAULT_DOCS_DIR,
+      testsDir: "tests",
+      srcDir: ["src"],
+      contextFilledCells: 0,
+      et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
+      pushover: "on",
+    }),
+  );
+
+  const recordedRequests: string[] = [];
+  setPiNotifyHttpsRequestForTests(((url: URL, options: Record<string, unknown>, callback?: (response: any) => void) => {
+    void options;
+    recordedRequests.push(url.toString());
+    const fakeResponse = {
+      on(_eventName: string, _handler: (...args: any[]) => void) {
+        return fakeResponse;
+      },
+      resume() {
+        return undefined;
+      },
+    };
+    callback?.(fakeResponse);
+    const fakeRequest = {
+      on(_eventName: string, _handler: (...args: any[]) => void) {
+        return fakeRequest;
+      },
+      end(_body?: string) {
+        return undefined;
+      },
+    };
+    return fakeRequest as unknown as ReturnType<typeof import("node:https").request>;
+  }) as typeof import("node:https").request);
+
+  try {
+    await pi.emit("session_start", { reason: "startup" }, ctx);
+    await pi.commands.get("req-analyze")!.handler("Inspect src/index.ts", ctx);
+    await pi.emit("agent_start", {}, ctx);
+    await pi.emit("agent_end", {
+      messages: [
+        {
+          role: "assistant",
+          stopReason: "stop",
+          content: [],
+        },
+      ],
+    }, ctx);
+    assert.deepEqual(recordedRequests, []);
+  } finally {
+    setPiNotifyHttpsRequestForTests(undefined);
+  }
+});
+
 test("sound toggle shortcut cycles persisted pi-notify sound levels", async () => {
   const cwd = createTempDir("pi-usereq-shortcut-notify-");
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
@@ -1914,13 +2132,12 @@ test("sound toggle shortcut cycles persisted pi-notify sound levels", async () =
     assert.equal(
       ctx.__state.statuses.get("pi-usereq"),
       buildExpectedFakeStatusText({
+        basePath: buildExpectedFakeBasePath(cwd),
         docsDir: DEFAULT_DOCS_DIR,
         testsDir: "tests",
         srcDir: ["src"],
-        toolCount: PI_USEREQ_DEFAULT_ENABLED_TOOL_NAMES.length,
         contextFilledCells: 0,
-        elapsed: "idle",
-        last: "N/A",
+        et: " ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--",
         sound: expectedSound,
       }),
     );
@@ -1939,11 +2156,11 @@ test("git-path tool derives the repository root at runtime", async () => {
     piUsereqExtension(pi);
     const result = await executeRegisteredTool(pi, "git-path", projectBase) as {
       content?: Array<{ text?: string }>;
-      details?: { result: { path_value: string }; runtime_paths: { git_path: string } };
+      details?: { result: { path_value: string } };
     };
 
     assert.deepEqual(JSON.parse(result.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(result.details)));
-    assert.equal(result.details?.result.path_value, result.details?.runtime_paths.git_path);
+    assert.equal(result.details?.result.path_value, projectBase);
   } finally {
     fs.rmSync(projectBase, { recursive: true, force: true });
   }
@@ -1979,6 +2196,26 @@ test("configuration menu can add static-check entries from raw specs", async ()
   assert.deepEqual(config["static-check"].Python, [{ module: "Command", cmd: "true" }]);
 });
 
+test("configuration menu hides removed static-check modules from user-facing actions", async () => {
+  const cwd = createTempDir("pi-usereq-menu-sc-hidden-");
+  fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
+  const pi = createFakePi();
+  piUsereqExtension(pi);
+  const command = pi.commands.get("pi-usereq");
+  assert.ok(command);
+  const ctx = createFakeCtx(cwd, {
+    selects: ["static-check"],
+  });
+
+  await command!.handler("", ctx);
+
+  const renderedStaticCheckMenu = (ctx.__state.customRenderLines[1] ?? []).join("\n");
+  assert.match(renderedStaticCheckMenu, /Add entry for supported language/);
+  assert.doesNotMatch(renderedStaticCheckMenu, /Pylance/);
+  assert.doesNotMatch(renderedStaticCheckMenu, /Ruff/);
+  assert.doesNotMatch(renderedStaticCheckMenu, /Dummy/);
+});
+
 test("configuration menu can add guided static-check entries for explicit supported languages", async () => {
   const cwd = createTempDir("pi-usereq-menu-sc-guided-");
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
@@ -1994,13 +2231,13 @@ test("configuration menu can add guided static-check entries for explicit suppor
         "static-check",
         "Add entry for supported language",
         "Python",
-        "Ruff",
         "Back",
         "Save and close",
       ],
+      inputs: ["true", ""],
     }),
   );
 
   const config = JSON.parse(fs.readFileSync(getProjectConfigPath(cwd), "utf8"));
-  assert.deepEqual(config["static-check"].Python, [{ module: "Ruff" }]);
+  assert.deepEqual(config["static-check"].Python, [{ module: "Command", cmd: "true" }]);
 });