pi-ui-extend 0.1.9 → 0.1.13

package/external/pi-tools-suite/src/lsp/_shared/output.ts

@@ -4,6 +4,7 @@ import type { CommandRunResult } from "./types";
 import { normalizeRelativePath, uriToFilePath } from "./paths";
 
 const DEFAULT_OUTPUT_LIMIT = 4000;
+export const LSP_DIAGNOSTIC_ICON = "\u{f0026}";
 
 export function textFromContent(content: Array<{ type: string; text?: string }>): string {
   return content
@@ -29,20 +30,20 @@ export function commandOutput(result: Pick<CommandRunResult, "stdout" | "stderr"
 
 export function formatWarnings(title: string, warnings: string[]): string {
   if (warnings.length === 0) return "";
-  return `${title}:\n\n${warnings.map((warning) => `⚠️ ${warning}`).join("\n")}`;
+  return `${title}:\n\n${warnings.map((warning) => `${LSP_DIAGNOSTIC_ICON} ${warning}`).join("\n")}`;
 }
 
 export function formatCommandIssue(toolId: string, action: string, result: CommandRunResult): string {
   const output = commandOutput(result);
   const suffix = result.killed ? " (killed/timeout)" : "";
-  if (!output) return `⚠️ ${toolId} ${action} failed with exit code ${result.code}${suffix}`;
-  return `⚠️ ${toolId} ${action} failed with exit code ${result.code}${suffix}:\n${output}`;
+  if (!output) return `${LSP_DIAGNOSTIC_ICON} ${toolId} ${action} failed with exit code ${result.code}${suffix}`;
+  return `${LSP_DIAGNOSTIC_ICON} ${toolId} ${action} failed with exit code ${result.code}${suffix}:\n${output}`;
 }
 
 export function formatDiagnosticOutput(toolId: string, output: string): string {
   const compact = truncateOutput(output);
-  if (!compact) return `⚠️ ${toolId} found issues`;
-  return `⚠️ ${toolId} found issues:\n${compact}`;
+  if (!compact) return `${LSP_DIAGNOSTIC_ICON} ${toolId} found issues`;
+  return `${LSP_DIAGNOSTIC_ICON} ${toolId} found issues:\n${compact}`;
 }
 
 function severityLabel(severity: number | undefined): string {
@@ -88,11 +89,11 @@ export function formatLspDiagnostics(serverId: string, file: string, diagnostics
   if (diagnostics.length === 0) return "";
   const rendered = diagnostics.slice(0, 20).map((diagnostic) => formatDiagnostic(file, diagnostic, root));
   if (diagnostics.length > 20) rendered.push(`… ${diagnostics.length - 20} more diagnostics`);
-  return `⚠️ ${serverId}:\n${rendered.join("\n")}`;
+  return `${LSP_DIAGNOSTIC_ICON} ${serverId}:\n${rendered.join("\n")}`;
 }
 
 export function hasIssueOutput(output: string): boolean {
-  return output.includes("⚠️");
+  return output.includes(LSP_DIAGNOSTIC_ICON) || output.includes("⚠️") || output.includes("⚠");
 }
 
 export function joinSections(title: string, lines: string[]): string {

package/external/pi-tools-suite/src/lsp/manager.ts

@@ -6,7 +6,7 @@ import { LspClient } from "./client";
 import { loadLspConfig } from "./_shared/config";
 import { isPathIncluded } from "./_shared/glob";
 import { filePathToUri, findProjectRoot, normalizeRelativePath, resolveCommand, toAbsolutePath } from "./_shared/paths";
-import { formatLspDiagnostics, formatWarnings, joinSections } from "./_shared/output";
+import { formatLspDiagnostics, formatWarnings, joinSections, LSP_DIAGNOSTIC_ICON } from "./_shared/output";
 import type { LspServerConfig, StoredDiagnostics } from "./_shared/types";
 import { clientKey, couldMatchBeforeRoot, fileSizeAllowed, languageIdForFile, readTextFile } from "./lsp-utils";
 import { localMarkdownDiagnostics } from "./markdown-diagnostics";
@@ -112,7 +112,7 @@ export class LspManager {
       try {
         const maxFileSizeBytes = match.server.maxFileSizeBytes ?? DEFAULT_MAX_FILE_SIZE_BYTES;
         if (!(await fileSizeAllowed(file, maxFileSizeBytes))) {
-          lines.push(`⚠️ ${match.server.id}: skipped ${match.relFile}; file exceeds maxFileSizeBytes (${maxFileSizeBytes})`);
+          lines.push(`${LSP_DIAGNOSTIC_ICON} ${match.server.id}: skipped ${match.relFile}; file exceeds maxFileSizeBytes (${maxFileSizeBytes})`);
           continue;
         }
 
@@ -173,14 +173,14 @@ export class LspManager {
         if (!isFreshDiagnosticsEntry(entry, startedAt, doc.version)) {
           const fallbackSuffix = tsserverFallbackError ? `; tsserver fallback failed: ${tsserverFallbackError}` : "";
           const pullSuffix = pullDiagnosticsError ? `; pull diagnostics failed: ${pullDiagnosticsError}` : "";
-          lines.push(`⚠️ ${match.server.id}: timed out after ${diagnosticsWaitMs}ms waiting for fresh diagnostics for ${match.relFile}${fallbackSuffix}${pullSuffix}`);
+          lines.push(`${LSP_DIAGNOSTIC_ICON} ${match.server.id}: timed out after ${diagnosticsWaitMs}ms waiting for fresh diagnostics for ${match.relFile}${fallbackSuffix}${pullSuffix}`);
           continue;
         }
         const diagnostics = diagnosticsWithLocalFallback(match.server.id, file, text, entry.diagnostics);
         if (diagnostics !== entry.diagnostics) this.diagnostics.set(match.server.id, match.root, filePathToUri(file), diagnostics, doc.version);
         lines.push(formatLspDiagnostics(match.server.id, file, diagnostics, match.root));
       } catch (error) {
-        lines.push(`⚠️ ${match.server.id}: ${(error as Error).message}`);
+        lines.push(`${LSP_DIAGNOSTIC_ICON} ${match.server.id}: ${(error as Error).message}`);
       }
     }
 

package/external/pi-tools-suite/src/repo-discovery/index.ts

@@ -112,9 +112,56 @@ async function initializeIndexedProject(pi: ExtensionAPI, cwd: string, signal: A
 	const indexerDir = path.join(projectRoot, ".indexer-cli");
 	if (directoryExists(indexerDir)) return { projectRoot, initialized: false, alreadyIndexed: true, output: "Project is already indexed." };
 
+	const idxCli = await ensureIdxCliAvailable(pi, projectRoot, signal);
+	if (!idxCli.available) {
+		return {
+			projectRoot,
+			initialized: false,
+			alreadyIndexed: false,
+			output: idxCli.output,
+			exitCode: idxCli.exitCode,
+			installedIdx: idxCli.installed,
+		};
+	}
+
 	const init = await pi.exec("idx", ["init"], { cwd: projectRoot, signal, timeout: 600_000 });
-	const output = [init.stdout, init.stderr].filter(Boolean).join(init.stdout && init.stderr ? "\n" : "").trim() || "No output";
-	return { projectRoot, initialized: (init.code ?? 0) === 0, alreadyIndexed: false, output, exitCode: init.code ?? 0 };
+	const initOutput = [init.stdout, init.stderr].filter(Boolean).join(init.stdout && init.stderr ? "\n" : "").trim() || "No output";
+	const output = idxCli.installed
+		? [`idx was not available; installed with npm install -g indexer-cli@latest:`, idxCli.output, `idx init output:`, initOutput].join("\n\n")
+		: initOutput;
+	return { projectRoot, initialized: (init.code ?? 0) === 0, alreadyIndexed: false, output, exitCode: init.code ?? 0, installedIdx: idxCli.installed };
+}
+
+async function ensureIdxCliAvailable(pi: ExtensionAPI, cwd: string, signal: AbortSignal | undefined) {
+	const check = await runCommandSafely(pi, "sh", ["-lc", "command -v idx"], { cwd, signal, timeout: 30_000 });
+	if ((check.code ?? 0) === 0) return { available: true, installed: false, output: "idx is available.", exitCode: 0 };
+
+	const install = await runCommandSafely(pi, "npm", ["install", "-g", "indexer-cli@latest"], { cwd, signal, timeout: 600_000 });
+	const installOutput = [install.stdout, install.stderr].filter(Boolean).join(install.stdout && install.stderr ? "\n" : "").trim() || "No output";
+	const exitCode = install.code ?? 0;
+	if (exitCode !== 0) {
+		return {
+			available: false,
+			installed: false,
+			output: [`idx is not available and npm install -g indexer-cli@latest failed:`, installOutput].join("\n\n"),
+			exitCode,
+		};
+	}
+
+	return { available: true, installed: true, output: installOutput, exitCode };
+}
+
+async function runCommandSafely(
+	pi: ExtensionAPI,
+	command: string,
+	args: string[],
+	options: { cwd?: string; signal?: AbortSignal; timeout?: number },
+): Promise<ExecResult> {
+	try {
+		return await pi.exec(command, args, options);
+	} catch (error) {
+		return { stdout: "", stderr: error instanceof Error ? error.message : String(error), code: 1 };
+	}
 }
 
 async function updateIndexerCli(pi: ExtensionAPI, cwd: string, signal: AbortSignal | undefined) {

package/external/pi-tools-suite/src/todo/index.ts

@@ -2,7 +2,7 @@ import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-a
 import { autoClearCompletedTodos } from "./state/auto-clear.js";
 import { loadPersistedPlan, syncPersistedPlan } from "./state/persistence.js";
 import { replayFromBranch } from "./state/replay.js";
-import { ACTIVE_STATUSES, selectVisibleTasks } from "./state/selectors.js";
+import { ACTIVE_STATUSES, isTaskBlocked, selectVisibleTasks } from "./state/selectors.js";
 import { getState, replaceState } from "./state/store.js";
 import { publishTodoState, registerTodosCommand, registerTodoTool } from "./todo.js";
 
@@ -17,7 +17,9 @@ function isAskUserToolName(toolName: string): boolean {
 }
 
 function getUnfinishedTodoNudge(): { signature: string; message: string } | undefined {
-	const unfinished = selectVisibleTasks(getState()).filter((task) => ACTIVE_STATUSES.has(task.status));
+	const visible = selectVisibleTasks(getState());
+	const byId = new Map(visible.map((task) => [task.id, task]));
+	const unfinished = visible.filter((task) => ACTIVE_STATUSES.has(task.status) && !isTaskBlocked(task, byId));
 	if (unfinished.length === 0) return undefined;
 
 	const signature = JSON.stringify(

package/external/pi-tools-suite/src/todo/state/selectors.ts

@@ -47,3 +47,7 @@ export function selectTodoCounts(state: TaskState): TodoCounts {
 }
 
 export const ACTIVE_STATUSES: ReadonlySet<TaskStatus> = new Set(["pending", "in_progress"]);
+
+export function isTaskBlocked(task: Task, byId: ReadonlyMap<number, Task>): boolean {
+	return (task.blockedBy ?? []).some((id) => byId.get(id)?.status !== "completed");
+}

package/external/pi-tools-suite/src/todo/todo.ts

@@ -23,7 +23,7 @@ import {
 } from "./state/persistence.js";
 import { AUTO_CLEAR_COMPLETED_MESSAGE, autoClearCompletedTodos } from "./state/auto-clear.js";
 import { replayFromBranch } from "./state/replay.js";
-import { selectTasksByStatus, selectTodoCounts } from "./state/selectors.js";
+import { isTaskBlocked, selectTasksByStatus, selectTodoCounts } from "./state/selectors.js";
 import { applyTaskMutation } from "./state/state-reducer.js";
 import { commitState, getState, replaceState } from "./state/store.js";
 import { buildToolResult, formatContent } from "./tool/response-envelope.js";
@@ -287,11 +287,7 @@ function filterCommandTasks(tasks: readonly Task[], options: TodosCommandOptions
 	if (!options.includeDeleted) view = view.filter((task) => task.status !== "deleted");
 	if (options.activeOnly) view = view.filter((task) => task.status === "pending" || task.status === "in_progress");
 	if (options.readyOnly) {
-		view = view.filter(
-			(task) =>
-				task.status === "pending" &&
-				(task.blockedBy ?? []).every((id) => byId.get(id)?.status === "completed"),
-		);
+		view = view.filter((task) => task.status === "pending" && !isTaskBlocked(task, byId));
 	}
 	if (options.status) view = view.filter((task) => task.status === options.status);
 	if (options.priority) view = view.filter((task) => task.priority === options.priority);

package/external/pi-tools-suite/src/todo/tool/response-envelope.ts

@@ -131,7 +131,7 @@ export function buildToolResult(
 	state: TaskState,
 	op: Op,
 ): { content: Array<{ type: "text"; text: string }>; details: TaskDetails } {
-	const text = formatContent(op, state);
+	const text = appendWorkflowReminder(formatContent(op, state), op, state);
 	const details: TaskDetails = {
 		action,
 		params: params as Record<string, unknown>,
@@ -141,3 +141,11 @@ export function buildToolResult(
 	};
 	return { content: [{ type: "text", text }], details };
 }
+
+function appendWorkflowReminder(text: string, op: Op, state: TaskState): string {
+	if (op.kind === "error" || op.kind === "export") return text;
+	const hasPending = state.tasks.some((task) => task.status === "pending");
+	const hasInProgress = state.tasks.some((task) => task.status === "in_progress");
+	if (!hasPending || hasInProgress) return text;
+	return `${text}\n\nReminder: pending todos exist but none is in_progress. Before starting work, call todo update on exactly one task with status in_progress and activeForm.`;
+}

package/external/pi-tools-suite/src/tool-descriptions.ts

@@ -256,7 +256,7 @@ export const TODO_TOOL_DESCRIPTION: ToolDescription = {
 		"When the user adds, removes, cancels, reprioritizes, or changes the goal, scope, requirements, constraints, or chosen approach, use `todo` before continuing to synchronize the plan: update still-relevant tasks, defer or delete obsolete tasks, add new required tasks, and adjust dependencies/order.",
 		"When your own investigation or verification discovers new facts that make the current todo plan stale, incomplete, impossible, unsafe, or no longer the best approach, use `todo` to revise the plan immediately instead of following outdated tasks.",
 		"Update todos as part of the workflow, not as end-of-task cleanup: whenever you start, finish, block, split, abandon, or materially change a step, call `todo` immediately before continuing.",
-		"Before starting work, mark exactly one task in_progress with activeForm (present-continuous label). Mark it completed immediately after the required verification, never in batches.",
+		"Before any non-trivial read/edit/test/tool work on a planned task, mark exactly one task in_progress with activeForm (present-continuous label); do this immediately after creating a plan if no task is active. Mark it completed immediately after the required verification, never in batches.",
 		"If implementation is partial, tests fail, or a blocker remains, keep the task in_progress and add/update a blocker task instead of completing it.",
 		"Never use `clear`, `delete`, or batch deletion to hide unfinished, stale, or forgotten todos. Defer obsolete items or update them with the reason; only delete when the user explicitly asks or the item was created by mistake.",
 		"Before giving a final response for work that used todos, ensure every visible todo is completed, deferred, or intentionally still in_progress with a blocker/explanation.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-ui-extend",
-  "version": "0.1.9",
+  "version": "0.1.13",
   "private": false,
   "type": "module",
   "bin": {
@@ -14,13 +14,16 @@
     "./sdk": {
       "types": "./dist/sdk.d.ts",
       "import": "./dist/sdk.js"
-    }
+    },
+    "./schemas/pix.json": "./schemas/pix.json",
+    "./schemas/pi-tools-suite.json": "./schemas/pi-tools-suite.json"
   },
   "files": [
     "bin",
     "docs",
     "dist",
     "extensions",
+    "schemas",
     "skills",
     "external/pi-tools-suite/index.ts",
     "external/pi-tools-suite/package.json",
@@ -34,22 +37,28 @@
     "predev": "npm run link:pix --silent",
     "dev": "mise exec node@24.16.0 -- tsx src/main.ts",
     "start": "mise exec node@24.16.0 -- tsx src/main.ts",
+    "prebuild:pix": "npm run generate-schemas --silent",
     "build:pix": "mise exec node@24.16.0 -- tsc -p tsconfig.json",
     "prelink:pix": "npm run build:pix --silent",
     "link:pix": "mise exec node@24.16.0 -- npm link --silent",
     "watch:pix": "npm run link:pix --silent && mise exec node@24.16.0 -- tsc -p tsconfig.json --watch --preserveWatchOutput --noEmitOnError",
     "check": "mise exec node@24.16.0 -- npm run check:inner",
+    "precheck:inner": "npm run generate-schemas --silent",
     "check:inner": "tsc --noEmit && npm run test:inner",
     "test": "mise exec node@24.16.0 -- npm run test:inner",
     "test:inner": "node --import tsx --test \"tests/**/*.test.ts\"",
     "test:tools-suite": "npm --prefix external/pi-tools-suite test",
     "test:coverage": "mise exec node@24.16.0 -- node --import tsx --test --experimental-test-coverage --test-coverage-include=src/**/*.ts --test-coverage-exclude=src/main.ts --test-coverage-lines=95 --test-coverage-branches=80 --test-coverage-functions=95 \"tests/**/*.test.ts\"",
     "update-sdk-references": "mise exec node@24.16.0 -- node .pi/skills/pi-sdk/scripts/update-references.mjs",
+    "prepack": "npm run generate-schemas --silent",
     "pack:dry-run": "npm pack --dry-run",
     "smoke-test": "mise exec node@24.16.0 -- bash scripts/smoke-test-package.sh",
     "release:check": "npm run check && npm run build:pix && npm run pack:dry-run",
     "publish-npm": "bash scripts/publish.sh",
-    "prepublishOnly": "npm run check && npm run build:pix"
+    "generate-schemas": "tsx scripts/generate-schemas.ts",
+    "generate-schemas:check": "tsx scripts/generate-schemas.ts --check",
+    "generate-schemas:watch": "tsx watch scripts/generate-schemas.ts",
+    "prepublishOnly": "npm run check && npm run build:pix && npm run generate-schemas"
   },
   "dependencies": {
     "@earendil-works/pi-tui": "0.77.0",