tdd-enforcer 0.2.9 → 0.3.1

package/adapters/pi/index.test.ts

@@ -430,10 +430,8 @@ describe("handleBeforeAgentStart", () => {
 		};
 	}
 
-	function makeEvent(): {
-		systemPromptOptions: { promptGuidelines: string[] };
-	} {
-		return { systemPromptOptions: { promptGuidelines: [] } };
+	function _makeEvent(): { systemPrompt: string } {
+		return { systemPrompt: "" };
 	}
 
 	beforeEach(() => {
@@ -441,7 +439,7 @@ describe("handleBeforeAgentStart", () => {
 		mockLoadTddState = vi.fn();
 	});
 
-	it("pushes guidelines when TDD enabled", async () => {
+	it("returns instructions when TDD enabled", async () => {
 		mockLoadTddState.mockReturnValue({
 			ok: true,
 			state: { enabled: true, current: "red" },
@@ -452,39 +450,32 @@ describe("handleBeforeAgentStart", () => {
 				timeoutSeconds: 30,
 			},
 		});
-		const event = makeEvent();
-		await handleBeforeAgentStart(
-			event as any,
+		const result = await handleBeforeAgentStart(
+			{ systemPrompt: "" } as any,
 			{ cwd: "/test" } as any,
 			makeDeps(),
 		);
-		expect(event.systemPromptOptions.promptGuidelines).toHaveLength(3);
-		expect(event.systemPromptOptions.promptGuidelines[0]).toContain(
-			"locked files will be blocked",
-		);
-		expect(event.systemPromptOptions.promptGuidelines[1]).toContain(
-			"next_tdd_phase",
-		);
-		expect(event.systemPromptOptions.promptGuidelines[2]).toContain(
-			"cycle so reverting is cheap",
-		);
+		expect(result).toBeDefined();
+		expect(result?.systemPrompt).toContain("TDD enforcement");
+		expect(result?.systemPrompt).toContain("locked files will be blocked");
+		expect(result?.systemPrompt).toContain("cycle so reverting is cheap");
+		expect(result?.systemPrompt).not.toContain("next_tdd_phase");
 	});
 
-	it("does not push guidelines when TDD not setup", async () => {
+	it("returns undefined when TDD not setup", async () => {
 		mockLoadTddState.mockReturnValue({
 			ok: false,
 			reason: "Missing .pi/tdd/",
 		});
-		const event = makeEvent();
-		await handleBeforeAgentStart(
-			event as any,
+		const result = await handleBeforeAgentStart(
+			{ systemPrompt: "" } as any,
 			{ cwd: "/test" } as any,
 			makeDeps(),
 		);
-		expect(event.systemPromptOptions.promptGuidelines).toHaveLength(0);
+		expect(result).toBeUndefined();
 	});
 
-	it("does not push guidelines when TDD disabled", async () => {
+	it("returns disabled message when TDD was disabled", async () => {
 		mockLoadTddState.mockReturnValue({
 			ok: true,
 			state: { enabled: false, current: "red" },
@@ -495,12 +486,12 @@ describe("handleBeforeAgentStart", () => {
 				timeoutSeconds: 30,
 			},
 		});
-		const event = makeEvent();
-		await handleBeforeAgentStart(
-			event as any,
+		const result = await handleBeforeAgentStart(
+			{ systemPrompt: "" } as any,
 			{ cwd: "/test" } as any,
 			makeDeps(),
 		);
-		expect(event.systemPromptOptions.promptGuidelines).toHaveLength(0);
+		expect(result).toBeDefined();
+		expect(result?.systemPrompt).toContain("was disabled");
 	});
 });

package/adapters/pi/index.ts

@@ -149,22 +149,28 @@ export async function handleTddStatus(
 }
 
 export async function handleBeforeAgentStart(
-	event: {
-		systemPromptOptions: { promptGuidelines: string[] };
-	},
+	event: { systemPrompt: string },
 	ctx: { cwd: string },
 	deps: {
 		loadTddState: typeof loadTddState;
 	},
-): Promise<void> {
+): Promise<{ systemPrompt: string } | undefined> {
 	const tdd = deps.loadTddState(ctx.cwd);
-	if (!tdd.ok || !tdd.state.enabled) return;
-
-	event.systemPromptOptions.promptGuidelines.push(
-		"You are working under TDD enforcement. Each phase restricts which files you can modify — locked files will be blocked automatically.",
-		"Use `next_tdd_phase` to advance through the cycle, `previous_tdd_phase` to revert a phase, `tdd_status` to check current phase and blocked file rules.",
-		"Minimise the scope of each TDD cycle so reverting is cheap.",
-	);
+	if (!tdd.ok) return;
+
+	if (tdd.state.enabled) {
+		return {
+			systemPrompt:
+				event.systemPrompt +
+				"\n\nYou are working under TDD enforcement. Each phase restricts which files you can modify — locked files will be blocked automatically.\n" +
+				"Minimise the scope of each TDD cycle so reverting is cheap.",
+		};
+	}
+	return {
+		systemPrompt:
+			event.systemPrompt +
+			"\n\nTDD enforcement was disabled. File restrictions are no longer enforced.",
+	};
 }
 
 export async function handleTddJump(

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "tdd-enforcer",
-	"version": "0.2.9",
+	"version": "0.3.1",
 	"type": "module",
 	"scripts": {
 		"fmt": "biome format --write .",