mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-10-19 20:44:16 +02:00
## Thinking Path > - Paperclip is the open source app people use to manage AI agents for work. > - A task conversation must let a user guide a running agent and resume stopped work. > - The active run owns its input protocol, even when the user changes the next model or effort. > - Queue delivery waits for a provider receipt, which must be able to persist during the request. > - A stopped startup also needs a clear user action that passes normal task admission. > - This pull request fixes steering delivery, makes queue actions immediate, and restores explicit continuation. > - The benefit is a responsive conversation that can recover without losing saved input. ## Linked Issues or Issue Description **What happened?** A queued message could change from Steer to Interrupt while a native run prepared. A steer request could wait on its own database lock and fail to deliver. A stopped startup could then leave the conversation without a working Retry or message continuation. Interrupt also waited for the server and showed a toast. **Expected behavior** The active run keeps its input protocol. Steer delivers input to that run. Steer and Interrupt clear the submitted queue rows and show the input in the conversation immediately. Failed delivery restores the latest queue with an inline error. An eligible stopped run offers Retry, and authenticated user input can start a fresh turn through normal task admission. **Steps to reproduce** 1. Start a task with a native Paperclip Runner. 2. Change the selected model or effort while that run prepares. 3. Queue a message and press Steer. 4. Observe the provider receipt and queue state during the request. 5. Stop a startup before its provider process begins, then try Retry or send a new message. 6. Repeat queued delivery with a legacy runner and press Interrupt. **Paperclip version or commit** Reproduced on the parent of this branch, `59c07ede7`. **Deployment mode** Authenticated private deployment. The fixes also cover local task conversations. Related work: Refs #12834, Refs #13354, Refs #13275. The open refactor in #13160 moves the same queue route; it does not fix the receipt lock or stopped-run continuation addressed here. ## What Changed - Select queue behavior from the active run's immutable dispatch and runtime resolution. - Leave the run row unlocked during provider acknowledgement, then lock and read it before merging the receipt. - Retain queued input if the target run stops during that wait. Keep inline delivery errors visible after empty queue updates. - Permit exact Retry and authenticated continuation after verified native startup cancellation. Preserve pause, approval, budget, ownership, and process-stop gates. - Carry undelivered native queue input into a fresh turn once the old execution is confirmed stopped. - Show Steer and Interrupt input in the conversation and clear submitted composer rows immediately. Restore the latest queue inline on failure. Remove delivery toasts. - Keep optimistic delivery stable across stale polls, empty queues, and paginated history. Preserve classic Interrupt error handling. - Document recovery and optimistic delivery behavior. Add regression tests across server, shared queue projection, and UI boundaries. ## Verification - Red-green regression tests reproduced the queue protocol, receipt lock, stopped-startup continuation, and optimistic delivery failures. - The focused server route, continuation, queue, and runner boundary suites passed during implementation. - The queue-route suite passes with 78 tests. The three complete conversation UI suites pass with 347 tests. - UI typecheck, production build, and `pnpm check:token-gates` pass. - Workspace `pnpm -r typecheck` and `pnpm build` pass. The local monolithic `pnpm test:run` is still running; remote CI verifies the complete suite on the latest commit. - All CI gates pass on `bd9031ad56abfcde13d13a13488c1b9217c2fd3a`, including the full test shards, runner verification, browser E2E, typecheck, release registry, and canary dry run. - Greptile reports 5/5 for that commit. Both review threads are resolved. ## Risks This changes queue display and explicit continuation admission. The UI must restore rejected delivery without losing other-session edits. The server must preserve concurrent provider result updates and must not resume a process whose stop is uncertain. Focused tests cover these boundaries. This change has no database migration. ## Model Used OpenAI Codex, an agent based on GPT-6. The exact runtime model ID and context window are not exposed in this session. Used reasoning, repository tools, code execution, and browser inspection. ## 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>
165 lines
6.0 KiB
TypeScript
165 lines
6.0 KiB
TypeScript
import type {
|
|
IssueComment,
|
|
IssueQueuedCommentProtocol,
|
|
IssueQueuedCommentQueue,
|
|
IssueQueuedCommentQueueState,
|
|
IssueQueuedCommentSteeringDisposition,
|
|
} from "@paperclipai/shared";
|
|
|
|
function record(value: unknown): Record<string, unknown> | null {
|
|
return typeof value === "object" && value !== null && !Array.isArray(value)
|
|
? (value as Record<string, unknown>)
|
|
: null;
|
|
}
|
|
|
|
const STEERING_DISPOSITIONS = new Set<IssueQueuedCommentSteeringDisposition>([
|
|
"available",
|
|
"unsupported",
|
|
"temporarily_unavailable",
|
|
]);
|
|
const QUEUE_STATES = new Set<IssueQueuedCommentQueueState>([
|
|
"deferred",
|
|
"queued",
|
|
]);
|
|
|
|
/** Defensive boundary shared by IssueDetail and Storybook queue fixtures. */
|
|
export function normalizeIssueQueuedCommentQueue(
|
|
value: unknown,
|
|
fallbackIssueId: string,
|
|
): IssueQueuedCommentQueue {
|
|
const source = record(value);
|
|
const seen = new Set<string>();
|
|
const entries = (Array.isArray(source?.entries) ? source.entries : [])
|
|
.flatMap((candidate, sourcePosition) => {
|
|
const entry = record(candidate);
|
|
const comment = record(entry?.comment);
|
|
const id = typeof comment?.id === "string" ? comment.id : "";
|
|
const body = typeof comment?.body === "string" ? comment.body : null;
|
|
if (!id || body === null || seen.has(id)) return [];
|
|
seen.add(id);
|
|
const position =
|
|
typeof entry?.position === "number" && Number.isFinite(entry.position)
|
|
? entry.position
|
|
: sourcePosition;
|
|
const response = record(entry?.source);
|
|
return [
|
|
{
|
|
comment: comment as unknown as IssueComment,
|
|
position,
|
|
...(response?.kind === "interaction" && typeof response.interactionId === "string"
|
|
&& typeof response.interactionKind === "string" ? { source: {
|
|
kind: "interaction" as const, interactionId: response.interactionId,
|
|
interactionKind: response.interactionKind,
|
|
requiresFreshSession: response.requiresFreshSession === true,
|
|
} } : {}),
|
|
canEdit: entry?.canEdit === true,
|
|
canDiscard: entry?.canDiscard === true,
|
|
},
|
|
];
|
|
})
|
|
.sort((left, right) => left.position - right.position)
|
|
.map((entry, position) => ({ ...entry, position }));
|
|
const disposition = source?.steeringDisposition;
|
|
const state = source?.state;
|
|
const wait = record(source?.executionWait);
|
|
|
|
return {
|
|
issueId:
|
|
typeof source?.issueId === "string" ? source.issueId : fallbackIssueId,
|
|
queueId: typeof source?.queueId === "string" ? source.queueId : null,
|
|
state:
|
|
typeof state === "string" &&
|
|
QUEUE_STATES.has(state as IssueQueuedCommentQueueState)
|
|
? (state as IssueQueuedCommentQueueState)
|
|
: null,
|
|
targetRunId:
|
|
typeof source?.targetRunId === "string" ? source.targetRunId : null,
|
|
revision:
|
|
typeof source?.revision === "string" ? source.revision : "unavailable",
|
|
protocol:
|
|
source?.protocol === "paperclip_runner_v1"
|
|
? "paperclip_runner_v1"
|
|
: "legacy",
|
|
steeringDisposition:
|
|
typeof disposition === "string" &&
|
|
STEERING_DISPOSITIONS.has(
|
|
disposition as IssueQueuedCommentSteeringDisposition,
|
|
)
|
|
? (disposition as IssueQueuedCommentSteeringDisposition)
|
|
: "unsupported",
|
|
entries,
|
|
executionWait: typeof wait?.reason === "string" && typeof wait?.message === "string"
|
|
? { reason: wait.reason, message: wait.message }
|
|
: null,
|
|
};
|
|
}
|
|
|
|
export interface PendingIssueQueuedComment {
|
|
comment: IssueComment;
|
|
targetRunId: string | null;
|
|
}
|
|
|
|
/**
|
|
* Keeps a just-submitted follow-up in the composer queue while the comment and
|
|
* queue endpoints acknowledge it independently. If the server has not yet
|
|
* included every local entry, queue mutations are withheld by clearing the
|
|
* queue id; local discard remains available through the optimistic comment
|
|
* cancellation path.
|
|
*/
|
|
export function mergePendingIssueQueuedComments(params: {
|
|
issueId: string;
|
|
authoritativeQueue: IssueQueuedCommentQueue | null | undefined;
|
|
pendingComments: PendingIssueQueuedComment[];
|
|
fallbackProtocol: IssueQueuedCommentProtocol;
|
|
}): IssueQueuedCommentQueue | null {
|
|
const authoritativeEntries = params.authoritativeQueue?.entries ?? [];
|
|
const authoritativeIds = new Set(
|
|
authoritativeEntries.map((entry) => entry.comment.id),
|
|
);
|
|
const pendingEntries = params.pendingComments
|
|
.filter(({ comment }) => !authoritativeIds.has(comment.id))
|
|
.map(({ comment }, position) => ({
|
|
comment,
|
|
position: authoritativeEntries.length + position,
|
|
canEdit: false,
|
|
canDiscard: true,
|
|
}));
|
|
const entries = [...authoritativeEntries, ...pendingEntries].map(
|
|
(entry, position) => ({ ...entry, position }),
|
|
);
|
|
if (entries.length === 0) return null;
|
|
|
|
const queueAcknowledged = pendingEntries.length === 0;
|
|
const authoritativeOwnsQueue = Boolean(params.authoritativeQueue?.queueId);
|
|
const fallbackTargetRunId =
|
|
params.pendingComments.find((entry) => entry.targetRunId)?.targetRunId ??
|
|
null;
|
|
// An empty snapshot predating this submission does not own its runtime.
|
|
// Use the active run until the server acknowledges an actual queue.
|
|
const protocol = authoritativeOwnsQueue
|
|
? params.authoritativeQueue!.protocol
|
|
: params.fallbackProtocol;
|
|
const targetRunId = authoritativeOwnsQueue
|
|
? (params.authoritativeQueue?.targetRunId ?? null)
|
|
: fallbackTargetRunId;
|
|
|
|
return {
|
|
issueId: params.authoritativeQueue?.issueId ?? params.issueId,
|
|
queueId: queueAcknowledged
|
|
? (params.authoritativeQueue?.queueId ?? null)
|
|
: null,
|
|
state:
|
|
params.authoritativeQueue?.state ?? (targetRunId ? "deferred" : null),
|
|
targetRunId,
|
|
revision: params.authoritativeQueue?.revision ?? "awaiting-server",
|
|
protocol,
|
|
steeringDisposition:
|
|
(authoritativeOwnsQueue ? params.authoritativeQueue?.steeringDisposition : undefined) ??
|
|
(protocol === "paperclip_runner_v1" && targetRunId
|
|
? "temporarily_unavailable"
|
|
: "unsupported"),
|
|
entries,
|
|
executionWait: params.authoritativeQueue?.executionWait ?? null,
|
|
};
|
|
}
|