feat: Improve theme not found handling

Modify  to return a boolean instead of throwing an error when a theme is not found. Update CLI startup and  hook to handle the boolean return value for more graceful error handling.

1 files changed, 2 insertions, 15 deletions

diff --git a/packages/cli/src/gemini.ts b/packages/cli/src/gemini.ts
index 32e34c20..1f48a99b 100644
--- a/packages/cli/src/gemini.ts
+++ b/packages/cli/src/gemini.ts
@@ -24,23 +24,10 @@ async function main() {
   const settings = loadSettings(process.cwd());
   const config = await loadCliConfig(settings.merged);
   if (settings.merged.theme) {
-    try {
-      themeManager.setActiveTheme(settings.merged.theme);
-    } catch (error: unknown) {
+    if (!themeManager.setActiveTheme(settings.merged.theme)) {
       // If the theme is not found during initial load, log a warning and continue.
       // The useThemeCommand hook in App.tsx will handle opening the dialog.
-      if (
-        error instanceof Error &&
-        error.message.includes('Theme') &&
-        error.message.includes('not found')
-      ) {
-        console.warn(
-          `Warning: ${error instanceof Error ? error.message : String(error)}`,
-        );
-      } else {
-        // Re-throw other errors to be caught by the main catch block
-        throw error;
-      }
+      console.warn(`Warning: Theme "${settings.merged.theme}" not found.`);
     }
   }