@agent-relay/dashboard-server 2.0.68 → 2.0.70

package/dist/server.js

@@ -1756,15 +1756,17 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
                                 existing.avatarUrl = user.avatarUrl;
                         }
                         else {
-                            const now = new Date().toISOString();
+                            // Use stable timestamps from the user/file data, not new Date(),
+                            // so getAllData() produces deterministic output for dedup comparison
+                            const stableTimestamp = user.lastSeen || user.connectedAt || new Date(remoteData.updatedAt).toISOString();
                             agentsMap.set(user.name, {
                                 name: user.name,
                                 role: 'User',
                                 cli: 'dashboard',
                                 messageCount: 0,
                                 status: 'online',
-                                lastSeen: now,
-                                lastActive: now,
+                                lastSeen: stableTimestamp,
+                                lastActive: stableTimestamp,
                                 needsAttention: false,
                                 avatarUrl: user.avatarUrl,
                             });
@@ -1954,18 +1956,21 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
             return true;
         });
         // Separate AI agents from human users
+        // Sort by name for deterministic JSON serialization (enables dedup comparison)
         const filteredAgents = validEntries
             .filter(agent => agent.cli !== 'dashboard')
             .map(agent => ({
             ...agent,
             isHuman: false,
-        }));
+        }))
+            .sort((a, b) => a.name.localeCompare(b.name));
         const humanUsers = validEntries
             .filter(agent => agent.cli === 'dashboard')
             .map(agent => ({
             ...agent,
             isHuman: true,
-        }));
+        }))
+            .sort((a, b) => a.name.localeCompare(b.name));
         return {
             agents: filteredAgents,
             users: humanUsers,
@@ -1978,6 +1983,7 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
     // Track clients that are still initializing (haven't received first data yet)
     // This prevents race conditions where broadcastData sends before initial data is sent
     const initializingClients = new WeakSet();
+    let lastBroadcastPayload = '';
     const broadcastData = async () => {
         try {
             const data = await getAllData();
@@ -1987,6 +1993,11 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
                 console.warn('[dashboard] Skipping broadcast - empty payload');
                 return;
             }
+            // Skip broadcast if data hasn't changed since last send
+            if (rawPayload === lastBroadcastPayload) {
+                return;
+            }
+            lastBroadcastPayload = rawPayload;
             // Push into buffer and wrap with sequence ID for replay support
             const seq = mainMessageBuffer.push('data', rawPayload);
             const payload = JSON.stringify({ seq, ...data });
@@ -2066,6 +2077,7 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
         }
         return { projects: [], messages: [], connected: false };
     };
+    let lastBridgeBroadcastPayload = '';
     const broadcastBridgeData = async () => {
         try {
             const data = await getBridgeData();
@@ -2075,6 +2087,11 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
                 console.warn('[dashboard] Skipping bridge broadcast - empty payload');
                 return;
             }
+            // Skip broadcast if data hasn't changed since last send
+            if (payload === lastBridgeBroadcastPayload) {
+                return;
+            }
+            lastBridgeBroadcastPayload = payload;
             wssBridge.clients.forEach(client => {
                 if (client.readyState === WebSocket.OPEN) {
                     try {
@@ -4840,6 +4857,41 @@ export async function startDashboard(portOrOptions, dataDirArg, teamDirArg, dbPa
             res.status(500).json({ success: false, error: safeMessage });
         }
     });
+    /**
+     * POST /api/repos/remove - Remove a cloned repo directory from the workspace
+     * Body: { fullName: "Owner/RepoName" }
+     * Used by cloud API when a user removes a repo from their workspace settings.
+     */
+    app.post('/api/repos/remove', async (req, res) => {
+        const { fullName } = req.body;
+        if (!fullName || typeof fullName !== 'string' || !fullName.includes('/')) {
+            return res.status(400).json({ success: false, error: 'fullName is required (e.g., "Owner/RepoName")' });
+        }
+        if (!/^[a-zA-Z0-9._-]+\/[a-zA-Z0-9._-]+$/.test(fullName)) {
+            return res.status(400).json({ success: false, error: 'Invalid repository name format' });
+        }
+        const repoName = fullName.split('/').pop();
+        const workspaceDir = process.env.WORKSPACE_DIR || path.dirname(projectRoot || dataDir);
+        const targetDir = path.join(workspaceDir, repoName);
+        // Verify the directory is inside the workspace dir (prevent path traversal)
+        const resolvedTarget = path.resolve(targetDir);
+        const resolvedWorkspace = path.resolve(workspaceDir);
+        if (!resolvedTarget.startsWith(resolvedWorkspace + path.sep)) {
+            return res.status(400).json({ success: false, error: 'Invalid path' });
+        }
+        if (!fs.existsSync(targetDir)) {
+            return res.json({ success: true, message: 'Directory does not exist', path: targetDir });
+        }
+        try {
+            fs.rmSync(targetDir, { recursive: true, force: true });
+            console.log(`[api/repos/remove] Removed directory: ${targetDir}`);
+            res.json({ success: true, path: targetDir });
+        }
+        catch (err) {
+            console.error('[api/repos/remove] Remove failed:', err.message);
+            res.status(500).json({ success: false, error: err.message || 'Remove failed' });
+        }
+    });
     /**
      * POST /api/spawn/architect - Spawn an Architect agent for bridge mode
      * Body: { cli?: string }
@@ -5781,12 +5833,15 @@ Start by greeting the project leads and asking for status updates.`;
         }
         return {};
     }
-    // Watch for changes
+    // Watch for changes - poll as a safety net for DB-backed storage mode.
+    // Real-time updates are already handled by explicit broadcastData() calls
+    // at every data mutation point (message send, spawn, release, cwd update, etc.).
+    // This interval only catches external/indirect changes (presence, DB edits).
     if (storage) {
         setInterval(() => {
             broadcastData().catch((err) => console.error('Broadcast failed', err));
             broadcastBridgeData().catch((err) => console.error('Bridge broadcast failed', err));
-        }, 1000);
+        }, 5000);
     }
     else {
         let fsWait = null;