@pi-unipi/web-api 0.1.4 → 0.1.6

package/README.md

@@ -128,6 +128,9 @@ web_llm_summarize(url: "https://example.com/research", prompt: "Extract key find
 
 Interactive settings dialog for managing providers and API keys.
 
+- **Auto-enable on key input** — provider is automatically enabled when you add a valid API key (no extra toggle step)
+- **Cursor memory** — last configured provider moves to the top of the list when you return to the menu
+
 ### /unipi:web-cache-clear
 
 Clear all cached web content.
@@ -145,8 +148,8 @@ Clear all cached web content.
 If you see "No search provider available":
 
 1. Run `/unipi:web-settings`
-2. Enable at least one provider
-3. Add API keys for paid providers
+2. Add API keys for paid providers (they auto-enable on key input)
+3. Or manually enable a free provider
 
 ### API key invalid
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/web-api",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "Web search, read, and summarize tools with provider-based backend selection for Pi coding agent",
   "type": "module",
   "license": "MIT",

package/src/commands.ts

@@ -4,7 +4,7 @@
  * Registers /unipi:web-settings and /unipi:web-cache-clear commands.
  */
 
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI, ExtensionCommandContext } from "@mariozechner/pi-coding-agent";
 import { UNIPI_PREFIX } from "@pi-unipi/core";
 import { showSettingsDialog } from "./tui/settings-dialog.js";
 import { webCache } from "./cache.js";
