@wardby/cli 0.4.1 → 0.5.1

package/prisma/migrations/20261009000002_local_review/migration.sql

@@ -0,0 +1,50 @@
+-- Additive: manual reviews of local repositories. A LocalPullRequest is a
+-- branch compared against a base in a local: repository, keyed by a durable
+-- serial number so the repo_* review tools address it like a pull request;
+-- LocalReview holds what review agents publish on it.
+
+-- CreateTable
+CREATE TABLE "LocalPullRequest" (
+    "id" TEXT NOT NULL,
+    "number" SERIAL NOT NULL,
+    "repository" TEXT NOT NULL,
+    "branch" TEXT NOT NULL,
+    "base" TEXT NOT NULL,
+    "runId" TEXT,
+    "createdAt" TIMESTAMP(3) NOT NULL DEFAULT CURRENT_TIMESTAMP,
+
+    CONSTRAINT "LocalPullRequest_pkey" PRIMARY KEY ("id")
+);
+
+-- CreateTable
+CREATE TABLE "LocalReview" (
+    "id" TEXT NOT NULL,
+    "pullRequestId" TEXT NOT NULL,
+    "agentId" TEXT NOT NULL,
+    "headSha" TEXT NOT NULL,
+    "verdict" TEXT NOT NULL,
+    "summary" TEXT NOT NULL,
+    "body" TEXT NOT NULL,
+    "comments" JSONB NOT NULL,
+    "createdAt" TIMESTAMP(3) NOT NULL DEFAULT CURRENT_TIMESTAMP,
+
+    CONSTRAINT "LocalReview_pkey" PRIMARY KEY ("id")
+);
+
+-- CreateIndex
+CREATE UNIQUE INDEX "LocalPullRequest_number_key" ON "LocalPullRequest"("number");
+
+-- CreateIndex
+CREATE UNIQUE INDEX "LocalPullRequest_runId_key" ON "LocalPullRequest"("runId");
+
+-- CreateIndex
+CREATE INDEX "LocalPullRequest_repository_branch_idx" ON "LocalPullRequest"("repository", "branch");
+
+-- CreateIndex
+CREATE INDEX "LocalReview_pullRequestId_createdAt_idx" ON "LocalReview"("pullRequestId", "createdAt");
+
+-- AddForeignKey
+ALTER TABLE "LocalPullRequest" ADD CONSTRAINT "LocalPullRequest_runId_fkey" FOREIGN KEY ("runId") REFERENCES "Run"("id") ON DELETE SET NULL ON UPDATE CASCADE;
+
+-- AddForeignKey
+ALTER TABLE "LocalReview" ADD CONSTRAINT "LocalReview_pullRequestId_fkey" FOREIGN KEY ("pullRequestId") REFERENCES "LocalPullRequest"("id") ON DELETE CASCADE ON UPDATE CASCADE;

package/prisma/schema.prisma

