wmdev 0.3.0 → 0.4.0

package/README.md

@@ -62,6 +62,10 @@ wmdev uses two config files in the project root:
 ### `.wmdev.yaml` schema
 
 ```yaml
+# Project name displayed in the sidebar header and browser tab title.
+# Falls back to "Dashboard" if omitted.
+name: string
+
 # Services to monitor — each maps a display name to a port env var.
 # The dashboard polls these ports and shows health status badges.
 # When portStart is set, wmdev auto-allocates ports for new worktrees
@@ -112,6 +116,8 @@ linkedRepos: []
 ### Example
 
 ```yaml
+name: My Project
+
 services:
   - name: BE
     portEnv: BACKEND_PORT
@@ -149,6 +155,7 @@ linkedRepos:
 
 | Parameter | Type | Required | Description |
 |-----------|------|----------|-------------|
+| `name` | string | no | Project name shown in the sidebar header and browser tab title. Defaults to "Dashboard" |
 | `services[].name` | string | yes | Display name shown in the dashboard |
 | `services[].portEnv` | string | yes | Env var containing the service port (read from each worktree's `.env.local`) |
 | `services[].portStart` | number | no | Base port for slot 0. When set, wmdev auto-allocates ports for new worktrees |

package/backend/dist/server.js

@@ -6941,8 +6941,13 @@ async function readEnvLocal(wtDir) {
     for (const line of text.split(`
 `)) {
       const match = line.match(/^(\w+)=(.*)$/);
-      if (match)
-        env[match[1]] = match[2];
+      if (match) {
+        let val = match[2];
+        if (val.length >= 2 && val.startsWith("'") && val.endsWith("'")) {
+          val = val.slice(1, -1);
+        }
+        env[match[1]] = val;
+      }
     }
     return env;
   } catch {
@@ -6959,12 +6964,14 @@ async function writeEnvLocal(wtDir, entries) {
 `);
   } catch {}
   for (const [key, value] of Object.entries(entries)) {
+    const needsQuoting = /[{}"\s$`\\!#|;&()<>]/.test(value);
+    const safe = needsQuoting ? `'${value}'` : value;
     const pattern = new RegExp(`^${key}=`);
     const idx = lines.findIndex((l) => pattern.test(l));
     if (idx >= 0) {
-      lines[idx] = `${key}=${value}`;
+      lines[idx] = `${key}=${safe}`;
     } else {
-      lines.push(`${key}=${value}`);
+      lines.push(`${key}=${safe}`);
     }
   }
   await Bun.write(filePath, lines.join(`
@@ -7098,6 +7105,7 @@ function loadConfig(dir) {
       alias: typeof r.alias === "string" ? r.alias : r.repo.split("/").pop()
     })) : [];
     return {
+      ...typeof parsed.name === "string" ? { name: parsed.name } : {},
       services: Array.isArray(parsed.services) ? parsed.services : DEFAULT_CONFIG.services,
       profiles: {
         default: defaultProfile?.name ? defaultProfile : DEFAULT_CONFIG.profiles.default,
@@ -8097,6 +8105,20 @@ function mapChecks(checks) {
     runId: parseRunId(c.detailsUrl)
   }));
 }
+function parseReviewComments(json) {
+  const raw = JSON.parse(json);
+  const sorted = raw.sort((a, b) => new Date(b.created_at).getTime() - new Date(a.created_at).getTime());
+  return sorted.slice(0, PR_FETCH_LIMIT).map((c) => ({
+    type: "inline",
+    author: c.user?.login ?? "unknown",
+    body: c.body ?? "",
+    createdAt: c.created_at ?? "",
+    path: c.path ?? "",
+    line: c.line ?? null,
+    diffHunk: c.diff_hunk ?? "",
+    isReply: c.in_reply_to_id !== undefined
+  }));
+}
 function parsePrResponse(json, repoLabel) {
   const prs = new Map;
   const entries = JSON.parse(json);
@@ -8111,6 +8133,7 @@ function parsePrResponse(json, repoLabel) {
       ciStatus: summarizeChecks(entry.statusCheckRollup),
       ciChecks: mapChecks(entry.statusCheckRollup),
       comments: (entry.comments ?? []).map((c) => ({
+        type: "comment",
         author: c.author?.login ?? "unknown",
         body: c.body ?? "",
         createdAt: c.createdAt ?? ""
@@ -8161,6 +8184,45 @@ async function fetchAllPrs(repoSlug, repoLabel, cwd) {
     return { ok: false, error: `failed to parse gh output for ${label}: ${err}` };
   }
 }
+async function mapWithConcurrency(items, limit, fn) {
+  const results = new Array(items.length);
+  let next = 0;
+  async function worker() {
+    while (next < items.length) {
+      const idx = next++;
+      results[idx] = await fn(items[idx]);
+    }
+  }
+  await Promise.all(Array.from({ length: Math.min(limit, items.length) }, () => worker()));
+  return results;
+}
+async function fetchReviewComments(prNumber, repoSlug, cwd) {
+  const repoFlag = repoSlug ? repoSlug : "{owner}/{repo}";
+  const args = [
+    "gh",
+    "api",
+    `repos/${repoFlag}/pulls/${prNumber}/comments`,
+    "--paginate"
+  ];
+  const proc = Bun.spawn(args, {
+    stdout: "pipe",
+    stderr: "pipe",
+    ...cwd ? { cwd } : {}
+  });
+  const timeout = Bun.sleep(GH_TIMEOUT_MS).then(() => {
+    proc.kill();
+    return "timeout";
+  });
+  const raceResult = await Promise.race([proc.exited, timeout]);
+  if (raceResult === "timeout" || raceResult !== 0)
+    return [];
+  try {
+    const json = await new Response(proc.stdout).text();
+    return parseReviewComments(json);
+  } catch {
+    return [];
+  }
+}
 async function fetchPrState(url) {
   const proc = Bun.spawn(["gh", "pr", "view", url, "--json", "state"], {
     stdout: "pipe",
@@ -8217,6 +8279,22 @@ async function syncPrStatus(getWorktreePaths, linkedRepos, projectDir) {
       branchPrs.set(branch, existing);
     }
   }
+  const reviewTuples = [];
+  for (const entries of branchPrs.values()) {
+    for (const entry of entries) {
+      if (entry.state === "open") {
+        const repoSlug = entry.repo ? linkedRepos.find((lr) => lr.alias === entry.repo)?.repo : undefined;
+        reviewTuples.push({ entry, repoSlug });
+      }
+    }
+  }
+  if (reviewTuples.length > 0) {
+    const reviewResults = await mapWithConcurrency(reviewTuples, 5, (t) => fetchReviewComments(t.entry.number, t.repoSlug, projectDir));
+    for (let i = 0;i < reviewTuples.length; i++) {
+      const entry = reviewTuples[i].entry;
+      entry.comments = [...entry.comments, ...reviewResults[i]].sort((a, b) => new Date(a.createdAt).getTime() - new Date(b.createdAt).getTime());
+    }
+  }
   const wtPaths = await getWorktreePaths();
   const seen = new Set;
   for (const [branch, entries] of branchPrs) {