@corbat-tech/coco 2.33.1 → 2.33.2

package/dist/cli/index.js

@@ -4220,6 +4220,10 @@ function buildMaxTokensParam(model, maxTokens) {
   }
   return { max_tokens: maxTokens };
 }
+function truncateToolDescription(description) {
+  if (description.length <= MAX_TOOL_DESCRIPTION_LENGTH) return description;
+  return description.slice(0, MAX_TOOL_DESCRIPTION_LENGTH - 1) + "\u2026";
+}
 function createOpenAIProvider(config) {
   const provider = new OpenAIProvider();
   if (config) {
@@ -4242,7 +4246,7 @@ function createKimiProvider(config) {
   }
   return provider;
 }
-var DEFAULT_MODEL2, CONTEXT_WINDOWS2, MODELS_WITHOUT_TEMPERATURE, LOCAL_MODEL_PATTERNS, MODELS_WITH_THINKING_MODE, OpenAIProvider;
+var DEFAULT_MODEL2, CONTEXT_WINDOWS2, MODELS_WITHOUT_TEMPERATURE, LOCAL_MODEL_PATTERNS, MODELS_WITH_THINKING_MODE, OpenAIProvider, MAX_TOOL_DESCRIPTION_LENGTH;
 var init_openai = __esm({
   "src/providers/openai.ts"() {
     init_errors();
@@ -4400,6 +4404,14 @@ var init_openai = __esm({
       supportsTemperature(model) {
         return !MODELS_WITHOUT_TEMPERATURE.some((m) => model.toLowerCase().includes(m.toLowerCase()));
       }
+      /**
+       * Whether this provider instance supports the Responses API for the given model.
+       * Subclasses (e.g. CopilotProvider) can override to force Chat Completions
+       * when their endpoint does not expose /v1/responses.
+       */
+      modelNeedsResponsesApi(model) {
+        return needsResponsesApi(model);
+      }
       /**
        * Get extra body parameters for API calls.
        * Honors the user's ThinkingMode for Kimi models; defaults to disabled
@@ -4419,7 +4431,7 @@ var init_openai = __esm({
       async chat(messages, options) {
         this.ensureInitialized();
         const model = options?.model ?? this.config.model ?? DEFAULT_MODEL2;
-        if (needsResponsesApi(model)) {
+        if (this.modelNeedsResponsesApi(model)) {
           return this.chatViaResponses(messages, options);
         }
         return withRetry(async () => {
@@ -4459,7 +4471,7 @@ var init_openai = __esm({
       async chatWithTools(messages, options) {
         this.ensureInitialized();
         const model = options?.model ?? this.config.model ?? DEFAULT_MODEL2;
-        if (needsResponsesApi(model)) {
+        if (this.modelNeedsResponsesApi(model)) {
           return this.chatWithToolsViaResponses(messages, options);
         }
         return withRetry(async () => {
@@ -4511,7 +4523,7 @@ var init_openai = __esm({
       async *stream(messages, options) {
         this.ensureInitialized();
         const model = options?.model ?? this.config.model ?? DEFAULT_MODEL2;
-        if (needsResponsesApi(model)) {
+        if (this.modelNeedsResponsesApi(model)) {
           yield* this.streamViaResponses(messages, options);
           return;
         }
@@ -4549,7 +4561,7 @@ var init_openai = __esm({
       async *streamWithTools(messages, options) {
         this.ensureInitialized();
         const model = options?.model ?? this.config.model ?? DEFAULT_MODEL2;
-        if (needsResponsesApi(model)) {
+        if (this.modelNeedsResponsesApi(model)) {
           yield* this.streamWithToolsViaResponses(messages, options);
           return;
         }
@@ -4801,7 +4813,7 @@ var init_openai = __esm({
         } catch {
           try {
             const model = this.config.model || DEFAULT_MODEL2;
-            if (needsResponsesApi(model)) {
+            if (this.modelNeedsResponsesApi(model)) {
               await this.client.responses.create({
                 model,
                 input: [{ role: "user", content: [{ type: "input_text", text: "Hi" }] }],
@@ -4942,7 +4954,7 @@ var init_openai = __esm({
           type: "function",
           function: {
             name: tool.name,
-            description: tool.description,
+            description: truncateToolDescription(tool.description),
             parameters: tool.input_schema
           }
         }));
@@ -5428,12 +5440,13 @@ var init_openai = __esm({
         return tools.map((tool) => ({
           type: "function",
           name: tool.name,
-          description: tool.description ?? void 0,
+          description: tool.description ? truncateToolDescription(tool.description) : void 0,
           parameters: tool.input_schema ?? null,
           strict: false
         }));
       }
     };
+    MAX_TOOL_DESCRIPTION_LENGTH = 1024;
   }
 });
 
@@ -6210,6 +6223,14 @@ var init_copilot2 = __esm({
       /**
        * Count tokens (approximate — Copilot models vary in tokenizer)
        */
+      /**
+       * The GitHub Copilot endpoint (api.githubcopilot.com) does not expose the
+       * OpenAI Responses API (/v1/responses). Always use Chat Completions so that
+       * gpt-5-mini and similar models work correctly with MCP tools.
+       */
+      modelNeedsResponsesApi(_model) {
+        return false;
+      }
       countTokens(text15) {
         if (!text15) return 0;
         return Math.ceil(text15.length / 3.5);
@@ -10786,7 +10807,7 @@ async function createDefaultReplConfig() {
   const model = await getLastUsedModel(providerType) || getDefaultModel(providerType);
   const persistedThinking = await getLastUsedThinking(providerType);
   const thinking = persistedThinking ?? resolveDefaultThinking(providerType, model);
-  const thinkingToStore = thinking === "off" ? void 0 : thinking;
+  const thinkingToStore = persistedThinking !== void 0 ? persistedThinking : thinking === "off" ? void 0 : thinking;
   return {
     provider: {
       type: providerType,
@@ -34319,6 +34340,12 @@ async function renderStartupPanel(session, gitCtx, mcpServers = []) {
   }
   const cocoStatus = isQualityLoop() ? chalk.magenta("  \u{1F504} quality mode: ") + chalk.green.bold("on") + chalk.dim(" \u2014 iterates until quality \u2265 85. /quality to disable") : chalk.dim("  \u{1F4A1} quality mode is Coco's edge for robust code. Enable with /quality on");
   console.log(cocoStatus);
+  if (thinkingCapability.supported) {
+    const modeLabel = formatThinkingMode(session.config.provider.thinking ?? "off");
+    console.log(
+      chalk.dim("  \u{1F9E0} reasoning: ") + chalk.magenta(modeLabel) + chalk.dim("  \xB7  /thinking to change")
+    );
+  }
   const skillTotal = session.skillRegistry?.size ?? 0;
   const hasSomething = skillTotal > 0 || mcpServers.length > 0;
   if (hasSomething) {
@@ -35928,6 +35955,7 @@ async function ensureConfiguredV2(config) {
   const preferredHasOpenAIOAuth = preferredProviderDef?.id === "openai" && hasOpenAIOAuthTokens;
   const preferredHasCopilotCreds = preferredProviderDef?.id === "copilot" && isProviderConfigured();
   const preferredIsConfigured = preferredIsLocal || preferredHasApiKey || preferredHasOpenAIOAuth || preferredHasCopilotCreds;
+  const preferredWasConfigured = Boolean(preferredProviderDef && preferredIsConfigured);
   let preferredWasConfiguredButUnavailable = false;
   let preferredUnavailableWasLocal = false;
   if (preferredProviderDef && preferredIsConfigured) {
@@ -35973,7 +36001,9 @@ async function ensureConfiguredV2(config) {
         }
         const provider = await createProvider(providerId, { model });
         if (await provider.isAvailable()) {
-          await saveProviderPreference(prov.id, model);
+          if (!preferredWasConfigured) {
+            await saveProviderPreference(prov.id, model);
+          }
           return {
             ...config,
             provider: {
@@ -35998,7 +36028,9 @@ async function ensureConfiguredV2(config) {
         const model = recommended?.id || openaiDef.models[0]?.id || "";
         const provider = await createProvider("codex", { model });
         if (await provider.isAvailable()) {
-          await saveProviderPreference("openai", model);
+          if (!preferredWasConfigured) {
+            await saveProviderPreference("openai", model);
+          }
           return {
             ...config,
             provider: {