@oh-my-pi/pi-coding-agent 4.3.0 → 4.3.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +11 -0
- package/package.json +5 -5
- package/src/cli/update-cli.ts +2 -2
- package/src/config.ts +5 -5
- package/src/core/auth-storage.ts +6 -1
- package/src/core/custom-commands/loader.ts +3 -1
- package/src/core/custom-tools/loader.ts +1 -18
- package/src/core/extensions/loader.ts +5 -21
- package/src/core/hooks/loader.ts +1 -18
- package/src/core/keybindings.ts +3 -1
- package/src/core/logger.ts +1 -2
- package/src/core/prompt-templates.ts +5 -4
- package/src/core/sdk.ts +5 -3
- package/src/core/skills.ts +5 -4
- package/src/core/tools/exa/mcp-client.ts +2 -2
- package/src/core/tools/task/agents.ts +5 -64
- package/src/core/tools/task/commands.ts +7 -33
- package/src/core/tools/task/discovery.ts +4 -66
- package/src/core/tools/task/executor.ts +32 -3
- package/src/core/tools/task/index.ts +11 -2
- package/src/core/tools/task/render.ts +25 -15
- package/src/core/tools/task/types.ts +3 -0
- package/src/core/tools/task/worker-protocol.ts +2 -1
- package/src/core/tools/task/worker.ts +2 -1
- package/src/core/tools/web-scrapers/huggingface.ts +1 -1
- package/src/core/tools/web-scrapers/readthedocs.ts +1 -1
- package/src/core/tools/web-scrapers/types.ts +1 -1
- package/src/core/tools/web-search/auth.ts +5 -3
- package/src/discovery/codex.ts +3 -1
- package/src/discovery/helpers.ts +124 -3
- package/src/migrations.ts +11 -9
- package/src/modes/interactive/components/extensions/state-manager.ts +19 -18
- package/src/prompts/agents/frontmatter.md +1 -0
- package/src/prompts/agents/reviewer.md +32 -4
- package/src/prompts/tools/task.md +3 -1
|
@@ -4,6 +4,33 @@ description: Code review specialist for quality and security analysis
|
|
|
4
4
|
tools: read, grep, find, ls, bash, report_finding
|
|
5
5
|
spawns: explore, task
|
|
6
6
|
model: pi/slow, gpt-5.2-codex, gpt-5.2, codex, gpt
|
|
7
|
+
output:
|
|
8
|
+
properties:
|
|
9
|
+
overall_correctness:
|
|
10
|
+
enum: [correct, incorrect]
|
|
11
|
+
explanation:
|
|
12
|
+
type: string
|
|
13
|
+
confidence:
|
|
14
|
+
type: number
|
|
15
|
+
optionalProperties:
|
|
16
|
+
findings:
|
|
17
|
+
elements:
|
|
18
|
+
properties:
|
|
19
|
+
title:
|
|
20
|
+
type: string
|
|
21
|
+
body:
|
|
22
|
+
type: string
|
|
23
|
+
priority:
|
|
24
|
+
type: number
|
|
25
|
+
confidence:
|
|
26
|
+
type: number
|
|
27
|
+
file_path:
|
|
28
|
+
type: string
|
|
29
|
+
line_start:
|
|
30
|
+
type: number
|
|
31
|
+
line_end:
|
|
32
|
+
type: number
|
|
33
|
+
required: [overall_correctness, explanation, confidence]
|
|
7
34
|
---
|
|
8
35
|
|
|
9
36
|
You are a senior engineer reviewing a proposed code change. Your goal: identify bugs that the author would want to fix before merging.
|
|
@@ -64,11 +91,12 @@ Each `report_finding` requires:
|
|
|
64
91
|
- `file_path`: Absolute path
|
|
65
92
|
- `line_start`, `line_end`: Range ≤10 lines, must overlap the diff
|
|
66
93
|
|
|
67
|
-
Final `complete` call:
|
|
94
|
+
Final `complete` call (payload goes under `data`):
|
|
68
95
|
|
|
69
|
-
- `overall_correctness`: "correct" (no bugs/blockers) or "incorrect"
|
|
70
|
-
- `explanation`: 1-3 sentences
|
|
71
|
-
- `confidence`: 0.0-1.0
|
|
96
|
+
- `data.overall_correctness`: "correct" (no bugs/blockers) or "incorrect"
|
|
97
|
+
- `data.explanation`: Plain text, 1-3 sentences summarizing your verdict. Do NOT include JSON, do NOT repeat findings here (they're already captured via `report_finding`).
|
|
98
|
+
- `data.confidence`: 0.0-1.0
|
|
99
|
+
- `data.findings`: Optional; MUST omit (it is populated from `report_finding` calls)
|
|
72
100
|
|
|
73
101
|
Correctness judgment ignores non-blocking issues (style, docs, nits).
|
|
74
102
|
|
|
@@ -12,12 +12,14 @@ If you discussed requirements, plans, schemas, or decisions with the user, you M
|
|
|
12
12
|
## Available Agents
|
|
13
13
|
|
|
14
14
|
{{#list agents prefix="- " join="\n"}}
|
|
15
|
-
{{name}}: {{description}} (Tools: {{default (join tools ", ") "All tools"}})
|
|
15
|
+
{{name}}: {{description}} (Tools: {{default (join tools ", ") "All tools"}}{{#if output}}, Output: structured{{/if}})
|
|
16
16
|
{{/list}}
|
|
17
17
|
{{#if moreAgents}}
|
|
18
18
|
...and {{moreAgents}} more agents
|
|
19
19
|
{{/if}}
|
|
20
20
|
|
|
21
|
+
Agents with "Output: structured" have a fixed schema enforced via frontmatter; your `output` parameter will be ignored for these agents.
|
|
22
|
+
|
|
21
23
|
## When NOT to Use
|
|
22
24
|
|
|
23
25
|
- Reading a specific file path → Use Read tool instead
|