@ktpartners/dgs-platform 2.7.4 → 2.8.0

package/deliver-great-systems/bin/lib/milestone.cjs

@@ -144,7 +144,7 @@ function cmdMilestoneComplete(cwd, version, options, raw) {
   // Archive REQUIREMENTS.md
   if (fs.existsSync(reqPath)) {
     const reqContent = fs.readFileSync(reqPath, 'utf-8');
-    const archiveHeader = `# Requirements Archive: ${version} ${milestoneName}\n\n**Archived:** ${today}\n**Status:** SHIPPED\n\nFor current requirements, see \`.planning/REQUIREMENTS.md\`.\n\n---\n\n`;
+    const archiveHeader = `# Requirements Archive: ${version} ${milestoneName}\n\n**Archived:** ${today}\n**Status:** SHIPPED\n\nFor current requirements, see \`REQUIREMENTS.md\`.\n\n---\n\n`;
     fs.writeFileSync(path.join(archiveDir, `${version}-REQUIREMENTS.md`), archiveHeader + reqContent, 'utf-8');
   }
 

package/deliver-great-systems/bin/lib/overlap.cjs

@@ -21,7 +21,7 @@ const SEVERITY_ORDER = { HIGH: 0, MEDIUM: 1, LOW: 2 };
 /**
  * Scan a project's plan files and extract repos and files from XML tags.
  *
- * Scans all *-PLAN.md files under .planning/projects/<slug>/phases/ subdirectories.
+ * Scans all *-PLAN.md files under projects/<slug>/phases/ subdirectories.
  * For each plan file, extracts all <repos> and <files> tags.
  *
  * @param {string} cwd - Working directory (product root)
@@ -97,7 +97,7 @@ function scanProjectPlanFiles(cwd, slug) {
  * Filter out phase directories that are in archived milestone paths.
  *
  * Archived paths match patterns like:
- *   .planning/milestones/v1.0-phases/01-setup
+ *   milestones/v1.0-phases/01-setup
  *
  * @param {string[]} phaseDirs - Array of full phase directory paths
  * @returns {string[]} Filtered array with archived paths removed
@@ -181,7 +181,7 @@ function classifyOverlapSeverity(overlapEntry) {
  * Get slugs of all active (non-completed) projects.
  *
  * Reads PROJECTS.md to get active project slugs. Falls back to scanning
- * .planning/ subfolders if PROJECTS.md is missing. Applies ghost project
+ * subfolders if PROJECTS.md is missing. Applies ghost project
  * guard (skips folders without STATE.md).
  *
  * @param {string} cwd - Working directory (product root)
@@ -200,7 +200,7 @@ function getActiveProjectSlugs(cwd) {
       });
   }
 
-  // Fallback: scan .planning/ subfolders
+  // Fallback: scan project subfolders
   const folders = getProjectFolders(cwd);
   return folders.filter(slug => {
     const state = readProjectState(cwd, slug);

package/deliver-great-systems/bin/lib/overlap.test.cjs

@@ -8,16 +8,17 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 
-const { createTempDir, cleanupDir, writeFile } = require('./test-helpers.cjs');
+const { createTempDir, cleanupDir, writeFile , initGitRepo } = require('./test-helpers.cjs');
+const { resetPaths } = require('./paths.cjs');
 
-// Helper to create a minimal project with STATE.md under .planning/projects/<slug>/
+// Helper to create a minimal project with STATE.md under projects/<slug>/
 function createProject(cwd, slug, status = 'Ready to execute') {
-  writeFile(cwd, `.planning/projects/${slug}/STATE.md`, `# Project State\n\nPhase: 1\nStatus: ${status}\nProgress: [░░░░░░░░░░] 0%\n`);
-  writeFile(cwd, `.planning/projects/${slug}/PROJECT.md`, `# Project: ${slug}\n`);
-  fs.mkdirSync(path.join(cwd, '.planning', 'projects', slug, 'phases'), { recursive: true });
+  writeFile(cwd, `projects/${slug}/STATE.md`, `# Project State\n\nPhase: 1\nStatus: ${status}\nProgress: [░░░░░░░░░░] 0%\n`);
+  writeFile(cwd, `projects/${slug}/PROJECT.md`, `# Project: ${slug}\n`);
+  fs.mkdirSync(path.join(cwd, 'projects', slug, 'phases'), { recursive: true });
 }
 
-// Helper to create a plan file with repos and files tags under .planning/projects/<slug>/phases/
+// Helper to create a plan file with repos and files tags under projects/<slug>/phases/
 function createPlanFile(cwd, slug, phaseDir, planName, tasks) {
   let content = `---\nphase: ${phaseDir}\nplan: 01\n---\n\n<tasks>\n`;
   for (const task of tasks) {
@@ -28,7 +29,7 @@ function createPlanFile(cwd, slug, phaseDir, planName, tasks) {
     content += `  <action>Do something</action>\n  <verify>Check it</verify>\n  <done>Done</done>\n</task>\n`;
   }
   content += '\n</tasks>\n';
-  writeFile(cwd, `.planning/projects/${slug}/phases/${phaseDir}/${planName}`, content);
+  writeFile(cwd, `projects/${slug}/phases/${phaseDir}/${planName}`, content);
 }
 
 // Helper to create PROJECTS.md
@@ -45,7 +46,7 @@ function createProjectsMd(cwd, projects) {
   for (const p of projects.filter(p => p.status === 'completed')) {
     content += `| ${p.name} | ${p.completed || ''} | |\n`;
   }
-  writeFile(cwd, '.planning/PROJECTS.md', content);
+  writeFile(cwd, 'PROJECTS.md', content);
 }
 
 const {
@@ -61,13 +62,13 @@ const {
 
 describe('scanProjectPlanFiles', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('returns empty array when project has no phases directory', () => {
     createProject(tmpDir, 'project-a');
     // Remove phases dir
-    fs.rmSync(path.join(tmpDir, '.planning', 'projects', 'project-a', 'phases'), { recursive: true, force: true });
+    fs.rmSync(path.join(tmpDir, 'projects', 'project-a', 'phases'), { recursive: true, force: true });
     const result = scanProjectPlanFiles(tmpDir, 'project-a');
     assert.ok(Array.isArray(result));
     assert.strictEqual(result.length, 0);
@@ -105,9 +106,9 @@ describe('scanProjectPlanFiles', () => {
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
     // Create a SUMMARY.md file (should be skipped)
-    writeFile(tmpDir, '.planning/projects/project-a/phases/01-setup/01-01-SUMMARY.md', '<repos>server</repos>\n<files>server/x.js</files>');
+    writeFile(tmpDir, 'projects/project-a/phases/01-setup/01-01-SUMMARY.md', '<repos>server</repos>\n<files>server/x.js</files>');
     // Create a CONTEXT.md file (should be skipped)
-    writeFile(tmpDir, '.planning/projects/project-a/phases/01-setup/01-CONTEXT.md', '<repos>other</repos>');
+    writeFile(tmpDir, 'projects/project-a/phases/01-setup/01-CONTEXT.md', '<repos>other</repos>');
     const result = scanProjectPlanFiles(tmpDir, 'project-a');
     assert.strictEqual(result.length, 1);
     // Only web-app from the PLAN.md, not server from SUMMARY.md
@@ -128,7 +129,7 @@ describe('scanProjectPlanFiles', () => {
 
   it('handles plan files without repos or files tags', () => {
     createProject(tmpDir, 'project-a');
-    writeFile(tmpDir, '.planning/projects/project-a/phases/01-setup/01-01-PLAN.md',
+    writeFile(tmpDir, 'projects/project-a/phases/01-setup/01-01-PLAN.md',
       '---\nphase: 01\n---\n\n<tasks>\n<task type="auto">\n  <name>Task 1</name>\n  <action>Do</action>\n</task>\n</tasks>\n'
     );
     const result = scanProjectPlanFiles(tmpDir, 'project-a');
@@ -142,13 +143,13 @@ describe('scanProjectPlanFiles', () => {
 
 describe('buildOverlapMatrix', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('returns empty overlap when only one project exists', () => {
     createProject(tmpDir, 'project-a');
     createProjectsMd(tmpDir, [{ name: 'project-a', status: 'Active' }]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -164,7 +165,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -185,7 +186,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/shared.js', 'web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -210,7 +211,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'completed', completed: '2026-01-01' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -227,7 +228,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'ghost-project', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -244,7 +245,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n| server | ./server |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n| server | ./server |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -264,7 +265,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-b', status: 'Active' },
       { name: 'project-c', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| shared | ./shared |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| shared | ./shared |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['shared/lib.js'], repos: ['shared'] },
     ]);
@@ -286,7 +287,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| services-api | ./services/api |\n| services-api-gateway | ./services/api-gateway |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| services-api | ./services/api |\n| services-api-gateway | ./services/api-gateway |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['services/api/shared.ts', 'services/api-gateway/handler.ts'], repos: ['services-api', 'services-api-gateway'] },
     ]);
@@ -314,7 +315,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     // Repo-relative paths: src/index.ts instead of web-app/src/index.ts
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['src/index.ts', 'src/app.ts'], repos: ['web-app'] },
@@ -341,7 +342,7 @@ describe('buildOverlapMatrix', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     // Legacy paths without repos tags — files include repo prefix
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/shared.js'] },
@@ -361,13 +362,13 @@ describe('buildOverlapMatrix', () => {
 
 describe('excludeArchivedPhases', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('keeps current phase directories', () => {
     const dirs = [
-      path.join(tmpDir, '.planning/projects/project-a/phases/01-setup'),
-      path.join(tmpDir, '.planning/projects/project-a/phases/02-build'),
+      path.join(tmpDir, 'projects/project-a/phases/01-setup'),
+      path.join(tmpDir, 'projects/project-a/phases/02-build'),
     ];
     const result = excludeArchivedPhases(dirs);
     assert.strictEqual(result.length, 2);
@@ -375,8 +376,8 @@ describe('excludeArchivedPhases', () => {
 
   it('excludes directories in milestones archive path', () => {
     const dirs = [
-      path.join(tmpDir, '.planning/projects/project-a/phases/01-setup'),
-      path.join(tmpDir, '.planning/milestones/v1.0-phases/01-setup'),
+      path.join(tmpDir, 'projects/project-a/phases/01-setup'),
+      path.join(tmpDir, 'milestones/v1.0-phases/01-setup'),
     ];
     const result = excludeArchivedPhases(dirs);
     assert.strictEqual(result.length, 1);
@@ -517,8 +518,8 @@ describe('classifyOverlapSeverity', () => {
 
 describe('buildOverlapMatrix severity', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('two projects sharing same file -> severity HIGH', () => {
     createProject(tmpDir, 'project-a');
@@ -527,7 +528,7 @@ describe('buildOverlapMatrix severity', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/shared.js'], repos: ['web-app'] },
     ]);
@@ -547,7 +548,7 @@ describe('buildOverlapMatrix severity', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/auth/login.js'], repos: ['web-app'] },
     ]);
@@ -567,7 +568,7 @@ describe('buildOverlapMatrix severity', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/auth/login.js'], repos: ['web-app'] },
     ]);
@@ -587,7 +588,7 @@ describe('buildOverlapMatrix severity', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| web-app | ./web-app |\n');
     createPlanFile(tmpDir, 'project-a', '01-setup', '01-01-PLAN.md', [
       { name: 'Task 1', files: ['web-app/src/a.js'], repos: ['web-app'] },
     ]);
@@ -607,7 +608,7 @@ describe('buildOverlapMatrix severity', () => {
       { name: 'project-a', status: 'Active' },
       { name: 'project-b', status: 'Active' },
     ]);
-    writeFile(tmpDir, '.planning/REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| server | ./server |\n| web-app | ./web-app |\n');
+    writeFile(tmpDir, 'REPOS.md', '# Repos\n\n| Name | Path |\n|------|------|\n| server | ./server |\n| web-app | ./web-app |\n');
     // Use separate plan files per repo so each plan has a single <repos> tag
     // This ensures files are mapped precisely to one repo each
     // project-a: server plan (shared.js) + web-app plan (auth)
@@ -694,8 +695,8 @@ describe('formatOverlapReport severity', () => {
 
 describe('getActiveProjectSlugs', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('returns active slugs from PROJECTS.md', () => {
     createProject(tmpDir, 'project-a');
@@ -724,7 +725,7 @@ describe('getActiveProjectSlugs', () => {
   it('falls back to scanning folders when PROJECTS.md missing', () => {
     createProject(tmpDir, 'project-a');
     createProject(tmpDir, 'project-b');
-    // No PROJECTS.md — should scan .planning/ subfolders
+    // No PROJECTS.md — should scan projects/ subfolders
     const result = getActiveProjectSlugs(tmpDir);
     assert.ok(result.includes('project-a'));
     assert.ok(result.includes('project-b'));
@@ -733,14 +734,14 @@ describe('getActiveProjectSlugs', () => {
   it('ghost project guard: skips folders without STATE.md', () => {
     createProject(tmpDir, 'project-a');
     // Create a ghost folder without STATE.md
-    fs.mkdirSync(path.join(tmpDir, '.planning', 'projects', 'ghost-project'), { recursive: true });
+    fs.mkdirSync(path.join(tmpDir, 'projects', 'ghost-project'), { recursive: true });
     const result = getActiveProjectSlugs(tmpDir);
     assert.ok(result.includes('project-a'));
     assert.ok(!result.includes('ghost-project'));
   });
 
   it('returns empty array when no projects exist', () => {
-    fs.mkdirSync(path.join(tmpDir, '.planning'), { recursive: true });
+    // No projects directory exists
     const result = getActiveProjectSlugs(tmpDir);
     assert.strictEqual(result.length, 0);
   });

package/deliver-great-systems/bin/lib/path-audit.test.cjs

@@ -7,9 +7,8 @@
  * 2. GATES on library source files (path.join(cwd, '.planning') pattern)
  * 3. Documents baselines for lib module raw string literals
  *
- * Allowlisted files (may legitimately reference .planning/):
- * - workflows/init-product.md — creates and describes both layouts
- * - workflows/help.md — user documentation explaining directory structure
+ * All file categories are gated -- no allowlisted files remain.
+ * init-product.md and help.md were cleaned in Phase 122.
  *
  * Allowlisted line patterns:
  * - HTML comments: <!-- .planning/ ... -->
@@ -45,30 +44,25 @@ const PROJECT_SCOPED_PATTERN = /\.planning\/(?:STATE|ROADMAP|PROJECT(?!S\.md)|RE
 
 /**
  * Workflow files allowlisted from comprehensive .planning/ scanning.
- * These files legitimately reference .planning/ for layout documentation.
+ * Empty -- all workflow files were cleaned in Phase 122.
  */
-const ALLOWLISTED_WORKFLOW_FILES = [
-  'workflows/init-product.md',  // Creates and describes both layouts
-  'workflows/help.md',          // User documentation explaining directory structure
-];
+const ALLOWLISTED_WORKFLOW_FILES = [];
 
 /**
  * Reference files allowlisted from comprehensive .planning/ scanning.
- * These files have legitimate .planning/ references in example content.
+ * Empty -- all reference files were cleaned in Phase 122.
  */
-const ALLOWLISTED_REFERENCE_FILES = [
-  'references/planning-config.md',  // Has .planning/ in .gitignore example block
-];
+const ALLOWLISTED_REFERENCE_FILES = [];
 
 /**
  * Product-level paths that are intentionally at .planning/ root and should
  * NOT be flagged by the audit.
  */
 const SAFE_PRODUCT_PATHS = [
-  '.planning/config.json',
-  '.planning/PROJECTS.md',
-  '.planning/REPOS.md',
-  '.planning/codebase',
+  'config.json',
+  'PROJECTS.md',
+  'REPOS.md',
+  'codebase',
 ];
 
 /**
@@ -288,9 +282,9 @@ describe('path audit: comprehensive .planning/ reference scanning', () => {
 
     // Allowlist must stay minimal
     assert.ok(
-      ALLOWLISTED_WORKFLOW_FILES.length <= 2,
-      `Workflow allowlist too large (${ALLOWLISTED_WORKFLOW_FILES.length}). ` +
-      'Only init-product.md and help.md should be allowlisted. Migrate others.'
+      ALLOWLISTED_WORKFLOW_FILES.length === 0,
+      `Workflow allowlist should be empty (found ${ALLOWLISTED_WORKFLOW_FILES.length}). ` +
+      'All workflow files were cleaned in Phase 122.'
     );
   });
 
@@ -302,9 +296,9 @@ describe('path audit: comprehensive .planning/ reference scanning', () => {
 
     // Allowlist must stay minimal
     assert.ok(
-      ALLOWLISTED_REFERENCE_FILES.length <= 1,
-      `Reference allowlist too large (${ALLOWLISTED_REFERENCE_FILES.length}). ` +
-      'Only planning-config.md should be allowlisted.'
+      ALLOWLISTED_REFERENCE_FILES.length === 0,
+      `Reference allowlist should be empty (found ${ALLOWLISTED_REFERENCE_FILES.length}). ` +
+      'All reference files were cleaned in Phase 122.'
     );
   });
 

package/deliver-great-systems/bin/lib/paths.cjs

@@ -1,28 +1,36 @@
 /**
  * paths.cjs -- Planning root detection and path constants
  *
- * LEAF MODULE: Zero DGS imports. Uses only Node.js built-in fs and path.
- * This module sits at the bottom of the dependency graph to prevent
- * circular dependencies with core.cjs and config.cjs.
+ * LEAF MODULE: Zero DGS imports. Uses only Node.js built-in fs, path,
+ * and child_process. This module sits at the bottom of the dependency
+ * graph to prevent circular dependencies with core.cjs and config.cjs.
  *
  * Exports:
  *   getPlanningRoot(cwd) -- Returns absolute path to planning root
  *   getPaths(cwd)        -- Returns frozen PATHS object with all standard paths
  *   initPaths(cwd)       -- Alias for getPaths (explicit cache population)
- *   resetPaths()         -- Clears the cwd-keyed cache (for test isolation)
+ *   resetPaths()         -- Clears caches (for test isolation)
+ *   isV2Install(cwd)     -- Detects V2 multi-project installation
  *
- * Detection cascade (getPlanningRoot):
- *   1. .planning/config.json or .planning/config.local.json exists -> .planning/ layout
- *   2. config.local.json at root with planningRoot: '.' -> root layout
- *      (legacy fallback: dgs.config.json at root with planningRoot: '.')
- *   3. dgs.config.json at root without planningRoot: '.' -> .planning/ layout
- *   4. PROJECT.md at root AND no .planning/ directory -> root layout (auto-detect)
- *   5. No signals -> .planning/ layout (default)
+ * Root detection (getPlanningRoot):
+ *   Single `git rev-parse --show-toplevel` call. Errors if not in a git repo.
+ *   Result cached per-process.
  */
 
 const fs = require('fs');
 const path = require('path');
+const { execSync } = require('child_process');
 
+/**
+ * Per-process cache for getPlanningRoot result.
+ * One git repo = one root per process. Cleared by resetPaths().
+ */
+let _planningRootCache = null;
+
+/**
+ * Per-cwd cache for getPaths results.
+ * Different cwd values produce different absolute paths.
+ */
 const _cache = new Map();
 
 /**
@@ -34,57 +42,50 @@ const PROJECTS_DIR = 'projects';
 
 /**
  * Detect the planning root directory for the given cwd.
+ * Uses `git rev-parse --show-toplevel` to find the git repo root.
+ * Result is cached per-process for performance.
  *
  * @param {string} [cwd] - Working directory (defaults to process.cwd())
- * @returns {string} Absolute path to the planning root
+ * @returns {string} Absolute path to the planning root (git repo root)
  */
 function getPlanningRoot(cwd) {
-  const resolved = path.resolve(cwd || process.cwd());
-  const dotPlanning = path.join(resolved, '.planning');
-
-  // Step 1: .planning/ layout -- config files inside .planning/
-  if (
-    fs.existsSync(path.join(dotPlanning, 'config.json')) ||
-    fs.existsSync(path.join(dotPlanning, 'config.local.json')) ||
-    fs.existsSync(path.join(dotPlanning, 'dgs.config.json'))
-  ) {
-    return dotPlanning;
-  }
-
-  // Step 2: Root layout -- config.local.json at repo root with planningRoot: '.'
-  const rootLocalConfigPath = path.join(resolved, 'config.local.json');
-  if (fs.existsSync(rootLocalConfigPath)) {
-    try {
-      const config = JSON.parse(fs.readFileSync(rootLocalConfigPath, 'utf-8'));
-      if (config.planningRoot === '.') return resolved;
-    } catch {
-      // Malformed JSON -- fall through
-    }
+  if (_planningRootCache !== null) return _planningRootCache;
+  try {
+    const root = execSync('git rev-parse --show-toplevel', {
+      cwd: cwd || process.cwd(),
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    }).trim();
+    _planningRootCache = root;
+    return root;
+  } catch {
+    process.stderr.write('Error: DGS requires a git repository. Run this command from inside a git repo.\n');
+    process.exit(1);
   }
+}
 
-  // Step 2b: Legacy fallback -- dgs.config.json at repo root
-  const rootConfigPath = path.join(resolved, 'dgs.config.json');
-  if (fs.existsSync(rootConfigPath)) {
+/**
+ * Detect if this is a v2 (multi-project) installation.
+ * Requires both file existence AND valid DGS-generated content.
+ * PROJECTS.md must start with "# Projects", REPOS.md with "# Repos".
+ *
+ * @param {string} cwd - Working directory
+ * @returns {boolean}
+ */
+function isV2Install(cwd) {
+  const checks = [
+    { file: 'PROJECTS.md', header: '# Projects' },
+    { file: 'REPOS.md', header: '# Repos' },
+  ];
+  const planRoot = getPlanningRoot(cwd);
+  for (const { file, header } of checks) {
+    const filePath = path.join(planRoot, file);
     try {
-      const config = JSON.parse(fs.readFileSync(rootConfigPath, 'utf-8'));
-      if (config.planningRoot === '.') return resolved;
-    } catch {
-      // Malformed JSON -- fall through to next check
-    }
-    // dgs.config.json exists but no planningRoot: '.' or malformed
-    return dotPlanning;
-  }
-
-  // Step 3: Auto-detect root layout -- PROJECT.md at root, no .planning/ dir
-  if (
-    fs.existsSync(path.join(resolved, 'PROJECT.md')) &&
-    !fs.existsSync(dotPlanning)
-  ) {
-    return resolved;
+      const content = fs.readFileSync(filePath, 'utf-8');
+      if (content.startsWith(header)) return true;
+    } catch { /* File doesn't exist or unreadable */ }
   }
-
-  // Step 4: Default -- .planning/
-  return dotPlanning;
+  return false;
 }
 
 /**
@@ -99,7 +100,6 @@ function getPaths(cwd) {
   if (_cache.has(resolved)) return _cache.get(resolved);
 
   const root = getPlanningRoot(resolved);
-  const isRoot = (root === resolved);
 
   const paths = Object.freeze({
     ROOT: root,
@@ -112,14 +112,12 @@ function getPaths(cwd) {
     MILESTONES: path.join(root, 'milestones'),
     CONFIG: path.join(root, 'config.json'),
     CONFIG_LOCAL: path.join(root, 'config.local.json'),
-    CONFIG_LEGACY: path.join(root, 'dgs.config.json'),
     QUICK: path.join(root, 'quick'),
     TODOS: path.join(root, 'todos'),
     RESEARCH: path.join(root, 'research'),
     DEBUG: path.join(root, 'debug'),
     ARCHIVE: path.join(root, 'archive'),
     PROJECTS: path.join(root, PROJECTS_DIR),
-    LAYOUT: isRoot ? 'root' : 'dotplanning',
   });
 
   _cache.set(resolved, paths);
@@ -138,10 +136,12 @@ function initPaths(cwd) {
 }
 
 /**
- * Clear all cached PATHS entries.
- * Call in afterEach for test isolation between test cases.
+ * Clear all cached values.
+ * Clears both the per-process getPlanningRoot cache and the per-cwd
+ * getPaths cache. Call in afterEach for test isolation between test cases.
  */
 function resetPaths() {
+  _planningRootCache = null;
   _cache.clear();
 }
 
@@ -150,5 +150,6 @@ module.exports = {
   getPaths,
   initPaths,
   resetPaths,
+  isV2Install,
   PROJECTS_DIR,
 };