fix(theme): make graphite the explicit default across startup modes (#200)

Author: elucid

src/core/config.test.ts

@@ -86,7 +86,7 @@ describe("config resolution", () => {
     });
   });
 
-  test("falls back to the global config path outside a repo", () => {
+  test("defaults unspecified themes to graphite, including piped pager-style patch input", () => {
     const home = createTempDir("hunk-config-home-");
     const cwd = createTempDir("hunk-config-cwd-");
 
@@ -96,6 +96,7 @@ describe("config resolution", () => {
     });
 
     expect(resolved.repoConfigPath).toBeUndefined();
+    expect(resolved.input.options.theme).toBe("graphite");
   });
 
   test("command-specific config sections also apply to show mode", () => {
@@ -196,4 +197,28 @@ describe("config resolution", () => {
     expect(bootstrap.initialShowHunkHeaders).toBe(false);
     expect(bootstrap.initialShowAgentNotes).toBe(true);
   });
+
+  test("loadAppBootstrap exposes graphite when no theme is configured", async () => {
+    const home = createTempDir("hunk-config-home-");
+    const repo = createTempDir("hunk-config-repo-");
+    createRepo(repo);
+
+    const before = join(repo, "before.ts");
+    const after = join(repo, "after.ts");
+    writeFileSync(before, "export const alpha = 1;\n");
+    writeFileSync(after, "export const alpha = 2;\n");
+
+    const resolved = resolveConfiguredCliInput(
+      {
+        kind: "diff",
+        left: before,
+        right: after,
+        options: {},
+      },
+      { cwd: repo, env: { HOME: home } },
+    );
+    const bootstrap = await loadAppBootstrap(resolved.input);
+
+    expect(bootstrap.initialTheme).toBe("graphite");
+  });
 });

src/core/config.ts

@@ -131,7 +131,9 @@ export function resolveConfiguredCliInput(
 
   let resolvedOptions: CommonOptions = {
     mode: DEFAULT_VIEW_PREFERENCES.mode,
-    theme: undefined,
+    // Keep the built-in theme default explicit so stdin-backed startup paths do not depend on
+    // renderer theme-mode detection for their initial palette.
+    theme: "graphite",
     agentContext: input.options.agentContext,
     pager: input.options.pager ?? false,
     watch: input.options.watch ?? false,

src/ui/lib/ui-lib.test.ts

@@ -359,13 +359,13 @@ describe("ui helpers", () => {
     ).toBe(16);
   });
 
-  test("resolveTheme falls back by requested id and renderer mode while lazily exposing syntax styles", () => {
+  test("resolveTheme falls back by requested id to graphite while lazily exposing syntax styles", () => {
     const midnight = resolveTheme("midnight", null);
     const missingLight = resolveTheme("missing", "light");
     const missingDark = resolveTheme("missing", "dark");
 
     expect(midnight.id).toBe("midnight");
-    expect(missingLight.id).toBe("paper");
+    expect(missingLight.id).toBe("graphite");
     expect(missingDark.id).toBe("graphite");
     expect(resolveTheme("ember", null).syntaxStyle).toBeDefined();
   });

src/ui/themes.ts

@@ -286,16 +286,12 @@ export const THEMES: AppTheme[] = [
   ),
 ];
 
-/** Resolve a named theme or fall back to a theme that matches the renderer mode. */
-export function resolveTheme(requested: string | undefined, themeMode: ThemeMode | null) {
+/** Resolve a named theme or fall back to Hunk's explicit built-in default. */
+export function resolveTheme(requested: string | undefined, _themeMode: ThemeMode | null) {
   const exact = THEMES.find((theme) => theme.id === requested);
   if (exact) {
     return exact;
   }
 
-  if (themeMode === "light") {
-    return THEMES.find((theme) => theme.id === "paper") ?? THEMES[0]!;
-  }
-
   return THEMES.find((theme) => theme.id === "graphite") ?? THEMES[0]!;
 }