@ktpartners/dgs-platform 2.7.4 → 2.8.0

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

@@ -1,5 +1,5 @@
 /**
- * Tests for core.cjs v2 path resolution functions
+ * Tests for core.cjs path resolution functions
  *
  * Uses Node.js built-in test runner (node:test) and assert (node:assert).
  * Each test creates an isolated temp directory fixture and cleans up after.
@@ -26,17 +26,17 @@ const {
   getProjectDir,
 } = require('./core.cjs');
 
-// ─── v1 Mode Tests ────────────────────────────────────────────────────────────
+// ─── Root layout (no v2 markers) Tests ───────────────────────────────────────
 
-describe('v1 mode (no v2 markers)', () => {
+describe('root layout (no v2 markers)', () => {
   let fixture;
 
   beforeEach(() => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/ROADMAP.md': '# Roadmap',
-      '.planning/phases/': null,
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'ROADMAP.md': '# Roadmap',
+      'phases/': null,
     });
   });
 
@@ -44,24 +44,24 @@ describe('v1 mode (no v2 markers)', () => {
     fixture.cleanup();
   });
 
-  it('getProjectRoot returns .planning', () => {
+  it('getProjectRoot returns .', () => {
     const result = getProjectRoot(fixture.cwd);
-    assert.equal(result, '.planning');
+    assert.equal(result, '.');
   });
 
-  it('resolveProjectPath with relative path returns .planning/<path>', () => {
+  it('resolveProjectPath with relative path returns relative path', () => {
     const result = resolveProjectPath(fixture.cwd, 'STATE.md');
-    assert.equal(result, path.join('.planning', 'STATE.md'));
+    assert.equal(result, 'STATE.md');
   });
 
   it('resolveProjectPath with nested path returns correct path', () => {
     const result = resolveProjectPath(fixture.cwd, 'phases/01-foo');
-    assert.equal(result, path.join('.planning', 'phases', '01-foo'));
+    assert.equal(result, path.join('phases', '01-foo'));
   });
 
-  it('resolveProjectPath without second arg returns .planning', () => {
+  it('resolveProjectPath without second arg returns .', () => {
     const result = resolveProjectPath(fixture.cwd);
-    assert.equal(result, '.planning');
+    assert.equal(result, '.');
   });
 
   it('isV2Install returns false', () => {
@@ -77,12 +77,13 @@ describe('v2 mode with current_project', () => {
 
   beforeEach(() => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'auth-overhaul' }),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
-      '.planning/REPOS.md': '# Repos\n\n| Name | Path |\n',
-      '.planning/projects/auth-overhaul/STATE.md': '# State',
-      '.planning/projects/auth-overhaul/ROADMAP.md': '# Roadmap',
-      '.planning/projects/auth-overhaul/phases/': null,
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'auth-overhaul' }),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'REPOS.md': '# Repos\n\n| Name | Path |\n',
+      'projects/auth-overhaul/STATE.md': '# State',
+      'projects/auth-overhaul/ROADMAP.md': '# Roadmap',
+      'projects/auth-overhaul/phases/': null,
     });
   });
 
@@ -90,19 +91,19 @@ describe('v2 mode with current_project', () => {
     fixture.cleanup();
   });
 
-  it('getProjectRoot returns .planning/projects/<project>', () => {
+  it('getProjectRoot returns projects/<project>', () => {
     const result = getProjectRoot(fixture.cwd);
-    assert.equal(result, path.join('.planning', 'projects', 'auth-overhaul'));
+    assert.equal(result, path.join('projects', 'auth-overhaul'));
   });
 
   it('resolveProjectPath with STATE.md returns project-qualified path', () => {
     const result = resolveProjectPath(fixture.cwd, 'STATE.md');
-    assert.equal(result, path.join('.planning', 'projects', 'auth-overhaul', 'STATE.md'));
+    assert.equal(result, path.join('projects', 'auth-overhaul', 'STATE.md'));
   });
 
   it('resolveProjectPath with nested path returns project-qualified path', () => {
     const result = resolveProjectPath(fixture.cwd, 'phases/01-foo');
-    assert.equal(result, path.join('.planning', 'projects', 'auth-overhaul', 'phases', '01-foo'));
+    assert.equal(result, path.join('projects', 'auth-overhaul', 'phases', '01-foo'));
   });
 });
 
@@ -111,8 +112,8 @@ describe('v2 mode with current_project', () => {
 describe('v2 mode without current_project (guard trigger)', () => {
   it('throws NO_CURRENT_PROJECT_V2 when PROJECTS.md exists with valid header', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
     });
 
     try {
@@ -127,8 +128,8 @@ describe('v2 mode without current_project (guard trigger)', () => {
 
   it('throws NO_CURRENT_PROJECT_V2 when REPOS.md exists with valid header', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/REPOS.md': '# Repos\n\n| Name | Path |\n',
+      'config.json': JSON.stringify({}),
+      'REPOS.md': '# Repos\n\n| Name | Path |\n',
     });
 
     try {
@@ -147,8 +148,8 @@ describe('v2 mode without current_project (guard trigger)', () => {
 describe('strict v2 marker validation', () => {
   it('isV2Install returns false when PROJECTS.md has non-DGS content', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': 'Shopping List\n- apples\n- bananas',
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': 'Shopping List\n- apples\n- bananas',
     });
 
     try {
@@ -160,8 +161,8 @@ describe('strict v2 marker validation', () => {
 
   it('isV2Install returns true when PROJECTS.md starts with # Projects', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n\n## Active\n',
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n\n## Active\n',
     });
 
     try {
@@ -173,8 +174,8 @@ describe('strict v2 marker validation', () => {
 
   it('isV2Install returns true when REPOS.md starts with # Repos', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/REPOS.md': '# Repos\n\n| Name | Path |\n',
+      'config.json': JSON.stringify({}),
+      'REPOS.md': '# Repos\n\n| Name | Path |\n',
     });
 
     try {
@@ -186,7 +187,7 @@ describe('strict v2 marker validation', () => {
 
   it('isV2Install returns false when neither file exists', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     try {
@@ -202,8 +203,9 @@ describe('strict v2 marker validation', () => {
 describe('project directory validation', () => {
   it('throws PROJECT_NOT_FOUND when current_project dir does not exist', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'ghost-project' }),
-      '.planning/PROJECTS.md': '# Projects\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'ghost-project' }),
+      'PROJECTS.md': '# Projects\n',
     });
 
     try {
@@ -222,9 +224,10 @@ describe('project directory validation', () => {
 describe('completed project guards', () => {
   it('getProjectRoot throws PROJECT_COMPLETED when project STATUS is completed', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'done-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\nCompleted: 2026-02-20\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'done-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\nCompleted: 2026-02-20\n',
     });
 
     try {
@@ -239,9 +242,10 @@ describe('completed project guards', () => {
 
   it('getProjectRoot throws PROJECT_COMPLETED for case-insensitive status (Completed)', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'done-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/done-proj/STATE.md': '# Project State\n\nStatus: Completed\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'done-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/done-proj/STATE.md': '# Project State\n\nStatus: Completed\n',
     });
 
     try {
@@ -256,14 +260,15 @@ describe('completed project guards', () => {
 
   it('getProjectRoot does NOT throw for active project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'active-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/active-proj/STATE.md': '# Project State\n\nStatus: Active\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'active-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/active-proj/STATE.md': '# Project State\n\nStatus: Active\n',
     });
 
     try {
       const result = getProjectRoot(fixture.cwd);
-      assert.equal(result, path.join('.planning', 'projects', 'active-proj'));
+      assert.equal(result, path.join('projects', 'active-proj'));
     } finally {
       fixture.cleanup();
     }
@@ -271,14 +276,15 @@ describe('completed project guards', () => {
 
   it('getProjectRoot does NOT throw when STATE.md is missing', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'no-state-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/no-state-proj/': null,
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'no-state-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/no-state-proj/': null,
     });
 
     try {
       const result = getProjectRoot(fixture.cwd);
-      assert.equal(result, path.join('.planning', 'projects', 'no-state-proj'));
+      assert.equal(result, path.join('projects', 'no-state-proj'));
     } finally {
       fixture.cleanup();
     }
@@ -286,9 +292,10 @@ describe('completed project guards', () => {
 
   it('requireProjectRoot propagates PROJECT_COMPLETED', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'done-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'done-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
     });
 
     try {
@@ -303,9 +310,10 @@ describe('completed project guards', () => {
 
   it('resolveProjectPath throws PROJECT_COMPLETED for completed project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'done-proj' }),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'done-proj' }),
+      'PROJECTS.md': '# Projects\n',
+      'projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
     });
 
     try {
@@ -320,8 +328,8 @@ describe('completed project guards', () => {
 
   it('isProjectCompleted returns true for completed project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
+      'config.json': JSON.stringify({}),
+      'projects/done-proj/STATE.md': '# Project State\n\nStatus: completed\n',
     });
 
     try {
@@ -333,8 +341,8 @@ describe('completed project guards', () => {
 
   it('isProjectCompleted returns false for active project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/projects/active-proj/STATE.md': '# Project State\n\nStatus: Active\n',
+      'config.json': JSON.stringify({}),
+      'projects/active-proj/STATE.md': '# Project State\n\nStatus: Active\n',
     });
 
     try {
@@ -346,8 +354,8 @@ describe('completed project guards', () => {
 
   it('isProjectCompleted returns false when STATE.md missing', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/projects/no-state-proj/': null,
+      'config.json': JSON.stringify({}),
+      'projects/no-state-proj/': null,
     });
 
     try {
@@ -361,11 +369,11 @@ describe('completed project guards', () => {
 // ─── getProjectFolders Tests ──────────────────────────────────────────────────
 
 describe('getProjectFolders', () => {
-  it('returns empty array for v1 install', () => {
+  it('returns empty array when no v2 markers exist', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/STATE.md': '# State',
-      '.planning/phases/': null,
+      'config.json': JSON.stringify({}),
+      'STATE.md': '# State',
+      'phases/': null,
     });
 
     try {
@@ -378,14 +386,14 @@ describe('getProjectFolders', () => {
 
   it('returns project folder names from projects/ directory', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/auth-overhaul/STATE.md': '# State',
-      '.planning/projects/dashboard-v2/STATE.md': '# State',
-      '.planning/phases/': null,
-      '.planning/codebase/': null,
-      '.planning/archive/': null,
-      '.planning/todos/': null,
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n',
+      'projects/auth-overhaul/STATE.md': '# State',
+      'projects/dashboard-v2/STATE.md': '# State',
+      'phases/': null,
+      'codebase/': null,
+      'archive/': null,
+      'todos/': null,
     });
 
     try {
@@ -398,10 +406,10 @@ describe('getProjectFolders', () => {
 
   it('only includes directories containing STATE.md', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/projects/valid-project/STATE.md': '# State',
-      '.planning/projects/empty-folder/': null,
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n',
+      'projects/valid-project/STATE.md': '# State',
+      'projects/empty-folder/': null,
     });
 
     try {
@@ -414,9 +422,9 @@ describe('getProjectFolders', () => {
 
   it('excludes dot-directories', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/projects/.hidden/STATE.md': '# State',
-      '.planning/projects/real-project/STATE.md': '# State',
+      'config.json': JSON.stringify({}),
+      'projects/.hidden/STATE.md': '# State',
+      'projects/real-project/STATE.md': '# State',
     });
 
     try {
@@ -427,21 +435,11 @@ describe('getProjectFolders', () => {
     }
   });
 
-  it('returns empty array when .planning does not exist', () => {
-    const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'dgs-test-'));
-    try {
-      const result = getProjectFolders(cwd);
-      assert.deepEqual(result, []);
-    } finally {
-      fs.rmSync(cwd, { recursive: true, force: true });
-    }
-  });
-
   it('returns empty array when projects/ directory does not exist', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n',
-      '.planning/phases/': null,
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n',
+      'phases/': null,
     });
 
     try {
@@ -476,37 +474,39 @@ describe('getV2Hint', () => {
 describe('edge cases', () => {
   it('resolveProjectPath with empty string returns project root', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     try {
       const result = resolveProjectPath(fixture.cwd, '');
-      assert.equal(result, '.planning');
+      assert.equal(result, '.');
     } finally {
       fixture.cleanup();
     }
   });
 
-  it('getProjectRoot handles missing .planning gracefully', () => {
-    const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'dgs-test-'));
+  it('getProjectRoot returns . when no markers exist', () => {
+    const fixture = createFixture({
+      'config.json': JSON.stringify({}),
+    });
+
     try {
-      // Should not crash — returns .planning (v1 fallback since no markers)
-      const result = getProjectRoot(cwd);
-      assert.equal(result, '.planning');
+      const result = getProjectRoot(fixture.cwd);
+      assert.equal(result, '.');
     } finally {
-      fs.rmSync(cwd, { recursive: true, force: true });
+      fixture.cleanup();
     }
   });
 
   it('treats empty string current_project as unset', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: '' }),
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: '' }),
     });
 
     try {
-      // Should return .planning (v1 fallback), not .planning/
       const result = getProjectRoot(fixture.cwd);
-      assert.equal(result, '.planning');
+      assert.equal(result, '.');
     } finally {
       fixture.cleanup();
     }
@@ -514,8 +514,9 @@ describe('edge cases', () => {
 
   it('rejects current_project with path traversal (..)', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: '../etc' }),
-      '.planning/PROJECTS.md': '# Projects\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: '../etc' }),
+      'PROJECTS.md': '# Projects\n',
     });
 
     try {
@@ -530,8 +531,9 @@ describe('edge cases', () => {
 
   it('rejects current_project with forward slash', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'foo/bar' }),
-      '.planning/PROJECTS.md': '# Projects\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'foo/bar' }),
+      'PROJECTS.md': '# Projects\n',
     });
 
     try {
@@ -546,8 +548,9 @@ describe('edge cases', () => {
 
   it('rejects current_project with backslash', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: 'foo\\bar' }),
-      '.planning/PROJECTS.md': '# Projects\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'foo\\bar' }),
+      'PROJECTS.md': '# Projects\n',
     });
 
     try {
@@ -564,12 +567,18 @@ describe('edge cases', () => {
 // ─── requireProjectRoot Tests ─────────────────────────────────────────────────
 
 describe('requireProjectRoot', () => {
-  it('returns .planning/projects/<project> when v2 install has current_project set and directory exists', () => {
-    const fixture = createTempProject({ version: 2, project: 'my-app' });
+  it('returns projects/<project> when v2 install has current_project set and directory exists', () => {
+    const fixture = createFixture({
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'my-app' }),
+      'PROJECTS.md': '# Projects\n',
+      'REPOS.md': '# Repos\n',
+      'projects/my-app/STATE.md': '# State',
+    });
 
     try {
       const result = requireProjectRoot(fixture.cwd);
-      assert.equal(result, path.join('.planning', 'projects', 'my-app'));
+      assert.equal(result, path.join('projects', 'my-app'));
     } finally {
       fixture.cleanup();
     }
@@ -577,8 +586,8 @@ describe('requireProjectRoot', () => {
 
   it('throws NO_CURRENT_PROJECT_V2 when v2 install has no current_project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
     });
 
     try {
@@ -593,8 +602,9 @@ describe('requireProjectRoot', () => {
 
   it('throws NO_CURRENT_PROJECT_V2 when v2 install has empty string current_project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: '' }),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: '' }),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
     });
 
     try {
@@ -609,8 +619,9 @@ describe('requireProjectRoot', () => {
 
   it('throws NO_CURRENT_PROJECT_V2 when v2 install has whitespace-only current_project', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ current_project: '   ' }),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: '   ' }),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
     });
 
     try {
@@ -624,13 +635,12 @@ describe('requireProjectRoot', () => {
   });
 
   it('throws PROJECT_NOT_FOUND when v2 install has current_project pointing to nonexistent directory', () => {
-    const fixture = createTempProject({ version: 2, project: 'real-project' });
-
-    // Manually update local config to point at a nonexistent project
-    fs.writeFileSync(
-      path.join(fixture.cwd, '.planning', 'config.local.json'),
-      JSON.stringify({ current_project: 'ghost-project' })
-    );
+    const fixture = createFixture({
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'ghost-project' }),
+      'PROJECTS.md': '# Projects\n',
+      'REPOS.md': '# Repos\n',
+    });
 
     try {
       assert.throws(
@@ -643,12 +653,12 @@ describe('requireProjectRoot', () => {
   });
 
   it('throws INVALID_PROJECT_NAME when current_project contains ..', () => {
-    const fixture = createTempProject({ version: 2, project: 'real-project' });
-
-    fs.writeFileSync(
-      path.join(fixture.cwd, '.planning', 'config.local.json'),
-      JSON.stringify({ current_project: '../etc' })
-    );
+    const fixture = createFixture({
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: '../etc' }),
+      'PROJECTS.md': '# Projects\n',
+      'REPOS.md': '# Repos\n',
+    });
 
     try {
       assert.throws(
@@ -660,21 +670,10 @@ describe('requireProjectRoot', () => {
     }
   });
 
-  it('returns .planning for v1 install (backward compatible)', () => {
-    const fixture = createTempProject({ version: 1 });
-
-    try {
-      const result = requireProjectRoot(fixture.cwd);
-      assert.equal(result, '.planning');
-    } finally {
-      fixture.cleanup();
-    }
-  });
-
   it('resolveProjectPath throws NO_CURRENT_PROJECT_V2 on v2 install without current_project (guard propagates)', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
-      '.planning/PROJECTS.md': '# Projects\n\n| Project | Status |\n',
+      'config.json': JSON.stringify({}),
+      'PROJECTS.md': '# Projects\n\n| Project | Status |\n',
     });
 
     try {
@@ -699,19 +698,18 @@ describe('root layout', () => {
   });
 
   it('loadConfig reads from root-layout config.json', () => {
-    fixture = createTempProject({ layout: 'root', withConfig: { model_profile: 'quality' } });
+    fixture = createTempProject({ withConfig: { model_profile: 'quality' } });
     const config = loadConfig(fixture.cwd);
     assert.equal(config.model_profile, 'quality');
   });
 
   it('loadConfig returns defaults when root-layout config.json has no model_profile', () => {
-    fixture = createTempProject({ layout: 'root' });
+    fixture = createTempProject();
     const config = loadConfig(fixture.cwd);
     assert.equal(config.model_profile, 'balanced');
   });
 
-  it('getProjectRoot returns . for v1 root-layout', () => {
-    // Create a root-layout v1 fixture (no v2 markers with valid headers)
+  it('getProjectRoot returns . for root-layout without v2 markers', () => {
     fixture = createFixture({
       'config.local.json': JSON.stringify({ planningRoot: '.' }),
       'config.json': JSON.stringify({}),
@@ -725,12 +723,12 @@ describe('root layout', () => {
   });
 
   it('isV2Install detects v2 markers in root-layout', () => {
-    fixture = createTempProject({ layout: 'root' });
+    fixture = createTempProject();
     const result = isV2Install(fixture.cwd);
     assert.equal(result, true);
   });
 
-  it('isV2Install returns false for root-layout v1', () => {
+  it('isV2Install returns false for root-layout without v2 markers', () => {
     fixture = createFixture({
       'config.local.json': JSON.stringify({ planningRoot: '.' }),
       'config.json': JSON.stringify({}),
@@ -755,7 +753,6 @@ describe('root layout', () => {
   });
 
   it('loadConfig reads config.json in root-layout auto-detect', () => {
-    // Root layout with config.json + PROJECT.md auto-detect (no config.local.json needed)
     fixture = createFixture({
       'PROJECT.md': '# Project\n',
       'config.json': JSON.stringify({ model_profile: 'budget' }),
@@ -769,13 +766,19 @@ describe('root layout', () => {
 // ─── getProjectDir Tests ─────────────────────────────────────────────────────
 
 describe('getProjectDir', () => {
-  it('returns absolute path under .planning/projects/<slug> for v2 install', () => {
-    const fixture = createTempProject({ version: 2, project: 'my-app' });
+  it('returns absolute path under projects/<slug> for v2 install', () => {
+    const fixture = createFixture({
+      'config.json': JSON.stringify({}),
+      'config.local.json': JSON.stringify({ current_project: 'my-app' }),
+      'PROJECTS.md': '# Projects\n',
+      'REPOS.md': '# Repos\n',
+      'projects/my-app/STATE.md': '# State',
+    });
 
     try {
       const result = getProjectDir(fixture.cwd, 'my-app');
       assert.ok(path.isAbsolute(result), 'Expected absolute path');
-      assert.equal(result, path.join(fixture.cwd, '.planning', 'projects', 'my-app'));
+      assert.equal(result, path.join(fixture.cwd, 'projects', 'my-app'));
     } finally {
       fixture.cleanup();
     }
@@ -783,7 +786,7 @@ describe('getProjectDir', () => {
 
   it('returns absolute path for any slug even if directory does not exist', () => {
     const fixture = createFixture({
-      '.planning/config.json': JSON.stringify({}),
+      'config.json': JSON.stringify({}),
     });
 
     try {
@@ -808,8 +811,8 @@ describe('config two-file merge', () => {
 
   it('loadConfig merges config.json and config.local.json', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ model_profile: 'quality' }),
-      '.planning/config.local.json': JSON.stringify({ current_project: 'my-app' }),
+      'config.json': JSON.stringify({ model_profile: 'quality' }),
+      'config.local.json': JSON.stringify({ current_project: 'my-app' }),
     });
     const config = loadConfig(fixture.cwd);
     assert.equal(config.model_profile, 'quality');
@@ -818,8 +821,8 @@ describe('config two-file merge', () => {
 
   it('loadConfig local overrides shared for overlapping keys', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ model_profile: 'speed' }),
-      '.planning/config.local.json': JSON.stringify({ model_profile: 'quality' }),
+      'config.json': JSON.stringify({ model_profile: 'speed' }),
+      'config.local.json': JSON.stringify({ model_profile: 'quality' }),
     });
     const config = loadConfig(fixture.cwd);
     assert.equal(config.model_profile, 'quality');
@@ -827,17 +830,10 @@ describe('config two-file merge', () => {
 
   it('loadConfig reads config.json when no config.local.json exists', () => {
     fixture = createFixture({
-      '.planning/config.json': JSON.stringify({ model_profile: 'speed' }),
+      'config.json': JSON.stringify({ model_profile: 'speed' }),
     });
     const config = loadConfig(fixture.cwd);
     assert.equal(config.model_profile, 'speed');
   });
 
-  it('loadConfig falls back to legacy dgs.config.json', () => {
-    fixture = createFixture({
-      '.planning/dgs.config.json': JSON.stringify({ model_profile: 'quality' }),
-    });
-    const config = loadConfig(fixture.cwd);
-    assert.equal(config.model_profile, 'quality');
-  });
 });