@zhushanwen/pi-goal 0.6.2-dev.0 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-goal",
-  "version": "0.6.2-dev.0",
+  "version": "0.6.3",
   "description": "Codex-style /goal command for Pi — persistent goal-driven autonomous loop with evidence-based completion, token/time budgets, blocked detection, and steering templates.",
   "type": "module",
   "main": "src/index.ts",
@@ -28,7 +28,7 @@
     "typebox": "*"
   },
   "dependencies": {
-    "@xyz-agent/extension-protocol": "^0.3.1-dev.0"
+    "@xyz-agent/extension-protocol": "^0.3.1"
   },
   "peerDependenciesMeta": {
     "@earendil-works/pi-tui": {

package/src/__tests__/goal-control-prompt.test.ts

@@ -61,6 +61,22 @@ describe("goal_control description — 含参数结构反例（Don't 段）", ()
 	});
 });
 
+describe("goal_control promptGuidelines — complete/report_blocked 主动触发引导（防不主动收尾）", () => {
+	// promptGuidelines 进 system prompt guidelines 段（强信号），是修复 complete/report_blocked
+	// 「该主动调但不调」的核心载体。用源码断言锁定两条正向触发引导，防止后续重构删掉。
+	it("源码含 promptGuidelines 字段", () => {
+		expect(ADAPTER_SRC).toMatch(/promptGuidelines:\s*\[/);
+	});
+
+	it("promptGuidelines 含 complete 主动触发信号（proactively + objective is actually achieved）", () => {
+		expect(ADAPTER_SRC).toMatch(/complete:.*proactively call.*objective is actually achieved/s);
+	});
+
+	it("promptGuidelines 含 report_blocked 主动触发信号（proactively + ≥3 approaches）", () => {
+		expect(ADAPTER_SRC).toMatch(/report_blocked:.*proactively call.*≥3 distinct alternative approaches/s);
+	});
+});
+
 describe("goal_control runtime 错误文案 — 含 'Correct:' 纠正正例（≥4 处）", () => {
 	it("源码含 ≥4 处 'Correct:' 纠错文案（4 条 required throw 各一带正例）", () => {
 		// objective/slug/evidence/reason 四条必填 throw 各应带完整 JSON 正例，

package/src/adapters/goal-control-adapter.ts

@@ -376,6 +376,18 @@ Don't:
 - complete when no goal is active — create or resume one first (fails with 'Goal mode not active').`,
 		promptSnippet:
 			"Use goal_control to manage the thread goal: create (with slug + objective, only when user asks) or complete (with evidence) or report_blocked (with reason, after trying alternatives).",
+		// promptGuidelines：进 system prompt guidelines 段（强信号位）。
+		// 聚焦 complete/report_blocked 的正向触发引导——这两个 action 是「该主动调但实际不调」
+		// 的核心症结（contextInjectionPrompt 规则 3/4 虽引导，但属 goal 活跃时才注入的 steering，
+		// 措辞被动；此处给 system prompt 层的常驻强信号）。
+		// create 不列：其劝退已在 description + promptSnippet 双重覆盖，此处第三重冗余反而稀释
+		// complete/report_blocked 的注意力权重（P14 约束衰减）。
+		promptGuidelines: [
+			// 全解耦下 todo 非硬前置——objective 实际达成才算（与 handleComplete「todo 由 AI 自判」一致）
+			"complete: proactively call when the active goal's objective is actually achieved, not merely in progress. Evidence must be concrete artifacts (files changed, tests green, commands run). Finishing all todos (incl. verification todos) is the usual readiness signal, but the real bar is the objective being met — you decide.",
+			// P3 数字阈值 ≥3，与 params.reason description 的 "at least 3 approaches" 双重冗余
+			"report_blocked: proactively call when genuinely blocked after ≥3 distinct alternative approaches — not for hard/slow work or uncertainty. State the blocker and what you tried. Do NOT silently stop or leave the goal hanging.",
+		],
 		executionMode: "sequential",
 		parameters: GoalControlParams,
 

package/src/projection/__tests__/prompts.test.ts

@@ -212,12 +212,11 @@ describe("contextInjectionPrompt", () => {
 		expect(out).toContain("test objective");
 	});
 
-	it("要求建 todo（含 verification todo，FR-6/#10）", () => {
+	it("要求建 todo（含 verification todo）", () => {
 		const state = makeState();
 		const out = contextInjectionPrompt(state, 0);
 		expect(out).toContain("todo tool");
-		expect(out).toContain("verification todos");
-		expect(out).toContain("isVerification intent");
+		expect(out).toContain("separate todo for verification checks");
 	});
 
 	it("无 goal_manager 引用（#1 清理后）", () => {

package/src/projection/prompts.ts

@@ -239,7 +239,7 @@ export function contextInjectionPrompt(
 		`Track work with todos:\n` +
 		`Before working, create todos for the task breakdown using the todo tool:\n` +
 		`- Each concrete step becomes a todo item\n` +
-		`- Include verification todos (e.g., 'run tests', 'typecheck') with isVerification intent\n` +
+		`- Add a separate todo for verification checks (e.g., 'run tests', 'typecheck')\n` +
 		`- Track progress by updating todo status as you complete items\n` +
 		planSection +
 		`\n` +