fix(rotation): strictly respect preference order in model switching

Author: Gooseware

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.2.8] - 2026-01-24
+
+### Fixed
+- **Model Selector Order** - Corrected `ModelSelector` to strictly respect the user's preference order defined in `quota.json`. Previously, it was incorrectly picking the model with the highest remaining quota percentage instead of following the priority list.
+- **Provider ID Support** - Improved `providerID` detection for more models (including GPT/OpenAI) and added support for explicit `provider/model` format in model IDs.
+
 ## [2.2.7] - 2026-01-24
 
 ### Fixed

package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-antigravity-autopilot",
-  "version": "2.2.7",
+  "version": "2.2.8",
   "description": "Intelligent quota monitoring and model auto-switching for OpenCode with Antigravity. Plugin-based, zero source modifications required.",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

src/plugin.ts

@@ -96,10 +96,14 @@ export const plugin: Plugin = async (ctx) => {
 
           output.message.model.modelID = result.nextModel;
 
-          if (result.nextModel.includes('claude')) {
+          if (result.nextModel.toLowerCase().includes('claude')) {
             output.message.model.providerID = 'anthropic';
-          } else if (result.nextModel.includes('gemini')) {
+          } else if (result.nextModel.toLowerCase().includes('gemini')) {
             output.message.model.providerID = 'google';
+          } else if (result.nextModel.toLowerCase().includes('gpt')) {
+            output.message.model.providerID = 'openai';
+          } else if (result.nextModel.includes('/')) {
+            output.message.model.providerID = result.nextModel.split('/')[0];
           }
 
           await ctx.client.tui.showToast({

src/rotation/QuotaTracker.ts

@@ -62,42 +62,31 @@ export class QuotaTracker {
   }
 
   getBestAvailableModel(candidates: string[]): string | null {
-    this.logger.debug('QuotaTracker', 'Finding best available model', {
+    this.logger.debug('QuotaTracker', 'Finding best available model (respecting preference order)', {
       candidatesCount: candidates.length,
       candidates,
     });
 
-    let bestModel: string | null = null;
-    let bestQuota = -1;
-
     for (const model of candidates) {
-      if (!this.isModelAvailable(model)) {
-        this.logger.debug('QuotaTracker', 'Model unavailable (below threshold)', { model });
-        continue;
-      }
-
-      const state = this.quotaState.get(model);
-      const quota = state?.quotaFraction ?? 1.0;
-
-      if (quota > bestQuota) {
-        bestQuota = quota;
-        bestModel = model;
+      if (this.isModelAvailable(model)) {
+        const state = this.quotaState.get(model);
+        const quotaPercentage = state ? `${(state.quotaFraction * 100).toFixed(1)}%` : 'unknown (assumed healthy)';
+        
+        this.logger.info('QuotaTracker', 'Selected model based on preference order', {
+          model,
+          quotaPercentage,
+        });
+        return model;
       }
+      
+      this.logger.debug('QuotaTracker', 'Model unavailable (below threshold), skipping', { model });
     }
 
-    if (bestModel) {
-      this.logger.info('QuotaTracker', 'Best model found', {
-        model: bestModel,
-        quotaFraction: bestQuota,
-        quotaPercentage: `${(bestQuota * 100).toFixed(1)}%`,
-      });
-    } else {
-      this.logger.warn('QuotaTracker', 'No available model found among candidates', {
-        candidatesCount: candidates.length,
-      });
-    }
+    this.logger.warn('QuotaTracker', 'No available model found among candidates', {
+      candidatesCount: candidates.length,
+    });
 
-    return bestModel;
+    return null;
   }
 
   getAllQuotaStates(): ModelQuotaState[] {