@raidou/pi-pm-subagents 0.1.3 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raidou/pi-pm-subagents",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Coordinator mode for pi: a read-only coordinator agent that drives isolated subagents, including a planner role.",
   "keywords": [
     "pi",
@@ -12,6 +12,10 @@
     "coordinator"
   ],
   "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/weirongxu/pi-pm-subagents.git"
+  },
   "type": "module",
   "pi": {
     "extensions": [

package/src/subagent/batcher.test.ts

@@ -145,6 +145,37 @@ describe('MessageBatcher', () => {
     expect(firstFlush?.[1]).toContain('Subagent activity update')
   })
 
+  it('formats plan items with reviewed-by-user tag', () => {
+    vi.useFakeTimers()
+    const flushed: string[][] = []
+    const batcher = new MessageBatcher((items) => flushed.push(items), 100)
+    const subagent = makeSubagent(1, 'plan-1', 'done')
+
+    batcher.add(subagent, 'plan', 'The implementation plan')
+    batcher.flushNow()
+
+    expect(flushed).toHaveLength(1)
+    const item = flushed[0]?.[0]
+    expect(item).toContain('<subagent-plan>')
+    expect(item).toContain('</subagent-plan>')
+    expect(item).toContain('<type>plan</type>')
+    expect(item).toContain('<reviewed-by-user>true</reviewed-by-user>')
+    expect(item).toContain('<message>The implementation plan</message>')
+  })
+
+  it('does not add reviewed-by-user tag to non-plan items', () => {
+    vi.useFakeTimers()
+    const flushed: string[][] = []
+    const batcher = new MessageBatcher((items) => flushed.push(items), 100)
+    const subagent = makeSubagent(1, 'task-1', 'done')
+
+    batcher.add(subagent, 'done', 'Task completed')
+    batcher.flushNow()
+
+    const item = flushed[0]?.[0]
+    expect(item).not.toContain('reviewed-by-user')
+  })
+
   it('resets timer on each add', () => {
     vi.useFakeTimers()
     const flushed: string[][] = []

package/src/subagent/batcher.ts

@@ -1,5 +1,11 @@
 import { formatSubagentSummary, type LiveSubagent } from './manager.js'
 
+const TAG_NAMES: Record<string, string> = {
+  done: 'subagent-done',
+  plan: 'subagent-plan',
+  notify: 'subagent-notify',
+}
+
 export class MessageBatcher {
   private items: string[] = []
   private timer: ReturnType<typeof setTimeout> | undefined
@@ -14,14 +20,15 @@ export class MessageBatcher {
   }
 
   add(subagent: LiveSubagent, type: string, message: string): void {
-    const tagName = type === 'done' ? 'subagent-done' : 'subagent-notify'
-    const item = [
-      `<${tagName}>`,
-      `<type>${type}</type>`,
+    const tagName = TAG_NAMES[type] ?? 'subagent-notify'
+    const lines = [`<${tagName}>`, `<type>${type}</type>`]
+    if (type === 'plan') lines.push('<reviewed-by-user>true</reviewed-by-user>')
+    lines.push(
       `<job>${formatSubagentSummary(subagent)}</job>`,
       `<message>${message}</message>`,
       `</${tagName}>`,
-    ].join('\n')
+    )
+    const item = lines.join('\n')
     this.items.push(item)
     if (this.timer !== undefined) clearTimeout(this.timer)
     this.timer = setTimeout(() => {

package/src/subagent/tools.ts

@@ -181,7 +181,7 @@ export function registerSubagentTools(
                         id: 'send-to-coordinator',
                         label: 'Send plan to coordinator',
                         action: () => {
-                          batcher.add(subagent, 'done', lastMessage)
+                          batcher.add(subagent, 'plan', lastMessage)
                         },
                       },
                       {