@sentry/junior 0.267.0 → 0.269.0

package/dist/{work-DUQoNcqP.js → work-BE_qTYGi.js}

@@ -1,15 +1,14 @@
-import { t as __exportAll } from "./rolldown-runtime-nwUcYXjP.js";
-import { At as PluginCredentialFailureError, Kt as failTurnRecord, Ln as getTerminalAssistantMessages, Nn as isTurnExecutionLimitExceededError, Pt as AuthorizationFlowDisabledError, Q as openConversationProjection, Ut as saveTurnCheckpoint, Wt as abandonTurnRecord, qt as getTurnRecord } from "./tasks-0ilmuCux.js";
-import { n as botConfig } from "./config-CdB6yJj5.js";
-import { f as getConversationStore, u as getConversationEventStore } from "./viewer-DrN3oC2X.js";
-import { m as sameActorIdentity, u as agentInstructionProvenance } from "./history-C84mlAwO.js";
-import { Q as stopChildAgentsOf, ct as buildSteeringPiMessage, et as recordSubagentEnded, lt as withoutSavedSteeringMessages, pt as readBlockedReport, tt as recordSubagentStarted } from "./agent-runner-DgNATN4D.js";
-import { E as markAgentInvocationSteered, H as TurnInputCommitLostError, K as isTurnInputCommitLostError, L as agentTypeInstructions, O as reportReplacedAgentInvocation, R as agentTypeStartsWithParentHistory, S as markAgentInvocationMailboxAppended, T as markAgentInvocationRunning, W as getConversationTurnBoundaryError, _ as getLatestAgentInvocationForConversation, c as getAgentInvocation, f as getAgentInvocationMessageId, m as getAgentInvocationTurnId, n as completeAgentInvocation, r as createAgentInvocation, s as getActiveAgentInvocationForConversation, w as markAgentInvocationPaused, y as isTerminalAgentInvocation } from "./store-6EetVP-m.js";
-import { n as getAssistantReplyText } from "./assistant-reply-CT7VcXQe.js";
-import { C as maybePostAutomatedTurnLimitNotice, T as resetAutomatedTurnLimit, w as recordFinishedTurnForAutomatedLimit } from "./session-stop-DkOjt46_.js";
-import { r as appendAndEnqueueInboundMessage } from "./store-DvNZIsCr.js";
-import { t as ConversationTurnLifecycleService } from "./turn-lifecycle-OzLIwvi1.js";
-import { n as getPersistedSandboxState, o as persistThreadStateById, r as getPersistedThreadState } from "./thread-state-DRmdJ2QN.js";
+import { Dn as failTurnRecord, Nt as openConversationProjection, On as getTurnRecord, Tn as abandonTurnRecord, dn as PluginCredentialFailureError, hn as AuthorizationFlowDisabledError, mr as isTurnExecutionLimitExceededError, vr as getTerminalAssistantMessages, wn as saveTurnCheckpoint } from "./turn-session-routing-BqO0qXLk.js";
+import { n as botConfig } from "./config-CPsE4T5L.js";
+import { j as SYSTEM_IDENTITY_ID, x as getConversationStore, y as getConversationEventStore } from "./viewer-DSbD6DF6.js";
+import { _ as sameActorIdentity, p as agentInstructionProvenance } from "./media-85PDJxO_.js";
+import { Et as withoutSavedSteeringMessages, Mt as readBlockedReport, Tt as buildSteeringPiMessage, _t as recordSubagentEnded, ht as stopChildAgentsOf, vt as recordSubagentStarted } from "./agent-runner-CrDrPUtY.js";
+import { E as markAgentInvocationSteered, H as TurnInputCommitLostError, K as isTurnInputCommitLostError, L as agentTypeInstructions, O as reportReplacedAgentInvocation, R as agentTypeStartsWithParentHistory, S as markAgentInvocationMailboxAppended, T as markAgentInvocationRunning, W as getConversationTurnBoundaryError, _ as getLatestAgentInvocationForConversation, c as getAgentInvocation, f as getAgentInvocationMessageId, m as getAgentInvocationTurnId, n as completeAgentInvocation, r as createAgentInvocation, s as getActiveAgentInvocationForConversation, w as markAgentInvocationPaused, y as isTerminalAgentInvocation } from "./store-1DVhoKOL.js";
+import { r as getAssistantReplyText } from "./interruption-marker-g6Xup3Fk.js";
+import { C as maybePostAutomatedTurnLimitNotice, D as recordFinishedTurnForAutomatedLimit, O as resetAutomatedTurnLimit } from "./session-stop-BQW_iFcd.js";
+import { r as appendAndEnqueueInboundMessage } from "./store-DCo2n5iJ.js";
+import { t as ConversationTurnLifecycleService } from "./turn-lifecycle-bf2fIYi_.js";
+import { a as persistThreadStateById, n as getPersistedSandboxState, r as getPersistedThreadState } from "./thread-state-CHy8LS0C.js";
 import { isDeepStrictEqual } from "node:util";
 import { z } from "zod";
 // src/chat/runtime/turn-execution.ts
