gentle-pi 0.6.0 → 0.6.1

package/assets/agents/jd-fix-agent.md

@@ -0,0 +1,16 @@
+---
+name: jd-fix-agent
+description: Judgment Day surgical fix agent for confirmed findings. Can edit code and run focused tests.
+tools: read, grep, glob, edit, write, bash
+---
+
+You are the Judgment Day fix agent for Gentle AI.
+
+Apply surgical fixes for confirmed Judgment Day findings only. Preserve the original design intent, keep the patch focused, and avoid unrelated refactors.
+
+Rules:
+
+- Edit only the files needed to resolve confirmed findings.
+- Add or update focused tests when the fix changes behavior.
+- Run the relevant tests when practical and report exact results.
+- Clearly list what was fixed, what was verified, and any remaining risks.

package/assets/agents/jd-judge-a.md

@@ -0,0 +1,16 @@
+---
+name: jd-judge-a
+description: Judgment Day blind adversarial reviewer A. Read-only; reports findings and does not fix code.
+tools: read, grep, glob, bash
+---
+
+You are Judgment Day judge A for Gentle AI.
+
+Run an independent, blind adversarial review of the assigned change. Focus on correctness, regressions, missing tests, unsafe behavior, and mismatches with the user's request.
+
+Rules:
+
+- Stay read-only. Do not edit files or apply fixes.
+- Do not coordinate with judge B before producing your review.
+- Report concrete findings with file paths, evidence, severity, and suggested verification.
+- If you find no confirmed issues, say so clearly.

package/assets/agents/jd-judge-b.md

@@ -0,0 +1,16 @@
+---
+name: jd-judge-b
+description: Judgment Day blind adversarial reviewer B. Read-only; independently reports findings and does not fix code.
+tools: read, grep, glob, bash
+---
+
+You are Judgment Day judge B for Gentle AI.
+
+Run an independent, blind adversarial review of the assigned change. Challenge assumptions from a different angle than judge A, with special attention to edge cases, test gaps, integration risks, and user-visible regressions.
+
+Rules:
+
+- Stay read-only. Do not edit files or apply fixes.
+- Work independently from judge A and do not rely on judge A's conclusions.
+- Report concrete findings with file paths, evidence, severity, and suggested verification.
+- If you find no confirmed issues, say so clearly.

package/extensions/gentle-ai.ts

@@ -438,7 +438,18 @@ const SDD_AGENT_NAMES = [
 ] as const;
 const SDD_AGENT_NAME_SET = new Set<string>(SDD_AGENT_NAMES);
 
