@kendoo.agentdesk/agentdesk 0.20.2 → 0.20.4

package/CHANGELOG.md

@@ -13,6 +13,13 @@ Internal refactors, infrastructure changes, and architectural notes are not list
 ### Added
 - `[UI]` Private-session sharing flow. When someone visits a session URL they don't own, they now see a friendly "Shh… this one's private" page with a one-click "Request access" button instead of a blank error. The session owner sees pending requests in a new header inbox and can grant viewer access for just that session or the whole project. Viewer-granted sessions show up in the teammate's sidebar tagged as a viewer.
 
+## [0.20.3] — 2026-04-19
+
+### Fixed
+- `[CLI]` Critical: `loadConfig` no longer lets a server row with `null` or missing fields override a fully-configured local `.agentdesk.json`. The merge now treats `null`/`undefined` from the server as "no value known" — only explicit non-null server values win. Previously a partially-synced project row on agentdesk.live would silently strip local tracker/repo/badge fields during the merge, and the cached-back write then persisted the stripped state to disk. On the next `init` run, the config looked empty even though the user had typed everything in.
+- `[CLI]` Existing-mode save in `init` preserves the full local `.agentdesk.json` — it only ensures `projectKey` is set, never replaces other fields. The previous behavior wrote `{ "projectKey": "..." }` as the entire file, which destroyed any data not also on the server.
+- `[CLI]` Auto-heal sync: when `loadConfig` sees a server row missing fields the local file has, it now pushes the merged config back to the server (fire-and-forget) so both sides converge. Projects that were set up before server-settings-push landed will heal themselves on next init.
+
 ## [0.20.2] — 2026-04-19
 
 ### Changed

package/cli/config.mjs

@@ -87,20 +87,32 @@ export async function loadConfig(dir, opts = {}) {
     }
   }
 
-  // Merge: DEFAULTS ← local ← server (UI is the single source of truth)
+  // Merge: DEFAULTS ← local ← server, where the "server overrides local"
+  // rule only kicks in when the server actually has a value. Null /
+  // undefined on the server means "the server doesn't know about this
+  // field yet" — local wins. This is what keeps a partially-configured
+  // server row from silently wiping a fully-configured local file on
+  // next read.
   let config = { ...DEFAULTS };
   if (localConfig) config = deepMerge(config, localConfig);
