pi-usereq 0.4.0 → 0.6.0

package/tests/attended-results-scenarios.ts

@@ -357,16 +357,6 @@ function buildStandaloneScenarios(): AttendedScenario[] {
         path.posix.join("standalone", "test-static-check-dummy", `${fixtureName}.json`),
         ["--test-static-check", "dummy", relativeFixturePath],
       ),
-      createStandaloneScenario(
-        `standalone/test-static-check-pylance/${fixtureName}`,
-        path.posix.join("standalone", "test-static-check-pylance", `${fixtureName}.json`),
-        ["--test-static-check", "pylance", relativeFixturePath],
-      ),
-      createStandaloneScenario(
-        `standalone/test-static-check-ruff/${fixtureName}`,
-        path.posix.join("standalone", "test-static-check-ruff", `${fixtureName}.json`),
-        ["--test-static-check", "ruff", relativeFixturePath],
-      ),
       createStandaloneScenario(
         `standalone/test-static-check-command/${fixtureName}`,
         path.posix.join("standalone", "test-static-check-command", `${fixtureName}.json`),
@@ -470,14 +460,14 @@ function buildProjectScenarios(): AttendedScenario[] {
             "--provider",
             "claude:prompts",
             "--enable-static-check",
-            "Python=Pylance",
+            "Python=Command,git,--version",
           ],
           cwd: projectBase,
           normalize: createProjectNormalizer(projectBase),
           cleanup: () => removePath(projectBase),
           postAssert: () => {
             const payload = JSON.parse(fs.readFileSync(getProjectConfigPath(projectBase), "utf8")) as UseReqConfig;
-            assert.deepEqual(payload["static-check"].Python, [{ module: "Pylance" }]);
+            assert.deepEqual(payload["static-check"].Python, [{ module: "Command", cmd: "git", params: ["--version"] }]);
           },
         };
       },
@@ -488,7 +478,7 @@ function buildProjectScenarios(): AttendedScenario[] {
           const script = [
             "from pathlib import Path",
             "from usereq.cli import save_config",
-            `save_config(Path(${JSON.stringify(projectBase)}), 'guidelines', 'docs', 'tests', ['src'], static_check_config={'Python': [{'module': 'Pylance'}]})`,
+            `save_config(Path(${JSON.stringify(projectBase)}), 'guidelines', 'docs', 'tests', ['src'], static_check_config={'Python': [{'module': 'Command', 'cmd': 'git', 'params': ['--version']} ]})`,
           ].join("\n");
           const result = runPythonInline(script, projectBase);
           return createProjectNormalizer(projectBase)(toArchivedCliResult(result));

package/tests/cli-command-option-parity.test.ts

@@ -583,6 +583,14 @@ const TARGET_CASES: TargetParityCase[] = [
       assert.equal(result.status, 1);
       assert.match(result.stderr, /not an executable program/);
       assert.ok(!fs.existsSync(getProjectConfigPath(projectBase)));
+
+      const removedModule = runNodeCli(
+        ["--base", projectBase, "--enable-static-check", "Python=Ruff"],
+        projectBase,
+      );
+      assert.equal(removedModule.status, 1);
+      assert.match(removedModule.stderr, /unknown module/i);
+      assert.ok(!fs.existsSync(getProjectConfigPath(projectBase)));
     },
   },
   {
@@ -594,16 +602,19 @@ const TARGET_CASES: TargetParityCase[] = [
           "--base",
           projectBase,
           "--enable-static-check",
-          "Python=Pylance",
+          "Python=Command,git,--version",
           "--enable-static-check",
           "C=Command,git,--version",
           "--enable-static-check",
-          "Python=Ruff",
+          "Python=Command,git,--help",
         ],
         projectBase,
       );
       assert.equal(persisted.status, 0, persisted.stderr);
-      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [{ module: "Pylance" }, { module: "Ruff" }]);
+      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [
+        { module: "Command", cmd: "git", params: ["--version"] },
+        { module: "Command", cmd: "git", params: ["--help"] },
+      ]);
       assert.deepEqual(getStaticCheckEntries(projectBase, "C"), [{ module: "Command", cmd: "git", params: ["--version"] }]);
     },
   },
