@ian2018cs/agenthub 0.1.75 → 0.1.76

package/package.json

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

package/server/routes/agents.js

@@ -5,7 +5,7 @@ import path from 'path';
 import { spawn } from 'child_process';
 import AdmZip from 'adm-zip';
 import { getUserPaths, getPublicPaths } from '../services/user-directories.js';
-import { scanAgents, ensureAgentRepo, incrementPatchVersion, publishAgentToRepo, invalidateAgentCache } from '../services/system-agent-repo.js';
+import { scanAgents, ensureAgentRepo, incrementPatchVersion, publishAgentToRepo } from '../services/system-agent-repo.js';
 import { ensureSystemRepo, SYSTEM_REPO_URL } from '../services/system-repo.js';
 import { ensureSystemMcpRepo, SYSTEM_MCP_REPO_URL } from '../services/system-mcp-repo.js';
 import { addProjectManually, loadProjectConfig, saveProjectConfig } from '../projects.js';
@@ -1310,7 +1310,7 @@ router.post('/submissions/:id/approve', async (req, res) => {
     if (updateAllUsers) {
       // Force-update all users who have this agent installed
       try {
-        const freshAgents = await scanAgents(true);
+        const freshAgents = await scanAgents();
         const publishedAgent = freshAgents.find(a =>
           a.dirName === submission.agent_name || a.name === submission.agent_name
         );

package/server/services/system-agent-repo.js

@@ -7,13 +7,6 @@ export const SYSTEM_AGENT_REPO_URL = 'git@git.amberweather.com:mcp-server/hupoer
 export const SYSTEM_AGENT_REPO_OWNER = 'mcp-server';
 export const SYSTEM_AGENT_REPO_NAME = 'hupoer-agents';
 
-// ─── In-memory cache ──────────────────────────────────────────────────────────
-const CACHE_TTL_MS = 5 * 60 * 1000; // 5 minutes
-let agentCache = { agents: null, cachedAt: 0 };
-
-export function invalidateAgentCache() {
-  agentCache = { agents: null, cachedAt: 0 };
-}
 
 function runGit(args, cwd = null) {
   return new Promise((resolve, reject) => {
@@ -42,7 +35,7 @@ function getRepoPath() {
 }
 
 /**
- * Ensure the agent repo is cloned locally. If already present, tries to pull.
+ * Ensure the agent repo is cloned locally. Does NOT pull; call pullAgentRepo() explicitly.
  * Returns the path to the local clone.
  */
 export async function ensureAgentRepo() {
@@ -50,12 +43,6 @@ export async function ensureAgentRepo() {
 
   try {
     await fs.access(repoPath);
-    // Already cloned — try to pull latest
-    try {
-      await runGit(['pull', '--ff-only'], repoPath);
-    } catch (err) {
-      console.log('[AgentRepo] Failed to pull, using existing clone:', err.message);
-    }
   } catch {
     // Not yet cloned
     await fs.mkdir(path.dirname(repoPath), { recursive: true });
@@ -71,6 +58,19 @@ export async function ensureAgentRepo() {
   return repoPath;
 }
 
+/**
+ * Pull latest from remote. Called explicitly on user-triggered refresh.
+ */
+export async function pullAgentRepo() {
+  const repoPath = getRepoPath();
+  try {
+    await runGit(['pull', '--ff-only'], repoPath);
+  } catch (err) {
+    console.log('[AgentRepo] Failed to pull:', err.message);
+    throw err;
+  }
+}
+
 /**
  * Parse agent.yaml from an agent directory.
  * Returns the parsed metadata object or null if invalid.
@@ -161,23 +161,25 @@ async function parseAgentYaml(agentDir) {
 
 /**
  * Scan the agent repo for available agents.
- * @param {boolean} force - When true, always git pull and re-scan (bypasses cache).
- *                          When false (default), returns cached result if < 5 min old.
+ * @param {boolean} pull - When true, git pull before scanning (e.g. user-triggered refresh).
+ *                         When false (default), read directly from local disk — fast path.
  * Returns array of agent metadata objects.
  */
-export async function scanAgents(force = false) {
-  // Return cached result if fresh and not forced
-  if (!force && agentCache.agents !== null && Date.now() - agentCache.cachedAt < CACHE_TTL_MS) {
-    return agentCache.agents;
-  }
-
+export async function scanAgents(pull = false) {
   let repoPath;
   try {
     repoPath = await ensureAgentRepo();
   } catch (err) {
     console.error('[AgentRepo] Could not access agent repo:', err.message);
-    // Return stale cache on network error rather than empty list
-    return agentCache.agents ?? [];
+    return [];
+  }
+
+  if (pull) {
+    try {
+      await pullAgentRepo();
+    } catch (err) {
+      console.warn('[AgentRepo] Pull failed, scanning existing local data:', err.message);
+    }
   }
 
   const agents = [];
@@ -211,7 +213,6 @@ export async function scanAgents(force = false) {
     });
   }
 
-  agentCache = { agents, cachedAt: Date.now() };
   return agents;
 }