pi-usereq 0.11.0 → 0.13.0

package/tests/attended-results-scenarios.ts

@@ -14,11 +14,13 @@ import { detectLanguage } from "../src/core/generate-markdown.js";
 import { LANGUAGE_TAGS } from "../src/core/find-constructs.js";
 import {
   createStaticCheckLanguageConfig,
-  getProjectConfigPath,
+  getDefaultConfig,
   type UseReqConfig,
 } from "../src/core/config.js";
 import {
   initFixtureRepo,
+  readGlobalConfigJson,
+  readProjectConfigJson,
   runNodeCli,
   runPythonCli,
   runPythonInline,
@@ -312,9 +314,9 @@ function buildStandaloneScenarios(): AttendedScenario[] {
         ["--files-tokens", relativeFixturePath],
       ),
       createStandaloneScenario(
-        `standalone/files-references/${fixtureName}`,
-        path.posix.join("standalone", "files-references", `${fixtureName}.json`),
-        ["--files-references", relativeFixturePath],
+        `standalone/files-summarize/${fixtureName}`,
+        path.posix.join("standalone", "files-summarize", `${fixtureName}.json`),
+        ["--files-summarize", relativeFixturePath],
       ),
       createStandaloneScenario(
         `standalone/files-compress/${fixtureName}`,
@@ -369,10 +371,10 @@ function getFailingStaticCheckConfig(): UseReqConfig["static-check"] {
  */
 function buildProjectScenarios(): AttendedScenario[] {
   return [
-    createProjectScenario("project/references", path.posix.join("project", "references.json"), () => {
+    createProjectScenario("project/summarize", path.posix.join("project", "summarize.json"), () => {
       const { projectBase } = initFixtureRepo();
       return {
-        args: ["--references"],
+        args: ["--summarize"],
         cwd: projectBase,
         normalize: createProjectNormalizer(projectBase),
         cleanup: () => removePath(projectBase),
@@ -450,10 +452,14 @@ function buildProjectScenarios(): AttendedScenario[] {
           normalize: createProjectNormalizer(projectBase),
           cleanup: () => removePath(projectBase),
           postAssert: () => {
-            const payload = JSON.parse(fs.readFileSync(getProjectConfigPath(projectBase), "utf8")) as UseReqConfig;
-            assert.deepEqual(payload["static-check"].Python, createStaticCheckLanguageConfig([
+            const defaultConfig = getDefaultConfig(projectBase);
+            const localPayload = readProjectConfigJson(projectBase) as unknown as Record<string, unknown>;
+            const globalPayload = readGlobalConfigJson() as unknown as Record<string, unknown>;
+            assert.equal((localPayload["static-check"] as Record<string, any>).Python.enabled, "enable");
+            assert.deepEqual((globalPayload["static-check"] as Record<string, any>).Python.checkers, [
+              ...defaultConfig["static-check"].Python.checkers,
               { module: "Command", cmd: "git", params: ["--version"] },
-            ], "enable"));
+            ]);
           },
         };
       },

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

@@ -12,6 +12,7 @@ import { spawnSync } from "node:child_process";
 import {
   createStaticCheckLanguageConfig,
   DEFAULT_DOCS_DIR,
+  getDefaultConfig,
   getProjectConfigPath,
   type UseReqConfig,
 } from "../src/core/config.js";
@@ -21,6 +22,7 @@ import {
   createTempDir,
   getFixtureFiles,
   initFixtureRepo,
+  readGlobalConfigJson,
   readProjectConfigJson,
   runNodeCli,
   runPythonCli,
@@ -115,11 +117,11 @@ function getFixtureTagFilter(filePath: string): string {
 }
 
 /**
- * @brief Returns the persisted per-language static-check config for one language.
- * @details Reads the raw project config JSON so tests can assert entry order, duplicate suppression, enable-flag persistence, and metadata preservation exactly as written. Runtime is O(n) in config file size. Side effects are limited to filesystem reads.
+ * @brief Returns the persisted local static-check config for one language.
+ * @details Reads the raw local project config JSON so tests can assert enable-flag persistence and local-scope shape exactly as written. Runtime is O(n) in config file size. Side effects are limited to filesystem reads.
  * @param[in] projectBase {string} Fixture project root.
  * @param[in] language {string} Canonical language key.
- * @return {Record<string, unknown> | undefined} Persisted language config object when present.
+ * @return {Record<string, unknown> | undefined} Persisted local language config object when present.
  */
 function getStaticCheckLanguageConfig(projectBase: string, language: string): Record<string, unknown> | undefined {
   const payload = readProjectConfigJson(projectBase);
@@ -128,14 +130,17 @@ function getStaticCheckLanguageConfig(projectBase: string, language: string): Re
 }
 
 /**
- * @brief Returns the persisted static-check entries for one language.
- * @details Reads the raw project config JSON and extracts the `checkers` array so tests can assert entry order and duplicate suppression exactly as written. Runtime is O(n) in config file size. Side effects are limited to filesystem reads.
- * @param[in] projectBase {string} Fixture project root.
+ * @brief Returns the persisted global static-check entries for one language.
+ * @details Reads the raw global config JSON and extracts the `checkers` array so tests can assert entry order and duplicate suppression exactly as written. Runtime is O(n) in config file size. Side effects are limited to filesystem reads.
+ * @param[in] projectBase {string} Fixture project root retained for stable call-site shape.
  * @param[in] language {string} Canonical language key.
- * @return {Array<Record<string, unknown>>} Persisted checker-entry array or an empty array.
+ * @return {Array<Record<string, unknown>>} Persisted global checker-entry array or an empty array.
  */
 function getStaticCheckEntries(projectBase: string, language: string): Array<Record<string, unknown>> {
-  const languageConfig = getStaticCheckLanguageConfig(projectBase, language);
+  void projectBase;
+  const payload = readGlobalConfigJson();
+  const staticCheck = (payload["static-check"] ?? {}) as Record<string, Record<string, unknown>>;
+  const languageConfig = staticCheck[language];
   return Array.isArray(languageConfig?.checkers)
     ? languageConfig.checkers as Array<Record<string, unknown>>
     : [];
@@ -171,18 +176,18 @@ const IMPORTED_CASE_MAPPINGS: ImportedCaseMapping[] = [
   { sourceId: "test_files_commands.py::TestFilesTokensCommand::test_files_tokens_no_base_required", targetId: "oracle::standalone::files-tokens-explicit-files" },
   { sourceId: "test_files_commands.py::TestFilesTokensCommand::test_files_tokens_skip_missing", targetId: "oracle::standalone::files-tokens-explicit-files" },
   { sourceId: "test_files_commands.py::TestFilesTokensCommand::test_files_tokens_all_missing_errors", targetId: "oracle::standalone::files-tokens-explicit-files" },
-  { sourceId: "test_files_commands.py::TestFilesReferencesCommand::test_files_references_basic", targetId: "direct::files-references-explicit-files" },
-  { sourceId: "test_files_commands.py::TestFilesReferencesCommand::test_files_references_no_base_required", targetId: "direct::files-references-explicit-files" },
-  { sourceId: "test_files_commands.py::TestFilesReferencesCommand::test_files_references_verbose_outputs_progress", targetId: "direct::files-references-verbose" },
+  { sourceId: "test_files_commands.py::TestFilesSummarizeCommand::test_files_summarize_basic", targetId: "direct::files-summarize-explicit-files" },
+  { sourceId: "test_files_commands.py::TestFilesSummarizeCommand::test_files_summarize_no_base_required", targetId: "direct::files-summarize-explicit-files" },
+  { sourceId: "test_files_commands.py::TestFilesSummarizeCommand::test_files_summarize_verbose_outputs_progress", targetId: "direct::files-summarize-verbose" },
   { sourceId: "test_files_commands.py::TestFilesCompressCommand::test_files_compress_basic", targetId: "oracle::standalone::files-compress-line-number-modes" },
   { sourceId: "test_files_commands.py::TestFilesCompressCommand::test_files_compress_no_base_required", targetId: "oracle::standalone::files-compress-line-number-modes" },
   { sourceId: "test_files_commands.py::TestFilesCompressCommand::test_files_compress_no_line_numbers_by_default", targetId: "oracle::standalone::files-compress-line-number-modes" },
   { sourceId: "test_files_commands.py::TestFilesCompressCommand::test_files_compress_enable_line_numbers", targetId: "oracle::standalone::files-compress-line-number-modes" },
   { sourceId: "test_files_commands.py::TestFilesCompressCommand::test_files_compress_verbose_outputs_progress", targetId: "oracle::standalone::files-compress-verbose" },
-  { sourceId: "test_files_commands.py::TestReferencesCommand::test_rejects_base", targetId: "direct::references-modes" },
-  { sourceId: "test_files_commands.py::TestReferencesCommand::test_references_with_here", targetId: "direct::references-modes" },
-  { sourceId: "test_files_commands.py::TestReferencesCommand::test_references_prepends_files_structure", targetId: "direct::references-modes" },
-  { sourceId: "test_files_commands.py::TestReferencesCommand::test_references_from_config", targetId: "direct::references-modes" },
+  { sourceId: "test_files_commands.py::TestSummarizeCommand::test_rejects_base", targetId: "direct::summarize-modes" },
+  { sourceId: "test_files_commands.py::TestSummarizeCommand::test_summarize_with_here", targetId: "direct::summarize-modes" },
+  { sourceId: "test_files_commands.py::TestSummarizeCommand::test_summarize_prepends_files_structure", targetId: "direct::summarize-modes" },
+  { sourceId: "test_files_commands.py::TestSummarizeCommand::test_summarize_from_config", targetId: "direct::summarize-modes" },
   { sourceId: "test_files_commands.py::TestCompressCommand::test_rejects_base", targetId: "oracle::project::compress-modes" },
   { sourceId: "test_files_commands.py::TestCompressCommand::test_compress_with_here", targetId: "oracle::project::compress-modes" },
   { sourceId: "test_files_commands.py::TestCompressCommand::test_compress_no_line_numbers_by_default", targetId: "oracle::project::compress-modes" },
@@ -254,7 +259,7 @@ const TARGET_CASES: TargetParityCase[] = [
     },
   },
   {
-    id: "direct::files-references-explicit-files",
+    id: "direct::files-summarize-explicit-files",
     run(t) {
       const filePath = createScratchFile(
         t,
@@ -262,8 +267,8 @@ const TARGET_CASES: TargetParityCase[] = [
         "/**\n * @file\n * @brief Sample file.\n */\n\n/**\n * @brief Build sample value.\n * @param[in] input {string} Caller text.\n * @return {string} Upper-cased text.\n */\nexport function buildSample(input: string): string {\n  return input.toUpperCase();\n}\n",
       );
       const cwd = path.dirname(filePath);
-      const node = runNodeCli(["--files-references", filePath], cwd);
-      const python = runPythonCli(["--files-references", filePath], cwd);
+      const node = runNodeCli(["--files-summarize", filePath], cwd);
+      const python = runPythonCli(["--files-summarize", filePath], cwd);
       assert.equal(node.status, python.status);
       assert.equal(node.stdout, python.stdout);
       assert.equal(node.stderr, python.stderr);
@@ -273,12 +278,12 @@ const TARGET_CASES: TargetParityCase[] = [
     },
   },
   {
-    id: "direct::files-references-verbose",
+    id: "direct::files-summarize-verbose",
     run(t) {
       const filePath = createScratchFile(t, "sample.py", "value = 1\n");
       const cwd = path.dirname(filePath);
-      const node = runNodeCli(["--verbose", "--files-references", filePath], cwd);
-      const python = runPythonCli(["--verbose", "--files-references", filePath], cwd);
+      const node = runNodeCli(["--verbose", "--files-summarize", filePath], cwd);
+      const python = runPythonCli(["--verbose", "--files-summarize", filePath], cwd);
       assert.equal(node.status, python.status);
       assert.equal(node.stdout, python.stdout);
       assert.equal(node.stderr, python.stderr);
@@ -302,15 +307,15 @@ const TARGET_CASES: TargetParityCase[] = [
     },
   },
   {
-    id: "direct::references-modes",
+    id: "direct::summarize-modes",
     run(t) {
       const { projectBase } = createFixtureRepo(t, { fixtures: [] });
       fs.writeFileSync(path.join(projectBase, "src", "alpha.ts"), "export const ALPHA = 1;\n", "utf8");
       fs.mkdirSync(path.join(projectBase, "src", "nested"), { recursive: true });
       fs.writeFileSync(path.join(projectBase, "src", "nested", "beta.ts"), "export function beta(): number {\n  return 2;\n}\n", "utf8");
 
-      const directNode = runNodeCli(["--references"], projectBase);
-      const directPython = runPythonCli(["--references"], projectBase);
+      const directNode = runNodeCli(["--summarize"], projectBase);
+      const directPython = runPythonCli(["--summarize"], projectBase);
       assert.equal(directNode.status, directPython.status);
       assert.equal(directNode.stdout, directPython.stdout);
       assert.equal(directNode.stderr, directPython.stderr);
@@ -318,13 +323,13 @@ const TARGET_CASES: TargetParityCase[] = [
       assert.match(directNode.stdout, /src\/nested\/beta\.ts/);
       assert.match(directNode.stdout, /# beta\.ts \| TypeScript/);
 
-      const hereNode = runNodeCli(["--here", "--references"], projectBase);
-      const herePython = runPythonCli(["--here", "--references"], projectBase);
+      const hereNode = runNodeCli(["--here", "--summarize"], projectBase);
+      const herePython = runPythonCli(["--here", "--summarize"], projectBase);
       assert.equal(hereNode.status, herePython.status);
       assert.equal(hereNode.stdout, herePython.stdout);
       assert.equal(hereNode.stderr, herePython.stderr);
 
-      const baseRejected = runNodeCli(["--base", projectBase, "--references"], projectBase);
+      const baseRejected = runNodeCli(["--base", projectBase, "--summarize"], projectBase);
       assert.notEqual(baseRejected.status, 0);
       assert.match(baseRejected.stderr, /do not allow --base/);
     },
@@ -594,6 +599,7 @@ const TARGET_CASES: TargetParityCase[] = [
       fs.mkdirSync(path.join(projectBase, "src"), { recursive: true });
       fs.mkdirSync(path.join(projectBase, "tests"), { recursive: true });
       fs.mkdirSync(path.join(projectBase, ...DEFAULT_DOCS_DIR.split("/")), { recursive: true });
+      const originalGlobalConfig = structuredClone(readGlobalConfigJson());
       const result = runNodeCli(
         ["--base", projectBase, "--enable-static-check", "C=Command,nonexistent_tool_xyz_12345"],
         projectBase,
@@ -601,6 +607,7 @@ const TARGET_CASES: TargetParityCase[] = [
       assert.equal(result.status, 1);
       assert.match(result.stderr, /not an executable program/);
       assert.ok(!fs.existsSync(getProjectConfigPath(projectBase)));
+      assert.deepEqual(readGlobalConfigJson(), originalGlobalConfig);
 
       const removedModule = runNodeCli(
         ["--base", projectBase, "--enable-static-check", "Python=Ruff"],
@@ -609,6 +616,7 @@ const TARGET_CASES: TargetParityCase[] = [
       assert.equal(removedModule.status, 1);
       assert.match(removedModule.stderr, /unknown module/i);
       assert.ok(!fs.existsSync(getProjectConfigPath(projectBase)));
+      assert.deepEqual(readGlobalConfigJson(), originalGlobalConfig);
     },
   },
   {
@@ -629,13 +637,18 @@ const TARGET_CASES: TargetParityCase[] = [
         projectBase,
       );
       assert.equal(persisted.status, 0, persisted.stderr);
+      const defaultConfig = getDefaultConfig(projectBase);
       assert.equal(getStaticCheckLanguageConfig(projectBase, "Python")?.enabled, "enable");
       assert.equal(getStaticCheckLanguageConfig(projectBase, "C")?.enabled, "enable");
       assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [
+        ...defaultConfig["static-check"].Python.checkers,
         { module: "Command", cmd: "git", params: ["--version"] },
         { module: "Command", cmd: "git", params: ["--help"] },
       ]);
-      assert.deepEqual(getStaticCheckEntries(projectBase, "C"), [{ module: "Command", cmd: "git", params: ["--version"] }]);
+      assert.deepEqual(getStaticCheckEntries(projectBase, "C"), [
+        ...defaultConfig["static-check"].C.checkers,
+        { module: "Command", cmd: "git", params: ["--version"] },
+      ]);
     },
   },
   {
@@ -658,10 +671,15 @@ const TARGET_CASES: TargetParityCase[] = [
         projectBase,
       );
       assert.equal(result.status, 0, result.stderr);
+      const defaultConfig = getDefaultConfig(projectBase);
       assert.equal(getStaticCheckLanguageConfig(projectBase, "Python")?.enabled, "enable");
       assert.equal(getStaticCheckLanguageConfig(projectBase, "C")?.enabled, "enable");
-      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [{ module: "Command", cmd: "git", params: ["--version"] }]);
+      assert.deepEqual(getStaticCheckEntries(projectBase, "Python"), [
+        ...defaultConfig["static-check"].Python.checkers,
+        { module: "Command", cmd: "git", params: ["--version"] },
+      ]);
       assert.deepEqual(getStaticCheckEntries(projectBase, "C"), [
+        ...defaultConfig["static-check"].C.checkers,
         { module: "Command", cmd: "git", params: ["--version"] },
         { module: "Command", cmd: "git", params: ["--help"] },
       ]);
@@ -709,7 +727,7 @@ const TARGET_CASES: TargetParityCase[] = [
     },
   },
   {
-    id: "direct::fixtures::files-references-fixture-shape",
+    id: "direct::fixtures::files-summarize-fixture-shape",
     async run(t) {
       for (const fixture of getFixtureFiles()) {
         if (path.basename(fixture) === TAB_SENSITIVE_FIXTURE_NAME) {
@@ -717,8 +735,8 @@ const TARGET_CASES: TargetParityCase[] = [
         }
         await t.test(path.basename(fixture), () => {
           const cwd = path.dirname(fixture);
-          const node = runNodeCli(["--files-references", fixture], cwd);
-          const python = runPythonCli(["--files-references", fixture], cwd);
+          const node = runNodeCli(["--files-summarize", fixture], cwd);
+          const python = runPythonCli(["--files-summarize", fixture], cwd);
           assert.equal(node.status, python.status);
           assert.equal(node.stdout, python.stdout);
           assert.equal(node.stderr, python.stderr);
@@ -748,9 +766,9 @@ const TARGET_CASES: TargetParityCase[] = [
       assert.ok(fixture, `missing ${TAB_SENSITIVE_FIXTURE_NAME}`);
       const cwd = path.dirname(fixture);
 
-      const references = runNodeCli(["--files-references", fixture], cwd);
-      assert.equal(references.status, 0, references.stderr);
-      assert.match(references.stdout, /\tvar wg sync\.WaitGroup/);
+      const summarize = runNodeCli(["--files-summarize", fixture], cwd);
+      assert.equal(summarize.status, 0, summarize.stderr);
+      assert.match(summarize.stdout, /\tvar wg sync\.WaitGroup/);
 
       const compress = runNodeCli(["--files-compress", fixture, "--enable-line-numbers"], cwd);
       assert.equal(compress.status, 0, compress.stderr);

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

@@ -12,22 +12,21 @@ import {
   type OfflineContractSnapshot,
 } from "../scripts/lib/extension-debug-harness.js";
 import { buildParityReport, type SdkContractSnapshot } from "../scripts/lib/sdk-smoke.js";
-import { getProjectConfigPath } from "../src/core/config.js";
+import { loadConfig } from "../src/core/config.js";
 import { PI_USEREQ_STATUS_HOOK_NAMES } from "../src/core/extension-status.js";
-import { initFixtureRepo } from "./helpers.js";
+import { initFixtureRepo, saveFixtureConfigs } from "./helpers.js";
 
 /**
  * @brief Persists a targeted enabled-tool list into a fixture project config.
- * @details Loads `.pi-usereq.json`, replaces the `enabled-tools` array with the supplied values, and writes the updated JSON back to disk with a trailing newline. Runtime is O(n) in config size. Side effects include filesystem reads and file overwrite.
+ * @details Loads the effective split config, replaces the global `enabled-tools` array with the supplied values, and rewrites the local/global plus oracle mirror files so offline harness tests observe the same runtime state. Runtime is O(n) in config size. Side effects include filesystem reads and file overwrite.
  * @param[in] projectBase {string} Fixture project root.
  * @param[in] enabledTools {string[]} Enabled-tool names to persist.
  * @return {void} No return value.
  */
 function writeEnabledTools(projectBase: string, enabledTools: string[]): void {
-  const configPath = getProjectConfigPath(projectBase);
-  const config = JSON.parse(fs.readFileSync(configPath, "utf8")) as Record<string, unknown>;
+  const config = loadConfig(projectBase);
   config["enabled-tools"] = enabledTools;
-  fs.writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`, "utf8");
+  saveFixtureConfigs(projectBase, config);
 }
 
 /**
@@ -237,7 +236,7 @@ test("replaySessionStart captures active tools, statuses, and cwd semantics", as
     assert.match(status, /<accent>status:<\/accent><warning>idle<\/warning>/);
     assert.match(status, new RegExp(`<accent>branch:<\\/accent><warning>${escapedBranch}<\\/warning>`));
     assert.doesNotMatch(status, /<accent>current-path:<\/accent>/);
-    assert.match(status, /<accent>context:<\/accent>▕_▏/);
+    assert.match(status, /<accent>context:<\/accent><warning>▕_▏<\/warning>/);
     assert.match(status, /<accent>elapsed:<\/accent><warning>⏱︎ --:-- ⚑ --:-- ⌛︎--:--<\/warning>/);
     assert.match(status, /<accent>sound:<\/accent><warning>none<\/warning>/);
     assert.doesNotMatch(status, /<accent>beep:<\/accent>/);
@@ -257,10 +256,9 @@ test("replaySessionStart captures active tools, statuses, and cwd semantics", as
 test("replayCommand captures prompt command payloads", async () => {
   const { projectBase } = initFixtureRepo();
   try {
-    const configPath = getProjectConfigPath(projectBase);
-    const config = JSON.parse(fs.readFileSync(configPath, "utf8")) as Record<string, unknown>;
+    const config = loadConfig(projectBase);
     config.GIT_WORKTREE_ENABLED = "disable";
-    fs.writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`, "utf8");
+    saveFixtureConfigs(projectBase, config);
     assert.equal(spawnSync("git", ["add", ".pi-usereq.json", ".req/config.json"], {
       cwd: projectBase,
       encoding: "utf8",