@ktpartners/dgs-platform 2.7.5 → 2.8.0

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

@@ -142,9 +142,9 @@ describe('context load-tier none', () => {
   beforeEach(() => {
     resetTierCache();
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
     });
   });
 
@@ -154,18 +154,18 @@ describe('context load-tier none', () => {
   });
 
   it('returns empty files array', () => {
-    const result = loadTierInternal('none', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('none', fixture.cwd, { cwd: fixture.cwd });
     assert.ok(Array.isArray(result.files), 'files should be array');
     assert.equal(result.files.length, 0, 'none tier should have 0 files');
   });
 
   it('returns tier name in output', () => {
-    const result = loadTierInternal('none', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('none', fixture.cwd, { cwd: fixture.cwd });
     assert.equal(result.tier, 'none', 'tier should be "none"');
   });
 
   it('returns empty scope', () => {
-    const result = loadTierInternal('none', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('none', fixture.cwd, { cwd: fixture.cwd });
     assert.deepEqual(result.scope, {}, 'scope should be empty');
   });
 });
@@ -186,12 +186,12 @@ describe('context load-tier lite', () => {
 
   it('returns files with correct paths for existing files', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
     });
 
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
     assert.equal(result.tier, 'lite');
     assert.ok(result.files.length > 0, 'should have some files');
 
@@ -204,12 +204,12 @@ describe('context load-tier lite', () => {
 
   it('returns category labels for each file', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
     });
 
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
 
     for (const file of result.files) {
       assert.ok(file.category, `file ${file.path} should have category`);
@@ -224,11 +224,11 @@ describe('context load-tier lite', () => {
   it('omits missing files silently', () => {
     // No STATE.md in this fixture
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
     });
 
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
     const filePaths = result.files.map(f => f.path);
     assert.ok(!filePaths.some(p => p.includes('STATE.md')), 'STATE.md should be omitted when missing');
     assert.ok(filePaths.some(p => p.includes('PROJECT.md')), 'PROJECT.md should still be present');
@@ -237,12 +237,12 @@ describe('context load-tier lite', () => {
 
   it('all files have exists: true', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
     });
 
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
     for (const file of result.files) {
       assert.equal(file.exists, true, `file ${file.path} should have exists: true`);
     }
@@ -265,15 +265,15 @@ describe('context load-tier planning', () => {
 
   it('includes all lite tier files plus planning files', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/REQUIREMENTS.md': '# Requirements',
-      '.planning/REPOS.md': '# Repos',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'REQUIREMENTS.md': '# Requirements',
+      'REPOS.md': '# Repos',
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('planning', fixture.cwd, { cwd: fixture.cwd });
     const filePaths = result.files.map(f => f.path);
 
     // Lite tier files
@@ -289,14 +289,14 @@ describe('context load-tier planning', () => {
 
   it('includes codebase docs via dynamic glob', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/codebase/CONVENTIONS.md': '# Conventions',
-      '.planning/codebase/TESTING.md': '# Testing',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'codebase/CONVENTIONS.md': '# Conventions',
+      'codebase/TESTING.md': '# Testing',
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('planning', fixture.cwd, { cwd: fixture.cwd });
     const filePaths = result.files.map(f => f.path);
 
     assert.ok(filePaths.some(p => p.includes('CONVENTIONS.md')), 'should include CONVENTIONS.md');
@@ -309,14 +309,14 @@ describe('context load-tier planning', () => {
 
   it('missing REPOS.md is silently omitted', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/REQUIREMENTS.md': '# Requirements',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'REQUIREMENTS.md': '# Requirements',
       // No REPOS.md
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('planning', fixture.cwd, { cwd: fixture.cwd });
     const filePaths = result.files.map(f => f.path);
     assert.ok(!filePaths.some(p => p.includes('REPOS.md')), 'REPOS.md should be omitted when missing');
     assert.ok(filePaths.some(p => p.includes('ROADMAP.md')), 'ROADMAP.md should be present');
@@ -339,17 +339,17 @@ describe('context load-tier with scope flags', () => {
 
   it('--phase adds phase-specific files to output', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/STATE.md': '# State',
-      '.planning/ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
-      '.planning/phases/01-test-phase/01-CONTEXT.md': '# Context',
-      '.planning/phases/01-test-phase/01-RESEARCH.md': '# Research',
-      '.planning/phases/01-test-phase/01-01-PLAN.md': '---\nphase: 01\nplan: 01\n---\n# Plan',
-      '.planning/phases/01-test-phase/01-01-SUMMARY.md': '---\nphase: 01\nplan: 01\n---\n# Summary',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'STATE.md': '# State',
+      'ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
+      'phases/01-test-phase/01-CONTEXT.md': '# Context',
+      'phases/01-test-phase/01-RESEARCH.md': '# Research',
+      'phases/01-test-phase/01-01-PLAN.md': '---\nphase: 01\nplan: 01\n---\n# Plan',
+      'phases/01-test-phase/01-01-SUMMARY.md': '---\nphase: 01\nplan: 01\n---\n# Summary',
     });
 
-    const result = loadTierInternal('execution', path.join(fixture.cwd, '.planning'), {
+    const result = loadTierInternal('execution', fixture.cwd, {
       cwd: fixture.cwd,
       phase: '1',
     });
@@ -363,14 +363,14 @@ describe('context load-tier with scope flags', () => {
 
   it('--idea adds idea file and docs to output', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/ideas/pending/005-better-caching.md': '# Better Caching',
-      '.planning/ideas/pending/005-better-caching/docs/research.md': '# Research on caching',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'ideas/pending/005-better-caching.md': '# Better Caching',
+      'ideas/pending/005-better-caching/docs/research.md': '# Research on caching',
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), {
+    const result = loadTierInternal('planning', fixture.cwd, {
       cwd: fixture.cwd,
       idea: '5',
     });
@@ -383,14 +383,14 @@ describe('context load-tier with scope flags', () => {
 
   it('--spec adds spec file and docs to output', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/specs/dgs-improved-context-spec.md': '---\nid: dgs-improved-context\nstatus: draft\n---\n# Improved Context',
-      '.planning/specs/dgs-improved-context-spec/docs/analysis.md': '# Analysis',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'specs/dgs-improved-context-spec.md': '---\nid: dgs-improved-context\nstatus: draft\n---\n# Improved Context',
+      'specs/dgs-improved-context-spec/docs/analysis.md': '# Analysis',
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), {
+    const result = loadTierInternal('planning', fixture.cwd, {
       cwd: fixture.cwd,
       spec: 'dgs-improved-context',
     });
@@ -403,15 +403,15 @@ describe('context load-tier with scope flags', () => {
 
   it('missing scope targets silently return no extra files', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/STATE.md': '# State',
-      '.planning/ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
-      '.planning/phases/01-test-phase/01-01-PLAN.md': '# Plan',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'STATE.md': '# State',
+      'ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
+      'phases/01-test-phase/01-01-PLAN.md': '# Plan',
     });
 
     // Phase 999 does not exist
-    const result = loadTierInternal('execution', path.join(fixture.cwd, '.planning'), {
+    const result = loadTierInternal('execution', fixture.cwd, {
       cwd: fixture.cwd,
       phase: '999',
     });
@@ -426,13 +426,13 @@ describe('context load-tier with scope flags', () => {
 
   it('approved spec is marked as truncated', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/specs/my-spec.md': '---\nid: my-spec\nstatus: approved\n---\n# My Spec\n\n## Section 1\nContent\n\n## Section 2\nContent\n\n## Section 3\nContent\n\n## Section 4\nShould be truncated',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'specs/my-spec.md': '---\nid: my-spec\nstatus: approved\n---\n# My Spec\n\n## Section 1\nContent\n\n## Section 2\nContent\n\n## Section 3\nContent\n\n## Section 4\nShould be truncated',
     });
 
-    const result = loadTierInternal('planning', path.join(fixture.cwd, '.planning'), {
+    const result = loadTierInternal('planning', fixture.cwd, {
       cwd: fixture.cwd,
       spec: 'my-spec',
     });
@@ -458,13 +458,13 @@ describe('graceful degradation', () => {
     resetTierCache();
   });
 
-  it('empty .planning directory returns empty files for lite tier', () => {
+  it('empty directory returns empty files for lite tier', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
       // No STATE.md, PROJECT.md etc.
     });
 
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
     assert.equal(result.tier, 'lite');
     // config.json exists but PROJECT.md and STATE.md don't
     assert.ok(result.files.length <= 1, 'should have at most config.json');
@@ -472,10 +472,10 @@ describe('graceful degradation', () => {
 
   it('non-existent tier name returns error in result', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
-    const result = loadTierInternal('invalid-name', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('invalid-name', fixture.cwd, { cwd: fixture.cwd });
     assert.ok(result.error, 'should have error field');
     assert.ok(result.error.includes('Unknown tier'), 'error should mention unknown tier');
     assert.equal(result.files.length, 0, 'should have empty files');
@@ -483,7 +483,7 @@ describe('graceful degradation', () => {
 
   it('invalid tier via CLI produces error on stderr', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     const stderr = runContextError(fixture.cwd, 'load-tier invalid-name');
@@ -491,14 +491,14 @@ describe('graceful degradation', () => {
     assert.ok(stderr.includes('invalid tier'), 'error should mention invalid tier');
   });
 
-  it('missing .planning/config.json falls back gracefully', () => {
+  it('missing config.json falls back gracefully', () => {
     fixture = createFixture({
-      '.planning/PROJECT.md': '# Project',
-      '.planning/STATE.md': '# State',
+      'PROJECT.md': '# Project',
+      'STATE.md': '# State',
     });
 
     // The config.json from tier definitions won't be found but that's fine
-    const result = loadTierInternal('lite', path.join(fixture.cwd, '.planning'), { cwd: fixture.cwd });
+    const result = loadTierInternal('lite', fixture.cwd, { cwd: fixture.cwd });
     assert.equal(result.tier, 'lite');
     // PROJECT.md and STATE.md should be found
     const filePaths = result.files.map(f => f.path);
@@ -523,9 +523,9 @@ describe('CLI integration', () => {
 
   it('context load-tier lite returns JSON with correct structure', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
     });
 
     const result = runContext(fixture.cwd, 'load-tier lite');
@@ -536,7 +536,7 @@ describe('CLI integration', () => {
 
   it('context load-tier none returns empty files via CLI', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     const result = runContext(fixture.cwd, 'load-tier none');
@@ -546,11 +546,11 @@ describe('CLI integration', () => {
 
   it('context load-tier planning includes inherited lite files via CLI', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/PROJECT.md': '# Project',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/REQUIREMENTS.md': '# Requirements',
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'PROJECT.md': '# Project',
+      'ROADMAP.md': '# Roadmap',
+      'REQUIREMENTS.md': '# Requirements',
     });
 
     const result = runContext(fixture.cwd, 'load-tier planning');
@@ -561,13 +561,13 @@ describe('CLI integration', () => {
 
   it('context load-tier with --phase flag works via CLI', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECT.md': '# Project',
-      '.planning/STATE.md': '# State',
-      '.planning/ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
-      '.planning/REQUIREMENTS.md': '# Requirements',
-      '.planning/phases/01-test-phase/01-CONTEXT.md': '# Context',
-      '.planning/phases/01-test-phase/01-01-PLAN.md': '---\nphase: 01\nplan: 01\n---\n# Plan',
+      'config.json': JSON.stringify({}),
+      'PROJECT.md': '# Project',
+      'STATE.md': '# State',
+      'ROADMAP.md': '# Roadmap\n\n## Phases\n\n### Phase 1: Test Phase\n',
+      'REQUIREMENTS.md': '# Requirements',
+      'phases/01-test-phase/01-CONTEXT.md': '# Context',
+      'phases/01-test-phase/01-01-PLAN.md': '---\nphase: 01\nplan: 01\n---\n# Plan',
     });
 
     const result = runContext(fixture.cwd, 'load-tier execution --phase 1');
@@ -578,7 +578,7 @@ describe('CLI integration', () => {
 
   it('context without subcommand produces error', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     const stderr = runContextError(fixture.cwd, '');
@@ -589,7 +589,7 @@ describe('CLI integration', () => {
 
   it('context load-tier without tier name produces usage error', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     const stderr = runContextError(fixture.cwd, 'load-tier');
@@ -600,7 +600,7 @@ describe('CLI integration', () => {
 
   it('context unknown-subcommand produces error', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     const stderr = runContextError(fixture.cwd, 'bogus');

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

@@ -5,7 +5,7 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
-const { getPlanningRoot, PROJECTS_DIR } = require('./paths.cjs');
+const { getPlanningRoot, isV2Install, PROJECTS_DIR } = require('./paths.cjs');
 
 // ─── Path helpers ────────────────────────────────────────────────────────────
 
@@ -102,17 +102,7 @@ function loadConfig(cwd) {
     }
   } catch { /* ignore */ }
 
-  // Fall back to legacy dgs.config.json if neither new file exists
-  let hasNewConfig = Object.keys(shared).length > 0 || Object.keys(local).length > 0;
-  if (!hasNewConfig) {
-    const legacyPath = path.join(root, 'dgs.config.json');
-    try {
-      if (fs.existsSync(legacyPath)) {
-        shared = JSON.parse(fs.readFileSync(legacyPath, 'utf-8'));
-        hasNewConfig = true;
-      }
-    } catch { /* ignore */ }
-  }
+  const hasNewConfig = Object.keys(shared).length > 0 || Object.keys(local).length > 0;
 
   if (!hasNewConfig) {
     return { ...defaults, current_project: null, v2_hint_shown: false, sync_hint_shown: false };
@@ -182,8 +172,7 @@ function isGitIgnored(cwd, targetPath) {
   try {
     // --no-index checks .gitignore rules regardless of whether the file is tracked.
     // Without it, git check-ignore returns "not ignored" for tracked files even when
-    // .gitignore explicitly lists them — a common source of confusion when .planning/
-    // was committed before being added to .gitignore.
+    // .gitignore explicitly lists them — ensure planning files are not accidentally ignored.
     execSync('git check-ignore -q --no-index -- ' + targetPath.replace(/[^a-zA-Z0-9._\-/]/g, ''), {
       cwd,
       stdio: 'pipe',
@@ -308,12 +297,12 @@ function findPhaseInternal(cwd, phase) {
 
   const normalized = normalizePhaseName(phase);
 
-  // Resolve phases directory: v2 uses .planning/<project>/phases/, v1 uses .planning/phases/
+  // Resolve phases directory: project root or '.' fallback
   let projectRoot;
   try {
     projectRoot = getProjectRoot(cwd);
   } catch {
-    projectRoot = '.planning';
+    projectRoot = '.';
   }
   const phasesRel = path.join(projectRoot, 'phases');
   const phasesDir = path.join(cwd, phasesRel);
@@ -393,12 +382,12 @@ function getArchivedPhaseDirs(cwd) {
 
 function getRoadmapPhaseInternal(cwd, phaseNum) {
   if (!phaseNum) return null;
-  // v2: look in project root (.planning/<project>/), v1: .planning/
+  // Look in project root (projects/<slug>/)
   let projectRoot;
   try {
     projectRoot = getProjectRoot(cwd);
   } catch {
-    projectRoot = path.relative(cwd, getPlanningRoot(cwd)) || '.planning';
+    projectRoot = path.relative(cwd, getPlanningRoot(cwd)) || '.';
   }
   const roadmapPath = path.join(cwd, projectRoot, 'ROADMAP.md');
   if (!fs.existsSync(roadmapPath)) return null;
@@ -554,8 +543,8 @@ function getMilestonePhaseFilter(cwd) {
 // ─── v2 Path Resolution ──────────────────────────────────────────────────────
 
 /**
- * System directories that live under .planning/ but are NOT project folders.
- * Kept as a safety net — getProjectFolders() now scans .planning/projects/ directly
+ * System directories that are NOT project folders.
+ * Kept as a safety net — getProjectFolders() scans projects/ directly
  * and no longer relies on this set for filtering. The 'projects' entry prevents
  * the projects container directory itself from being mistaken for a project.
  */
@@ -564,32 +553,6 @@ const SYSTEM_DIRS = new Set([
   'debug', 'milestones', 'research', PROJECTS_DIR,
 ]);
 
-/**
- * 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 content = fs.readFileSync(filePath, 'utf-8');
-      if (content.startsWith(header)) return true;
-    } catch {
-      // File doesn't exist or unreadable — not a marker
-    }
-  }
-  return false;
-}
-
 /**
  * Construct the absolute path to a project directory.
  *
@@ -606,9 +569,9 @@ function getProjectDir(cwd, slug) {
 
 /**
  * Get the project root directory (relative path).
- * - v2 with current_project set: returns .planning/projects/<project>
+ * - v2 with current_project set: returns projects/<project>
  * - v2 without current_project: throws Error('NO_CURRENT_PROJECT_V2')
- * - v1 (no v2 markers): returns '.planning'
+ * - No v2 markers: returns '.'
  *
  * @param {string} cwd - Working directory
  * @returns {string} Relative path to project root
@@ -618,10 +581,8 @@ function getProjectRoot(cwd) {
   const config = loadConfig(cwd);
   const currentProject = (config.current_project || '').trim();
 
-  // Determine layout: root layout when planning root equals cwd
-  const planRoot = getPlanningRoot(cwd);
-  const isRootLayout = (planRoot === path.resolve(cwd));
-  const planRootRel = isRootLayout ? '.' : '.planning';
+  // Root layout: planning root IS the repo root
+  const planRootRel = '.';
 
   if (currentProject) {
     // Validate project name — must be a simple slug, no path traversal
@@ -649,7 +610,7 @@ function getProjectRoot(cwd) {
     throw new Error('NO_CURRENT_PROJECT_V2');
   }
 
-  // v1 fallback — no v2 markers, return planning root relative path
+  // No v2 markers — return root
   return planRootRel;
 }
 
@@ -674,8 +635,7 @@ function requireProjectRoot(cwd) {
 
 /**
  * Resolve a project-scoped relative path.
- * e.g., resolveProjectPath(cwd, 'STATE.md') → '.planning/projects/auth-overhaul/STATE.md' (v2)
- *        resolveProjectPath(cwd, 'STATE.md') → '.planning/STATE.md' (v1)
+ * e.g., resolveProjectPath(cwd, 'STATE.md') → 'projects/auth-overhaul/STATE.md'
  *        resolveProjectPath(cwd) → project root
  *
  * @param {string} cwd - Working directory
@@ -692,7 +652,7 @@ function resolveProjectPath(cwd, relativePath) {
  *
  * This is intentionally lightweight — reads STATUS line from STATE.md using
  * safeReadFile (already in core.cjs) to avoid circular dependency on projects.cjs.
- * Reads from .planning/projects/<slug>/STATE.md.
+ * Reads from projects/<slug>/STATE.md.
  *
  * @param {string} cwd - Working directory
  * @param {string} slug - Project slug
@@ -711,7 +671,7 @@ function isProjectCompleted(cwd, slug) {
 
 /**
  * List project folders in a v2 install.
- * Scans .planning/projects/ directly and returns directories that:
+ * Scans projects/ directly and returns directories that:
  * - Don't start with '.'
  * - Contain a STATE.md file (qualifying marker for project folders)
  *