@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.
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@mrclrchtr/supi-core",
3
- "version": "2.6.0",
3
+ "version": "2.6.1",
4
4
  "description": "SuPi core — shared infrastructure for SuPi extensions (XML context tags, config system)",
5
5
  "license": "MIT",
6
6
  "repository": {
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@mrclrchtr/supi-review",
3
- "version": "2.6.0",
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.0"
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, modelRegistry, cwd, snapshot, serializedContext, note, signal, onProgress } =
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,
@@ -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;