free-coding-models 0.3.16 → 0.3.18

package/src/tool-launchers.js

@@ -43,6 +43,7 @@ import { getApiKey } from './config.js'
 import { ENV_VAR_NAMES, isWindows } from './provider-metadata.js'
 import { getToolMeta } from './tool-metadata.js'
 import { PROVIDER_METADATA } from './provider-metadata.js'
+import { resolveToolBinaryPath } from './tool-bootstrap.js'
 
 const OPENAI_COMPAT_ENV_KEYS = [
   'OPENAI_API_KEY',
@@ -127,6 +128,10 @@ function applyOpenAiCompatEnv(env, apiKey, baseUrl, modelId) {
   return env
 }
 
+function resolveLaunchCommand(mode, fallbackCommand) {
+  return resolveToolBinaryPath(mode) || fallbackCommand
+}
+
 /**
  * 📖 resolveLauncherModelId returns the provider-native id used by the direct
  * 📖 launchers. Legacy bridge-specific model remapping has been removed.
@@ -562,35 +567,35 @@ export async function startExternalTool(mode, model, config) {
   printConfigArtifacts(meta.label, launchPlan.configArtifacts)
 
   if (mode === 'aider') {
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'crush') {
     console.log(chalk.dim('  📖 Crush will use the provider directly for this launch.'))
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'goose') {
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'qwen') {
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'openhands') {
     console.log(chalk.dim(`  📖 OpenHands launched with model: ${model.modelId}`))
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'amp') {
     console.log(chalk.dim(`  📖 Amp config updated with model: ${model.modelId}`))
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   if (mode === 'pi') {
     // 📖 Pi supports --provider and --model flags for guaranteed auto-selection
-    return spawnCommand(launchPlan.command, launchPlan.args, launchPlan.env)
+    return spawnCommand(resolveLaunchCommand(mode, launchPlan.command), launchPlan.args, launchPlan.env)
   }
 
   console.log(chalk.red(`  X Unsupported external tool mode: ${mode}`))

package/src/ui-config.js

@@ -1,49 +1,42 @@
 /**
  * @file ui-config.js
- * @description Central configuration for TUI visual styling.
+ * @description Central configuration helpers for TUI separators and spacing.
  * 
  * @details
- * This module centralizes all visual styling constants for the TUI interface.
- * By keeping colors, separators, and spacing in one place, it becomes easy to
- * customize the look and feel without modifying rendering logic.
+ * This module centralizes the shared table separators used by the TUI. The
+ * theme can change at runtime, so separators must be generated lazily instead
+ * of frozen once at import time.
  * 
  * 📖 Configuration:
- * - BORDER_COLOR: Color of column separators (vertical bars)
- * - BORDER_STYLE: Style of separators (dim, bold, etc.)
- * - HORIZONTAL_SEPARATOR: Character used for horizontal lines
- * - HORIZONTAL_STYLE: Style of horizontal lines
- * - COLUMN_SPACING: Space between columns
+ * - `getVerticalSeparator()` — theme-aware vertical divider
+ * - `getHorizontalLine()` — theme-aware horizontal divider
+ * - `getColumnSpacing()` — formatted spacing wrapper around the divider
  * 
  * @see render-table.js - uses these constants for rendering
  * @see tier-colors.js - for tier-specific color definitions
  */
 
-import chalk from 'chalk';
+import { themeColors } from './theme.js'
 
-// 📖 Column separator (vertical bar) configuration
-export const BORDER_COLOR = chalk.rgb(255, 140, 0); // Gentle dark orange
-export const BORDER_STYLE = 'dim'; // Options: 'dim', 'bold', 'underline', 'inverse', etc.
-export const VERTICAL_SEPARATOR = BORDER_COLOR[BORDER_STYLE]('│');
+// 📖 Column separator stays subtle so it improves scanability without turning
+// 📖 the table into a bright fence.
+export function getVerticalSeparator() {
+  return themeColors.border('│')
+}
 
-// 📖 Horizontal separator configuration
-export const HORIZONTAL_SEPARATOR = '─'; // Unicode horizontal line
-export const HORIZONTAL_STYLE = 'dim'; // Options: 'dim', 'bold', etc.
-export const HORIZONTAL_LINE = chalk[HORIZONTAL_STYLE](HORIZONTAL_SEPARATOR);
+export function getHorizontalLine() {
+  return themeColors.dim('─')
+}
 
-// 📖 Column spacing configuration
-export const COLUMN_SPACING = ` ${VERTICAL_SEPARATOR} `; // Space around vertical separator
+export function getColumnSpacing() {
+  return ` ${getVerticalSeparator()} `
+}
 
-// 📖 Optional: Add more UI styling constants here as needed
-export const TABLE_PADDING = 1; // Padding around table edges
+export const TABLE_PADDING = 1
 
-// 📖 Export all constants for easy import
 export default {
-  BORDER_COLOR,
-  BORDER_STYLE,
-  VERTICAL_SEPARATOR,
-  HORIZONTAL_SEPARATOR,
-  HORIZONTAL_STYLE,
-  HORIZONTAL_LINE,
-  COLUMN_SPACING,
+  getVerticalSeparator,
+  getHorizontalLine,
+  getColumnSpacing,
   TABLE_PADDING
-};
+}