fixing MLX native support, the option wasn't showing for mac silicon chips, removed secret requirement for feedback.

Author: igorcosta

src/config.ts

@@ -17,6 +17,7 @@ const DEFAULT_BASE_URL = 'https://openrouter.ai/api/v1';
 const DEFAULT_OLLAMA_URL = 'http://localhost:11434';
 const DEFAULT_LLAMACPP_URL = 'http://localhost:8080';
 const DEFAULT_OPENAI_URL = 'https://api.openai.com/v1';
+const DEFAULT_MLX_URL = 'http://localhost:8080';
 
 interface LegacyConfigShape {
   api_key?: string;
@@ -176,7 +177,8 @@ function isModernConfig(config: AutohandConfig | LegacyConfigShape): config is A
   return typeof (config as AutohandConfig).openrouter === 'object' ||
     typeof (config as AutohandConfig).ollama === 'object' ||
     typeof (config as AutohandConfig).llamacpp === 'object' ||
-    typeof (config as AutohandConfig).openai === 'object';
+    typeof (config as AutohandConfig).openai === 'object' ||
+    typeof (config as AutohandConfig).mlx === 'object';
 }
 
 function isLegacyConfig(config: AutohandConfig | LegacyConfigShape): config is LegacyConfigShape {
@@ -241,7 +243,8 @@ export function getProviderConfig(config: AutohandConfig, provider?: ProviderNam
     openrouter: config.openrouter,
     ollama: config.ollama,
     llamacpp: config.llamacpp,
-    openai: config.openai
+    openai: config.openai,
+    mlx: config.mlx
   };
 
   const entry = configByProvider[chosen];
@@ -279,6 +282,8 @@ function defaultBaseUrlFor(provider: ProviderName, port?: number): string | unde
       return p ? `http://localhost:${p}` : DEFAULT_LLAMACPP_URL;
     case 'openai':
       return DEFAULT_OPENAI_URL;
+    case 'mlx':
+      return p ? `http://localhost:${p}` : DEFAULT_MLX_URL;
     default:
       return undefined;
   }

src/core/agent.ts

@@ -15,6 +15,7 @@ import { FileActionManager } from '../actions/filesystem.js';
 import { saveConfig, getProviderConfig } from '../config.js';
 import type { LLMProvider } from '../providers/LLMProvider.js';
 import { ProviderFactory, ProviderNotConfiguredError } from '../providers/ProviderFactory.js';
+import { isMLXSupported } from '../utils/platform.js';
 import { readInstruction } from '../ui/inputPrompt.js';
 // showFilePalette is imported dynamically to avoid bundling ink in standalone binary
 import {
@@ -238,7 +239,10 @@ export class AutohandAgent {
       }
     });
     this.errorLogger = new ErrorLogger(packageJson.version);
-    this.feedbackManager = new FeedbackManager();
+    this.feedbackManager = new FeedbackManager({
+      apiBaseUrl: runtime.config.api?.baseUrl || 'https://api.autohand.ai',
+      cliVersion: packageJson.version
+    });
     this.skillsRegistry = new SkillsRegistry(AUTOHAND_PATHS.skills);
     this.telemetryManager = new TelemetryManager({
       enabled: runtime.config.telemetry?.enabled === true,
@@ -1018,14 +1022,17 @@ If lint or tests fail, report the issues but do NOT commit.`;
   private async promptModelSelection(): Promise<void> {
     try {
       // Show all providers with status indicators
-      const allProviders: ProviderName[] = ['openrouter', 'ollama', 'llamacpp', 'openai'];
+      // Use ProviderFactory to get platform-aware list (includes MLX on Apple Silicon)
+      const allProviders = ProviderFactory.getProviderNames();
       const providerChoices = allProviders.map(name => {
         const isConfigured = this.isProviderConfigured(name);
         const indicator = isConfigured ? chalk.green('●') : chalk.red('○');
         const current = name === this.activeProvider ? chalk.cyan(' (current)') : '';
+        // Add Apple Silicon indicator for MLX
+        const siliconNote = name === 'mlx' ? chalk.gray(' (Apple Silicon)') : '';
         return {
           name,
-          message: `${indicator} ${name}${current}`,
+          message: `${indicator} ${name}${current}${siliconNote}`,
           value: name
         };
       });
@@ -1088,6 +1095,9 @@ If lint or tests fail, report the issues but do NOT commit.`;
       case 'openai':
         await this.configureOpenAI();
         break;
+      case 'mlx':
+        await this.configureMLX();
+        break;
     }
   }
 
