@ian2018cs/agenthub 0.1.78 → 0.1.79

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ian2018cs/agenthub",
-  "version": "0.1.78",
+  "version": "0.1.79",
   "description": "A web-based UI for AI Agents",
   "type": "module",
   "main": "server/index.js",

package/server/routes/agents.js

@@ -347,6 +347,47 @@ async function ensureSharedGitRepo(repoUrl, branch) {
   return targetPath;
 }
 
+/**
+ * Try to move an existing local git clone directly into the shared location,
+ * avoiding a fresh network clone. Only succeeds when:
+ *   - localPath is a real directory (not a symlink) with a .git folder
+ *   - The shared target path does not yet exist
+ * Returns the shared path on success, or null if promotion is not possible.
+ */
+async function tryPromoteLocalRepoToShared(localPath, repoUrl, branch) {
+  const parsed = parseGitUrl(repoUrl);
+  if (!parsed) return null;
+
+  const publicPaths = getPublicPaths();
+  const targetPath = path.join(publicPaths.gitRepoDir, parsed.owner, parsed.repo, branch);
+
+  // Shared repo already exists — nothing to promote
+  try {
+    await fs.access(targetPath);
+    return null;
+  } catch {}
+
+  // Source must be a real git directory (not a symlink, must have .git)
+  try {
+    const stat = await fs.lstat(localPath);
+    if (!stat.isDirectory() || stat.isSymbolicLink()) return null;
+    await fs.access(path.join(localPath, '.git'));
+  } catch {
+    return null;
+  }
+
+  try {
+    await fs.mkdir(path.dirname(targetPath), { recursive: true });
+    await fs.rename(localPath, targetPath);
+    console.log(`[GitRepo] Promoted local repo to shared: ${localPath} → ${targetPath}`);
+    return targetPath;
+  } catch (e) {
+    // rename may fail on cross-device links — fall back to normal clone
+    console.warn(`[GitRepo] Could not promote local repo (will clone instead): ${e.message}`);
+    return null;
+  }
+}
+
 /**
  * Create a symlink from a project subdirectory to a shared git repo.
  * Removes existing directory or symlink at the target location.
@@ -513,6 +554,7 @@ async function scanClaudeMdRefs(projectPath) {
     if (!ref) continue;
     if (/^https?:\/\/|^mailto:|^\/\//.test(ref)) continue; // skip absolute URLs
     if (path.isAbsolute(ref)) continue; // skip absolute paths
+    if (ref.endsWith('/')) continue; // skip directory references like [src/utils/]
     if (seen.has(ref)) continue;
     seen.add(ref);
 
@@ -1530,8 +1572,22 @@ router.post('/submissions/:id/approve', async (req, res) => {
         for (const gitRepo of yamlGitRepos) {
           if (!gitRepo.name || !gitRepo.repo) continue;
           try {
-            // Ensure shared git repo exists
-            const sharedPath = await ensureSharedGitRepo(gitRepo.repo, gitRepo.branch || 'main');
+            // Try to promote the submitter's existing local clone to shared location first
+            // (avoids a full network re-clone if the user already has the repo cloned)
+            let sharedPath = null;
+            if (submitterUuid) {
+              const submitterProjectDir = path.join(getUserPaths(submitterUuid).projectsDir, submission.agent_name);
+              const localRepoPath = path.join(submitterProjectDir, gitRepo.name);
+              sharedPath = await tryPromoteLocalRepoToShared(localRepoPath, gitRepo.repo, gitRepo.branch || 'main');
+              if (sharedPath) {
+                console.log(`[AgentApprove] Promoted local repo "${gitRepo.name}" to shared (no re-clone needed)`);
+              }
+            }
+
+            // Fall back to normal clone/update if promotion was not possible
+            if (!sharedPath) {
+              sharedPath = await ensureSharedGitRepo(gitRepo.repo, gitRepo.branch || 'main');
+            }
             console.log(`[AgentApprove] Ensured shared git repo "${gitRepo.name}" at ${sharedPath}`);
 
             // Replace submitter's original subdirectory with symlink to shared repo