-  if (serverConfig) config = deepMerge(config, serverConfig);
-
-  // First-time sync: if local exists but server is empty, push local up.
-  if (localConfig && !serverConfig && apiKey && serverUrl && projectName) {
-    pushConfig(apiKey, serverUrl, projectName, localConfig).catch(() => {});
+  if (serverConfig) config = mergeNonNull(config, serverConfig);
+
+  // Auto-heal sync: if local has fields the server is missing (either
+  // no row at all or a partial row), push the merged config up so the
+  // server catches up. Fire-and-forget — we already have the right
+  // answer in `config` for the current caller.
+  const shouldHeal = localConfig && apiKey && serverUrl && projectName && (
+    !serverConfig || hasFieldsServerLacks(localConfig, serverConfig)
+  );
+  if (shouldHeal) {
+    pushConfig(apiKey, serverUrl, projectName, stripCredentials(config)).catch(() => {});
   }
 
-  // Cache the merged, credential-free config back to disk so the local file
-  // stays a readable view of the authoritative state. Only when we actually
-  // fetched from the server — offline runs must not silently mutate the
-  // user's local file.
+  // Cache the merged, credential-free config back to disk so the local
+  // file stays a readable view of the authoritative state. Only when we
+  // actually fetched from the server — offline runs must not silently
+  // mutate the user's local file. With mergeNonNull above, we're
+  // guaranteed this write never strips existing local fields.
   if (serverConfig) {
     writeLocalConfig(configPath, config);
   }
@@ -108,6 +120,23 @@ export async function loadConfig(dir, opts = {}) {
   return config;
 }
 
+// True when `local` has any non-null leaf that's null/missing on `server`.
+// Walked recursively for nested blocks (linear, jira, github). Used to
+// decide whether it's worth pushing a heal sync.
+function hasFieldsServerLacks(local, server) {
+  const isObj = v => v && typeof v === "object" && !Array.isArray(v);
+  for (const [key, v] of Object.entries(local || {})) {
+    if (v === null || v === undefined) continue;
+    const s = server?.[key];
+    if (isObj(v)) {
+      if (!isObj(s) || hasFieldsServerLacks(v, s)) return true;
+    } else {
+      if (s === null || s === undefined) return true;
+    }
+  }
+  return false;
+}
+
 // Push a config object to the server. Caller is responsible for deciding when.
 export async function pushConfig(apiKey, serverUrl, projectName, payload) {
   if (!apiKey || !serverUrl || !projectName) return { ok: false, error: "missing_auth" };
@@ -185,3 +214,19 @@ function deepMerge(defaults, overrides) {
   }
   return result;
 }
+
+// Like deepMerge, but null/undefined on the overrides side never
+// clobbers a non-null value already in base. Used for the local ← server
+// step so an empty server row can't wipe a fully-populated local file.
+function mergeNonNull(base, overrides) {
+  const result = { ...base };
+  for (const [key, value] of Object.entries(overrides)) {
+    if (value === null || value === undefined) continue;
+    if (value && typeof value === "object" && !Array.isArray(value) && base[key] && typeof base[key] === "object") {
+      result[key] = mergeNonNull(base[key], value);
+    } else {
+      result[key] = value;
+    }
+  }
+  return result;
+}

package/cli/init.mjs

@@ -484,12 +484,21 @@ async function runWizard({ cwd, apiKey, mode, projectKey }) {
   }
   console.log("");
 
-  // Write local .agentdesk.json. Existing mode writes only projectKey
-  // (server owns the rest and loadConfig will re-populate on the next
-  // run); new mode writes the full config.
+  // Write local .agentdesk.json.
+  //   • Existing mode: preserve whatever's already on disk, ensuring
+  //     only that `projectKey` is set. Never clobber tracker/github/
+  //     badge fields the user may have locally even when the server
+  //     hasn't caught up yet (auto-heal push in config.mjs will
+  //     reconcile the server side).
+  //   • New mode: write the full config we just built.
   const configPath = join(currentCwd, ".agentdesk.json");
   if (isExisting) {
-    writeFileSync(configPath, JSON.stringify({ projectKey: computedKey }, null, 2) + "\n");
+    let existingOnDisk = {};
+    if (existsSync(configPath)) {
+      try { existingOnDisk = JSON.parse(readFileSync(configPath, "utf-8")); } catch {}
+    }
+    const merged = { ...existingOnDisk, projectKey: computedKey };
+    writeFileSync(configPath, JSON.stringify(merged, null, 2) + "\n");
   } else {
     writeFileSync(configPath, JSON.stringify(config, null, 2) + "\n");
   }

package/cli/prompt.mjs

@@ -68,7 +68,7 @@ export function buildPrompt({ taskId, taskLink, description, createTask, tracker
       } else {
         createInstr += `- ⚠ No Linear team key is configured. Ask the user which team this issue should live in BEFORE creating it.\n`;
       }
-      createInstr += `- Set the title based on the description below\n- After creation, use the returned identifier (e.g., ${linearTeamKey || "KEN"}-530) as the task ID for the rest of the session\n`;
+      createInstr += `- Set the title based on the description below\n- **Assign the issue to the connected user.** First fetch the viewer id with \`{ viewer { id } }\`, then pass that id as \`assigneeId\` in the \`issueCreate\` input.\n- After creation, use the returned identifier (e.g., ${linearTeamKey || "KEN"}-530) as the task ID for the rest of the session\n`;
     } else if (tracker === "jira") {
       const jiraProject = config.jira?.project;
       createInstr += `Create a Jira issue:\n- Endpoint: ${config.jira?.baseUrl || ""}/rest/api/3/issue\n- Auth: Basic auth with $JIRA_EMAIL and $JIRA_API_TOKEN\n`;
@@ -77,9 +77,9 @@ export function buildPrompt({ taskId, taskLink, description, createTask, tracker
       } else {
         createInstr += `- ⚠ No Jira project key is configured. Ask the user which project this task should live in BEFORE creating it.\n`;
       }
-      createInstr += `- Set the summary based on the description below\n- After creation, use the returned key (e.g., ${jiraProject || "PROJ"}-42) as the task ID for the rest of the session\n`;
+      createInstr += `- Set the summary based on the description below\n- **Assign the issue to the connected user.** First GET \`${config.jira?.baseUrl || ""}/rest/api/3/myself\` to fetch your \`accountId\`, then include \`fields.assignee = { "accountId": "<that id>" }\` in the create payload.\n- After creation, use the returned key (e.g., ${jiraProject || "PROJ"}-42) as the task ID for the rest of the session\n`;
     } else if (tracker === "github") {
-      createInstr += `Create a GitHub issue:\n- Run: gh issue create --title "..." --body "..."\n- After creation, use the returned issue number as the task ID for the rest of the session\n`;
+      createInstr += `Create a GitHub issue:\n- Run: gh issue create --title "..." --body "..." --assignee @me\n- After creation, use the returned issue number as the task ID for the rest of the session\n`;
     }
     createInstr += `\nTask description: ${description}\n`;
     createInstr += `\nAfter creating the task, Jane MUST:\n`;
@@ -461,14 +461,16 @@ export function buildPhasedPrompt({ phase, taskId, taskLink, description, create
       if (linearTeamKey) {
         createInstr += `**Team MUST be \`${linearTeamKey}\`.** Resolve its id via \`{ teams(filter: { key: { eq: "${linearTeamKey}" } }) { nodes { id } } }\` and pass as \`teamId\` in \`issueCreate\`. Do NOT pick any other team.\n`;
       }
+      createInstr += `**Assign to the connected user:** fetch \`{ viewer { id } }\` and pass that id as \`assigneeId\` in \`issueCreate\`.\n`;
     } else if (tracker === "jira") {
       const jiraProject = config.jira?.project;
       createInstr += `Create a Jira issue at ${config.jira?.baseUrl || ""}.\n`;
       if (jiraProject) {
         createInstr += `**Project MUST be \`${jiraProject}\`.** Set \`fields.project.key = "${jiraProject}"\` in the create payload. Do NOT pick any other project.\n`;
       }
+      createInstr += `**Assign to the connected user:** GET \`${config.jira?.baseUrl || ""}/rest/api/3/myself\` to fetch your \`accountId\`, then include \`fields.assignee = { "accountId": "<that id>" }\` in the create payload.\n`;
     } else if (tracker === "github") {
-      createInstr += `Create a GitHub issue: gh issue create --title "..." --body "..."\n`;
+      createInstr += `Create a GitHub issue: gh issue create --title "..." --body "..." --assignee @me\n`;
     }
     createInstr += `\nTask description: ${description}\n`;
     createInstr += `\nAfter creating, output: TASK_ID: <identifier>\n`;

package/cli/stream-parser.mjs

@@ -39,11 +39,32 @@ export function createStreamParser({ teamNames, callbacks }) {
 
   function detectPhase(text) {
     const upper = text.trim().toUpperCase();
-    if (upper.startsWith("PHASE") || upper.startsWith("---") || upper.startsWith("#")) {
-      for (const p of PHASE_NAMES) {
-        if (upper.includes(p)) return p;
-      }
-    }
+    if (!upper) return null;
+
+    const phaseAlt = PHASE_NAMES.join("|");
+
+    // Canonical signal: "# PHASE: EXECUTION" / "PHASE EXECUTION" / "## EXECUTION".
+    // Accepted anywhere in a line so it survives leading bullets, emoji, etc.
+    const explicit = upper.match(new RegExp(`(?:^|\\s)#*\\s*PHASE\\s*[:\\-]?\\s*(${phaseAlt})\\b`));
+    if (explicit) return explicit[1];
+
+    // Markdown heading line that names a phase: "# EXECUTION", "## PLAN", etc.
+    const headingMatch = upper.match(new RegExp(`^#+\\s*(${phaseAlt})\\b`));
+    if (headingMatch) return headingMatch[1];
+
+    // Divider followed by a phase name: "--- EXECUTION".
+    const dividerMatch = upper.match(new RegExp(`^-{3,}\\s*(${phaseAlt})\\b`));
+    if (dividerMatch) return dividerMatch[1];
+
+    // Natural-language transitions ("entering EXECUTION", "moving to PLAN",
+    // "now in REVIEW", "EXECUTION phase begins"). Defensive fallback for when
+    // the canonical marker is missed.
+    const verbMatch = upper.match(new RegExp(`\\b(?:ENTERING|MOVING\\s+TO|STARTING|BEGIN(?:NING)?|NOW\\s+IN|TRANSITION(?:ING)?\\s+TO|PROCEED(?:ING)?\\s+TO|ADVANCING\\s+TO)\\s+(?:THE\\s+)?(?:PHASE\\s+)?(${phaseAlt})\\b(?:\\s+PHASE)?`));
+    if (verbMatch) return verbMatch[1];
+
+    const phaseSuffixMatch = upper.match(new RegExp(`\\b(${phaseAlt})\\s+PHASE\\s+(?:BEGINS?|STARTS?|HAS\\s+BEGUN|IS\\s+UNDERWAY)\\b`));
+    if (phaseSuffixMatch) return phaseSuffixMatch[1];
+
     return null;
   }
 
@@ -86,12 +107,6 @@ export function createStreamParser({ teamNames, callbacks }) {
           if (block.type === "text" && block.text.trim()) {
             const text = block.text.trim().replace(/\*+/g, "");
 
-            const phase = detectPhase(text);
-            if (phase) {
-              callbacks.onPhaseChange?.({ phase, timestamp: timestamp() });
-              continue;
-            }
-
             // Detect TASK_ID announcement
             const taskIdMatch = text.match(/TASK_ID:\s*(\S+)/);
             if (taskIdMatch) {
@@ -110,6 +125,13 @@ export function createStreamParser({ teamNames, callbacks }) {
               const currentLine = textLines[i].trim();
               if (!currentLine) { i++; continue; }
 
+              const phase = detectPhase(currentLine);
+              if (phase) {
+                callbacks.onPhaseChange?.({ phase, timestamp: timestamp() });
+                i++;
+                continue;
+              }
+
               const detected = detectAgent(currentLine);
               if (detected) {
                 let msg = detected.rest;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kendoo.agentdesk/agentdesk",
-  "version": "0.20.2",
+  "version": "0.20.4",
   "description": "AI team orchestrator for Claude Code — run collaborative agent sessions from your terminal",
   "type": "module",
   "bin": {

package/prompts/team.md

@@ -314,8 +314,22 @@ Keep comments to bullet points.
 
 ---
 
+## PHASE SIGNALS (MANDATORY — DASHBOARD DEPENDS ON THIS)
+
+Whenever the team transitions phases, Jane MUST print a single bare line in this exact form on its own line, with nothing else on that line:
+
+```
+# PHASE: <NAME>
+```
+
+Where `<NAME>` is one of `INTAKE`, `PLAN`, `EXECUTION`, `REVIEW`, `SUMMARY`. Examples: `# PHASE: PLAN`, `# PHASE: EXECUTION`, `# PHASE: SUMMARY`. This must be printed BEFORE any agent dialogue for the new phase. Without this marker the dashboard timeline gets stuck and the user can't tell which phase the team is in. Emit it even if it feels redundant.
+
+---
+
 ## INTAKE
 
+Begin this phase by printing `# PHASE: INTAKE` on its own line.
+
 **If a CREATE TASK section exists above**, Dennis executes the tracker API call to create the task. Jane then announces the task ID and what the team will be working on. Output `TASK_ID: <identifier>`, set status to "In Progress". Only then continue.
 
 **Reminder: Jane does NOT use tools or reference code during INTAKE. Dennis handles all tool calls (fetching tasks, reading files, checking branches). Jane interprets the findings in product terms.**
@@ -445,6 +459,8 @@ After Dennis reports his assessment findings, Jane evaluates whether the task is
 
 ## PLAN (1-2 rounds max)
 
+Begin this phase by printing `# PHASE: PLAN` on its own line.
+
 Jane restates the task in product terms (no code, no file names, no technical jargon). Then each agent contributes their perspective in one pass:
 
 {{PLANNING_ORDER}}
@@ -457,12 +473,16 @@ After the first round, Jane asks for objections. If none, declare the plan final
 
 ## EXECUTION
 
+Begin this phase by printing `# PHASE: EXECUTION` on its own line.
+
 {{EXECUTION_STEPS}}
 
 ---
 
 ## SUMMARY
 
+Begin this phase by printing `# PHASE: SUMMARY` on its own line.
+
 Jane dictates the summary content in product terms; Dennis executes all tracker commands:
 1. Verify Bart posted the PR link. If not, Dennis posts it now.
 2. Dennis transitions the task to "In Review".