@raidou/pi-pm-subagents 0.1.2 → 0.1.4

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Weirong Xu <weirongxu.raidou@gmail.com>
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the "Software"),
+to deal in the Software without restriction, including without limitation
+the rights to use, copy, modify, merge, publish, distribute, sublicense,
+and/or sell copies of the Software, and to permit persons to whom the
+Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included
+in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
+DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
+OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raidou/pi-pm-subagents",
-  "version": "0.1.2",
+  "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,12 +12,23 @@
     "coordinator"
   ],
   "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/weirongxu/pi-pm-subagents.git"
+  },
   "type": "module",
   "pi": {
     "extensions": [
       "./src/index.ts"
     ]
   },
+  "files": [
+    "src",
+    "agents",
+    "README.zh-CN.md",
+    "example-prompts",
+    "pm-subagents-prompts"
+  ],
   "scripts": {
     "test": "pnpm test:types && pnpm test:lint && pnpm test:unit",
     "test:types": "tsc --noEmit",

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)
                         },
                       },
                       {

package/.prettierrc

@@ -1,7 +0,0 @@
-{
-  "singleQuote": true,
-  "trailingComma": "all",
-  "arrowParens": "always",
-  "semi": false,
-  "printWidth": 80
-}

package/AGENTS.md

@@ -1 +0,0 @@
-- use `pnpm test` to check and fix after edit, always test types, lint, and unit (if have)

package/eslint.config.mjs

@@ -1,14 +0,0 @@
-// @ts-check
-import { tsconfig } from '@raidou/eslint-config-base'
-import { defineConfig } from 'eslint/config'
-export default defineConfig([
-  {
-    files: [
-      'src/**/*.{mjs,ts,tsx,js,jsx}',
-      'app/*.{mjs,ts,tsx,js,jsx}',
-      'app/src/**/*.{mjs,ts,tsx,js,jsx}',
-      '*.{mjs,ts,tsx,js,jsx}',
-    ],
-    extends: [tsconfig],
-  },
-])

package/pnpm-workspace.yaml

@@ -1,5 +0,0 @@
-allowBuilds:
-  '@google/genai': false
-  protobufjs: false
-minimumReleaseAgeExclude:
-  - typebox@1.3.9

package/tsconfig.json

@@ -1,24 +0,0 @@
-{
-  "compilerOptions": {
-    "outDir": "dist",
-    "jsx": "react-jsx",
-    "declaration": false,
-    "allowJs": true,
-    "lib": ["DOM", "ES2022"],
-    "target": "ES2022",
-    "module": "nodenext",
-    "stripInternal": true,
-    "resolveJsonModule": true,
-    "moduleResolution": "nodenext",
-    "esModuleInterop": true,
-    "sourceMap": true,
-    "strict": true,
-    "allowImportingTsExtensions": true,
-    "rewriteRelativeImportExtensions": true,
-    "forceConsistentCasingInFileNames": true,
-    "noUncheckedIndexedAccess": true,
-    "verbatimModuleSyntax": true,
-    "skipLibCheck": true
-  },
-  "include": ["src", "*.ts", "*.tsx", "*.js", "*.jsx", "*.mjs"]
-}

package/vitest.config.ts

@@ -1,8 +0,0 @@
-import { defineConfig } from 'vitest/config'
-
-export default defineConfig({
-  test: {
-    globals: false,
-    include: ['src/**/*.test.ts'],
-  },
-})