@@ -1281,6 +1291,67 @@ If lint or tests fail, report the issues but do NOT commit.`;
     }
   }
 
+  private async configureMLX(): Promise<void> {
+    try {
+      console.log(chalk.cyan('MLX Configuration (Apple Silicon)'));
+      console.log(chalk.gray('MLX provides local LLM inference optimized for Apple Silicon.'));
+      console.log(chalk.gray('Make sure mlx-lm server is running: mlx_lm.server --model <model-name>\n'));
+
+      // Try to fetch available models from MLX server
+      const mlxUrl = 'http://localhost:8080';
+      let availableModels: string[] = [];
+
+      try {
+        const response = await fetch(`${mlxUrl}/v1/models`);
+        if (response.ok) {
+          const data = await response.json();
+          availableModels = data.data?.map((m: any) => m.id) || [];
+        }
+      } catch {
+        console.log(chalk.yellow('⚠ Could not connect to MLX server. Make sure it\'s running.\n'));
+      }
+
+      let modelAnswer;
+      if (availableModels.length > 0) {
+        modelAnswer = await enquirer.prompt<{ model: string }>([
+          {
+            type: 'select',
+            name: 'model',
+            message: 'Select a model',
+            choices: availableModels
+          }
+        ]);
+      } else {
+        modelAnswer = await enquirer.prompt<{ model: string }>([
+          {
+            type: 'input',
+            name: 'model',
+            message: 'Enter model name (e.g., mlx-community/Llama-3.2-3B-Instruct-4bit)',
+            initial: 'mlx-community/Llama-3.2-3B-Instruct-4bit'
+          }
+        ]);
+      }
+
+      this.runtime.config.mlx = {
+        baseUrl: mlxUrl,
+        model: modelAnswer.model
+      };
+
+      this.runtime.config.provider = 'mlx';
+      this.runtime.options.model = modelAnswer.model;
+      await saveConfig(this.runtime.config);
+      this.resetLlmClient('mlx', modelAnswer.model);
+
+      console.log(chalk.green('\n✓ MLX configured successfully!'));
+    } catch (error) {
+      if ((error as any).name === 'ExitPromptError' || (error as Error).message?.includes('canceled')) {
+        console.log(chalk.gray('\nConfiguration cancelled.'));
+        return;
+      }
+      throw error;
+    }
+  }
+
   private async changeProviderModel(provider: ProviderName): Promise<void> {
     try {
       const currentSettings = getProviderConfig(this.runtime.config, provider);
@@ -3956,6 +4027,7 @@ If lint or tests fail, report the issues but do NOT commit.`;
     if (this.runtime.config.ollama) providers.push('ollama');
     if (this.runtime.config.llamacpp) providers.push('llamacpp');
     if (this.runtime.config.openai) providers.push('openai');
+    if (this.runtime.config.mlx) providers.push('mlx');
     return providers.length ? providers : ['openrouter'];
   }
 
@@ -3964,7 +4036,8 @@ If lint or tests fail, report the issues but do NOT commit.`;
       openrouter: this.runtime.config.openrouter ?? (this.runtime.config.openrouter = { apiKey: '', model }),
       ollama: this.runtime.config.ollama ?? (this.runtime.config.ollama = { model }),
       llamacpp: this.runtime.config.llamacpp ?? (this.runtime.config.llamacpp = { model }),
-      openai: this.runtime.config.openai ?? (this.runtime.config.openai = { model })
+      openai: this.runtime.config.openai ?? (this.runtime.config.openai = { model }),
+      mlx: this.runtime.config.mlx ?? (this.runtime.config.mlx = { model })
     };
     cfgMap[provider].model = model;
     this.activeProvider = provider;

src/feedback/FeedbackApiClient.ts

@@ -25,8 +25,6 @@ export interface FeedbackApiConfig {
   offlineQueue: boolean;
   /** CLI version for analytics */
   cliVersion: string;
-  /** Company secret for API authentication */
-  companySecret: string;
 }
 
 export interface FeedbackSubmission extends FeedbackResponse {
@@ -62,8 +60,7 @@ const DEFAULT_CONFIG: FeedbackApiConfig = {
   timeout: 5000,
   maxRetries: 3,
   offlineQueue: true,
-  cliVersion: '0.1.0',
-  companySecret: ''
+  cliVersion: '0.1.0'
 };
 
 // ============ FeedbackApiClient ============
@@ -163,21 +160,19 @@ export class FeedbackApiClient {
 
   /**
    * Send feedback to API endpoint
+   * No authentication required - endpoint is rate-limited instead
    */
   private async sendToApi(submission: FeedbackSubmission): Promise<ApiResponse> {
     const controller = new AbortController();
     const timeoutId = setTimeout(() => controller.abort(), this.config.timeout);
 
-    // Build auth token: {device_id}.{company_secret}
-    const authToken = `${submission.deviceId}.${this.config.companySecret}`;
-
     try {
       const response = await fetch(`${this.config.baseUrl}/v1/feedback`, {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',
-          'Authorization': `Bearer ${authToken}`,
-          'X-CLI-Version': this.config.cliVersion
+          'X-CLI-Version': this.config.cliVersion,
+          'X-Device-ID': submission.deviceId
         },
         body: JSON.stringify(submission),
         signal: controller.signal