@@ -20,26 +20,24 @@ export const WEB_COMMANDS = {
  */
 export function registerWebCommands(pi: ExtensionAPI): void {
   // --- /unipi:web-settings command ---
-  pi.registerCommand({
-    name: `${UNIPI_PREFIX}${WEB_COMMANDS.SETTINGS}`,
+  pi.registerCommand(`${UNIPI_PREFIX}${WEB_COMMANDS.SETTINGS}`, {
     description: "Configure web API providers and API keys",
-    handler: async (_args, _ctx) => {
-      await showSettingsDialog(pi);
+    handler: async (_args, ctx) => {
+      await showSettingsDialog(ctx);
     },
   });
 
   // --- /unipi:web-cache-clear command ---
-  pi.registerCommand({
-    name: `${UNIPI_PREFIX}${WEB_COMMANDS.CACHE_CLEAR}`,
+  pi.registerCommand(`${UNIPI_PREFIX}${WEB_COMMANDS.CACHE_CLEAR}`, {
     description: "Clear all cached web content",
-    handler: async (_args, _ctx) => {
+    handler: async (_args, ctx) => {
       const stats = webCache.getStats();
       const cleared = webCache.clear();
 
-      await pi.ui.notify({
-        message: `Cache cleared: ${cleared} entries removed (${stats.totalSizeBytes} bytes freed)`,
-        level: "success",
-      });
+      ctx.ui.notify(
+        `Cache cleared: ${cleared} entries removed (${stats.totalSizeBytes} bytes freed)`,
+        "info",
+      );
     },
   });
 }

package/src/tui/settings-dialog.ts

@@ -5,7 +5,7 @@
  * Uses pi's TUI components for provider selection and key input.
  */
 
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionCommandContext } from "@mariozechner/pi-coding-agent";
 import { registry } from "../providers/registry.js";
 import {
   getApiKey,
@@ -21,8 +21,9 @@ import { getProviderOptions, getProviderStatuses } from "./provider-selector.js"
  * Show settings dialog.
  * This is the main entry point for /unipi:web-settings command.
  */
-export async function showSettingsDialog(pi: ExtensionAPI): Promise<void> {
+export async function showSettingsDialog(ctx: ExtensionCommandContext): Promise<void> {
   let running = true;
+  let lastSelected: string | undefined;
 
   while (running) {
     const options = getProviderOptions();
@@ -34,20 +35,37 @@ export async function showSettingsDialog(pi: ExtensionAPI): Promise<void> {
       description: "Exit settings",
     });
 
+    // Move last selected provider to top of list for quick re-entry
+    if (lastSelected && lastSelected !== "__exit__") {
+      const idx = options.findIndex(o => o.value === lastSelected);
+      if (idx > 0) {
+        const [item] = options.splice(idx, 1);
+        options.unshift(item);
+      }
+    }
+
     // Show provider list
-    const selected = await pi.ui.select({
-      title: "Web API Settings",
-      message: "Select a provider to configure:",
-      options,
+    const labels = options.map(o => o.value === "__exit__" ? o.label : `${o.label} — ${o.description}`);
+    const selected = await ctx.ui.select(
+      "Web API Settings",
+      labels,
+    );
+    // Map label back to value
+    const selectedOpt = options.find(o => {
+      const full = o.value === "__exit__" ? o.label : `${o.label} — ${o.description}`;
+      return full === selected;
     });
+    const selectedValue = selectedOpt?.value;
 
-    if (!selected || selected === "__exit__") {
+    if (!selectedValue || selectedValue === "__exit__") {
       running = false;
       continue;
     }
 
+    lastSelected = selectedValue;
+
     // Show provider configuration
-    await configureProvider(pi, selected);
+    await configureProvider(ctx, selectedValue);
   }
 }
 
@@ -55,15 +73,15 @@ export async function showSettingsDialog(pi: ExtensionAPI): Promise<void> {
  * Configure a specific provider.
  */
 async function configureProvider(
-  pi: ExtensionAPI,
+  ctx: ExtensionCommandContext,
   providerId: string
 ): Promise<void> {
   const provider = registry.getProvider(providerId);
   if (!provider) {
-    await pi.ui.notify({
-      message: `Provider "${providerId}" not found`,
-      level: "error",
-    });
+    ctx.ui.notify(
+      `Provider "${providerId}" not found`,
+      "error",
+    );
     return;
   }
 
@@ -110,32 +128,34 @@ async function configureProvider(
     description: "Return to provider list",
   });
 
-  const selected = await pi.ui.select({
-    title: `Configure ${provider.name}`,
-    message: `Capabilities: ${provider.capabilities.join(", ")}`,
-    options,
-  });
+  const labels = options.map(o => `${o.label} — ${o.description}`);
+  const selected = await ctx.ui.select(
+    `Configure ${provider.name} (${provider.capabilities.join(", ")})`,
+    labels,
+  );
+  const selectedOpt = options.find(o => `${o.label} — ${o.description}` === selected);
+  const selectedValue = selectedOpt?.value;
 
-  switch (selected) {
+  switch (selectedValue) {
     case "__toggle__":
       setProviderEnabled(providerId, !enabled);
-      await pi.ui.notify({
-        message: `${provider.name} ${!enabled ? "enabled" : "disabled"}`,
-        level: "success",
-      });
+      ctx.ui.notify(
+        `${provider.name} ${!enabled ? "enabled" : "disabled"}`,
+        "info",
+      );
       break;
 
     case "__add_key__":
     case "__update_key__":
-      await inputApiKey(pi, providerId, provider.name);
+      await inputApiKey(ctx, providerId, provider.name);
       break;
 
     case "__remove_key__":
       removeApiKey(providerId);
-      await pi.ui.notify({
-        message: `API key removed for ${provider.name}`,
-        level: "success",
-      });
+      ctx.ui.notify(
+        `API key removed for ${provider.name}`,
+        "info",
+      );
       break;
 
     case "__back__":
@@ -148,30 +168,25 @@ async function configureProvider(
  * Input API key for a provider.
  */
 async function inputApiKey(
-  pi: ExtensionAPI,
+  ctx: ExtensionCommandContext,
   providerId: string,
   providerName: string
 ): Promise<void> {
-  const apiKey = await pi.ui.input({
-    title: `API Key for ${providerName}`,
-    message: `Enter API key (env: ${registry.getProvider(providerId)?.apiKeyEnv || "N/A"}):`,
-    placeholder: "sk-...",
-    validate: async (value: string) => {
-      if (!value || value.trim().length === 0) {
-        return "API key cannot be empty";
-      }
-      if (!validateApiKeyFormat(providerId, value)) {
-        return "API key format looks invalid";
-      }
-      return null;
-    },
-  });
+  const envHint = registry.getProvider(providerId)?.apiKeyEnv || "N/A";
+  const apiKey = await ctx.ui.input(
+    `API Key for ${providerName} (env: ${envHint})`,
+    "sk-...",
+  );
 
   if (apiKey) {
     setApiKey(providerId, apiKey);
-    await pi.ui.notify({
-      message: `API key saved for ${providerName}`,
-      level: "success",
-    });
+    // Auto-enable provider on successful key input
+    if (!isProviderEnabled(providerId)) {
+      setProviderEnabled(providerId, true);
+    }
+    ctx.ui.notify(
+      `API key saved for ${providerName} — enabled`,
+      "info",
+    );
   }
 }