gentle-pi 0.3.2 → 0.3.3

package/extensions/gentle-ai.ts

@@ -130,6 +130,7 @@ const SDD_AGENT_NAMES = [
 	"sdd-verify",
 	"sdd-archive",
 ] as const;
+const SDD_AGENT_NAME_SET = new Set<string>(SDD_AGENT_NAMES);
 
 type SddAgentName = (typeof SDD_AGENT_NAMES)[number];
 type ThinkingLevel = "off" | "minimal" | "low" | "medium" | "high" | "xhigh";
@@ -170,6 +171,34 @@ const MODEL_CONTROL_OPTIONS = [
 	CUSTOM_MODEL,
 ] as const;
 
+function readStringPath(value: unknown, path: string[]): string | undefined {
+	let current = value;
+	for (const key of path) {
+		if (!isRecord(current)) return undefined;
+		current = current[key];
+	}
+	return typeof current === "string" ? current : undefined;
+}
+
+function isSddAgentStartEvent(event: unknown): boolean {
+	const candidates = [
+		readStringPath(event, ["agentName"]),
+		readStringPath(event, ["agent"]),
+		readStringPath(event, ["name"]),
+		readStringPath(event, ["agent", "name"]),
+		readStringPath(event, ["subagent", "name"]),
+	]
+		.filter((value): value is string => value !== undefined)
+		.map((value) => value.trim());
+	if (candidates.some((value) => SDD_AGENT_NAME_SET.has(value))) return true;
+
+	const systemPrompt = readStringPath(event, ["systemPrompt"]) ?? "";
+	return SDD_AGENT_NAMES.some((name) => {
+		const phase = name.replace(/^sdd-/, "");
+		return new RegExp(`\\bSDD ${phase} executor\\b`, "i").test(systemPrompt);
+	});
+}
+
 function evaluateDeniedCommand(
 	command: string,
 ): ToolCallEventResult | undefined {
@@ -1209,7 +1238,10 @@ export default function gentleAi(pi: ExtensionAPI): void {
 		return { action: "continue" };
 	});
 
-	pi.on("before_agent_start", (event, ctx) => {
+	pi.on("before_agent_start", async (event, ctx) => {
+		if (isSddAgentStartEvent(event) && !getSddPreflightPreferences(ctx)) {
+			await runSddPreflight(ctx);
+		}
 		const prefs = getSddPreflightPreferences(ctx);
 		const sddPrompt = prefs ? `\n\n${renderSddPreflightPrompt(prefs)}` : "";
 		return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gentle-pi",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "Turn Pi into el Gentleman: a senior-architect development harness with SDD/OpenSpec, subagents, strict TDD evidence, review guardrails, and skill discovery.",
   "license": "MIT",
   "type": "module",

package/tests/runtime-harness.mjs

@@ -162,9 +162,14 @@ async function run() {
 	const promptCwd = await tempWorkspace();
 	try {
 		const promptHook = hooks.get("before_agent_start")[0];
-		const promptResult = promptHook({ systemPrompt: "base" }, createCtx(promptCwd));
+		const promptResult = await promptHook({ systemPrompt: "base" }, createCtx(promptCwd));
 		assert.match(promptResult.systemPrompt, /base/);
 		assert.match(promptResult.systemPrompt, /el Gentleman/);
+		assert.equal(
+			existsSync(join(promptCwd, ".pi", "agents", "sdd-apply.md")),
+			false,
+			"normal agent startup must not run SDD preflight",
+		);
 	} finally {
 		await rm(promptCwd, { recursive: true, force: true });
 	}
@@ -297,7 +302,7 @@ async function run() {
 		await inputHook({ text: "/sdd-plan another change", source: "interactive" }, ctx);
 		assert.equal(ctx.ui.selections.length, 3, "preflight should run only once per session");
 		const promptHook = hooks.get("before_agent_start")[0];
-		const promptResult = promptHook({ systemPrompt: "base" }, ctx);
+		const promptResult = await promptHook({ systemPrompt: "base" }, ctx);
 		assert.match(promptResult.systemPrompt, /SDD Session Preflight/);
 		assert.match(promptResult.systemPrompt, /Execution mode: interactive/);
 	} finally {
@@ -317,6 +322,34 @@ async function run() {
 		await rm(commandSddCwd, { recursive: true, force: true });
 	}
 
+	const sddAgentGuardCwd = await tempWorkspace();
+	try {
+		const ctx = createCtx(sddAgentGuardCwd, true, "sdd-agent-guard-session");
+		const promptHook = hooks.get("before_agent_start")[0];
+		const promptResult = await promptHook(
+			{
+				systemPrompt: "You are the SDD proposal executor for Gentle AI.",
+			},
+			ctx,
+		);
+		assert.equal(existsSync(join(sddAgentGuardCwd, ".pi", "agents", "sdd-apply.md")), true);
+		assert.equal(existsSync(join(sddAgentGuardCwd, ".pi", "chains", "sdd-full.chain.md")), true);
+		assert.equal(ctx.ui.selections.length, 3);
+		assert.match(promptResult.systemPrompt, /SDD Session Preflight/);
+		assert.match(ctx.ui.notifications.at(-1).message, /SDD preflight complete/);
+
+		await promptHook(
+			{
+				agentName: "sdd-tasks",
+				systemPrompt: "You are the SDD tasks executor for Gentle AI.",
+			},
+			ctx,
+		);
+		assert.equal(ctx.ui.selections.length, 3, "SDD agent guard should reuse session choices");
+	} finally {
+		await rm(sddAgentGuardCwd, { recursive: true, force: true });
+	}
+
 	const invalidPreflightCwd = await tempWorkspace();
 	try {
 		await writeFile(globalModelsPath, "{ invalid json");