@hamp10/agentforge 0.2.45 → 0.2.47

package/package.json

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

package/scripts/check-task-semantics.js

@@ -532,6 +532,33 @@ try {
     /current repair strategy is not converging/i,
     'repeated visual failures should force a strategy change instead of another micro-edit loop'
   );
+  assert.match(
+    worker._formatVisualFailureFocusNudge(
+      'gamma-ai: Visual warning: target hero text is clipped.',
+      'Work on the Example.com listing pages for Alpha.ai, Beta.ai, and Gamma.ai. Delete and rebuild Alpha.ai and Beta.ai from a clean start, and fix Gamma.ai so all three are visually polished.'
+    ),
+    /Keep other requested target page\(s\) intact[\s\S]*alpha-ai, beta-ai/i,
+    'visual-warning retries should focus on the failing target instead of recreating unrelated completed targets'
+  );
+  const focusedRepairLead = worker._formatVisualRepairTaskLead(
+    'Work on the Example.com listing pages for Alpha.ai, Beta.ai, and Gamma.ai. Delete and rebuild Alpha.ai and Beta.ai from a clean start, and fix Gamma.ai so all three are visually polished.',
+    'gamma-ai: Visual warning: target hero text is clipped.'
+  );
+  assert.match(
+    focusedRepairLead,
+    /visual repair pass for currently failing target page\(s\): gamma-ai/i,
+    'focused visual repair retries should name the failing target set'
+  );
+  assert.match(
+    focusedRepairLead,
+    /Do not repeat initial delete\/rebuild\/reset\/setup work[\s\S]*alpha-ai, beta-ai/i,
+    'focused visual repair retries should not replay clean-start work for already-built non-failing targets'
+  );
+  assert.doesNotMatch(
+    focusedRepairLead,
+    /The task is:|Delete and rebuild Alpha\.ai/i,
+    'focused visual repair retries should not put the original reset-heavy task text at the top of the retry prompt'
+  );
   assert.match(
     worker._buildScopedUiTargetSetNudge(baseline, message),
     /one delivery set/i,
@@ -1177,6 +1204,11 @@ assert.match(
   /_formatRepeatedVisualRepairNudge[\s\S]*current repair strategy is not converging/i,
   'repeated visual failures should inject a generic strategy-change nudge'
 );
+assert.match(
+  workerSource,
+  /_formatVisualFailureFocusNudge[\s\S]*Do not delete or recreate already-built target pages during a visual-warning repair/i,
+  'visual-warning retries should preserve completed targets outside the failing page'
+);
 assert.match(
   workerSource,
   /_formatUiVerificationFailureLogSummary[\s\S]*UI visual verification issue\(s\)/i,

package/src/worker.js

@@ -1476,6 +1476,38 @@ export class AgentForgeWorker extends EventEmitter {
     ].join('\n');
   }
 
+  _formatVisualFailureFocusNudge(output, userMessage) {
+    const { slugs: allSlugs } = this._extractExplicitScope(userMessage);
+    const failedSlugs = this._extractUiVerificationFailureSlugs(output, userMessage);
+    if (allSlugs.length < 2 || failedSlugs.length === 0 || failedSlugs.length >= allSlugs.length) return '';
+    const untouched = allSlugs.filter(slug => !failedSlugs.includes(slug));
+    return [
+      `Current visual verification warnings identify target page(s): ${failedSlugs.join(', ')}.`,
+      untouched.length > 0
+        ? `Keep other requested target page(s) intact unless their own verification reports a concrete issue: ${untouched.join(', ')}.`
+        : '',
+      'If the original task included delete/rebuild or clean-start language, treat that as already satisfied for target files that now exist. Do not delete or recreate already-built target pages during a visual-warning repair unless the warning is for that same page.',
+      'After fixing the failing target-owned structure, re-verify every requested target page locally before commit or completion.',
+    ].filter(Boolean).join('\n');
+  }
+
+  _formatVisualRepairTaskLead(userMessage, output) {
+    const { slugs: allSlugs } = this._extractExplicitScope(userMessage);
+    const failedSlugs = this._extractUiVerificationFailureSlugs(output, userMessage);
+    if (allSlugs.length >= 2 && failedSlugs.length > 0 && failedSlugs.length < allSlugs.length) {
+      const preservedSlugs = allSlugs.filter(slug => !failedSlugs.includes(slug));
+      return [
+        `Original scoped UI target set: ${allSlugs.join(', ')}.`,
+        `This retry is a visual repair pass for currently failing target page(s): ${failedSlugs.join(', ')}.`,
+        preservedSlugs.length > 0
+          ? `Do not repeat initial delete/rebuild/reset/setup work for target page(s) not currently failing: ${preservedSlugs.join(', ')}.`
+          : '',
+        'Continue from the current repo state and change only files needed to repair the failing target page(s), then re-verify every requested target page locally before completion or delivery.',
+      ].filter(Boolean).join('\n');
+    }
+    return `The task is: "${userMessage}"`;
+  }
+
   _formatUiVerificationFailureLogSummary(nudge) {
     const lines = String(nudge || '')
       .split('\n')
@@ -4409,13 +4441,15 @@ export class AgentForgeWorker extends EventEmitter {
               const repairBudget = consumeUiRepairNudge('visual verification warnings', uiVerificationFailureNudge);
               const scopedTargetSetNudge = this._formatScopedUiTargetSetReminder(scopeAwareUserMessage);
               const repeatedVisualRepairNudge = this._formatRepeatedVisualRepairNudge(uiRepairNudgeCount);
+              const visualFailureFocusNudge = this._formatVisualFailureFocusNudge(output, scopeAwareUserMessage);
+              const visualRepairTaskLead = this._formatVisualRepairTaskLead(scopeAwareUserMessage, output);
               const visualFailureSummary = this._formatUiVerificationFailureLogSummary(uiVerificationFailureNudge);
               if (nudgeCount < MAX_NUDGES) {
                 console.log(`[${taskId}] UI completion still had visual verification warnings — nudging (${nudgeCount}/${MAX_NUDGES}, total UI repairs ${repairBudget})`);
                 if (visualFailureSummary) {
                   console.log(`[${taskId}] UI visual verification issue(s): ${visualFailureSummary}`);
                 }
-                iterationMessage = withTaskContext(`The task is: "${userMessage}"\n\n${[uiVerificationFailureNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\nFix the visible UI issues, verify again with the browser, and only then end with ✓ TASK_COMPLETE.`);
+                iterationMessage = withTaskContext(`${visualRepairTaskLead}\n\n${[uiVerificationFailureNudge, visualFailureFocusNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\nFix the visible UI issues, verify again with the browser, and only then end with ✓ TASK_COMPLETE.`);
                 continue;
               }
               throw new Error('UI task claimed completion while visual verification still reported issues');
@@ -4681,13 +4715,15 @@ export class AgentForgeWorker extends EventEmitter {
                 const repairBudget = consumeUiRepairNudge('publish visual verification warnings', uiVerificationFailureNudge);
                 const scopedTargetSetNudge = this._formatScopedUiTargetSetReminder(scopeAwareUserMessage);
                 const repeatedVisualRepairNudge = this._formatRepeatedVisualRepairNudge(uiRepairNudgeCount);
+                const visualFailureFocusNudge = this._formatVisualFailureFocusNudge(output, scopeAwareUserMessage);
+                const visualRepairTaskLead = this._formatVisualRepairTaskLead(scopeAwareUserMessage, output);
                 const visualFailureSummary = this._formatUiVerificationFailureLogSummary(uiVerificationFailureNudge);
                 if (nudgeCount < MAX_NUDGES) {
                   console.log(`[${taskId}] Publish evidence still had visual verification warnings — continuing (${nudgeCount}/${MAX_NUDGES}, total UI repairs ${repairBudget})`);
                   if (visualFailureSummary) {
                     console.log(`[${taskId}] UI visual verification issue(s): ${visualFailureSummary}`);
                   }
-                  iterationMessage = withTaskContext(`The task is: "${userMessage}"\n\n${[uiVerificationFailureNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\nContinue from the current repo state, fix the visible UI issues, verify again, then commit/push any additional changes before reporting delivery complete.`);
+                  iterationMessage = withTaskContext(`${visualRepairTaskLead}\n\n${[uiVerificationFailureNudge, visualFailureFocusNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\nContinue from the current repo state, fix the visible UI issues, verify again, then commit/push any additional changes before reporting delivery complete.`);
                   continue;
                 }
                 throw new Error('Publish task still had visual verification issues after repeated retries');
@@ -4825,6 +4861,8 @@ export class AgentForgeWorker extends EventEmitter {
             );
             const scopedTargetSetNudge = this._formatScopedUiTargetSetReminder(scopeAwareUserMessage);
             const repeatedVisualRepairNudge = this._formatRepeatedVisualRepairNudge(uiVerificationRetryCount);
+            const visualFailureFocusNudge = this._formatVisualFailureFocusNudge(output, scopeAwareUserMessage);
+            const visualRepairTaskLead = this._formatVisualRepairTaskLead(scopeAwareUserMessage, output);
             const visualFailureSummary = this._formatUiVerificationFailureLogSummary(visualVerificationFailureNudge);
             nudgeCount = 0;
             console.log(`[${taskId}] UI task visual verification still reported visible issues — retrying (${uiVerificationRetryCount}/${UI_REPAIR_NUDGE_LIMIT}, total UI repairs ${repairBudget})`);
@@ -4834,7 +4872,7 @@ export class AgentForgeWorker extends EventEmitter {
             const retryInstruction = generatedResetNudge
               ? 'Rebuild the removed target page file(s) from already-inspected project context and comparable existing pages. Do not reuse the rejected generated page as the basis for the next attempt. Address every requested target that is currently missing or changed in one coherent pass, fix the visible issues reported by local browser verification, reopen each edited target screen locally after the final edit, and only then end with ✓ TASK_COMPLETE.'
               : 'Continue from the current changed files and latest browser evidence. Fix the visible issues reported by the local browser verification, reopen each edited target screen locally after the final edit, and only then end with ✓ TASK_COMPLETE.';
-            iterationMessage = withTaskContext(`The task is: "${userMessage}"\n\n${[visualVerificationFailureNudge, generatedResetNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\n${retryInstruction}`);
+            iterationMessage = withTaskContext(`${visualRepairTaskLead}\n\n${[visualVerificationFailureNudge, visualFailureFocusNudge, generatedResetNudge, scopedTargetSetNudge, repeatedVisualRepairNudge].filter(Boolean).join('\n\n')}\n\n${retryInstruction}`);
           } else if (hasMissingLocalUiVerification) {
             uiVerificationRetryCount++;
             const uiVerificationFailureDetails = this._extractUiVerificationFailureDetails(output);