@mrclrchtr/supi-review 2.6.0 → 2.6.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/node_modules/@mrclrchtr/supi-core/package.json +1 -1
- package/package.json +2 -2
- package/src/history/synthesize.ts +1 -5
- package/src/review.ts +0 -2
- package/src/tool/agent-review-tools.ts +0 -2
- package/src/tool/agent-review-workflow.ts +0 -5
- package/src/tool/brief-runner.ts +0 -1
- package/src/tool/review-runner.ts +0 -1
- package/src/types.ts +0 -3
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@mrclrchtr/supi-review",
|
|
3
|
-
"version": "2.6.
|
|
3
|
+
"version": "2.6.1",
|
|
4
4
|
"description": "SuPi Review extension — session-aware review command and agent tools",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"repository": {
|
|
@@ -31,7 +31,7 @@
|
|
|
31
31
|
"README.md"
|
|
32
32
|
],
|
|
33
33
|
"dependencies": {
|
|
34
|
-
"@mrclrchtr/supi-core": "2.6.
|
|
34
|
+
"@mrclrchtr/supi-core": "2.6.1"
|
|
35
35
|
},
|
|
36
36
|
"bundledDependencies": [
|
|
37
37
|
"@mrclrchtr/supi-core"
|
|
@@ -1,4 +1,3 @@
|
|
|
1
|
-
import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
|
|
2
1
|
import { listReviewInstructionBlocks } from "../target/packet.ts";
|
|
3
2
|
import { runBriefSynthesis } from "../tool/brief-runner.ts";
|
|
4
3
|
import type {
|
|
@@ -15,7 +14,6 @@ export const BRIEF_SYNTHESIS_PROMPT_VERSION = "1";
|
|
|
15
14
|
|
|
16
15
|
export interface SynthesizeReviewBriefOptions {
|
|
17
16
|
model: ReviewModelSelection;
|
|
18
|
-
modelRegistry: ModelRegistry;
|
|
19
17
|
cwd: string;
|
|
20
18
|
snapshot: ReviewSnapshot;
|
|
21
19
|
serializedContext: string;
|
|
@@ -28,13 +26,11 @@ export interface SynthesizeReviewBriefOptions {
|
|
|
28
26
|
export function synthesizeReviewBrief(
|
|
29
27
|
options: SynthesizeReviewBriefOptions,
|
|
30
28
|
): Promise<BriefSynthesisRunResult> {
|
|
31
|
-
const { model,
|
|
32
|
-
options;
|
|
29
|
+
const { model, cwd, snapshot, serializedContext, note, signal, onProgress } = options;
|
|
33
30
|
|
|
34
31
|
return runBriefSynthesis({
|
|
35
32
|
prompt: buildBriefSynthesisPrompt(snapshot, serializedContext, note),
|
|
36
33
|
model: model.model,
|
|
37
|
-
modelRegistry,
|
|
38
34
|
cwd,
|
|
39
35
|
signal,
|
|
40
36
|
onProgress,
|
package/src/review.ts
CHANGED
|
@@ -64,7 +64,6 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
|
|
|
64
64
|
(signal: AbortSignal, onProgress: (p: WidgetProgress) => void) =>
|
|
65
65
|
synthesizeReviewBrief({
|
|
66
66
|
model,
|
|
67
|
-
modelRegistry: ctx.modelRegistry,
|
|
68
67
|
cwd: ctx.cwd,
|
|
69
68
|
snapshot,
|
|
70
69
|
serializedContext,
|
|
@@ -114,7 +113,6 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
|
|
|
114
113
|
runReviewer({
|
|
115
114
|
prompt: plan.packet.prompt,
|
|
116
115
|
model: plan.model,
|
|
117
|
-
modelRegistry: ctx.modelRegistry,
|
|
118
116
|
cwd: ctx.cwd,
|
|
119
117
|
signal,
|
|
120
118
|
snapshot: plan.snapshot,
|
|
@@ -81,7 +81,6 @@ export function registerAgentReviewTools(
|
|
|
81
81
|
note,
|
|
82
82
|
serializedContext,
|
|
83
83
|
model,
|
|
84
|
-
modelRegistry: ctx.modelRegistry,
|
|
85
84
|
signal,
|
|
86
85
|
planStore,
|
|
87
86
|
onProgress: (progress) => {
|
|
@@ -136,7 +135,6 @@ export function registerAgentReviewTools(
|
|
|
136
135
|
critique: input.critique,
|
|
137
136
|
revisedBrief: input.revisedBrief,
|
|
138
137
|
reviewers: input.reviewers,
|
|
139
|
-
modelRegistry: ctx.modelRegistry,
|
|
140
138
|
signal,
|
|
141
139
|
planStore,
|
|
142
140
|
onBriefEvaluation: (evaluation) => {
|
|
@@ -1,4 +1,3 @@
|
|
|
1
|
-
import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
|
|
2
1
|
import {
|
|
3
2
|
checkReviewSnapshotFreshness,
|
|
4
3
|
fingerprintReviewSnapshot,
|
|
@@ -32,7 +31,6 @@ export interface PrepareAgentReviewWorkflowInput {
|
|
|
32
31
|
note?: string;
|
|
33
32
|
serializedContext: string;
|
|
34
33
|
model: ReviewModelSelection;
|
|
35
|
-
modelRegistry: ModelRegistry;
|
|
36
34
|
signal?: AbortSignal;
|
|
37
35
|
onProgress?: (progress: ReviewProgress) => void;
|
|
38
36
|
planStore: ReviewPlanStore;
|
|
@@ -51,7 +49,6 @@ export interface RunAgentReviewWorkflowInput {
|
|
|
51
49
|
critique: BriefCritique;
|
|
52
50
|
revisedBrief?: Omit<SynthesizedReviewBrief, "note">;
|
|
53
51
|
reviewers: ReviewerAssignment[];
|
|
54
|
-
modelRegistry: ModelRegistry;
|
|
55
52
|
signal?: AbortSignal;
|
|
56
53
|
onBriefEvaluation?: (evaluation: BriefEvaluation) => void;
|
|
57
54
|
onReviewerProgress?: (reviewerId: string, progress: ReviewProgress) => void;
|
|
@@ -79,7 +76,6 @@ export async function prepareAgentReviewPlan(
|
|
|
79
76
|
|
|
80
77
|
const synthesis = await synthesizeReviewBrief({
|
|
81
78
|
model: input.model,
|
|
82
|
-
modelRegistry: input.modelRegistry,
|
|
83
79
|
cwd: input.cwd,
|
|
84
80
|
snapshot,
|
|
85
81
|
serializedContext: input.serializedContext,
|
|
@@ -192,7 +188,6 @@ async function runAssignment(
|
|
|
192
188
|
const rawResult = await runReviewer({
|
|
193
189
|
prompt: packet.prompt,
|
|
194
190
|
model: plan.model,
|
|
195
|
-
modelRegistry: input.modelRegistry,
|
|
196
191
|
cwd: input.cwd,
|
|
197
192
|
signal: input.signal,
|
|
198
193
|
snapshot: plan.snapshot,
|
package/src/tool/brief-runner.ts
CHANGED
|
@@ -67,7 +67,6 @@ async function createBriefSession(
|
|
|
67
67
|
const { session } = await createAgentSession({
|
|
68
68
|
cwd: invocation.cwd,
|
|
69
69
|
model: invocation.model,
|
|
70
|
-
modelRegistry: invocation.modelRegistry,
|
|
71
70
|
thinkingLevel: clampThinkingLevel(invocation.model, "max"),
|
|
72
71
|
tools: ["submit_review_brief"],
|
|
73
72
|
customTools: [submitBriefTool],
|
|
@@ -42,7 +42,6 @@ async function createReviewerSession(
|
|
|
42
42
|
const { session } = await createAgentSession({
|
|
43
43
|
cwd: invocation.cwd,
|
|
44
44
|
model: invocation.model.model,
|
|
45
|
-
modelRegistry: invocation.modelRegistry,
|
|
46
45
|
thinkingLevel: clampThinkingLevel(invocation.model.model, "max"),
|
|
47
46
|
tools: [
|
|
48
47
|
"read",
|
package/src/types.ts
CHANGED
|
@@ -1,5 +1,4 @@
|
|
|
1
1
|
import type { Model } from "@earendil-works/pi-ai";
|
|
2
|
-
import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
|
|
3
2
|
|
|
4
3
|
/** Inclusive 1-based line range reported by the reviewer. */
|
|
5
4
|
export interface ReviewLineRange {
|
|
@@ -326,7 +325,6 @@ export interface BriefSynthesisInvocation {
|
|
|
326
325
|
prompt: string;
|
|
327
326
|
// biome-ignore lint/suspicious/noExplicitAny: Model<any> is pi's canonical type
|
|
328
327
|
model: Model<any>;
|
|
329
|
-
modelRegistry?: ModelRegistry;
|
|
330
328
|
cwd: string;
|
|
331
329
|
signal?: AbortSignal;
|
|
332
330
|
timeoutMs?: number;
|
|
@@ -336,7 +334,6 @@ export interface BriefSynthesisInvocation {
|
|
|
336
334
|
export interface ReviewInvocation {
|
|
337
335
|
prompt: string;
|
|
338
336
|
model: ReviewModelSelection;
|
|
339
|
-
modelRegistry?: ModelRegistry;
|
|
340
337
|
cwd: string;
|
|
341
338
|
signal?: AbortSignal;
|
|
342
339
|
snapshot: ReviewSnapshot;
|