gsd-pi 2.30.0 → 2.31.0-dev.6ffc032

package/src/resources/extensions/gsd/tests/plan-slice-prompt.test.ts

@@ -28,18 +28,12 @@ const BASE_VARS = {
   sourceFilePaths: "- **Requirements**: `.gsd/REQUIREMENTS.md`",
 };
 
-test("plan-slice prompt: commit step present when commit_docs=true", () => {
-  const result = loadPrompt("plan-slice", { ...BASE_VARS, commitInstruction: "Commit: `docs(S01): add slice plan`" });
-  assert.ok(result.includes("docs(S01): add slice plan"));
+test("plan-slice prompt: commit instruction says do not commit (external state)", () => {
+  const result = loadPrompt("plan-slice", { ...BASE_VARS, commitInstruction: "Do not commit planning artifacts — .gsd/ is managed externally." });
+  assert.ok(result.includes("Do not commit planning artifacts"));
   assert.ok(!result.includes("{{commitInstruction}}"));
 });
 
-test("plan-slice prompt: no commit step when commit_docs=false", () => {
-  const result = loadPrompt("plan-slice", { ...BASE_VARS, commitInstruction: "Do not commit — planning docs are not tracked in git for this project." });
-  assert.ok(!result.includes("docs(S01): add slice plan"));
-  assert.ok(result.includes("Do not commit"));
-});
-
 test("plan-slice prompt: all variables substituted", () => {
   const result = loadPrompt("plan-slice", { ...BASE_VARS, commitInstruction: "Commit: `docs(S01): add slice plan`" });
   assert.ok(!result.includes("{{"));

package/src/resources/extensions/gsd/tests/preferences.test.ts

@@ -1,6 +1,6 @@
 /**
  * Preferences tests — consolidated from:
- *   - preferences-git.test.ts (git.isolation, git.merge_to_main, git.commit_docs)
+ *   - preferences-git.test.ts (git.isolation, git.merge_to_main)
  *   - preferences-hooks.test.ts (post-unit + pre-dispatch hook config)
  *   - preferences-mode.test.ts (solo/team mode defaults, overrides)
  *   - preferences-models.test.ts (model config parsing, OpenRouter, CRLF)
@@ -39,14 +39,6 @@ test("git.merge_to_main produces deprecation warning", () => {
   }
 });
 
-test("git.commit_docs accepts boolean, rejects string", () => {
-  const { errors: e1, preferences: p1 } = validatePreferences({ git: { commit_docs: false } });
-  assert.equal(e1.length, 0);
-  assert.equal(p1.git?.commit_docs, false);
-
-  const { errors: e2 } = validatePreferences({ git: { commit_docs: "no" as any } });
-  assert.ok(e2.length > 0);
-});
 
 test("getIsolationMode defaults to worktree when no prefs file", { skip: "requires no global ~/.gsd/preferences.md" }, () => {
   assert.equal(getIsolationMode(), "worktree");

package/src/resources/extensions/gsd/tests/worktree.test.ts

@@ -108,10 +108,7 @@ async function main(): Promise<void> {
     assertEq(readIntegrationBranch(repo, "M001"), "f-123-thing",
       "captureIntegrationBranch records the current branch");
 
-    // Verify it was committed (not just written to disk)
-    const logOut = run("git log --oneline -1", repo);
-    assertTrue(logOut.includes("integration branch"), "metadata committed to git");
-
+    // .gsd/ metadata is written to disk only (not committed) since commit_docs removal
     rmSync(repo, { recursive: true, force: true });
   }
 

package/src/resources/extensions/gsd/worktree.ts

@@ -56,13 +56,13 @@ export function setActiveMilestoneId(basePath: string, milestoneId: string | nul
  * record when the user starts from a different branch (#300). Always a no-op
  * if on a GSD slice branch.
  */
-export function captureIntegrationBranch(basePath: string, milestoneId: string, options?: { commitDocs?: boolean }): void {
+export function captureIntegrationBranch(basePath: string, milestoneId: string): void {
   // In a worktree, the base branch is implicit (worktree/<name>).
   // Writing it to META.json would leave stale metadata after merge back to main.
   if (detectWorktreeName(basePath)) return;
   const svc = getService(basePath);
   const current = svc.getCurrentBranch();
-  writeIntegrationBranch(basePath, milestoneId, current, options);
+  writeIntegrationBranch(basePath, milestoneId, current);
 }
 
 // ─── Pure Utility Functions (unchanged) ────────────────────────────────────