@ktpartners/dgs-platform 2.7.5 → 2.8.0

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

@@ -4,9 +4,9 @@
  * Provides the data model and operations for document management:
  * add, list, remove, move, and INDEX.md maintenance.
  * Documents are stored in scope-specific docs/ directories:
- * - Product-level: .planning/docs/
- * - Idea-scoped: .planning/ideas/{state}/{idea-slug}/docs/
- * - Spec-scoped: .planning/specs/{spec-slug}/docs/
+ * - Product-level: docs/
+ * - Idea-scoped: ideas/{state}/{idea-slug}/docs/
+ * - Spec-scoped: specs/{spec-slug}/docs/
  * Text extraction produces .extracted.txt sidecars for PDF, XLSX, CSV, DOCX.
  * Large extractions also generate .summary.txt sidecars.
  */

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

@@ -13,6 +13,9 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 
+const { initGitRepo } = require('./test-helpers.cjs');
+const { resetPaths } = require('./paths.cjs');
+
 const {
   resolveDocsDir,
   scanDocsDir,
@@ -29,15 +32,18 @@ describe('docs', () => {
 
     beforeEach(() => {
       tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'dgs-resolve-docs-'));
+      tmpDir = fs.realpathSync(tmpDir);
+      initGitRepo(tmpDir);
     });
 
     afterEach(() => {
+      resetPaths();
       fs.rmSync(tmpDir, { recursive: true, force: true });
     });
 
-    it('product scope returns .planning/docs/product', () => {
+    it('product scope returns docs/product at root', () => {
       const result = resolveDocsDir(tmpDir, 'product');
-      assert.equal(result, path.join(tmpDir, '.planning', 'docs', 'product'));
+      assert.equal(result, path.join(tmpDir, 'docs', 'product'));
     });
 
     it('product scope path ends with /docs/product', () => {
@@ -47,11 +53,12 @@ describe('docs', () => {
     });
 
     it('idea scope returns docs path for idea found in pending', () => {
-      // Create the fixture directory that resolveDocsDir searches for
-      fs.mkdirSync(path.join(tmpDir, '.planning', 'ideas', 'pending', 'my-idea'), { recursive: true });
-      const result = resolveDocsDir(tmpDir, 'idea', 'my-idea');
-      assert.ok(result.includes(path.join('ideas', 'pending', 'my-idea', 'docs')),
-        `Expected ideas/pending/my-idea/docs in path, got: ${result}`);
+      // Create the idea file that resolveDocsDir searches for
+      fs.mkdirSync(path.join(tmpDir, 'ideas', 'pending'), { recursive: true });
+      fs.writeFileSync(path.join(tmpDir, 'ideas', 'pending', '001-my-idea.md'), '---\nid: 1\ntitle: "My Idea"\n---\n');
+      const result = resolveDocsDir(tmpDir, 'idea', '1');
+      assert.ok(result.includes(path.join('ideas', 'pending', '001-my-idea', 'docs')),
+        `Expected ideas/pending/001-my-idea/docs in path, got: ${result}`);
     });
 
     it('spec scope returns correct path', () => {

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

@@ -420,7 +420,7 @@ function createRepoBranches(cwd, repoNames, branchName, config, baseBranch) {
             reason: 'base_branch_missing',
             error: `Base branch '${baseBranch}' does not exist in repo '${name}'. ` +
                    `Checked both local and remote (origin/${baseBranch}). ` +
-                   `Configure git.base_branch in .planning/config.json or create the branch first.`,
+                   `Configure git.base_branch in config.json or create the branch first.`,
             repo: name,
           };
         }
@@ -578,7 +578,7 @@ function updateRepoStatus(cwd, projectSlug, repoResults) {
 }
 
 /**
- * Get project slugs from .planning directory.
+ * Get project slugs from projects directory.
  * Looks for subdirectories that contain STATE.md (project folders).
  *
  * @param {string} cwd - Product root

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

@@ -11,15 +11,13 @@ const os = require('os');
 const { execSync } = require('child_process');
 
 const { createTempDir, cleanupDir, initGitRepo } = require('./test-helpers.cjs');
+const { resetPaths } = require('./paths.cjs');
 
 /**
- * Helper: create a product root with .planning/REPOS.md and git repos.
+ * Helper: create a product root with REPOS.md and git repos.
  */
 function setupProductRoot(tmpDir, repoNames) {
-  // Create .planning with REPOS.md
-  const planningDir = path.join(tmpDir, '.planning');
-  fs.mkdirSync(planningDir, { recursive: true });
-
+  // Create REPOS.md at root
   let reposMd = '# Repos\n\n';
   reposMd += '| Name | Path | GitHub URL | Description |\n';
   reposMd += '|------|------|------------|-------------|\n';
@@ -33,7 +31,7 @@ function setupProductRoot(tmpDir, repoNames) {
     execSync('git commit -m "add readme"', { cwd: path.join(tmpDir, name), stdio: 'pipe' });
   }
 
-  fs.writeFileSync(path.join(planningDir, 'REPOS.md'), reposMd);
+  fs.writeFileSync(path.join(tmpDir, 'REPOS.md'), reposMd);
   return tmpDir;
 }
 
@@ -57,8 +55,8 @@ const {
 
 describe('resolveRepoRelativePath', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('resolves src/index.ts in repo web-app to correct absolute path', () => {
     setupProductRoot(tmpDir, ['web-app']);
@@ -76,17 +74,17 @@ describe('resolveRepoRelativePath', () => {
   });
 
   it('works with ../repo paths in REPOS.md (sibling layout)', () => {
-    // Create a sibling-layout product root
+    // Create a sibling-layout product root with its own git repo
     const productRoot = path.join(tmpDir, 'product-root');
-    const planningDir = path.join(productRoot, '.planning');
-    fs.mkdirSync(planningDir, { recursive: true });
+    initGitRepo(productRoot);
+    resetPaths();
 
     let reposMd = '# Repos\n\n';
     reposMd += '| Name | Path | GitHub URL | Description |\n';
     reposMd += '|------|------|------------|-------------|\n';
     reposMd += '| api-service | ../api-service | | Test repo |\n';
 
-    fs.writeFileSync(path.join(planningDir, 'REPOS.md'), reposMd);
+    fs.writeFileSync(path.join(productRoot, 'REPOS.md'), reposMd);
     fs.mkdirSync(path.join(tmpDir, 'api-service'), { recursive: true });
 
     const result = resolveRepoRelativePath(productRoot, 'api-service', 'src/server.ts');
@@ -115,8 +113,8 @@ describe('resolveRepoRelativePath', () => {
 
 describe('preflightCheck', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('all repos clean — passed: true', () => {
     setupProductRoot(tmpDir, ['repo-a', 'repo-b']);
@@ -166,7 +164,7 @@ describe('preflightCheck', () => {
 
   it('SUMMARY.md with status: partial — passed: false, partial_execution set', () => {
     setupProductRoot(tmpDir, ['repo-a']);
-    const phaseDir = path.join(tmpDir, '.planning', 'phases', '04-test');
+    const phaseDir = path.join(tmpDir, 'phases', '04-test');
     fs.mkdirSync(phaseDir, { recursive: true });
     fs.writeFileSync(path.join(phaseDir, '04-01-SUMMARY.md'),
       '---\nphase: 04-test\nplan: 01\nstatus: partial\n---\n# Summary\n');
@@ -177,7 +175,7 @@ describe('preflightCheck', () => {
 
   it('SUMMARY.md with status: complete — passed: true', () => {
     setupProductRoot(tmpDir, ['repo-a']);
-    const phaseDir = path.join(tmpDir, '.planning', 'phases', '04-test');
+    const phaseDir = path.join(tmpDir, 'phases', '04-test');
     fs.mkdirSync(phaseDir, { recursive: true });
     fs.writeFileSync(path.join(phaseDir, '04-01-SUMMARY.md'),
       '---\nphase: 04-test\nplan: 01\nstatus: complete\n---\n# Summary\n');
@@ -191,8 +189,8 @@ describe('preflightCheck', () => {
 
 describe('commitPerRepo', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   const options = {
     type: 'feat',
@@ -329,8 +327,8 @@ describe('buildPlanningCommitBody', () => {
 
 describe('detectRepoChanges', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('repo with modified file — listed', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -388,8 +386,8 @@ describe('detectRepoChanges', () => {
 
 describe('preflightCheck edge cases', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('empty repoNames array — passed: true (vacuously)', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -409,8 +407,8 @@ describe('preflightCheck edge cases', () => {
 
 describe('commitPerRepo edge cases', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   const options = {
     type: 'feat',
@@ -429,8 +427,8 @@ describe('commitPerRepo edge cases', () => {
 
 describe('detectRepoChanges edge cases', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('repo with deleted file — listed', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -444,8 +442,8 @@ describe('detectRepoChanges edge cases', () => {
 
 describe('roundtrip integration', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('detectRepoChanges -> commitPerRepo -> buildPlanningCommitBody produces coherent result', () => {
     setupProductRoot(tmpDir, ['repo-a', 'repo-b']);
@@ -481,8 +479,8 @@ describe('roundtrip integration', () => {
 
 describe('createRepoBranches', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('branching_strategy none — created: false', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -529,8 +527,8 @@ describe('createRepoBranches', () => {
 
 describe('createRepoBranches prefix collision', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('detects prefix collision when creating similar branch name', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -587,8 +585,8 @@ describe('createRepoBranches prefix collision', () => {
 
 describe('createRepoBranches project-scoped prefix collision', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('no collision when project slugs differ by / delimiter (checkout vs checkout-v2)', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -646,8 +644,8 @@ describe('createRepoBranches project-scoped prefix collision', () => {
 
 describe('createRepoBranches base_branch', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('without baseBranch creates branch from HEAD (backwards compat)', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -737,8 +735,8 @@ describe('createRepoBranches base_branch', () => {
 
 describe('reportBranchesToMerge', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('repos with the branch — listed', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -765,16 +763,16 @@ describe('reportBranchesToMerge', () => {
 
 describe('updateRepoStatus', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('creates Repo Status section in STATE.md', () => {
     setupProductRoot(tmpDir, ['repo-a']);
     // Create initial STATE.md
-    fs.writeFileSync(path.join(tmpDir, '.planning', 'STATE.md'), '# Project State\n\n## Current Position\nPhase: 4\n');
+    fs.writeFileSync(path.join(tmpDir, 'STATE.md'), '# Project State\n\n## Current Position\nPhase: 4\n');
     const repoResults = [{ repo: 'repo-a', status: 'complete', sha: 'abc1234' }];
     updateRepoStatus(tmpDir, null, repoResults);
-    const content = fs.readFileSync(path.join(tmpDir, '.planning', 'STATE.md'), 'utf-8');
+    const content = fs.readFileSync(path.join(tmpDir, 'STATE.md'), 'utf-8');
     assert.ok(content.includes('## Repo Status'));
     assert.ok(content.includes('repo-a'));
     assert.ok(content.includes('abc1234'));
@@ -782,11 +780,11 @@ describe('updateRepoStatus', () => {
 
   it('updates existing Repo Status section (replace, not duplicate)', () => {
     setupProductRoot(tmpDir, ['repo-a']);
-    fs.writeFileSync(path.join(tmpDir, '.planning', 'STATE.md'),
+    fs.writeFileSync(path.join(tmpDir, 'STATE.md'),
       '# State\n\n## Repo Status\n\n| Repo | Branch | Last Commit | Touched By |\n|------|--------|-------------|------------|\n| old | main | old123 | - |\n\n## Other\n');
     const repoResults = [{ repo: 'repo-a', status: 'complete', sha: 'new456' }];
     updateRepoStatus(tmpDir, null, repoResults);
-    const content = fs.readFileSync(path.join(tmpDir, '.planning', 'STATE.md'), 'utf-8');
+    const content = fs.readFileSync(path.join(tmpDir, 'STATE.md'), 'utf-8');
     assert.ok(content.includes('new456'));
     assert.ok(!content.includes('old123'));
     // Should only have one Repo Status section
@@ -796,20 +794,20 @@ describe('updateRepoStatus', () => {
 
   it('handles repo with no branch (shows current branch)', () => {
     setupProductRoot(tmpDir, ['repo-a']);
-    fs.writeFileSync(path.join(tmpDir, '.planning', 'STATE.md'), '# State\n');
+    fs.writeFileSync(path.join(tmpDir, 'STATE.md'), '# State\n');
     const repoResults = [{ repo: 'repo-a', status: 'complete', sha: 'abc1234' }];
     updateRepoStatus(tmpDir, null, repoResults);
-    const content = fs.readFileSync(path.join(tmpDir, '.planning', 'STATE.md'), 'utf-8');
+    const content = fs.readFileSync(path.join(tmpDir, 'STATE.md'), 'utf-8');
     // Should show 'main' or 'master' as current branch
     assert.ok(content.includes('main') || content.includes('master'));
   });
 
   it('works when STATE.md has no existing Repo Status section', () => {
     setupProductRoot(tmpDir, ['repo-a']);
-    fs.writeFileSync(path.join(tmpDir, '.planning', 'STATE.md'), '# State\n\n## Current Position\nPhase: 4\n');
+    fs.writeFileSync(path.join(tmpDir, 'STATE.md'), '# State\n\n## Current Position\nPhase: 4\n');
     const repoResults = [{ repo: 'repo-a', status: 'complete', sha: 'abc1234' }];
     updateRepoStatus(tmpDir, null, repoResults);
-    const content = fs.readFileSync(path.join(tmpDir, '.planning', 'STATE.md'), 'utf-8');
+    const content = fs.readFileSync(path.join(tmpDir, 'STATE.md'), 'utf-8');
     assert.ok(content.includes('## Repo Status'));
   });
 });
@@ -818,8 +816,8 @@ describe('updateRepoStatus', () => {
 
 describe('CLI integration: commit preflight', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('commit preflight with clean repos — returns passed: true via CLI', () => {
     setupProductRoot(tmpDir, ['repo-a', 'repo-b']);
@@ -862,8 +860,8 @@ describe('CLI integration: commit preflight', () => {
 
 describe('CLI integration: commit --multi-repo', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('commit --multi-repo with changes in 2 repos — both committed', () => {
     setupProductRoot(tmpDir, ['repo-a', 'repo-b']);
@@ -938,8 +936,8 @@ describe('CLI integration: commit --multi-repo', () => {
 
 describe('retryCommitPerRepo', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   const options = {
     type: 'feat',
@@ -1141,8 +1139,8 @@ describe('retryCommitPerRepo', () => {
 
 describe('detectManualResolution', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('clean working tree with all expected files present -- resolved: true, reason: manually_resolved', () => {
     setupProductRoot(tmpDir, ['repo-a']);
@@ -1200,8 +1198,8 @@ describe('detectManualResolution', () => {
 
 describe('cmdCommitMultiRepo advisory preflight', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('dirty repo produces warning but commit proceeds for other repos', () => {
     setupProductRoot(tmpDir, ['repo-a', 'repo-b']);
@@ -1249,7 +1247,7 @@ describe('cmdCommitMultiRepo advisory preflight', () => {
     setupProductRoot(tmpDir, ['repo-a']);
 
     // Create a phase dir with a partial SUMMARY.md
-    const phaseDir = path.join(tmpDir, '.planning', 'phases', '04-test');
+    const phaseDir = path.join(tmpDir, 'phases', '04-test');
     fs.mkdirSync(phaseDir, { recursive: true });
     fs.writeFileSync(path.join(phaseDir, '04-01-SUMMARY.md'),
       '---\nphase: 04-test\nplan: 01\nstatus: partial\n---\n# Summary\n');
@@ -1314,10 +1312,10 @@ describe('cmdCommitMultiRepo advisory preflight', () => {
  * Returns { rootDir, productRoot } where productRoot is the planning repo cwd.
  */
 function setupFlatLayout(tmpDir, repoNames) {
-  // Create planning repo (product root)
+  // Create planning repo (product root) with its own git repo
   const productRoot = path.join(tmpDir, 'planning-repo');
-  const planningDir = path.join(productRoot, '.planning');
-  fs.mkdirSync(planningDir, { recursive: true });
+  initGitRepo(productRoot);
+  resetPaths();
 
   let reposMd = '# Repos\n\n';
   reposMd += '| Name | Path | GitHub URL | Description |\n';
@@ -1333,7 +1331,7 @@ function setupFlatLayout(tmpDir, repoNames) {
     execSync('git commit -m "add readme"', { cwd: path.join(tmpDir, name), stdio: 'pipe' });
   }
 
-  fs.writeFileSync(path.join(planningDir, 'REPOS.md'), reposMd);
+  fs.writeFileSync(path.join(productRoot, 'REPOS.md'), reposMd);
   return { rootDir: tmpDir, productRoot };
 }
 
@@ -1341,8 +1339,8 @@ function setupFlatLayout(tmpDir, repoNames) {
 
 describe('resolveRepoPath with ../repo paths', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('resolves ../api-service to absolute path of sibling directory', () => {
     const { productRoot } = setupFlatLayout(tmpDir, ['api-service']);
@@ -1380,8 +1378,8 @@ describe('resolveRepoPath with ../repo paths', () => {
 
 describe('multi-repo execution with ../repo paths', () => {
   let tmpDir;
-  beforeEach(() => { tmpDir = createTempDir(); });
-  afterEach(() => { cleanupDir(tmpDir); });
+  beforeEach(() => { tmpDir = createTempDir(); tmpDir = fs.realpathSync(tmpDir); initGitRepo(tmpDir); });
+  afterEach(() => { resetPaths(); cleanupDir(tmpDir); });
 
   it('preflightCheck passes for clean sibling repos', () => {
     const { productRoot } = setupFlatLayout(tmpDir, ['repo-a', 'repo-b']);

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

@@ -7,7 +7,7 @@
  * discuss-save, research-save, appendDiscussionEntry, and appendResearchEntry.
  *
  * Ideas are stored as markdown files with YAML frontmatter in
- * .planning/ideas/{pending,done,rejected}/ directories.
+ * ideas/{pending,done,rejected}/ directories.
  * A manifest.json tracks the next available sequential ID (never reused).
  *
  * Section order: Body > ## Notes > ## Discussion Log > ## Research Log
@@ -23,7 +23,7 @@ const { extractNameFromAuthor } = require('./identity.cjs');
 // ─── Manifest Management ─────────────────────────────────────────────────────
 
 /**
- * Load the ideas manifest from .planning/ideas/manifest.json.
+ * Load the ideas manifest from ideas/manifest.json.
  * Returns { next_id: N }. If missing, returns { next_id: 1 }.
  *
  * @param {string} cwd - Working directory
@@ -42,7 +42,7 @@ function loadManifest(cwd) {
 }
 
 /**
- * Save the ideas manifest to .planning/ideas/manifest.json.
+ * Save the ideas manifest to ideas/manifest.json.
  *
  * @param {string} cwd - Working directory
  * @param {{ next_id: number }} manifest
@@ -75,7 +75,7 @@ function allocateId(cwd) {
 const IDEA_STATES = ['pending', 'done', 'rejected', 'consolidated'];
 
 /**
- * Ensure .planning/ideas/{pending,done,rejected}/ directories exist.
+ * Ensure ideas/{pending,done,rejected}/ directories exist.
  *
  * @param {string} cwd - Working directory
  */