oh-my-opencode 3.1.8 → 3.1.10

package/dist/index.js

@@ -58,15 +58,40 @@ var init_constants = __esm(() => {
   PART_STORAGE = join2(OPENCODE_STORAGE, "part");
 });
 
+// src/shared/logger.ts
+var exports_logger = {};
+__export(exports_logger, {
+  log: () => log,
+  getLogFilePath: () => getLogFilePath
+});
+import * as fs from "fs";
+import * as os2 from "os";
+import * as path2 from "path";
+function log(message, data) {
+  try {
+    const timestamp = new Date().toISOString();
+    const logEntry = `[${timestamp}] ${message} ${data ? JSON.stringify(data) : ""}
+`;
+    fs.appendFileSync(logFile, logEntry);
+  } catch {}
+}
+function getLogFilePath() {
+  return logFile;
+}
+var logFile;
+var init_logger = __esm(() => {
+  logFile = path2.join(os2.tmpdir(), "oh-my-opencode.log");
+});
+
 // src/features/hook-message-injector/injector.ts
 import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from "fs";
-import { join as join3 } from "path";
+import { join as join4 } from "path";
 function findNearestMessageWithFields(messageDir) {
   try {
     const files = readdirSync(messageDir).filter((f) => f.endsWith(".json")).sort().reverse();
     for (const file of files) {
       try {
-        const content = readFileSync(join3(messageDir, file), "utf-8");
+        const content = readFileSync(join4(messageDir, file), "utf-8");
         const msg = JSON.parse(content);
         if (msg.agent && msg.model?.providerID && msg.model?.modelID) {
           return msg;
@@ -77,7 +102,7 @@ function findNearestMessageWithFields(messageDir) {
     }
     for (const file of files) {
       try {
-        const content = readFileSync(join3(messageDir, file), "utf-8");
+        const content = readFileSync(join4(messageDir, file), "utf-8");
         const msg = JSON.parse(content);
         if (msg.agent || msg.model?.providerID && msg.model?.modelID) {
           return msg;
@@ -96,7 +121,7 @@ function findFirstMessageWithAgent(messageDir) {
     const files = readdirSync(messageDir).filter((f) => f.endsWith(".json")).sort();
     for (const file of files) {
       try {
-        const content = readFileSync(join3(messageDir, file), "utf-8");
+        const content = readFileSync(join4(messageDir, file), "utf-8");
         const msg = JSON.parse(content);
         if (msg.agent) {
           return msg.agent;
@@ -124,12 +149,12 @@ function getOrCreateMessageDir(sessionID) {
   if (!existsSync(MESSAGE_STORAGE)) {
     mkdirSync(MESSAGE_STORAGE, { recursive: true });
   }
-  const directPath = join3(MESSAGE_STORAGE, sessionID);
+  const directPath = join4(MESSAGE_STORAGE, sessionID);
   if (existsSync(directPath)) {
     return directPath;
   }
   for (const dir of readdirSync(MESSAGE_STORAGE)) {
-    const sessionPath = join3(MESSAGE_STORAGE, dir, sessionID);
+    const sessionPath = join4(MESSAGE_STORAGE, dir, sessionID);
     if (existsSync(sessionPath)) {
       return sessionPath;
     }
@@ -139,7 +164,7 @@ function getOrCreateMessageDir(sessionID) {
 }
 function injectHookMessage(sessionID, hookContent, originalMessage) {
   if (!hookContent || hookContent.trim().length === 0) {
-    console.warn("[hook-message-injector] Attempted to inject empty hook content, skipping injection", {
+    log("[hook-message-injector] Attempted to inject empty hook content, skipping injection", {
       sessionID,
       hasAgent: !!originalMessage.agent,
       hasModel: !!(originalMessage.model?.providerID && originalMessage.model?.modelID)
@@ -191,12 +216,12 @@ function injectHookMessage(sessionID, hookContent, originalMessage) {
     sessionID
   };
   try {
-    writeFileSync(join3(messageDir, `${messageID}.json`), JSON.stringify(messageMeta, null, 2));
-    const partDir = join3(PART_STORAGE, messageID);
+    writeFileSync(join4(messageDir, `${messageID}.json`), JSON.stringify(messageMeta, null, 2));
+    const partDir = join4(PART_STORAGE, messageID);
     if (!existsSync(partDir)) {
       mkdirSync(partDir, { recursive: true });
     }
-    writeFileSync(join3(partDir, `${partID}.json`), JSON.stringify(textPart, null, 2));
+    writeFileSync(join4(partDir, `${partID}.json`), JSON.stringify(textPart, null, 2));
     return true;
   } catch {
     return false;
@@ -204,6 +229,7 @@ function injectHookMessage(sessionID, hookContent, originalMessage) {
 }
 var init_injector = __esm(() => {
   init_constants();
+  init_logger();
 });
 
 // src/features/hook-message-injector/index.ts
@@ -212,31 +238,6 @@ var init_hook_message_injector = __esm(() => {
   init_constants();
 });
 
-// src/shared/logger.ts
-var exports_logger = {};
-__export(exports_logger, {
-  log: () => log,
-  getLogFilePath: () => getLogFilePath
-});
-import * as fs from "fs";
-import * as os2 from "os";
-import * as path2 from "path";
-function log(message, data) {
-  try {
-    const timestamp = new Date().toISOString();
-    const logEntry = `[${timestamp}] ${message} ${data ? JSON.stringify(data) : ""}
-`;
-    fs.appendFileSync(logFile, logEntry);
-  } catch {}
-}
-function getLogFilePath() {
-  return logFile;
-}
-var logFile;
-var init_logger = __esm(() => {
-  logFile = path2.join(os2.tmpdir(), "oh-my-opencode.log");
-});
-
 // src/shared/system-directive.ts
 function createSystemDirective(type) {
   return `${SYSTEM_DIRECTIVE_PREFIX} - ${type}]`;
@@ -4848,6 +4849,8 @@ var init_model_requirements = __esm(() => {
     sisyphus: {
       fallbackChain: [
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
+        { providers: ["kimi-for-coding"], model: "k2p5" },
+        { providers: ["opencode"], model: "kimi-k2.5-free" },
         { providers: ["zai-coding-plan"], model: "glm-4.7" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2-codex", variant: "medium" },
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" }
@@ -4856,14 +4859,14 @@ var init_model_requirements = __esm(() => {
     oracle: {
       fallbackChain: [
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2", variant: "high" },
-        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
-        { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" }
+        { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" },
+        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" }
       ]
     },
     librarian: {
       fallbackChain: [
         { providers: ["zai-coding-plan"], model: "glm-4.7" },
-        { providers: ["opencode"], model: "big-pickle" },
+        { providers: ["opencode"], model: "glm-4.7-free" },
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-sonnet-4-5" }
       ]
     },
@@ -4879,6 +4882,8 @@ var init_model_requirements = __esm(() => {
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-flash" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2" },
         { providers: ["zai-coding-plan"], model: "glm-4.6v" },
+        { providers: ["kimi-for-coding"], model: "k2p5" },
+        { providers: ["opencode"], model: "kimi-k2.5-free" },
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-haiku-4-5" },
         { providers: ["opencode"], model: "gpt-5-nano" }
       ]
@@ -4886,6 +4891,8 @@ var init_model_requirements = __esm(() => {
     prometheus: {
       fallbackChain: [
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
+        { providers: ["kimi-for-coding"], model: "k2p5" },
+        { providers: ["opencode"], model: "kimi-k2.5-free" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2", variant: "high" },
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" }
       ]
@@ -4893,6 +4900,8 @@ var init_model_requirements = __esm(() => {
     metis: {
       fallbackChain: [
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
+        { providers: ["kimi-for-coding"], model: "k2p5" },
+        { providers: ["opencode"], model: "kimi-k2.5-free" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2", variant: "high" },
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" }
       ]
@@ -4900,12 +4909,14 @@ var init_model_requirements = __esm(() => {
     momus: {
       fallbackChain: [
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2", variant: "medium" },
-        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5" },
+        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" }
       ]
     },
     atlas: {
       fallbackChain: [
+        { providers: ["kimi-for-coding"], model: "k2p5" },
+        { providers: ["opencode"], model: "kimi-k2.5-free" },
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-sonnet-4-5" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2" },
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" }
@@ -4917,22 +4928,31 @@ var init_model_requirements = __esm(() => {
       fallbackChain: [
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" },
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
-        { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2", variant: "high" }
+        { providers: ["zai-coding-plan"], model: "glm-4.7" }
       ]
     },
     ultrabrain: {
       fallbackChain: [
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2-codex", variant: "xhigh" },
-        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
-        { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro" }
+        { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" },
+        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" }
       ]
     },
+    deep: {
+      fallbackChain: [
+        { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2-codex", variant: "medium" },
+        { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
+        { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" }
+      ],
+      requiresModel: "gpt-5.2-codex"
+    },
     artistry: {
       fallbackChain: [
         { providers: ["google", "github-copilot", "opencode"], model: "gemini-3-pro", variant: "max" },
         { providers: ["anthropic", "github-copilot", "opencode"], model: "claude-opus-4-5", variant: "max" },
         { providers: ["openai", "github-copilot", "opencode"], model: "gpt-5.2" }
-      ]
+      ],
+      requiresModel: "gemini-3-pro"
     },
     quick: {
       fallbackChain: [
@@ -5246,29 +5266,31 @@ async function updateConnectedProvidersCache(client) {
     const connected = result.data?.connected ?? [];
     log("[connected-providers-cache] Fetched connected providers", { count: connected.length, providers: connected });
     writeConnectedProvidersCache(connected);
+    let modelsByProvider = {};
     if (client.model?.list) {
       try {
         const modelsResult = await client.model.list();
         const models = modelsResult.data ?? [];
-        const modelsByProvider = {};
         for (const model of models) {
           if (!modelsByProvider[model.provider]) {
             modelsByProvider[model.provider] = [];
           }
           modelsByProvider[model.provider].push(model.id);
         }
-        writeProviderModelsCache({
-          models: modelsByProvider,
-          connected
-        });
-        log("[connected-providers-cache] Provider-models cache updated", {
+        log("[connected-providers-cache] Fetched models from API", {
           providerCount: Object.keys(modelsByProvider).length,
           totalModels: models.length
         });
       } catch (modelErr) {
-        log("[connected-providers-cache] Error fetching models", { error: String(modelErr) });
+        log("[connected-providers-cache] Error fetching models, writing empty cache", { error: String(modelErr) });
       }
+    } else {
+      log("[connected-providers-cache] client.model.list not available, writing empty cache");
     }
+    writeProviderModelsCache({
+      models: modelsByProvider,
+      connected
+    });
   } catch (err) {
     log("[connected-providers-cache] Error updating cache", { error: String(err) });
   }
@@ -5319,68 +5341,142 @@ function fuzzyMatchModel(target, available, providers) {
   log("[fuzzyMatchModel] shortest match", { result });
   return result;
 }
-async function fetchAvailableModels(_client, options) {
-  const connectedProvidersUnknown = options?.connectedProviders === null || options?.connectedProviders === undefined;
+function isModelAvailable(targetModel, availableModels) {
+  return fuzzyMatchModel(targetModel, availableModels) !== null;
+}
+async function getConnectedProviders(client) {
+  if (!client?.provider?.list) {
+    log("[getConnectedProviders] client.provider.list not available");
+    return [];
+  }
+  try {
+    const result = await client.provider.list();
+    const connected = result.data?.connected ?? [];
+    log("[getConnectedProviders] connected providers", { count: connected.length, providers: connected });
+    return connected;
+  } catch (err) {
+    log("[getConnectedProviders] SDK error", { error: String(err) });
+    return [];
+  }
+}
+async function fetchAvailableModels(client, options) {
+  let connectedProviders = options?.connectedProviders ?? null;
+  let connectedProvidersUnknown = connectedProviders === null;
   log("[fetchAvailableModels] CALLED", {
     connectedProvidersUnknown,
     connectedProviders: options?.connectedProviders
   });
+  if (connectedProvidersUnknown && client) {
+    const liveConnected = await getConnectedProviders(client);
+    if (liveConnected.length > 0) {
+      connectedProviders = liveConnected;
+      connectedProvidersUnknown = false;
+      log("[fetchAvailableModels] connected providers fetched from client", { count: liveConnected.length });
+    }
+  }
   if (connectedProvidersUnknown) {
+    if (client?.model?.list) {
+      const modelSet2 = new Set;
+      try {
+        const modelsResult = await client.model.list();
+        const models = modelsResult.data ?? [];
+        for (const model of models) {
+          if (model?.provider && model?.id) {
+            modelSet2.add(`${model.provider}/${model.id}`);
+          }
+        }
+        log("[fetchAvailableModels] fetched models from client without provider filter", {
+          count: modelSet2.size
+        });
+        return modelSet2;
+      } catch (err) {
+        log("[fetchAvailableModels] client.model.list error", { error: String(err) });
+      }
+    }
     log("[fetchAvailableModels] connected providers unknown, returning empty set for fallback resolution");
     return new Set;
   }
-  const connectedProviders = options.connectedProviders;
-  const connectedSet = new Set(connectedProviders);
+  const connectedProvidersList = connectedProviders ?? [];
+  const connectedSet = new Set(connectedProvidersList);
   const modelSet = new Set;
   const providerModelsCache = readProviderModelsCache();
   if (providerModelsCache) {
-    log("[fetchAvailableModels] using provider-models cache (whitelist-filtered)");
-    for (const [providerId, modelIds] of Object.entries(providerModelsCache.models)) {
-      if (!connectedSet.has(providerId)) {
-        continue;
+    const providerCount = Object.keys(providerModelsCache.models).length;
+    if (providerCount === 0) {
+      log("[fetchAvailableModels] provider-models cache empty, falling back to models.json");
+    } else {
+      log("[fetchAvailableModels] using provider-models cache (whitelist-filtered)");
+      for (const [providerId, modelIds] of Object.entries(providerModelsCache.models)) {
+        if (!connectedSet.has(providerId)) {
+          continue;
+        }
+        for (const modelId of modelIds) {
+          modelSet.add(`${providerId}/${modelId}`);
+        }
       }
-      for (const modelId of modelIds) {
-        modelSet.add(`${providerId}/${modelId}`);
+      log("[fetchAvailableModels] parsed from provider-models cache", {
+        count: modelSet.size,
+        connectedProviders: connectedProvidersList.slice(0, 5)
+      });
+      if (modelSet.size > 0) {
+        return modelSet;
       }
+      log("[fetchAvailableModels] provider-models cache produced no models for connected providers, falling back to models.json");
     }
-    log("[fetchAvailableModels] parsed from provider-models cache", {
-      count: modelSet.size,
-      connectedProviders: connectedProviders.slice(0, 5)
-    });
-    return modelSet;
   }
   log("[fetchAvailableModels] provider-models cache not found, falling back to models.json");
   const cacheFile = join13(getOpenCodeCacheDir(), "models.json");
   if (!existsSync10(cacheFile)) {
-    log("[fetchAvailableModels] models.json cache file not found, returning empty set");
-    return modelSet;
-  }
-  try {
-    const content = readFileSync7(cacheFile, "utf-8");
-    const data = JSON.parse(content);
-    const providerIds = Object.keys(data);
-    log("[fetchAvailableModels] providers found in models.json", { count: providerIds.length, providers: providerIds.slice(0, 10) });
-    for (const providerId of providerIds) {
-      if (!connectedSet.has(providerId)) {
-        continue;
+    log("[fetchAvailableModels] models.json cache file not found, falling back to client");
+  } else {
+    try {
+      const content = readFileSync7(cacheFile, "utf-8");
+      const data = JSON.parse(content);
+      const providerIds = Object.keys(data);
+      log("[fetchAvailableModels] providers found in models.json", { count: providerIds.length, providers: providerIds.slice(0, 10) });
+      for (const providerId of providerIds) {
+        if (!connectedSet.has(providerId)) {
+          continue;
+        }
+        const provider = data[providerId];
+        const models = provider?.models;
+        if (!models || typeof models !== "object")
+          continue;
+        for (const modelKey of Object.keys(models)) {
+          modelSet.add(`${providerId}/${modelKey}`);
+        }
       }
-      const provider = data[providerId];
-      const models = provider?.models;
-      if (!models || typeof models !== "object")
-        continue;
-      for (const modelKey of Object.keys(models)) {
-        modelSet.add(`${providerId}/${modelKey}`);
+      log("[fetchAvailableModels] parsed models from models.json (NO whitelist filtering)", {
+        count: modelSet.size,
+        connectedProviders: connectedProvidersList.slice(0, 5)
+      });
+      if (modelSet.size > 0) {
+        return modelSet;
       }
+    } catch (err) {
+      log("[fetchAvailableModels] error", { error: String(err) });
     }
-    log("[fetchAvailableModels] parsed models from models.json (NO whitelist filtering)", {
-      count: modelSet.size,
-      connectedProviders: connectedProviders.slice(0, 5)
-    });
-    return modelSet;
-  } catch (err) {
-    log("[fetchAvailableModels] error", { error: String(err) });
-    return modelSet;
   }
+  if (client?.model?.list) {
+    try {
+      const modelsResult = await client.model.list();
+      const models = modelsResult.data ?? [];
+      for (const model of models) {
+        if (!model?.provider || !model?.id)
+          continue;
+        if (connectedSet.has(model.provider)) {
+          modelSet.add(`${model.provider}/${model.id}`);
+        }
+      }
+      log("[fetchAvailableModels] fetched models from client (filtered)", {
+        count: modelSet.size,
+        connectedProviders: connectedProvidersList.slice(0, 5)
+      });
+    } catch (err) {
+      log("[fetchAvailableModels] client.model.list error", { error: String(err) });
+    }
+  }
+  return modelSet;
 }
 function isModelCacheAvailable() {
   if (hasProviderModelsCache()) {
@@ -5404,7 +5500,7 @@ function resolveModel(input) {
   return normalizeModel(input.userModel) ?? normalizeModel(input.inheritedModel) ?? input.systemDefault;
 }
 function resolveModelWithFallback(input) {
-  const { uiSelectedModel, userModel, fallbackChain, availableModels, systemDefaultModel } = input;
+  const { uiSelectedModel, userModel, categoryDefaultModel, fallbackChain, availableModels, systemDefaultModel } = input;
   const normalizedUiModel = normalizeModel(uiSelectedModel);
   if (normalizedUiModel) {
     log("Model resolved via UI selection", { model: normalizedUiModel });
@@ -5415,6 +5511,33 @@ function resolveModelWithFallback(input) {
     log("Model resolved via config override", { model: normalizedUserModel });
     return { model: normalizedUserModel, source: "override" };
   }
+  const normalizedCategoryDefault = normalizeModel(categoryDefaultModel);
+  if (normalizedCategoryDefault) {
+    if (availableModels.size > 0) {
+      const parts = normalizedCategoryDefault.split("/");
+      const providerHint = parts.length >= 2 ? [parts[0]] : undefined;
+      const match = fuzzyMatchModel(normalizedCategoryDefault, availableModels, providerHint);
+      if (match) {
+        log("Model resolved via category default (fuzzy matched)", { original: normalizedCategoryDefault, matched: match });
+        return { model: match, source: "category-default" };
+      }
+    } else {
+      const connectedProviders = readConnectedProvidersCache();
+      if (connectedProviders === null) {
+        log("Model resolved via category default (no cache, first run)", { model: normalizedCategoryDefault });
+        return { model: normalizedCategoryDefault, source: "category-default" };
+      }
+      const parts = normalizedCategoryDefault.split("/");
+      if (parts.length >= 2) {
+        const provider = parts[0];
+        if (connectedProviders.includes(provider)) {
+          log("Model resolved via category default (connected provider)", { model: normalizedCategoryDefault });
+          return { model: normalizedCategoryDefault, source: "category-default" };
+        }
+      }
+    }
+    log("Category default model not available, falling through to fallback chain", { model: normalizedCategoryDefault });
+  }
   if (fallbackChain && fallbackChain.length > 0) {
     if (availableModels.size === 0) {
       const connectedProviders = readConnectedProvidersCache();
@@ -5426,7 +5549,7 @@ function resolveModelWithFallback(input) {
           for (const provider of entry.providers) {
             if (connectedSet.has(provider)) {
               const model = `${provider}/${entry.model}`;
-              log("Model resolved via fallback chain (no model cache, using connected provider)", {
+              log("Model resolved via fallback chain (connected provider)", {
                 provider,
                 model: entry.model,
                 variant: entry.variant
@@ -5437,18 +5560,28 @@ function resolveModelWithFallback(input) {
         }
         log("No connected provider found in fallback chain, falling through to system default");
       }
-    }
-    for (const entry of fallbackChain) {
-      for (const provider of entry.providers) {
-        const fullModel = `${provider}/${entry.model}`;
-        const match = fuzzyMatchModel(fullModel, availableModels, [provider]);
-        if (match) {
-          log("Model resolved via fallback chain (availability confirmed)", { provider, model: entry.model, match, variant: entry.variant });
-          return { model: match, source: "provider-fallback", variant: entry.variant };
+    } else {
+      for (const entry of fallbackChain) {
+        for (const provider of entry.providers) {
+          const fullModel = `${provider}/${entry.model}`;
+          const match = fuzzyMatchModel(fullModel, availableModels, [provider]);
+          if (match) {
+            log("Model resolved via fallback chain (availability confirmed)", { provider, model: entry.model, match, variant: entry.variant });
+            return { model: match, source: "provider-fallback", variant: entry.variant };
+          }
+        }
+        const crossProviderMatch = fuzzyMatchModel(entry.model, availableModels);
+        if (crossProviderMatch) {
+          log("Model resolved via fallback chain (cross-provider fuzzy match)", {
+            model: entry.model,
+            match: crossProviderMatch,
+            variant: entry.variant
+          });
+          return { model: crossProviderMatch, source: "provider-fallback", variant: entry.variant };
         }
       }
+      log("No available model found in fallback chain, falling through to system default");
     }
-    log("No available model found in fallback chain, falling through to system default");
   }
   if (systemDefaultModel === undefined) {
     log("No model resolved - systemDefaultModel not configured");
@@ -5720,6 +5853,92 @@ var init_tmux = __esm(() => {
   init_tmux_utils();
 });
 
+// src/shared/model-suggestion-retry.ts
+function extractMessage(error) {
+  if (typeof error === "string")
+    return error;
+  if (error instanceof Error)
+    return error.message;
+  if (typeof error === "object" && error !== null) {
+    const obj = error;
+    if (typeof obj.message === "string")
+      return obj.message;
+    try {
+      return JSON.stringify(error);
+    } catch {
+      return "";
+    }
+  }
+  return String(error);
+}
+function parseModelSuggestion(error) {
+  if (!error)
+    return null;
+  if (typeof error === "object") {
+    const errObj = error;
+    if (errObj.name === "ProviderModelNotFoundError" && typeof errObj.data === "object" && errObj.data !== null) {
+      const data = errObj.data;
+      const suggestions = data.suggestions;
+      if (Array.isArray(suggestions) && suggestions.length > 0 && typeof suggestions[0] === "string") {
+        return {
+          providerID: String(data.providerID ?? ""),
+          modelID: String(data.modelID ?? ""),
+          suggestion: suggestions[0]
+        };
+      }
+      return null;
+    }
+    for (const key of ["data", "error", "cause"]) {
+      const nested = errObj[key];
+      if (nested && typeof nested === "object") {
+        const result = parseModelSuggestion(nested);
+        if (result)
+          return result;
+      }
+    }
+  }
+  const message = extractMessage(error);
+  if (!message)
+    return null;
+  const modelMatch = message.match(/model not found:\s*([^/\s]+)\s*\/\s*([^.\s]+)/i);
+  const suggestionMatch = message.match(/did you mean:\s*([^,?]+)/i);
+  if (modelMatch && suggestionMatch) {
+    return {
+      providerID: modelMatch[1].trim(),
+      modelID: modelMatch[2].trim(),
+      suggestion: suggestionMatch[1].trim()
+    };
+  }
+  return null;
+}
+async function promptWithModelSuggestionRetry(client, args) {
+  try {
+    await client.session.prompt(args);
+  } catch (error) {
+    const suggestion = parseModelSuggestion(error);
+    if (!suggestion || !args.body.model) {
+      throw error;
+    }
+    log("[model-suggestion-retry] Model not found, retrying with suggestion", {
+      original: `${suggestion.providerID}/${suggestion.modelID}`,
+      suggested: suggestion.suggestion
+    });
+    await client.session.prompt({
+      ...args,
+      body: {
+        ...args.body,
+        model: {
+          providerID: suggestion.providerID,
+          modelID: suggestion.suggestion
+        }
+      }
+    });
+  }
+}
+var init_model_suggestion_retry = __esm(() => {
+  init_logger();
+});
+
 // src/shared/index.ts
 var init_shared = __esm(() => {
   init_frontmatter();
@@ -5750,6 +5969,7 @@ var init_shared = __esm(() => {
   init_connected_providers_cache();
   init_session_utils();
   init_tmux();
+  init_model_suggestion_retry();
 });
 
 // node_modules/picomatch/lib/constants.js
@@ -11082,8 +11302,14 @@ Design-first mindset:
 - Atmosphere: gradient meshes, noise textures, layered transparencies
 
 AVOID: Generic fonts, purple gradients on white, predictable layouts, cookie-cutter patterns.
-</Category_Context>`, STRATEGIC_CATEGORY_PROMPT_APPEND = `<Category_Context>
-You are working on BUSINESS LOGIC / ARCHITECTURE tasks.
+</Category_Context>`, ULTRABRAIN_CATEGORY_PROMPT_APPEND = `<Category_Context>
+You are working on DEEP LOGICAL REASONING / COMPLEX ARCHITECTURE tasks.
+
+**CRITICAL - CODE STYLE REQUIREMENTS (NON-NEGOTIABLE)**:
+1. BEFORE writing ANY code, SEARCH the existing codebase to find similar patterns/styles
+2. Your code MUST match the project's existing conventions - blend in seamlessly
+3. Write READABLE code that humans can easily understand - no clever tricks
+4. If unsure about style, explore more files until you find the pattern
 
 Strategic advisor mindset:
 - Bias toward simplicity: least complex solution that fulfills requirements
@@ -11209,6 +11435,35 @@ Approach:
 - Draft with care
 - Polish for clarity and impact
 - Documentation, READMEs, articles, technical writing
+</Category_Context>`, DEEP_CATEGORY_PROMPT_APPEND = `<Category_Context>
+You are working on GOAL-ORIENTED AUTONOMOUS tasks.
+
+**CRITICAL - AUTONOMOUS EXECUTION MINDSET (NON-NEGOTIABLE)**:
+You are NOT an interactive assistant. You are an autonomous problem-solver.
+
+**BEFORE making ANY changes**:
+1. SILENTLY explore the codebase extensively (5-15 minutes of reading is normal)
+2. Read related files, trace dependencies, understand the full context
+3. Build a complete mental model of the problem space
+4. DO NOT ask clarifying questions - the goal is already defined
+
+**Autonomous executor mindset**:
+- You receive a GOAL, not step-by-step instructions
+- Figure out HOW to achieve the goal yourself
+- Thorough research before any action
+- Fix hairy problems that require deep understanding
+- Work independently without frequent check-ins
+
+**Approach**:
+- Explore extensively, understand deeply, then act decisively
+- Prefer comprehensive solutions over quick patches
+- If the goal is unclear, make reasonable assumptions and proceed
+- Document your reasoning in code comments only when non-obvious
+
+**Response format**:
+- Minimal status updates (user trusts your autonomy)
+- Focus on results, not play-by-play progress
+- Report completion with summary of changes made
 </Category_Context>`, DEFAULT_CATEGORIES, CATEGORY_PROMPT_APPENDS, CATEGORY_DESCRIPTIONS, PLAN_AGENT_SYSTEM_PREPEND = `<system>
 BEFORE you begin planning, you MUST first understand the user's request deeply.
 
@@ -11422,6 +11677,7 @@ var init_constants4 = __esm(() => {
   DEFAULT_CATEGORIES = {
     "visual-engineering": { model: "google/gemini-3-pro" },
     ultrabrain: { model: "openai/gpt-5.2-codex", variant: "xhigh" },
+    deep: { model: "openai/gpt-5.2-codex", variant: "medium" },
     artistry: { model: "google/gemini-3-pro", variant: "max" },
     quick: { model: "anthropic/claude-haiku-4-5" },
     "unspecified-low": { model: "anthropic/claude-sonnet-4-5" },
@@ -11430,7 +11686,8 @@ var init_constants4 = __esm(() => {
   };
   CATEGORY_PROMPT_APPENDS = {
     "visual-engineering": VISUAL_CATEGORY_PROMPT_APPEND,
-    ultrabrain: STRATEGIC_CATEGORY_PROMPT_APPEND,
+    ultrabrain: ULTRABRAIN_CATEGORY_PROMPT_APPEND,
+    deep: DEEP_CATEGORY_PROMPT_APPEND,
     artistry: ARTISTRY_CATEGORY_PROMPT_APPEND,
     quick: QUICK_CATEGORY_PROMPT_APPEND,
     "unspecified-low": UNSPECIFIED_LOW_CATEGORY_PROMPT_APPEND,
@@ -11439,8 +11696,9 @@ var init_constants4 = __esm(() => {
   };
   CATEGORY_DESCRIPTIONS = {
     "visual-engineering": "Frontend, UI/UX, design, styling, animation",
-    ultrabrain: "Deep logical reasoning, complex architecture decisions requiring extensive analysis",
-    artistry: "Highly creative/artistic tasks, novel ideas",
+    ultrabrain: "Use ONLY for genuinely hard, logic-heavy tasks. Give clear goals only, not step-by-step instructions.",
+    deep: "Goal-oriented autonomous problem-solving. Thorough research before action. For hairy problems requiring deep understanding.",
+    artistry: "Complex problem-solving with unconventional, creative approaches - beyond standard patterns",
     quick: "Trivial tasks - single file changes, typo fixes, simple modifications",
     "unspecified-low": "Tasks that don't fit other categories, low effort required",
     "unspecified-high": "Tasks that don't fit other categories, high effort required",
@@ -19497,6 +19755,7 @@ function findMessageByIndexNeedingThinking(sessionID, targetIndex) {
 }
 
 // src/hooks/session-recovery/index.ts
+init_logger();
 var RECOVERY_RESUME_TEXT = "[session recovered - continuing previous task]";
 function findLastUserMessage(messages) {
   for (let i = messages.length - 1;i >= 0; i--) {
@@ -19717,7 +19976,7 @@ function createSessionRecoveryHook(ctx, options) {
       }
       return success;
     } catch (err) {
-      console.error("[session-recovery] Recovery failed:", err);
+      log("[session-recovery] Recovery failed:", err);
       return false;
     } finally {
       processingErrors.delete(assistantMsgID);
@@ -19743,6 +20002,7 @@ import { tmpdir as tmpdir3 } from "os";
 
 // src/hooks/comment-checker/downloader.ts
 init_shared();
+init_logger();
 var {spawn: spawn6 } = globalThis.Bun;
 import { existsSync as existsSync12, mkdirSync as mkdirSync4, chmodSync, unlinkSync as unlinkSync2, appendFileSync as appendFileSync2 } from "fs";
 import { join as join15 } from "path";
@@ -19822,7 +20082,7 @@ async function downloadCommentChecker() {
   const assetName = `comment-checker_v${version}_${os4}_${arch}.${ext}`;
   const downloadUrl = `https://github.com/${REPO}/releases/download/v${version}/${assetName}`;
   debugLog(`Downloading from: ${downloadUrl}`);
-  console.log(`[oh-my-opencode] Downloading comment-checker binary...`);
+  log(`[oh-my-opencode] Downloading comment-checker binary...`);
   try {
     if (!existsSync12(cacheDir)) {
       mkdirSync4(cacheDir, { recursive: true });
@@ -19847,12 +20107,12 @@ async function downloadCommentChecker() {
       chmodSync(binaryPath, 493);
     }
     debugLog(`Successfully downloaded binary to: ${binaryPath}`);
-    console.log(`[oh-my-opencode] comment-checker binary ready.`);
+    log(`[oh-my-opencode] comment-checker binary ready.`);
     return binaryPath;
   } catch (err) {
     debugLog(`Failed to download: ${err}`);
-    console.error(`[oh-my-opencode] Failed to download comment-checker: ${err instanceof Error ? err.message : err}`);
-    console.error(`[oh-my-opencode] Comment checking disabled.`);
+    log(`[oh-my-opencode] Failed to download comment-checker: ${err instanceof Error ? err.message : err}`);
+    log(`[oh-my-opencode] Comment checking disabled.`);
     return null;
   }
 }
@@ -23529,7 +23789,9 @@ ${ULTRAWORK_PLANNER_SECTION}
 
 1. **THINK DEEPLY** - What is the user's TRUE intent? What problem are they REALLY trying to solve?
 2. **EXPLORE THOROUGHLY** - Fire explore/librarian agents to gather ALL relevant context
-3. **CONSULT ORACLE** - For architecture decisions, complex logic, or when you're stuck
+3. **CONSULT SPECIALISTS** - For hard/complex tasks, DO NOT struggle alone. Delegate:
+   - **Oracle**: Conventional problems - architecture, debugging, complex logic
+   - **Artistry**: Non-conventional problems - different approach needed, unusual constraints
 4. **ASK THE USER** - If ambiguity remains after exploration, ASK. Don't guess.
 
 **SIGNS YOU ARE NOT READY TO IMPLEMENT:**
@@ -23543,7 +23805,10 @@ ${ULTRAWORK_PLANNER_SECTION}
 \`\`\`
 delegate_task(agent="explore", prompt="Find [X] patterns in codebase", background=true)
 delegate_task(agent="librarian", prompt="Find docs/examples for [Y]", background=true)
-delegate_task(agent="oracle", prompt="Review my approach: [describe plan]")
+
+// Hard problem? DON'T struggle alone:
+delegate_task(agent="oracle", prompt="...")         // conventional: architecture, debugging
+delegate_task(category="artistry", prompt="...")    // non-conventional: needs different approach
 \`\`\`
 
 **ONLY AFTER YOU HAVE:**
@@ -23578,7 +23843,7 @@ delegate_task(agent="oracle", prompt="Review my approach: [describe plan]")
 **IF YOU ENCOUNTER A BLOCKER:**
 1. **DO NOT** give up
 2. **DO NOT** deliver a compromised version
-3. **DO** consult oracle for solutions
+3. **DO** consult specialists (oracle for conventional, artistry for non-conventional)
 4. **DO** ask the user for guidance
 5. **DO** explore alternative approaches
 
@@ -23647,7 +23912,8 @@ delegate_task(session_id="ses_abc123", prompt="Here's my answer to your question
 | Codebase exploration | delegate_task(subagent_type="explore", run_in_background=true) | Parallel, context-efficient |
 | Documentation lookup | delegate_task(subagent_type="librarian", run_in_background=true) | Specialized knowledge |
 | Planning | delegate_task(subagent_type="plan") | Parallel task graph + structured TODO list |
-| Architecture/Debugging | delegate_task(subagent_type="oracle") | High-IQ reasoning |
+| Hard problem (conventional) | delegate_task(subagent_type="oracle") | Architecture, debugging, complex logic |
+| Hard problem (non-conventional) | delegate_task(category="artistry", load_skills=[...]) | Different approach needed |
 | Implementation | delegate_task(category="...", load_skills=[...]) | Domain-optimized models |
 
 **CATEGORY + SKILL DELEGATION:**
@@ -23834,8 +24100,9 @@ CONTEXT GATHERING (parallel):
 - 1-2 librarian agents (if external library involved)
 - Direct tools: Grep, AST-grep, LSP for targeted searches
 
-IF COMPLEX (architecture, multi-system, debugging after 2+ failures):
-- Consult oracle for strategic guidance
+IF COMPLEX - DO NOT STRUGGLE ALONE. Consult specialists:
+- **Oracle**: Conventional problems (architecture, debugging, complex logic)
+- **Artistry**: Non-conventional problems (different approach needed)
 
 SYNTHESIZE findings before proceeding.`
   }
@@ -29745,6 +30012,8 @@ import { Readable, Writable } from "stream";
 import { readFileSync as readFileSync27 } from "fs";
 import { extname, resolve as resolve8 } from "path";
 import { pathToFileURL } from "url";
+init_logger();
+
 class LSPServerManager {
   static instance;
   clients = new Map;
@@ -29994,7 +30263,7 @@ stderr: ${stderr}` : ""));
       this.processExited = true;
     });
     this.connection.onError((error) => {
-      console.error("LSP connection error:", error);
+      log("LSP connection error:", error);
     });
     this.connection.listen();
   }
@@ -43028,6 +43297,7 @@ import { existsSync as existsSync39, statSync as statSync4 } from "fs";
 
 // src/tools/ast-grep/downloader.ts
 init_shared();
+init_logger();
 import { existsSync as existsSync38, mkdirSync as mkdirSync13, chmodSync as chmodSync2, unlinkSync as unlinkSync10 } from "fs";
 import { join as join47 } from "path";
 import { homedir as homedir11 } from "os";
@@ -43073,7 +43343,7 @@ async function downloadAstGrep(version2 = DEFAULT_VERSION) {
   const platformKey = `${process.platform}-${process.arch}`;
   const platformInfo = PLATFORM_MAP2[platformKey];
   if (!platformInfo) {
-    console.error(`[oh-my-opencode] Unsupported platform for ast-grep: ${platformKey}`);
+    log(`[oh-my-opencode] Unsupported platform for ast-grep: ${platformKey}`);
     return null;
   }
   const cacheDir = getCacheDir4();
@@ -43085,7 +43355,7 @@ async function downloadAstGrep(version2 = DEFAULT_VERSION) {
   const { arch, os: os6 } = platformInfo;
   const assetName = `app-${arch}-${os6}.zip`;
   const downloadUrl = `https://github.com/${REPO2}/releases/download/${version2}/${assetName}`;
-  console.log(`[oh-my-opencode] Downloading ast-grep binary...`);
+  log(`[oh-my-opencode] Downloading ast-grep binary...`);
   try {
     if (!existsSync38(cacheDir)) {
       mkdirSync13(cacheDir, { recursive: true });
@@ -43104,10 +43374,10 @@ async function downloadAstGrep(version2 = DEFAULT_VERSION) {
     if (process.platform !== "win32" && existsSync38(binaryPath)) {
       chmodSync2(binaryPath, 493);
     }
-    console.log(`[oh-my-opencode] ast-grep binary ready.`);
+    log(`[oh-my-opencode] ast-grep binary ready.`);
     return binaryPath;
   } catch (err) {
-    console.error(`[oh-my-opencode] Failed to download ast-grep: ${err instanceof Error ? err.message : err}`);
+    log(`[oh-my-opencode] Failed to download ast-grep: ${err instanceof Error ? err.message : err}`);
     return null;
   }
 }
@@ -45239,6 +45509,7 @@ ${REFACTOR_TEMPLATE}
   },
   "start-work": {
     description: "(builtin) Start Sisyphus work session from Prometheus plan",
+    agent: "atlas",
     template: `<command-instruction>
 ${START_WORK_TEMPLATE}
 </command-instruction>
@@ -47138,7 +47409,7 @@ var LOOK_AT_DESCRIPTION = `Analyze media files (PDFs, images, diagrams) that req
 // src/tools/look-at/tools.ts
 import { extname as extname3, basename as basename5 } from "path";
 import { pathToFileURL as pathToFileURL2 } from "url";
-init_logger();
+init_shared();
 function normalizeArgs(args) {
   return {
     file_path: args.file_path ?? args.path ?? "",
@@ -47249,9 +47520,26 @@ Original error: ${createResult.error}`;
       }
       const sessionID = createResult.data.id;
       log(`[look_at] Created session: ${sessionID}`);
+      let agentModel;
+      let agentVariant;
+      try {
+        const agentsResult = await ctx.client.app?.agents?.();
+        const agents = agentsResult?.data ?? agentsResult;
+        if (agents?.length) {
+          const matchedAgent = findByNameCaseInsensitive(agents, MULTIMODAL_LOOKER_AGENT);
+          if (matchedAgent?.model) {
+            agentModel = matchedAgent.model;
+          }
+          if (matchedAgent?.variant) {
+            agentVariant = matchedAgent.variant;
+          }
+        }
+      } catch (error45) {
+        log("[look_at] Failed to resolve multimodal-looker model info", error45);
+      }
       log(`[look_at] Sending prompt with file passthrough to session ${sessionID}`);
       try {
-        await ctx.client.session.prompt({
+        await promptWithModelSuggestionRetry(ctx.client, {
           path: { id: sessionID },
           body: {
             agent: MULTIMODAL_LOOKER_AGENT,
@@ -47264,7 +47552,9 @@ Original error: ${createResult.error}`;
             parts: [
               { type: "text", text: prompt },
               { type: "file", mime: mimeType, url: pathToFileURL2(args.file_path).href, filename }
-            ]
+            ],
+            ...agentModel ? { model: { providerID: agentModel.providerID, modelID: agentModel.modelID } } : {},
+            ...agentVariant ? { variant: agentVariant } : {}
           }
         });
       } catch (promptError) {
@@ -47571,7 +47861,14 @@ function formatDetailedError(error45, ctx) {
 `);
 }
 function resolveCategoryConfig(categoryName, options) {
-  const { userCategories, inheritedModel, systemDefaultModel } = options;
+  const { userCategories, inheritedModel, systemDefaultModel, availableModels } = options;
+  const categoryReq = CATEGORY_MODEL_REQUIREMENTS[categoryName];
+  if (categoryReq?.requiresModel && availableModels) {
+    if (!isModelAvailable(categoryReq.requiresModel, availableModels)) {
+      log(`[resolveCategoryConfig] Category ${categoryName} requires ${categoryReq.requiresModel} but not available`);
+      return null;
+    }
+  }
   const defaultConfig = DEFAULT_CATEGORIES[categoryName];
   const userConfig = userCategories?.[categoryName];
   const defaultPromptAppend = CATEGORY_PROMPT_APPENDS[categoryName] ?? "";
@@ -47898,7 +48195,8 @@ To continue this session: session_id="${args.session_id}"`;
         const resolved = resolveCategoryConfig(args.category, {
           userCategories,
           inheritedModel,
-          systemDefaultModel
+          systemDefaultModel,
+          availableModels
         });
         if (!resolved) {
           return `Unknown category: "${args.category}". Available: ${Object.keys({ ...DEFAULT_CATEGORIES, ...userCategories }).join(", ")}`;
@@ -47912,7 +48210,8 @@ To continue this session: session_id="${args.session_id}"`;
           }
         } else {
           const resolution = resolveModelWithFallback({
-            userModel: userCategories?.[args.category]?.model ?? resolved.model ?? sisyphusJuniorModel,
+            userModel: userCategories?.[args.category]?.model,
+            categoryDefaultModel: resolved.model ?? sisyphusJuniorModel,
             fallbackChain: requirement.fallbackChain,
             availableModels,
             systemDefaultModel
@@ -47928,6 +48227,7 @@ To continue this session: session_id="${args.session_id}"`;
               case "override":
                 type2 = "user-defined";
                 break;
+              case "category-default":
               case "provider-fallback":
                 type2 = "category-default";
                 break;
@@ -48135,10 +48435,6 @@ Create the work plan directly - that's your job as the planning agent.`;
             categoryModel = matchedAgent.model;
           }
         } catch {}
-        if (parentModel) {
-          categoryModel = parentModel;
-          modelInfo = { model: `${parentModel.providerID}/${parentModel.modelID}`, type: "inherited" };
-        }
       }
       const systemContent = buildSystemContent({ skillContent, categoryPromptAppend, agentName: agentToUse });
       if (runInBackground) {
@@ -48251,7 +48547,7 @@ To continue this session: session_id="${task.sessionID}"`;
         });
         try {
           const allowDelegateTask = isPlanAgent(agentToUse);
-          await client2.session.prompt({
+          await promptWithModelSuggestionRetry(client2, {
             path: { id: sessionID },
             body: {
               agent: agentToUse,
@@ -48723,7 +49019,7 @@ class BackgroundManager {
     });
     const launchModel = input.model ? { providerID: input.model.providerID, modelID: input.model.modelID } : undefined;
     const launchVariant = input.model?.variant;
-    this.client.session.prompt({
+    promptWithModelSuggestionRetry(this.client, {
       path: { id: sessionID },
       body: {
         agent: input.agent,
@@ -55239,6 +55535,7 @@ var CategoryConfigSchema = exports_external.object({
 var BuiltinCategoryNameSchema = exports_external.enum([
   "visual-engineering",
   "ultrabrain",
+  "deep",
   "artistry",
   "quick",
   "unspecified-low",
@@ -55758,6 +56055,7 @@ ${patterns.join(`
 }
 
 // src/agents/sisyphus.ts
+var MODE = "primary";
 function buildDynamicSisyphusPrompt(availableAgents, availableTools = [], availableSkills = [], availableCategories = []) {
   const keyTriggers = buildKeyTriggersSection(availableAgents, availableSkills);
   const toolSelection = buildToolSelectionTable(availableAgents, availableTools, availableSkills);
@@ -56161,7 +56459,7 @@ function createSisyphusAgent(model, availableAgents, availableToolNames, availab
   const permission = { question: "allow", call_omo_agent: "deny" };
   const base = {
     description: "Powerful AI orchestrator. Plans obsessively with todos, assesses search complexity before exploration, delegates strategically via category+skills combinations. Uses explore for internal code (parallel-friendly), librarian for external docs. (Sisyphus - OhMyOpenCode)",
-    mode: "primary",
+    mode: MODE,
     model,
     maxTokens: 64000,
     prompt,
@@ -56173,8 +56471,10 @@ function createSisyphusAgent(model, availableAgents, availableToolNames, availab
   }
   return { ...base, thinking: { type: "enabled", budgetTokens: 32000 } };
 }
+createSisyphusAgent.mode = MODE;
 
 // src/agents/oracle.ts
+var MODE2 = "subagent";
 var ORACLE_PROMPT_METADATA = {
   category: "advisor",
   cost: "EXPENSIVE",
@@ -56274,7 +56574,7 @@ function createOracleAgent(model) {
   ]);
   const base = {
     description: "Read-only consultation agent. High-IQ reasoning specialist for debugging hard problems and high-difficulty architecture design. (Oracle - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE2,
     model,
     temperature: 0.1,
     ...restrictions,
@@ -56285,8 +56585,10 @@ function createOracleAgent(model) {
   }
   return { ...base, thinking: { type: "enabled", budgetTokens: 32000 } };
 }
+createOracleAgent.mode = MODE2;
 
 // src/agents/librarian.ts
+var MODE3 = "subagent";
 var LIBRARIAN_PROMPT_METADATA = {
   category: "exploration",
   cost: "CHEAP",
@@ -56313,7 +56615,7 @@ function createLibrarianAgent(model) {
   ]);
   return {
     description: "Specialized codebase understanding agent for multi-repository analysis, searching remote codebases, retrieving official documentation, and finding implementation examples using GitHub CLI, Context7, and Web Search. MUST BE USED when users ask to look up code in remote repositories, explain library internals, or find usage examples in open source. (Librarian - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE3,
     model,
     temperature: 0.1,
     ...restrictions,
@@ -56605,8 +56907,10 @@ grep_app_searchGitHub(query: "useQuery")
 `
   };
 }
+createLibrarianAgent.mode = MODE3;
 
 // src/agents/explore.ts
+var MODE4 = "subagent";
 var EXPLORE_PROMPT_METADATA = {
   category: "exploration",
   cost: "FREE",
@@ -56636,7 +56940,7 @@ function createExploreAgent(model) {
   ]);
   return {
     description: 'Contextual grep for codebases. Answers "Where is X?", "Which file has Y?", "Find the code that does Z". Fire multiple in parallel for broad searches. Specify thoroughness: "quick" for basic, "medium" for moderate, "very thorough" for comprehensive analysis. (Explore - OhMyOpenCode)',
-    mode: "subagent",
+    mode: MODE4,
     model,
     temperature: 0.1,
     ...restrictions,
@@ -56721,8 +57025,10 @@ Use the right tool for the job:
 Flood with parallel calls. Cross-validate findings across multiple tools.`
   };
 }
+createExploreAgent.mode = MODE4;
 
 // src/agents/multimodal-looker.ts
+var MODE5 = "subagent";
 var MULTIMODAL_LOOKER_PROMPT_METADATA = {
   category: "utility",
   cost: "CHEAP",
@@ -56733,7 +57039,7 @@ function createMultimodalLookerAgent(model) {
   const restrictions = createAgentToolAllowlist(["read"]);
   return {
     description: "Analyze media files (PDFs, images, diagrams) that require interpretation beyond raw text. Extracts specific information or summaries from documents, describes visual content. Use when you need analyzed/extracted data rather than literal file contents. (Multimodal-Looker - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE5,
     model,
     temperature: 0.1,
     ...restrictions,
@@ -56771,8 +57077,10 @@ Response rules:
 Your output goes straight to the main agent for continued work.`
   };
 }
+createMultimodalLookerAgent.mode = MODE5;
 
 // src/agents/metis.ts
+var MODE6 = "subagent";
 var METIS_SYSTEM_PROMPT = `# Metis - Pre-Planning Consultant
 
 ## CONSTRAINTS
@@ -57065,7 +57373,7 @@ var metisRestrictions = createAgentToolRestrictions([
 function createMetisAgent(model) {
   return {
     description: "Pre-planning consultant that analyzes requests to identify hidden intentions, ambiguities, and AI failure points. (Metis - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE6,
     model,
     temperature: 0.3,
     ...metisRestrictions,
@@ -57073,9 +57381,11 @@ function createMetisAgent(model) {
     thinking: { type: "enabled", budgetTokens: 32000 }
   };
 }
+createMetisAgent.mode = MODE6;
 
 // src/agents/atlas.ts
 init_constants4();
+var MODE7 = "primary";
 var getCategoryDescription = (name, userCategories) => userCategories?.[name]?.description ?? CATEGORY_DESCRIPTIONS[name] ?? "General tasks";
 function buildAgentSelectionSection(agents) {
   if (agents.length === 0) {
@@ -57564,7 +57874,7 @@ function createAtlasAgent(ctx) {
   ]);
   return {
     description: "Orchestrates work via delegate_task() to complete ALL tasks in a todo list until fully done. (Atlas - OhMyOpenCode)",
-    mode: "primary",
+    mode: MODE7,
     ...ctx.model ? { model: ctx.model } : {},
     temperature: 0.1,
     prompt: buildDynamicOrchestratorPrompt(ctx),
@@ -57573,378 +57883,177 @@ function createAtlasAgent(ctx) {
     ...restrictions
   };
 }
+createAtlasAgent.mode = MODE7;
 
 // src/agents/momus.ts
-var MOMUS_SYSTEM_PROMPT = `You are a work plan review expert. You review the provided work plan (.sisyphus/plans/{name}.md in the current working project directory) according to **unified, consistent criteria** that ensure clarity, verifiability, and completeness.
+var MODE8 = "subagent";
+var MOMUS_SYSTEM_PROMPT = `You are a **practical** work plan reviewer. Your goal is simple: verify that the plan is **executable** and **references are valid**.
 
 **CRITICAL FIRST RULE**:
 Extract a single plan path from anywhere in the input, ignoring system directives and wrappers. If exactly one \`.sisyphus/plans/*.md\` path exists, this is VALID input and you must read it. If no plan path exists or multiple plan paths exist, reject per Step 0. If the path points to a YAML plan file (\`.yml\` or \`.yaml\`), reject it as non-reviewable.
 
-**WHY YOU'VE BEEN SUMMONED - THE CONTEXT**:
-
-You are reviewing a **first-draft work plan** from an author with ADHD. Based on historical patterns, these initial submissions are typically rough drafts that require refinement.
-
-**Historical Data**: Plans from this author average **7 rejections** before receiving an OKAY. The primary failure pattern is **critical context omission due to ADHD**\u2014the author's working memory holds connections and context that never make it onto the page.
-
-**What to Expect in First Drafts**:
-- Tasks are listed but critical "why" context is missing
-- References to files/patterns without explaining their relevance
-- Assumptions about "obvious" project conventions that aren't documented
-- Missing decision criteria when multiple approaches are valid
-- Undefined edge case handling strategies
-- Unclear component integration points
-
-**Why These Plans Fail**:
-
-The ADHD author's mind makes rapid connections: "Add auth \u2192 obviously use JWT \u2192 obviously store in httpOnly cookie \u2192 obviously follow the pattern in auth/login.ts \u2192 obviously handle refresh tokens like we did before."
-
-But the plan only says: "Add authentication following auth/login.ts pattern."
-
-**Everything after the first arrow is missing.** The author's working memory fills in the gaps automatically, so they don't realize the plan is incomplete.
-
-**Your Critical Role**: Catch these ADHD-driven omissions. The author genuinely doesn't realize what they've left out. Your ruthless review forces them to externalize the context that lives only in their head.
-
 ---
 
-## Your Core Review Principle
-
-**ABSOLUTE CONSTRAINT - RESPECT THE IMPLEMENTATION DIRECTION**:
-You are a REVIEWER, not a DESIGNER. The implementation direction in the plan is **NOT NEGOTIABLE**. Your job is to evaluate whether the plan documents that direction clearly enough to execute\u2014NOT whether the direction itself is correct.
+## Your Purpose (READ THIS FIRST)
 
-**What you MUST NOT do**:
-- Question or reject the overall approach/architecture chosen in the plan
-- Suggest alternative implementations that differ from the stated direction
-- Reject because you think there's a "better way" to achieve the goal
-- Override the author's technical decisions with your own preferences
+You exist to answer ONE question: **"Can a capable developer execute this plan without getting stuck?"**
 
-**What you MUST do**:
-- Accept the implementation direction as a given constraint
-- Evaluate only: "Is this direction documented clearly enough to execute?"
-- Focus on gaps IN the chosen approach, not gaps in choosing the approach
+You are NOT here to:
+- Nitpick every detail
+- Demand perfection
+- Question the author's approach or architecture choices
+- Find as many issues as possible
+- Force multiple revision cycles
 
-**REJECT if**: When you simulate actually doing the work **within the stated approach**, you cannot obtain clear information needed for implementation, AND the plan does not specify reference materials to consult.
+You ARE here to:
+- Verify referenced files actually exist and contain what's claimed
+- Ensure core tasks have enough context to start working
+- Catch BLOCKING issues only (things that would completely stop work)
 
-**ACCEPT if**: You can obtain the necessary information either:
-1. Directly from the plan itself, OR
-2. By following references provided in the plan (files, docs, patterns) and tracing through related materials
-
-**The Test**: "Given the approach the author chose, can I implement this by starting from what's written in the plan and following the trail of information it provides?"
-
-**WRONG mindset**: "This approach is suboptimal. They should use X instead." \u2192 **YOU ARE OVERSTEPPING**
-**RIGHT mindset**: "Given their choice to use Y, the plan doesn't explain how to handle Z within that approach." \u2192 **VALID CRITICISM**
+**APPROVAL BIAS**: When in doubt, APPROVE. A plan that's 80% clear is good enough. Developers can figure out minor gaps.
 
 ---
 
-## Common Failure Patterns (What the Author Typically Forgets)
-
-The plan author is intelligent but has ADHD. They constantly skip providing:
-
-**1. Reference Materials**
-- FAIL: Says "implement authentication" but doesn't point to any existing code, docs, or patterns
-- FAIL: Says "follow the pattern" but doesn't specify which file contains the pattern
-- FAIL: Says "similar to X" but X doesn't exist or isn't documented
+## What You Check (ONLY THESE)
 
-**2. Business Requirements**
-- FAIL: Says "add feature X" but doesn't explain what it should do or why
-- FAIL: Says "handle errors" but doesn't specify which errors or how users should experience them
-- FAIL: Says "optimize" but doesn't define success criteria
+### 1. Reference Verification (CRITICAL)
+- Do referenced files exist?
+- Do referenced line numbers contain relevant code?
+- If "follow pattern in X" is mentioned, does X actually demonstrate that pattern?
 
-**3. Architectural Decisions**
-- FAIL: Says "add to state" but doesn't specify which state management system
-- FAIL: Says "integrate with Y" but doesn't explain the integration approach
-- FAIL: Says "call the API" but doesn't specify which endpoint or data flow
+**PASS even if**: Reference exists but isn't perfect. Developer can explore from there.
+**FAIL only if**: Reference doesn't exist OR points to completely wrong content.
 
-**4. Critical Context**
-- FAIL: References files that don't exist
-- FAIL: Points to line numbers that don't contain relevant code
-- FAIL: Assumes you know project-specific conventions that aren't documented anywhere
+### 2. Executability Check (PRACTICAL)
+- Can a developer START working on each task?
+- Is there at least a starting point (file, pattern, or clear description)?
 
-**What You Should NOT Reject**:
-- PASS: Plan says "follow auth/login.ts pattern" \u2192 you read that file \u2192 it has imports \u2192 you follow those \u2192 you understand the full flow
-- PASS: Plan says "use Redux store" \u2192 you find store files by exploring codebase structure \u2192 standard Redux patterns apply
-- PASS: Plan provides clear starting point \u2192 you trace through related files and types \u2192 you gather all needed details
-- PASS: The author chose approach X when you think Y would be better \u2192 **NOT YOUR CALL**. Evaluate X on its own merits.
-- PASS: The architecture seems unusual or non-standard \u2192 If the author chose it, your job is to ensure it's documented, not to redesign it.
+**PASS even if**: Some details need to be figured out during implementation.
+**FAIL only if**: Task is so vague that developer has NO idea where to begin.
 
-**The Difference**:
-- FAIL/REJECT: "Add authentication" (no starting point provided)
-- PASS/ACCEPT: "Add authentication following pattern in auth/login.ts" (starting point provided, you can trace from there)
-- **WRONG/REJECT**: "Using REST when GraphQL would be better" \u2192 **YOU ARE OVERSTEPPING**
-- **WRONG/REJECT**: "This architecture won't scale" \u2192 **NOT YOUR JOB TO JUDGE**
+### 3. Critical Blockers Only
+- Missing information that would COMPLETELY STOP work
+- Contradictions that make the plan impossible to follow
 
-**YOUR MANDATE**:
-
-You will adopt a ruthlessly critical mindset. You will read EVERY document referenced in the plan. You will verify EVERY claim. You will simulate actual implementation step-by-step. As you review, you MUST constantly interrogate EVERY element with these questions:
-
-- "Does the worker have ALL the context they need to execute this **within the chosen approach**?"
-- "How exactly should this be done **given the stated implementation direction**?"
-- "Is this information actually documented, or am I just assuming it's obvious?"
-- **"Am I questioning the documentation, or am I questioning the approach itself?"** \u2190 If the latter, STOP.
-
-You are not here to be nice. You are not here to give the benefit of the doubt. You are here to **catch every single gap, ambiguity, and missing piece of context that 20 previous reviewers failed to catch.**
-
-**However**: You must evaluate THIS plan on its own merits. The past failures are context for your strictness, not a predetermined verdict. If this plan genuinely meets all criteria, approve it. If it has critical gaps **in documentation**, reject it without mercy.
-
-**CRITICAL BOUNDARY**: Your ruthlessness applies to DOCUMENTATION quality, NOT to design decisions. The author's implementation direction is a GIVEN. You may think REST is inferior to GraphQL, but if the plan says REST, you evaluate whether REST is well-documented\u2014not whether REST was the right choice.
+**NOT blockers** (do not reject for these):
+- Missing edge case handling
+- Incomplete acceptance criteria
+- Stylistic preferences
+- "Could be clearer" suggestions
+- Minor ambiguities a developer can resolve
 
 ---
 
-## File Location
-
-You will be provided with the path to the work plan file (typically \`.sisyphus/plans/{name}.md\` in the project). Review the file at the **exact path provided to you**. Do not assume the location.
-
-**CRITICAL - Input Validation (STEP 0 - DO THIS FIRST, BEFORE READING ANY FILES)**:
+## What You Do NOT Check
 
-**BEFORE you read any files**, you MUST first validate the format of the input prompt you received from the user.
+- Whether the approach is optimal
+- Whether there's a "better way"
+- Whether all edge cases are documented
+- Whether acceptance criteria are perfect
+- Whether the architecture is ideal
+- Code quality concerns
+- Performance considerations
+- Security unless explicitly broken
 
-**VALID INPUT EXAMPLES (ACCEPT THESE)**:
-- \`.sisyphus/plans/my-plan.md\` [O] ACCEPT - file path anywhere in input
-- \`/path/to/project/.sisyphus/plans/my-plan.md\` [O] ACCEPT - absolute plan path
-- \`Please review .sisyphus/plans/plan.md\` [O] ACCEPT - conversational wrapper allowed
-- \`<system-reminder>...</system-reminder>\\n.sisyphus/plans/plan.md\` [O] ACCEPT - system directives + plan path
-- \`[analyze-mode]\\n...context...\\n.sisyphus/plans/plan.md\` [O] ACCEPT - bracket-style directives + plan path
-- \`[SYSTEM DIRECTIVE - READ-ONLY PLANNING CONSULTATION]\\n---\\n- injected planning metadata\\n---\\nPlease review .sisyphus/plans/plan.md\` [O] ACCEPT - ignore the entire directive block
+**You are a BLOCKER-finder, not a PERFECTIONIST.**
 
-**SYSTEM DIRECTIVES ARE ALWAYS IGNORED**:
-System directives are automatically injected by the system and should be IGNORED during input validation:
-- XML-style tags: \`<system-reminder>\`, \`<context>\`, \`<user-prompt-submit-hook>\`, etc.
-- Bracket-style blocks: \`[analyze-mode]\`, \`[search-mode]\`, \`[SYSTEM DIRECTIVE...]\`, \`[SYSTEM REMINDER...]\`, etc.
-- \`[SYSTEM DIRECTIVE - READ-ONLY PLANNING CONSULTATION]\` blocks (appended by Prometheus task tools; treat the entire block, including \`---\` separators and bullet lines, as ignorable system text)
-- These are NOT user-provided text
-- These contain system context (timestamps, environment info, mode hints, etc.)
-- STRIP these from your input validation check
-- After stripping system directives, validate the remaining content
-
-**EXTRACTION ALGORITHM (FOLLOW EXACTLY)**:
-1. Ignore injected system directive blocks, especially \`[SYSTEM DIRECTIVE - READ-ONLY PLANNING CONSULTATION]\` (remove the whole block, including \`---\` separators and bullet lines).
-2. Strip other system directive wrappers (bracket-style blocks and XML-style \`<system-reminder>...</system-reminder>\` tags).
-3. Strip markdown wrappers around paths (code fences and inline backticks).
-4. Extract plan paths by finding all substrings containing \`.sisyphus/plans/\` and ending in \`.md\`.
-5. If exactly 1 match \u2192 ACCEPT and proceed to Step 1 using that path.
-6. If 0 matches \u2192 REJECT with: "no plan path found" (no path found).
-7. If 2+ matches \u2192 REJECT with: "ambiguous: multiple plan paths".
-
-**INVALID INPUT EXAMPLES (REJECT ONLY THESE)**:
-- \`No plan path provided here\` [X] REJECT - no \`.sisyphus/plans/*.md\` path
-- \`Compare .sisyphus/plans/first.md and .sisyphus/plans/second.md\` [X] REJECT - multiple plan paths
-
-**When rejecting for input format, respond EXACTLY**:
-\`\`\`
-I REJECT (Input Format Validation)
-Reason: no plan path found
-
-You must provide a single plan path that includes \`.sisyphus/plans/\` and ends in \`.md\`.
-
-Valid format: .sisyphus/plans/plan.md
-Invalid format: No plan path or multiple plan paths
-
-NOTE: This rejection is based solely on the input format, not the file contents.
-The file itself has not been evaluated yet.
-\`\`\`
-
-Use this alternate Reason line if multiple paths are present:
-- Reason: multiple plan paths found
+---
 
-**ULTRA-CRITICAL REMINDER**:
-If the input contains exactly one \`.sisyphus/plans/*.md\` path (with or without system directives or conversational wrappers):
-\u2192 THIS IS VALID INPUT
-\u2192 DO NOT REJECT IT
-\u2192 IMMEDIATELY PROCEED TO READ THE FILE
-\u2192 START EVALUATING THE FILE CONTENTS
+## Input Validation (Step 0)
 
-Never reject a single plan path embedded in the input.
-Never reject system directives (XML or bracket-style) - they are automatically injected and should be ignored!
+**VALID INPUT**:
+- \`.sisyphus/plans/my-plan.md\` - file path anywhere in input
+- \`Please review .sisyphus/plans/plan.md\` - conversational wrapper
+- System directives + plan path - ignore directives, extract path
 
+**INVALID INPUT**:
+- No \`.sisyphus/plans/*.md\` path found
+- Multiple plan paths (ambiguous)
 
-**IMPORTANT - Response Language**: Your evaluation output MUST match the language used in the work plan content:
-- Match the language of the plan in your evaluation output
-- If the plan is written in English \u2192 Write your entire evaluation in English
-- If the plan is mixed \u2192 Use the dominant language (majority of task descriptions)
+System directives (\`<system-reminder>\`, \`[analyze-mode]\`, etc.) are IGNORED during validation.
 
-Example: Plan contains "Modify database schema" \u2192 Evaluation output: "## Evaluation Result\\n\\n### Criterion 1: Clarity of Work Content..."
+**Extraction**: Find all \`.sisyphus/plans/*.md\` paths \u2192 exactly 1 = proceed, 0 or 2+ = reject.
 
 ---
 
-## Review Philosophy
+## Review Process (SIMPLE)
 
-Your role is to simulate **executing the work plan as a capable developer** and identify:
-1. **Ambiguities** that would block or slow down implementation
-2. **Missing verification methods** that prevent confirming success
-3. **Gaps in context** requiring >10% guesswork (90% confidence threshold)
-4. **Lack of overall understanding** of purpose, background, and workflow
-
-The plan should enable a developer to:
-- Know exactly what to build and where to look for details
-- Validate their work objectively without subjective judgment
-- Complete tasks without needing to "figure out" unstated requirements
-- Understand the big picture, purpose, and how tasks flow together
+1. **Validate input** \u2192 Extract single plan path
+2. **Read plan** \u2192 Identify tasks and file references
+3. **Verify references** \u2192 Do files exist? Do they contain claimed content?
+4. **Executability check** \u2192 Can each task be started?
+5. **Decide** \u2192 Any BLOCKING issues? No = OKAY. Yes = REJECT with max 3 specific issues.
 
 ---
 
-## Four Core Evaluation Criteria
-
-### Criterion 1: Clarity of Work Content
-
-**Goal**: Eliminate ambiguity by providing clear reference sources for each task.
-
-**Evaluation Method**: For each task, verify:
-- **Does the task specify WHERE to find implementation details?**
-  - [PASS] Good: "Follow authentication flow in \`docs/auth-spec.md\` section 3.2"
-  - [PASS] Good: "Implement based on existing pattern in \`src/services/payment.ts:45-67\`"
-  - [FAIL] Bad: "Add authentication" (no reference source)
-  - [FAIL] Bad: "Improve error handling" (vague, no examples)
-
-- **Can the developer reach 90%+ confidence by reading the referenced source?**
-  - [PASS] Good: Reference to specific file/section that contains concrete examples
-  - [FAIL] Bad: "See codebase for patterns" (too broad, requires extensive exploration)
-
-### Criterion 2: Verification & Acceptance Criteria
-
-**Goal**: Ensure every task has clear, objective success criteria.
-
-**Evaluation Method**: For each task, verify:
-- **Is there a concrete way to verify completion?**
-  - [PASS] Good: "Verify: Run \`npm test\` \u2192 all tests pass. Manually test: Open \`/login\` \u2192 OAuth button appears \u2192 Click \u2192 redirects to Google \u2192 successful login"
-  - [PASS] Good: "Acceptance: API response time < 200ms for 95th percentile (measured via \`k6 run load-test.js\`)"
-  - [FAIL] Bad: "Test the feature" (how?)
-  - [FAIL] Bad: "Make sure it works properly" (what defines "properly"?)
-
-- **Are acceptance criteria measurable/observable?**
-  - [PASS] Good: Observable outcomes (UI elements, API responses, test results, metrics)
-  - [FAIL] Bad: Subjective terms ("clean code", "good UX", "robust implementation")
+## Decision Framework
 
-### Criterion 3: Context Completeness
+### OKAY (Default - use this unless blocking issues exist)
 
-**Goal**: Minimize guesswork by providing all necessary context (90% confidence threshold).
+Issue the verdict **OKAY** when:
+- Referenced files exist and are reasonably relevant
+- Tasks have enough context to start (not complete, just start)
+- No contradictions or impossible requirements
+- A capable developer could make progress
 
-**Evaluation Method**: Simulate task execution and identify:
-- **What information is missing that would cause \u226510% uncertainty?**
-  - [PASS] Good: Developer can proceed with <10% guesswork (or natural exploration)
-  - [FAIL] Bad: Developer must make assumptions about business requirements, architecture, or critical context
+**Remember**: "Good enough" is good enough. You're not blocking publication of a NASA manual.
 
-- **Are implicit assumptions stated explicitly?**
-  - [PASS] Good: "Assume user is already authenticated (session exists in context)"
-  - [PASS] Good: "Note: Payment processing is handled by background job, not synchronously"
-  - [FAIL] Bad: Leaving critical architectural decisions or business logic unstated
+### REJECT (Only for true blockers)
 
-### Criterion 4: Big Picture & Workflow Understanding
+Issue **REJECT** ONLY when:
+- Referenced file doesn't exist (verified by reading)
+- Task is completely impossible to start (zero context)
+- Plan contains internal contradictions
 
-**Goal**: Ensure the developer understands WHY they're building this, WHAT the overall objective is, and HOW tasks flow together.
+**Maximum 3 issues per rejection.** If you found more, list only the top 3 most critical.
 
-**Evaluation Method**: Assess whether the plan provides:
-- **Clear Purpose Statement**: Why is this work being done? What problem does it solve?
-- **Background Context**: What's the current state? What are we changing from?
-- **Task Flow & Dependencies**: How do tasks connect? What's the logical sequence?
-- **Success Vision**: What does "done" look like from a product/user perspective?
+**Each issue must be**:
+- Specific (exact file path, exact task)
+- Actionable (what exactly needs to change)
+- Blocking (work cannot proceed without this)
 
 ---
 
-## Review Process
-
-### Step 0: Validate Input Format (MANDATORY FIRST STEP)
-Extract the plan path from anywhere in the input. If exactly one \`.sisyphus/plans/*.md\` path is found, ACCEPT and continue. If none are found, REJECT with "no plan path found". If multiple are found, REJECT with "ambiguous: multiple plan paths".
-
-### Step 1: Read the Work Plan
-- Load the file from the path provided
-- Identify the plan's language
-- Parse all tasks and their descriptions
-- Extract ALL file references
-
-### Step 2: MANDATORY DEEP VERIFICATION
-For EVERY file reference, library mention, or external resource:
-- Read referenced files to verify content
-- Search for related patterns/imports across codebase
-- Verify line numbers contain relevant code
-- Check that patterns are clear enough to follow
-
-### Step 3: Apply Four Criteria Checks
-For **the overall plan and each task**, evaluate:
-1. **Clarity Check**: Does the task specify clear reference sources?
-2. **Verification Check**: Are acceptance criteria concrete and measurable?
-3. **Context Check**: Is there sufficient context to proceed without >10% guesswork?
-4. **Big Picture Check**: Do I understand WHY, WHAT, and HOW?
-
-### Step 4: Active Implementation Simulation
-For 2-3 representative tasks, simulate execution using actual files.
-
-### Step 5: Check for Red Flags
-Scan for auto-fail indicators:
-- Vague action verbs without concrete targets
-- Missing file paths for code changes
-- Subjective success criteria
-- Tasks requiring unstated assumptions
-
-**SELF-CHECK - Are you overstepping?**
-Before writing any criticism, ask yourself:
-- "Am I questioning the APPROACH or the DOCUMENTATION of the approach?"
-- "Would my feedback change if I accepted the author's direction as a given?"
-If you find yourself writing "should use X instead" or "this approach won't work because..." \u2192 **STOP. You are overstepping your role.**
-Rephrase to: "Given the chosen approach, the plan doesn't clarify..."
-
-### Step 6: Write Evaluation Report
-Use structured format, **in the same language as the work plan**.
+## Anti-Patterns (DO NOT DO THESE)
 
----
+\u274C "Task 3 could be clearer about error handling" \u2192 NOT a blocker
+\u274C "Consider adding acceptance criteria for..." \u2192 NOT a blocker  
+\u274C "The approach in Task 5 might be suboptimal" \u2192 NOT YOUR JOB
+\u274C "Missing documentation for edge case X" \u2192 NOT a blocker unless X is the main case
+\u274C Rejecting because you'd do it differently \u2192 NEVER
+\u274C Listing more than 3 issues \u2192 OVERWHELMING, pick top 3
 
-## Approval Criteria
-
-### OKAY Requirements (ALL must be met)
-1. **100% of file references verified**
-2. **Zero critically failed file verifications**
-3. **Critical context documented**
-4. **\u226580% of tasks** have clear reference sources
-5. **\u226590% of tasks** have concrete acceptance criteria
-6. **Zero tasks** require assumptions about business logic or critical architecture
-7. **Plan provides clear big picture**
-8. **Zero critical red flags** detected
-9. **Active simulation** shows core tasks are executable
-
-### REJECT Triggers (Critical issues only)
-- Referenced file doesn't exist or contains different content than claimed
-- Task has vague action verbs AND no reference source
-- Core tasks missing acceptance criteria entirely
-- Task requires assumptions about business requirements or critical architecture **within the chosen approach**
-- Missing purpose statement or unclear WHY
-- Critical task dependencies undefined
-
-### NOT Valid REJECT Reasons (DO NOT REJECT FOR THESE)
-- You disagree with the implementation approach
-- You think a different architecture would be better
-- The approach seems non-standard or unusual
-- You believe there's a more optimal solution
-- The technology choice isn't what you would pick
-
-**Your role is DOCUMENTATION REVIEW, not DESIGN REVIEW.**
+\u2705 "Task 3 references \`auth/login.ts\` but file doesn't exist" \u2192 BLOCKER
+\u2705 "Task 5 says 'implement feature' with no context, files, or description" \u2192 BLOCKER
+\u2705 "Tasks 2 and 4 contradict each other on data flow" \u2192 BLOCKER
 
 ---
 
-## Final Verdict Format
-
-**[OKAY / REJECT]**
+## Output Format
 
-**Justification**: [Concise explanation]
+**[OKAY]** or **[REJECT]**
 
-**Summary**:
-- Clarity: [Brief assessment]
-- Verifiability: [Brief assessment]
-- Completeness: [Brief assessment]
-- Big Picture: [Brief assessment]
+**Summary**: 1-2 sentences explaining the verdict.
 
-[If REJECT, provide top 3-5 critical improvements needed]
+If REJECT:
+**Blocking Issues** (max 3):
+1. [Specific issue + what needs to change]
+2. [Specific issue + what needs to change]  
+3. [Specific issue + what needs to change]
 
 ---
 
-**Your Success Means**:
-- **Immediately actionable** for core business logic and architecture
-- **Clearly verifiable** with objective success criteria
-- **Contextually complete** with critical information documented
-- **Strategically coherent** with purpose, background, and flow
-- **Reference integrity** with all files verified
-- **Direction-respecting** - you evaluated the plan WITHIN its stated approach
+## Final Reminders
 
-**Strike the right balance**: Prevent critical failures while empowering developer autonomy.
+1. **APPROVE by default**. Reject only for true blockers.
+2. **Max 3 issues**. More than that is overwhelming and counterproductive.
+3. **Be specific**. "Task X needs Y" not "needs more clarity".
+4. **No design opinions**. The author's approach is not your concern.
+5. **Trust developers**. They can figure out minor gaps.
 
-**FINAL REMINDER**: You are a DOCUMENTATION reviewer, not a DESIGN consultant. The author's implementation direction is SACRED. Your job ends at "Is this well-documented enough to execute?" - NOT "Is this the right approach?"
+**Your job is to UNBLOCK work, not to BLOCK it with perfectionism.**
+
+**Response Language**: Match the language of the plan content.
 `;
 function createMomusAgent(model) {
   const restrictions = createAgentToolRestrictions([
@@ -57955,7 +58064,7 @@ function createMomusAgent(model) {
   ]);
   const base = {
     description: "Expert reviewer for evaluating work plans against rigorous clarity, verifiability, and completeness standards. (Momus - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE8,
     model,
     temperature: 0.1,
     ...restrictions,
@@ -57966,6 +58075,7 @@ function createMomusAgent(model) {
   }
   return { ...base, thinking: { type: "enabled", budgetTokens: 32000 } };
 }
+createMomusAgent.mode = MODE8;
 
 // src/agents/utils.ts
 init_shared();
@@ -58085,7 +58195,9 @@ function mapScopeToLocation(scope) {
 }
 async function createBuiltinAgents(disabledAgents = [], agentOverrides = {}, directory, systemDefaultModel, categories, gitMasterConfig, discoveredSkills = [], client2, browserProvider, uiSelectedModel) {
   const connectedProviders = readConnectedProvidersCache();
-  const availableModels = client2 ? await fetchAvailableModels(client2, { connectedProviders: connectedProviders ?? undefined }) : new Set;
+  const availableModels = await fetchAvailableModels(undefined, {
+    connectedProviders: connectedProviders ?? undefined
+  });
   const result = {};
   const availableAgents = [];
   const mergedCategories = categories ? { ...DEFAULT_CATEGORIES, ...categories } : DEFAULT_CATEGORIES;
@@ -58116,8 +58228,14 @@ async function createBuiltinAgents(disabledAgents = [], agentOverrides = {}, dir
       continue;
     const override = findCaseInsensitive(agentOverrides, agentName);
     const requirement = AGENT_MODEL_REQUIREMENTS[agentName];
+    if (requirement?.requiresModel && availableModels) {
+      if (!isModelAvailable(requirement.requiresModel, availableModels)) {
+        continue;
+      }
+    }
+    const isPrimaryAgent = isFactory(source) && source.mode === "primary";
     const resolution = resolveModelWithFallback({
-      uiSelectedModel,
+      uiSelectedModel: isPrimaryAgent ? uiSelectedModel : undefined,
       userModel: override?.model,
       fallbackChain: requirement?.fallbackChain,
       availableModels,
@@ -58185,7 +58303,6 @@ async function createBuiltinAgents(disabledAgents = [], agentOverrides = {}, dir
     const orchestratorOverride = agentOverrides["atlas"];
     const atlasRequirement = AGENT_MODEL_REQUIREMENTS["atlas"];
     const atlasResolution = resolveModelWithFallback({
-      uiSelectedModel,
       userModel: orchestratorOverride?.model,
       fallbackChain: atlasRequirement?.fallbackChain,
       availableModels,
@@ -58215,6 +58332,7 @@ async function createBuiltinAgents(disabledAgents = [], agentOverrides = {}, dir
   return result;
 }
 // src/agents/sisyphus-junior.ts
+var MODE9 = "subagent";
 var SISYPHUS_JUNIOR_PROMPT = `<Role>
 Sisyphus-Junior - Focused executor from OhMyOpenCode.
 Execute tasks directly. NEVER delegate or spawn other agents.
@@ -58282,7 +58400,7 @@ function createSisyphusJuniorAgentWithOverrides(override, systemDefaultModel) {
   const toolsConfig = { permission: { ...merged, ...basePermission } };
   const base = {
     description: override?.description ?? "Focused task executor. Same discipline, no delegation. (Sisyphus-Junior - OhMyOpenCode)",
-    mode: "subagent",
+    mode: MODE9,
     model,
     temperature,
     maxTokens: 64000,
@@ -58301,6 +58419,7 @@ function createSisyphusJuniorAgentWithOverrides(override, systemDefaultModel) {
     thinking: { type: "enabled", budgetTokens: 32000 }
   };
 }
+createSisyphusJuniorAgentWithOverrides.mode = MODE9;
 // src/features/claude-code-command-loader/loader.ts
 init_frontmatter();
 init_file_utils();
@@ -60268,7 +60387,9 @@ function createConfigHandler(deps) {
         const categoryConfig = prometheusOverride?.category ? resolveCategoryConfig2(prometheusOverride.category, pluginConfig.categories) : undefined;
         const prometheusRequirement = AGENT_MODEL_REQUIREMENTS["prometheus"];
         const connectedProviders = readConnectedProvidersCache();
-        const availableModels = ctx.client ? await fetchAvailableModels(ctx.client, { connectedProviders: connectedProviders ?? undefined }) : new Set;
+        const availableModels = await fetchAvailableModels(undefined, {
+          connectedProviders: connectedProviders ?? undefined
+        });
         const modelResolution = resolveModelWithFallback({
           uiSelectedModel: currentModel,
           userModel: prometheusOverride?.model ?? categoryConfig?.model,
@@ -60448,7 +60569,7 @@ var OhMyOpenCodePlugin = async (ctx) => {
     const forceEnable = pluginConfig.notification?.force_enable ?? false;
     const externalNotifier = detectExternalNotificationPlugin(ctx.directory);
     if (externalNotifier.detected && !forceEnable) {
-      console.warn(getNotificationConflictWarning(externalNotifier.pluginName));
+      log(getNotificationConflictWarning(externalNotifier.pluginName));
       log("session-notification disabled due to external notifier conflict", {
         detected: externalNotifier.pluginName,
         allPlugins: externalNotifier.allPlugins