pullfrog 0.1.6 → 0.1.7

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.
@@ -8,6 +8,17 @@ import { type AgentResult, type AgentRunContext, type AgentUsage, type PostRunIs
8
8
  *
9
9
  * the gate is anchored to `hadProgressComment` so silent runs (non-issue
10
10
  * events, dispatcher skipped seeding) don't fire a nudge there's no UI for.
11
+ *
12
+ * `Review` and `IncrementalReview` have different valid exits:
13
+ * - Review: only `create_pull_request_review` counts. `report_progress` is
14
+ * not a substitute — a Review run that exits with just a summary comment
15
+ * has produced nothing reviewable on the PR. matches the hard-fail
16
+ * message at `expected = "create_pull_request_review"` below.
17
+ * - IncrementalReview: `report_progress` is a legitimate "no review
18
+ * warranted" exit, so either toolState flag short-circuits.
19
+ * splitting per mode also closes the bypass where a subagent (e.g. a
20
+ * `task`-dispatched `reviewfrog` lens) calls `report_progress` and silences
21
+ * the gate even though the orchestrator never submitted a review.
11
22
  */
12
23
  export declare function getUnsubmittedReview(toolState: ToolState): "Review" | "IncrementalReview" | null;
13
24
  /**
@@ -56,6 +67,16 @@ export declare function buildPostRunPrompt(issues: PostRunIssues): string;
56
67
  * the file is the single source of truth — there is no separate MCP tool
57
68
  * call. the server reads the file at end-of-run and persists any edits to
58
69
  * `Repo.learnings`.
70
+ *
71
+ * the prompt copy is shaped by repo-wide audits of the actual content the
72
+ * agent has been writing (issue #619 in pullfrog/app). recurring failure
73
+ * modes the framing pushes back on:
74
+ * - massive multi-paragraph "bullets" that are really mini-articles
75
+ * - PR-/review-/commit-/date-anchored facts that decay within weeks
76
+ * - rediscovery of pullfrog-tool quirks that belong in tool descriptions,
77
+ * not per-repo learnings
78
+ * - sections growing into giant flat lists with no internal structure,
79
+ * forcing future runs to read kilobytes to find one fact
59
80
  */
60
81
  export declare function buildLearningsReflectionPrompt(filePath: string): string;
61
82
  /**
@@ -0,0 +1,19 @@
1
+ /**
2
+ * Derive a cheaper subagent model override from the orchestrator's resolved
3
+ * model spec.
4
+ *
5
+ * This is a pure registry lookup: every alias in `action/models.ts` declares
6
+ * its own `subagentModel` (alias key in the same provider). At runtime we
7
+ * reverse-lookup the orchestrator's resolved slug to find the alias that
8
+ * produced it, follow the `subagentModel` pointer, and return the target
9
+ * alias's resolve / openRouterResolve depending on which route the
10
+ * orchestrator was using.
11
+ *
12
+ * Returns `{ reviewer: undefined }` when the orchestrator's alias has no
13
+ * `subagentModel` (e.g. it's already at a sufficiently cheap tier, or its
14
+ * provider doesn't have a clean cheaper-but-capable sibling). See models.ts
15
+ * for the wiring + per-provider rationale.
16
+ */
17
+ export declare function deriveSubagentModels(orchestratorSpec: string | undefined): {
18
+ reviewer: string | undefined;
19
+ };