@@ -41,6 +41,10 @@ model Agent {
   // Phase 3: tools + multi-turn loop.
   tools    AgentTool[]
   maxTurns Int         @default(10)
+  /// Sub-agent delegations one run may make (delegate_to_*), each to a different sub-agent. 1 = route to exactly one.
+  maxDelegationsPerRun Int @default(1)
+  /// Native leads: the delegate_to_* calls of one model turn run at the same time. False = one after another.
+  parallelDelegations Boolean @default(false)
 
   // Phase 4: ownership. Null = owner-less, a transition state only: access
   // then comes from ResourceGrant rows alone (the migration gives everyone
@@ -52,6 +56,7 @@ model Agent {
   datastores   AgentDatastore[]
   webhooks     Webhook[]
   repositories AgentRepository[]
+  deferredReviews DeferredReview[]
   issueProjects AgentIssueProject[]
 
   /// Self-defects opt-in: the Jira project and issue type wardby files the agent's own failures into. Both or neither.
@@ -350,6 +355,8 @@ model Run {
   attribution      RunAttribution?
   modelUsage       RunModelUsage[]
   serviceStatuses  CodingRunServiceStatus[]
+  /// A manual review of a local branch: this run is the review run dispatched for it.
+  localPullRequest LocalPullRequest?
 
   /// Un-phased: sub-agent dispatch (see AgentSubAgent). Null for a
   /// top-level run; set when this run was created by a parent run's
@@ -398,6 +405,8 @@ model CodingAgentProfile {
   defaultTask              String?
   allowWebhookTaskOverride Boolean @default(false)
   timeoutSec               Int     @default(1800)
+  /// Claude Code runs only: the most agent turns a run may take. Null = the worker default (200).
+  maxTurns                 Int?
   /// Unused; dropped after rollout (see the package-registry plan, Task 14).
   allowedEgress            Json    @default("[]")
   protectedPaths           Json
@@ -464,6 +473,8 @@ model CodingRun {
   /// Fixed at dispatch from the profile's debugTraceUntil: the worker writes a
   /// full stream trace to its own pod log (never to the database).
   debugTrace        Boolean             @default(false)
+  /// Fixed at dispatch from the profile's maxTurns: the Claude Code worker's turn limit. Null = the worker default.
+  maxTurns          Int?
   /// Coding-run services resolved at dispatch from the base branch's
   /// .wardby/services.yaml: one snapshot per service (the catalog entry minus
   /// kind), fixed for the run's lifetime. Empty = none.
@@ -471,6 +482,10 @@ model CodingRun {
   /// The originating tracker issue ("jira" + key), set by the control plane from the run tree, never from model output.
   issueProvider     String?
   issueKey          String?
+  /// The branch a run pushed to a local repository (no pull request exists); null otherwise.
+  resultBranch      String?
+  /// The commit the run started from (the base ref's tip when the workspace was prepared).
+  baseSha           String?
   createdAt         DateTime            @default(now())
   updatedAt         DateTime            @updatedAt
   proxySession      CodingProxySession?
@@ -1059,11 +1074,19 @@ model AgentRepository {
   repository String
   /// "read" or "write".
   access     String
-  /// Subset of "pull_request", "mention", "push".
+  /// Subset of "pull_request", "mention", "push", "review_fix".
   triggers   String[] @default([])
   /// Only with the "pull_request" trigger (required there); names the agent's
   /// check. Unique per repository across all links (NULLs are distinct).
   checkName  String?
+  /// Only with the "review_fix" trigger: how many automatic fix rounds a
+  /// pull request gets before wardby stops. Null = the default (2).
+  reviewFixMaxRounds Int?
+  /// Only with the "pull_request" trigger: holds this agent's review on a
+  /// pushed head until that head's own CI finishes (see DeferredReview),
+  /// instead of racing it. Default off: existing links review immediately,
+  /// exactly as today.
+  waitForCi  Boolean  @default(false)
   createdAt  DateTime @default(now())
 
   /// How this link was authorized: "host_permission" (the agent owner's
@@ -1080,6 +1103,31 @@ model AgentRepository {
   @@index([provider, repository])
 }
 
+/// A pull-request review deferred because its head's CI was still pending
+/// (or had not reported) when the triggering push event arrived
+/// (AgentRepository.waitForCi). Started either when that head's CI finishes
+/// (the ci_completed host event) or by the reconciler's fallback sweep after
+/// DEFERRED_REVIEW_MAX_WAIT_MS. Idempotent per
+/// (provider, repository, prNumber, headSha, agentId): re-deferring the same
+/// head for the same agent is a no-op, not a duplicate row.
+model DeferredReview {
+  id         String   @id @default(cuid())
+  provider   String
+  repository String
+  prNumber   Int
+  headSha    String
+  agentId    String
+  agent      Agent    @relation(fields: [agentId], references: [id], onDelete: Cascade)
+  /// The agent's check name at defer time, so the delayed review starts
+  /// against the same check even if the link's checkName changes later.
+  checkName  String
+  createdAt  DateTime @default(now())
+
+  @@unique([provider, repository, prNumber, headSha, agentId])
+  @@index([provider, repository, headSha])
+  @@index([createdAt])
+}
+
 /// The host check a run owns, so the review tool can complete it and the
 /// run finalizer can close it if the run ends without publishing.
 model RunHostCheck {
@@ -1094,6 +1142,18 @@ model RunHostCheck {
   /// existed: such a run publishes without a check.
   prNumber    Int?
   completedAt DateTime?
+  /// The verdict the run published on its own check ("APPROVE",
+  /// "CHANGES_REQUESTED", "COMMENT"); null until then, or when the check was
+  /// closed without one (superseded, or by the finalizer).
+  verdict     String?
+  /// The published review's summary and body (capped), kept for an automatic
+  /// fix round started when the run finishes.
+  reviewBody  String?
+  /// The run published COMMENT while CI on its head was still running or had
+  /// not reported yet. CI finishing on that head later re-runs the review.
+  ciPendingAtReview Boolean?
+  /// When CI finishing on this head re-ran the review: at most once per head.
+  ciRereviewAt      DateTime?
 }
 
 /// De-duplicates host webhook redeliveries; pruned after 7 days.
@@ -1371,3 +1431,40 @@ model ModelCatalogEntry {
 
   @@id([provider, modelId])
 }
+
+/// A local "pull request": a branch compared against a base in a local:
+/// repository (git folder on the wardby host), keyed by a durable number so
+/// the repo_* review tools work against it like a GitHub pull request.
+model LocalPullRequest {
+  id         String        @id @default(cuid())
+  number     Int           @unique @default(autoincrement())
+  /// "local:/abs/path"
+  repository String
+  branch     String
+  base       String
+  /// The review run dispatched for this pull request.
+  runId      String?       @unique
+  run        Run?          @relation(fields: [runId], references: [id], onDelete: SetNull)
+  createdAt  DateTime      @default(now())
+  reviews    LocalReview[]
+
+  @@index([repository, branch])
+}
+
+/// A review (or a plain comment, verdict COMMENT) published on a LocalPullRequest.
+model LocalReview {
+  id            String           @id @default(cuid())
+  pullRequestId String
+  pullRequest   LocalPullRequest @relation(fields: [pullRequestId], references: [id], onDelete: Cascade)
+  agentId       String
+  headSha       String
+  /// APPROVE | CHANGES_REQUESTED | COMMENT
+  verdict       String
+  summary       String
+  body          String
+  /// [{path, line, side, severity, body}]
+  comments      Json
+  createdAt     DateTime         @default(now())
+
+  @@index([pullRequestId, createdAt])
+}