pi-spark 0.9.2 → 0.9.3

package/extensions/models/index.ts

@@ -54,8 +54,8 @@ export default function (pi: ExtensionAPI) {
       parameters: Type.Object({
         action: StringEnum(["list", "current"] as const, {
           description:
-            "\"list\" returns all currently usable models with metadata; " +
-            "\"current\" returns the active model with metadata and thinking level.",
+            "\"current\" returns the active model with metadata and thinking level; " +
+            "\"list\" returns all currently usable models with metadata.",
         }),
         provider: Type.Optional(Type.String({
           description: "For list: filter by provider, case-insensitive substring (e.g., \"vercel\", \"moonshot\")",
@@ -70,7 +70,7 @@ export default function (pi: ExtensionAPI) {
           description: "For list: maximum number of models to return"
         })),
       }),
-      renderCall(args, theme, { expanded }) {
+      renderCall(args, theme) {
         let text = `${theme.bold(theme.fg("toolTitle", "model"))} ${theme.fg("accent", args.action)}`;
 
         if (args.provider) text += theme.fg("muted", ` provider:${args.provider}`);
@@ -78,8 +78,6 @@ export default function (pi: ExtensionAPI) {
         if (args.offset !== undefined || args.limit !== undefined) text += theme.fg("warning", ` from:${args.offset ?? 1}`);
         if (args.limit !== undefined) text += theme.fg("warning", ` to:${(args.offset ?? 1) + args.limit - 1}`);
 
-        if (!expanded) text += theme.fg("dim", ` (${keyText("app.tools.expand")} to expand)`);
-
         return new Text(text, 0, 0);
       },
       renderResult(result, { expanded }, theme, context) {
@@ -109,7 +107,7 @@ export default function (pi: ExtensionAPI) {
         const models = details.models ?? [];
         const total = details.total ?? models.length;
 
-        if (expanded) {
+        if (models.length > 0 && expanded) {
           models.forEach((model) => {
             container.addChild(new Text(theme.fg("muted", formatModel(model.provider, model.id)), 0, 0));
           });
@@ -117,15 +115,17 @@ export default function (pi: ExtensionAPI) {
           container.addChild(new Spacer(1));
         }
 
-        const summary = `${models.length !== total ? `${models.length} of ` : ""}${total} ${filtered ? "matched" : "available"} model${total === 1 ? "" : "s"} listed`;
-        container.addChild(new Text(theme.fg("muted", summary) + (details.truncation?.truncated ? theme.fg("warning", " (truncated)") : ""), 0, 0));
+        const summary = total > 0 ? `${models.length !== total ? `${models.length} of ` : ""}${total} ${filtered ? "matched" : "available"} model${total === 1 ? "" : "s"} listed` : `0 models ${filtered ? "matched" : "available"}`;
+        const truncatedHint = details.truncation?.truncated ? theme.fg("warning", " (truncated)") : "";
+        const expandHint = models.length > 0 && !expanded ? theme.fg("dim", ` (${keyText("app.tools.expand")} to expand)`) : "";
+        container.addChild(new Text(theme.fg("muted", summary) + truncatedHint + expandHint, 0, 0));
 
         return container;
       },
       async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
         if (params.action === "current") {
           const model = ctx.model;
-          if (!model) throw new Error("No model is currently selected.");
+          if (!model) throw new Error("No model is currently selected");
 
           const thinkingLevel = pi.getThinkingLevel();
           const current = { model: toMetadata(model), thinkingLevel };
@@ -149,7 +149,7 @@ export default function (pi: ExtensionAPI) {
 
         if (total === 0) {
           return {
-            content: [{ type: "text", text: `0 models ${filtered ? "matched" : "available"}.` }],
+            content: [{ type: "text", text: `0 models ${filtered ? "matched" : "available"}` }],
             details: { action: "list", models: [], total } satisfies ModelToolDetails,
           };
         }
@@ -157,7 +157,7 @@ export default function (pi: ExtensionAPI) {
         // Convert from 1-indexed offset to 0-indexed array access.
         const startIndex = params.offset ? Math.max(0, params.offset - 1) : 0;
         if (startIndex >= total) {
-          throw new Error(`Offset ${params.offset} is beyond end of list (${total} models total).`);
+          throw new Error(`Offset ${params.offset} is beyond end of list (${total} models total)`);
         }
 
         const endIndex = params.limit !== undefined ? Math.min(startIndex + params.limit, total) : total;

package/extensions/name/index.ts

@@ -72,12 +72,12 @@ export default function (pi: ExtensionAPI) {
       },
       async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
         const name = sanitizeText(params.name);
-        if (!name) throw new Error("Session name was empty after normalization. Provide a short, non-empty phrase.");
+        if (!name) throw new Error("Session name was empty after normalization; provide a short, non-empty phrase");
 
         const previous = pi.getSessionName() ?? null;
         if (previous === name) {
           return {
-            content: [{ type: "text", text: `Session is already named "${name}". No change.` }],
+            content: [{ type: "text", text: `Session is already named "${name}"; no change` }],
             details: { changed: false, previous },
           };
         }
@@ -85,7 +85,7 @@ export default function (pi: ExtensionAPI) {
         pi.setSessionName(name);
 
         return {
-          content: [{ type: "text", text: `${previous ? `Renamed session from "${previous}" to "${name}".` : `Named session "${name}".`}` }],
+          content: [{ type: "text", text: `${previous ? `Renamed session from "${previous}" to "${name}"` : `Named session "${name}"`}` }],
           details: { changed: true, previous },
         };
       },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-spark",
-  "version": "0.9.2",
+  "version": "0.9.3",
   "description": "A small, opinionated collection of pi extensions",
   "keywords": [
     "pi-coding-agent",