mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-10-08 11:13:44 +02:00
## Thinking Path > - Paperclip manages tasks across persistent agent sessions. > - The full Runner E2E catalog exposed failures in session restoration, tool validation, and test controls. > - These failures prevented valid work from resuming or made a valid interaction fail the test. > - Invalid completion reports also reached finalization before the provider received useful feedback. > - This pull request repairs those boundaries without changing production prompts or approval policy. > - Focused regressions and fresh paid cases verify each fix. ## Linked Issues or Issue Description Follow-up to #13655. Stacked on the trusted worker prerequisite fix in #13674. **What happened?** Read-only skill uploads failed in resumed Daytona sandboxes. Invalid criterion IDs escaped tool validation. A progress event could park a run before its tool response settled. Partial question forms hid required answers. Two test assumptions rejected valid plan keys or failed to navigate an optional question page. **What did you expect to happen?** Resume identical skill bundles, give repairable feedback for malformed completion calls, preserve in-flight tool responses, show all required questions, and test the rendered workflow accurately. **Steps to reproduce** Inspect the failed cases in https://github.com/paperclipai/paperclip/actions/runs/35417932353. Fresh campaigns: https://github.com/paperclipai/paperclip/actions/runs/35444497313 and https://github.com/paperclipai/paperclip/actions/runs/35445327618. The later backup cleanup is tested in https://github.com/paperclipai/paperclip/actions/runs/35446477285. Combined report: https://pages.paperclip.ing/runner-e2e-operational-35444497313/investigation.html. ## What Changed - Compare immutable archives before reusing read-only Daytona bundles. Reject corrupted content and preserve unrelated files. - Validate exact criterion IDs before accepting completion. OpenCode returns a tool error instead of emitting a result that terminates runnerd. - Complete the activity item for rejected OpenCode calls. - Remove retired read-only harness backups without altering live files or following symlinks. A fresh paid rerun exposed this later checkpoint-cleanup failure. - Exclude progress messages from the governed-wait completion boundary. - Reject newly created question forms that omit questions or contradict their stored answer semantics. Keep historical rows readable. - Navigate all rendered question pages and recognize revision-bound descriptive plan keys in the continuation suite. ## Verification - Harness unit suite: 383 tests pass. Harness typecheck passes. - Native session executor and status corpus: 381 tests pass. - Shared question and interaction-service tests: 42 pass; native question bridge and executor: 360 pass. Daytona sync: 21 pass, including foreign-owner archives and corrupted immutable content. - OpenCode driver: 29 tests pass, including wrong, missing, and duplicate criterion IDs followed by a valid retry. - Repository typecheck and build pass. The later OpenCode activity fix also passes its package build. - The latest commit passes all 52 PR checks and Greptile 5/5. The backup-cleanup fix also passes 351 related local tests and server typecheck. Local full-suite coverage completed across runs. adapter-auth-signal-routes and pipelines-routes encountered transient socket resets; both pass on retry, and all remaining 24 serialized files pass. Paid reruns are complete: 27 of 29 unique cases pass using the latest recording per case. Both Daytona controller-restart cases still fail with runner_state_identity_mismatch; the report describes this remaining runtime issue. Eight affected cells need #13674 on master before their rerun. ## Risks Creation rejects inconsistent dual question representations but does not change historical records. Immutable bundle comparison must verify bytes before skipping extraction. Completion feedback must use the contract bound to the current run. Durable suspension and approval checks remain enforced. Production prompts are unchanged. ## Model Used OpenAI GPT-6 via Codex, with repository inspection, code editing, and test execution. The exact API model ID and context-window size are not exposed in this session. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work - [x] I have searched GitHub for duplicate or related PRs and linked them above - [x] I have either (a) linked existing issues with `Fixes: #` / `Closes #` / `Refs #` OR (b) described the issue in-PR following the relevant issue template - [x] I have not referenced internal/instance-local Paperclip issues or links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip` URLs) - [x] My branch name describes the change (e.g. `docs/...`, `fix/...`) and contains no internal Paperclip ticket id or instance-derived details - [x] I have run tests locally and they pass - [x] I have added or updated tests where applicable - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [x] All Paperclip CI gates are green - [x] Greptile is 5/5 with no open P2s, recommendations, or follow-ups - [x] I will address all Greptile and reviewer comments before requesting merge --------- Co-authored-by: Paperclip <noreply@paperclip.ing>
604 lines
20 KiB
TypeScript
604 lines
20 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import {
|
|
ISSUE_THREAD_INTERACTION_CANONICAL_RESOLVER_POLICIES,
|
|
ISSUE_THREAD_INTERACTION_LEGACY_RESOLVER_POLICY_ALIASES,
|
|
legacyIssueThreadInteractionResolverPolicyAlias,
|
|
normalizeIssueThreadInteractionResolverPolicy,
|
|
} from "./constants.js";
|
|
import {
|
|
acceptIssueThreadInteractionSchema,
|
|
askUserQuestionsResultSchema,
|
|
askUserQuestionsPayloadSchema,
|
|
createIssueThreadInteractionSchema,
|
|
paperclipQuestionSetPayloadSchema,
|
|
requestConfirmationPayloadSchema,
|
|
requestConfirmationResultSchema,
|
|
requestItemVerdictsResultSchema,
|
|
submitIssueThreadInteractionVerdictsSchema,
|
|
} from "./validators/issue.js";
|
|
|
|
describe("issue thread interaction schemas", () => {
|
|
it("defines canonical resolver policies and normalizes compatibility aliases", () => {
|
|
expect(ISSUE_THREAD_INTERACTION_CANONICAL_RESOLVER_POLICIES).toEqual([
|
|
"anyone",
|
|
"not_creator",
|
|
"human_only",
|
|
]);
|
|
expect(ISSUE_THREAD_INTERACTION_LEGACY_RESOLVER_POLICY_ALIASES).toEqual([
|
|
"board_or_agents",
|
|
"board_only",
|
|
]);
|
|
expect(normalizeIssueThreadInteractionResolverPolicy("board_or_agents")).toBe("anyone");
|
|
expect(normalizeIssueThreadInteractionResolverPolicy("board_only")).toBe("human_only");
|
|
expect(normalizeIssueThreadInteractionResolverPolicy("not_creator")).toBe("not_creator");
|
|
expect(legacyIssueThreadInteractionResolverPolicyAlias("anyone")).toBe("board_or_agents");
|
|
expect(legacyIssueThreadInteractionResolverPolicyAlias("not_creator")).toBeNull();
|
|
});
|
|
|
|
it.each(["anyone", "not_creator", "human_only", "board_or_agents", "board_only"] as const)(
|
|
"accepts resolver policy input %s",
|
|
(resolverPolicy) => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_confirmation",
|
|
resolverPolicy,
|
|
payload: { version: 1, prompt: "Proceed?" },
|
|
});
|
|
expect(parsed.resolverPolicy).toBe(resolverPolicy);
|
|
},
|
|
);
|
|
|
|
it("parses request_confirmation payloads with default no-wake continuation", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Apply this plan?",
|
|
acceptLabel: "Apply",
|
|
rejectLabel: "Revise",
|
|
rejectRequiresReason: true,
|
|
rejectReasonLabel: "What needs to change?",
|
|
declineReasonPlaceholder: "Optional: tell the agent what you'd change.",
|
|
detailsMarkdown: "The current plan document will be accepted as-is.",
|
|
supersedeOnUserComment: true,
|
|
},
|
|
});
|
|
|
|
expect(parsed).toMatchObject({
|
|
kind: "request_confirmation",
|
|
continuationPolicy: "none",
|
|
payload: {
|
|
prompt: "Apply this plan?",
|
|
acceptLabel: "Apply",
|
|
rejectLabel: "Revise",
|
|
rejectRequiresReason: true,
|
|
rejectReasonLabel: "What needs to change?",
|
|
allowDeclineReason: true,
|
|
declineReasonPlaceholder: "Optional: tell the agent what you'd change.",
|
|
supersedeOnUserComment: true,
|
|
},
|
|
});
|
|
});
|
|
|
|
it("round-trips versioned tool action payload and lifecycle metadata", () => {
|
|
const payload = requestConfirmationPayloadSchema.parse({
|
|
version: 1,
|
|
prompt: "Approve send_email?",
|
|
toolAction: {
|
|
version: 1,
|
|
actionRequestId: "11111111-1111-4111-8111-111111111111",
|
|
invocationId: "22222222-2222-4222-8222-222222222222",
|
|
toolName: "send_email",
|
|
toolDisplayName: "Send email",
|
|
connectionId: "33333333-3333-4333-8333-333333333333",
|
|
applicationId: "44444444-4444-4444-8444-444444444444",
|
|
appDisplayName: "Gmail",
|
|
risk: "write",
|
|
previewMarkdown: "Send an email to the reviewed recipient.",
|
|
argumentsSummaryJson: '{"to":"recipient@example.com"}',
|
|
argumentsHash: "reviewed-arguments-hash",
|
|
expiresAt: "2026-07-12T16:00:00.000Z",
|
|
},
|
|
});
|
|
const result = requestConfirmationResultSchema.parse({
|
|
version: 1,
|
|
outcome: "accepted",
|
|
toolAction: {
|
|
version: 1,
|
|
status: "executed",
|
|
errorCode: null,
|
|
errorMessage: null,
|
|
updatedAt: "2026-07-12T15:05:00.000Z",
|
|
},
|
|
});
|
|
|
|
expect(payload.toolAction).toMatchObject({
|
|
version: 1,
|
|
toolDisplayName: "Send email",
|
|
risk: "write",
|
|
argumentsHash: "reviewed-arguments-hash",
|
|
});
|
|
expect(result.toolAction).toMatchObject({ version: 1, status: "executed" });
|
|
expect(requestConfirmationPayloadSchema.parse({ version: 1, prompt: "Legacy confirmation?" }).toolAction)
|
|
.toBeUndefined();
|
|
});
|
|
|
|
it("parses superseded confirmation results with a replacement pointer", () => {
|
|
const result = requestConfirmationResultSchema.parse({
|
|
version: 1,
|
|
outcome: "superseded_by_newer_request",
|
|
supersededByInteractionId: "11111111-1111-4111-8111-111111111111",
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
version: 1,
|
|
outcome: "superseded_by_newer_request",
|
|
supersededByInteractionId: "11111111-1111-4111-8111-111111111111",
|
|
});
|
|
});
|
|
|
|
it("accepts run-attributed agent item verdict results and rejects missing runs", () => {
|
|
const result = {
|
|
version: 1,
|
|
outcome: "resolved",
|
|
complete: true,
|
|
items: [{
|
|
id: "api",
|
|
verdict: "approve",
|
|
resolvedByAgentId: "11111111-1111-4111-8111-111111111111",
|
|
resolvedByRunId: "22222222-2222-4222-8222-222222222222",
|
|
resolvedAt: "2026-08-14T12:00:00.000Z",
|
|
}],
|
|
};
|
|
expect(requestItemVerdictsResultSchema.parse(result)).toMatchObject(result);
|
|
expect(() => requestItemVerdictsResultSchema.parse({
|
|
...result,
|
|
items: [{ ...result.items[0], resolvedByRunId: undefined }],
|
|
})).toThrow("resolvedByRunId is required for an agent resolver");
|
|
});
|
|
|
|
it("accepts issue document targets for request_confirmation interactions", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_confirmation",
|
|
continuationPolicy: "wake_assignee_on_accept",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Accept the latest plan revision?",
|
|
allowDeclineReason: false,
|
|
target: {
|
|
type: "issue_document",
|
|
issueId: "11111111-1111-4111-8111-111111111111",
|
|
documentId: "22222222-2222-4222-8222-222222222222",
|
|
key: "plan",
|
|
revisionId: "33333333-3333-4333-8333-333333333333",
|
|
revisionNumber: 2,
|
|
label: "Plan v2",
|
|
href: "/issues/PAP-123#document-plan",
|
|
},
|
|
},
|
|
});
|
|
|
|
expect(parsed.kind).toBe("request_confirmation");
|
|
if (parsed.kind !== "request_confirmation") return;
|
|
expect(parsed.payload.target).toMatchObject({
|
|
type: "issue_document",
|
|
key: "plan",
|
|
revisionNumber: 2,
|
|
label: "Plan v2",
|
|
href: "/issues/PAP-123#document-plan",
|
|
});
|
|
});
|
|
|
|
it("accepts custom targets for request_confirmation interactions", () => {
|
|
for (const href of [
|
|
"https://example.com/checklist",
|
|
"http://example.com/checklist",
|
|
"/PAP/issues/PAP-123#document-plan",
|
|
"#document-plan",
|
|
]) {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Proceed with the external checklist?",
|
|
target: {
|
|
type: "custom",
|
|
key: "external-checklist",
|
|
revisionId: "checklist-v1",
|
|
revisionNumber: 1,
|
|
label: "Checklist v1",
|
|
href,
|
|
},
|
|
},
|
|
});
|
|
|
|
expect(parsed.kind).toBe("request_confirmation");
|
|
if (parsed.kind !== "request_confirmation") return;
|
|
expect(parsed.payload.target).toMatchObject({
|
|
type: "custom",
|
|
key: "external-checklist",
|
|
label: "Checklist v1",
|
|
href,
|
|
});
|
|
}
|
|
});
|
|
|
|
it("parses ask_user_questions supersede flags and expired results", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "ask_user_questions",
|
|
payload: {
|
|
version: 1,
|
|
title: "Choose scope",
|
|
supersedeOnUserComment: false,
|
|
questions: [
|
|
{
|
|
id: "scope",
|
|
prompt: "Which scope should I use?",
|
|
selectionMode: "single",
|
|
options: [{ id: "small", label: "Small" }],
|
|
},
|
|
],
|
|
},
|
|
});
|
|
|
|
expect(parsed).toMatchObject({
|
|
kind: "ask_user_questions",
|
|
continuationPolicy: "wake_assignee",
|
|
payload: {
|
|
supersedeOnUserComment: false,
|
|
},
|
|
});
|
|
|
|
expect(askUserQuestionsResultSchema.parse({
|
|
version: 1,
|
|
answers: [],
|
|
expirationReason: "superseded_by_comment",
|
|
commentId: "11111111-1111-4111-8111-111111111111",
|
|
summaryMarkdown: null,
|
|
})).toMatchObject({
|
|
expirationReason: "superseded_by_comment",
|
|
commentId: "11111111-1111-4111-8111-111111111111",
|
|
});
|
|
});
|
|
|
|
it("retains canonical runner question sets without narrowing their public bounds", () => {
|
|
const questionSet = {
|
|
schema: "paperclip.question_set.v1" as const,
|
|
title: "Runner input",
|
|
questions: [{
|
|
id: "deployment-color",
|
|
prompt: "Which deployment color should the runner use?",
|
|
required: true,
|
|
answerMode: "single_select" as const,
|
|
options: [{ id: "blue", label: "Blue" }],
|
|
}],
|
|
};
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "ask_user_questions",
|
|
continuationPolicy: "none",
|
|
resolverPolicy: "human_only",
|
|
payload: {
|
|
version: 1,
|
|
questions: [{
|
|
id: "deployment-color",
|
|
prompt: "Which deployment color should the runner use?",
|
|
selectionMode: "single",
|
|
allowOther: false,
|
|
options: [{ id: "blue", label: "Blue" }],
|
|
}],
|
|
questionSet,
|
|
},
|
|
});
|
|
expect(parsed.kind).toBe("ask_user_questions");
|
|
if (parsed.kind !== "ask_user_questions") return;
|
|
expect(parsed.payload.questionSet).toEqual(questionSet);
|
|
|
|
expect(() => paperclipQuestionSetPayloadSchema.parse({
|
|
...questionSet,
|
|
questions: [{ ...questionSet.questions[0], answerMode: "text", options: questionSet.questions[0].options }],
|
|
})).toThrow("text questions cannot define options");
|
|
});
|
|
|
|
it("rejects creation of a form that hides required questions, while retaining historical readability", () => {
|
|
const payload = {
|
|
version: 1,
|
|
questions: [
|
|
{ id: "club_name", prompt: "Club name?", selectionMode: "single", required: true, options: [{ id: "text", label: "Answer", freeText: true }] },
|
|
{ id: "audience", prompt: "Audience?", selectionMode: "single", required: true, options: [{ id: "beginners", label: "Beginners" }, { id: "everyone", label: "Everyone" }] },
|
|
],
|
|
questionSet: { schema: "paperclip.question_set.v1", questions: [{ id: "club_name", prompt: "Club name?", answerMode: "text", required: true }] },
|
|
};
|
|
expect(() => createIssueThreadInteractionSchema.parse({ kind: "ask_user_questions", payload })).toThrow("must present every questions entry");
|
|
expect(askUserQuestionsPayloadSchema.parse(payload).questions).toHaveLength(2);
|
|
const complete = { ...payload, questionSet: { ...payload.questionSet, questions: [...payload.questionSet.questions, { id: "audience", prompt: "Audience?", answerMode: "single_select", required: true, options: payload.questions[1].options }] } };
|
|
expect(createIssueThreadInteractionSchema.parse({ kind: "ask_user_questions", payload: complete }).payload).toMatchObject({ questionSet: { questions: expect.any(Array) } });
|
|
});
|
|
|
|
it.each([
|
|
["required", { required: false }],
|
|
["answer mode", { answerMode: "multi_select" }],
|
|
["option IDs", { options: [{ id: "different", label: "Blue" }] }],
|
|
["option labels", { options: [{ id: "blue", label: "Red" }] }],
|
|
["prompt", { prompt: "A different question?" }],
|
|
])("rejects conflicting canonical %s at creation", (_name, changes) => {
|
|
const payload = {
|
|
version: 1,
|
|
questions: [{ id: "color", prompt: "Color?", required: true, selectionMode: "single", options: [{ id: "blue", label: "Blue" }] }],
|
|
questionSet: { schema: "paperclip.question_set.v1", questions: [{ id: "color", prompt: "Color?", required: true, answerMode: "single_select", options: [{ id: "blue", label: "Blue" }], ...changes }] },
|
|
};
|
|
expect(() => createIssueThreadInteractionSchema.parse({ kind: "ask_user_questions", payload })).toThrow("must match");
|
|
expect(askUserQuestionsPayloadSchema.parse(payload).questions).toHaveLength(1);
|
|
});
|
|
|
|
it("rejects unsafe request_confirmation target hrefs", () => {
|
|
const base = {
|
|
kind: "request_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Proceed?",
|
|
target: {
|
|
type: "custom",
|
|
key: "external-checklist",
|
|
revisionId: "checklist-v1",
|
|
label: "Checklist v1",
|
|
},
|
|
},
|
|
} as const;
|
|
|
|
for (const href of [
|
|
"javascript:alert(1)",
|
|
"data:text/html,hi",
|
|
"//evil.example/path",
|
|
"file:///tmp/x",
|
|
"mailto:user@example.com",
|
|
"slack://channel?id=1",
|
|
"vscode://file/tmp/x",
|
|
"ftp://example.com/file",
|
|
]) {
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
target: {
|
|
...base.payload.target,
|
|
href,
|
|
},
|
|
},
|
|
})).toThrow("href must be a root-relative path, same-page fragment, or http(s) URL");
|
|
}
|
|
});
|
|
|
|
it("parses request_checkbox_confirmation payloads with checkbox defaults", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_checkbox_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Which items should be archived?",
|
|
options: [
|
|
{ id: "item-1", label: "Draft report" },
|
|
{ id: "item-2", label: "Old screenshot", description: "Captured during QA." },
|
|
],
|
|
defaultSelectedOptionIds: ["item-2"],
|
|
minSelected: 0,
|
|
maxSelected: 2,
|
|
acceptLabel: "Archive selected",
|
|
rejectRequiresReason: true,
|
|
target: {
|
|
type: "issue_document",
|
|
key: "plan",
|
|
revisionId: "33333333-3333-4333-8333-333333333333",
|
|
revisionNumber: 2,
|
|
},
|
|
},
|
|
});
|
|
|
|
expect(parsed).toMatchObject({
|
|
kind: "request_checkbox_confirmation",
|
|
continuationPolicy: "wake_assignee",
|
|
payload: {
|
|
allowDeclineReason: true,
|
|
defaultSelectedOptionIds: ["item-2"],
|
|
minSelected: 0,
|
|
maxSelected: 2,
|
|
},
|
|
});
|
|
});
|
|
|
|
it("rejects invalid request_checkbox_confirmation option references and bounds", () => {
|
|
const base = {
|
|
kind: "request_checkbox_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Which items should be archived?",
|
|
options: [
|
|
{ id: "item-1", label: "Draft report" },
|
|
{ id: "item-2", label: "Old screenshot" },
|
|
],
|
|
},
|
|
} as const;
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
options: [
|
|
{ id: "item-1", label: "Draft report" },
|
|
{ id: "item-1", label: "Duplicate" },
|
|
],
|
|
},
|
|
})).toThrow("Option ids must be unique within one checkbox confirmation");
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
defaultSelectedOptionIds: ["missing"],
|
|
},
|
|
})).toThrow("defaultSelectedOptionIds must reference existing option ids");
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
defaultSelectedOptionIds: ["item-1"],
|
|
minSelected: 2,
|
|
},
|
|
})).toThrow("defaultSelectedOptionIds must satisfy minSelected");
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
minSelected: 2,
|
|
maxSelected: 1,
|
|
},
|
|
})).toThrow("maxSelected must be greater than or equal to minSelected");
|
|
});
|
|
|
|
it("rejects unsafe request_checkbox_confirmation target hrefs", () => {
|
|
const base = {
|
|
kind: "request_checkbox_confirmation",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Which items should be archived?",
|
|
options: [{ id: "item-1", label: "Draft report" }],
|
|
target: {
|
|
type: "custom",
|
|
key: "external-checklist",
|
|
revisionId: "checklist-v1",
|
|
label: "Checklist v1",
|
|
},
|
|
},
|
|
} as const;
|
|
|
|
for (const href of ["file:///tmp/x", "slack://channel?id=1", "vscode://file/tmp/x"]) {
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
target: {
|
|
...base.payload.target,
|
|
href,
|
|
},
|
|
},
|
|
})).toThrow("href must be a root-relative path, same-page fragment, or http(s) URL");
|
|
}
|
|
});
|
|
|
|
it("accepts empty checkbox selections and rejects duplicate selected option ids", () => {
|
|
expect(acceptIssueThreadInteractionSchema.parse({ selectedOptionIds: [] })).toEqual({
|
|
selectedOptionIds: [],
|
|
});
|
|
|
|
expect(() => acceptIssueThreadInteractionSchema.parse({
|
|
selectedOptionIds: ["item-1", "item-1"],
|
|
})).toThrow("selectedOptionIds must be unique");
|
|
});
|
|
|
|
it("parses request_item_verdicts payloads with defaults", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_item_verdicts",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Review these generated items.",
|
|
items: [
|
|
{ id: "api", label: "API route", description: "Server submit endpoint" },
|
|
{ id: "docs", label: "Docs", previewMarkdown: "Document the route." },
|
|
],
|
|
},
|
|
});
|
|
|
|
expect(parsed).toMatchObject({
|
|
kind: "request_item_verdicts",
|
|
continuationPolicy: "wake_assignee",
|
|
payload: {
|
|
verdicts: ["approve", "reject"],
|
|
requireReasonOn: ["reject"],
|
|
allowBulkApprove: true,
|
|
},
|
|
});
|
|
});
|
|
|
|
it("accepts request_item_verdicts defer when enabled explicitly", () => {
|
|
const parsed = createIssueThreadInteractionSchema.parse({
|
|
kind: "request_item_verdicts",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Review these generated items.",
|
|
items: [{ id: "api", label: "API route" }],
|
|
verdicts: ["approve", "reject", "defer"],
|
|
requireReasonOn: ["reject", "defer"],
|
|
},
|
|
});
|
|
|
|
expect(parsed).toMatchObject({
|
|
kind: "request_item_verdicts",
|
|
payload: {
|
|
verdicts: ["approve", "reject", "defer"],
|
|
requireReasonOn: ["reject", "defer"],
|
|
},
|
|
});
|
|
});
|
|
|
|
it("rejects invalid request_item_verdicts item and reason references", () => {
|
|
const base = {
|
|
kind: "request_item_verdicts",
|
|
payload: {
|
|
version: 1,
|
|
prompt: "Review these generated items.",
|
|
items: [
|
|
{ id: "api", label: "API route" },
|
|
{ id: "docs", label: "Docs" },
|
|
],
|
|
},
|
|
} as const;
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
items: [],
|
|
},
|
|
})).toThrow();
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
items: [
|
|
{ id: "api", label: "API route" },
|
|
{ id: "api", label: "Duplicate" },
|
|
],
|
|
},
|
|
})).toThrow("Item ids must be unique within one item verdict request");
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
items: Array.from({ length: 201 }, (_value, index) => ({
|
|
id: `item-${index}`,
|
|
label: `Item ${index}`,
|
|
})),
|
|
},
|
|
})).toThrow();
|
|
|
|
expect(() => createIssueThreadInteractionSchema.parse({
|
|
...base,
|
|
payload: {
|
|
...base.payload,
|
|
verdicts: ["approve", "reject"],
|
|
requireReasonOn: ["defer"],
|
|
},
|
|
})).toThrow("requireReasonOn must reference enabled verdicts");
|
|
});
|
|
|
|
it("rejects duplicate request_item_verdicts submit ids", () => {
|
|
expect(() => submitIssueThreadInteractionVerdictsSchema.parse({
|
|
verdicts: [
|
|
{ id: "api", verdict: "approve" },
|
|
{ id: "api", verdict: "reject", reason: "Needs revision" },
|
|
],
|
|
})).toThrow("verdict item ids must be unique");
|
|
});
|
|
});
|