@xynogen/pix-core 0.2.4 → 0.3.1

package/README.md

@@ -1,35 +1,38 @@
 # pix-core
 
-Pi coding agent extension — core UI/UX bundle.
+Pi coding agent extension — core UI/UX meta-package.
+
+Installing `pix-core` pulls in all of the packages below as npm dependencies — no source code of its own.
 
 ## What's included
 
-| Extension | Type | Description |
-|---|---|---|
-| `welcome` | lifecycle | ASCII π banner + startup health checks (version, auth, models, gitignore) |
-| `footer` | UI | Status bar: mode / git branch / model / cost / tps |
-| `models` | command | `/models` — enhanced model picker with BenchLM rank, context, cost |
-| `update` | command | `/update` — self-update Pi + refresh extensions from dotfiles |
-| `lg` | command | `/lg` — summarize unstaged git changes with per-file +/- counts |
-| `yeet` | command | `/yeet` — stage all, commit, and push current changes |
-| `copy-all` | command | `/copy-all` — copy the whole conversation to the clipboard |
-| `diff` | command | `/diff` — list/open files changed during the last agent run |
-| `todo` | tool | durable execution checklist (survives compaction) |
-| `toolbox` | tool | `search` (fuzzy-find every tool / MCP tool / skill / command), `enable` / `disable` (turn a gated tool on/off by name) |
-| `nudges` | hooks | model-steering reminders (tools / skill / capability) |
+| Package | Description |
+|---|---|
+| `pix-welcome` | ASCII π banner + startup health checks (version, auth, models, gitignore) |
+| `pix-footer` | Status bar: mode / git branch / model / cost / live TPS |
+| `pix-models` | `/models` — enhanced model picker with BenchLM rank, context, cost |
+| `pix-update` | `/update` — self-update Pi + refresh extensions |
+| `pix-commands` | `/diff` and `/clear` slash commands |
+| `pix-diagnostics` | Compact LSP diagnostic widget |
+| `pix-prompts` | System-prompt injection (AGENTS.md + repo directive files) |
+| `pix-skills` | Agent skill loader (`read_skills` tool + 21 bundled skills) |
+| `pix-nudge` | Tool + capability nudge hooks |
 
 ## Install
 
 ```bash
-pi install git:github.com/xynogen/pix-core
+pi install npm:@xynogen/pix-core
 ```
 
-> **Requires** [pix-data](https://github.com/xynogen/pix-data) for shared models.dev + BenchLM cache.
-> Install both:
-> ```bash
-> pi install git:github.com/xynogen/pix-data
-> pi install git:github.com/xynogen/pix-core
-> ```
+> Includes the core pix UI/UX packages and installs their dependencies.
+
+## Full distro
+
+To install the complete pix suite (all packages + Pi itself):
+
+```bash
+curl -fsSL https://raw.githubusercontent.com/xynogen/pix-mono/main/scripts/install.sh | sh
+```
 
 ## License
 

package/package.json

@@ -1,26 +1,15 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.2.4",
-	"description": "Pi extension — core UI/UX bundle (welcome banner, footer, model picker, self-update)",
+	"version": "0.3.1",
+	"description": "Pi extension bundle — installs all core pix-* packages",
 	"type": "module",
-	"main": "src/index.ts",
 	"scripts": {
 		"test": "bun test"
 	},
 	"files": [
-		"src",
-		"skills",
 		"README.md",
 		"LICENSE"
 	],
-	"pi": {
-		"extensions": [
-			"src/index.ts"
-		],
-		"skills": [
-			"./skills"
-		]
-	},
 	"keywords": [
 		"pi",
 		"pi-package",
@@ -42,12 +31,17 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-data": "*",
+		"@xynogen/pix-welcome": "*",
+		"@xynogen/pix-footer": "*",
+		"@xynogen/pix-diagnostics": "*",
+		"@xynogen/pix-prompts": "*",
 		"@xynogen/pix-skills": "*",
-		"typebox": "^1.1.38"
+		"@xynogen/pix-models": "*",
+		"@xynogen/pix-update": "*",
+		"@xynogen/pix-commands": "*",
+		"@xynogen/pix-nudge": "*"
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*",
-		"@earendil-works/pi-tui": "*"
+		"@earendil-works/pi-coding-agent": "*"
 	}
 }

package/skills/ask-user/SKILL.md

