gentle-pi 0.4.2 → 0.4.4

package/tests/runtime-harness.mjs

@@ -31,6 +31,10 @@ const EXPECTED_COMMANDS = [
 	"gentle-ai:install-sdd",
 	"gentle-ai:sdd-preflight",
 	"gentle:sdd-preflight",
+	"sdd-status",
+	"gentle-ai:sdd-status",
+	"sdd-continue",
+	"gentle-ai:sdd-continue",
 	"gentle:models",
 	"gentle-ai:models",
 	"gentleman:models",
@@ -246,6 +250,25 @@ async function run() {
 		assert.match(onboardPromptResult.systemPrompt, /onboard base/);
 		assert.match(onboardPromptResult.systemPrompt, /## SDD Session Preflight/);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-onboard.md")), true);
+		await mkdir(join(promptCwd, "openspec", "changes", "status-demo", "specs", "demo"), { recursive: true });
+		await writeFile(join(promptCwd, "openspec", "changes", "status-demo", "proposal.md"), "# Proposal\n");
+		await writeFile(join(promptCwd, "openspec", "changes", "status-demo", "specs", "demo", "spec.md"), "# Spec\n");
+		await writeFile(join(promptCwd, "openspec", "changes", "status-demo", "design.md"), "# Design\n");
+		await writeFile(join(promptCwd, "openspec", "changes", "status-demo", "tasks.md"), "# Tasks\n\n- [ ] 1.1 Implement demo\n");
+		const applyPromptResult = await promptHook(
+			{ agentName: "sdd-apply", systemPrompt: "apply base" },
+			createCtx(promptCwd, true, "sdd-apply-session"),
+		);
+		assert.match(applyPromptResult.systemPrompt, /## Native SDD Status Engine/);
+		assert.match(applyPromptResult.systemPrompt, /"changeName": "status-demo"/);
+		assert.match(applyPromptResult.systemPrompt, /### apply instructions/);
+		const statusCtx = createCtx(promptCwd, true);
+		await commands.get("sdd-status").handler("status-demo --json", statusCtx);
+		assert.match(statusCtx.ui.notifications.at(-1).message, /"schemaName": "gentle-pi\.sdd-status"/);
+		const continueCtx = createCtx(promptCwd, true);
+		await commands.get("sdd-continue").handler("status-demo", continueCtx);
+		assert.match(continueCtx.ui.notifications.at(-1).message, /Native SDD Dispatcher/);
+		assert.match(continueCtx.ui.notifications.at(-1).message, /nextPhase: sdd-apply/);
 	} finally {
 		await rm(promptCwd, { recursive: true, force: true });
 	}
@@ -311,8 +334,10 @@ async function run() {
 		);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-apply.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "chains", "sdd-full.chain.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md")), true);
 		await writeFile(join(globalAgentHome, "agents", "sdd-apply.md"), "stale global apply\n");
 		await writeFile(join(globalAgentHome, "chains", "sdd-full.chain.md"), "stale global chain\n");
+		await writeFile(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md"), "stale global status contract\n");
 		await mkdir(join(noUiCwd, ".pi", "agents"), { recursive: true });
 		await writeFile(join(noUiCwd, ".pi", "agents", "sdd-apply.md"), "project override must stay\n");
 		for (const handler of hooks.get("session_start")) {
@@ -328,6 +353,11 @@ async function run() {
 			"stale global chain\n",
 			"session_start must refresh stale global SDD chains",
 		);
+		assert.notEqual(
+			await readFile(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md"), "utf8"),
+			"stale global status contract\n",
+			"session_start must refresh stale global SDD support files",
+		);
 		assert.equal(
 			await readFile(join(noUiCwd, ".pi", "agents", "sdd-apply.md"), "utf8"),
 			"project override must stay\n",
@@ -402,7 +432,9 @@ async function run() {
 		assert.equal(existsSync(join(lazySddCwd, ".pi", "agents", "sdd-apply.md")), false);
 		assert.equal(existsSync(join(lazySddCwd, ".pi", "chains", "sdd-full.chain.md")), false);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-apply.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-status.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-sync.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "chains", "sdd-full.chain.md")), true);
 		assert.equal(ctx.ui.selections.length, 3);
 		assert.equal(ctx.ui.selections[0].label, "SDD execution mode");
@@ -436,7 +468,9 @@ async function run() {
 		assert.equal(existsSync(join(lazySddCwd, ".pi", "agents", "sdd-apply.md")), false);
 		assert.equal(existsSync(join(lazySddCwd, ".pi", "chains", "sdd-full.chain.md")), false);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-apply.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-status.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-sync.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "chains", "sdd-full.chain.md")), true);
 		const globalSddApply = await readFile(
 			join(globalAgentHome, "agents", "sdd-apply.md"),
@@ -605,6 +639,8 @@ async function run() {
 		assert.match(ctx.ui.notifications.at(-1).message, /Global Gentle AI SDD assets installed/);
 		assert.equal(existsSync(join(installCwd, ".pi", "agents", "sdd-apply.md")), false);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-apply.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-status.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md")), true);
 	} finally {
 		await rm(installCwd, { recursive: true, force: true });
 	}
@@ -613,9 +649,11 @@ async function run() {
 	try {
 		await mkdir(join(staleAssetsCwd, ".pi", "agents"), { recursive: true });
 		await mkdir(join(staleAssetsCwd, ".pi", "chains"), { recursive: true });
+		await mkdir(join(staleAssetsCwd, ".pi", "gentle-ai", "support"), { recursive: true });
 		await writeFile(join(staleAssetsCwd, ".pi", "agents", "sdd-apply.md"), "stale apply\n");
 		await writeFile(join(staleAssetsCwd, ".pi", "agents", "sdd-spec.md"), "stale spec\n");
 		await writeFile(join(staleAssetsCwd, ".pi", "chains", "sdd-full.chain.md"), "stale chain\n");
+		await writeFile(join(staleAssetsCwd, ".pi", "gentle-ai", "support", "sdd-status-contract.md"), "stale status contract\n");
 		const ctx = createCtx(staleAssetsCwd, true);
 		await commands.get("gentle-ai:status").handler("", ctx);
 		assert.match(ctx.ui.notifications.at(-1).message, /Project-local SDD override drift: \d+ file\(s\)/);
@@ -638,7 +676,9 @@ async function run() {
 		assert.equal(existsSync(join(sddCwd, ".pi", "agents", "sdd-apply.md")), false);
 		assert.equal(existsSync(join(sddCwd, ".pi", "chains", "sdd-full.chain.md")), false);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-apply.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-status.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "agents", "sdd-sync.md")), true);
+		assert.equal(existsSync(join(globalAgentHome, "gentle-ai", "support", "sdd-status-contract.md")), true);
 		assert.equal(existsSync(join(globalAgentHome, "chains", "sdd-full.chain.md")), true);
 		assert.equal(ctx.ui.selections.length, 3);
 		assert.match(ctx.ui.notifications[0].message, /SDD preflight complete/);

package/tests/sdd-status.test.ts

@@ -0,0 +1,291 @@
+import assert from "node:assert/strict";
+import { mkdirSync, writeFileSync } from "node:fs";
+import { mkdtemp } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { dirname, join } from "node:path";
+import test from "node:test";
+import {
+	listActiveOpenSpecChanges,
+	parseSddStatusCommandArgs,
+	renderPhaseInstructions,
+	renderSddStatusMarkdown,
+	resolveSddStatus,
+} from "../lib/sdd-status.ts";
+
+async function workspace(): Promise<string> {
+	return mkdtemp(join(tmpdir(), "gentle-pi-sdd-status-"));
+}
+
+function write(path: string, content: string): void {
+	mkdirSync(dirname(path), { recursive: true });
+	writeFileSync(path, content);
+}
+
+function seedChange(cwd: string, change = "add-auth"): string {
+	const root = join(cwd, "openspec", "changes", change);
+	write(join(root, "proposal.md"), "# Proposal\n");
+	write(join(root, "specs", "auth", "spec.md"), "# Auth Spec\n");
+	write(join(root, "design.md"), "# Design\n");
+	write(
+		join(root, "tasks.md"),
+		`# Tasks
+
+- [x] 1.1 Build foundation
+- [ ] 1.2 Wire routes
+`,
+	);
+	return root;
+}
+
+test("listActiveOpenSpecChanges excludes archive and sorts active changes", async () => {
+	const cwd = await workspace();
+	mkdirSync(join(cwd, "openspec", "changes", "b-change"), { recursive: true });
+	mkdirSync(join(cwd, "openspec", "changes", "a-change"), { recursive: true });
+	mkdirSync(join(cwd, "openspec", "changes", "archive", "2026-01-01-old"), { recursive: true });
+
+	assert.deepEqual(listActiveOpenSpecChanges(cwd), ["a-change", "b-change"]);
+});
+
+test("resolveSddStatus blocks when there are no active changes", async () => {
+	const cwd = await workspace();
+	mkdirSync(join(cwd, "openspec", "changes"), { recursive: true });
+
+	const status = resolveSddStatus({ cwd });
+
+	assert.equal(status.changeName, null);
+	assert.match(status.blockedReasons[0], /No active SDD changes/);
+	assert.equal(status.dependencies.apply, "blocked");
+});
+
+test("resolveSddStatus blocks when change selection is ambiguous", async () => {
+	const cwd = await workspace();
+	mkdirSync(join(cwd, "openspec", "changes", "first"), { recursive: true });
+	mkdirSync(join(cwd, "openspec", "changes", "second"), { recursive: true });
+
+	const status = resolveSddStatus({ cwd });
+
+	assert.equal(status.changeName, null);
+	assert.match(status.blockedReasons[0], /ambiguous/);
+});
+
+test("resolveSddStatus selects the only active change and counts task progress", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+
+	const status = resolveSddStatus({ cwd, includeInstructions: true });
+
+	assert.equal(status.changeName, "add-auth");
+	assert.equal(status.changeRoot, root);
+	assert.equal(status.artifacts.proposal, "done");
+	assert.equal(status.artifacts.specs, "done");
+	assert.deepEqual(status.taskProgress, {
+		total: 2,
+		complete: 1,
+		remaining: 1,
+		unchecked: ["- [ ] 1.2 Wire routes"],
+	});
+	assert.equal(status.applyState, "ready");
+	assert.equal(status.dependencies.apply, "ready");
+	assert.match(status.instructions?.apply.join("\n") ?? "", /persisted task checkboxes/);
+});
+
+test("resolveSddStatus marks apply all_done and verify ready when tasks are checked", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Build foundation\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.applyState, "all_done");
+	assert.equal(status.dependencies.apply, "all_done");
+	assert.equal(status.dependencies.verify, "ready");
+});
+
+test("resolveSddStatus blocks sync when verify report is not clearly passing", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "apply-progress.md"), "# Apply\n\nSome work completed.\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nTODO: tests not run yet\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.dependencies.verify, "ready");
+	assert.equal(status.dependencies.sync, "blocked");
+	assert.equal(status.dependencies.archive, "blocked");
+});
+
+test("resolveSddStatus rejects negated pass and sync-complete phrases", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nStatus: not passed\n");
+	write(join(root, "sync-report.md"), "# Sync\n\nSync complete: no\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.dependencies.verify, "ready");
+	assert.equal(status.dependencies.sync, "blocked");
+	assert.equal(status.dependencies.archive, "blocked");
+	assert.notEqual(status.nextRecommended, "sdd-archive");
+});
+
+test("resolveSddStatus blocks sync when verify report contains critical text", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "verify-report.md"), "# Verify\n\nCRITICAL: missing tests\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.dependencies.sync, "blocked");
+	assert.equal(status.dependencies.archive, "blocked");
+});
+
+test("resolveSddStatus reports same-domain collisions", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd, "current");
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nPASS\n");
+	write(join(cwd, "openspec", "changes", "other", "specs", "auth", "spec.md"), "# Other\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "current" });
+
+	assert.deepEqual(status.collisions.map((collision) => collision.domain), ["auth"]);
+	assert.equal(status.collisions[0].changes[0].change, "other");
+	assert.equal(status.dependencies.sync, "blocked");
+});
+
+test("resolveSddStatus blocks apply when tasks has no checkboxes", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "tasks.md"), "# Tasks\n\nImplementation notes only.\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.taskProgress.total, 0);
+	assert.equal(status.applyState, "blocked");
+	assert.equal(status.dependencies.apply, "blocked");
+	assert.match(status.blockedReasons.join("\n"), /no implementation task checkboxes/);
+});
+
+test("resolveSddStatus marks legacy flat specs partial and blocks sync", async () => {
+	const cwd = await workspace();
+	write(join(cwd, "openspec", "changes", "legacy", "proposal.md"), "# Proposal\n");
+	write(join(cwd, "openspec", "changes", "legacy", "spec.md"), "# Flat\n");
+	write(join(cwd, "openspec", "changes", "legacy", "design.md"), "# Design\n");
+	write(join(cwd, "openspec", "changes", "legacy", "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(cwd, "openspec", "changes", "legacy", "verify-report.md"), "# Verify\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "legacy" });
+
+	assert.equal(status.artifacts.specs, "partial");
+	assert.match(status.blockedReasons.join("\n"), /Legacy flat spec/);
+	assert.equal(status.dependencies.sync, "blocked");
+});
+
+test("resolveSddStatus accepts nested domain specs even when a legacy flat spec also exists", async () => {
+	const cwd = await workspace();
+	write(join(cwd, "openspec", "changes", "mixed", "proposal.md"), "# Proposal\n");
+	write(join(cwd, "openspec", "changes", "mixed", "spec.md"), "# Flat\n");
+	write(join(cwd, "openspec", "changes", "mixed", "specs", "parent", "child", "spec.md"), "# Nested\n");
+	write(join(cwd, "openspec", "changes", "mixed", "design.md"), "# Design\n");
+	write(join(cwd, "openspec", "changes", "mixed", "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "mixed" });
+
+	assert.equal(status.artifacts.specs, "done");
+	assert.equal(status.legacyFlatSpec?.hasDomainSpecs, true);
+	assert.doesNotMatch(status.blockedReasons.join("\n"), /Legacy flat spec/);
+});
+
+test("resolveSddStatus blocks sync when core artifacts are missing even with clean verify", async () => {
+	const cwd = await workspace();
+	write(join(cwd, "openspec", "changes", "thin", "proposal.md"), "# Proposal\n");
+	write(join(cwd, "openspec", "changes", "thin", "design.md"), "# Design\n");
+	write(join(cwd, "openspec", "changes", "thin", "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(cwd, "openspec", "changes", "thin", "verify-report.md"), "# Verify\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "thin" });
+
+	assert.match(status.blockedReasons.join("\n"), /domain specs are missing or partial/);
+	assert.equal(status.dependencies.sync, "blocked");
+	assert.notEqual(status.nextRecommended, "sdd-sync");
+});
+
+test("resolveSddStatus blocks stale sync report when current verify is not passing", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nStatus: not passed\n");
+	write(join(root, "sync-report.md"), "# Sync\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.dependencies.verify, "ready");
+	assert.equal(status.dependencies.sync, "blocked");
+	assert.equal(status.dependencies.archive, "blocked");
+	assert.notEqual(status.nextRecommended, "sdd-archive");
+});
+
+test("resolveSddStatus blocks archive when required artifacts are missing", async () => {
+	const cwd = await workspace();
+	write(join(cwd, "openspec", "changes", "thin", "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(cwd, "openspec", "changes", "thin", "verify-report.md"), "# Verify\n\nPASS\n");
+	write(join(cwd, "openspec", "changes", "thin", "sync-report.md"), "# Sync\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "thin" });
+
+	assert.match(status.blockedReasons.join("\n"), /proposal\.md is missing/);
+	assert.equal(status.dependencies.archive, "blocked");
+	assert.notEqual(status.nextRecommended, "sdd-archive");
+});
+
+test("resolveSddStatus reports partial core artifacts as blockers", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "proposal.md"), "");
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nPASS\n");
+	write(join(root, "sync-report.md"), "# Sync\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.artifacts.proposal, "partial");
+	assert.match(status.blockedReasons.join("\n"), /proposal\.md is empty or partial/);
+	assert.equal(status.dependencies.archive, "blocked");
+});
+
+test("resolveSddStatus marks archive ready only after clean verify, sync, and complete tasks", async () => {
+	const cwd = await workspace();
+	const root = seedChange(cwd);
+	write(join(root, "tasks.md"), "# Tasks\n\n- [x] 1.1 Done\n");
+	write(join(root, "verify-report.md"), "# Verify\n\nPASS\n");
+	write(join(root, "sync-report.md"), "# Sync\n\nPASS\n");
+
+	const status = resolveSddStatus({ cwd, changeName: "add-auth" });
+
+	assert.equal(status.dependencies.archive, "ready");
+	assert.equal(status.nextRecommended, "sdd-archive");
+	assert.match(renderPhaseInstructions(status).archive.join("\n"), /CRITICAL verification issues have no override/);
+});
+
+test("renderSddStatusMarkdown includes structured JSON", async () => {
+	const cwd = await workspace();
+	seedChange(cwd);
+
+	const markdown = renderSddStatusMarkdown(resolveSddStatus({ cwd }));
+
+	assert.match(markdown, /## SDD Status: add-auth/);
+	assert.match(markdown, /```json/);
+	assert.match(markdown, /"schemaName": "gentle-pi.sdd-status"/);
+});
+
+test("parseSddStatusCommandArgs extracts change and json flag", () => {
+	assert.deepEqual(parseSddStatusCommandArgs("add-auth --json"), {
+		changeName: "add-auth",
+		json: true,
+	});
+	assert.deepEqual(parseSddStatusCommandArgs("--json"), {
+		changeName: undefined,
+		json: true,
+	});
+});