@vercel/factory 0.0.8 → 0.0.10

package/dist/eve/index.d.mts

@@ -1,5 +1,5 @@
-import { c as Finding, k as FactoryStores, t as ApprovalPolicy } from "../approval-qkxww0wJ.mjs";
-import { d as GitHubReviewComment } from "../github-BdWWueJP.mjs";
+import { A as FactoryStores, c as Finding, t as ApprovalPolicy } from "../approval-CDiBco2V.mjs";
+import { T as PullRequestReviewComment, b as GitHubReviewComment, j as SubmittedPullRequestReview } from "../github-wThBfbm-.mjs";
 import { z } from "zod";
 //#region src/eve/fetch.d.ts
 /** The subset of the eve sandbox handle a credentialed git operation needs. */
@@ -362,7 +362,7 @@ declare function invokeAgentTool(stores: FactoryStores, options: InvokeAgentOpti
 }, {
   taskId: string;
   kind: string;
-  state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+  state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
 }> & {
   execute(input: {
     repositoryIds: string[];
@@ -374,7 +374,77 @@ declare function invokeAgentTool(stores: FactoryStores, options: InvokeAgentOpti
   }, ctx: import("eve/tools").ToolContext): Promise<{
     taskId: string;
     kind: string;
-    state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
+  }>;
+};
+//#endregion
+//#region src/eve/post-review.d.ts
+interface PublishedReview {
+  taskId: string;
+  changeId: string;
+  repository: string;
+  headSha: string;
+  verdict: "approve" | "request_changes" | "comment";
+  summary: string;
+  comments: readonly PullRequestReviewComment[];
+  review: SubmittedPullRequestReview;
+}
+interface PostReviewToolOptions {
+  tokenForRepository: (repository: string) => Promise<string>;
+  onPublished?: (published: PublishedReview) => Promise<void>;
+  fetch?: typeof globalThis.fetch;
+}
+/**
+ * Publish an independent review of an existing pull request and finish the
+ * review task without changing the pull request's factory lifecycle.
+ */
+declare function postReviewTool(stores: FactoryStores, options: PostReviewToolOptions): import("eve/tools").ToolDefinition<{
+  taskId: string;
+  changeId: string;
+  verdict: "approve" | "request_changes" | "comment";
+  headSha: string;
+  summary: string;
+  comments: {
+    path: string;
+    line: number;
+    body: string;
+  }[];
+}, {
+  recorded: false;
+  reason: string;
+  verdict?: undefined;
+  review?: undefined;
+  taskState?: undefined;
+} | {
+  recorded: true;
+  verdict: "approve" | "request_changes" | "comment";
+  review: SubmittedPullRequestReview;
+  taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
+  reason?: undefined;
+}> & {
+  execute(input: {
+    taskId: string;
+    changeId: string;
+    verdict: "approve" | "request_changes" | "comment";
+    headSha: string;
+    summary: string;
+    comments: {
+      path: string;
+      line: number;
+      body: string;
+    }[];
+  }, ctx: import("eve/tools").ToolContext): Promise<{
+    recorded: false;
+    reason: string;
+    verdict?: undefined;
+    review?: undefined;
+    taskState?: undefined;
+  } | {
+    recorded: true;
+    verdict: "approve" | "request_changes" | "comment";
+    review: SubmittedPullRequestReview;
+    taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
+    reason?: undefined;
   }>;
 };
 //#endregion
@@ -484,8 +554,8 @@ declare function recordVerificationTool(stores: FactoryStores): import("eve/tool
 } | {
   recorded: true;
   verdict: "pass" | "fail";
-  changeState: "rolled_back" | "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "closed";
-  taskState: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+  changeState: "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "rolled_back" | "closed";
+  taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
   reason?: undefined;
 }> & {
   execute(input: {
@@ -511,8 +581,8 @@ declare function recordVerificationTool(stores: FactoryStores): import("eve/tool
   } | {
     recorded: true;
     verdict: "pass" | "fail";
-    changeState: "rolled_back" | "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "closed";
-    taskState: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    changeState: "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "rolled_back" | "closed";
+    taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
     reason?: undefined;
   }>;
 };
@@ -562,8 +632,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
   }, {
     schemaVersion: 1;
     id: string;
+    repositoryIds: string[];
     kind: string;
-    state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
     origin: {
       signalId?: string | undefined;
       parentTaskId?: string | undefined;
@@ -576,7 +647,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     attempt: number;
     createdAt: string;
     updatedAt: string;
-    repositoryIds?: string[] | undefined;
     area?: string | undefined;
     approval?: "required" | "granted" | "denied" | undefined;
     dependsOn?: string[] | undefined;
@@ -604,8 +674,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     }, ctx: import("eve/tools").ToolContext): Promise<{
       schemaVersion: 1;
       id: string;
+      repositoryIds: string[];
       kind: string;
-      state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+      state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
       origin: {
         signalId?: string | undefined;
         parentTaskId?: string | undefined;
@@ -618,7 +689,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       attempt: number;
       createdAt: string;
       updatedAt: string;
-      repositoryIds?: string[] | undefined;
       area?: string | undefined;
       approval?: "required" | "granted" | "denied" | undefined;
       dependsOn?: string[] | undefined;
@@ -713,7 +783,7 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
         scopeMatches: boolean;
         previewUrl?: string | undefined;
       };
-      state: "rolled_back" | "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "closed";
+      state: "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "rolled_back" | "closed";
       createdAt: string;
       updatedAt: string;
       taskId?: string | undefined;
@@ -724,8 +794,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     task: {
       schemaVersion: 1;
       id: string;
+      repositoryIds: string[];
       kind: string;
-      state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+      state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
       origin: {
         signalId?: string | undefined;
         parentTaskId?: string | undefined;
@@ -738,7 +809,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       attempt: number;
       createdAt: string;
       updatedAt: string;
-      repositoryIds?: string[] | undefined;
       area?: string | undefined;
       approval?: "required" | "granted" | "denied" | undefined;
       dependsOn?: string[] | undefined;
@@ -785,7 +855,7 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
           scopeMatches: boolean;
           previewUrl?: string | undefined;
         };
-        state: "rolled_back" | "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "closed";
+        state: "verifying" | "ready" | "merged" | "deployed" | "rolled_out" | "rolled_back" | "closed";
         createdAt: string;
         updatedAt: string;
         taskId?: string | undefined;
@@ -796,8 +866,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       task: {
         schemaVersion: 1;
         id: string;
+        repositoryIds: string[];
         kind: string;
-        state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+        state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
         origin: {
           signalId?: string | undefined;
           parentTaskId?: string | undefined;
@@ -810,7 +881,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
         attempt: number;
         createdAt: string;
         updatedAt: string;
-        repositoryIds?: string[] | undefined;
         area?: string | undefined;
         approval?: "required" | "granted" | "denied" | undefined;
         dependsOn?: string[] | undefined;
@@ -841,7 +911,7 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     taskState?: undefined;
   } | {
     finished: true;
-    taskState: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
     reason?: undefined;
   }> & {
     execute(input: {
@@ -853,7 +923,7 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       taskState?: undefined;
     } | {
       finished: true;
-      taskState: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+      taskState: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
       reason?: undefined;
     }>;
   };
@@ -863,8 +933,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
   }, {
     schemaVersion: 1;
     id: string;
+    repositoryIds: string[];
     kind: string;
-    state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
     origin: {
       signalId?: string | undefined;
       parentTaskId?: string | undefined;
@@ -877,7 +948,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     attempt: number;
     createdAt: string;
     updatedAt: string;
-    repositoryIds?: string[] | undefined;
     area?: string | undefined;
     approval?: "required" | "granted" | "denied" | undefined;
     dependsOn?: string[] | undefined;
@@ -903,8 +973,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     }, ctx: import("eve/tools").ToolContext): Promise<{
       schemaVersion: 1;
       id: string;
+      repositoryIds: string[];
       kind: string;
-      state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+      state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
       origin: {
         signalId?: string | undefined;
         parentTaskId?: string | undefined;
@@ -917,7 +988,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       attempt: number;
       createdAt: string;
       updatedAt: string;
-      repositoryIds?: string[] | undefined;
       area?: string | undefined;
       approval?: "required" | "granted" | "denied" | undefined;
       dependsOn?: string[] | undefined;
@@ -950,8 +1020,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
   }, {
     schemaVersion: 1;
     id: string;
+    repositoryIds: string[];
     kind: string;
-    state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+    state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
     origin: {
       signalId?: string | undefined;
       parentTaskId?: string | undefined;
@@ -964,7 +1035,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     attempt: number;
     createdAt: string;
     updatedAt: string;
-    repositoryIds?: string[] | undefined;
     area?: string | undefined;
     approval?: "required" | "granted" | "denied" | undefined;
     dependsOn?: string[] | undefined;
@@ -996,8 +1066,9 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
     }, ctx: import("eve/tools").ToolContext): Promise<{
       schemaVersion: 1;
       id: string;
+      repositoryIds: string[];
       kind: string;
-      state: "succeeded" | "failed" | "needs_human" | "queued" | "running" | "verifying" | "cancelled";
+      state: "queued" | "running" | "verifying" | "succeeded" | "failed" | "needs_human" | "cancelled";
       origin: {
         signalId?: string | undefined;
         parentTaskId?: string | undefined;
@@ -1010,7 +1081,6 @@ declare function factoryTools(stores: FactoryStores, options?: FactoryToolsOptio
       attempt: number;
       createdAt: string;
       updatedAt: string;
-      repositoryIds?: string[] | undefined;
       area?: string | undefined;
       approval?: "required" | "granted" | "denied" | undefined;
       dependsOn?: string[] | undefined;
@@ -1099,5 +1169,5 @@ declare function factoryHooks(stores: FactoryStores): {
 };
 type FactoryHooks = ReturnType<typeof factoryHooks>;
 //#endregion
-export { type CommandSandbox, type DeepsecScanOptions, FactoryHooks, FactoryTools, FactoryToolsOptions, type FetchPullRequestOptions, type FetchPullRequestResult, GateVerdict, type InvokeAgentOptions, type MergeConflictPayload, type MergeConflictToolOptions, RecordChangeInput, type RecordFindingOptions, type RecordFindingResult, type ReviewCommentToolOptions, deepsecScanTool, factoryHooks, factoryTools, fetchPullRequestTool, findingFromDeepsec, invokeAgentTool, mergeConflictPayloadSchema, mergeConflictTools, recordFinding, recordFindingTool, recordVerificationTool, reviewCommentTools };
+export { type CommandSandbox, type DeepsecScanOptions, FactoryHooks, FactoryTools, FactoryToolsOptions, type FetchPullRequestOptions, type FetchPullRequestResult, GateVerdict, type InvokeAgentOptions, type MergeConflictPayload, type MergeConflictToolOptions, type PostReviewToolOptions, type PublishedReview, RecordChangeInput, type RecordFindingOptions, type RecordFindingResult, type ReviewCommentToolOptions, deepsecScanTool, factoryHooks, factoryTools, fetchPullRequestTool, findingFromDeepsec, invokeAgentTool, mergeConflictPayloadSchema, mergeConflictTools, postReviewTool, recordFinding, recordFindingTool, recordVerificationTool, reviewCommentTools };
 //# sourceMappingURL=index.d.mts.map

package/dist/eve/index.mjs

@@ -1,6 +1,7 @@
 import { i as repositoryIdSchema, o as signalIdSchema, p as vocabularySchema, s as taskIdSchema, t as changeIdSchema } from "../id-DwCihQr4.mjs";
-import { S as findingFingerprint, b as requiresApproval, g as dispatchTask, p as levelSchema, s as taskQuestionSchema, u as changeEvidenceSchema, w as findingSchema, y as defaultApprovalPolicy } from "../task-BxTU3MJb.mjs";
-import { answerReviewComment, readPullRequestHead, readReviewComment } from "../github.mjs";
+import { c as findingSchema, i as requiresApproval, o as findingFingerprint, r as defaultApprovalPolicy } from "../approval-DdvEKLiE.mjs";
+import { g as taskQuestionSchema, i as levelSchema, s as dispatchTask, t as changeEvidenceSchema } from "../change-BPYhWKuL.mjs";
+import { answerReviewComment, readPullRequestHead, readReviewComment, submitPullRequestReview } from "../github.mjs";
 import { z } from "zod";
 import { defineHook } from "eve/hooks";
 import { defineTool } from "eve/tools";
@@ -474,6 +475,86 @@ function invokeAgentTool(stores, options) {
 	});
 }
 //#endregion
+//#region src/eve/post-review.ts
+const reviewCommentSchema = z.strictObject({
+	path: z.string().min(1).max(500).describe("Repository-relative file path."),
+	line: z.number().int().min(1).describe("Line in the pull request's new version."),
+	body: z.string().min(1).max(5e3).describe("Markdown explaining the finding and a concrete fix.")
+});
+/**
+* Publish an independent review of an existing pull request and finish the
+* review task without changing the pull request's factory lifecycle.
+*/
+function postReviewTool(stores, options) {
+	return defineTool({
+		description: "Publish a completed code review to the existing pull request and finish your code_review task. Use request_changes only for blocking findings.",
+		inputSchema: z.object({
+			taskId: taskIdSchema.describe("Your code_review task."),
+			changeId: changeIdSchema.describe("The pull request change from your starting message."),
+			verdict: z.enum([
+				"approve",
+				"request_changes",
+				"comment"
+			]),
+			headSha: z.string().regex(/^[0-9a-f]{40}$/).describe("Commit returned by fetch_pull_request."),
+			summary: z.string().min(1).max(5e3).describe("Markdown summary of what you reviewed and found."),
+			comments: z.array(reviewCommentSchema).max(50).default([])
+		}),
+		async execute({ taskId, changeId, verdict, headSha, summary, comments }) {
+			const task = await stores.tasks.get(taskId);
+			if (task === null || task.kind !== "code_review") return {
+				recorded: false,
+				reason: `${taskId} is not a code_review task`
+			};
+			if (task.state !== "running") return {
+				recorded: false,
+				reason: `task is ${task.state}, not running`
+			};
+			const change = await stores.changes.get(changeId);
+			if (change === null) return {
+				recorded: false,
+				reason: `change ${changeId} does not exist`
+			};
+			if (change.state !== "verifying" && change.state !== "ready") return {
+				recorded: false,
+				reason: `change is ${change.state}, not open for review`
+			};
+			const repository = await stores.repositories.get(change.repositoryId);
+			if (repository === null) return {
+				recorded: false,
+				reason: `repository ${change.repositoryId} does not exist`
+			};
+			const review = await submitPullRequestReview({
+				repository: repository.slug,
+				token: await options.tokenForRepository(repository.slug),
+				fetch: options.fetch,
+				pullRequestNumber: change.prNumber,
+				commitId: headSha,
+				event: verdict === "approve" ? "APPROVE" : verdict === "request_changes" ? "REQUEST_CHANGES" : "COMMENT",
+				body: summary,
+				comments
+			});
+			await options.onPublished?.({
+				taskId,
+				changeId,
+				repository: repository.slug,
+				headSha,
+				verdict,
+				summary,
+				comments,
+				review
+			});
+			await stores.tasks.transition(taskId, "verifying", { reason: summary });
+			return {
+				recorded: true,
+				verdict,
+				review,
+				taskState: (await stores.tasks.transition(taskId, "succeeded")).state
+			};
+		}
+	});
+}
+//#endregion
 //#region src/eve/review.ts
 function reviewCommentTools(stores, options) {
 	const repoPath = options.repoPath ?? "repo";
@@ -848,6 +929,6 @@ function factoryHooks(stores) {
 	} } }) };
 }
 //#endregion
-export { deepsecScanTool, factoryHooks, factoryTools, fetchPullRequestTool, findingFromDeepsec, invokeAgentTool, mergeConflictPayloadSchema, mergeConflictTools, recordFinding, recordFindingTool, recordVerificationTool, reviewCommentTools };
+export { deepsecScanTool, factoryHooks, factoryTools, fetchPullRequestTool, findingFromDeepsec, invokeAgentTool, mergeConflictPayloadSchema, mergeConflictTools, postReviewTool, recordFinding, recordFindingTool, recordVerificationTool, reviewCommentTools };
 
 //# sourceMappingURL=index.mjs.map