@xynogen/pix-core 0.1.4 → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.1.4",
+	"version": "0.1.6",
 	"description": "Pi extension — core UI/UX bundle (welcome banner, footer, model picker, self-update)",
 	"type": "module",
 	"main": "src/index.ts",

package/src/index.ts

@@ -16,6 +16,7 @@
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import registerSkillLoader from "@xynogen/pix-skills";
 import registerAgentSop from "./commands/agent-sop/agent-sop.ts";
 import registerClear from "./commands/clear/clear.ts";
 import registerCopyAll from "./commands/copy-all/copy-all.ts";
@@ -34,6 +35,7 @@ import registerWelcome from "./ui/welcome.ts";
 
 export default function (pi: ExtensionAPI): void {
 	registerAgentSop(pi);
+	registerSkillLoader(pi);
 	registerWelcome(pi);
 	registerFooter(pi);
 	registerDiagnostics(pi);

package/src/lib/data.ts

@@ -11,6 +11,11 @@
  *   models.ts  — lookupModelsDev, lookupBenchmark
  */
 
+export type {
+	BenchmarkEntry,
+	ModelsDevApi,
+	ModelsDevModel,
+} from "../../../pix-data/src/index.ts";
 export {
 	benchmark,
 	buildModelsDevIndex,
@@ -21,12 +26,7 @@ export {
 	lookupInIndex,
 	lookupModelsDev,
 	modelsDev,
-} from "@xynogen/pix-data";
-export type {
-	BenchmarkEntry,
-	ModelsDevApi,
-	ModelsDevModel,
-} from "@xynogen/pix-data";
+} from "../../../pix-data/src/index.ts";
 
 export default function (_pi: unknown): void {
 	// pix-data warms this cache on startup — nothing to do here.

package/src/nudge/capability.test.ts

@@ -1,6 +1,6 @@
+import { describe, expect, test } from "bun:test";
 import { mkdirSync, rmSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
-import { describe, expect, test } from "bun:test";
 import {
 	buildOrientation,
 	CAPABILITY_REMINDER,
@@ -48,8 +48,8 @@ describe("CAPABILITY_REMINDER", () => {
 		expect(CAPABILITY_REMINDER.toLowerCase()).toContain("improvis");
 	});
 
-	test("nudges model to call skill() when a skill matches", () => {
-		expect(CAPABILITY_REMINDER).toContain("skill()");
+	test("nudges model to call read_skills() when a skill matches", () => {
+		expect(CAPABILITY_REMINDER).toContain("read_skills()");
 	});
 
 	test("points at /toolbox slash command for discovery (not a function call)", () => {
@@ -132,9 +132,9 @@ describe("buildOrientation", () => {
 		expect(out).toContain("2 skills");
 	});
 
-	test("explains how to use skill() and /toolbox for discovery", () => {
+	test("explains how to use read_skills() and /toolbox for discovery", () => {
 		const out = buildOrientation([tool("read", "builtin")], []);
-		expect(out).toContain("skill()");
+		expect(out).toContain("read_skills()");
 		expect(out).toContain("/toolbox");
 		// toolbox must NOT appear as a function call
 		expect(out).not.toContain("toolbox(");

package/src/nudge/capability.ts

@@ -10,14 +10,14 @@
  *   1. FIRST prompt of the session — an orientation block: a high-level
  *      description of WHAT is available (counts of tools / MCP tools / skills)
  *      and HOW to explore it. We deliberately do NOT dump the whole inventory
- *      every turn — the model should call skill() for skills and use /toolbox
+ *      every turn — the model should call read_skills() for skills and use /toolbox
  *      (slash command, user-facing) to discover/enable gated tools.
  *   2. EVERY subsequent prompt — the terse one-line CAPABILITY_REMINDER, a
- *      cheap (~40 tok) reinforcement that steers toward skill() and /toolbox.
+ *      cheap (~40 tok) reinforcement that steers toward read_skills() and /toolbox.
  *
  * NOTE: `toolbox` is a slash command only (/toolbox) — NOT a model-callable
  * function tool. The model cannot call toolbox() in function definitions.
- * The `skill` tool IS model-callable: skill() lists/loads bundled skills.
+ * The `read_skills` tool IS model-callable: read_skills() lists/loads bundled skills.
  */
 
 import { existsSync } from "node:fs";
@@ -34,7 +34,7 @@ type LoadedSkill = NonNullable<BuildSystemPromptOptions["skills"]>[number];
 export const CAPABILITY_REMINDER =
 	"Reminder — check knowledge resources " +
 	"(skills/tools/MCP/web/user) before improvising. " +
-	"Matching skill? Call skill() first. " +
+	"Matching skill? Call read_skills() first. " +
 	"Use /toolbox to discover/enable gated tools. " +
 	"All tools callable via function definitions.";
 
@@ -128,7 +128,7 @@ export function buildOrientation(
 	if (gateLine) lines.push(gateLine);
 	lines.push(
 		"Don't improvise what a capability covers — ask the user, search the web, or pull docs first.",
-		"`skill()` lists/loads bundled skills — call it when a skill matches your task. " +
+		"`read_skills()` lists/loads bundled skills — call it when a skill matches your task. " +
 			"/toolbox (slash command) discovers and enables gated tools.",
 	);
 	if (skillNames.length) {

package/src/tool/todo/todo.test.ts

@@ -1,5 +1,12 @@
 import { describe, expect, test } from "bun:test";
-import registerTodo from "./todo.ts";
+import registerTodo, { renderTodoLines, type TodoItem } from "./todo.ts";
+
+// Stub theme tags each fragment with its color/bold so assertions can verify
+// which status got which tint, without depending on real ANSI codes.
+const tagTheme = {
+	fg: (color: string, text: string) => `[${color}]${text}[/]`,
+	bold: (text: string) => `<b>${text}</b>`,
+};
 
 // ─── Helpers ────────────────────────────────────────────────────────────────
 
@@ -600,3 +607,40 @@ describe("restore", () => {
 		expect(text(result)).toBe("(no todos)");
 	});
 });
+
+describe("renderTodoLines (colored TUI render)", () => {
+	const items: TodoItem[] = [
+		{ id: 1, text: "alpha", status: "done" },
+		{ id: 2, text: "bravo", status: "in_progress" },
+		{ id: 3, text: "charlie", status: "pending" },
+		{ id: 4, text: "delta", status: "blocked" },
+	];
+
+	test("empty list renders muted placeholder", () => {
+		expect(renderTodoLines([], tagTheme)).toBe("[muted](no todos)[/]");
+	});
+
+	test("tints each glyph by status", () => {
+		const out = renderTodoLines(items, tagTheme);
+		expect(out).toContain("[success]●[/]"); // done
+		expect(out).toContain("[accent]◐[/]"); // in_progress
+		expect(out).toContain("[muted]○[/]"); // pending
+		expect(out).toContain("[error]⊘[/]"); // blocked
+	});
+
+	test("highlights the in-progress row bold + accent", () => {
+		const out = renderTodoLines(items, tagTheme);
+		expect(out).toContain("<b>[accent]2. bravo[/]</b>");
+	});
+
+	test("dims completed rows and uses text color for active-but-not-running", () => {
+		const out = renderTodoLines(items, tagTheme);
+		expect(out).toContain("[muted]1. alpha[/]"); // done body muted
+		expect(out).toContain("[text]3. charlie[/]"); // pending body text
+	});
+
+	test("shows the done/total count header", () => {
+		const out = renderTodoLines(items, tagTheme);
+		expect(out).toContain("[muted]Todos 1/4 done:[/]");
+	});
+});

package/src/tool/todo/todo.ts

@@ -10,6 +10,7 @@
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
 export type TodoStatus = "pending" | "in_progress" | "done" | "blocked";
@@ -27,6 +28,38 @@ const TODO_GLYPH: Record<TodoStatus, string> = {
 	blocked: "⊘",
 };
 
+/** Theme color key per status — drives both glyph and (for active) row tint. */
+const TODO_COLOR: Record<TodoStatus, string> = {
+	pending: "muted",
+	in_progress: "accent",
+	done: "success",
+	blocked: "error",
+};
+
+export type TodoTheme = {
+	fg: (color: string, text: string) => string;
+	bold: (text: string) => string;
+};
+
+/** Colored checklist for the TUI: glyphs tinted by status, active row bold. */
+export function renderTodoLines(items: TodoItem[], theme: TodoTheme): string {
+	if (!items.length) return theme.fg("muted", "(no todos)");
+	const done = items.filter((t) => t.status === "done").length;
+	const head = theme.fg("muted", `Todos ${done}/${items.length} done:`);
+	const lines = items.map((t) => {
+		const color = TODO_COLOR[t.status];
+		const glyph = theme.fg(color, TODO_GLYPH[t.status]);
+		const body = `${t.id}. ${t.text}`;
+		// Highlight the in-flight task so the eye lands on it first.
+		const label =
+			t.status === "in_progress"
+				? theme.bold(theme.fg("accent", body))
+				: theme.fg(t.status === "done" ? "muted" : "text", body);
+		return `${glyph} ${label}`;
+	});
+	return `${head}\n${lines.join("\n")}`;
+}
+
 const parseItems = (raw: string): string[] =>
 	raw
 		.split("\n")
@@ -102,6 +135,10 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				}),
 			),
 		}),
+		renderResult(_result, _options, theme) {
+			return new Text(renderTodoLines(todos, theme as TodoTheme), 0, 0);
+		},
+
 		async execute(_id, params) {
 			// AgentToolResult now requires a `details` field. These todo results have
 			// no structured details, so emit `undefined` via small local helpers.