-type SddAgentName = (typeof SDD_AGENT_NAMES)[number];
+const JUDGMENT_DAY_AGENT_NAMES = [
+	"jd-judge-a",
+	"jd-judge-b",
+	"jd-fix-agent",
+] as const;
+
+const CORE_MODEL_AGENT_NAMES = [
+	...SDD_AGENT_NAMES,
+	...JUDGMENT_DAY_AGENT_NAMES,
+] as const;
+const CORE_MODEL_AGENT_NAME_SET = new Set<string>(CORE_MODEL_AGENT_NAMES);
+
 type ThinkingLevel = "off" | "minimal" | "low" | "medium" | "high" | "xhigh";
 interface AgentRoutingEntry {
 	model?: string;
@@ -998,14 +1009,7 @@ function listDiscoverableAgents(cwd: string): AgentEntry[] {
 	];
 	const byName = new Map<string, AgentEntry>();
 	for (const agent of agents) byName.set(agent.name, agent);
-	const discovered = Array.from(byName.values());
-	const sddFirst = SDD_AGENT_NAMES.map((name) =>
-		discovered.find((agent) => agent.name === name),
-	).filter((agent): agent is AgentEntry => agent !== undefined);
-	const rest = discovered
-		.filter((agent) => !SDD_AGENT_NAMES.includes(agent.name as SddAgentName))
-		.sort((left, right) => left.name.localeCompare(right.name));
-	return [...sddFirst, ...rest];
+	return orderDiscoverableAgents(Array.from(byName.values()));
 }
 
 async function listDiscoverableAgentsAsync(cwd: string): Promise<AgentEntry[]> {
@@ -1030,14 +1034,17 @@ async function listDiscoverableAgentsAsync(cwd: string): Promise<AgentEntry[]> {
 	}
 	const byName = new Map<string, AgentEntry>();
 	for (const agent of agents) byName.set(agent.name, agent);
-	const discovered = Array.from(byName.values());
-	const sddFirst = SDD_AGENT_NAMES.map((name) =>
-		discovered.find((agent) => agent.name === name),
+	return orderDiscoverableAgents(Array.from(byName.values()));
+}
+
+function orderDiscoverableAgents(agents: AgentEntry[]): AgentEntry[] {
+	const coreFirst = CORE_MODEL_AGENT_NAMES.map((name) =>
+		agents.find((agent) => agent.name === name),
 	).filter((agent): agent is AgentEntry => agent !== undefined);
-	const rest = discovered
-		.filter((agent) => !SDD_AGENT_NAMES.includes(agent.name as SddAgentName))
+	const rest = agents
+		.filter((agent) => !CORE_MODEL_AGENT_NAME_SET.has(agent.name))
 		.sort((left, right) => left.name.localeCompare(right.name));
-	return [...sddFirst, ...rest];
+	return [...coreFirst, ...rest];
 }
 
 function projectSettingsPath(cwd: string): string {
@@ -1904,6 +1911,8 @@ async function applyReviewGate(
 export const __testing = {
 	listAgentsFromDir,
 	listAgentsFromDirAsync,
+	listDiscoverableAgents,
+	orderDiscoverableAgents,
 	classifyGuardedCommand,
 	loadRuntimeGuardrailsConfig,
 	buildGentlePrompt,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gentle-pi",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "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/gentle-ai.test.ts

@@ -34,3 +34,52 @@ test("agent discovery skips skills directories", async (t) => {
 		["reviewer", "worker"],
 	);
 });
+
+test("agent model discovery prioritizes SDD and Judgment Day agents", (t) => {
+	const root = mkdtempSync(join(tmpdir(), "gentle-pi-model-agents-"));
+	t.after(() => rmSync(root, { recursive: true, force: true }));
+	writeMarkdown(join(root, "zeta.md"), "name: zeta\n");
+	writeMarkdown(join(root, "jd-fix-agent.md"), "name: jd-fix-agent\n");
+	writeMarkdown(join(root, "sdd-apply.md"), "name: sdd-apply\n");
+	writeMarkdown(join(root, "alpha.md"), "name: alpha\n");
+	writeMarkdown(join(root, "jd-judge-b.md"), "name: jd-judge-b\n");
+	writeMarkdown(join(root, "sdd-init.md"), "name: sdd-init\n");
+	writeMarkdown(join(root, "jd-judge-a.md"), "name: jd-judge-a\n");
+
+	const discovered = __testing.listAgentsFromDir(root, "user");
+	const ordered = __testing.orderDiscoverableAgents(discovered);
+
+	assert.deepEqual(
+		ordered.map((agent) => agent.name),
+		[
+			"sdd-init",
+			"sdd-apply",
+			"jd-judge-a",
+			"jd-judge-b",
+			"jd-fix-agent",
+			"alpha",
+			"zeta",
+		],
+	);
+});
+
+test("discoverable model agents include installed Judgment Day agents", (t) => {
+	const root = mkdtempSync(join(tmpdir(), "gentle-pi-installed-agents-"));
+	const previousHome = process.env.GENTLE_PI_AGENT_HOME;
+	process.env.GENTLE_PI_AGENT_HOME = root;
+	t.after(() => {
+		if (previousHome === undefined) delete process.env.GENTLE_PI_AGENT_HOME;
+		else process.env.GENTLE_PI_AGENT_HOME = previousHome;
+		rmSync(root, { recursive: true, force: true });
+	});
+	writeMarkdown(join(root, "agents", "jd-judge-a.md"), "name: jd-judge-a\n");
+	writeMarkdown(join(root, "agents", "jd-judge-b.md"), "name: jd-judge-b\n");
+	writeMarkdown(join(root, "agents", "jd-fix-agent.md"), "name: jd-fix-agent\n");
+
+	const discovered = __testing.listDiscoverableAgents(root).map((agent) => agent.name);
+
+	assert.deepEqual(
+		discovered.filter((name) => name.startsWith("jd-")),
+		["jd-judge-a", "jd-judge-b", "jd-fix-agent"],
+	);
+});