@xynogen/pix-core 0.1.3 → 0.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.1.3",
+	"version": "0.1.5",
 	"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) {