quadwork 1.6.3 → 1.8.0

package/server/routes.js

@@ -8,6 +8,8 @@ const fs = require("fs");
 const path = require("path");
 const os = require("os");
 
+const multer = require("multer");
+
 const router = express.Router();
 
 const CONFIG_DIR = path.join(os.homedir(), ".quadwork");
@@ -113,18 +115,32 @@ function chatAuthHeaders(token) {
 }
 
 router.get("/api/chat", async (req, res) => {
+  const projectId = req.query.project;
   const apiPath = req.query.path || "/api/messages";
-  const { url: base, token } = getChattrConfig(req.query.project);
+  const { url: base, token } = getChattrConfig(projectId);
 
-  const fwd = new URLSearchParams();
-  for (const [k, v] of Object.entries(req.query)) {
-    if (k !== "path") fwd.set(k, String(v));
-  }
-  if (token) fwd.set("token", token);
+  const buildUrl = (tok) => {
+    const fwd = new URLSearchParams();
+    for (const [k, v] of Object.entries(req.query)) {
+      if (k !== "path") fwd.set(k, String(v));
+    }
+    if (tok) fwd.set("token", tok);
+    return `${base}${apiPath}?${fwd.toString()}`;
+  };
 
-  const url = `${base}${apiPath}?${fwd.toString()}`;
   try {
-    const r = await fetch(url, { headers: chatAuthHeaders(token) });
+    const r = await fetch(buildUrl(token), { headers: chatAuthHeaders(token) });
+    // #448: on 401/403, re-sync the session token from AC and retry
+    // once. The stored token may be stale after an AC restart.
+    if ((r.status === 401 || r.status === 403) && projectId) {
+      try { await syncChattrToken(projectId); } catch {}
+      const { token: refreshed } = getChattrConfig(projectId);
+      if (refreshed && refreshed !== token) {
+        const retry = await fetch(buildUrl(refreshed), { headers: chatAuthHeaders(refreshed) });
+        if (!retry.ok) return res.status(retry.status).json({ error: `AgentChattr returned ${retry.status}` });
+        return res.json(await retry.json());
+      }
+    }
     if (!r.ok) return res.status(r.status).json({ error: `AgentChattr returned ${r.status}` });
     res.json(await r.json());
   } catch (err) {
@@ -977,6 +993,53 @@ router.post("/api/chat", async (req, res) => {
   }
 });
 
+// ─── Image upload (#466) ──────────────────────────────────────────────────
+
+const UPLOAD_MAX_BYTES = 10 * 1024 * 1024; // 10MB
+const ALLOWED_MIME = new Set(["image/png", "image/jpeg", "image/gif", "image/webp"]);
+const uploadStorage = multer.diskStorage({
+  destination: (req, _file, cb) => {
+    const projectId = req.query.project || "";
+    if (!projectId || /[/\\]/.test(projectId)) return cb(new Error("Invalid project"));
+    const dir = path.join(CONFIG_DIR, projectId, "uploads");
+    fs.mkdirSync(dir, { recursive: true });
+    cb(null, dir);
+  },
+  filename: (_req, file, cb) => {
+    const ext = path.extname(file.originalname) || ".png";
+    cb(null, `upload-${Date.now()}${ext}`);
+  },
+});
+const upload = multer({
+  storage: uploadStorage,
+  limits: { fileSize: UPLOAD_MAX_BYTES },
+  fileFilter: (_req, file, cb) => {
+    if (ALLOWED_MIME.has(file.mimetype)) cb(null, true);
+    else cb(new Error(`Unsupported type: ${file.mimetype}`));
+  },
+});
+
+router.post("/api/upload", upload.single("file"), (req, res) => {
+  if (!req.file) return res.status(400).json({ error: "No file uploaded" });
+  return res.json({
+    ok: true,
+    path: req.file.path,
+    name: req.file.filename,
+  });
+});
+
+// Serve uploaded images for thumbnail rendering
+router.get("/api/uploads/:project/:filename", (req, res) => {
+  const { project, filename } = req.params;
+  // Sanitize to prevent directory traversal
+  if (/[/\\]/.test(project) || /[/\\]/.test(filename)) {
+    return res.status(400).json({ error: "Invalid path" });
+  }
+  const filePath = path.join(CONFIG_DIR, project, "uploads", filename);
+  if (!fs.existsSync(filePath)) return res.status(404).json({ error: "Not found" });
+  res.sendFile(filePath);
+});
+
 // ─── Projects (dashboard aggregation) ──────────────────────────────────────
 
 function ghJson(args) {
@@ -1674,184 +1737,7 @@ router.get("/api/batch-progress", async (req, res) => {
   res.json(data);
 });
 
-// ─── Memory ────────────────────────────────────────────────────────────────
-
-function getProject(projectId) {
-  try {
-    const cfg = JSON.parse(fs.readFileSync(CONFIG_PATH, "utf-8"));
-    return cfg.projects?.find((p) => p.id === projectId) || null;
-  } catch {
-    return null;
-  }
-}
-
-function getMemoryPaths(project) {
-  const workDir = project.working_dir || "";
-  return {
-    cardsDir: project.memory_cards_dir || path.join(workDir, "..", "agent-memory", "archive", "v2", "cards"),
-    sharedMemoryPath: project.shared_memory_path || path.join(workDir, "..", "agent-memory", "central", "short-term", "agent-os.md"),
-    butlerDir: project.butler_scripts_dir || path.join(workDir, "..", "agent-memory", "scripts"),
-  };
-}
-
-function findMdFiles(dir) {
-  const results = [];
-  if (!fs.existsSync(dir)) return results;
-  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
-    const full = path.join(dir, entry.name);
-    if (entry.isDirectory()) results.push(...findMdFiles(full));
-    else if (entry.name.endsWith(".md")) results.push(full);
-  }
-  return results;
-}
-
-function parseFrontmatter(content) {
-  const fm = {};
-  const match = content.match(/^---\n([\s\S]*?)\n---/);
-  if (!match) return fm;
-  for (const line of match[1].split("\n")) {
-    const idx = line.indexOf(":");
-    if (idx > 0) {
-      const key = line.slice(0, idx).trim();
-      let val = line.slice(idx + 1).trim();
-      if (val.startsWith("[") && val.endsWith("]")) val = val.slice(1, -1).trim();
-      fm[key] = val;
-    }
-  }
-  return fm;
-}
-
-router.get("/api/memory", (req, res) => {
-  const projectId = req.query.project || "";
-  const action = req.query.action || "cards";
-  const project = getProject(projectId);
-  if (!project) return res.status(404).json({ error: "Project not found" });
-
-  const paths = getMemoryPaths(project);
-
-  if (action === "cards") {
-    const search = req.query.search || "";
-    try {
-      const files = findMdFiles(paths.cardsDir);
-      const cards = files.map((fullPath) => {
-        const content = fs.readFileSync(fullPath, "utf-8");
-        const fm = parseFrontmatter(content);
-        const relPath = path.relative(paths.cardsDir, fullPath);
-        const body = content.replace(/^---\n[\s\S]*?\n---\n?/, "").trim();
-        const firstLine = body.split("\n")[0]?.replace(/^#\s*/, "").trim();
-        return {
-          file: relPath,
-          title: firstLine || fm.id || path.basename(fullPath, ".md"),
-          date: fm.at || "",
-          agent: fm.by || "",
-          tags: fm.tags || "",
-          content: body,
-        };
-      });
-      cards.sort((a, b) => b.date.localeCompare(a.date));
-      if (search) {
-        const q = search.toLowerCase();
-        return res.json(cards.filter((c) =>
-          c.title.toLowerCase().includes(q) || c.agent.toLowerCase().includes(q) || c.tags.toLowerCase().includes(q) || c.content.toLowerCase().includes(q)
-        ));
-      }
-      return res.json(cards);
-    } catch {
-      return res.json([]);
-    }
-  }
-
-  if (action === "status") {
-    const agents = project.agents || {};
-    const status = {};
-    for (const [id, agent] of Object.entries(agents)) {
-      const targetPath = path.join(agent.cwd || "", "shared-memory.md");
-      if (fs.existsSync(targetPath)) {
-        const stat = fs.statSync(targetPath);
-        status[id] = { injected: true, lastModified: stat.mtime.toISOString() };
-      } else {
-        status[id] = { injected: false, lastModified: null };
-      }
-    }
-    const sourceExists = fs.existsSync(paths.sharedMemoryPath);
-    return res.json({ agents: status, sourceExists });
-  }
-
-  if (action === "shared-memory") {
-    try {
-      const content = fs.readFileSync(paths.sharedMemoryPath, "utf-8");
-      return res.json({ content, path: paths.sharedMemoryPath });
-    } catch {
-      return res.json({ content: "", path: paths.sharedMemoryPath });
-    }
-  }
-
-  if (action === "settings") {
-    return res.json({
-      memory_cards_dir: project.memory_cards_dir || "",
-      shared_memory_path: project.shared_memory_path || "",
-      butler_scripts_dir: project.butler_scripts_dir || "",
-    });
-  }
-
-  res.status(400).json({ error: "Unknown action" });
-});
-
-router.post("/api/memory", (req, res) => {
-  const projectId = req.query.project || "";
-  const action = req.query.action || "";
-  const project = getProject(projectId);
-  if (!project) return res.status(404).json({ error: "Project not found" });
-
-  const paths = getMemoryPaths(project);
-
-  if (action === "butler") {
-    const allowed = ["butler-scan.sh", "butler-consolidate.sh", "inject.sh"];
-    const command = req.body.command;
-    if (!allowed.includes(command)) return res.json({ ok: false, error: `Unknown command: ${command}` });
-    const scriptPath = path.join(paths.butlerDir, command);
-    if (!fs.existsSync(scriptPath)) return res.json({ ok: false, error: `Script not found: ${scriptPath}` });
-    try {
-      const output = execFileSync("bash", [scriptPath], {
-        encoding: "utf-8",
-        timeout: 30000,
-        cwd: path.dirname(paths.butlerDir),
-      });
-      return res.json({ ok: true, output });
-    } catch (err) {
-      return res.json({ ok: false, error: err.message });
-    }
-  }
-
-  if (action === "save-memory") {
-    try {
-      const dir = path.dirname(paths.sharedMemoryPath);
-      if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
-      fs.writeFileSync(paths.sharedMemoryPath, req.body.content);
-      return res.json({ ok: true });
-    } catch (err) {
-      return res.json({ ok: false, error: err.message });
-    }
-  }
-
-  if (action === "save-settings") {
-    try {
-      const cfg = JSON.parse(fs.readFileSync(CONFIG_PATH, "utf-8"));
-      const proj = cfg.projects?.find((p) => p.id === projectId);
-      if (!proj) return res.json({ ok: false, error: "Project not found" });
-      const s = req.body;
-      if (s.memory_cards_dir !== undefined) proj.memory_cards_dir = s.memory_cards_dir || undefined;
-      if (s.shared_memory_path !== undefined) proj.shared_memory_path = s.shared_memory_path || undefined;
-      if (s.butler_scripts_dir !== undefined) proj.butler_scripts_dir = s.butler_scripts_dir || undefined;
-      fs.writeFileSync(CONFIG_PATH, JSON.stringify(cfg, null, 2));
-      return res.json({ ok: true });
-    } catch (err) {
-      return res.json({ ok: false, error: err.message });
-    }
-  }
-
-  res.status(400).json({ error: "Unknown action" });
-});
+// #445: Memory section (agent-memory butler integration) removed.
 
 // ─── Setup ─────────────────────────────────────────────────────────────────
 
@@ -2378,9 +2264,12 @@ router.post("/api/rename", (req, res) => {
 // ─── Telegram ──────────────────────────────────────────────────────────────
 
 const BRIDGE_DIR = path.join(CONFIG_DIR, "agentchattr-telegram");
+// #444: pin agentchattr-telegram to a known commit (same pattern as
+// AGENTCHATTR_PIN in bin/quadwork.js for bcurts/agentchattr).
+const AGENTCHATTR_TELEGRAM_PIN = "4a6b45f1794c612328b9d5ee6d6fcb3f77015abc";
 
 function telegramPidFile(projectId) {
-  return path.join(CONFIG_DIR, `telegram-bridge-${projectId}.pid`);
+  return path.join(CONFIG_DIR, `tg-bridge-${projectId}.pid`);
 }
 
 function telegramConfigToml(projectId) {
@@ -2388,7 +2277,7 @@ function telegramConfigToml(projectId) {
 }
 
 // #383: path to a project's AgentChattr config.toml. The install
-// handler patches this file to declare the `telegram-bridge` agent
+// handler patches this file to declare the `tg` agent
 // so AC's registry accepts the bridge's register call.
 function projectAgentchattrConfigPath(projectId) {
   return path.join(CONFIG_DIR, projectId, "agentchattr", "config.toml");
@@ -2415,7 +2304,12 @@ function resolveProjectAgentchattrUrl(cfg, project) {
 // AC message IDs advances the cursor past the other project's range,
 // silently killing AC→TG forwarding for that project.
 function buildTelegramBridgeToml(tg, projectId) {
-  const cursorFile = path.join(CONFIG_DIR, `telegram-bridge-cursor-${projectId}.json`);
+  const cursorFile = path.join(CONFIG_DIR, `tg-bridge-cursor-${projectId}.json`);
+  // #439: migrate old cursor file so the bridge doesn't replay history
+  const oldCursor = path.join(CONFIG_DIR, `telegram-bridge-cursor-${projectId}.json`);
+  if (!fs.existsSync(cursorFile) && fs.existsSync(oldCursor)) {
+    fs.renameSync(oldCursor, cursorFile);
+  }
   return (
     `[telegram]\n` +
     `bot_token = "${tg.bot_token}"\n` +
@@ -2427,14 +2321,18 @@ function buildTelegramBridgeToml(tg, projectId) {
 
 // #383 Bug 3: AC's registry rejects any base name not pre-declared
 // in config.toml with `400 unknown base`. The bridge registers as
-// `telegram-bridge`, so every per-project AC config must declare it.
-// Idempotent: only appends if the section is not already present.
+// `tg` (#439: renamed from `telegram-bridge`), so every per-project
+// AC config must declare it. Idempotent: only appends if the section
+// is not already present. Also migrates old `[agents.telegram-bridge]`.
 function patchAgentchattrConfigForTelegramBridge(tomlText) {
-  if (/^\[agents\.telegram-bridge\]\s*$/m.test(tomlText)) {
-    return { text: tomlText, changed: false };
+  // #439: migrate old slug if present
+  const original = tomlText;
+  tomlText = tomlText.replace(/^\[agents\.telegram-bridge\]\s*$/m, "[agents.tg]");
+  if (/^\[agents\.tg\]\s*$/m.test(tomlText)) {
+    return { text: tomlText, changed: tomlText !== original };
   }
   const sep = tomlText.length === 0 || tomlText.endsWith("\n") ? "" : "\n";
-  const block = `\n[agents.telegram-bridge]\nlabel = "Telegram Bridge"\n`;
+  const block = `\n[agents.tg]\nlabel = "Telegram Bridge"\n`;
   return { text: tomlText + sep + block, changed: true };
 }
 
@@ -2456,7 +2354,7 @@ function buildTelegramBridgeSpawnEnv(parentEnv) {
 // config parse, auth failure) are recoverable instead of
 // /dev/null'd by `stdio: "ignore"`.
 function telegramBridgeLog(projectId) {
-  return path.join(CONFIG_DIR, `telegram-bridge-${projectId}.log`);
+  return path.join(CONFIG_DIR, `tg-bridge-${projectId}.log`);
 }
 
 // Tail the last N lines of a file without reading the whole thing
@@ -2676,6 +2574,12 @@ router.post("/api/telegram", async (req, res) => {
       try {
         if (!fs.existsSync(BRIDGE_DIR)) {
           execFileSync("gh", ["repo", "clone", "realproject7/agentchattr-telegram", BRIDGE_DIR], { encoding: "utf-8", timeout: 30000 });
+          // #444: pin to a known commit after clone
+          try {
+            execFileSync("git", ["-C", BRIDGE_DIR, "checkout", "-B", "pinned", AGENTCHATTR_TELEGRAM_PIN], { encoding: "utf-8", timeout: 30000 });
+          } catch {
+            console.warn(`[telegram] WARNING: could not check out agentchattr-telegram pin ${AGENTCHATTR_TELEGRAM_PIN}; falling back to default branch.`);
+          }
         }
         // #380: create the dedicated venv if missing. `python3 -m venv`
         // builds a fresh isolated environment that bypasses PEP 668
@@ -2705,16 +2609,14 @@ router.post("/api/telegram", async (req, res) => {
             `pip output tail:\n${pipOutput.split("\n").slice(-10).join("\n")}`,
         });
       }
-      // #383 Bug 3: ensure every known project's AC config declares
-      // the `telegram-bridge` agent. Without this, AC's registry
-      // rejects the bridge's register call with `400 unknown base`
-      // and the bridge enters an infinite re-register loop.
-      // Idempotent — append-only, skips configs that already have
-      // the section. Does NOT restart AC servers; the operator
-      // must click SERVER → Restart to load the new agent slug.
+      // #383 Bug 3 / #457: ensure every known project's AC config
+      // declares the `tg` agent and migrates old `telegram-bridge`
+      // slug. Restarts AC for projects whose config changed so the
+      // new slug loads immediately.
       const patched = [];
       try {
         const cfgAll = JSON.parse(fs.readFileSync(CONFIG_PATH, "utf-8"));
+        const serverPort = cfgAll.port || 8400;
         for (const proj of cfgAll.projects || []) {
           if (!proj || !proj.id) continue;
           const acPath = projectAgentchattrConfigPath(proj.id);
@@ -2725,6 +2627,14 @@ router.post("/api/telegram", async (req, res) => {
             if (changed) {
               fs.writeFileSync(acPath, text);
               patched.push(proj.id);
+              // #457: restart AC so it loads the new agent slug
+              setTimeout(async () => {
+                try {
+                  await fetch(`http://127.0.0.1:${serverPort}/api/agentchattr/${encodeURIComponent(proj.id)}/restart`, {
+                    method: "POST",
+                  });
+                } catch {}
+              }, 1000);
             }
           } catch {}
         }
@@ -2855,7 +2765,7 @@ router.post("/api/telegram", async (req, res) => {
         const acUrl = resolveProjectAgentchattrUrl(cfg, project);
         if (acUrl) {
           const acPort = new URL(acUrl).port || "8300";
-          await fetch(`http://127.0.0.1:${acPort}/api/deregister/telegram-bridge`, {
+          await fetch(`http://127.0.0.1:${acPort}/api/deregister/tg`, {
             method: "POST",
             signal: AbortSignal.timeout(3000),
           }).catch(() => {});
@@ -2938,7 +2848,7 @@ const DISCORD_BRIDGE_SRC = path.join(__dirname, "..", "bridges", "discord");
 const DISCORD_BRIDGE_DIR = path.join(CONFIG_DIR, "agentchattr-discord");
 
 function discordPidFile(projectId) {
-  return path.join(CONFIG_DIR, `discord-bridge-${projectId}.pid`);
+  return path.join(CONFIG_DIR, `dc-bridge-${projectId}.pid`);
 }
 
 function discordConfigToml(projectId) {
@@ -2946,11 +2856,16 @@ function discordConfigToml(projectId) {
 }
 
 function discordBridgeLog(projectId) {
-  return path.join(CONFIG_DIR, `discord-bridge-${projectId}.log`);
+  return path.join(CONFIG_DIR, `dc-bridge-${projectId}.log`);
 }
 
 function buildDiscordBridgeToml(dc, projectId) {
-  const cursorFile = path.join(CONFIG_DIR, `discord-bridge-cursor-${projectId}.json`);
+  const cursorFile = path.join(CONFIG_DIR, `dc-bridge-cursor-${projectId}.json`);
+  // #439: migrate old cursor file so the bridge doesn't replay history
+  const oldCursor = path.join(CONFIG_DIR, `discord-bridge-cursor-${projectId}.json`);
+  if (!fs.existsSync(cursorFile) && fs.existsSync(oldCursor)) {
+    fs.renameSync(oldCursor, cursorFile);
+  }
   return (
     `[discord]\n` +
     `bot_token = "${dc.bot_token}"\n` +
@@ -2961,11 +2876,14 @@ function buildDiscordBridgeToml(dc, projectId) {
 }
 
 function patchAgentchattrConfigForDiscordBridge(tomlText) {
-  if (/^\[agents\.discord-bridge\]\s*$/m.test(tomlText)) {
-    return { text: tomlText, changed: false };
+  // #439: migrate old slug if present
+  const original = tomlText;
+  tomlText = tomlText.replace(/^\[agents\.discord-bridge\]\s*$/m, "[agents.dc]");
+  if (/^\[agents\.dc\]\s*$/m.test(tomlText)) {
+    return { text: tomlText, changed: tomlText !== original };
   }
   const sep = tomlText.length === 0 || tomlText.endsWith("\n") ? "" : "\n";
-  const block = `\n[agents.discord-bridge]\nlabel = "Discord Bridge"\n`;
+  const block = `\n[agents.dc]\nlabel = "Discord Bridge"\n`;
   return { text: tomlText + sep + block, changed: true };
 }
 
@@ -3147,10 +3065,12 @@ router.post("/api/discord", async (req, res) => {
             `pip output tail:\n${pipOutput.split("\n").slice(-10).join("\n")}`,
         });
       }
-      // Patch all project AC configs with [agents.discord-bridge]
+      // #457: Patch all project AC configs with [agents.dc] and
+      // migrate old `discord-bridge` slug. Restart AC for changed projects.
       const patched = [];
       try {
         const cfgAll = JSON.parse(fs.readFileSync(CONFIG_PATH, "utf-8"));
+        const serverPort = cfgAll.port || 8400;
         for (const proj of cfgAll.projects || []) {
           if (!proj || !proj.id) continue;
           const acPath = projectAgentchattrConfigPath(proj.id);
@@ -3161,6 +3081,14 @@ router.post("/api/discord", async (req, res) => {
             if (changed) {
               fs.writeFileSync(acPath, text);
               patched.push(proj.id);
+              // #457: restart AC so it loads the new agent slug
+              setTimeout(async () => {
+                try {
+                  await fetch(`http://127.0.0.1:${serverPort}/api/agentchattr/${encodeURIComponent(proj.id)}/restart`, {
+                    method: "POST",
+                  });
+                } catch {}
+              }, 1000);
             }
           } catch {}
         }
@@ -3246,7 +3174,7 @@ router.post("/api/discord", async (req, res) => {
         const acUrl = resolveProjectAgentchattrUrl(cfg, project);
         if (acUrl) {
           const acPort = new URL(acUrl).port || "8300";
-          await fetch(`http://127.0.0.1:${acPort}/api/deregister/discord-bridge`, {
+          await fetch(`http://127.0.0.1:${acPort}/api/deregister/dc`, {
             method: "POST",
             signal: AbortSignal.timeout(3000),
           }).catch(() => {});
@@ -3374,7 +3302,7 @@ module.exports.parseActiveBatch = parseActiveBatch;
 // summarizeItems for the batch-progress fixture test.
 module.exports.buildNoPrRow = buildNoPrRow;
 module.exports.summarizeItems = summarizeItems;
-// #353: expose readLastLines for the telegram-bridge test.
+// #353: expose readLastLines for the tg-bridge test.
 module.exports.readLastLines = readLastLines;
 // #380: expose checkTelegramBridgePythonDeps so the bridge test can
 // exercise the venv-path interpreter argument round trip.
@@ -3390,6 +3318,7 @@ module.exports.checkDiscordBridgePythonDeps = checkDiscordBridgePythonDeps;
 module.exports.buildDiscordBridgeToml = buildDiscordBridgeToml;
 module.exports.patchAgentchattrConfigForDiscordBridge = patchAgentchattrConfigForDiscordBridge;
 module.exports.buildDiscordBridgeSpawnEnv = buildDiscordBridgeSpawnEnv;
+module.exports.projectAgentchattrConfigPath = projectAgentchattrConfigPath;
 // #236: expose sendViaWebSocket so the chat-ws-send regression test
 // can verify the ack/body/error paths against a fake AC ws server.
 module.exports.sendViaWebSocket = sendViaWebSocket;

package/server/routes.telegramBridge.test.js

@@ -183,7 +183,7 @@ try {
   assert.match(toml13, /agentchattr_url = "http:\/\/127\.0\.0\.1:8301"/);
   // #404: cursor_file must be per-project so multiple bridges
   // don't clobber each other's position.
-  assert.match(toml13, /cursor_file = ".*telegram-bridge-cursor-testproject\.json"/);
+  assert.match(toml13, /cursor_file = ".*tg-bridge-cursor-testproject\.json"/);
   // Must NOT emit a separate [agentchattr] section — the bridge
   // would silently ignore it.
   assert.equal(toml13.includes("\n[agentchattr]\n"), false);
@@ -196,19 +196,19 @@ try {
     "[agents.head]\nlabel = \"Head\"\n\n[agents.dev]\nlabel = \"Dev\"\n";
   const first = patchAgentchattrConfigForTelegramBridge(baseConfig);
   assert.equal(first.changed, true);
-  assert.match(first.text, /^\[agents\.telegram-bridge\]$/m);
+  assert.match(first.text, /^\[agents\.tg\]$/m);
   assert.match(first.text, /label = "Telegram Bridge"/);
   // Running a second time is a no-op.
   const second = patchAgentchattrConfigForTelegramBridge(first.text);
   assert.equal(second.changed, false);
   assert.equal(second.text, first.text);
-  // A config that was hand-patched during diagnosis is recognized
-  // as already-correct — do not clobber the operator's edit.
+  // #439: a config with old slug [agents.telegram-bridge] is migrated
+  // to [agents.tg] and flagged as changed.
   const handPatched =
     baseConfig + "\n[agents.telegram-bridge]\nlabel = \"Telegram Bridge\"\n";
   const third = patchAgentchattrConfigForTelegramBridge(handPatched);
-  assert.equal(third.changed, false);
-  assert.equal(third.text, handPatched);
+  assert.equal(third.changed, true);
+  assert.match(third.text, /^\[agents\.tg\]$/m);
 
   // 15) #383 Bug 4: buildTelegramBridgeSpawnEnv strips the three
   //     env vars the upstream bridge treats as higher-precedence

package/templates/config.toml

@@ -38,15 +38,15 @@ color = "#da7756"
 label = "Builder"
 
 # #383: AC's registry rejects bases not declared in config.toml.
-# The Telegram bridge registers as `telegram-bridge`, so every
-# per-project AC config must declare it. The bridge has no
-# command/cwd of its own — it is a long-running external client
-# that posts to AC's HTTP API.
-[agents.telegram-bridge]
+# The Telegram bridge registers as `tg` (#439: renamed from
+# `telegram-bridge`), so every per-project AC config must declare
+# it. The bridge has no command/cwd of its own — it is a long-running
+# external client that posts to AC's HTTP API.
+[agents.tg]
 label = "Telegram Bridge"
 
-# #399: Discord bridge registers as `discord-bridge`.
-[agents.discord-bridge]
+# #399/#439: Discord bridge registers as `dc` (renamed from `discord-bridge`).
+[agents.dc]
 label = "Discord Bridge"
 
 [routing]

package/out/_next/static/ZT6D996Dw9auBgcm_HHTY/_ssgManifest.js

@@ -1 +0,0 @@
-self.__SSG_MANIFEST=new Set(["\u002Fproject\u002F[id]","\u002Fproject\u002F[id]\u002Fmemory","\u002Fproject\u002F[id]\u002Fqueue"]);self.__SSG_MANIFEST_CB&&self.__SSG_MANIFEST_CB()