@zephyrdeng/pi-review 0.9.1 → 0.10.0

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.
Files changed (41) hide show
  1. package/README.md +1 -1
  2. package/README.zh-CN.md +1 -1
  3. package/dist/json-events.d.ts +1 -0
  4. package/dist/json-events.d.ts.map +1 -1
  5. package/dist/json-events.js +12 -2
  6. package/dist/json-events.js.map +1 -1
  7. package/dist/json-events.test.js +22 -0
  8. package/dist/json-events.test.js.map +1 -1
  9. package/dist/meta-footer.d.ts.map +1 -1
  10. package/dist/meta-footer.js +6 -1
  11. package/dist/meta-footer.js.map +1 -1
  12. package/dist/meta-footer.test.js +42 -1
  13. package/dist/meta-footer.test.js.map +1 -1
  14. package/dist/panel-aggregate.d.ts +7 -0
  15. package/dist/panel-aggregate.d.ts.map +1 -1
  16. package/dist/panel-aggregate.js +19 -1
  17. package/dist/panel-aggregate.js.map +1 -1
  18. package/dist/panel-aggregate.test.js +56 -1
  19. package/dist/panel-aggregate.test.js.map +1 -1
  20. package/dist/panel-cli.test.js +42 -0
  21. package/dist/panel-cli.test.js.map +1 -1
  22. package/dist/panel-view.d.ts.map +1 -1
  23. package/dist/panel-view.js +9 -3
  24. package/dist/panel-view.js.map +1 -1
  25. package/dist/panel-view.test.js +62 -0
  26. package/dist/panel-view.test.js.map +1 -1
  27. package/dist/panel.d.ts.map +1 -1
  28. package/dist/panel.js +9 -4
  29. package/dist/panel.js.map +1 -1
  30. package/dist/review-events.d.ts +1 -0
  31. package/dist/review-events.d.ts.map +1 -1
  32. package/dist/types.d.ts +6 -0
  33. package/dist/types.d.ts.map +1 -1
  34. package/extensions/panel-tool.test.ts +42 -0
  35. package/extensions/panel-tool.ts +3 -1
  36. package/extensions/rv-interactive.test.ts +179 -8
  37. package/extensions/rv-interactive.ts +64 -61
  38. package/package.json +1 -1
  39. package/skills/pi-review/SKILL.md +7 -6
  40. package/skills/pi-review/references/codex-tools.md +3 -3
  41. package/skills/pi-review/references/model-selection.md +2 -2
@@ -6,11 +6,11 @@ Use this after `pi-review models` (or `pi-review models <search>`). **Never inve
6
6
 
7
7
  1. Run `pi-review models` (narrow with search if the user named a vendor or model family).
8
8
  2. Infer **review profile** from the target (see table below).
9
- 3. Walk the **priority list** for that profile; pick the **first** row whose `idContains` matches some listed model id (case-insensitive substring). If several ids match one row, prefer the **newest** id (higher version suffix, e.g. `2.7` over `2.5`, `4-8` over `4-6`).
9
+ 3. **Cross-model review**: identify the host session's own model — the model that wrote or will fix the code under review; its id is in your own session context. Walk the **priority list** for that profile skipping rows in the host's model family, and pick the **first** remaining row whose `idContains` matches some listed model id (case-insensitive substring). Same-family reviewers share the author model's blind spots; return to the host's own family only when no other family matches the catalog. On a panel, spread reviewer models so the host's family stays a minority. If several ids match one row, prefer the **newest** id (higher version suffix, e.g. `2.7` over `2.5`, `4-8` over `4-6`).
10
10
  4. Set `--model <exact-listed-provider/model>` from the catalog. Add `:thinking` only if that model supports it in Pi and the preset suggests it (see thinking column).
11
11
  - User “max / 最高档” maps to Pi thinking `xhigh` when supported; otherwise fall back to the highest available level.
12
12
  - User “low / off / medium / high” maps to that exact level when supported — **do not upgrade** a user-specified low/off to high/xhigh.
13
- 5. If **no** priority row matches anything in the catalog, either omit `--model` (Pi default) or pick the best **reasoning** model with a large context window from the list—state the choice in one line to the user.
13
+ 5. When the user named no model, the walk above yields a **proposal, not a decision**: present 2–3 exact catalog ids (cross-model candidates first, one line of why each, plus “Pi default” as an explicit option) and ask the user to pick before running the review. A user-named model skips the ask and resolves per **User-named model** below. If **no** priority row matches anything in the catalog, propose the best **reasoning** models with a large context window from the list the same way.
14
14
 
15
15
  ## Profiles
16
16