@ktpartners/dgs-platform 2.7.0 → 2.7.1

package/agents/dgs-executor.md

@@ -517,6 +517,56 @@ node "$HOME/.claude/deliver-great-systems/bin/dgs-tools.cjs" commit "docs({phase
 Separate from per-task commits — captures execution results only.
 </final_commit>
 
+<codereview_gate>
+Check if code review is enabled:
+
+```bash
+CODEREVIEW=$(node "$HOME/.claude/deliver-great-systems/bin/dgs-tools.cjs" config-get workflow.codereview 2>/dev/null || echo "false")
+```
+
+If `CODEREVIEW` is `true`:
+
+Display:
+```
+------------------------------------------------------------
+ DGS > SPAWNING CODE REVIEW
+------------------------------------------------------------
+
+ Reviewing {phase}-{plan} changes across 3 passes...
+```
+
+Compute diff reference for the plan's task commits:
+
+```bash
+FIRST_TASK_COMMIT=$(git log --oneline --grep="feat(${PHASE}-${PLAN}):" --grep="fix(${PHASE}-${PLAN}):" --grep="test(${PHASE}-${PLAN}):" --grep="refactor(${PHASE}-${PLAN}):" --reverse | head -1 | cut -d' ' -f1)
+```
+
+If FIRST_TASK_COMMIT is empty (no task commits found), skip codereview with message: "No task commits found for {phase}-{plan}, skipping code review."
+
+Otherwise, invoke the codereview workflow:
+
+```
+@~/.claude/deliver-great-systems/workflows/codereview.md
+```
+
+With inputs:
+- PHASE: ${PHASE}
+- PLAN: ${PLAN}
+- PLAN_PATH: ${phase_dir}/{phase}-{plan}-PLAN.md
+- PHASE_DIR: ${phase_dir}
+- DIFF_REF: ${FIRST_TASK_COMMIT}^..HEAD
+
+After codereview completes:
+- If auto-fixes were committed, the fix commit hash is noted in the codereview output
+- The SUMMARY.md already has codereview findings appended and CODEREVIEW.md has been created with the full report (both done by codereview workflow)
+- If the codereview workflow committed auto-fixes, amend the metadata commit to include the updated SUMMARY.md and CODEREVIEW.md:
+  ```bash
+  node "$HOME/.claude/deliver-great-systems/bin/dgs-tools.cjs" commit "" --files ${phase_dir}/{phase}-{plan}-SUMMARY.md ${phase_dir}/{phase}-{plan}-CODEREVIEW.md --amend
+  ```
+
+If `CODEREVIEW` is `false` or absent: skip silently (no output).
+</codereview_gate>
+
 <completion_format>
 ```markdown
 ## PLAN COMPLETE
@@ -546,5 +596,6 @@ Plan execution complete when:
 - [ ] STATE.md updated (position, decisions, issues, session)
 - [ ] ROADMAP.md updated with plan progress (via `roadmap update-plan-progress`)
 - [ ] Final metadata commit made (includes SUMMARY.md, STATE.md, ROADMAP.md)
+- [ ] Codereview gate checked (if enabled: codereview invoked, if auto-fixes committed: metadata commit amended)
 - [ ] Completion format returned to orchestrator
 </success_criteria>

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

@@ -505,12 +505,12 @@ describe('v2 mode with project: init todos', () => {
     fixture.cleanup();
   });
 
-  it('returns project-qualified pending_dir', () => {
-    assert.equal(result.pending_dir, path.join('.planning', 'projects', 'test-project', 'todos', 'pending'));
+  it('returns product-level pending_dir', () => {
+    assert.equal(result.pending_dir, path.join('.planning', 'todos', 'pending'));
   });
 
-  it('returns project-qualified completed_dir', () => {
-    assert.equal(result.completed_dir, path.join('.planning', 'projects', 'test-project', 'todos', 'completed'));
+  it('returns product-level completed_dir', () => {
+    assert.equal(result.completed_dir, path.join('.planning', 'todos', 'completed'));
   });
 
   it('returns dgs_mode v2', () => {

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

@@ -138,7 +138,7 @@ function createBackupTag(cwd) {
 // ─── Migration Planning ─────────────────────────────────────────────────────
 
 // Directories to move (project-level, order: directories first)
-const DIRS_TO_MOVE = ['phases', 'research', 'todos', 'quick', 'debug'];
+const DIRS_TO_MOVE = ['phases', 'research', 'quick', 'debug'];
 
 // Files to move (project-level)
 const FILES_TO_MOVE = ['PROJECT.md', 'REQUIREMENTS.md', 'ROADMAP.md', 'STATE.md'];

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

@@ -327,13 +327,13 @@ describe('migrateV1ToV2', () => {
     assert.ok(log.includes('rename'));
   });
 
-  it('moves todos/ directory if it exists', () => {
+  it('preserves todos/ at product level (not moved to project)', () => {
     writeFile(tmpDir, '.planning/todos/pending/task1.md', '# Todo 1\n');
     createV1Install(tmpDir, 'Test App');
     migrateV1ToV2(tmpDir, 'test-app');
 
-    assert.ok(fs.existsSync(path.join(tmpDir, '.planning', 'projects', 'test-app', 'todos', 'pending', 'task1.md')));
-    assert.ok(!fs.existsSync(path.join(tmpDir, '.planning', 'todos')));
+    assert.ok(fs.existsSync(path.join(tmpDir, '.planning', 'todos', 'pending', 'task1.md')));
+    assert.ok(!fs.existsSync(path.join(tmpDir, '.planning', 'projects', 'test-app', 'todos')));
   });
 
   it('moves quick/ directory if it exists', () => {
@@ -407,18 +407,15 @@ describe('collectMigrationMoves', () => {
     writeFile(tmpDir, '.planning/PROJECT.md', '# Project: Test\n');
     writeFile(tmpDir, '.planning/phases/01-setup/01-01-PLAN.md', '---\n---\n');
     writeFile(tmpDir, '.planning/research/notes.md', '# Notes\n');
-    writeFile(tmpDir, '.planning/todos/pending/task.md', '# Todo\n');
 
     const { moves } = collectMigrationMoves(tmpDir, 'test');
 
     const phasesMove = moves.find(m => m.relSource.endsWith('phases'));
     const researchMove = moves.find(m => m.relSource.endsWith('research'));
-    const todosMove = moves.find(m => m.relSource.endsWith('todos'));
     const projectMove = moves.find(m => m.relSource.endsWith('PROJECT.md'));
 
     assert.strictEqual(phasesMove.isDir, true);
     assert.strictEqual(researchMove.isDir, true);
-    assert.strictEqual(todosMove.isDir, true);
     assert.strictEqual(projectMove.isDir, false);
   });
 });

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

@@ -41,7 +41,7 @@ const COMPREHENSIVE_PATTERN = /\.planning\//;
  * multi-project install because they resolve to the root instead of
  * .planning/<project>/<path>.
  */
-const PROJECT_SCOPED_PATTERN = /\.planning\/(?:STATE|ROADMAP|PROJECT(?!S\.md)|REQUIREMENTS|phases|archive|quick|todos|debug|research)/;
+const PROJECT_SCOPED_PATTERN = /\.planning\/(?:STATE|ROADMAP|PROJECT(?!S\.md)|REQUIREMENTS|phases|archive|quick|debug|research)/;
 
 /**
  * Workflow files allowlisted from comprehensive .planning/ scanning.

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

@@ -14,7 +14,7 @@ const { getPlanningRoot } = require('./paths.cjs');
 
 // ─── Constants ──────────────────────────────────────────────────────────────
 
-const STANDARD_DIRS = ['phases', 'research', 'todos', 'quick', 'debug'];
+const STANDARD_DIRS = ['phases', 'research', 'quick', 'debug'];
 
 // ─── Project Subfolder Creation ─────────────────────────────────────────────
 

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

@@ -64,7 +64,7 @@ describe('createProjectSubfolder', () => {
 
   it('creates all required subdirectories', () => {
     createProjectSubfolder(tmpDir, 'auth-overhaul', 'Auth Overhaul');
-    const expectedDirs = ['phases', 'research', 'todos', 'quick', 'debug'];
+    const expectedDirs = ['phases', 'research', 'quick', 'debug'];
     for (const dir of expectedDirs) {
       assert.ok(
         fs.existsSync(path.join(tmpDir, '.planning', 'projects', 'auth-overhaul', dir)),

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

@@ -571,7 +571,7 @@ function cmdStateSnapshot(cwd, raw) {
 function getMilestonePhaseFilter(cwd) {
   const milestonePhaseNums = new Set();
   try {
-    const roadmap = fs.readFileSync(path.join(cwd, '.planning', 'ROADMAP.md'), 'utf-8');
+    const roadmap = fs.readFileSync(path.join(getPlanningRoot(cwd), 'ROADMAP.md'), 'utf-8');
     const phasePattern = /#{2,4}\s*Phase\s+(\d+[A-Z]?(?:\.\d+)*)\s*:/gi;
     let m;
     while ((m = phasePattern.exec(roadmap)) !== null) {
@@ -632,7 +632,7 @@ function buildStateFrontmatter(bodyContent, cwd) {
 
   if (cwd) {
     try {
-      const phasesDir = path.join(cwd, '.planning', 'phases');
+      const phasesDir = path.join(getPlanningRoot(cwd), 'phases');
       if (fs.existsSync(phasesDir)) {
         const isDirInMilestone = getMilestonePhaseFilter(cwd);
         const phaseDirs = fs.readdirSync(phasesDir, { withFileTypes: true })

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

@@ -228,8 +228,8 @@ function createTempProject(options) {
     }
 
     if (withTodos) {
-      fs.mkdirSync(path.join(cwd, '.planning', PROJECTS_DIR, project, 'todos', 'pending'), { recursive: true });
-      fs.mkdirSync(path.join(cwd, '.planning', PROJECTS_DIR, project, 'todos', 'completed'), { recursive: true });
+      fs.mkdirSync(path.join(cwd, '.planning', 'todos', 'pending'), { recursive: true });
+      fs.mkdirSync(path.join(cwd, '.planning', 'todos', 'completed'), { recursive: true });
     }
 
     if (withResearch) {

package/package.json

@@ -4,7 +4,7 @@
   "bugs": {
     "url": "https://github.com/KT-Partners-Ltd/dgs-platform-docs/issues"
   },
-  "version": "2.7.0",
+  "version": "2.7.1",
   "description": "Deliver Great Systems Platform — A meta-prompting, context engineering and spec-driven development system for Claude Code and Gemini by KT Partners.",
   "bin": {
     "dgs": "bin/install.js"