@hamp10/agentforge 0.2.25 → 0.2.26

package/package.json

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

package/scripts/check-task-semantics.js

@@ -668,6 +668,7 @@ try {
 const openClawSource = readFileSync(new URL('../src/OpenClawCLI.js', import.meta.url), 'utf-8');
 const workerSource = readFileSync(new URL('../src/worker.js', import.meta.url), 'utf-8');
 const workerBinSource = readFileSync(new URL('../bin/agentforge.js', import.meta.url), 'utf-8');
+const selfUpdateSource = readFileSync(new URL('../src/selfUpdate.js', import.meta.url), 'utf-8');
 const defaultGuidesSource = readFileSync(new URL('../src/default-task-guides.js', import.meta.url), 'utf-8');
 const browserSource = readFileSync(new URL('../src/browser.js', import.meta.url), 'utf-8');
 const previewServerSource = readFileSync(new URL('../src/preview-server.js', import.meta.url), 'utf-8');
@@ -838,16 +839,41 @@ assert.match(
   /app\.delete\('\/api\/agents\/delete\/cleanup-empty'[\s\S]*activeTasks[\s\S]*type: 'agents_pruned'/i,
   'empty-agent cleanup should preserve live tasks and broadcast removed agents'
 );
+assert.match(
+  serverSource,
+  /CREATE TABLE IF NOT EXISTS agent_flows[\s\S]*config JSONB DEFAULT NULL/i,
+  'agent flow emergency schema patch should include config so model routing survives skipped migrations'
+);
+assert.match(
+  serverSource,
+  /ALTER TABLE agent_flows ADD COLUMN IF NOT EXISTS config JSONB DEFAULT NULL/i,
+  'agent flow startup should self-heal missing config column'
+);
+assert.match(
+  serverSource,
+  /ON CONFLICT \(id\)[\s\S]*WHERE agent_flows\.user_id = EXCLUDED\.user_id[\s\S]*RETURNING id/i,
+  'flow upserts should not update another user flow with the same id'
+);
 assert.match(
   dashboardSource,
   /confirmDeleteAgent[\s\S]*await fetch[\s\S]*removeAgentFromLocalState/i,
   'single-agent delete should only remove local UI after the server confirms deletion'
 );
+assert.match(
+  dashboardSource,
+  /confirmPruneAgents[\s\S]*\/api\/agents\/delete\/prune\?keep=15[\s\S]*removeAgentFromLocalState/i,
+  'dashboard should expose old-agent pruning through the safe keep-latest endpoint'
+);
 assert.match(
   dashboardSource,
   /function finalizeLiveFeedBeforeGuide[\s\S]*commitChunkBuffer[\s\S]*hideTypingIndicator/i,
   'live Guide messages should close the current visible agent feed before rendering the user bubble'
 );
+assert.match(
+  selfUpdateSource,
+  /ensureWritableGlobalInstallEnv[\s\S]*canWriteNpmRoot[\s\S]*configureUserOwnedNpmGlobalEnv/i,
+  'self-update should preflight npm global permissions before printing a failed system-prefix install'
+);
 assert.match(
   dashboardSource,
   /async function sendGuideMessage[\s\S]*agent\._guideInflight[\s\S]*finalizeLiveFeedBeforeGuide\(agentId\)[\s\S]*addMessage/i,
@@ -953,6 +979,16 @@ assert.doesNotMatch(
   /taskRequiresVisualVerification\s*&&\s*directIteration\s*<=\s*1\s*&&\s*explicitScopeForTask\.pageOnly/s,
   'comparable UI context gate should apply on retries, not only first iteration'
 );
+assert.match(
+  openClawSource,
+  /read-only project context, not as a target page/,
+  'read-only comparable page inspection should not be narrated as target-page work'
+);
+assert.match(
+  openClawSource,
+  /verifiedLocalUrl = isLocalUiUrl\(resultUrl \|\| url\)/,
+  'scoped auto-verification should count a clean inferred local target URL even when browser output formatting is imperfect'
+);
 assert.match(
   openClawSource,
   /const minRatio = isLargeText \? 3 : 4\.5;/,

package/src/OpenClawCLI.js

@@ -4629,8 +4629,10 @@ export class OpenClawCLI extends EventEmitter {
           recordDirectToolSummary('browser', result);
           continue;
         }
-        rememberLocalScopedUrl(browserResultUrl(result));
-        if (isLocalUiVerificationResult(result)) {
+        const resultUrl = browserResultUrl(result);
+        rememberLocalScopedUrl(resultUrl);
+        const verifiedLocalUrl = isLocalUiUrl(resultUrl || url);
+        if (isLocalUiVerificationResult(result) || verifiedLocalUrl) {
           recordCleanLocalVerification(result, [slug]);
           verifiedAny = true;
         }
@@ -4766,6 +4768,13 @@ export class OpenClawCLI extends EventEmitter {
         return `I am checking the live browser${url} to judge the actual product surface before deciding the next change.`;
       }
       if (name === 'read_file' || name === 'read') {
+        const scope = extractExplicitScope(task);
+        const targetText = String(targetPath || base || '').toLowerCase();
+        const looksLikePageSource = /\.(html?|css|s[ac]ss|jsx?|tsx?|vue|svelte|astro|mdx?)$/i.test(targetText);
+        const isNamedTarget = scope.slugs.length > 0 && scopeSlugsMatchingText(targetText, scope.slugs).length > 0;
+        if (base && looksLikePageSource && scope.slugs.length > 0 && !isNamedTarget) {
+          return `I am reading ${base} as read-only project context, not as a target page.`;
+        }
         return base
           ? `I am reading ${base} to work from the current implementation instead of guessing.`
           : `I am reading the current code before making the next change.`;
@@ -5097,6 +5106,10 @@ export class OpenClawCLI extends EventEmitter {
                     lastDirectVisualWarning = visualWarning;
                   } else if (!taskRequiresVisualVerification || isLocalUiVerificationResult(result)) {
                     recordCleanLocalVerification(result);
+                  } else {
+                    const verifiedUrl = resultUrl || args?.url || '';
+                    const verifiedSlugs = isLocalUiUrl(verifiedUrl) ? scopeSlugsInText(verifiedUrl) : [];
+                    if (verifiedSlugs.length > 0) recordCleanLocalVerification(result, verifiedSlugs);
                   }
                 }
               }

package/src/selfUpdate.js

@@ -6,8 +6,8 @@
  * prevent infinite re-exec loops).
  */
 
-import { execSync, spawn } from 'child_process';
-import { mkdirSync } from 'fs';
+import { execFileSync, execSync, spawn } from 'child_process';
+import { mkdirSync, rmSync } from 'fs';
 import { homedir } from 'os';
 import path from 'path';
 
@@ -75,22 +75,58 @@ export async function checkAndUpdate(packageName, currentVersion) {
 
 function installGlobalPackage(pkg, { force = false } = {}) {
   const forceFlag = force ? ' --force' : '';
+  const preflightEnv = ensureWritableGlobalInstallEnv();
   try {
-    execSync(`npm install -g${forceFlag} ${pkg}`, { stdio: 'inherit' });
+    execSync(`npm install -g${forceFlag} ${pkg}`, { stdio: 'inherit', env: preflightEnv });
     return;
   } catch (error) {
     if (!looksLikePermissionFailure(error)) throw error;
   }
 
+  const env = configureUserOwnedNpmGlobalEnv();
+  execSync(`npm install -g${forceFlag} ${pkg}`, { stdio: 'inherit', env });
+}
+
+function npmConfigGet(args) {
+  try {
+    return execFileSync('npm', args, {
+      encoding: 'utf8',
+      stdio: ['ignore', 'pipe', 'ignore'],
+    }).trim();
+  } catch {
+    return '';
+  }
+}
+
+function canWriteNpmRoot(rootDir) {
+  if (!rootDir) return false;
+  const testDir = path.join(rootDir, `.agentforge-write-test-${process.pid}`);
+  try {
+    mkdirSync(testDir, { recursive: true });
+    rmSync(testDir, { recursive: true, force: true });
+    return true;
+  } catch {
+    try { rmSync(testDir, { recursive: true, force: true }); } catch {}
+    return false;
+  }
+}
+
+function ensureWritableGlobalInstallEnv() {
+  const root = npmConfigGet(['root', '-g']);
+  if (canWriteNpmRoot(root)) return process.env;
+  return configureUserOwnedNpmGlobalEnv();
+}
+
+function configureUserOwnedNpmGlobalEnv() {
   const prefix = path.join(homedir(), '.npm-global');
   mkdirSync(path.join(prefix, 'bin'), { recursive: true });
   mkdirSync(path.join(prefix, 'lib', 'node_modules'), { recursive: true });
-  execSync(`npm config set prefix "${prefix}"`, { stdio: 'inherit' });
+  execFileSync('npm', ['config', 'set', 'prefix', prefix], { stdio: ['ignore', 'ignore', 'inherit'] });
   const env = {
     ...process.env,
     PATH: `${path.join(prefix, 'bin')}${path.delimiter}${process.env.PATH || ''}`,
   };
-  execSync(`npm install -g${forceFlag} ${pkg}`, { stdio: 'inherit', env });
+  return env;
 }
 
 function looksLikePermissionFailure(error) {