@hamp10/agentforge 0.2.37 → 0.2.38

package/package.json

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

package/scripts/check-task-semantics.js

@@ -35,6 +35,12 @@ const cases = [
     absent: ['example-com', 'make', 'fix'],
     pageOnly: true,
   },
+  {
+    text: 'Work on the Example.com listing pages for AlphaBoard.ai, BetaMatch.ai, and GammaForge.ai. Make all three pages visually polished and consistent with the rest of the site.',
+    slugs: ['alphaboard-ai', 'betamatch-ai', 'gammaforge-ai'],
+    absent: ['example-com', 'three', 'all-three', 'visual', 'polished'],
+    pageOnly: true,
+  },
   {
     text: 'Work on the Example.com listing pages for AlphaBoard and BetaMatch. Delete and rebuild those two listing page implementations from a clean start, preserving the same URLs and site conventions. Fix the readability and design issues. Only change those two listing pages.',
     slugs: ['alphaboard', 'betamatch'],
@@ -249,6 +255,33 @@ try {
     'clean-start rebuild tasks should still require the scoped target page to be recreated'
   );
   git(fixture.repo, ['restore', '--', 'public_html/domains/alpha.html']);
+  const nestedAlphaRel = `${path.basename(fixture.repo)}/public_html/domains/alpha.html`;
+  rmSync(path.join(fixture.repo, nestedAlphaRel), { force: true });
+  assert.equal(
+    worker._buildDeletedScopedPageNudge(
+      [{ root: fixture.repo, head: fixture.head }],
+      'Delete and rebuild the Alpha listing page from a clean start, preserving the same URL. Only change that listing page.'
+    ),
+    '',
+    'tracked same-name nested project copies should not count as missing scoped page sources'
+  );
+  assert.equal(
+    worker._scopeAllowsChangedPath(
+      { root: fixture.repo, head: fixture.head },
+      nestedAlphaRel,
+      ['alpha'],
+      true,
+      'Delete and rebuild the Alpha listing page from a clean start. Only change that listing page.'
+    ),
+    false,
+    'tracked same-name nested project copies should be treated as out-of-scope changes'
+  );
+  assert.equal(
+    worker._allCurrentPageSourceFiles(fixture.repo).includes(nestedAlphaRel),
+    false,
+    'tracked same-name nested project copies should not pollute current page source discovery'
+  );
+  git(fixture.repo, ['restore', '--', nestedAlphaRel]);
   const projectsRoot = mkdtempSync(path.join(tmpdir(), 'agentforge-project-list-'));
   let agentWorkspace = null;
   try {

package/src/taskSemantics.js

@@ -11,8 +11,9 @@ const GENERIC_SCOPE_WORDS = new Set([
   'website', 'app', 'readability', 'readable', 'legibility', 'contrast',
   'quality', 'visual', 'content', 'layout', 'styling', 'polish',
   'current', 'target', 'targets', 'requested', 'same', 'rest', 'live',
-  'local', 'itself', 'only', 'those', 'these', 'this', 'that', 'two',
-  'both', 'change', 'edit', 'modify', 'touch', 'make', 'improve', 'fix',
+  'local', 'itself', 'only', 'all', 'those', 'these', 'this', 'that',
+  'one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine',
+  'ten', 'both', 'change', 'edit', 'modify', 'touch', 'make', 'improve', 'fix',
   'update', 'redesign', 'owned', 'source', 'style', 'styles', 'global',
   'shared', 'reference', 'references', 'broaden', 'work', 'wrong', 'bad',
   'broken',

package/src/worker.js

@@ -650,6 +650,22 @@ export class AgentForgeWorker extends EventEmitter {
     return /\.(?:html?|xhtml|astro|mdx?)$/i.test(String(relativePath || ''));
   }
 
+  _nestedProjectCopyCanonicalRel(repo, rel) {
+    const normalized = String(rel || '').replace(/^[/\\]+/, '');
+    if (!repo || !normalized) return '';
+    const parts = normalized.split(/[\\/]+/).filter(Boolean);
+    if (parts.length < 2 || parts[0].toLowerCase() !== path.basename(repo).toLowerCase()) return '';
+    return parts.slice(1).join('/');
+  }
+
+  _isNestedProjectCopyRel(repo, rel, ref = 'HEAD') {
+    const canonicalRel = this._nestedProjectCopyCanonicalRel(repo, rel);
+    if (!canonicalRel) return false;
+    const canonicalAbs = path.join(repo, ...canonicalRel.split('/'));
+    if (existsSync(canonicalAbs) || existsSync(path.dirname(canonicalAbs))) return true;
+    return this._gitPathExistsAtRef(repo, ref || 'HEAD', canonicalRel);
+  }
+
   _isBroadReferenceSourcePath(relativePath) {
     const normalized = String(relativePath || '').replace(/\\/g, '/').toLowerCase();
     const parts = normalized.split('/').filter(Boolean);
@@ -727,6 +743,7 @@ export class AgentForgeWorker extends EventEmitter {
     const names = new Set();
     const add = (output) => {
       for (const rel of String(output || '').split('\n').map(line => line.trim()).filter(Boolean)) {
+        if (this._isNestedProjectCopyRel(repo, rel)) continue;
         if (this._isPageSourcePath(rel)) names.add(rel);
       }
     };
@@ -785,6 +802,7 @@ export class AgentForgeWorker extends EventEmitter {
 
   _scopeAllowsChangedPath(baseline, rel, allowedSlugs, pageOnly, userMessage = '') {
     const lower = String(rel || '').toLowerCase();
+    if (this._isNestedProjectCopyRel(baseline?.root, rel, baseline?.head || 'HEAD')) return false;
     const slugAllowed = this._scopeSlugsMatchingText(lower, allowedSlugs).length > 0;
     if (slugAllowed) {
       if (this._isNewHtmlPageOutsideExistingCollection(baseline, rel, allowedSlugs, pageOnly)) return false;
@@ -988,13 +1006,12 @@ export class AgentForgeWorker extends EventEmitter {
       const deleted = [];
       const status = this._gitStatusPorcelain(baseline.root, 10000);
       for (const rawLine of String(status || '').split('\n').filter(Boolean)) {
-        const x = rawLine[0] || ' ';
-        const y = rawLine[1] || ' ';
-        if (x !== 'D' && y !== 'D') continue;
-        const rel = rawLine.length >= 3 && rawLine[2] === ' ' ? rawLine.slice(3) : rawLine.trim().replace(/^..\s+/, '');
-        const pathName = rel.includes(' -> ') ? rel.split(' -> ').pop() : rel;
+        const statusCode = rawLine.slice(0, 2);
+        if (!statusCode.includes('D')) continue;
+        const pathName = this._parseGitStatusPaths(rawLine)[0] || '';
         const lower = String(pathName || '').toLowerCase();
         if (!this._isPageSourcePath(lower)) continue;
+        if (this._isNestedProjectCopyRel(baseline.root, pathName, baseline.head || 'HEAD')) continue;
         if (this._scopeSlugsMatchingText(lower, allowedSlugs).length === 0) continue;
         deleted.push(pathName);
       }