@zephyrdeng/pi-review 0.4.9 → 0.5.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/README.md +14 -4
- package/README.zh-CN.md +9 -3
- package/dist/args.d.ts +1 -0
- package/dist/args.d.ts.map +1 -1
- package/dist/args.js +9 -2
- package/dist/args.js.map +1 -1
- package/dist/cli.js +12 -1
- package/dist/cli.js.map +1 -1
- package/dist/install-args.d.ts +4 -0
- package/dist/install-args.d.ts.map +1 -0
- package/dist/install-args.js +47 -0
- package/dist/install-args.js.map +1 -0
- package/dist/install-args.test.d.ts +2 -0
- package/dist/install-args.test.d.ts.map +1 -0
- package/dist/install-args.test.js +21 -0
- package/dist/install-args.test.js.map +1 -0
- package/dist/install.d.ts +8 -0
- package/dist/install.d.ts.map +1 -0
- package/dist/install.js +46 -0
- package/dist/install.js.map +1 -0
- package/dist/types.d.ts +5 -1
- package/dist/types.d.ts.map +1 -1
- package/package.json +2 -2
- package/skills/pi-review/SKILL.md +28 -8
- package/skills/pi-review/references/codex-tools.md +34 -0
package/README.md
CHANGED
|
@@ -42,15 +42,23 @@ Dev dependencies use the [public npm registry](https://registry.npmjs.org/) (see
|
|
|
42
42
|
npm install -g @zephyrdeng/pi-review
|
|
43
43
|
```
|
|
44
44
|
|
|
45
|
-
### Pi
|
|
45
|
+
### One-shot (Pi + other agents)
|
|
46
46
|
|
|
47
47
|
```bash
|
|
48
|
-
|
|
48
|
+
npx @zephyrdeng/pi-review install
|
|
49
49
|
```
|
|
50
50
|
|
|
51
|
-
|
|
51
|
+
Runs `pi install npm:@zephyrdeng/pi-review` when the Pi CLI is on PATH, then installs the agent skill for Claude Code, Codex, and Cursor (via the [skills CLI](https://www.npmjs.com/package/skills), non-interactive `-y`). Forward extra flags to the skill step, e.g. `npx @zephyrdeng/pi-review install --agent claude-code codex -y` or `npx @zephyrdeng/pi-review install --agents-only --all`.
|
|
52
|
+
|
|
53
|
+
Use `--pi-only` or `--agents-only` to run one side. For Pi-only use, **do not** also run `install-skill` — the npm Pi package already exposes the skill via `pi.skills`.
|
|
54
|
+
|
|
55
|
+
### Pi package only
|
|
56
|
+
|
|
57
|
+
```bash
|
|
58
|
+
pi install npm:@zephyrdeng/pi-review
|
|
59
|
+
```
|
|
52
60
|
|
|
53
|
-
|
|
61
|
+
### Agent skill only (Claude Code, Codex, Cursor, ...)
|
|
54
62
|
|
|
55
63
|
```bash
|
|
56
64
|
npx @zephyrdeng/pi-review install-skill
|
|
@@ -146,6 +154,8 @@ pi-review --continue <sessionHandle> --mode challenge --model provider/model --
|
|
|
146
154
|
|
|
147
155
|
Agent hosts like Claude Code, Cursor, and Codex typically buffer a Bash tool's stdout until the command exits, so a multi-minute review can look like a silent wait followed by one large dump — even though `pi-review` streams by default. `--progress-log <path>` sidesteps this: it runs the child in `--mode json` and writes its event log to a file in real time, independent of what the calling tool does with stdout.
|
|
148
156
|
|
|
157
|
+
The bundled **pi-review** agent skill tells parent agents to use `--progress-log` by default on those hosts (Pi `/rv` stays on default streaming). Details: [`skills/pi-review/SKILL.md`](skills/pi-review/SKILL.md) and [`skills/pi-review/references/codex-tools.md`](skills/pi-review/references/codex-tools.md).
|
|
158
|
+
|
|
149
159
|
```bash
|
|
150
160
|
# Start the review in the background, writing structured progress events to a file
|
|
151
161
|
pi-review --progress-log /tmp/pi-review.jsonl -- @src/foo.ts &
|
package/README.zh-CN.md
CHANGED
|
@@ -25,12 +25,16 @@
|
|
|
25
25
|
|
|
26
26
|
```bash
|
|
27
27
|
npm install -g @zephyrdeng/pi-review
|
|
28
|
-
#
|
|
28
|
+
# 一键：Pi 包 + Claude Code / Codex / Cursor 等 skill（推荐首次安装）
|
|
29
|
+
npx @zephyrdeng/pi-review install
|
|
30
|
+
# 仅 Pi 包
|
|
29
31
|
pi install npm:@zephyrdeng/pi-review
|
|
30
|
-
#
|
|
32
|
+
# 仅各 Agent skill
|
|
31
33
|
npx @zephyrdeng/pi-review install-skill
|
|
32
34
|
```
|
|
33
35
|
|
|
36
|
+
`install` 会在有 Pi CLI 时执行 `pi install npm:@zephyrdeng/pi-review`，再通过 [skills CLI](https://www.npmjs.com/package/skills) 非交互安装 agent skill。只用 Pi 时**不要**再跑 `install-skill`，避免与 `pi.skills` 重复冲突。可用 `--pi-only` / `--agents-only` 拆分。
|
|
37
|
+
|
|
34
38
|
## 快速开始
|
|
35
39
|
|
|
36
40
|
```bash
|
|
@@ -62,7 +66,9 @@ pi-review models
|
|
|
62
66
|
|
|
63
67
|
跟进审查时，`--continue` 与首次 `/rv` 一样可**选填** `--mode`、`--model`（以及后续 CLI 支持的其它选项由 skill 直接调用 CLI 时传入）。
|
|
64
68
|
|
|
65
|
-
`/rv`
|
|
69
|
+
`/rv` 会向主会话注入**英文任务说明**（**仅 Pi 宿主**：默认流式、**不要**自动加 `--no-stream` / `--progress-log`），由主 Agent 调用 `pi-review` CLI。普通 `/rv @path` 无需额外参数。需要整段缓冲时显式 `/rv --no-stream`。
|
|
70
|
+
|
|
71
|
+
**Claude Code / Codex 等 Agent 宿主**：Bash 工具会缓冲 stdout，不能指望“默认流式”在对话里逐字出现；**pi-review skill 默认建议** `--progress-log <path>` + 后台运行 + tail 日志。详见 `skills/pi-review/SKILL.md`、`skills/pi-review/references/codex-tools.md` 与英文 README。
|
|
66
72
|
|
|
67
73
|
## 输出格式
|
|
68
74
|
|
package/dist/args.d.ts
CHANGED
package/dist/args.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"args.d.ts","sourceRoot":"","sources":["../src/args.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;
|
|
1
|
+
{"version":3,"file":"args.d.ts","sourceRoot":"","sources":["../src/args.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAG7C,wBAAgB,KAAK,CAAC,QAAQ,SAAI,GAAG,KAAK,CAsCzC;AAUD,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU,CAkGpD"}
|
package/dist/args.js
CHANGED
|
@@ -1,9 +1,11 @@
|
|
|
1
|
-
|
|
1
|
+
import { parseInstallCommand } from "./install-args.js";
|
|
2
|
+
export function usage(exitCode = 0) {
|
|
2
3
|
const out = `Usage:
|
|
3
4
|
pi-review models [search]
|
|
4
5
|
pi-review [options] -- <@files|text...>
|
|
5
6
|
pi-review update Update to the latest version
|
|
6
|
-
pi-review install
|
|
7
|
+
pi-review install [options] Pi package + agent skills (one-shot)
|
|
8
|
+
pi-review install-skill [options] Install skill to AI agents only
|
|
7
9
|
pi-review uninstall-skill [options] Remove skill from AI agents
|
|
8
10
|
|
|
9
11
|
Requires Pi CLI (https://pi.dev) to be installed and configured.
|
|
@@ -29,6 +31,8 @@ Examples:
|
|
|
29
31
|
pi-review -- @src/foo.ts
|
|
30
32
|
pi-review --model openai/gpt-5.5 -- @src/foo.ts
|
|
31
33
|
pi-review --mode challenge --keep-session -- @design.md
|
|
34
|
+
pi-review install
|
|
35
|
+
pi-review install --agent claude-code codex -y
|
|
32
36
|
pi-review install-skill
|
|
33
37
|
`;
|
|
34
38
|
(exitCode === 0 ? process.stdout : process.stderr).write(out);
|
|
@@ -47,6 +51,9 @@ export function parseArgs(argv) {
|
|
|
47
51
|
if (argv[0] === "models") {
|
|
48
52
|
return { command: "models", search: argv.slice(1), mode: "code", skills: [], payload: [], keepSession: false, stream: true };
|
|
49
53
|
}
|
|
54
|
+
if (argv[0] === "install") {
|
|
55
|
+
return parseInstallCommand(argv.slice(1));
|
|
56
|
+
}
|
|
50
57
|
if (argv[0] === "install-skill") {
|
|
51
58
|
return { command: "install-skill", extraArgs: argv.slice(1), mode: "code", skills: [], payload: [], keepSession: false, stream: true };
|
|
52
59
|
}
|
package/dist/args.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"args.js","sourceRoot":"","sources":["../src/args.ts"],"names":[],"mappings":"
|
|
1
|
+
{"version":3,"file":"args.js","sourceRoot":"","sources":["../src/args.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,mBAAmB,EAAE,MAAM,mBAAmB,CAAC;AAExD,MAAM,UAAU,KAAK,CAAC,QAAQ,GAAG,CAAC;IAChC,MAAM,GAAG,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAkCb,CAAC;IACA,CAAC,QAAQ,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC9D,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;AACzB,CAAC;AAED,SAAS,YAAY,CAAC,IAAY,EAAE,IAAc;IAChD,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAC1C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,cAAc,IAAI,qBAAqB,CAAC,CAAC;QAC9D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IACD,OAAO,IAAI,CAAC,KAAK,EAAG,CAAC;AACvB,CAAC;AAED,MAAM,UAAU,SAAS,CAAC,IAAc;IACtC,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,QAAQ;QAAE,KAAK,CAAC,CAAC,CAAC,CAAC;IAC5E,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,QAAQ,EAAE,CAAC;QACzB,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC/H,CAAC;IACD,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;QAC1B,OAAO,mBAAmB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;IAC5C,CAAC;IACD,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,eAAe,EAAE,CAAC;QAChC,OAAO,EAAE,OAAO,EAAE,eAAe,EAAE,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IACzI,CAAC;IACD,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,iBAAiB,EAAE,CAAC;QAClC,OAAO,EAAE,OAAO,EAAE,iBAAiB,EAAE,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC3I,CAAC;IACD,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,QAAQ,EAAE,CAAC;QACzB,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IACxG,CAAC;IAED,IAAI,IAAI,GAAG,MAAM,CAAC;IAClB,IAAI,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QACxC,IAAI,GAAG,IAAI,CAAC,KAAK,EAAG,CAAC;IACvB,CAAC;IAED,MAAM,OAAO,GAAe;QAC1B,OAAO,EAAE,QAAQ;QACjB,IAAI;QACJ,MAAM,EAAE,EAAE;QACV,OAAO,EAAE,EAAE;QACX,WAAW,EAAE,KAAK;QAClB,MAAM,EAAE,IAAI;KACb,CAAC;IAEF,OAAO,IAAI,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,EAAG,CAAC;QAC1B,IAAI,GAAG,KAAK,IAAI,EAAE,CAAC;YACjB,OAAO,CAAC,OAAO,GAAG,IAAI,CAAC,KAAK,EAAE,CAAC;YAC/B,MAAM;QACR,CAAC;QACD,QAAQ,GAAG,EAAE,CAAC;YACZ,KAAK,IAAI,CAAC;YACV,KAAK,QAAQ;gBACX,KAAK,CAAC,CAAC,CAAC,CAAC;gBACT,MAAM;YACR,KAAK,QAAQ;gBACX,OAAO,CAAC,IAAI,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBACvC,MAAM;YACR,KAAK,gBAAgB;gBACnB,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC;gBAC3B,MAAM;YACR,KAAK,YAAY;gBACf,OAAO,CAAC,cAAc,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBACjD,MAAM;YACR,KAAK,SAAS;gBACZ,OAAO,CAAC,KAAK,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBACxC,MAAM;YACR,KAAK,YAAY;gBACf,OAAO,CAAC,QAAQ,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBAC3C,MAAM;YACR,KAAK,YAAY;gBACf,OAAO,CAAC,QAAQ,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBAC3C,MAAM;YACR,KAAK,SAAS;gBACZ,OAAO,CAAC,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC;gBAC7C,MAAM;YACR,KAAK,SAAS;gBACZ,OAAO,CAAC,KAAK,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBACxC,MAAM;YACR,KAAK,QAAQ;gBACX,OAAO,CAAC,IAAI,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBACvC,MAAM;YACR,KAAK,aAAa;gBAChB,OAAO,CAAC,MAAM,GAAG,KAAK,CAAC;gBACvB,MAAM;YACR,KAAK,gBAAgB;gBACnB,OAAO,CAAC,WAAW,GAAG,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBAC9C,MAAM;YACR;gBACE,IAAI,GAAG,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;oBACzB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,8BAA8B,GAAG,IAAI,CAAC,CAAC;oBAC5D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;gBAClB,CAAC;gBACD,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;gBACnC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC;gBAChB,MAAM;QACV,CAAC;IACH,CAAC;IAED,IAAI,OAAO,CAAC,WAAW,IAAI,OAAO,CAAC,cAAc,EAAE,CAAC;QAClD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oEAAoE,CAAC,CAAC;QAC3F,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,IAAI,OAAO,CAAC,WAAW,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,CAAC;QAC3C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,6DAA6D,CAAC,CAAC;QACpF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}
|
package/dist/cli.js
CHANGED
|
@@ -1,10 +1,14 @@
|
|
|
1
1
|
#!/usr/bin/env node
|
|
2
|
-
import { parseArgs } from "./args.js";
|
|
2
|
+
import { parseArgs, usage } from "./args.js";
|
|
3
|
+
import { isInstallHelp } from "./install-args.js";
|
|
3
4
|
import { runModels, runReview } from "./review.js";
|
|
4
5
|
import { resolveConfig } from "./config.js";
|
|
5
6
|
import { installSkill, uninstallSkill } from "./skill.js";
|
|
6
7
|
import { runUpdate } from "./update.js";
|
|
8
|
+
import { runInstall } from "./install.js";
|
|
7
9
|
const parsed = parseArgs(process.argv.slice(2));
|
|
10
|
+
if (isInstallHelp(parsed))
|
|
11
|
+
usage(0);
|
|
8
12
|
if (parsed.command === "models") {
|
|
9
13
|
const config = resolveConfig();
|
|
10
14
|
runModels(config.piBin, parsed.search || []);
|
|
@@ -12,6 +16,13 @@ if (parsed.command === "models") {
|
|
|
12
16
|
else if (parsed.command === "update") {
|
|
13
17
|
runUpdate();
|
|
14
18
|
}
|
|
19
|
+
else if (parsed.command === "install") {
|
|
20
|
+
runInstall({
|
|
21
|
+
pi: parsed.installPi !== false,
|
|
22
|
+
agents: parsed.installAgents !== false,
|
|
23
|
+
agentArgs: parsed.extraArgs ?? [],
|
|
24
|
+
});
|
|
25
|
+
}
|
|
15
26
|
else if (parsed.command === "install-skill") {
|
|
16
27
|
installSkill(parsed.extraArgs);
|
|
17
28
|
}
|
package/dist/cli.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;
|
|
1
|
+
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":";AACA,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,MAAM,WAAW,CAAC;AAC7C,OAAO,EAAE,aAAa,EAAE,MAAM,mBAAmB,CAAC;AAClD,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AACnD,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAC5C,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,YAAY,CAAC;AAC1D,OAAO,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AACxC,OAAO,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAE1C,MAAM,MAAM,GAAG,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;AAChD,IAAI,aAAa,CAAC,MAAM,CAAC;IAAE,KAAK,CAAC,CAAC,CAAC,CAAC;AAEpC,IAAI,MAAM,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;IAChC,MAAM,MAAM,GAAG,aAAa,EAAE,CAAC;IAC/B,SAAS,CAAC,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC;AAC/C,CAAC;KAAM,IAAI,MAAM,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;IACvC,SAAS,EAAE,CAAC;AACd,CAAC;KAAM,IAAI,MAAM,CAAC,OAAO,KAAK,SAAS,EAAE,CAAC;IACxC,UAAU,CAAC;QACT,EAAE,EAAE,MAAM,CAAC,SAAS,KAAK,KAAK;QAC9B,MAAM,EAAE,MAAM,CAAC,aAAa,KAAK,KAAK;QACtC,SAAS,EAAE,MAAM,CAAC,SAAS,IAAI,EAAE;KAClC,CAAC,CAAC;AACL,CAAC;KAAM,IAAI,MAAM,CAAC,OAAO,KAAK,eAAe,EAAE,CAAC;IAC9C,YAAY,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;AACjC,CAAC;KAAM,IAAI,MAAM,CAAC,OAAO,KAAK,iBAAiB,EAAE,CAAC;IAChD,cAAc,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;AACnC,CAAC;KAAM,CAAC;IACN,MAAM,SAAS,CAAC,MAAM,CAAC,CAAC;AAC1B,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install-args.d.ts","sourceRoot":"","sources":["../src/install-args.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAE7C,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU,CA4C9D;AAED,wBAAgB,aAAa,CAAC,MAAM,EAAE,UAAU,GAAG,OAAO,CAEzD"}
|
|
@@ -0,0 +1,47 @@
|
|
|
1
|
+
export function parseInstallCommand(argv) {
|
|
2
|
+
let installPi = true;
|
|
3
|
+
let installAgents = true;
|
|
4
|
+
const agentArgs = [];
|
|
5
|
+
for (let i = 0; i < argv.length; i++) {
|
|
6
|
+
const arg = argv[i];
|
|
7
|
+
if (arg === "-h" || arg === "--help") {
|
|
8
|
+
return {
|
|
9
|
+
command: "install",
|
|
10
|
+
extraArgs: ["--help"],
|
|
11
|
+
mode: "code",
|
|
12
|
+
skills: [],
|
|
13
|
+
payload: [],
|
|
14
|
+
keepSession: false,
|
|
15
|
+
stream: true,
|
|
16
|
+
installPi: true,
|
|
17
|
+
installAgents: true,
|
|
18
|
+
};
|
|
19
|
+
}
|
|
20
|
+
if (arg === "--pi-only") {
|
|
21
|
+
installPi = true;
|
|
22
|
+
installAgents = false;
|
|
23
|
+
continue;
|
|
24
|
+
}
|
|
25
|
+
if (arg === "--agents-only") {
|
|
26
|
+
installPi = false;
|
|
27
|
+
installAgents = true;
|
|
28
|
+
continue;
|
|
29
|
+
}
|
|
30
|
+
agentArgs.push(arg);
|
|
31
|
+
}
|
|
32
|
+
return {
|
|
33
|
+
command: "install",
|
|
34
|
+
extraArgs: agentArgs,
|
|
35
|
+
mode: "code",
|
|
36
|
+
skills: [],
|
|
37
|
+
payload: [],
|
|
38
|
+
keepSession: false,
|
|
39
|
+
stream: true,
|
|
40
|
+
installPi,
|
|
41
|
+
installAgents,
|
|
42
|
+
};
|
|
43
|
+
}
|
|
44
|
+
export function isInstallHelp(parsed) {
|
|
45
|
+
return parsed.command === "install" && (parsed.extraArgs?.includes("--help") || parsed.extraArgs?.includes("-h") || false);
|
|
46
|
+
}
|
|
47
|
+
//# sourceMappingURL=install-args.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install-args.js","sourceRoot":"","sources":["../src/install-args.ts"],"names":[],"mappings":"AAEA,MAAM,UAAU,mBAAmB,CAAC,IAAc;IAChD,IAAI,SAAS,GAAG,IAAI,CAAC;IACrB,IAAI,aAAa,GAAG,IAAI,CAAC;IACzB,MAAM,SAAS,GAAa,EAAE,CAAC;IAE/B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACrC,MAAM,GAAG,GAAG,IAAI,CAAC,CAAC,CAAE,CAAC;QACrB,IAAI,GAAG,KAAK,IAAI,IAAI,GAAG,KAAK,QAAQ,EAAE,CAAC;YACrC,OAAO;gBACL,OAAO,EAAE,SAAS;gBAClB,SAAS,EAAE,CAAC,QAAQ,CAAC;gBACrB,IAAI,EAAE,MAAM;gBACZ,MAAM,EAAE,EAAE;gBACV,OAAO,EAAE,EAAE;gBACX,WAAW,EAAE,KAAK;gBAClB,MAAM,EAAE,IAAI;gBACZ,SAAS,EAAE,IAAI;gBACf,aAAa,EAAE,IAAI;aACpB,CAAC;QACJ,CAAC;QACD,IAAI,GAAG,KAAK,WAAW,EAAE,CAAC;YACxB,SAAS,GAAG,IAAI,CAAC;YACjB,aAAa,GAAG,KAAK,CAAC;YACtB,SAAS;QACX,CAAC;QACD,IAAI,GAAG,KAAK,eAAe,EAAE,CAAC;YAC5B,SAAS,GAAG,KAAK,CAAC;YAClB,aAAa,GAAG,IAAI,CAAC;YACrB,SAAS;QACX,CAAC;QACD,SAAS,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACtB,CAAC;IAED,OAAO;QACL,OAAO,EAAE,SAAS;QAClB,SAAS,EAAE,SAAS;QACpB,IAAI,EAAE,MAAM;QACZ,MAAM,EAAE,EAAE;QACV,OAAO,EAAE,EAAE;QACX,WAAW,EAAE,KAAK;QAClB,MAAM,EAAE,IAAI;QACZ,SAAS;QACT,aAAa;KACd,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,MAAkB;IAC9C,OAAO,MAAM,CAAC,OAAO,KAAK,SAAS,IAAI,CAAC,MAAM,CAAC,SAAS,EAAE,QAAQ,CAAC,QAAQ,CAAC,IAAI,MAAM,CAAC,SAAS,EAAE,QAAQ,CAAC,IAAI,CAAC,IAAI,KAAK,CAAC,CAAC;AAC7H,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install-args.test.d.ts","sourceRoot":"","sources":["../src/install-args.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
import assert from "node:assert/strict";
|
|
2
|
+
import test from "node:test";
|
|
3
|
+
import { parseInstallCommand } from "./install-args.js";
|
|
4
|
+
test("install defaults to pi + agents", () => {
|
|
5
|
+
const p = parseInstallCommand([]);
|
|
6
|
+
assert.equal(p.command, "install");
|
|
7
|
+
assert.equal(p.installPi, true);
|
|
8
|
+
assert.equal(p.installAgents, true);
|
|
9
|
+
});
|
|
10
|
+
test("install --pi-only", () => {
|
|
11
|
+
const p = parseInstallCommand(["--pi-only"]);
|
|
12
|
+
assert.equal(p.installPi, true);
|
|
13
|
+
assert.equal(p.installAgents, false);
|
|
14
|
+
});
|
|
15
|
+
test("install --agents-only forwards args", () => {
|
|
16
|
+
const p = parseInstallCommand(["--agents-only", "--agent", "codex", "-y"]);
|
|
17
|
+
assert.equal(p.installPi, false);
|
|
18
|
+
assert.equal(p.installAgents, true);
|
|
19
|
+
assert.deepEqual(p.extraArgs, ["--agent", "codex", "-y"]);
|
|
20
|
+
});
|
|
21
|
+
//# sourceMappingURL=install-args.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install-args.test.js","sourceRoot":"","sources":["../src/install-args.test.ts"],"names":[],"mappings":"AAAA,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,mBAAmB,EAAE,MAAM,mBAAmB,CAAC;AAExD,IAAI,CAAC,iCAAiC,EAAE,GAAG,EAAE;IAC3C,MAAM,CAAC,GAAG,mBAAmB,CAAC,EAAE,CAAC,CAAC;IAClC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;IACnC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;IAChC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC;AACtC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,mBAAmB,EAAE,GAAG,EAAE;IAC7B,MAAM,CAAC,GAAG,mBAAmB,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC;IAC7C,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;IAChC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,aAAa,EAAE,KAAK,CAAC,CAAC;AACvC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,qCAAqC,EAAE,GAAG,EAAE;IAC/C,MAAM,CAAC,GAAG,mBAAmB,CAAC,CAAC,eAAe,EAAE,SAAS,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;IAC3E,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,EAAE,KAAK,CAAC,CAAC;IACjC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC;IACpC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,EAAE,CAAC,SAAS,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;AAC5D,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1,8 @@
|
|
|
1
|
+
export interface InstallOptions {
|
|
2
|
+
pi: boolean;
|
|
3
|
+
agents: boolean;
|
|
4
|
+
/** Forwarded to install-skill / skills CLI (e.g. --agent codex -y). */
|
|
5
|
+
agentArgs: string[];
|
|
6
|
+
}
|
|
7
|
+
export declare function runInstall(options: InstallOptions): never;
|
|
8
|
+
//# sourceMappingURL=install.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install.d.ts","sourceRoot":"","sources":["../src/install.ts"],"names":[],"mappings":"AAuCA,MAAM,WAAW,cAAc;IAC7B,EAAE,EAAE,OAAO,CAAC;IACZ,MAAM,EAAE,OAAO,CAAC;IAChB,uEAAuE;IACvE,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB;AAED,wBAAgB,UAAU,CAAC,OAAO,EAAE,cAAc,GAAG,KAAK,CAkBzD"}
|
package/dist/install.js
ADDED
|
@@ -0,0 +1,46 @@
|
|
|
1
|
+
import { spawnSync } from "node:child_process";
|
|
2
|
+
import { installSkill } from "./skill.js";
|
|
3
|
+
const PI_PACKAGE = "npm:@zephyrdeng/pi-review";
|
|
4
|
+
const DEFAULT_AGENT_ARGS = ["-y", "--agent", "claude-code", "codex", "cursor"];
|
|
5
|
+
function hasPiCli() {
|
|
6
|
+
const result = spawnSync("pi", ["--version"], {
|
|
7
|
+
encoding: "utf8",
|
|
8
|
+
timeout: 10_000,
|
|
9
|
+
stdio: "pipe",
|
|
10
|
+
});
|
|
11
|
+
return result.status === 0;
|
|
12
|
+
}
|
|
13
|
+
function runPiInstall() {
|
|
14
|
+
if (!hasPiCli()) {
|
|
15
|
+
process.stderr.write("pi-review install: Pi CLI not found — skipped Pi package. Install Pi from https://pi.dev then: pi install npm:@zephyrdeng/pi-review\n");
|
|
16
|
+
return false;
|
|
17
|
+
}
|
|
18
|
+
process.stdout.write(`Running: pi install ${PI_PACKAGE}\n`);
|
|
19
|
+
const result = spawnSync("pi", ["install", PI_PACKAGE], {
|
|
20
|
+
encoding: "utf8",
|
|
21
|
+
stdio: "inherit",
|
|
22
|
+
});
|
|
23
|
+
if (result.status !== 0) {
|
|
24
|
+
process.stderr.write(`pi-review install: pi install failed (exit ${result.status ?? "unknown"})\n`);
|
|
25
|
+
return false;
|
|
26
|
+
}
|
|
27
|
+
process.stdout.write("Pi: npm package registered (/rv extension + skill via pi.skills — no extra copy under ~/.agents for Pi).\n");
|
|
28
|
+
return true;
|
|
29
|
+
}
|
|
30
|
+
export function runInstall(options) {
|
|
31
|
+
if (!options.pi && !options.agents) {
|
|
32
|
+
process.stderr.write("pi-review install: nothing to do\n");
|
|
33
|
+
process.exit(2);
|
|
34
|
+
}
|
|
35
|
+
let piOk = true;
|
|
36
|
+
if (options.pi) {
|
|
37
|
+
piOk = runPiInstall();
|
|
38
|
+
}
|
|
39
|
+
if (options.agents) {
|
|
40
|
+
const args = options.agentArgs.length > 0 ? options.agentArgs : DEFAULT_AGENT_ARGS;
|
|
41
|
+
process.stdout.write(`Running: pi-review install-skill ${args.join(" ")}\n`);
|
|
42
|
+
installSkill(args);
|
|
43
|
+
}
|
|
44
|
+
process.exit(piOk ? 0 : 1);
|
|
45
|
+
}
|
|
46
|
+
//# sourceMappingURL=install.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"install.js","sourceRoot":"","sources":["../src/install.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAE1C,MAAM,UAAU,GAAG,2BAA2B,CAAC;AAE/C,MAAM,kBAAkB,GAAG,CAAC,IAAI,EAAE,SAAS,EAAE,aAAa,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;AAE/E,SAAS,QAAQ;IACf,MAAM,MAAM,GAAG,SAAS,CAAC,IAAI,EAAE,CAAC,WAAW,CAAC,EAAE;QAC5C,QAAQ,EAAE,MAAM;QAChB,OAAO,EAAE,MAAM;QACf,KAAK,EAAE,MAAM;KACd,CAAC,CAAC;IACH,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,YAAY;IACnB,IAAI,CAAC,QAAQ,EAAE,EAAE,CAAC;QAChB,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,uIAAuI,CACxI,CAAC;QACF,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,uBAAuB,UAAU,IAAI,CAAC,CAAC;IAC5D,MAAM,MAAM,GAAG,SAAS,CAAC,IAAI,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC,EAAE;QACtD,QAAQ,EAAE,MAAM;QAChB,KAAK,EAAE,SAAS;KACjB,CAAC,CAAC;IACH,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACxB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,8CAA8C,MAAM,CAAC,MAAM,IAAI,SAAS,KAAK,CAAC,CAAC;QACpG,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,4GAA4G,CAC7G,CAAC;IACF,OAAO,IAAI,CAAC;AACd,CAAC;AASD,MAAM,UAAU,UAAU,CAAC,OAAuB;IAChD,IAAI,CAAC,OAAO,CAAC,EAAE,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,CAAC;QACnC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oCAAoC,CAAC,CAAC;QAC3D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,IAAI,IAAI,GAAG,IAAI,CAAC;IAChB,IAAI,OAAO,CAAC,EAAE,EAAE,CAAC;QACf,IAAI,GAAG,YAAY,EAAE,CAAC;IACxB,CAAC;IAED,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;QACnB,MAAM,IAAI,GAAG,OAAO,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,kBAAkB,CAAC;QACnF,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oCAAoC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QAC7E,YAAY,CAAC,IAAI,CAAC,CAAC;IACrB,CAAC;IAED,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAC7B,CAAC"}
|
package/dist/types.d.ts
CHANGED
|
@@ -1,7 +1,11 @@
|
|
|
1
1
|
export declare const VERDICTS: readonly ["approve", "request_changes", "needs_clarification", "blocked"];
|
|
2
2
|
export type Verdict = (typeof VERDICTS)[number];
|
|
3
3
|
export interface ParsedArgs {
|
|
4
|
-
command: "review" | "models" | "install-skill" | "uninstall-skill" | "update";
|
|
4
|
+
command: "review" | "models" | "install" | "install-skill" | "uninstall-skill" | "update";
|
|
5
|
+
/** For `install`: run `pi install npm:@zephyrdeng/pi-review` */
|
|
6
|
+
installPi?: boolean;
|
|
7
|
+
/** For `install`: run agent skill install (skills CLI / Claude fallback) */
|
|
8
|
+
installAgents?: boolean;
|
|
5
9
|
extraArgs?: string[];
|
|
6
10
|
mode: string;
|
|
7
11
|
skills: string[];
|
package/dist/types.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,QAAQ,2EAA4E,CAAC;AAClG,MAAM,MAAM,OAAO,GAAG,CAAC,OAAO,QAAQ,CAAC,CAAC,MAAM,CAAC,CAAC;AAEhD,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,QAAQ,GAAG,QAAQ,GAAG,eAAe,GAAG,iBAAiB,GAAG,QAAQ,CAAC;
|
|
1
|
+
{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,QAAQ,2EAA4E,CAAC;AAClG,MAAM,MAAM,OAAO,GAAG,CAAC,OAAO,QAAQ,CAAC,CAAC,MAAM,CAAC,CAAC;AAEhD,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,QAAQ,GAAG,QAAQ,GAAG,SAAS,GAAG,eAAe,GAAG,iBAAiB,GAAG,QAAQ,CAAC;IAC1F,gEAAgE;IAChE,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,4EAA4E;IAC5E,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,WAAW,EAAE,OAAO,CAAC;IACrB,kGAAkG;IAClG,MAAM,EAAE,OAAO,CAAC;IAChB,yFAAyF;IACzF,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,EAAE,CAAC;CACnB;AAED,MAAM,WAAW,YAAY;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,MAAM,EAAE,GAAG,MAAM,CAAC;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;CACvB;AAED,MAAM,WAAW,WAAW;IAC1B,OAAO,EAAE,OAAO,CAAC;IACjB,aAAa,EAAE,QAAQ,GAAG,UAAU,GAAG,eAAe,CAAC;IACvD,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,UAAU;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;IACjB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;CAClB"}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@zephyrdeng/pi-review",
|
|
3
|
-
"version": "0.
|
|
3
|
+
"version": "0.5.1",
|
|
4
4
|
"description": "Isolated AI-powered code and plan reviews from the command line",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"author": "ZephyrDeng <zephyrTang@aliyun.com>",
|
|
@@ -67,7 +67,7 @@
|
|
|
67
67
|
"build": "tsc",
|
|
68
68
|
"check": "tsc --noEmit",
|
|
69
69
|
"smoke": "node dist/cli.js --help >/dev/null",
|
|
70
|
-
"test": "npx --yes tsx --test src/child-process.test.ts src/review.meta.test.ts src/meta-footer.test.ts src/json-events.test.ts extensions/rv-prompts.test.ts",
|
|
70
|
+
"test": "npx --yes tsx --test src/child-process.test.ts src/review.meta.test.ts src/meta-footer.test.ts src/json-events.test.ts src/install-args.test.ts extensions/rv-prompts.test.ts",
|
|
71
71
|
"test:rv": "npx --yes tsx --test extensions/rv-prompts.test.ts",
|
|
72
72
|
"prepublishOnly": "npm run build",
|
|
73
73
|
"pack:dry": "npm pack --dry-run"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pi-review
|
|
3
|
-
description: Use pi-review to delegate isolated code, diff, repository, architecture, or plan reviews to a fresh Pi session and return only the review conclusion. Use when the user asks for Pi review, isolated review, code review, plan review, challenge review, review status/progress, or wants to avoid context pollution.
|
|
3
|
+
description: Use pi-review to delegate isolated code, diff, repository, architecture, or plan reviews to a fresh Pi session and return only the review conclusion. On Claude Code or Codex, default to --progress-log with a background shell run and tail the log (chat tools buffer stdout). Use when the user asks for Pi review, isolated review, code review, plan review, challenge review, review status/progress, or wants to avoid context pollution.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Pi Review
|
|
@@ -23,17 +23,36 @@ node ../../bin/pi-review.js --help
|
|
|
23
23
|
|
|
24
24
|
When resolving this fallback, use the actual directory that contains this `SKILL.md`.
|
|
25
25
|
|
|
26
|
+
## Codex / Claude Code
|
|
27
|
+
|
|
28
|
+
- Map skill mentions of `Bash` to your host shell tool.
|
|
29
|
+
- Read **[references/codex-tools.md](./references/codex-tools.md)** (next to this `SKILL.md`) for Codex/Claude Code defaults: `--progress-log` + background run + tail.
|
|
30
|
+
- Do not use a blocking foreground shell alone for live progress; follow **Default workflow by host** below.
|
|
31
|
+
|
|
32
|
+
## Default workflow by host
|
|
33
|
+
|
|
34
|
+
| Host | How to run `pi-review` |
|
|
35
|
+
|------|-------------------------|
|
|
36
|
+
| **Pi** (`/rv`) | Foreground CLI, **default streaming** only. Do **not** add `--no-stream` or `--progress-log` unless the user explicitly asks. |
|
|
37
|
+
| **Claude Code, Codex, Cursor**, and similar AI agents | **Default: `--progress-log <path>` + background run + tail** (see below). Do **not** rely on foreground Bash stdout for “streaming” — the tool buffers until exit. |
|
|
38
|
+
| Scripts / CI | Foreground is fine; use `--no-stream` only when you must buffer until exit. |
|
|
39
|
+
|
|
26
40
|
## Pi host (`/rv`)
|
|
27
41
|
|
|
28
42
|
- In **Pi interactive** sessions, run `pi-review` with **default streaming only** — do **not** add `--no-stream` or `--progress-log` unless the user explicitly asks.
|
|
29
43
|
- The `/rv` extension injects these rules into the parent agent; ordinary `/rv @path` needs no extra flags.
|
|
30
44
|
|
|
31
|
-
## Streaming vs agent hosts
|
|
45
|
+
## Streaming vs agent hosts (Claude Code / Codex)
|
|
32
46
|
|
|
33
|
-
- `pi-review` **streams** child `pi` stdout/stderr to the process terminal by default (`--no-stream` buffers until exit).
|
|
34
|
-
- **
|
|
35
|
-
-
|
|
36
|
-
-
|
|
47
|
+
- `pi-review` **streams** child `pi` stdout/stderr to the process terminal by default (`--no-stream` buffers until exit). That helps **Pi** and real terminals, not most agent chat UIs.
|
|
48
|
+
- **Claude Code, Cursor, Codex**, and similar hosts **buffer tool stdout** until the bash command exits, so the chat shows the review **all at once** even though the CLI is streaming. That is expected—not a misconfiguration.
|
|
49
|
+
- Default `text` mode also produces **no stdout** during tool-use/thinking (only the final answer), so even a local terminal stays quiet for most of a multi-minute review.
|
|
50
|
+
- **Recommended for Claude Code / Codex (live progress):** always use `--progress-log <path>` unless the user explicitly wants a silent blocking wait:
|
|
51
|
+
1. Pick a writable path (e.g. `/tmp/pi-review-<id>.jsonl`).
|
|
52
|
+
2. Start `pi-review --progress-log <path> ...` in a **background** or **async** shell invocation (not a blocking foreground Bash that you wait on with no updates).
|
|
53
|
+
3. While the child runs, **tail** the log and summarize milestones to the user (filter out `message_update` / `tool_execution_update` unless they asked for token-level detail).
|
|
54
|
+
4. When the process exits, show the Markdown body and ASCII `── pi-review` footer from the command result (or re-read the final output as the skill’s step 4 describes).
|
|
55
|
+
- Fall back to a **local terminal** with default streaming only if the host cannot run background commands or tail a file.
|
|
37
56
|
|
|
38
57
|
## Review status (no slash command)
|
|
39
58
|
|
|
@@ -87,8 +106,9 @@ Do not implement findings from a status check; status is informational only.
|
|
|
87
106
|
- Use `--continue <handle>` only with a **Session** path from a prior ASCII footer or `sessionHandle` in `PI_REVIEW_META_JSON`.
|
|
88
107
|
- Put file references as `@path` after `--`.
|
|
89
108
|
- Do not ask `pi-review` to edit, patch, commit, or implement.
|
|
90
|
-
-
|
|
91
|
-
-
|
|
109
|
+
- **Pi `/rv`:** default streaming; no `--progress-log` unless the user asked.
|
|
110
|
+
- **Claude Code / Codex / Cursor:** **include `--progress-log <path>` by default**; background the CLI and tail the log for user-visible progress. Foreground Bash alone does not stream in chat.
|
|
111
|
+
- Add `--no-stream` only when the caller must buffer until exit (cannot combine with `--progress-log`).
|
|
92
112
|
Completion criterion: the command includes the resolved model/default choice and a concrete review target.
|
|
93
113
|
|
|
94
114
|
4. Return the result:
|
|
@@ -0,0 +1,34 @@
|
|
|
1
|
+
# Agent host tool notes (Codex / Claude Code)
|
|
2
|
+
|
|
3
|
+
Skills may use Claude Code tool names. On **Codex** or **Claude Code**, map them to your native tools:
|
|
4
|
+
|
|
5
|
+
| Skill references | Typical host equivalent |
|
|
6
|
+
|-----------------|-------------------------|
|
|
7
|
+
| `Bash` (run commands) | Native shell / terminal tool |
|
|
8
|
+
| `Read`, `Write`, `Edit` | Native file tools |
|
|
9
|
+
| `Skill` | Load and follow the skill instructions directly |
|
|
10
|
+
|
|
11
|
+
General Codex subagent and worktree mappings live in the **using-superpowers** skill (`references/codex-tools.md`) when that skill is installed; this file is **pi-review–specific** and ships with the pi-review package.
|
|
12
|
+
|
|
13
|
+
## Long-running `pi-review` (default on Codex / Claude Code)
|
|
14
|
+
|
|
15
|
+
Native shell tools usually **buffer stdout/stderr until the command exits**. A multi-minute `pi-review` run can look like a silent wait, then one large dump—even though the CLI streams by default. Default `text` mode also emits **no stdout** during tool-use/thinking (only the final answer).
|
|
16
|
+
|
|
17
|
+
When the **pi-review** skill applies on Codex, Claude Code, or Cursor:
|
|
18
|
+
|
|
19
|
+
1. **Default:** `pi-review --progress-log <path> ...` (writable file, e.g. `/tmp/pi-review-<id>.jsonl`).
|
|
20
|
+
2. Start that command in a **background** or **async** shell invocation—not a single blocking call with no interim updates.
|
|
21
|
+
3. **Tail** the log while the child runs; summarize milestones to the user (skip `message_update` / `tool_execution_update` unless they want token-level detail).
|
|
22
|
+
4. After exit, show the Markdown review and ASCII `── pi-review` footer (skill step 4).
|
|
23
|
+
|
|
24
|
+
**Pi interactive (`/rv`)** is different: foreground `pi-review`, default streaming only—do **not** auto-add `--progress-log` unless the user asked.
|
|
25
|
+
|
|
26
|
+
See the parent skill: sections *Default workflow by host* and *Streaming vs agent hosts (Claude Code / Codex)*.
|
|
27
|
+
|
|
28
|
+
Example tail (milestone events only):
|
|
29
|
+
|
|
30
|
+
```bash
|
|
31
|
+
tail -f -n +1 /tmp/pi-review.jsonl | jq -c --unbuffered '
|
|
32
|
+
select(.type != "message_update" and .type != "tool_execution_update")
|
|
33
|
+
'
|
|
34
|
+
```
|