npm - @hamp10/agentforge - Versions diffs - 0.2.40 → 0.2.41 - Mend

@hamp10/agentforge 0.2.40 → 0.2.41

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/package.json +1 -1
package/scripts/check-task-semantics.js +43 -0
package/src/worker.js +42 -4

package/package.json CHANGED Viewed

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

package/scripts/check-task-semantics.js CHANGED Viewed

@@ -340,6 +340,49 @@ try {
     /removed 1 rejected generated scoped file/i,
     'retry feedback should tell the agent that generated scoped pages were removed as failed output'
   );
+  const generatedDeltaRel = 'public_html/domains/delta.html';
+  const generatedEpsilonRel = 'public_html/domains/epsilon.html';
+  writeFileSync(
+    path.join(fixture.repo, generatedDeltaRel),
+    '<!doctype html><html><body><main><section><h1>Delta</h1><p>New scoped page.</p></section></main></body></html>'
+  );
+  writeFileSync(
+    path.join(fixture.repo, generatedEpsilonRel),
+    '<!doctype html><html><body><main><section><h1>Epsilon</h1><p>New scoped page.</p></section></main></body></html>'
+  );
+  const multiGeneratedQualityMessage = 'Work on the Example.com listing pages for Delta.ai and Epsilon.ai. Make both pages visually polished.';
+  assert.deepEqual(
+    worker._extractUiVerificationFailureSlugs(
+      'delta.html: Visual warning: hero text fails local contrast verification.',
+      multiGeneratedQualityMessage
+    ),
+    ['delta-ai'],
+    'visual verification parsing should identify the scoped page named by a page-prefixed warning'
+  );
+  assert.deepEqual(
+    worker._extractUiVerificationFailureSlugs(
+      'Visual warning: hero text fails local contrast verification.',
+      multiGeneratedQualityMessage
+    ),
+    [],
+    'generic visual warnings should not be treated as evidence to reset every generated scoped page'
+  );
+  worker._restoreGeneratedScopedUiTargets(
+    [{ root: fixture.repo, head: fixture.head, initialDirtyPaths: [] }],
+    multiGeneratedQualityMessage,
+    { onlySlugs: ['delta-ai'] }
+  );
+  assert.equal(
+    existsSync(path.join(fixture.repo, generatedDeltaRel)),
+    false,
+    'page-aware generated resets should remove the rejected generated page'
+  );
+  assert.equal(
+    existsSync(path.join(fixture.repo, generatedEpsilonRel)),
+    true,
+    'page-aware generated resets should preserve unrelated generated scoped pages'
+  );
+  rmSync(path.join(fixture.repo, generatedEpsilonRel), { force: true });
   const projectsRoot = mkdtempSync(path.join(tmpdir(), 'agentforge-project-list-'));
   let agentWorkspace = null;
   try {

package/src/worker.js CHANGED Viewed

@@ -945,11 +945,19 @@ export class AgentForgeWorker extends EventEmitter {
     return restored;
   }
-  _restoreGeneratedScopedUiTargets(repoBaselines, userMessage) {
+  _scopedUiPathMatchesAnySlug(relativePath, slugs) {
+    const matches = this._scopeSlugsMatchingText(relativePath, slugs);
+    return matches.length > 0;
+  }
+  _restoreGeneratedScopedUiTargets(repoBaselines, userMessage, options = {}) {
     if (!this._isBroadUiQualityTask(userMessage)) return [];
     if (!Array.isArray(repoBaselines) || repoBaselines.length === 0) return [];
     const { slugs: allowedSlugs, pageOnly } = this._extractExplicitScope(userMessage);
     if (allowedSlugs.length === 0 || !pageOnly) return [];
+    const onlySlugs = Array.isArray(options.onlySlugs)
+      ? [...new Set(options.onlySlugs.map(slug => this._scopeSlug(slug)).filter(Boolean))]
+      : [];
     const restored = [];
     for (const baseline of repoBaselines) {
@@ -970,6 +978,7 @@ export class AgentForgeWorker extends EventEmitter {
       const files = [...names]
         .filter(rel => !initialDirty.has(rel))
         .filter(rel => this._scopeAllowsChangedPath(baseline, rel, allowedSlugs, pageOnly, userMessage))
+        .filter(rel => onlySlugs.length === 0 || this._scopedUiPathMatchesAnySlug(rel, onlySlugs))
         .filter(rel => !this._gitPathExistsAtRef(baseline.root, baseline.head || 'HEAD', rel))
         .sort();
       if (files.length === 0) continue;
@@ -4765,8 +4774,11 @@ export class AgentForgeWorker extends EventEmitter {
               throw new Error('UI task failed visual verification after repeated repair attempts');
             }
             const repairBudget = consumeUiRepairNudge('visual verification warnings', visualVerificationFailureNudge);
+            const visualFailureSlugs = this._extractUiVerificationFailureSlugs(output, scopeAwareUserMessage);
             const generatedResetNudge = this._formatGeneratedScopedUiResetNudge(
-              this._restoreGeneratedScopedUiTargets(repoBaselines, scopeAwareUserMessage)
+              visualFailureSlugs.length > 0
+                ? this._restoreGeneratedScopedUiTargets(repoBaselines, scopeAwareUserMessage, { onlySlugs: visualFailureSlugs })
+                : []
             );
             nudgeCount = 0;
             console.log(`[${taskId}] UI task visual verification still reported visible issues — retrying (${uiVerificationRetryCount}/${UI_REPAIR_NUDGE_LIMIT}, total UI repairs ${repairBudget})`);
@@ -4782,8 +4794,11 @@ export class AgentForgeWorker extends EventEmitter {
               throw new Error('UI task failed visual verification after repeated repair attempts');
             }
             const repairBudget = consumeUiRepairNudge('missing local visual verification', uiVerificationFailureDetails);
+            const visualFailureSlugs = this._extractUiVerificationFailureSlugs(output, scopeAwareUserMessage);
             const generatedResetNudge = this._formatGeneratedScopedUiResetNudge(
-              this._restoreGeneratedScopedUiTargets(repoBaselines, scopeAwareUserMessage)
+              visualFailureSlugs.length > 0
+                ? this._restoreGeneratedScopedUiTargets(repoBaselines, scopeAwareUserMessage, { onlySlugs: visualFailureSlugs })
+                : []
             );
             nudgeCount = 0;
             console.log(`[${taskId}] UI task missing local visual verification — retrying with local-app repair instruction (${uiVerificationRetryCount}/${UI_REPAIR_NUDGE_LIMIT}, total UI repairs ${repairBudget})`);
@@ -5876,7 +5891,7 @@ end tell`.trim().replace(/\n/g, '\n');
       .split('\n')
       .map(line => line.trim())
       .filter(line =>
-        /^(?:Style|Visual) warning:/i.test(line) ||
+        /(?:^|[\s:])(?:Style|Visual) warning:/i.test(line) ||
         /Browser failed to load/i.test(line) ||
         /changed UI was not successfully loaded and inspected/i.test(line) ||
         /latest visual verification still reported/i.test(line) ||
@@ -5886,6 +5901,29 @@ end tell`.trim().replace(/\n/g, '\n');
     return unique.join('\n').slice(0, 1800);
   }
+  _extractUiVerificationFailureSlugs(output, userMessage) {
+    const { slugs: allowedSlugs } = this._extractExplicitScope(userMessage);
+    if (allowedSlugs.length === 0) return [];
+    const lines = String(output || '')
+      .split('\n')
+      .map(line => line.trim())
+      .filter(line =>
+        /(?:^|[\s:])(?:Style|Visual) warning:/i.test(line) ||
+        /Browser failed to load/i.test(line) ||
+        /changed UI was not successfully loaded and inspected/i.test(line) ||
+        /latest visual verification still reported/i.test(line) ||
+        /This is not complete/i.test(line)
+      );
+    if (lines.length === 0) return [];
+    const matched = new Set();
+    for (const line of lines) {
+      for (const slug of this._scopeSlugsMatchingText(line, allowedSlugs)) {
+        matched.add(slug);
+      }
+    }
+    return [...matched];
+  }
   _buildUiVerificationFailureNudge(output) {
     const details = this._extractUiVerificationFailureDetails(output);
     const genericFailure =