@tritard/waterbrother 0.16.96 → 0.16.97

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tritard/waterbrother",
-  "version": "0.16.96",
+  "version": "0.16.97",
   "description": "Waterbrother: bring-your-own-model coding CLI with local tools, sessions, operator modes, and approval controls",
   "type": "module",
   "bin": {

package/src/gateway.js

@@ -834,7 +834,7 @@ function parseTelegramStateIntent(text = "") {
   if (/\bwhat can i do here\b/.test(lower) || /\bhow do i use this room\b/.test(lower) || /\bhow do i use this chat\b/.test(lower) || /\bwhat do i do here\b/.test(lower)) {
     return { action: "room-guidance" };
   }
-  if (/\bwho are the bots\b/.test(lower) || /\bwho are the agents\b/.test(lower) || /\bwhat bots are here\b/.test(lower) || /\bwhat agents are here\b/.test(lower)) {
+  if (/\bwho are the (?:bots|boys)\b/.test(lower) || /\bwho are the agents\b/.test(lower) || /\bwhat (?:bots|boys) are here\b/.test(lower) || /\bwhat agents are here\b/.test(lower)) {
     return { action: "agent-list" };
   }
   if (/\bwhich terminals are live\b/.test(lower) || /\bwhich bots are live\b/.test(lower) || /\bwho is live\b/.test(lower) || /\bwhat terminals are live\b/.test(lower)) {

package/src/shared-project.js

@@ -54,11 +54,13 @@ function normalizeParticipant(participant = {}) {
 }
 
 function normalizeAgent(agent = {}) {
+  const rawLabel = String(agent.label || agent.name || "").trim();
+  const cleanLabel = rawLabel.replace(/\bundefined\b/ig, "").replace(/\s+/g, " ").trim();
   return {
     id: String(agent.id || "").trim(),
     ownerId: String(agent.ownerId || "").trim(),
     ownerName: String(agent.ownerName || "").trim(),
-    label: String(agent.label || agent.name || "").trim(),
+    label: cleanLabel,
     surface: String(agent.surface || "").trim(),
     role: AGENT_ROLES.includes(String(agent.role || "").trim()) ? String(agent.role).trim() : "standby",
     provider: String(agent.provider || "").trim(),
@@ -71,6 +73,42 @@ function normalizeAgent(agent = {}) {
   };
 }
 
+function isBrokenAgent(agent = {}) {
+  const ownerId = String(agent.ownerId || "").trim();
+  const ownerName = String(agent.ownerName || "").trim();
+  const label = String(agent.label || "").trim();
+  return !ownerId && !ownerName && !label;
+}
+
+function dedupeAgents(agents = []) {
+  const bySemanticKey = new Map();
+  for (const agent of agents) {
+    const normalized = normalizeAgent(agent);
+    if (!normalized.id || isBrokenAgent(normalized)) continue;
+    const semanticKey = [
+      String(normalized.ownerId || "").trim(),
+      String(normalized.ownerName || "").trim().toLowerCase(),
+      String(normalized.label || "").trim().toLowerCase(),
+      String(normalized.surface || "").trim().toLowerCase(),
+      String(normalized.role || "").trim().toLowerCase(),
+      String(normalized.provider || "").trim().toLowerCase(),
+      String(normalized.model || "").trim().toLowerCase(),
+      String(normalized.chatId || "").trim()
+    ].join("|");
+    const prior = bySemanticKey.get(semanticKey);
+    if (!prior) {
+      bySemanticKey.set(semanticKey, normalized);
+      continue;
+    }
+    const priorUpdated = Date.parse(String(prior.updatedAt || "").trim()) || 0;
+    const nextUpdated = Date.parse(String(normalized.updatedAt || "").trim()) || 0;
+    if (nextUpdated >= priorUpdated) {
+      bySemanticKey.set(semanticKey, normalized);
+    }
+  }
+  return [...bySemanticKey.values()];
+}
+
 function areAgentsEquivalent(left = {}, right = {}) {
   return [
     "ownerId",
@@ -119,12 +157,8 @@ function buildProjectParticipants(project = {}) {
 }
 
 function buildProjectAgents(project = {}) {
-  const existing = new Map(
-    (Array.isArray(project.agents) ? project.agents : [])
-      .map((agent) => normalizeAgent(agent))
-      .filter((agent) => agent.id)
-      .map((agent) => [agent.id, agent])
-  );
+  const existingAgents = dedupeAgents(Array.isArray(project.agents) ? project.agents : []);
+  const existing = new Map(existingAgents.map((agent) => [agent.id, agent]));
   const next = [];
   const activeOperatorId = String(project.activeOperator?.id || "").trim();
   if (activeOperatorId) {
@@ -145,7 +179,7 @@ function buildProjectAgents(project = {}) {
     if (next.some((item) => item.id === agent.id)) continue;
     next.push(normalizeAgent(agent));
   }
-  return next;
+  return dedupeAgents(next);
 }
 
 function memberRoleWeight(role = "") {
@@ -433,7 +467,12 @@ async function recordSharedProjectEvent(cwd, project, text, { type = "note", act
 export async function loadSharedProject(cwd) {
   try {
     const raw = await fs.readFile(sharedFilePath(cwd), "utf8");
-    return normalizeSharedProject(JSON.parse(raw), cwd);
+    const parsed = JSON.parse(raw);
+    const normalized = normalizeSharedProject(parsed, cwd);
+    if (JSON.stringify(parsed) !== JSON.stringify(normalized)) {
+      await writeJsonAtomically(sharedFilePath(cwd), normalized);
+    }
+    return normalized;
   } catch (error) {
     if (error?.code === "ENOENT") return null;
     throw error;