From babbf5970231db1b932b4b331aea85a56e31e761 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 11 May 2026 12:10:57 +0000
Subject: [PATCH 1/2] feat: show and edit agent Path in Agent management UI

Agent-Logs-Url: https://github.com/unbug/tday/sessions/e4c43ea7-9485-4ba8-972c-186a856acf18

Co-authored-by: unbug <799578+unbug@users.noreply.github.com>
---
 apps/desktop/src/main/index.ts                |  7 +-
 .../renderer/src/Settings/AgentsSection.tsx   | 64 ++++++++++++++++++-
 packages/shared/src/index.ts                  |  5 ++
 3 files changed, 73 insertions(+), 3 deletions(-)

diff --git a/apps/desktop/src/main/index.ts b/apps/desktop/src/main/index.ts
index fe44be5..5be3445 100644
--- a/apps/desktop/src/main/index.ts
+++ b/apps/desktop/src/main/index.ts
@@ -170,8 +170,9 @@ function registerIpc(): void {
     const out: AgentInfo[] = [];
     for (const [id, spec] of Object.entries(INSTALL_SPECS) as Array<[AgentId, AgentInstallSpec | undefined]>) {
       const settings = agents.agents?.[id] ?? {};
-      const bin = settings.bin ?? spec?.bin ?? id;
-      const detect = id === 'pi' ? PiAdapter.detect(bin) : detectGeneric(bin);
+      const effectiveBin = settings.bin ?? spec?.bin ?? id;
+      const detect = id === 'pi' ? PiAdapter.detect(effectiveBin) : detectGeneric(effectiveBin);
+      const resolved = resolveExecutable(effectiveBin, process.env);
       out.push({
         id,
         displayName: spec?.displayName ?? id,
@@ -181,6 +182,8 @@ function registerIpc(): void {
         providerId: settings.providerId,
         model: settings.model,
         isDefault: id === defaultId,
+        bin: settings.bin,
+        resolvedPath: resolved.resolved,
       });
     }
     return out;
diff --git a/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx b/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
index da7544e..6b508be 100644
--- a/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
+++ b/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
@@ -13,7 +13,11 @@ function buildAgentsCfg(agentList: AgentInfo[]): AgentsConfig {
     agents: Object.fromEntries(
       agentList.map((a) => [
         a.id,
-        { providerId: a.providerId || undefined, model: a.model || undefined },
+        {
+          bin: a.bin || undefined,
+          providerId: a.providerId || undefined,
+          model: a.model || undefined,
+        },
       ]),
     ) as AgentsConfig['agents'],
   };
@@ -88,6 +92,27 @@ export function AgentsSection({
     }
   };
 
+  const setAgentBin = (agentId: string, bin: string) => {
+    const next = agents.map((a) => (a.id === agentId ? { ...a, bin: bin || undefined } : a));
+    onAgentsChange(next);
+  };
+
+  const flushAgentBin = () => persistAgents(agents);
+
+  const resetAgentBin = (agentId: string) => {
+    const next = agents.map((a) => (a.id === agentId ? { ...a, bin: undefined } : a));
+    onAgentsChange(next);
+    persistAgents(next);
+  };
+
+  const browseAgentBin = async (agentId: string) => {
+    const picked = await window.tday.pickFile();
+    if (!picked) return;
+    const next = agents.map((a) => (a.id === agentId ? { ...a, bin: picked } : a));
+    onAgentsChange(next);
+    persistAgents(next);
+  };
+
   const installAgent = async (agentId: string, action: 'install' | 'update' | 'uninstall') => {
     setInstallingId(agentId);
     setInstallPct(0);
@@ -288,6 +313,43 @@ export function AgentsSection({
                 </label>
               </div>
 
+              {/* Path */}
+              <div>
+                <span className="mb-1 block text-[10px] uppercase tracking-wider text-zinc-500">
+                  Path
+                </span>
+                <div className="flex items-center gap-1.5">
+                  <input
+                    className="input flex-1 font-mono text-[11px]"
+                    placeholder={a.resolvedPath ?? ''}
+                    value={a.bin ?? ''}
+                    onChange={(e) => setAgentBin(a.id, e.target.value)}
+                    onBlur={() => flushAgentBin()}
+                  />
+                  <button
+                    onClick={() => void browseAgentBin(a.id)}
+                    className="shrink-0 rounded-md border border-zinc-700 px-2 py-1 text-[11px] text-zinc-300 hover:bg-zinc-800"
+                    title="Browse for executable"
+                  >
+                    …
+                  </button>
+                  {a.bin ? (
+                    <button
+                      onClick={() => resetAgentBin(a.id)}
+                      className="shrink-0 rounded-md border border-zinc-800 px-2 py-1 text-[11px] text-zinc-500 hover:bg-zinc-800 hover:text-zinc-300"
+                      title="Reset to default path"
+                    >
+                      Reset
+                    </button>
+                  ) : null}
+                </div>
+                {a.resolvedPath ? (
+                  <div className="mt-0.5 truncate text-[10px] text-zinc-600" title={a.resolvedPath}>
+                    {a.resolvedPath}
+                  </div>
+                ) : null}
+              </div>
+
               {/* Default for new tabs */}
               <div className="flex items-center justify-end border-b border-zinc-800/40 pb-3">
                 <label className="flex cursor-pointer items-center gap-2 text-[11px] text-zinc-400">
diff --git a/packages/shared/src/index.ts b/packages/shared/src/index.ts
index 1d1a276..339b673 100644
--- a/packages/shared/src/index.ts
+++ b/packages/shared/src/index.ts
@@ -179,6 +179,11 @@ export interface AgentInfo {
   model?: string;
   /** True if this agent is configured as the default for new tabs. */
   isDefault?: boolean;
+  /** User-configured executable name or absolute path override (agents.json bin field).
+   *  Undefined when using the built-in default binary name. */
+  bin?: string;
+  /** Resolved absolute path of the effective executable (null when not found on PATH). */
+  resolvedPath?: string | null;
 }
 
 /** Static config loaded from ~/.tday/providers.json (v0.1.0). */

From bb63c776b3195e163fec0579e91edf771e83e275 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 11 May 2026 12:12:35 +0000
Subject: [PATCH 2/2] fix: address code review feedback - trim bin input,
 improve path hint readability

Agent-Logs-Url: https://github.com/unbug/tday/sessions/e4c43ea7-9485-4ba8-972c-186a856acf18

Co-authored-by: unbug <799578+unbug@users.noreply.github.com>
---
 apps/desktop/src/renderer/src/Settings/AgentsSection.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx b/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
index 6b508be..0d40e31 100644
--- a/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
+++ b/apps/desktop/src/renderer/src/Settings/AgentsSection.tsx
@@ -93,7 +93,7 @@ export function AgentsSection({
   };
 
   const setAgentBin = (agentId: string, bin: string) => {
-    const next = agents.map((a) => (a.id === agentId ? { ...a, bin: bin || undefined } : a));
+    const next = agents.map((a) => (a.id === agentId ? { ...a, bin: bin.trim() || undefined } : a));
     onAgentsChange(next);
   };
 
@@ -344,7 +344,7 @@ export function AgentsSection({
                   ) : null}
                 </div>
                 {a.resolvedPath ? (
-                  <div className="mt-0.5 truncate text-[10px] text-zinc-600" title={a.resolvedPath}>
+                  <div className="mt-0.5 truncate text-[11px] text-zinc-500" title={a.resolvedPath}>
                     {a.resolvedPath}
                   </div>
                 ) : null}