fix: use || instead of ?? for binary path fallback to handle empty string default

Author: cursoragent

apps/server/src/git/Layers/ClaudeTextGeneration.ts

@@ -95,7 +95,7 @@ const makeClaudeTextGeneration = Effect.gen(function* () {
 
       const runClaudeCommand = Effect.gen(function* () {
         const command = ChildProcess.make(
-          claudeSettings?.binaryPath ?? "claude",
+          claudeSettings?.binaryPath || "claude",
           [
             "-p",
             "--output-format",

apps/server/src/git/Layers/CodexTextGeneration.ts

@@ -153,7 +153,7 @@ const makeCodexTextGeneration = Effect.gen(function* () {
         const reasoningEffort =
           modelSelection.options?.reasoningEffort ?? CODEX_GIT_TEXT_GENERATION_REASONING_EFFORT;
         const command = ChildProcess.make(
-          codexSettings?.binaryPath ?? "codex",
+          codexSettings?.binaryPath || "codex",
           [
             "exec",
             "--ephemeral",