@zephyrdeng/pi-review 0.12.1 → 0.14.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.
- package/README.md +18 -9
- package/README.zh-CN.md +12 -3
- package/dist/args.d.ts +12 -0
- package/dist/args.d.ts.map +1 -1
- package/dist/args.js +18 -5
- package/dist/args.js.map +1 -1
- package/dist/args.test.js +4 -2
- package/dist/args.test.js.map +1 -1
- package/dist/index.d.ts +6 -6
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +4 -4
- package/dist/index.js.map +1 -1
- package/dist/panel-cli.test.js +35 -0
- package/dist/panel-cli.test.js.map +1 -1
- package/dist/panel-config.d.ts +7 -0
- package/dist/panel-config.d.ts.map +1 -1
- package/dist/panel-config.js +12 -0
- package/dist/panel-config.js.map +1 -1
- package/dist/panel.d.ts +7 -1
- package/dist/panel.d.ts.map +1 -1
- package/dist/panel.js +44 -1
- package/dist/panel.js.map +1 -1
- package/dist/provider-probe.test.d.ts +2 -0
- package/dist/provider-probe.test.d.ts.map +1 -0
- package/dist/provider-probe.test.js +192 -0
- package/dist/provider-probe.test.js.map +1 -0
- package/dist/review.d.ts +44 -2
- package/dist/review.d.ts.map +1 -1
- package/dist/review.js +140 -1
- package/dist/review.js.map +1 -1
- package/dist/skill-update.test.js +61 -3
- package/dist/skill-update.test.js.map +1 -1
- package/dist/skill.d.ts +28 -0
- package/dist/skill.d.ts.map +1 -1
- package/dist/skill.js +155 -31
- package/dist/skill.js.map +1 -1
- package/dist/types.d.ts +26 -2
- package/dist/types.d.ts.map +1 -1
- package/dist/types.js.map +1 -1
- package/extensions/panel-tool.test.ts +15 -1
- package/extensions/panel-tool.ts +49 -19
- package/extensions/rv-completions.ts +1 -1
- package/extensions/rv-prompts.test.ts +13 -3
- package/extensions/rv-prompts.ts +28 -15
- package/package.json +1 -1
- package/skills/pi-review/SKILL.md +1 -1
- package/skills/pi-review/references/codex-tools.md +5 -5
- package/skills/pi-review/references/model-selection.md +1 -1
package/dist/skill.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"skill.js","sourceRoot":"","sources":["../src/skill.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAEzC,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAC/D,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;AACnD,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,YAAY,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC;AACzE,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,EAAE,UAAU,CAAC,CAAC;AAC/D,MAAM,IAAI,GAAG,sBAAsB,CAAC;AACpC,MAAM,UAAU,GAAG,WAAW,CAAC;AAE/B,uFAAuF;AACvF,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,IAAI,EAAE,SAAS,EAAE,aAAa,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;
|
|
1
|
+
{"version":3,"file":"skill.js","sourceRoot":"","sources":["../src/skill.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAEzC,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAC/D,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;AACnD,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,YAAY,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC;AACzE,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,EAAE,UAAU,CAAC,CAAC;AAC/D,MAAM,IAAI,GAAG,sBAAsB,CAAC;AACpC,MAAM,UAAU,GAAG,WAAW,CAAC;AAE/B,6DAA6D;AAC7D,MAAM,CAAC,MAAM,oBAAoB,GAAG;IAClC,aAAa;IACb,OAAO;IACP,QAAQ;IACR,8EAA8E;IAC9E,aAAa;IACb,iBAAiB;CACT,CAAC;AAEX,uFAAuF;AACvF,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,IAAI,EAAE,SAAS,EAAE,GAAG,oBAAoB,CAAC,CAAC;AAEnF,mEAAmE;AACnE,MAAM,SAAS,GAAG,KAAK,CAAC;AACxB,MAAM,iBAAiB,GAAG,CAAC,aAAa,EAAE,iBAAiB,CAAU,CAAC;AACtE,MAAM,oBAAoB,GAAG,IAAI,GAAG,CAAS,iBAAiB,CAAC,CAAC;AAoBhE;;;;GAIG;AACH,MAAM,UAAU,6BAA6B,CAAC,IAAI,GAAG,EAAE,CAAC,OAAO,EAAE;IAC/D,OAAO;QACL;YACE,EAAE,EAAE,aAAa;YACjB,KAAK,EAAE,aAAa;YACpB,GAAG,EAAE,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,CAAC;SACtD;QACD;YACE,EAAE,EAAE,KAAK;YACT,KAAK,EAAE,mBAAmB;YAC1B,GAAG,EAAE,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU,CAAC;SAChE;KACF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAAC,IAAc;IACjD,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,EAAE,CACpC,GAAG,KAAK,SAAS,CAAC,CAAC,CAAC,CAAC,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CACnD,CAAC;IACF,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;QAC3B,IAAI,oBAAoB,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;YAClC,IAAI,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS;YAClC,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACtB,CAAC;QACD,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAChB,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,mBAAmB,CAAC,IAAc;IAChD,MAAM,UAAU,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;IAC9C,IAAI,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC;QAAE,OAAO,KAAK,CAAC;IAE/C,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,UAAU,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QAC3C,MAAM,KAAK,GAAG,UAAU,CAAC,CAAC,CAAE,CAAC;QAC7B,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,IAAI;YAAE,SAAS;QACpD,CAAC,IAAI,CAAC,CAAC;QACP,OAAO,CAAC,GAAG,UAAU,CAAC,MAAM,IAAI,CAAC,UAAU,CAAC,CAAC,CAAE,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAChE,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAE,CAAC,CAAC;YAC5B,CAAC,IAAI,CAAC,CAAC;QACT,CAAC;QACD,CAAC,IAAI,CAAC,CAAC;IACT,CAAC;IACD,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;AAC9C,CAAC;AAED,gEAAgE;AAChE,MAAM,UAAU,gCAAgC,CAC9C,OAAiB,EAAE,EACnB,IAAI,GAAG,EAAE,CAAC,OAAO,EAAE;IAEnB,MAAM,GAAG,GAAG,6BAA6B,CAAC,IAAI,CAAC,CAAC;IAChD,MAAM,QAAQ,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;IAC3C,IAAI,QAAQ,KAAK,KAAK;QAAE,OAAO,GAAG,CAAC;IAEnC,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,KAAK,MAAM,KAAK,IAAI,QAAQ,EAAE,CAAC;QAC7B,IAAI,KAAK,KAAK,aAAa;YAAE,IAAI,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;QACrD,IAAI,KAAK,KAAK,SAAS,IAAI,oBAAoB,CAAC,GAAG,CAAC,KAAK,CAAC;YAAE,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC9E,CAAC;IACD,OAAO,GAAG,CAAC,MAAM,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;AACrD,CAAC;AAED,SAAS,YAAY;IACnB,MAAM,MAAM,GAAG,SAAS,CAAC,KAAK,EAAE,CAAC,QAAQ,EAAE,WAAW,CAAC,EAAE;QACvD,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,OAAO,CAAC,IAAc,EAAE,GAAG,KAAe;IACjD,OAAO,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC;AAC7C,CAAC;AAED,SAAS,mBAAmB,CAAC,IAAc;IACzC,MAAM,UAAU,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,OAAO,CAAC,UAAU,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IACnE,MAAM,MAAM,GAAG,SAAS,CACtB,KAAK,EACL,CAAC,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,GAAG,UAAU,CAAC,EACxE;QACE,KAAK,EAAE,SAAS;QAChB,QAAQ,EAAE,MAAM;KACjB,CACF,CAAC;IACF,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,kBAAkB,CAAC,OAAiB,EAAE;IAC7C,MAAM,UAAU,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,OAAO,CAAC,UAAU,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IACnE,MAAM,GAAG,GAAG,OAAO,CAAC,UAAU,EAAE,IAAI,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IAC7D,MAAM,MAAM,GAAG,SAAS,CACtB,KAAK,EACL,CAAC,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,GAAG,MAAM,EAAE,GAAG,GAAG,EAAE,GAAG,UAAU,CAAC,EAClE;QACE,KAAK,EAAE,SAAS;QAChB,QAAQ,EAAE,MAAM;KACjB,CACF,CAAC;IACF,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAc;IAC3C,MAAM,UAAU,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,OAAO,CAAC,UAAU,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IACnE,MAAM,MAAM,GAAG,SAAS,CACtB,KAAK,EACL,CAAC,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,GAAG,MAAM,EAAE,GAAG,UAAU,CAAC,EAC1D;QACE,KAAK,EAAE,SAAS;QAChB,QAAQ,EAAE,MAAM;KACjB,CACF,CAAC;IACF,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,aAAa,CAAC,MAAc,EAAE,OAAe;IACpD,EAAE,CAAC,SAAS,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC3C,KAAK,MAAM,KAAK,IAAI,EAAE,CAAC,WAAW,CAAC,MAAM,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,EAAE,CAAC;QACpE,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;QAC3C,MAAM,EAAE,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;QAC1C,IAAI,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;YACxB,aAAa,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;QAC1B,CAAC;aAAM,IAAI,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC;YAC1B,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;QAC5B,CAAC;IACH,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB;IACzB,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,YAAY,EAAE,cAAc,CAAC,EAAE,MAAM,CAAC,CAEtF,CAAC;IACF,OAAO,GAAG,CAAC,OAAO,CAAC;AACrB,CAAC;AAED,SAAS,aAAa,CAAC,YAAsB,EAAE;IAC7C,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC,EAAE,CAAC;QAClC,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,QAAQ;YAChB,OAAO,EAAE,iCAAiC,aAAa,EAAE;SAC1D,CAAC;IACJ,CAAC;IAED,MAAM,OAAO,GAAG,gCAAgC,CAAC,SAAS,CAAC,CAAC;IAC5D,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzB,MAAM,OAAO,GACX,uGAAuG,CAAC;QAC1G,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;QACrC,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;IAClD,CAAC;IAED,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;IACrC,MAAM,SAAS,GAAa,EAAE,CAAC;IAC/B,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,8EAA8E;QAC9E,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACxD,aAAa,CAAC,iBAAiB,EAAE,MAAM,CAAC,GAAG,CAAC,CAAC;QAC7C,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,UAAU,CAAC,CAAC;QACrD,MAAM,IAAI,GAAG,8BAA8B,OAAO,OAAO,MAAM,CAAC,KAAK,KAAK,UAAU,EAAE,CAAC;QACvF,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,IAAI,IAAI,CAAC,CAAC;QAClC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACvB,CAAC;IAED,OAAO;QACL,EAAE,EAAE,IAAI;QACR,MAAM,EAAE,QAAQ;QAChB,OAAO,EAAE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC;KAC9B,CAAC;AACJ,CAAC;AAED,SAAS,eAAe,CAAC,YAAsB,EAAE;IAC/C,MAAM,OAAO,GAAG,gCAAgC,CAAC,SAAS,CAAC,CAAC;IAC5D,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzB,MAAM,OAAO,GACX,yGAAyG,CAAC;QAC5G,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;QACrC,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;IAClD,CAAC;IAED,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;YAC/B,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;YAC3B,SAAS;QACX,CAAC;QACD,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC3C,MAAM,IAAI,GAAG,gCAAgC,MAAM,CAAC,KAAK,KAAK,MAAM,CAAC,GAAG,EAAE,CAAC;QAC3E,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,IAAI,IAAI,CAAC,CAAC;QAClC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACrB,CAAC;IAED,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzB,MAAM,OAAO,GAAG,2BAA2B,CAAC;QAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;QACrC,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;IACjD,CAAC;IAED,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,sBAAsB,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACrE,CAAC;IAED,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;AACrE,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAAC,YAAsB,EAAE;IACtD,IAAI,YAAY,EAAE,EAAE,CAAC;QACnB,MAAM,EAAE,GAAG,mBAAmB,CAAC,SAAS,CAAC,CAAC;QAC1C,OAAO;YACL,EAAE;YACF,MAAM,EAAE,YAAY;YACpB,OAAO,EAAE,EAAE,CAAC,CAAC,CAAC,0CAA0C,CAAC,CAAC,CAAC,2BAA2B;SACvF,CAAC;IACJ,CAAC;IAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,wFAAwF,CACzF,CAAC;IACF,OAAO,aAAa,CAAC,SAAS,CAAC,CAAC;AAClC,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc,CAAC,YAAsB,EAAE;IACrD,IAAI,YAAY,EAAE,EAAE,CAAC;QACnB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,8CAA8C,CAAC,CAAC;QACrE,IAAI,kBAAkB,CAAC,SAAS,CAAC,EAAE,CAAC;YAClC,OAAO;gBACL,EAAE,EAAE,IAAI;gBACR,MAAM,EAAE,YAAY;gBACpB,OAAO,EAAE,wCAAwC;aAClD,CAAC;QACJ,CAAC;QAED,yEAAyE;QACzE,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;QAC/E,MAAM,IAAI,GAAG,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,wBAAwB,CAAC;QACzE,MAAM,EAAE,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACrC,OAAO;YACL,EAAE;YACF,MAAM,EAAE,YAAY;YACpB,OAAO,EAAE,EAAE;gBACT,CAAC,CAAC,4CAA4C;gBAC9C,CAAC,CAAC,wCAAwC;SAC7C,CAAC;IACJ,CAAC;IAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,kFAAkF,CACnF,CAAC;IACF,OAAO,aAAa,CAAC,SAAS,CAAC,CAAC;AAClC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,YAAsB,EAAE;IACxD,IAAI,YAAY,EAAE,EAAE,CAAC;QACnB,MAAM,EAAE,GAAG,qBAAqB,CAAC,SAAS,CAAC,CAAC;QAC5C,OAAO;YACL,EAAE;YACF,MAAM,EAAE,YAAY;YACpB,OAAO,EAAE,EAAE,CAAC,CAAC,CAAC,wCAAwC,CAAC,CAAC,CAAC,0BAA0B;SACpF,CAAC;IACJ,CAAC;IAED,OAAO,eAAe,CAAC,SAAS,CAAC,CAAC;AACpC,CAAC;AAED,MAAM,UAAU,YAAY,CAAC,YAAsB,EAAE;IACnD,MAAM,MAAM,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC;IAC1C,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAClC,CAAC;AAED,MAAM,UAAU,cAAc,CAAC,YAAsB,EAAE;IACrD,MAAM,MAAM,GAAG,iBAAiB,CAAC,SAAS,CAAC,CAAC;IAC5C,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAClC,CAAC"}
|
package/dist/types.d.ts
CHANGED
|
@@ -48,7 +48,7 @@ export interface ParsedArgs {
|
|
|
48
48
|
command: "review" | "loop" | "models" | "install" | "install-skill" | "uninstall-skill" | "update";
|
|
49
49
|
/** For `install`: run `pi install npm:@zephyrdeng/pi-review` */
|
|
50
50
|
installPi?: boolean;
|
|
51
|
-
/** For `install`: run agent skill install (skills CLI / Claude fallback) */
|
|
51
|
+
/** For `install`: run agent skill install (skills CLI / Claude + agy direct fallback) */
|
|
52
52
|
installAgents?: boolean;
|
|
53
53
|
extraArgs?: string[];
|
|
54
54
|
mode: string;
|
|
@@ -121,7 +121,7 @@ export interface ReviewPreset {
|
|
|
121
121
|
}
|
|
122
122
|
export interface VerdictInfo {
|
|
123
123
|
verdict: Verdict;
|
|
124
|
-
verdictSource: "parsed" | "fallback" | "runtime_error";
|
|
124
|
+
verdictSource: "parsed" | "fallback" | "runtime_error" | "config_error";
|
|
125
125
|
parseError?: string;
|
|
126
126
|
}
|
|
127
127
|
/**
|
|
@@ -133,6 +133,15 @@ export interface VerdictInfo {
|
|
|
133
133
|
* require a bump on their own; bump only for a breaking shape change.
|
|
134
134
|
*/
|
|
135
135
|
export declare const REVIEW_META_VERSION: 1;
|
|
136
|
+
/**
|
|
137
|
+
* Machine-readable hint that the requested provider only exists with host Pi
|
|
138
|
+
* extensions loaded, so the run must be retried with
|
|
139
|
+
* `PI_REVIEW_CHILD_EXTENSIONS=1` (issue #8).
|
|
140
|
+
*/
|
|
141
|
+
export interface ExtensionHint {
|
|
142
|
+
provider: string;
|
|
143
|
+
availableViaExtension: boolean;
|
|
144
|
+
}
|
|
136
145
|
export interface ReviewMeta extends StructuredReviewResult {
|
|
137
146
|
/** Schema discriminator for `PI_REVIEW_META_JSON`; see `REVIEW_META_VERSION`. */
|
|
138
147
|
metaVersion: typeof REVIEW_META_VERSION;
|
|
@@ -144,6 +153,13 @@ export interface ReviewMeta extends StructuredReviewResult {
|
|
|
144
153
|
/** Token usage from the child pi --mode json stream, when available. */
|
|
145
154
|
usage?: TokenUsage;
|
|
146
155
|
sessionHandle?: string;
|
|
156
|
+
/**
|
|
157
|
+
* Present when the requested provider only exists with host Pi extensions
|
|
158
|
+
* loaded, but the review child runs with --no-extensions (issue #8). The
|
|
159
|
+
* host agent should re-run with `PI_REVIEW_CHILD_EXTENSIONS=1` (or pass it
|
|
160
|
+
* in the environment) before retrying.
|
|
161
|
+
*/
|
|
162
|
+
extensionHint?: ExtensionHint;
|
|
147
163
|
}
|
|
148
164
|
export interface ReviewRunResult {
|
|
149
165
|
meta: ReviewMeta;
|
|
@@ -252,6 +268,12 @@ export interface PanelAggregationResult extends StructuredReviewResult, PanelFie
|
|
|
252
268
|
}
|
|
253
269
|
/** Review metadata for a panel evaluation; additive over ReviewMeta. */
|
|
254
270
|
export interface PanelReviewMeta extends ReviewMeta, PanelFields {
|
|
271
|
+
/**
|
|
272
|
+
* All extension-only providers that blocked reviewers before spawn
|
|
273
|
+
* (issue #8), deduplicated by provider. Unlike single-review runs a panel
|
|
274
|
+
* may span several extension providers, so every hint is preserved here.
|
|
275
|
+
*/
|
|
276
|
+
extensionHints?: ExtensionHint[];
|
|
255
277
|
}
|
|
256
278
|
/** A reviewer's structured submission fed to the pure aggregation seam. */
|
|
257
279
|
export interface ReviewerSubmission {
|
|
@@ -265,6 +287,8 @@ export interface ReviewerSubmission {
|
|
|
265
287
|
usage?: TokenUsage;
|
|
266
288
|
durationMs: number;
|
|
267
289
|
result: StructuredReviewResult;
|
|
290
|
+
/** Extension-only provider hint, when this reviewer was blocked before spawn (issue #8). */
|
|
291
|
+
extensionHint?: ReviewMeta["extensionHint"];
|
|
268
292
|
}
|
|
269
293
|
/** One reviewer definition inside a named panel preset. */
|
|
270
294
|
export interface PanelReviewerSpec {
|
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,eAAO,MAAM,eAAe,8DAA+D,CAAC;AAC5F,MAAM,MAAM,YAAY,GAAG,CAAC,OAAO,eAAe,CAAC,CAAC,MAAM,CAAC,CAAC;AAE5D,MAAM,WAAW,aAAa;IAC5B,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB;;;;;;;OAOG;IACH,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,oGAAoG;IACpG,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB;;;;;OAKG;IACH,QAAQ,CAAC,EAAE,eAAe,CAAC;CAC5B;AAED,oFAAoF;AACpF,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;;;OAKG;IACH,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CAC3B;AAED,MAAM,WAAW,sBAAuB,SAAQ,WAAW;IACzD,MAAM,EAAE,YAAY,CAAC;IACrB,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,eAAe,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,QAAQ,GAAG,MAAM,GAAG,QAAQ,GAAG,SAAS,GAAG,eAAe,GAAG,iBAAiB,GAAG,QAAQ,CAAC;IACnG,gEAAgE;IAChE,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,
|
|
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,eAAO,MAAM,eAAe,8DAA+D,CAAC;AAC5F,MAAM,MAAM,YAAY,GAAG,CAAC,OAAO,eAAe,CAAC,CAAC,MAAM,CAAC,CAAC;AAE5D,MAAM,WAAW,aAAa;IAC5B,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,OAAO,CAAC;IACpB;;;;;;;OAOG;IACH,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,oGAAoG;IACpG,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB;;;;;OAKG;IACH,QAAQ,CAAC,EAAE,eAAe,CAAC;CAC5B;AAED,oFAAoF;AACpF,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;;;OAKG;IACH,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CAC3B;AAED,MAAM,WAAW,sBAAuB,SAAQ,WAAW;IACzD,MAAM,EAAE,YAAY,CAAC;IACrB,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,eAAe,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,QAAQ,GAAG,MAAM,GAAG,QAAQ,GAAG,SAAS,GAAG,eAAe,GAAG,iBAAiB,GAAG,QAAQ,CAAC;IACnG,gEAAgE;IAChE,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,yFAAyF;IACzF,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;;;;;OAKG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,kFAAkF;IAClF,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,yDAAyD;IACzD,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,sEAAsE;IACtE,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B;;;OAGG;IACH,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,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;IAClB,yEAAyE;IACzE,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,wCAAwC;IACxC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,qEAAqE;IACrE,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,qDAAqD;IACrD,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,6DAA6D;IAC7D,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,2CAA2C;IAC3C,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB;;;OAGG;IACH,cAAc,CAAC,EAAE,MAAM,EAAE,CAAC;IAC1B,oEAAoE;IACpE,YAAY,CAAC,EAAE,cAAc,CAAC;IAC9B,mGAAmG;IACnG,EAAE,CAAC,EAAE,KAAK,CAAC;IACX,kFAAkF;IAClF,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,2FAA2F;IAC3F,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,0GAA0G;IAC1G,MAAM,CAAC,EAAE,OAAO,CAAC;CAClB;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,GAAG,cAAc,CAAC;IACxE,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,mBAAmB,EAAG,CAAU,CAAC;AAE9C;;;;GAIG;AACH,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,qBAAqB,EAAE,OAAO,CAAC;CAChC;AAED,MAAM,WAAW,UAAW,SAAQ,sBAAsB;IACxD,iFAAiF;IACjF,WAAW,EAAE,OAAO,mBAAmB,CAAC;IACxC,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,gGAAgG;IAChG,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,wEAAwE;IACxE,KAAK,CAAC,EAAE,UAAU,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;;;OAKG;IACH,aAAa,CAAC,EAAE,aAAa,CAAC;CAC/B;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,UAAU,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,kGAAkG;IAClG,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,4EAA4E;IAC5E,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;CAC/B;AAMD,eAAO,MAAM,kBAAkB,qDAAsD,CAAC;AACtF,MAAM,MAAM,eAAe,GAAG,CAAC,OAAO,kBAAkB,CAAC,CAAC,MAAM,CAAC,CAAC;AAElE,2EAA2E;AAC3E,MAAM,WAAW,UAAU;IACzB,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,mEAAmE;AACnE,eAAO,MAAM,aAAa,IAAI,CAAC;AAE/B,yDAAyD;AACzD,eAAO,MAAM,qBAAqB,yCAA0C,CAAC;AAE7E,mEAAmE;AACnE,eAAO,MAAM,uBAAuB,IAAI,CAAC;AAEzC,iFAAiF;AACjF,eAAO,MAAM,mCAAmC,MAAM,CAAC;AAEvD,mFAAmF;AACnF,MAAM,WAAW,aAAa;IAC5B,oEAAoE;IACpE,EAAE,EAAE,MAAM,CAAC;IACX,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,aAAa,CAAC;CACxB;AAED;;;;GAIG;AACH,MAAM,WAAW,cAAc;IAC7B,gEAAgE;IAChE,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,EAAE,OAAO,CAAC;IACnB,mEAAmE;IACnE,YAAY,EAAE,MAAM,CAAC;IACrB,6DAA6D;IAC7D,sBAAsB,EAAE,MAAM,CAAC;IAC/B,qBAAqB,EAAE,MAAM,EAAE,CAAC;IAChC,gBAAgB,EAAE,MAAM,EAAE,CAAC;CAC5B;AAED,MAAM,MAAM,WAAW,GAAG,SAAS,GAAG,aAAa,GAAG,SAAS,CAAC;AAEhE,wEAAwE;AACxE,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,qFAAqF;IACrF,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,qFAAqF;IACrF,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,UAAU,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,YAAY,CAAC;IACrB,OAAO,EAAE,OAAO,CAAC;IACjB,aAAa,EAAE,WAAW,CAAC,eAAe,CAAC,CAAC;IAC5C,6EAA6E;IAC7E,WAAW,EAAE,OAAO,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,kFAAkF;AAClF,MAAM,WAAW,WAAW;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,eAAe,EAAE,eAAe,CAAC;IACjC,kBAAkB,EAAE,MAAM,CAAC;IAC3B,WAAW,EAAE,WAAW,CAAC;IACzB,iBAAiB,EAAE,cAAc,EAAE,CAAC;IACpC,UAAU,EAAE,cAAc,EAAE,CAAC;IAC7B,SAAS,EAAE,eAAe,EAAE,CAAC;IAC7B,gBAAgB,EAAE,OAAO,CAAC;IAC1B,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC9B;;;;;;;OAOG;IACH,cAAc,CAAC,EAAE,aAAa,EAAE,CAAC;CAClC;AAED,mDAAmD;AACnD,MAAM,WAAW,sBAAuB,SAAQ,sBAAsB,EAAE,WAAW;CAAG;AAEtF,wEAAwE;AACxE,MAAM,WAAW,eAAgB,SAAQ,UAAU,EAAE,WAAW;IAC9D;;;;OAIG;IACH,cAAc,CAAC,EAAE,aAAa,EAAE,CAAC;CAClC;AAED,2EAA2E;AAC3E,MAAM,WAAW,kBAAkB;IACjC,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,qFAAqF;IACrF,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,qFAAqF;IACrF,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,UAAU,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,sBAAsB,CAAC;IAC/B,4FAA4F;IAC5F,aAAa,CAAC,EAAE,UAAU,CAAC,eAAe,CAAC,CAAC;CAC7C;AAED,2DAA2D;AAC3D,MAAM,WAAW,iBAAiB;IAChC,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,mCAAmC;AACnC,MAAM,WAAW,WAAW;IAC1B,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,iBAAiB,EAAE,CAAC;IAC/B,SAAS,CAAC,EAAE,eAAe,CAAC;IAC5B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB"}
|
package/dist/types.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,QAAQ,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,qBAAqB,EAAE,SAAS,CAAU,CAAC;AAGlG,MAAM,CAAC,MAAM,eAAe,GAAG,CAAC,OAAO,EAAE,cAAc,EAAE,aAAa,EAAE,SAAS,CAAU,CAAC;AAmI5F;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,CAAU,CAAC;
|
|
1
|
+
{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,QAAQ,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,qBAAqB,EAAE,SAAS,CAAU,CAAC;AAGlG,MAAM,CAAC,MAAM,eAAe,GAAG,CAAC,OAAO,EAAE,cAAc,EAAE,aAAa,EAAE,SAAS,CAAU,CAAC;AAmI5F;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,CAAU,CAAC;AA8C9C,8EAA8E;AAC9E,eAAe;AACf,8EAA8E;AAE9E,MAAM,CAAC,MAAM,kBAAkB,GAAG,CAAC,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,WAAW,CAAU,CAAC;AActF,mEAAmE;AACnE,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,CAAC;AAE/B,yDAAyD;AACzD,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,CAAU,CAAC;AAE7E,mEAAmE;AACnE,MAAM,CAAC,MAAM,uBAAuB,GAAG,CAAC,CAAC;AAEzC,iFAAiF;AACjF,MAAM,CAAC,MAAM,mCAAmC,GAAG,GAAG,CAAC"}
|
|
@@ -4,6 +4,7 @@ import { createPanelViewState, reducePanelEvent } from "@zephyrdeng/pi-review";
|
|
|
4
4
|
import {
|
|
5
5
|
buildPanelResultContent,
|
|
6
6
|
expandHint,
|
|
7
|
+
normalizePanelToolParams,
|
|
7
8
|
panelToolParamError,
|
|
8
9
|
PanelResultView,
|
|
9
10
|
registerPanelReviewTool,
|
|
@@ -18,11 +19,24 @@ const theme = {
|
|
|
18
19
|
test("panel tool boundary rejects single-reviewer and out-of-range widths", () => {
|
|
19
20
|
assert.match(panelToolParamError({ target: "@src", reviewers: 1 }) ?? "", /between 2 and 8/);
|
|
20
21
|
assert.match(panelToolParamError({ target: "@src", reviewers: 9 }) ?? "", /between 2 and 8/);
|
|
21
|
-
assert.
|
|
22
|
+
assert.equal(panelToolParamError({ target: "@src", panel: "code-experts", reviewers: 2 }), undefined);
|
|
22
23
|
assert.equal(panelToolParamError({ target: "@src", reviewers: 2 }), undefined);
|
|
23
24
|
assert.equal(panelToolParamError({ target: "@src", panel: "code-experts" }), undefined);
|
|
24
25
|
});
|
|
25
26
|
|
|
27
|
+
test("panel + reviewers prefers panel instead of hard-failing", () => {
|
|
28
|
+
const normalized = normalizePanelToolParams({
|
|
29
|
+
target: "@src",
|
|
30
|
+
panel: "code-experts",
|
|
31
|
+
reviewers: 3,
|
|
32
|
+
});
|
|
33
|
+
assert.ok(!("error" in normalized));
|
|
34
|
+
assert.equal(normalized.panel, "code-experts");
|
|
35
|
+
assert.equal(normalized.reviewers, undefined);
|
|
36
|
+
assert.match(normalized.note ?? "", /ignored reviewers=3/);
|
|
37
|
+
assert.match(normalized.note ?? "", /code-experts/);
|
|
38
|
+
});
|
|
39
|
+
|
|
26
40
|
test("registered pi_review rejects reviewers=1 before spawning the panel CLI", async () => {
|
|
27
41
|
let registered: { execute: (...args: any[]) => Promise<any> } | undefined;
|
|
28
42
|
registerPanelReviewTool({
|
package/extensions/panel-tool.ts
CHANGED
|
@@ -13,6 +13,7 @@ import {
|
|
|
13
13
|
formatTokens,
|
|
14
14
|
formatUsage,
|
|
15
15
|
reducePanelEvent,
|
|
16
|
+
resolveExclusivePanelShape,
|
|
16
17
|
spawnStreamingChild,
|
|
17
18
|
type PanelReviewMeta,
|
|
18
19
|
type PanelViewState,
|
|
@@ -147,15 +148,21 @@ function reviewerSummaryLines(state: PanelViewState): string[] {
|
|
|
147
148
|
* Full panel conclusion for the parent LLM and expanded TUI.
|
|
148
149
|
* Pure plain text / ASCII only — Pi does not render markdown in tool results.
|
|
149
150
|
*/
|
|
150
|
-
export function buildPanelResultContent(state: PanelViewState, error?: string): string {
|
|
151
|
+
export function buildPanelResultContent(state: PanelViewState, error?: string, note?: string): string {
|
|
151
152
|
const meta = state.meta;
|
|
152
|
-
if (!meta && !error)
|
|
153
|
-
|
|
153
|
+
if (!meta && !error) {
|
|
154
|
+
return note ? `${PANEL_TOOL_DISPLAY_NAME}: ${note}\n\nPanel completed: unknown` : "Panel completed: unknown";
|
|
155
|
+
}
|
|
156
|
+
if (!meta) {
|
|
157
|
+
const body = error ?? "Panel completed: unknown";
|
|
158
|
+
return note ? `${PANEL_TOOL_DISPLAY_NAME}: ${note}\n\n${body}` : body;
|
|
159
|
+
}
|
|
154
160
|
|
|
155
161
|
const findings = formatFindingsPlain(meta);
|
|
156
162
|
const ascii = formatPanelMetaAscii(meta);
|
|
157
163
|
const summaries = reviewerSummaryLines(state);
|
|
158
164
|
const parts = [
|
|
165
|
+
note ? `${PANEL_TOOL_DISPLAY_NAME}: ${note}` : "",
|
|
159
166
|
findings,
|
|
160
167
|
ascii,
|
|
161
168
|
summaries.length ? `Reviewer summaries:\n${summaries.join("\n")}` : "",
|
|
@@ -259,15 +266,35 @@ export type PanelToolParams = {
|
|
|
259
266
|
reviewers?: number;
|
|
260
267
|
};
|
|
261
268
|
|
|
262
|
-
|
|
263
|
-
|
|
269
|
+
export type NormalizedPanelToolParams = {
|
|
270
|
+
panel?: string;
|
|
271
|
+
reviewers?: number;
|
|
272
|
+
/** Set when both panel and reviewers were provided and reviewers was dropped. */
|
|
273
|
+
note?: string;
|
|
274
|
+
};
|
|
275
|
+
|
|
276
|
+
/**
|
|
277
|
+
* Normalize agent tool args. Named presets own width/roles; if both `panel` and
|
|
278
|
+
* `reviewers` are set, prefer panel (common host habit) instead of hard-failing.
|
|
279
|
+
*/
|
|
280
|
+
export function normalizePanelToolParams(params: PanelToolParams): NormalizedPanelToolParams | { error: string } {
|
|
264
281
|
if (params.reviewers !== undefined) {
|
|
265
282
|
if (!Number.isSafeInteger(params.reviewers) || params.reviewers < 2 || params.reviewers > 8) {
|
|
266
|
-
return "reviewers must be an integer between 2 and 8; use the shell CLI for a single reviewer";
|
|
283
|
+
return { error: "reviewers must be an integer between 2 and 8; use the shell CLI for a single reviewer" };
|
|
267
284
|
}
|
|
268
|
-
if (params.panel) return "panel cannot be combined with reviewers";
|
|
269
285
|
}
|
|
270
|
-
|
|
286
|
+
const shape = resolveExclusivePanelShape(params);
|
|
287
|
+
const note =
|
|
288
|
+
shape.ignoredReviewers !== undefined
|
|
289
|
+
? `ignored reviewers=${shape.ignoredReviewers}; panel "${shape.panel}" defines the reviewer set`
|
|
290
|
+
: undefined;
|
|
291
|
+
return { panel: shape.panel, reviewers: shape.reviewers, ...(note ? { note } : {}) };
|
|
292
|
+
}
|
|
293
|
+
|
|
294
|
+
/** Runtime guard for direct tool calls; schema validation is not the only boundary. */
|
|
295
|
+
export function panelToolParamError(params: PanelToolParams): string | undefined {
|
|
296
|
+
const normalized = normalizePanelToolParams(params);
|
|
297
|
+
return "error" in normalized ? normalized.error : undefined;
|
|
271
298
|
}
|
|
272
299
|
|
|
273
300
|
export function registerPanelReviewTool(pi: ExtensionAPI): void {
|
|
@@ -279,8 +306,8 @@ export function registerPanelReviewTool(pi: ExtensionAPI): void {
|
|
|
279
306
|
parameters: Type.Object({
|
|
280
307
|
target: Type.String({ description: "Review target as given by the user, such as @src, @src/foo.ts, or a free-text brief. Directories stay tool path targets; only real files are attached." }),
|
|
281
308
|
mode: Type.Optional(Type.String({ description: "Review mode; defaults to code." })),
|
|
282
|
-
panel: Type.Optional(Type.String({ description: "Named panel preset; defaults to code-experts when reviewers is omitted." })),
|
|
283
|
-
reviewers: Type.Optional(Type.Number({ minimum: 2, maximum: 8, description: "Independent reviewer count (2-8).
|
|
309
|
+
panel: Type.Optional(Type.String({ description: "Named panel preset; defaults to code-experts when reviewers is omitted. If both panel and reviewers are set, panel wins." })),
|
|
310
|
+
reviewers: Type.Optional(Type.Number({ minimum: 2, maximum: 8, description: "Independent reviewer count (2-8). Omit when panel is set (panel wins if both appear). Single review uses the shell CLI." })),
|
|
284
311
|
reviewerModels: Type.Optional(Type.Array(Type.String(), { description: "Per-reviewer models as id=provider/model[:thinking] (e.g. r1=openai/gpt-5.6-sol:low). Trailing :thinking wins over shared thinking." })),
|
|
285
312
|
consensus: Type.Optional(Type.String({ description: "any | quorum | majority | unanimous" })),
|
|
286
313
|
minAgree: Type.Optional(Type.Number({ description: "Quorum threshold when consensus=quorum" })),
|
|
@@ -325,20 +352,21 @@ export function registerPanelReviewTool(pi: ExtensionAPI): void {
|
|
|
325
352
|
},
|
|
326
353
|
});
|
|
327
354
|
// Preserve the full panel configuration and enforce the panel-only tool boundary.
|
|
328
|
-
const
|
|
329
|
-
if (
|
|
355
|
+
const normalized = normalizePanelToolParams(params);
|
|
356
|
+
if ("error" in normalized) {
|
|
330
357
|
setAmbientStatus(toolCtx, undefined);
|
|
331
358
|
return {
|
|
332
|
-
content: [{ type: "text", text: `${PANEL_TOOL_DISPLAY_NAME}: ${
|
|
333
|
-
details: { state, error:
|
|
359
|
+
content: [{ type: "text", text: `${PANEL_TOOL_DISPLAY_NAME}: ${normalized.error}` }],
|
|
360
|
+
details: { state, error: normalized.error },
|
|
334
361
|
isError: true,
|
|
335
362
|
};
|
|
336
363
|
}
|
|
364
|
+
const shapeNote = normalized.note;
|
|
337
365
|
const args = [cliPath, "--mode", params.mode ?? "code", "--output-format", "events-jsonl"];
|
|
338
|
-
if (
|
|
339
|
-
args.push("--reviewers", String(
|
|
366
|
+
if (normalized.reviewers !== undefined) {
|
|
367
|
+
args.push("--reviewers", String(normalized.reviewers));
|
|
340
368
|
} else {
|
|
341
|
-
args.push("--panel",
|
|
369
|
+
args.push("--panel", normalized.panel ?? "code-experts");
|
|
342
370
|
}
|
|
343
371
|
for (const mapping of params.reviewerModels ?? []) args.push("--reviewer-model", mapping);
|
|
344
372
|
if (params.consensus) args.push("--consensus", params.consensus);
|
|
@@ -376,14 +404,16 @@ export function registerPanelReviewTool(pi: ExtensionAPI): void {
|
|
|
376
404
|
const details: PanelToolDetails = { state, ...(error ? { error } : {}) };
|
|
377
405
|
return {
|
|
378
406
|
// Parent LLM must receive the full conclusion; details remain the TUI rendering source.
|
|
379
|
-
content: [{ type: "text", text: buildPanelResultContent(state, error) }],
|
|
407
|
+
content: [{ type: "text", text: buildPanelResultContent(state, error, shapeNote) }],
|
|
380
408
|
details,
|
|
381
409
|
...(error ? { isError: true } : {}),
|
|
382
410
|
};
|
|
383
411
|
},
|
|
384
412
|
renderCall(args, theme, context) {
|
|
385
413
|
const text = context.lastComponent instanceof Text ? context.lastComponent : new Text("", 0, 0);
|
|
386
|
-
const
|
|
414
|
+
const shape = resolveExclusivePanelShape(args);
|
|
415
|
+
const width =
|
|
416
|
+
shape.reviewers !== undefined ? `reviewers=${shape.reviewers}` : (shape.panel ?? "code-experts");
|
|
387
417
|
text.setText(theme.fg("toolTitle", theme.bold(`${PANEL_TOOL_DISPLAY_NAME} `)) + theme.fg("muted", `${width} · ${args.mode ?? "code"} · ${args.target}`));
|
|
388
418
|
return text;
|
|
389
419
|
},
|
|
@@ -77,7 +77,7 @@ export const FLAG_HINTS: Record<string, string> = {
|
|
|
77
77
|
"--max-rounds": "Loop hard budget: how many review gates (not reviewer count)",
|
|
78
78
|
"--until": "Loop stop goal: clean",
|
|
79
79
|
"--reviewers": "Panel width: independent reviewers per gate (1-8)",
|
|
80
|
-
"--panel": "Named expert panel preset (
|
|
80
|
+
"--panel": "Named expert panel preset (if both set with --reviewers, --panel wins)",
|
|
81
81
|
"--reviewer-model": "Per-reviewer model: pick rK= then model (repeatable)",
|
|
82
82
|
"--consensus": "any | quorum | majority | unanimous",
|
|
83
83
|
"--min-agree": "Quorum threshold (quorum only)",
|
|
@@ -84,7 +84,8 @@ describe("parseRvArgs / validateRvParsed", () => {
|
|
|
84
84
|
assert.match(prompt, /natural-language review request/);
|
|
85
85
|
assert.match(prompt, /Do not expand directory targets into multi-file lists/);
|
|
86
86
|
assert.match(prompt, /panel="code-experts"/);
|
|
87
|
-
assert.match(prompt, /
|
|
87
|
+
assert.match(prompt, /Pass only one of panel or reviewers/);
|
|
88
|
+
assert.match(prompt, /panel wins/);
|
|
88
89
|
assert.match(prompt, /Slash commands select strategy only/);
|
|
89
90
|
assert.doesNotMatch(prompt, /@src\/panel\.ts|@src\/json-events\.ts/);
|
|
90
91
|
});
|
|
@@ -235,10 +236,19 @@ describe("parseRvArgs / validateRvParsed", () => {
|
|
|
235
236
|
]);
|
|
236
237
|
});
|
|
237
238
|
|
|
238
|
-
it("
|
|
239
|
+
it("panel + reviewers is accepted and builders prefer panel", () => {
|
|
239
240
|
const both = parseRvArgs("--reviewers 3 --panel code-experts @src", "loop");
|
|
240
|
-
assert.equal(validateRvParsed(both).ok,
|
|
241
|
+
assert.equal(validateRvParsed(both).ok, true);
|
|
242
|
+
const argv = buildPiReviewArgv(both, "@src");
|
|
243
|
+
assert.ok(argv.includes("--panel"));
|
|
244
|
+
assert.ok(argv.includes("code-experts"));
|
|
245
|
+
assert.ok(!argv.includes("--reviewers"));
|
|
246
|
+
const instruction = buildPiReviewToolCallInstruction(both, "@src");
|
|
247
|
+
assert.match(instruction, /panel="code-experts"/);
|
|
248
|
+
assert.doesNotMatch(instruction, /reviewers=3/);
|
|
249
|
+
});
|
|
241
250
|
|
|
251
|
+
it("rejects incompatible panel strategy combinations", () => {
|
|
242
252
|
const badConsensus = parseRvArgs("--reviewers 3 --consensus nope @src", "loop");
|
|
243
253
|
assert.equal(validateRvParsed(badConsensus).ok, false);
|
|
244
254
|
|
package/extensions/rv-prompts.ts
CHANGED
|
@@ -52,7 +52,7 @@ export type RvParsed = {
|
|
|
52
52
|
/** Panel width: independent reviewers for one gate (and each loop round). */
|
|
53
53
|
reviewers?: number;
|
|
54
54
|
reviewersError?: string;
|
|
55
|
-
/** Named panel preset (e.g. code-experts).
|
|
55
|
+
/** Named panel preset (e.g. code-experts). If both panel and reviewers are set, panel wins. */
|
|
56
56
|
panel?: string;
|
|
57
57
|
/** Per-reviewer model overrides: id=provider/model (repeatable). */
|
|
58
58
|
reviewerModels?: string[];
|
|
@@ -403,31 +403,31 @@ export function validateRvParsed(parsed: RvParsed): RvValidation {
|
|
|
403
403
|
if (parsed.until !== undefined && parsed.strategy !== "loop") {
|
|
404
404
|
return { ok: false, message: "--until is only valid with /rv-loop." };
|
|
405
405
|
}
|
|
406
|
-
|
|
407
|
-
return { ok: false, message: "--reviewers cannot be used with --panel" };
|
|
408
|
-
}
|
|
406
|
+
// panel + reviewers is allowed at parse time; builders prefer panel (see preferPanelShape).
|
|
409
407
|
if (parsed.reviewers !== undefined && (parsed.reviewers < 1 || parsed.reviewers > RV_MAX_REVIEWERS)) {
|
|
410
408
|
return { ok: false, message: `--reviewers must be between 1 and ${RV_MAX_REVIEWERS}` };
|
|
411
409
|
}
|
|
410
|
+
const shape = preferPanelShape(parsed);
|
|
412
411
|
const hasPanelOnlyOptions =
|
|
413
412
|
parsed.consensus !== undefined ||
|
|
414
413
|
parsed.minAgree !== undefined ||
|
|
415
414
|
parsed.consensusModel !== undefined ||
|
|
416
415
|
parsed.concurrency !== undefined ||
|
|
417
416
|
(parsed.reviewerModels?.length ?? 0) > 0;
|
|
418
|
-
if (
|
|
417
|
+
if (shape.reviewers === 1 && !shape.panel && hasPanelOnlyOptions) {
|
|
419
418
|
return { ok: false, message: "panel options require --reviewers > 1 or --panel" };
|
|
420
419
|
}
|
|
421
420
|
if (parsed.minAgree !== undefined) {
|
|
422
421
|
if (parsed.consensus && parsed.consensus !== "quorum") {
|
|
423
422
|
return { ok: false, message: "--min-agree is only valid with --consensus quorum" };
|
|
424
423
|
}
|
|
425
|
-
|
|
426
|
-
|
|
424
|
+
// Named panels resolve width later; only compare against explicit numeric width.
|
|
425
|
+
if (shape.reviewers !== undefined && parsed.minAgree > shape.reviewers) {
|
|
426
|
+
return { ok: false, message: `--min-agree ${parsed.minAgree} cannot exceed reviewers ${shape.reviewers}` };
|
|
427
427
|
}
|
|
428
428
|
}
|
|
429
|
-
if (parsed.concurrency !== undefined &&
|
|
430
|
-
return { ok: false, message: `--concurrency ${parsed.concurrency} cannot exceed reviewers ${
|
|
429
|
+
if (parsed.concurrency !== undefined && shape.reviewers !== undefined && parsed.concurrency > shape.reviewers) {
|
|
430
|
+
return { ok: false, message: `--concurrency ${parsed.concurrency} cannot exceed reviewers ${shape.reviewers}` };
|
|
431
431
|
}
|
|
432
432
|
for (const flag of FLAGS_REQUIRING_VALUE) {
|
|
433
433
|
if (parsed.target === flag) {
|
|
@@ -445,6 +445,17 @@ export function shellQuote(value: string): string {
|
|
|
445
445
|
return `'${value.replace(/'/g, `'"'"'`)}'`;
|
|
446
446
|
}
|
|
447
447
|
|
|
448
|
+
/** Prefer named panel over numeric width when both appear (matches CLI/tool soft-resolve). */
|
|
449
|
+
export function preferPanelShape(parsed: Pick<RvParsed, "panel" | "reviewers">): {
|
|
450
|
+
panel?: string;
|
|
451
|
+
reviewers?: number;
|
|
452
|
+
} {
|
|
453
|
+
if (parsed.panel !== undefined && parsed.reviewers !== undefined) {
|
|
454
|
+
return { panel: parsed.panel };
|
|
455
|
+
}
|
|
456
|
+
return { panel: parsed.panel, reviewers: parsed.reviewers };
|
|
457
|
+
}
|
|
458
|
+
|
|
448
459
|
/** Same optional flags for initial review and --continue follow-up (all optional). */
|
|
449
460
|
export function buildPiReviewArgv(parsed: RvParsed, target: string): string[] {
|
|
450
461
|
const parts = ["pi-review"];
|
|
@@ -470,8 +481,9 @@ export function buildPiReviewArgv(parsed: RvParsed, target: string): string[] {
|
|
|
470
481
|
if (parsed.model) parts.push("--model", parsed.model);
|
|
471
482
|
if (parsed.thinking) parts.push("--thinking", parsed.thinking);
|
|
472
483
|
if (parsed.noStream) parts.push("--no-stream");
|
|
473
|
-
|
|
474
|
-
if (
|
|
484
|
+
const shape = preferPanelShape(parsed);
|
|
485
|
+
if (shape.panel) parts.push("--panel", shape.panel);
|
|
486
|
+
if (shape.reviewers !== undefined) parts.push("--reviewers", String(shape.reviewers));
|
|
475
487
|
for (const mapping of parsed.reviewerModels ?? []) parts.push("--reviewer-model", mapping);
|
|
476
488
|
if (parsed.consensus) parts.push("--consensus", parsed.consensus);
|
|
477
489
|
if (parsed.minAgree !== undefined) parts.push("--min-agree", String(parsed.minAgree));
|
|
@@ -488,12 +500,13 @@ export function formatPiReviewCommandLine(parsed: RvParsed, target: string): str
|
|
|
488
500
|
|
|
489
501
|
/** Build the native pi_review tool call instruction without dropping panel config. */
|
|
490
502
|
export function buildPiReviewToolCallInstruction(parsed: RvParsed, target: string): string {
|
|
503
|
+
const shape = preferPanelShape(parsed);
|
|
491
504
|
const fields: string[] = [
|
|
492
505
|
`target=${JSON.stringify(target)}`,
|
|
493
506
|
`mode=${JSON.stringify(parsed.mode)}`,
|
|
494
507
|
];
|
|
495
|
-
if (
|
|
496
|
-
if (
|
|
508
|
+
if (shape.panel) fields.push(`panel=${JSON.stringify(shape.panel)}`);
|
|
509
|
+
if (shape.reviewers !== undefined) fields.push(`reviewers=${shape.reviewers}`);
|
|
497
510
|
if (parsed.reviewerModels?.length) fields.push(`reviewerModels=${JSON.stringify(parsed.reviewerModels)}`);
|
|
498
511
|
if (parsed.consensus) fields.push(`consensus=${JSON.stringify(parsed.consensus)}`);
|
|
499
512
|
if (parsed.minAgree !== undefined) fields.push(`minAgree=${parsed.minAgree}`);
|
|
@@ -502,10 +515,10 @@ export function buildPiReviewToolCallInstruction(parsed: RvParsed, target: strin
|
|
|
502
515
|
if (parsed.model) fields.push(`model=${JSON.stringify(parsed.model)}`);
|
|
503
516
|
if (parsed.thinking) fields.push(`thinking=${JSON.stringify(parsed.thinking)}`);
|
|
504
517
|
// Default panel when /rv has no explicit width/preset: keep code-experts.
|
|
505
|
-
if (parsed.strategy === "panel" && !
|
|
518
|
+
if (parsed.strategy === "panel" && !shape.panel && shape.reviewers === undefined) {
|
|
506
519
|
fields.push(`panel=${JSON.stringify("code-experts")}`);
|
|
507
520
|
}
|
|
508
|
-
return `Call pi_review with ${fields.join(", ")}. Treat target as a natural-language review request. Do not expand directory targets into multi-file lists.
|
|
521
|
+
return `Call pi_review with ${fields.join(", ")}. Treat target as a natural-language review request. Do not expand directory targets into multi-file lists. Pass only one of panel or reviewers (if both appear, panel wins). Keep reviewerModels/consensus/minAgree when provided.`;
|
|
509
522
|
}
|
|
510
523
|
|
|
511
524
|
export function orchestrationLocaleNote(locale: RvLocale = "en"): string {
|
package/package.json
CHANGED
|
@@ -27,7 +27,7 @@ node ../../bin/pi-review.js --help
|
|
|
27
27
|
|------|-------------------------|
|
|
28
28
|
| **Pi** (`/rv`) | New panel reviews call the native `pi_review` API tool, shown to users as **Pi Review Panel**, which launches the packaged CLI in event mode and renders live reviewer state. Continuations, kept sessions, `/rv-loop`, and explicit `--no-stream` use the shell CLI. |
|
|
29
29
|
| **Pi terminal** | Foreground `pi-review`; text deltas stream live by default. |
|
|
30
|
-
| **Claude Code / Codex / Cursor** and similar agent hosts | These hosts buffer a shell tool's output until the command exits, so a foreground run looks like a silent wait. **Panel review (`--reviewers`/`--panel`) default: `--ui web --ui-url-file <path>` + background run** — get the dashboard in front of the user: on Claude Code the CLI auto-open lands in the system default browser; on Codex open the `--ui-url-file` URL in the host's built-in browser (the CLI cannot reach a GUI there); hand over the URL as fallback either way. **Single review / `loop` default: `--progress-log <path>` + background run + tail the log** (`--ui web` requires an active panel and rejects `loop`). See [references/codex-tools.md](./references/codex-tools.md). Map skill mentions of `Bash` to your host shell tool. |
|
|
30
|
+
| **Claude Code / Codex / Cursor / agy (Antigravity)** and similar agent hosts | These hosts buffer a shell tool's output until the command exits, so a foreground run looks like a silent wait. **Panel review (`--reviewers`/`--panel`) default: `--ui web --ui-url-file <path>` + background run** — get the dashboard in front of the user: on Claude Code the CLI auto-open lands in the system default browser; on Codex open the `--ui-url-file` URL in the host's built-in browser (the CLI cannot reach a GUI there); on agy prefer the host browser / share the URL the same way; hand over the URL as fallback either way. **Single review / `loop` default: `--progress-log <path>` + background run + tail the log** (`--ui web` requires an active panel and rejects `loop`). See [references/codex-tools.md](./references/codex-tools.md). Map skill mentions of `Bash` to your host shell tool. |
|
|
31
31
|
| Scripts / CI | Foreground is fine; add `--no-stream` only when the caller must buffer until exit (cannot combine with `--progress-log`). |
|
|
32
32
|
|
|
33
33
|
`--progress-log <path>` tees a compact `--mode json` event stream to a file for outside observation and debugging (each `message_update` reduces its cumulative message snapshots to `usage` to keep the file small; add `--progress-log-raw` for the verbatim stream); it is never a prerequisite for metrics. Token usage accumulates by default, and `pi-review` writes milestone notices (`pi-review: review started`, `pi-review: tool <name> started/finished`, `pi-review: review finished`) to stderr.
|
|
@@ -1,6 +1,6 @@
|
|
|
1
|
-
# Agent host tool notes (Codex / Claude Code)
|
|
1
|
+
# Agent host tool notes (Codex / Claude Code / Cursor / agy)
|
|
2
2
|
|
|
3
|
-
Skills may use Claude Code tool names. On **Codex** or **
|
|
3
|
+
Skills may use Claude Code tool names. On **Codex**, **Claude Code**, **Cursor**, or **agy (Antigravity)**, map them to your native tools:
|
|
4
4
|
|
|
5
5
|
| Skill references | Typical host equivalent |
|
|
6
6
|
|-----------------|-------------------------|
|
|
@@ -10,13 +10,13 @@ Skills may use Claude Code tool names. On **Codex** or **Claude Code**, map them
|
|
|
10
10
|
|
|
11
11
|
This file is **pi-review–specific** and ships with the pi-review package.
|
|
12
12
|
|
|
13
|
-
## Long-running `pi-review` (default on Codex / Claude Code)
|
|
13
|
+
## Long-running `pi-review` (default on Codex / Claude Code / Cursor / agy)
|
|
14
14
|
|
|
15
15
|
Native shell tools on these hosts **buffer stdout/stderr until the command exits**. A multi-minute foreground `pi-review` run looks like a silent wait, then one large dump — never promise live foreground streaming. Which side channel to use depends on the run type: panel review gets the browser dashboard; single review and `loop` get the tailed progress log.
|
|
16
16
|
|
|
17
17
|
### Panel review: browser dashboard (default)
|
|
18
18
|
|
|
19
|
-
`--reviewers <n>` or `--panel <name>` activates panel review. On Codex, Claude Code, or
|
|
19
|
+
`--reviewers <n>` or `--panel <name>` activates panel review. On Codex, Claude Code, Cursor, or agy, default to the dashboard instead of a tailed log:
|
|
20
20
|
|
|
21
21
|
1. Run `pi-review --reviewers <n> --ui web --ui-url-file <path> -- <target>` (or `--panel <name>`) in the **background**. Pass `--no-ui-open` only when the user asked not to open a browser (CI, remote shells).
|
|
22
22
|
2. Read `<path>` shortly after launch — it is written atomically as soon as the dashboard server is ready, well before reviewers finish (poll a few times a second for a couple of seconds if the file is not there yet). Then get the dashboard in front of the user: on **Claude Code** the CLI auto-open already lands in the system default browser; on **Codex** the CLI cannot reach a GUI browser, so open the URL from `<path>` in the host's built-in browser yourself. Share the URL either way as a fallback: *"Dashboard opened in your browser (URL: `<url>`). I'll report back when the review completes."*
|
|
@@ -32,7 +32,7 @@ Native shell tools on these hosts **buffer stdout/stderr until the command exits
|
|
|
32
32
|
|
|
33
33
|
**Pi interactive (`/rv`)**: foreground `pi-review`, default streaming. Text deltas appear live on the terminal; no `--progress-log`/`--ui web` unless the user asked — Pi already renders panel progress natively.
|
|
34
34
|
|
|
35
|
-
See the parent skill
|
|
35
|
+
See the parent skill section *Default workflow by host* (Claude Code / Codex / Cursor / agy row).
|
|
36
36
|
|
|
37
37
|
### Loop closeout
|
|
38
38
|
|
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
|
|
3
3
|
Use this after `pi-review models` (or `pi-review models <search>`). **Never invent** `provider/model` IDs—only IDs that appear in that output.
|
|
4
4
|
|
|
5
|
-
## Workflow (Claude Code, Codex, Cursor, Pi parent agent)
|
|
5
|
+
## Workflow (Claude Code, Codex, Cursor, agy, Pi parent agent)
|
|
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).
|