@@ -1,48 +0,0 @@
----
-name: ask-user
-description: "MUST use before high-stakes/irreversible decisions or when requirements are ambiguous. Gather context, present 2-5 options via ask_user, get explicit choice, then proceed."
-metadata:
-  short-description: Decision gate for ambiguity and high-stakes choices
----
-
-# ask_user decision gate
-
-Decision control, not chit-chat.
-
-## Gate (call ask_user before proceeding if ANY true)
-- changes architecture/schema/API/deploy/security
-- costly to undo (big refactor, migration, destructive edit, prod behavior)
-- requirements unclear/conflicting/missing
-- multiple valid options, trade-off is preference-dependent
-- about to assume something that changes implementation
-
-Skip only if user already gave an explicit decision for THIS exact trade-off.
-
-## Handshake
-1. classify step: high_stakes | ambiguous | both | clear. clear → no gate.
-2. gather evidence first (read/bash/web/ref). don't ask blind.
-3. synthesize: 3-7 bullets — state, constraints, trade-offs, recommendation.
-4. ask ONE focused question: `question`, `context`(summary), `options`(2-5),
-   `allowMultiple:false` unless truly independent, `allowFreeform:true`.
-   `inline` displayMode when the preceding summary must stay visible.
-5. commit: restate decision, say next step, proceed.
-6. re-ask only on materially new ambiguity. no confirm loops.
-
-## Budget (anti-overasking)
-- max 1 call per boundary normally; max 2 if first is unclear/cancelled.
-- never re-ask same trade-off without new evidence.
-- attempt 2 = narrower question: [Proceed w/ recommended] [Choose other (freeform)] [Stop].
-- after attempt 2: high_stakes/both → STOP, mark blocked.
-  ambiguous-only + user says "your call" → take most reversible default, state assumptions.
-
-## Quality
-- question: concrete decision boundary, one decision only.
-- options: short, outcome-oriented, explicit trade-offs; add description when non-obvious.
-
-## Anti-patterns
-asking without context · trivial formatting choices · forcing options when freeform fits ·
-repeat questions w/o new info · proceeding high-stakes after unclear/cancelled answer.
-
-## On cancel / unclear
-Pause, explain what's blocked. At most one narrower follow-up.
-Then: high-stakes → stay blocked until explicit decision; ambiguity-only → proceed only if user delegated.

package/src/commands/agent-sop/agent-sop.ts

@@ -1,58 +0,0 @@
-/**
- * agent-sop — inject AGENT.md (from pix-skills) into system prompt
- *
- * Reads AGENT.md from the @xynogen/pix-skills package and appends it to the
- * system prompt on every agent start via `before_agent_start`.
- *
- * This is the "register skill" mechanism for the agent operating spec — no
- * static SKILL.md file needed. The content becomes part of the model's
- * standing instructions.
- */
-
-import { existsSync, readFileSync } from "node:fs";
-import { createRequire } from "node:module";
-import { resolve } from "node:path";
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-/** Resolve the absolute path to AGENT.md inside @xynogen/pix-skills. */
-function resolveAgentMdPath(): string | null {
-	try {
-		const require = createRequire(import.meta.url);
-		const pkgJson = require.resolve("@xynogen/pix-skills/package.json");
-		return resolve(pkgJson, "..", "AGENT.md");
-	} catch {
-		return null;
-	}
-}
-
-/** Read and return AGENT.md content, or null if unavailable. */
-function loadAgentMd(): string | null {
-	const p = resolveAgentMdPath();
-	if (!p || !existsSync(p)) return null;
-	try {
-		return readFileSync(p, "utf-8");
-	} catch {
-		return null;
-	}
-}
-
-export default function registerAgentSop(pi: ExtensionAPI): void {
-	// Load once at startup (content is static per session).
-	const agentMdContent = loadAgentMd();
-
-	if (!agentMdContent) {
-		// Silent skip — pix-skills might not be installed.
-		return;
-	}
-
-	pi.on("before_agent_start", async (event) => {
-		// Skip if already injected (idempotent check via a simple marker).
-		if (event.systemPrompt.includes("pix-agent-sop")) {
-			return;
-		}
-
-		return {
-			systemPrompt: `${event.systemPrompt}\n\n<pix-agent-sop>\n${agentMdContent}\n</pix-agent-sop>`,
-		};
-	});
-}

package/src/commands/clear/clear.ts

@@ -1,32 +0,0 @@
-import type {
-	ExtensionAPI,
-	ExtensionCommandContext,
-} from "@earendil-works/pi-coding-agent";
-
-async function clearCache(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
-	ctx.ui.notify("Clearing ~/.cache/pi", "info");
-	const result = await pi.exec("/bin/sh", ["-lc", 'rm -rf "$HOME/.cache/pi"'], {
-		timeout: 10_000,
-	});
-	const output = [result.stdout, result.stderr]
-		.filter(Boolean)
-		.join("\n")
-		.trim();
-	if ((result.code ?? 0) !== 0) {
-		ctx.ui.notify(`Cache clear failed. ${output || "No output."}`, "error");
-		return;
-	}
-	ctx.ui.notify(
-		"~/.cache/pi cleared. Run /reload to apply changes.",
-		"warning",
-	);
-}
-
-export default function (pi: ExtensionAPI) {
-	pi.registerCommand("clear", {
-		description: "Remove ~/.cache/pi and reload",
-		handler: async (_args, ctx) => {
-			await clearCache(pi, ctx);
-		},
-	});
-}

package/src/commands/diff/diff.ts

