comisai 1.0.30 → 1.0.32

package/node_modules/@comis/cli/dist/wizard/steps/04-credentials.js

@@ -16,6 +16,7 @@
  * @module
  */
 import { updateState, sectionSeparator, info, validateApiKey, getKeyPrefix, } from "../index.js";
+import { getModels } from "@mariozechner/pi-ai";
 // ---------- Provider Help URLs ----------
 const PROVIDER_HELP_URLS = {
     anthropic: "https://console.anthropic.com/settings/keys",
@@ -62,18 +63,75 @@ const AUTH_METHOD_PROVIDERS = {
     },
 };
 // ---------- Provider Validation Endpoints ----------
-const PROVIDER_VALIDATION = {
-    anthropic: { baseUrl: "https://api.anthropic.com", path: "/v1/models" },
-    openai: { baseUrl: "https://api.openai.com", path: "/v1/models" },
-    google: { baseUrl: "https://generativelanguage.googleapis.com", path: "/v1/models" },
-    groq: { baseUrl: "https://api.groq.com", path: "/openai/v1/models" },
-    mistral: { baseUrl: "https://api.mistral.ai", path: "/v1/models" },
-    deepseek: { baseUrl: "https://api.deepseek.com", path: "/v1/models" },
-    xai: { baseUrl: "https://api.x.ai", path: "/v1/models" },
-    together: { baseUrl: "https://api.together.xyz", path: "/v1/models" },
-    cerebras: { baseUrl: "https://api.cerebras.ai", path: "/v1/models" },
-    openrouter: { baseUrl: "https://openrouter.ai", path: "/api/v1/models" },
+/**
+ * Path suffixes per provider, RELATIVE to the pi-ai catalog baseUrl.
+ *
+ * Pi-ai's catalog baseUrl shape is NOT uniform across providers:
+ *   - HOST-ONLY for anthropic ("https://api.anthropic.com"), mistral, deepseek
+ *     -> path here must include the version prefix ("/v1/models").
+ *   - PREFIXED with the version segment for openai ("https://api.openai.com/v1"),
+ *     google ("/v1beta"), groq ("/openai/v1"), xai ("/v1"), cerebras ("/v1"),
+ *     openrouter ("/api/v1") -> path here must NOT repeat the version segment;
+ *     append "/models" only.
+ *
+ * Composing entry.baseUrl + entry.path therefore produces the canonical /models
+ * endpoint for each provider (e.g., https://api.openai.com/v1/models,
+ * https://generativelanguage.googleapis.com/v1beta/models,
+ * https://api.groq.com/openai/v1/models).
+ *
+ * Follow-up to 260501-kqq Sub-Fix C: that migration replaced the static
+ * PROVIDER_VALIDATION map (which had host-only baseUrls + correct /v1/models
+ * suffixes) with the catalog-driven `getValidationEndpoint` helper, but the
+ * path-table values were copied verbatim -- producing double-prefixed URLs
+ * (e.g., https://api.openai.com/v1/v1/models -> 404) for the 6 providers
+ * whose catalog baseUrl includes the version segment. 260501-mvw corrects
+ * the table; the helper itself is unchanged.
+ *
+ * Drift risk: if pi-ai upgrades a provider's baseUrl AND its path convention
+ * changes, this table must be updated. Acceptable trade-off -- explicit
+ * beats clever (auto-detection of duplicated path segments could mask
+ * legitimate future shape changes).
+ *
+ * Excluded: `together` and `ollama` are NOT in pi-ai 0.71.0's catalog
+ * (`getModels(p)[0]?.baseUrl` returns undefined for both). The line-130
+ * fallback (`if (!entry) return { valid: true };`) handles them by
+ * skipping live validation entirely. For `together` this is a deliberate
+ * behavior change vs the pre-260501-kqq state -- live validation against
+ * api.together.xyz is now skipped. Users can still target Together via
+ * the synthetic `custom` endpoint route.
+ */
+const PROVIDER_VALIDATION_PATHS = {
+    // Catalog baseUrl is HOST-ONLY for these providers -> path needs the /v1 prefix.
+    anthropic: "/v1/models",
+    mistral: "/v1/models",
+    deepseek: "/v1/models",
+    // Catalog baseUrl ALREADY INCLUDES the version prefix for these providers
+    // (e.g., openai's baseUrl is "https://api.openai.com/v1", openrouter's is
+    // "https://openrouter.ai/api/v1") -- append /models only.
+    openai: "/models",
+    google: "/models",
+    groq: "/models",
+    xai: "/models",
+    cerebras: "/models",
+    openrouter: "/models",
 };
+/**
+ * Resolve the validation endpoint for a provider by reading the catalog
+ * baseUrl from pi-ai (260501-gyy precedent: builtin-provider-guard.ts:45)
+ * and combining it with a known path from PROVIDER_VALIDATION_PATHS.
+ *
+ * Returns `undefined` for providers not in the catalog (or providers
+ * with no models, e.g., ollama with no remote endpoint) -- callers
+ * skip live validation in that case.
+ */
+function getValidationEndpoint(provider) {
+    const baseUrl = getModels(provider)[0]?.baseUrl;
+    if (!baseUrl)
+        return undefined;
+    // eslint-disable-next-line security/detect-object-injection -- read of static const map indexed by validated provider string
+    const path = PROVIDER_VALIDATION_PATHS[provider] ?? "/v1/models";
+    return { baseUrl, path };
+}
 // ---------- Live Validation ----------
 /**
  * Validate an API key against the provider's /models endpoint.
@@ -91,7 +149,7 @@ async function validateKeyLive(provider, apiKey, authMethod) {
     if (authMethod === "oauth") {
         return { valid: true };
     }
-    const entry = PROVIDER_VALIDATION[provider];
+    const entry = getValidationEndpoint(provider);
     if (!entry) {
         return { valid: true };
     }

package/node_modules/@comis/cli/dist/wizard/steps/05-agent.js

@@ -11,20 +11,6 @@
  */
 import { updateState, sectionSeparator, validateAgentName, } from "../index.js";
 import { createModelCatalog } from "@comis/agent";
-// ---------- Recommended Models Per Provider ----------
-const RECOMMENDED_MODELS = {
-    anthropic: "claude-sonnet-4-5-20250929",
-    openai: "gpt-4o",
-    google: "gemini-2.0-flash",
-    groq: "llama-3.3-70b-versatile",
-    mistral: "mistral-large-latest",
-    deepseek: "deepseek-chat",
-    xai: "grok-2",
-    together: "meta-llama/Meta-Llama-3.1-70B-Instruct-Turbo",
-    cerebras: "llama-3.3-70b",
-    openrouter: "anthropic/claude-sonnet-4-5-20250929",
-    ollama: "llama3",
-};
 // ---------- Helpers ----------
 function formatModelHint(m) {
     const parts = [];
@@ -67,8 +53,6 @@ export const agentStep = {
         catch {
             // Fallback: empty catalog, will use recommended default only
         }
-        const providerId = state.provider?.id ?? "";
-        const recommended = RECOMMENDED_MODELS[providerId];
         // 4. QuickStart flow -- use "default" so the daemon picks the model at runtime
         if (state.flow === "quickstart") {
             prompter.log.info("Model: default");
@@ -87,11 +71,13 @@ export const agentStep = {
                 label: "Custom model ID...",
                 hint: "Enter a model ID manually",
             });
+            // Initial value: state.model takes precedence; otherwise the
+            // first catalog entry. No hardcoded recommendation lookup --
+            // pi-ai's catalog ordering is the source of truth.
             const chosen = await prompter.select({
                 message: "Select a model",
                 options,
-                initialValue: state.model
-                    ?? (recommended && catalogModels.some((m) => m.modelId === recommended) ? recommended : undefined),
+                initialValue: state.model ?? catalogModels[0]?.modelId,
             });
             if (chosen === "__custom__") {
                 selectedModel = await prompter.text({
@@ -107,8 +93,8 @@ export const agentStep = {
             // Empty catalog -- go directly to custom text input
             selectedModel = await prompter.text({
                 message: "Model ID",
-                placeholder: recommended ?? "model-name",
-                defaultValue: state.model ?? recommended,
+                placeholder: state.model ?? "model-name",
+                defaultValue: state.model,
             });
         }
         return updateState(state, { agentName, model: selectedModel });

package/node_modules/@comis/cli/dist/wizard/types.d.ts

@@ -134,20 +134,6 @@ export type WizardResult = {
     envPath?: string;
     error?: WizardError;
 };
-/** Supported provider entry for selection prompts. */
-export type SupportedProvider = {
-    id: string;
-    label: string;
-    hint?: string;
-    category: "recommended" | "other" | "local" | "custom";
-};
-/**
- * All supported LLM providers, grouped by category.
- *
- * Categories: recommended (top picks), other (cloud APIs),
- * local (self-hosted), custom (user-defined endpoints).
- */
-export declare const SUPPORTED_PROVIDERS: readonly SupportedProvider[];
 /** Supported channel entry for selection prompts. */
 export type SupportedChannel = {
     type: ChannelConfig["type"];

package/node_modules/@comis/cli/dist/wizard/types.js

@@ -9,30 +9,6 @@
  */
 /** Starting state for a new wizard run. */
 export const INITIAL_STATE = { completedSteps: [] };
-/**
- * All supported LLM providers, grouped by category.
- *
- * Categories: recommended (top picks), other (cloud APIs),
- * local (self-hosted), custom (user-defined endpoints).
- */
-export const SUPPORTED_PROVIDERS = [
-    // Recommended
-    { id: "anthropic", label: "Anthropic (Claude)", hint: "Recommended for agents", category: "recommended" },
-    { id: "openai", label: "OpenAI (GPT)", hint: "GPT-4o, o1, o3 models", category: "recommended" },
-    // Other Providers
-    { id: "google", label: "Google (Gemini)", hint: "Gemini models", category: "other" },
-    { id: "groq", label: "Groq", hint: "Fast inference (Llama, Mixtral)", category: "other" },
-    { id: "mistral", label: "Mistral", hint: "Mistral models", category: "other" },
-    { id: "deepseek", label: "DeepSeek", hint: "DeepSeek models", category: "other" },
-    { id: "xai", label: "xAI (Grok)", hint: "Grok models", category: "other" },
-    { id: "together", label: "Together AI", hint: "Open-source model hosting", category: "other" },
-    { id: "cerebras", label: "Cerebras", hint: "Fast inference", category: "other" },
-    { id: "openrouter", label: "OpenRouter", hint: "Multi-provider routing", category: "other" },
-    // Local
-    { id: "ollama", label: "Ollama (local)", hint: "No API key needed", category: "local" },
-    // Custom
-    { id: "custom", label: "Custom endpoint", hint: "OpenAI-compatible API", category: "custom" },
-];
 /** All supported chat channels with credential guidance. */
 export const SUPPORTED_CHANNELS = [
     { type: "telegram", label: "Telegram", credentialHint: "Bot token from @BotFather" },

package/node_modules/@comis/cli/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/cli",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Command-line interface for the Comis AI agent platform",

package/node_modules/@comis/core/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/core",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Core domain types, ports, event bus, security, and config for Comis",

package/node_modules/@comis/daemon/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/daemon",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Background daemon and orchestrator for the Comis platform",

package/node_modules/@comis/gateway/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/gateway",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "HTTP, JSON-RPC, and WebSocket gateway for Comis",

package/node_modules/@comis/infra/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/infra",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Structured logging infrastructure for Comis",

package/node_modules/@comis/memory/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/memory",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "SQLite memory, embeddings, and RAG storage for Comis agents",

package/node_modules/@comis/scheduler/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/scheduler",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Task scheduling and cron management for Comis",

package/node_modules/@comis/shared/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/shared",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Shared types and utilities for the Comis platform",

package/node_modules/@comis/skills/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comis/skills",
   "private": true,
-  "version": "1.0.30",
+  "version": "1.0.32",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",
   "description": "Skill system, MCP integration, and tool sandbox for Comis agents",

package/node_modules/@comis/web/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comis/web",
-  "version": "1.0.30",
+  "version": "1.0.32",
   "description": "Web dashboard SPA for Comis agent management",
   "author": "Moshe Anconina",
   "license": "Apache-2.0",