pi-usereq 0.33.0 → 0.35.0

package/tests/extension-registration.test.ts

@@ -475,10 +475,39 @@ function buildExpectedFakeContextBar(options: {
   return formatFakeThemeForeground("warning", "▕█▏");
 }
 
+/**
+ * @brief Reads the repository package manifest fields used by status-bar tests.
+ * @details Loads `../package.json` relative to this test module so status-bar assertions derive the expected runtime extension name and version from repository metadata instead of hard-coded literals. Runtime is O(n) in file size. Side effects are limited to one filesystem read.
+ * @return {{ name: string; version: string }} Expected runtime extension package metadata.
+ */
+function readExpectedRuntimeExtensionMetadata(): { name: string; version: string } {
+  const packageJson = JSON.parse(
+    fs.readFileSync(new URL("../package.json", import.meta.url), "utf8"),
+  ) as { name?: unknown; version?: unknown };
+  return {
+    name: typeof packageJson.name === "string" && packageJson.name.trim() !== ""
+      ? packageJson.name.trim()
+      : "pi-usereq",
+    version: typeof packageJson.version === "string" && packageJson.version.trim() !== ""
+      ? packageJson.version.trim()
+      : "unknown",
+  };
+}
+
+/**
+ * @brief Builds the expected fake runtime extension identity prefix for status assertions.
+ * @details Renders `<extension>:v<version>` using the same accent-label and warning-value markers consumed by the fake status theme so prefix assertions remain aligned with runtime package metadata. Runtime is O(n) in rendered text length. No external state is mutated.
+ * @return {string} Encoded extension identity prefix.
+ */
+function buildExpectedFakeExtensionIdentityStatusField(): string {
+  const metadata = readExpectedRuntimeExtensionMetadata();
+  return `${formatFakeThemeForeground("accent", `${metadata.name}:`)}${formatFakeThemeForeground("warning", `v${metadata.version}`)}`;
+}
+
 /**
  * @brief Builds the expected fake pi-usereq status-bar string for assertions.
- * @details Reconstructs the field order, workflow-state highlighting, branch field, icon-based context gauge, consolidated elapsed field, and sound field emitted by the extension using deterministic fake theme markers. Runtime is O(1) plus optional git execution for branch discovery. No external state is mutated.
- * @param[in] options {{ workflowState?: string; basePath?: string; contextFilledCells: number; contextPercent?: number | null; et: string; sound?: string }} Expected status facts for rendered `status`, `branch`, `context`, `elapsed`, and `sound` fields.
+ * @details Reconstructs the runtime extension identity prefix, workflow-state highlighting, branch field, icon-based context gauge, consolidated elapsed field, and sound field emitted by the extension using deterministic fake theme markers. Runtime is O(1) plus optional git execution for branch discovery. No external state is mutated.
+ * @param[in] options {{ workflowState?: string; basePath?: string; contextFilledCells: number; contextPercent?: number | null; et: string; sound?: string }} Expected status facts for rendered runtime extension prefix, `status`, `branch`, `context`, `elapsed`, and `sound` fields.
  * @return {string} Encoded status-bar string.
  */
 function buildExpectedFakeStatusText(options: {
@@ -504,6 +533,7 @@ function buildExpectedFakeStatusText(options: {
     percent: options.contextPercent,
   });
   return [
+    buildExpectedFakeExtensionIdentityStatusField(),
     `${formatFakeThemeForeground("accent", "status:")}${workflowStateValue}`,
     buildField("branch", branchValue),
     `${formatFakeThemeForeground("accent", "context:")}${contextBar}`,
@@ -842,6 +872,7 @@ test("extension registers prompt and config commands while exposing tool capabil
     "debug-compress",
     "debug-references",
     "debug-static-check",
+    "debug-summarize",
     "debug-tokens",
     "pi-usereq-show-config",
     "test-static-check",
@@ -869,7 +900,7 @@ test("extension registers prompt and config commands while exposing tool capabil
 
 /**
  * @brief Verifies debug tool wrapper commands register only when the local debug flag is enabled.
- * @details Activates the extension once from a default-disabled fixture and once from a fixture whose local config enables debug tool wrapper commands, then asserts the four debug slash commands appear only in the enabled runtime. Runtime is dominated by fixture setup and extension registration. Side effects are limited to temporary filesystem mutation and process cwd changes.
+ * @details Activates the extension once from a default-disabled fixture and once from a fixture whose local config enables debug tool wrapper commands, then asserts the five debug slash commands appear only in the enabled runtime. Runtime is dominated by fixture setup and extension registration. Side effects are limited to temporary filesystem mutation and process cwd changes.
  * @return {void} No return value.
  * @throws {AssertionError} Throws when command registration ignores the local debug tool wrapper flag.
  * @satisfies TST-115
@@ -881,6 +912,7 @@ test("debug tool commands register only when enabled in local config", () => {
     "debug-compress",
     "debug-references",
     "debug-static-check",
+    "debug-summarize",
     "debug-tokens",
   ];
 
@@ -1606,7 +1638,7 @@ test("static-check tools return monolithic text payloads with minimal execution
 
 /**
  * @brief Verifies debug tool wrapper commands mirror tool content and reject when disabled.
- * @details Enables debug tool wrapper commands for one fixture project, compares each debug slash-command editor payload against the matching tool `content[0].text`, then disables the flag and verifies the already-registered wrapper rejects further execution. Runtime is dominated by tool execution plus temporary git fixture setup. Side effects are limited to temporary filesystem mutation, editor-text capture, and process cwd changes.
+ * @details Enables debug tool wrapper commands for one fixture project, compares the five debug slash-command editor payloads against the matching tool `content[0].text`, then disables the flag and verifies the already-registered wrappers reject further execution. Runtime is dominated by tool execution plus temporary git fixture setup. Side effects are limited to temporary filesystem mutation, editor-text capture, and process cwd changes.
  * @return {Promise<void>} Promise resolved when the verification completes.
  * @throws {AssertionError} Throws when a debug wrapper output diverges from the wrapped tool text or disabled execution is accepted.
  * @satisfies TST-116
@@ -1643,6 +1675,9 @@ test("debug tool wrapper commands mirror tool content and reject when disabled",
     const staticCheckToolResult = await executeRegisteredTool(pi, "static-check", projectBase) as {
       content?: Array<{ type: string; text?: string }>;
     };
+    const summarizeToolResult = await executeRegisteredTool(pi, "summarize", projectBase, {}) as {
+      content?: Array<{ type: string; text?: string }>;
+    };
     const tokensToolResult = await executeRegisteredTool(pi, "tokens", projectBase, {}) as {
       content?: Array<{ type: string; text?: string }>;
     };
@@ -1659,6 +1694,10 @@ test("debug tool wrapper commands mirror tool content and reject when disabled",
     await pi.commands.get("debug-static-check")!.handler("", staticCheckCtx);
     assert.equal(staticCheckCtx.__state.editorText, staticCheckToolResult.content?.[0]?.text ?? "");
 
+    const summarizeCtx = createFakeCtx(projectBase);
+    await pi.commands.get("debug-summarize")!.handler("", summarizeCtx);
+    assert.equal(summarizeCtx.__state.editorText, summarizeToolResult.content?.[0]?.text ?? "");
+
     const tokensCtx = createFakeCtx(projectBase);
     await pi.commands.get("debug-tokens")!.handler("", tokensCtx);
     assert.equal(tokensCtx.__state.editorText, tokensToolResult.content?.[0]?.text ?? "");
@@ -1666,12 +1705,20 @@ test("debug tool wrapper commands mirror tool content and reject when disabled",
     writeProjectConfigOverrides(projectBase, {
       DEBUG_TOOL_COMMANDS_ENABLED: "disable",
     });
-    const disabledCtx = createFakeCtx(projectBase);
-    await assert.rejects(
-      async () => pi.commands.get("debug-tokens")!.handler("", disabledCtx),
-      /disabled by Debug > Enable debug commands for tools/u,
-    );
-    assert.equal(disabledCtx.__state.editorText, "");
+    for (const commandName of [
+      "debug-compress",
+      "debug-references",
+      "debug-static-check",
+      "debug-summarize",
+      "debug-tokens",
+    ]) {
+      const disabledCtx = createFakeCtx(projectBase);
+      await assert.rejects(
+        async () => pi.commands.get(commandName)!.handler("", disabledCtx),
+        /disabled by Debug > Enable debug commands for tools/u,
+      );
+      assert.equal(disabledCtx.__state.editorText, "");
+    }
   } finally {
     process.chdir(previousCwd);
     fs.rmSync(projectBase, { recursive: true, force: true });
@@ -1968,6 +2015,10 @@ test("debug menu dims locked rows and persists debug settings with focus-preserv
   const debugCalls = ctx.__state.selectCalls.filter((call) => call.title === "Debug");
   assert.match(renderedInitialDebugMenu, /<dim>Log file/);
   assert.match(renderedInitialDebugMenu, /<dim>Enable debug commands for tools/);
+  assert.ok(
+    renderedInitialDebugMenu.indexOf("<dim>Enable debug commands for tools")
+      < renderedInitialDebugMenu.indexOf("<dim>Log file"),
+  );
   assert.match(renderedInitialDebugMenu, /<dim>Log on status/);
   assert.match(renderedInitialDebugMenu, /<dim>Status changes/);
   assert.match(renderedInitialDebugMenu, /<dim>Workflow events/);
@@ -1981,6 +2032,7 @@ test("debug menu dims locked rows and persists debug settings with focus-preserv
   assert.ok(pi.commands.get("debug-compress"));
   assert.ok(pi.commands.get("debug-references"));
   assert.ok(pi.commands.get("debug-static-check"));
+  assert.ok(pi.commands.get("debug-summarize"));
   assert.ok(pi.commands.get("debug-tokens"));
   assert.deepEqual(persistedConfig.DEBUG_ENABLED_TOOLS, ["files-tokens"]);
   assert.deepEqual(persistedConfig.DEBUG_ENABLED_PROMPTS, ["req-analyze"]);
@@ -2023,8 +2075,8 @@ test("debug menu rows derive from canonical tool and prompt inventories", async
 
   assert.deepEqual(ctx.__state.selectCalls[1]?.items ?? [], [
     "Debug",
-    "Log file",
     "Enable debug commands for tools",
+    "Log file",
     "Log on status",
     "Status changes",
     "Workflow events",
@@ -5003,6 +5055,27 @@ test("session_start applies configured pi-usereq startup tools", async () => {
   );
 });
 
+/**
+ * @brief Verifies the status bar prepends runtime extension identity before the `status` field.
+ * @details Emits `session_start`, reads the rendered footer text, and asserts the prefix starts with the runtime package name plus `v<version>` before `status:` using the same accent-label and warning-value markers used by other non-error status fields. Runtime is O(1). Side effects are limited to temporary filesystem mutation and in-memory status capture.
+ * @return {Promise<void>} Promise resolved after the status-prefix assertion completes.
+ * @throws {AssertionError} Throws when the footer omits or misorders the runtime extension identity prefix.
+ * @satisfies TST-031, TST-033, TST-049
+ */
+test("session_start status bar prepends runtime extension identity before status", async () => {
+  const cwd = createTempDir("pi-usereq-extension-identity-status-");
+  fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });
+  const pi = createFakePi();
+  piUsereqExtension(pi);
+  const ctx = createFakeCtx(cwd);
+
+  await pi.emit("session_start", { reason: "startup" }, ctx);
+
+  const statusText = ctx.__state.statuses.get("pi-usereq") ?? "";
+  const expectedPrefix = `${buildExpectedFakeExtensionIdentityStatusField()}${formatFakeThemeForeground("dim", " • ")}${formatFakeThemeForeground("accent", "status:")}`;
+  assert.ok(statusText.startsWith(expectedPrefix));
+});
+
 test("session_start renders the 0-percent context icon when context usage is empty", async () => {
   const cwd = createTempDir("pi-usereq-context-empty-status-");
   fs.mkdirSync(path.dirname(getProjectConfigPath(cwd)), { recursive: true });

package/tests/release-workflow.test.ts

@@ -1,4 +1,5 @@
 import assert from "node:assert/strict";
+import { spawnSync } from "node:child_process";
 import fs from "node:fs";
 import path from "node:path";
 import test from "node:test";
@@ -16,6 +17,61 @@ function readReleaseWorkflow(): string {
   );
 }
 
+/**
+ * @brief Verifies the release workflow YAML parses successfully via Python.
+ * @details Resolves a project-local `.venv/bin/python` first, then falls back to `python` and `python3`, executes an inline `yaml.safe_load(...)` check against `.github/workflows/release-npm.yml`, accepts the `PyYAML not available` skip path, and fails on non-zero exit status or `YAML ERROR` output. Runtime is dominated by subprocess probing and one Python execution. Side effects are limited to subprocess creation and filesystem reads performed by the Python snippet.
+ * @return {void} No return value.
+ * @throws {AssertionError} Throws when no usable Python interpreter exists or the workflow YAML fails to parse.
+ * @satisfies TST-039
+ */
+function assertReleaseWorkflowYamlSyntax(): void {
+  const workflowPath = path.join(
+    process.cwd(),
+    ".github",
+    "workflows",
+    "release-npm.yml",
+  );
+  const pythonCandidates = [
+    path.join(process.cwd(), ".venv", "bin", "python"),
+    "python",
+    "python3",
+  ];
+  const pythonCommand = pythonCandidates.find((candidate) => {
+    if (candidate.includes(path.sep) && !fs.existsSync(candidate)) {
+      return false;
+    }
+    const probe = spawnSync(candidate, ["--version"], { encoding: "utf8" });
+    return !probe.error && probe.status === 0;
+  });
+  assert.ok(pythonCommand, "python executable not found for release workflow validation");
+
+  const pythonCode = [
+    "from pathlib import Path",
+    "import sys",
+    `p = Path(${JSON.stringify(workflowPath)})`,
+    "text = p.read_text()",
+    "try:",
+    "    import yaml",
+    "except Exception as error:",
+    "    print(f'PyYAML not available: {error}')",
+    "    sys.exit(0)",
+    "try:",
+    "    yaml.safe_load(text)",
+    "    print('YAML OK')",
+    "except Exception as error:",
+    "    print(f'YAML ERROR: {error}')",
+    "    sys.exit(1)",
+  ].join("\n");
+  const result = spawnSync(pythonCommand, ["-c", pythonCode], {
+    cwd: process.cwd(),
+    encoding: "utf8",
+  });
+  const output = `${result.stdout}${result.stderr}`;
+
+  assert.equal(result.status, 0, output || result.error?.message || "unknown python execution failure");
+  assert.doesNotMatch(output, /YAML ERROR:/u);
+}
+
 /**
  * @brief Describes the package manifest fields asserted by npm release tests.
  * @details Constrains the parsed `package.json` shape to the publication identity and provenance metadata required by the release workflow tests. The interface is compile-time only and introduces no runtime side effects.
@@ -75,6 +131,11 @@ function assertCanonicalPackageProvenanceMetadata(): void {
   assert.equal(manifest.homepage, "https://github.com/Ogekuri/PI-useReq#readme");
 }
 
+test(
+  "release workflow remains valid YAML when parsed by Python",
+  assertReleaseWorkflowYamlSyntax,
+);
+
 test(
   "release workflow keeps the current tag trigger and gates release work on origin/master ancestry",
   () => {