@@ -1,32 +0,0 @@
-/**
- * /diff — explain unstaged git diff with per-file +/- counts.
- *
- * The agent runs `git status` + `git diff`, then replies with:
- *   1. 1–2 sentence explanation of what changed
- *   2. Per-file +/- line counts
- *   3. Total +/- line count
- */
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-const DIFF_PROMPT = `Run git status and inspect the unstaged git diff, then respond with only:
-
-1. A short 1-2 sentence explanation of what changed and why it matters.
-2. A list of changed unstaged files with their +/- line counts.
-3. A total +/- line count at the bottom.
-
-Keep it concise. Use git commands to calculate the line counts. Base the summary on the actual diff, not only filenames. Do not include staged changes unless they also have unstaged modifications.`;
-
-export default function (pi: ExtensionAPI) {
-	pi.registerCommand("diff", {
-		description: "Explain unstaged git diff with per-file +/- counts",
-		handler: async (_args, ctx) => {
-			if (!ctx.isIdle()) {
-				pi.sendUserMessage(DIFF_PROMPT, { deliverAs: "followUp" });
-				ctx.ui.notify("Queued /diff after the current turn finishes.", "info");
-				return;
-			}
-			pi.sendUserMessage(DIFF_PROMPT);
-		},
-	});
-}

package/src/commands/models/models.test.ts

@@ -1,95 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { benchStars, fmtCost, fmtCtx, sortModels } from "./models.ts";
-
-describe("fmtCtx", () => {
-	it("formats 0 as 0", () => expect(fmtCtx(0)).toBe("0"));
-	it("formats small numbers as-is", () => expect(fmtCtx(512)).toBe("512"));
-	it("formats thousands as Nk", () => {
-		expect(fmtCtx(128_000)).toBe("128k");
-		expect(fmtCtx(8_192)).toBe("8k");
-	});
-	it("formats millions as NM", () => {
-		expect(fmtCtx(1_000_000)).toBe("1M");
-		expect(fmtCtx(2_000_000)).toBe("2M");
-		expect(fmtCtx(1_500_000)).toBe("1.5M");
-	});
-});
-
-describe("fmtCost", () => {
-	it("returns — for undefined entry", () =>
-		expect(fmtCost(undefined)).toBe("—"));
-	it("returns — when no cost field", () => expect(fmtCost({})).toBe("—"));
-	it("returns free when both 0", () => {
-		expect(fmtCost({ cost: { input: 0, output: 0 } })).toBe("free");
-	});
-	it("formats input/output costs", () => {
-		expect(fmtCost({ cost: { input: 3, output: 15 } })).toBe("3.00/15.00");
-	});
-	it("handles missing input/output as 0", () => {
-		expect(fmtCost({ cost: {} })).toBe("free");
-	});
-});
-
-describe("benchStars", () => {
-	it("gives 5 stars for score >= 90", () => {
-		expect(benchStars(95).filled).toBe(5);
-		expect(benchStars(90).filled).toBe(5);
-	});
-	it("gives 4 stars for 80-89", () => {
-		expect(benchStars(85).filled).toBe(4);
-		expect(benchStars(80).filled).toBe(4);
-	});
-	it("gives 3 stars for 70-79", () => {
-		expect(benchStars(75).filled).toBe(3);
-	});
-	it("gives 2 stars for 50-69", () => {
-		expect(benchStars(60).filled).toBe(2);
-		expect(benchStars(50).filled).toBe(2);
-	});
-	it("gives 1 star for score < 50", () => {
-		expect(benchStars(30).filled).toBe(1);
-		expect(benchStars(0).filled).toBe(1);
-	});
-	it("gives 1 star for null/undefined", () => {
-		expect(benchStars(null).filled).toBe(1);
-		expect(benchStars(undefined).filled).toBe(1);
-	});
-	it("filled + empty always = 5", () => {
-		for (const s of [0, 50, 70, 80, 90, 100]) {
-			const { filled, empty } = benchStars(s);
-			expect(filled + empty).toBe(5);
-		}
-	});
-});
-
-describe("sortModels", () => {
-	const models = [
-		{ provider: "a", id: "m1", name: "Zebra", score: 80 },
-		{ provider: "a", id: "m2", name: "Alpha", score: 95 },
-		{ provider: "a", id: "m3", name: "Middle", score: null },
-		{ provider: "a", id: "m4", name: "Beta", score: 80 },
-	];
-
-	it("sorts by score descending", () => {
-		const sorted = sortModels(models);
-		expect(sorted[0].name).toBe("Alpha"); // score 95
-	});
-
-	it("breaks score ties alphabetically by name", () => {
-		const sorted = sortModels(models);
-		const tiedIdx = sorted.findIndex((m) => m.name === "Beta");
-		const zebraIdx = sorted.findIndex((m) => m.name === "Zebra");
-		expect(tiedIdx).toBeLessThan(zebraIdx); // Beta before Zebra, both score 80
-	});
-
-	it("puts null score models last", () => {
-		const sorted = sortModels(models);
-		expect(sorted[sorted.length - 1].name).toBe("Middle");
-	});
-
-	it("does not mutate the original array", () => {
-		const original = [...models];
-		sortModels(models);
-		expect(models).toEqual(original);
-	});
-});