@@ -86,16 +85,6 @@ async function loadForkHistory(invocation) {
 	return forkHistoryFrom((await openConversationProjection({ conversationId: invocation.parentConversationId })).messages, invocation.parentToolCallId);
 }
 // src/chat/agent-invocations/work.ts
-var work_exports = /* @__PURE__ */ __exportAll({
-	agentInvocationInputText: () => agentInvocationInputText,
-	agentInvocationRunPolicy: () => agentInvocationRunPolicy,
-	buildAgentInvocationInboundMessage: () => buildAgentInvocationInboundMessage,
-	createAgentInvocationWorker: () => createAgentInvocationWorker,
-	createAndEnqueueAgentInvocation: () => createAndEnqueueAgentInvocation,
-	enqueueAgentInvocation: () => enqueueAgentInvocation,
-	resolveAgentInvocationId: () => resolveAgentInvocationId,
-	routeAgentInvocationWork: () => routeAgentInvocationWork
-});
 const agentInvocationMailboxMetadataSchema = z.object({
 	agentInvocationId: z.string().min(1),
 	kind: z.literal("agent_invocation")
@@ -121,6 +110,7 @@ function buildAgentInvocationInboundMessage(invocation, nowMs = Date.now()) {
 		inboundMessageId: getAgentInvocationMessageId(invocation.invocationId),
 		input: {
 			authorId: invocation.invocationId,
+			authorIdentityId: SYSTEM_IDENTITY_ID,
 			text: "[agent invocation]",
 			metadata: {
 				agentInvocationId: invocation.invocationId,
@@ -188,8 +178,7 @@ async function resolveAgentInvocationId(context) {
 	if (context.attempt.messages.length > 0) return;
 	const active = await getActiveAgentInvocationForConversation(context.conversationId);
 	if (active) return active.invocationId;
-	const latest = await getLatestAgentInvocationForConversation(context.conversationId);
-	return latest?.status === "stopped" ? latest.invocationId : void 0;
+	return (await getLatestAgentInvocationForConversation(context.conversationId))?.invocationId;
 }
 /**
 * Give queued parent messages to the running agent invocation of a child as
@@ -267,9 +256,9 @@ function agentInvocationTurnEnd(terminal) {
 /** Project the terminal invocation into the idempotent turn lifecycle. */
 async function persistTerminalLifecycle(terminal) {
 	const turnId = getAgentInvocationTurnId(terminal.invocationId);
-	if (terminal.status === "stopped") await abandonTurnRecord({
+	if (terminal.status !== "completed") await abandonTurnRecord({
 		conversationId: terminal.childConversationId,
-		errorMessage: "Turn cancelled",
+		errorMessage: terminal.status === "stopped" ? "Turn cancelled" : terminal.errorMessage,
 		turnId
 	});
 	await finishTurn(terminal.childConversationId, turnId, agentInvocationTurnEnd(terminal));
@@ -506,8 +495,7 @@ function createAgentInvocationWorker(agentRunner, options = {}) {
 				turnId
 			}));
 		} catch (error) {
-			if (!(error instanceof AgentRunError)) throw error;
-			const runError = error.cause;
+			const runError = error instanceof AgentRunError ? error.cause : error;
 			if (isInvocationInputCommitLost(runError)) return { status: "lost_lease" };
 			if (stopSignal?.aborted) return await finishStopped();
 			const blocking = blockingInvocationError(runError);
@@ -553,11 +541,4 @@ function createAgentInvocationWorker(agentRunner, options = {}) {
 		return { status: "completed" };
 	};
 }
-/** Route invocation-backed work before falling through to existing consumers. */
-function routeAgentInvocationWork(options) {
-	return async (context) => {
-		const invocationId = await resolveAgentInvocationId(context);
-		return invocationId ? await options.invocationWorker(context, invocationId) : await options.fallbackWorker(context);
-	};
-}
-export { work_exports as a, resolveAgentInvocationId as i, createAndEnqueueAgentInvocation as n, AgentRunError as o, enqueueAgentInvocation as r, executeTurn as s, createAgentInvocationWorker as t };
+export { AgentRunError as a, resolveAgentInvocationId as i, createAndEnqueueAgentInvocation as n, executeTurn as o, enqueueAgentInvocation as r, createAgentInvocationWorker as t };

package/dist/{worker-D1l4Ugk0.js → worker-Cm9_sWXg.js}

@@ -1,10 +1,10 @@
 import { t as __exportAll } from "./rolldown-runtime-nwUcYXjP.js";
-import { fn as hasConversationStop, hn as isInvalidConversationRecordError, mn as isFinalAttempt } from "./tasks-0ilmuCux.js";
-import { y as isProviderRetryError } from "./client-BjgbMawj.js";
-import { i as getChatConfig } from "./config-CdB6yJj5.js";
-import { T as withLogContext, b as setSpanAttributes, h as logWarn, m as logInfo, p as logException } from "./logging-DuZ1A4ab.js";
-import { f as getTurnRequestDeadline, m as ConversationQueueMessageRejectedError } from "./job-queue-C0BZSa7r.js";
-import { C as requestAnotherSlice, T as startConversationWork, b as recordConversationRetry, d as countPendingConversationMessages, f as deadLetterAttempt, g as getConversationWorkState, i as beginConversationResume, l as completeConversationStop, m as ensureConversationWake, o as checkInConversationWork, p as drainConversationMailbox, s as clearConsumedConversationWake, t as ackMessages, u as completeConversationWork, x as releaseConversationWork, y as recordAttemptFailure } from "./store-DvNZIsCr.js";
+import { Xn as isInvalidConversationRecordError, Yn as isFinalAttempt, qn as hasConversationStop } from "./turn-session-routing-BqO0qXLk.js";
+import { y as isProviderRetryError } from "./client-Cqf4TboL.js";
+import { i as getChatConfig } from "./config-CPsE4T5L.js";
+import { T as withLogContext, b as setSpanAttributes, h as logWarn, m as logInfo, p as logException } from "./logging-n7knse-O.js";
+import { f as getTurnRequestDeadline, m as ConversationQueueMessageRejectedError } from "./job-queue-DI-k-Bbk.js";
+import { C as requestAnotherSlice, T as startConversationWork, b as recordConversationRetry, d as countPendingConversationMessages, f as deadLetterAttempt, g as getConversationWorkState, i as beginConversationResume, l as completeConversationStop, m as ensureConversationWake, o as checkInConversationWork, p as drainConversationMailbox, s as clearConsumedConversationWake, t as ackMessages, u as completeConversationWork, x as releaseConversationWork, y as recordAttemptFailure } from "./store-DCo2n5iJ.js";
 // src/chat/task-execution/attempt-selection.ts
 function selectContiguousTurnBatch(messages) {
 	const first = messages[0];
@@ -116,6 +116,14 @@ async function recordFailedDeliveryAttempt(args) {
 function isTerminalFailure(failure) {
 	return failure.status === "recorded" && failure.deadLetteredMessages.length > 0 && failure.pendingCount === 0;
 }
+/**
+* Return whether a failure of this attempt can end the conversation work. The
+* retry limit applies to every attempt, also one with no messages, such as a
+* resume. A message also ends it at its own delivery limit.
+*/
+function isLastConversationAttempt(work, messages) {
+	return (work.execution.retryCount ?? 0) + 1 >= 5 || messages.some((message) => isFinalAttempt(message));
+}
 function logRetryExhausted(args) {
 	logException(args.error, "conversation.work.retry.exhausted", {
 		"app.run.id": args.work.execution.runId ?? "unknown",
@@ -403,7 +411,7 @@ async function processConversationWorkInContext(message, options) {
 					conversationId,
 					destination,
 					drain,
-					isFinalAttempt: attemptMessages.some((message) => isFinalAttempt(message)),
+					isFinalAttempt: isLastConversationAttempt(leasedWork, attemptMessages),
 					messages: attemptMessages
 				},
 				conversationId,

package/migrations/0065_user_customization.sql

@@ -0,0 +1 @@
+ALTER TABLE "junior_users" ADD COLUMN "customization" text;

package/migrations/0066_review_items.sql

@@ -0,0 +1,53 @@
+CREATE TABLE "junior_review_item_assignees" (
+	"item_id" text NOT NULL,
+	"user_id" text NOT NULL,
+	CONSTRAINT "junior_review_item_assignees_item_id_user_id_pk" PRIMARY KEY("item_id","user_id")
+);
+--> statement-breakpoint
+CREATE TABLE "junior_review_item_conversations" (
+	"item_id" text NOT NULL,
+	"conversation_id" text NOT NULL,
+	CONSTRAINT "junior_review_item_conversations_item_id_conversation_id_pk" PRIMARY KEY("item_id","conversation_id")
+);
+--> statement-breakpoint
+CREATE TABLE "junior_review_items" (
+	"id" text PRIMARY KEY NOT NULL,
+	"kind" text NOT NULL,
+	"status" text NOT NULL,
+	"visibility" text NOT NULL,
+	"title" text NOT NULL,
+	"body" text NOT NULL,
+	"request" jsonb,
+	"answer" jsonb,
+	"assigned_to_everyone" boolean DEFAULT false NOT NULL,
+	"dedupe_key" text NOT NULL,
+	"subject_id" text,
+	"subject_title" text,
+	"subject_url" text,
+	"added_by_conversation_id" text NOT NULL,
+	"provider" text,
+	"provider_tenant_id" text,
+	"automation_id" text,
+	"closed_at" timestamp with time zone,
+	"closed_by_user_id" text,
+	"close_reason" text,
+	"created_at" timestamp with time zone NOT NULL,
+	"updated_at" timestamp with time zone NOT NULL,
+	CONSTRAINT "junior_review_items_kind_check" CHECK (("junior_review_items"."kind" = 'notice' and "junior_review_items"."request" is null and "junior_review_items"."answer" is null) or ("junior_review_items"."kind" = 'action' and "junior_review_items"."request" is not null)),
+	CONSTRAINT "junior_review_items_status_check" CHECK (("junior_review_items"."status" = 'open' and "junior_review_items"."closed_at" is null) or ("junior_review_items"."status" in ('done', 'dismissed') and "junior_review_items"."closed_at" is not null)),
+	CONSTRAINT "junior_review_items_tenant_check" CHECK (("junior_review_items"."provider" is null) = ("junior_review_items"."provider_tenant_id" is null)),
+	CONSTRAINT "junior_review_items_visibility_check" CHECK ("junior_review_items"."visibility" = 'public' or not "junior_review_items"."assigned_to_everyone")
+);
+--> statement-breakpoint
+ALTER TABLE "junior_review_item_assignees" ADD CONSTRAINT "junior_review_item_assignees_item_id_junior_review_items_id_fk" FOREIGN KEY ("item_id") REFERENCES "public"."junior_review_items"("id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_item_assignees" ADD CONSTRAINT "junior_review_item_assignees_user_id_junior_users_id_fk" FOREIGN KEY ("user_id") REFERENCES "public"."junior_users"("id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_item_conversations" ADD CONSTRAINT "junior_review_item_conversations_item_id_junior_review_items_id_fk" FOREIGN KEY ("item_id") REFERENCES "public"."junior_review_items"("id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_item_conversations" ADD CONSTRAINT "junior_review_item_conversations_conversation_id_junior_conversations_conversation_id_fk" FOREIGN KEY ("conversation_id") REFERENCES "public"."junior_conversations"("conversation_id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_items" ADD CONSTRAINT "junior_review_items_automation_id_junior_automations_id_fk" FOREIGN KEY ("automation_id") REFERENCES "public"."junior_automations"("id") ON DELETE set null ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_items" ADD CONSTRAINT "junior_review_items_closed_by_user_id_junior_users_id_fk" FOREIGN KEY ("closed_by_user_id") REFERENCES "public"."junior_users"("id") ON DELETE no action ON UPDATE no action;--> statement-breakpoint
+CREATE INDEX "junior_review_item_assignees_user_idx" ON "junior_review_item_assignees" USING btree ("user_id");--> statement-breakpoint
+CREATE INDEX "junior_review_item_conversations_conversation_idx" ON "junior_review_item_conversations" USING btree ("conversation_id");--> statement-breakpoint
+CREATE UNIQUE INDEX "junior_review_items_open_dedupe_uidx" ON "junior_review_items" USING btree ("dedupe_key") WHERE "junior_review_items"."status" = 'open';--> statement-breakpoint
+CREATE INDEX "junior_review_items_status_idx" ON "junior_review_items" USING btree ("status","updated_at","id");--> statement-breakpoint
+CREATE INDEX "junior_review_items_subject_idx" ON "junior_review_items" USING btree ("subject_id","status");--> statement-breakpoint
+CREATE INDEX "junior_review_items_automation_idx" ON "junior_review_items" USING btree ("automation_id","status");

package/migrations/0067_review_item_dedupe_tenant.sql

@@ -0,0 +1,2 @@
+DROP INDEX "junior_review_items_open_dedupe_uidx";--> statement-breakpoint
+CREATE UNIQUE INDEX "junior_review_items_open_dedupe_uidx" ON "junior_review_items" USING btree (coalesce("provider", ''),coalesce("provider_tenant_id", ''),"dedupe_key") WHERE "junior_review_items"."status" = 'open';

package/migrations/0068_review_item_slack_cards.sql

@@ -0,0 +1,10 @@
+CREATE TABLE "junior_review_item_slack_cards" (
+	"item_id" text NOT NULL,
+	"conversation_id" text NOT NULL,
+	"team_id" text NOT NULL,
+	"channel_id" text NOT NULL,
+	"message_ts" text NOT NULL,
+	CONSTRAINT "junior_review_item_slack_cards_item_id_conversation_id_pk" PRIMARY KEY("item_id","conversation_id")
+);
+--> statement-breakpoint
+ALTER TABLE "junior_review_item_slack_cards" ADD CONSTRAINT "junior_review_item_slack_cards_item_id_junior_review_items_id_fk" FOREIGN KEY ("item_id") REFERENCES "public"."junior_review_items"("id") ON DELETE cascade ON UPDATE no action;

package/migrations/0069_system_identity.sql

@@ -0,0 +1,25 @@
+-- Create the one system Identity with the fixed id `system`. Code writes this
+-- id without a read, and Message and Conversation rows reference it through
+-- foreign keys, so the row must exist. ON CONFLICT DO NOTHING makes a second
+-- run do nothing.
+INSERT INTO "junior_identities" (
+	"id",
+	"kind",
+	"provider",
+	"provider_tenant_id",
+	"provider_subject_id",
+	"display_name",
+	"created_at",
+	"updated_at"
+)
+VALUES (
+	'system',
+	'system',
+	'junior',
+	'',
+	'system',
+	'System',
+	CURRENT_TIMESTAMP,
+	CURRENT_TIMESTAMP
+)
+ON CONFLICT DO NOTHING;

package/migrations/0070_review_item_foreign_keys.sql

@@ -0,0 +1,5 @@
+ALTER TABLE "junior_review_items" DROP CONSTRAINT "junior_review_items_closed_by_user_id_junior_users_id_fk";
+--> statement-breakpoint
+ALTER TABLE "junior_review_item_slack_cards" ADD CONSTRAINT "junior_review_item_slack_cards_conversation_id_junior_conversations_conversation_id_fk" FOREIGN KEY ("conversation_id") REFERENCES "public"."junior_conversations"("conversation_id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_items" ADD CONSTRAINT "junior_review_items_added_by_conversation_id_junior_conversations_conversation_id_fk" FOREIGN KEY ("added_by_conversation_id") REFERENCES "public"."junior_conversations"("conversation_id") ON DELETE cascade ON UPDATE no action;--> statement-breakpoint
+ALTER TABLE "junior_review_items" ADD CONSTRAINT "junior_review_items_closed_by_user_id_junior_users_id_fk" FOREIGN KEY ("closed_by_user_id") REFERENCES "public"."junior_users"("id") ON DELETE set null ON UPDATE no action;

package/migrations/AGENTS.md

@@ -0,0 +1,17 @@
+# SQL Migrations
+
+`src/db/schema.ts` is the Drizzle schema entry point. This directory is the
+append-only history that brings an existing database to that schema.
+`src/chat/conversations/sql/migrations.ts` owns loading, locking, and the
+bridge-version guard.
+
+## Rules
+
+- Never edit, rename, reorder, or delete an applied migration or its
+  metadata. Add a new migration to correct it.
+- Change the table under `src/db/schema/`, then run
+  `pnpm --filter @sentry/junior db:generate --name <name>`. Commit the SQL,
+  `meta/_journal.json`, and the snapshot together.
+- Put fixed row transformations in the migration that needs them.
+- A schema without the core Drizzle journal must first upgrade through
+  `@sentry/junior@0.107.1`. Do not remove that guard.