@hamp10/agentforge 0.2.20 → 0.2.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hamp10/agentforge",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "description": "AgentForge worker — connect your machine to agentforge.ai",
   "type": "module",
   "bin": {

package/src/OllamaAgent.js

@@ -418,6 +418,7 @@ B6. npm install: cd ${projectsDir}/PROJECT_NAME && /usr/local/bin/npm init -y &&
 B7. After starting server, verify: sleep 3 && curl -s -o /dev/null -w '%{http_code}' http://localhost:PORT — if 000, check /tmp/server.log and fix the error.
 B8. PORT MANAGEMENT: Check port before starting: lsof -i :PORT | head -3. If in use: kill old process, restart. If crashed: restart. If busy with something else: pick different port.
 B9. EXPRESS WILDCARD ROUTE: NEVER write app.get('*', ...) — crashes in newer versions. Use app.use((req, res) => { ... }) instead.
+B10a. STATIC FILE PATHS: ALWAYS use path.join(__dirname, 'public') for express.static and file reads — NEVER './public' or 'public'. Relative paths break when server starts with nohup from a different directory. __dirname is always the directory containing server.js.
 B10. MANDATORY SCREENSHOT QA: After curl returns 200, call screenshot_and_describe with send_to_user:true. You are NOT done until the screenshot shows the real working app.
 B11. ALWAYS open the finished app: bash open http://localhost:PORT
 B12. CANVAS GAMES: canvas 800×600, dark background #1a1a2e, all elements clearly visible. Dark theme, styled UI.
@@ -874,6 +875,22 @@ B16. TEST LIKE A USER: Scroll, click buttons, simulate actions, check different
               this.emit('agent_image', { agentId, image: result });
             }
 
+            // ── Bash: curl returned 000 or 404 — force log read and targeted fix ──
+            if (name === 'bash') {
+              const resultStr = String(result).trim();
+              const isCurlZero = resultStr === '000' || resultStr.endsWith('\n000') || /\b000$/.test(resultStr);
+              const isCurl404 = resultStr === '404' || resultStr.endsWith('\n404') || /\b404$/.test(resultStr);
+              if (isCurlZero || isCurl404) {
+                const logRead = await this._executeTool('bash', { command: 'cat /tmp/server.log 2>/dev/null | tail -40 || echo "No server.log found"' }, workDir, agentId);
+                if (isCurlZero) {
+                  messages.push({ role: 'user', content: `[bash result]: 000 (connection refused — server is NOT running)\n\nCrash log:\n${logRead}\n\nThe server crashed or never started. Fix the actual error shown above. Do NOT assume it is running. Do NOT change the port. Make a targeted fix to the code then restart.` });
+                } else {
+                  messages.push({ role: 'user', content: `[bash result]: 404 (server is running but root route not found)\n\nServer log:\n${logRead}\n\nCommon cause: static files path is wrong. In server.js ALWAYS use path.join(__dirname, 'public') — NEVER './public' or 'public' — relative paths break when server is started with nohup. Fix the static file path and restart. Do NOT rewrite the whole file.` });
+                }
+                continue;
+              }
+            }
+
             // ALL models get tool results fed back — no model should run blind.
             // This is the core of the observe → reason → act loop: every tool result
             // must be in context so the model can see what happened and react correctly.
@@ -889,11 +906,12 @@ B16. TEST LIKE A USER: Scroll, click buttons, simulate actions, check different
                 if (name === 'screenshot_and_describe') {
                   const screenshotResult = String(result);
                   const isLocalhost = (parsedArgs.url || '').includes('localhost') || (parsedArgs.url || '').includes('127.0.0.1');
-                  // Server unreachable on localhost — force bash restart (only for local servers, not public URLs)
+                  // Server unreachable on localhost — read crash log and force a fix
                   if (screenshotResult.includes('SERVER IS NOT REACHABLE') && isLocalhost) {
                     const portMatch = (parsedArgs.url || '').match(/:(\d+)/);
                     const port = portMatch ? portMatch[1] : '????';
-                    messages.push({ role: 'user', content: `The local server on port ${port} is not running. Restart it with bash — find the project directory, then: pkill -f 'node.*${port}' 2>/dev/null; sleep 1; cd /path/to/project && nohup /usr/local/bin/node server.js > /tmp/server.log 2>&1 & sleep 3 && curl -s -o /dev/null -w '%{http_code}' http://localhost:${port}` });
+                    const logRead = await this._executeTool('bash', { command: 'cat /tmp/server.log 2>/dev/null | tail -40 || echo "No server.log found"' }, workDir, agentId);
+                    messages.push({ role: 'user', content: `The server on port ${port} is NOT running. Here are the crash logs:\n\n${logRead}\n\nThe server crashed or failed to start. Read the error above and fix the bug in your code. Then restart:\n  pkill -f 'node.*${port}' 2>/dev/null; nohup /usr/local/bin/node server.js > /tmp/server.log 2>&1 &\nDo NOT assume it is running. Do NOT change the port. Fix the actual error first.` });
                   }
                   // Public URL unreachable — try web_fetch instead
                   else if (screenshotResult.includes('SERVER IS NOT REACHABLE') && !isLocalhost) {