@@ -616,9 +627,9 @@ const TARGET_CASES: TargetParityCase[] = [
           "--base",
           projectBase,
           "--enable-static-check",
-          "Python=Ruff",
+          "Python=Command,git,--version",
           "--enable-static-check",
-          "Python=Ruff",
+          "Python=Command,git,--version",
           "--enable-static-check",
           "C=Command,git,--version",
           "--enable-static-check",
@@ -627,7 +638,7 @@ const TARGET_CASES: TargetParityCase[] = [
         projectBase,
       );
       assert.equal(result.status, 0, result.stderr);
-      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [{ module: "Ruff" }]);
+      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [{ module: "Command", cmd: "git", params: ["--version"] }]);
       assert.deepEqual(getStaticCheckEntries(projectBase, "C"), [
         { module: "Command", cmd: "git", params: ["--version"] },
         { module: "Command", cmd: "git", params: ["--help"] },
@@ -641,7 +652,7 @@ const TARGET_CASES: TargetParityCase[] = [
       const preloaded = {
         ...config,
         "static-check": {
-          Python: [{ module: "Pylance", meta: "legacy-note" } as unknown as Record<string, unknown>],
+          Python: [{ module: "Dummy", meta: "debug-note" } as unknown as Record<string, unknown>],
           JavaScript: [{ module: "Command", cmd: "git", params: ["--version"] }],
         } as unknown as UseReqConfig["static-check"],
       } as UseReqConfig;
@@ -651,9 +662,7 @@ const TARGET_CASES: TargetParityCase[] = [
           "--base",
           projectBase,
           "--enable-static-check",
-          "Python=Pylance",
-          "--enable-static-check",
-          "Python=Ruff",
+          "Python=Command,git,--version",
           "--enable-static-check",
           "JavaScript=Command,git,--help",
         ],
@@ -661,8 +670,8 @@ const TARGET_CASES: TargetParityCase[] = [
       );
       assert.equal(result.status, 0, result.stderr);
       assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [
-        { module: "Pylance", meta: "legacy-note" },
-        { module: "Ruff" },
+        { module: "Dummy" },
+        { module: "Command", cmd: "git", params: ["--version"] },
       ]);
       assert.deepEqual(getStaticCheckEntries(projectBase, "JavaScript"), [
         { module: "Command", cmd: "git", params: ["--version"] },

package/tests/debug-extension-harness.test.ts

@@ -165,9 +165,9 @@ test("inspectExtension surfaces agent-oriented find tool descriptions and schema
 
     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("Supported tags [Typescript]:")));
-    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("Regex rule:")));
     assert.match(String((find.parameters as { description?: string } | undefined)?.description ?? ""), /Regex matches construct names only/);
@@ -185,7 +185,7 @@ test("inspectExtension surfaces agent-oriented compression tool descriptions and
 
     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.match(String((filesCompress.parameters as { description?: string } | undefined)?.description ?? ""), /structured compressed lines/);
     assert.match(compress.description ?? "", /configured project source directories/);
@@ -212,7 +212,7 @@ test("inspectExtension surfaces structured utility tool descriptions and schema
     assert.ok(docsCheck, "missing docs-check tool");
     assert.ok(gitWtCreate, "missing git-wt-create tool");
 
-    assert.match(gitPath.description ?? "", /JSON-first payload/);
+    assert.match(gitPath.description ?? "", /token-optimized JSON payload/);
     assert.ok(gitPath.promptGuidelines?.some((line) => line.includes("Output contract:")));
     assert.match(String((gitPath.parameters as { description?: string } | undefined)?.description ?? ""), /path_value/);
 
@@ -220,9 +220,9 @@ test("inspectExtension surfaces structured utility tool descriptions and schema
     assert.ok(filesStaticCheck.promptGuidelines?.some((line) => line.includes("Failure contract:")));
     assert.match(String((filesStaticCheck.parameters as { description?: string } | undefined)?.description ?? ""), /configured checker modules/);
 
-    assert.match(gitCheck.description ?? "", /JSON-first payload/);
+    assert.match(gitCheck.description ?? "", /token-optimized JSON payload/);
     assert.ok(gitCheck.promptGuidelines?.some((line) => line.includes("Behavior contract:")));
-    assert.match(String((gitCheck.parameters as { description?: string } | undefined)?.description ?? ""), /clean-versus-error repository status/);
+    assert.match(String((gitCheck.parameters as { description?: string } | undefined)?.description ?? ""), /aggregate repository status fields/);
 
     assert.match(docsCheck.description ?? "", /remediation prompt commands/);
     assert.ok(docsCheck.promptGuidelines?.some((line) => line.includes("Specialization trigger:")));
@@ -246,18 +246,18 @@ test("replaySessionStart captures active tools, statuses, and cwd semantics", as
     assert.deepEqual([...report.activeTools].sort(), ["git-path", "static-check"]);
     assert.equal(report.effectiveCtxCwd, projectBase);
     assert.equal(report.effectiveProcessCwd, projectBase);
-    assert.match(
-      status,
-      /<accent>git:<\/accent><warning>.*<\/warning><dim> • <\/dim><accent>base:<\/accent><warning>\.<\/warning><dim> • <\/dim><accent>docs:<\/accent><warning>pi-usereq\/docs<\/warning><dim> • <\/dim><accent>tests:<\/accent><warning>tests<\/warning><dim> • <\/dim><accent>src:<\/accent><warning>src<\/warning><dim> • <\/dim><accent>tools:<\/accent><warning>2<\/warning>/,
-    );
-    assert.match(
-      status,
-      /<accent>context:<\/accent><bg-from-fg-accent><warning>CLEAR<\/warning><\/bg-from-fg-accent>/,
-    );
-    assert.match(status, /<accent>elapsed:<\/accent><warning>idle<\/warning>/);
-    assert.match(status, /<accent>last:<\/accent><warning>N\/A<\/warning>/);
-    assert.match(status, /<accent>beep:<\/accent><warning>none<\/warning>/);
+    const normalizedBasePath = projectBase.split(path.sep).join("/");
+    assert.match(status, new RegExp(`<accent>base:</accent><warning>${normalizedBasePath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}</warning>`));
+    assert.match(status, /<accent>context:<\/accent><bg-from-fg-accent><warning>◀ CLEAR ▶ <\/warning><\/bg-from-fg-accent>/);
+    assert.match(status, /<accent>elapsed:<\/accent><warning> ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--<\/warning>/);
+    assert.match(status, /<accent>beep:<\/accent><warning>end,esc,err<\/warning>/);
     assert.match(status, /<accent>sound:<\/accent><warning>none<\/warning>/);
+    assert.match(status, /<accent>pushover:<\/accent><warning>off<\/warning>/);
+    assert.doesNotMatch(status, /<accent>docs:<\/accent>/);
+    assert.doesNotMatch(status, /<accent>src:<\/accent>/);
+    assert.doesNotMatch(status, /<accent>tests:<\/accent>/);
+    assert.doesNotMatch(status, /<accent>git:<\/accent>/);
+    assert.doesNotMatch(status, /<accent>tools:<\/accent>/);
     assert.equal(report.ui.notifications.length, 0);
   } finally {
     fs.rmSync(projectBase, { recursive: true, force: true });
@@ -294,7 +294,8 @@ test("replayCommand captures interactive UI side effects", async () => {
 
     assert.deepEqual(report.activeTools, []);
     assert.ok(report.ui.notifications.some((entry) => entry.message === "Disabled all configurable active tools"));
-    assert.match(report.ui.statuses["pi-usereq"] ?? "", /<accent>tools:<\/accent><warning>0<\/warning>/);
+    assert.doesNotMatch(report.ui.statuses["pi-usereq"] ?? "", /<accent>tools:<\/accent>/);
+    assert.match(report.ui.statuses["pi-usereq"] ?? "", /<accent>elapsed:<\/accent><warning> ⏱︎ --:-- ⚑ --:-- ⌛︎ --:--<\/warning>/);
   } finally {
     fs.rmSync(projectBase, { recursive: true, force: true });
   }
@@ -307,9 +308,7 @@ test("replayTool captures tool results and cwd semantics", async () => {
     const toolResult = report.toolResult as {
       content?: Array<{ type: string; text?: string }>;
       details?: {
-        request: { project_base_path: string };
         result: { path_value: string; path_present: boolean };
-        runtime_paths: { git_path: string };
         execution: { code: number };
       };
     };
@@ -317,8 +316,7 @@ test("replayTool captures tool results and cwd semantics", async () => {
     assert.equal(report.effectiveCtxCwd, projectBase);
     assert.equal(report.effectiveProcessCwd, projectBase);
     assert.deepEqual(JSON.parse(toolResult.content?.[0]?.text ?? "{}"), JSON.parse(JSON.stringify(toolResult.details)));
-    assert.equal(toolResult.details?.request.project_base_path, projectBase);
-    assert.equal(toolResult.details?.result.path_value, toolResult.details?.runtime_paths.git_path);
+    assert.equal(toolResult.details?.result.path_value, projectBase);
     assert.equal(toolResult.details?.result.path_present, true);
     assert.equal(toolResult.details?.execution.code, 0);
   } finally {