pi-improve-interation 0.0.1 → 0.0.2

package/README.en.md

@@ -4,7 +4,7 @@ English | [中文](./README.md)
 
 ## Summary
 
-A companion pi package that layers enhancements on top of interactions you already use — pi-native and upstream pi plugins alike. One idea only: keep the interactions you know, make them stronger.
+A companion pi package that layers enhancements on top of interactions you already use, whether pi-native or from upstream pi plugins. The interactions stay the same; they just get stronger.
 
 - No fork of upstream: public extension APIs only, so an upstream upgrade costs at most one adapter change
 - No parallel concepts: it enhances the `@` you already type and the work you already delegate

package/README.md

@@ -4,7 +4,7 @@
 
 ## Summary
 
-给 pi 原生与上游 pi 插件**已有交互**叠加增强的伴生包。核心只有一件事：让你继续用熟悉的交互，但更强。
+给 pi 原生与上游 pi 插件已有交互叠加增强的伴生包。交互不变，能力更强。
 
 - 不 fork 上游：只走宿主公开扩展 API，上游升级最多改一处 adapter
 - 不引入平行概念：增强的就是你已经在敲的 `@`、已经在派的活

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-improve-interation",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Companion enhancement layer over pi-subagents (and future open-source pi packages) — adapter framework for orchestration add-ons, no fork required",
   "type": "module",
   "license": "MIT",

package/src/commands/delegation.ts

@@ -1,15 +1,16 @@
 /**
  * @agent:<name> mention → delegation instruction for the main session.
  *
- * Design (the main agent stays in charge of the actual child prompt):
+ * Design (the main agent is the dispatcher, not the prompt author):
  * 1. the mention is routing only — "@agent:reviewer" names the target agent;
  *    the remaining message text is the task
- * 2. the transform makes the convention explicit: the main agent receives a
- *    "delegate via the subagent tool" instruction, not a bare @token to guess
- * 3. the main agent decides the exact child prompt (carrying context), runs
- *    the agent through pi-subagents' `subagent` tool, and integrates results
- * 4. when no task text accompanies the mention, the main agent picks the most
- *    useful task itself and asks before delegating when scope is unclear
+ * 2. the transform keeps the main agent's effort low: it must quickly grasp
+ *    the user's intent, then only organize the work and optimize the handoff
+ *    prompt — not recreate the handoff from scratch
+ * 3. the main agent dispatches immediately and integrates the child result;
+ *    it does not re-derive context or pause to ask on a clear task
+ * 4. when no task text accompanies the mention, the main agent infers the
+ *    most useful task from the conversation
  */
 export const AGENT_MENTION_TOKEN_PATTERN = /(?:^|\s)@agent:([\w.-]+)/;
 
@@ -18,15 +19,15 @@ export function buildDelegationInstruction(
 	task: string,
 ): string {
 	const trimmed = task.trim();
-	const taskSection =
+	const rawMessage =
 		trimmed.length > 0
-			? `Task: ${trimmed}`
-			: "Task: not specified — pick the most useful task for this agent given the current conversation.";
+			? trimmed
+			: "(no explicit task — infer the most useful one from this conversation)";
 	return [
-		`Delegate to the ${agentName} subagent via the subagent tool.`,
+		"[Must] The user wants to use a subagent. Quickly grasp the intent of the user's raw message below, then focus solely on how to organize the work of the specified agent(s) and how to optimize the prompt you hand off. Then dispatch the subagent.",
 		"",
-		taskSection,
+		`Select AGENTS: ${agentName}`,
 		"",
-		"You own the handoff: decide the exact child prompt yourself (carry over relevant context from this conversation), run the agent, and integrate its result here. If the task is ambiguous, ask me before delegating.",
+		`User Request: \n${rawMessage}`,
 	].join("\n");
 }

package/test/agent-palette.test.ts

@@ -65,16 +65,14 @@ test("mergeAgentEntries: later groups override earlier by name", () => {
 
 test("buildDelegationInstruction: with task and without task", () => {
 	const withTask = buildDelegationInstruction("reviewer", "  review my diff  ");
-	assert.match(
-		withTask,
-		/^Delegate to the reviewer subagent via the subagent tool\./,
-	);
-	assert.match(withTask, /Task: review my diff/);
-	assert.match(withTask, /decide the exact child prompt yourself/);
+	assert.match(withTask, /^\[Must\]/);
+	assert.match(withTask, /Select AGENTS: reviewer/);
+	assert.match(withTask, /User Request: \nreview my diff/);
 
 	const withoutTask = buildDelegationInstruction("scout", "");
-	assert.match(withoutTask, /Task: not specified/);
-	assert.match(withoutTask, /ask me before delegating/);
+	assert.match(withoutTask, /Select AGENTS: scout/);
+	assert.match(withoutTask, /User Request:/);
+	assert.match(withoutTask, /no explicit task/);
 });
 
 function makeCurrent() {
@@ -192,11 +190,9 @@ test("input handler transforms @agent: mentions into delegation instructions", (
 		text: string;
 	};
 	assert.equal(transformed.action, "transform");
-	assert.match(
-		transformed.text,
-		/^Delegate to the reviewer subagent via the subagent tool\./,
-	);
-	assert.match(transformed.text, /Task: review my diff/);
+	assert.match(transformed.text, /^\[Must\]/);
+	assert.match(transformed.text, /Select AGENTS: reviewer/);
+	assert.match(transformed.text, /User Request: \nreview my diff/);
 	assert.doesNotMatch(transformed.text, /@agent:/);
 
 	const passthrough = handler({ text: "hello world" }) as { action: string };