@sentry/junior-scheduler 0.72.0 → 0.73.0

package/dist/index.js

@@ -125,7 +125,8 @@ function buildScheduledTaskRunPrompt(args) {
 // src/store.ts
 import {
   agentPluginCredentialSubjectSchema,
-  destinationSchema
+  destinationSchema,
+  isSlackDestination
 } from "@sentry/junior-plugin-api";
 
 // src/cadence.ts
@@ -662,7 +663,7 @@ function parseStoredTask(value) {
   }
   const record = value;
   const destination = destinationSchema.safeParse(record.destination);
-  if (!destination.success) {
+  if (!destination.success || !isSlackDestination(destination.data)) {
     return void 0;
   }
   const credentialSubject = record.credentialSubject === void 0 ? void 0 : agentPluginCredentialSubjectSchema.safeParse(record.credentialSubject);
@@ -1050,7 +1051,8 @@ import { Type } from "@sinclair/typebox";
 import {
   AgentPluginToolInputError,
   agentPluginCredentialSubjectSchema as agentPluginCredentialSubjectSchema2,
-  destinationSchema as destinationSchema2
+  destinationSchema as destinationSchema2,
+  isSlackDestination as isSlackDestination2
 } from "@sentry/junior-plugin-api";
 var TASK_ID_PREFIX = "sched";
 var MAX_LISTED_TASKS = 50;
@@ -1058,30 +1060,36 @@ var DEFAULT_SCHEDULE_TIMEZONE = "America/Los_Angeles";
 function throwToolInputError(error) {
   throw new AgentPluginToolInputError(error);
 }
-function requireActiveDestination(context) {
-  const parsed = destinationSchema2.safeParse(context.destination);
+function requireActiveConversation(context) {
+  const parsed = destinationSchema2.safeParse(context.source);
   if (!parsed.success) {
-    const destination = context.destination;
+    const source = context.source;
     const issues = parsed.error.issues;
-    if (!destination || destination.platform !== "slack") {
-      throwToolInputError("No active Slack destination is available.");
+    if (!source || source.platform !== "slack") {
+      throwToolInputError("No active Slack conversation is available.");
     }
     if (issues.some((issue) => issue.code === "unrecognized_keys")) {
       throwToolInputError(
-        "Active Slack destination must not include unknown fields."
+        "Active Slack conversation must not include unknown fields."
       );
     }
     if (issues.some((issue) => issue.path[0] === "channelId")) {
-      throwToolInputError("Active Slack destination channel is invalid.");
+      throwToolInputError("Active Slack conversation channel is invalid.");
     }
     if (issues.some((issue) => issue.path[0] === "teamId")) {
-      throwToolInputError("Active Slack destination workspace is invalid.");
+      throwToolInputError("Active Slack conversation workspace is invalid.");
     }
-    throwToolInputError("No active Slack destination is available.");
+    throwToolInputError("No active Slack conversation is available.");
+  }
+  if (!isSlackDestination2(parsed.data)) {
+    throwToolInputError("No active Slack conversation is available.");
   }
   return parsed.data;
 }
 function requireRequester(context) {
+  if (context.requester?.platform !== "slack") {
+    throwToolInputError("No active Slack requester context is available.");
+  }
   const userId = context.requester?.userId?.trim();
   if (!userId || userId.toLowerCase() === "unknown") {
     throwToolInputError("No active Slack requester context is available.");
@@ -1132,13 +1140,13 @@ function sameDestination(task, destination) {
   return taskDestination.platform === "slack" && taskDestination.teamId === destination.teamId && taskDestination.channelId === destination.channelId;
 }
 async function getWritableTask(args) {
-  const destination = requireActiveDestination(args.context);
+  const destination = requireActiveConversation(args.context);
   const task = await createSchedulerStore(args.context.state).getTask(
     args.taskId
   );
   if (!task || task.status === "deleted") {
     throwToolInputError(
-      "Scheduled task was not found in the active destination."
+      "Scheduled task was not found in the active Slack conversation."
     );
   }
   if (!sameDestination(task, destination)) {
@@ -1281,7 +1289,7 @@ function createSlackScheduleCreateTaskTool(context) {
       )
     }),
     execute: async (input) => {
-      const destination = requireActiveDestination(context);
+      const destination = requireActiveConversation(context);
       const requester = requireRequester(context);
       const nowMs = Date.now();
       const timezone = input.timezone ?? getDefaultScheduleTimezone();
@@ -1340,7 +1348,7 @@ function createSlackScheduleListTasksTool(context) {
     annotations: { readOnlyHint: true, destructiveHint: false },
     inputSchema: Type.Object({}),
     execute: async () => {
-      const destination = requireActiveDestination(context);
+      const destination = requireActiveConversation(context);
       const tasks = await createSchedulerStore(context.state).listTasksForTeam(
         destination.teamId
       );
@@ -1358,12 +1366,12 @@ function createSlackScheduleListTasksTool(context) {
 }
 function createSlackScheduleUpdateTaskTool(context) {
   return tool({
-    description: "Edit, pause, resume, or reschedule an existing Junior scheduled task in the active Slack conversation. Use only task IDs returned for this destination. Do not move scheduled tasks across conversations.",
+    description: "Edit, pause, resume, or reschedule an existing Junior scheduled task in the active Slack conversation. Use only task IDs returned for this conversation. Do not move scheduled tasks across conversations.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
-        description: "ID of the task to update. Must be from this active Slack destination."
+        description: "ID of the task to update. Must be from this active Slack conversation."
       }),
       task: Type.Optional(Type.String({ minLength: 1, maxLength: 4e3 })),
       schedule: Type.Optional(Type.String({ minLength: 1, maxLength: 300 })),
@@ -1459,12 +1467,12 @@ function createSlackScheduleUpdateTaskTool(context) {
 }
 function createSlackScheduleDeleteTaskTool(context) {
   return tool({
-    description: "Delete one scheduled Junior task from the active Slack conversation. Use only task IDs returned for this destination. Do not delete schedules from threads, other channels, or another user's DM.",
+    description: "Delete one scheduled Junior task from the active Slack conversation. Use only task IDs returned for this conversation. Do not delete schedules from threads, other channels, or another user's DM.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
-        description: "ID of the task to delete. Must be from this active Slack destination."
+        description: "ID of the task to delete. Must be from this active Slack conversation."
       })
     }),
     execute: async ({ task_id }) => {
@@ -1487,12 +1495,12 @@ function createSlackScheduleDeleteTaskTool(context) {
 }
 function createSlackScheduleRunTaskNowTool(context) {
   return tool({
-    description: "Queue an existing active scheduled Junior task to run as soon as possible, without changing its cadence. Use when the user asks to run an existing scheduled task now. Use only task IDs returned for this destination.",
+    description: "Queue an existing active scheduled Junior task to run as soon as possible, without changing its cadence. Use when the user asks to run an existing scheduled task now. Use only task IDs returned for this conversation.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
-        description: "ID of the active task to run now. Must be from this active Slack destination."
+        description: "ID of the active task to run now. Must be from this active Slack conversation."
       })
     }),
     execute: async ({ task_id }) => {
@@ -1535,9 +1543,13 @@ function shouldSkipRun(task, run) {
 }
 function createSchedulerToolContext(ctx) {
   return {
-    credentialSubject: ctx.credentialSubject,
-    destination: ctx.destination?.platform === "slack" ? ctx.destination : void 0,
-    requester: ctx.requester,
+    credentialSubject: ctx.slack?.credentialSubject,
+    source: ctx.source.platform === "slack" ? {
+      platform: "slack",
+      teamId: ctx.source.teamId,
+      channelId: ctx.source.channelId
+    } : void 0,
+    requester: ctx.requester?.platform === "slack" ? ctx.requester : void 0,
     state: ctx.state,
     userText: ctx.userText
   };
@@ -1785,12 +1797,13 @@ function createSchedulerPlugin() {
   return defineJuniorPlugin({
     manifest: {
       name: "scheduler",
+      displayName: "Scheduler",
       description: "Scheduled Junior task management and heartbeat dispatch"
     },
     legacyStatePrefixes: ["junior:scheduler"],
     hooks: {
       tools(ctx) {
-        if (!ctx.destination || ctx.destination.platform !== "slack" || !ctx.requester?.userId) {
+        if (ctx.source.platform !== "slack" || ctx.requester?.platform !== "slack") {
           return {};
         }
         const context = createSchedulerToolContext(ctx);

package/dist/schedule-tools.d.ts

@@ -1,18 +1,18 @@
-import { type AgentPluginCredentialSubject, type Destination, type Requester, type AgentPluginState, type AgentPluginToolDefinition } from "@sentry/junior-plugin-api";
+import { type AgentPluginCredentialSubject, type AgentPluginState, type AgentPluginToolDefinition, type SlackDestination, type SlackRequester } from "@sentry/junior-plugin-api";
 export interface SchedulerToolContext {
     credentialSubject?: AgentPluginCredentialSubject;
-    destination?: Destination;
-    requester?: Requester;
+    requester?: SlackRequester;
+    source?: SlackDestination;
     state: AgentPluginState;
     userText?: string;
 }
 /** Create a tool that stores a scheduled task for the active Slack context. */
 export declare function createSlackScheduleCreateTaskTool(context: SchedulerToolContext): AgentPluginToolDefinition<any>;
-/** Create a tool that lists scheduled tasks for the active Slack destination. */
+/** Create a tool that lists scheduled tasks for the active Slack conversation. */
 export declare function createSlackScheduleListTasksTool(context: SchedulerToolContext): AgentPluginToolDefinition<any>;
-/** Create a tool that edits a scheduled task in the active Slack destination. */
+/** Create a tool that edits a scheduled task in the active Slack conversation. */
 export declare function createSlackScheduleUpdateTaskTool(context: SchedulerToolContext): AgentPluginToolDefinition<any>;
-/** Create a tool that removes a scheduled task from the active Slack destination. */
+/** Create a tool that removes a scheduled task from the active Slack conversation. */
 export declare function createSlackScheduleDeleteTaskTool(context: SchedulerToolContext): AgentPluginToolDefinition<any>;
 /** Create a tool that marks an existing scheduled task due immediately. */
 export declare function createSlackScheduleRunTaskNowTool(context: SchedulerToolContext): AgentPluginToolDefinition<any>;

package/dist/types.d.ts

@@ -1,4 +1,4 @@
-import type { AgentPluginCredentialSubject, Destination } from "@sentry/junior-plugin-api";
+import type { AgentPluginCredentialSubject, SlackDestination } from "@sentry/junior-plugin-api";
 export type ScheduledTaskStatus = "active" | "paused" | "blocked" | "deleted";
 export type ScheduledRunStatus = "pending" | "running" | "completed" | "failed" | "blocked" | "skipped";
 export interface ScheduledTaskPrincipal {
@@ -47,7 +47,7 @@ export interface ScheduledTask {
     createdBy: ScheduledTaskPrincipal;
     conversationAccess?: ScheduledTaskConversationAccess;
     credentialSubject?: AgentPluginCredentialSubject;
-    destination: Destination;
+    destination: SlackDestination;
     executionActor?: ScheduledTaskExecutionActor;
     lastRunAtMs?: number;
     nextRunAtMs?: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/junior-scheduler",
-  "version": "0.72.0",
+  "version": "0.73.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -23,7 +23,7 @@
   ],
   "dependencies": {
     "@sinclair/typebox": "^0.34.49",
-    "@sentry/junior-plugin-api": "0.72.0"
+    "@sentry/junior-plugin-api": "0.73.0"
   },
   "devDependencies": {
     "@types/node": "^25.9.1",

package/src/plugin.ts

@@ -1,9 +1,9 @@
 import {
   defineJuniorPlugin,
-  type Destination,
   type Dispatch,
   type AgentPluginToolDefinition,
   type PluginOperationalReportContent,
+  type SlackDestination,
   type ToolRegistrationHookContext,
 } from "@sentry/junior-plugin-api";
 import { buildScheduledTaskRunPrompt } from "./prompt";
@@ -54,10 +54,16 @@ function createSchedulerToolContext(
   ctx: ToolRegistrationHookContext,
 ): SchedulerToolContext {
   return {
-    credentialSubject: ctx.credentialSubject,
-    destination:
-      ctx.destination?.platform === "slack" ? ctx.destination : undefined,
-    requester: ctx.requester,
+    credentialSubject: ctx.slack?.credentialSubject,
+    source:
+      ctx.source.platform === "slack"
+        ? {
+            platform: "slack",
+            teamId: ctx.source.teamId,
+            channelId: ctx.source.channelId,
+          }
+        : undefined,
+    requester: ctx.requester?.platform === "slack" ? ctx.requester : undefined,
     state: ctx.state,
     userText: ctx.userText,
   };
@@ -183,7 +189,7 @@ function formatTimestamp(timestampMs: number | undefined): string {
     : "none";
 }
 
-function destinationLabel(destination: Destination): string {
+function destinationLabel(destination: SlackDestination): string {
   if (destination.channelId.startsWith("D")) {
     return "Direct Message";
   }
@@ -358,15 +364,15 @@ export function createSchedulerPlugin() {
   return defineJuniorPlugin({
     manifest: {
       name: "scheduler",
+      displayName: "Scheduler",
       description: "Scheduled Junior task management and heartbeat dispatch",
     },
     legacyStatePrefixes: ["junior:scheduler"],
     hooks: {
       tools(ctx) {
         if (
-          !ctx.destination ||
-          ctx.destination.platform !== "slack" ||
-          !ctx.requester?.userId
+          ctx.source.platform !== "slack" ||
+          ctx.requester?.platform !== "slack"
         ) {
           return {} as Record<string, AgentPluginToolDefinition<any>>;
         }

package/src/schedule-tools.ts

@@ -4,11 +4,12 @@ import {
   AgentPluginToolInputError,
   agentPluginCredentialSubjectSchema,
   destinationSchema,
+  isSlackDestination,
   type AgentPluginCredentialSubject,
-  type Destination,
-  type Requester,
   type AgentPluginState,
   type AgentPluginToolDefinition,
+  type SlackDestination,
+  type SlackRequester,
 } from "@sentry/junior-plugin-api";
 import { buildCalendarRecurrence, parseScheduleTimestamp } from "./cadence";
 import { sanitizeScheduledTaskPrincipal } from "./identity";
@@ -25,8 +26,8 @@ import type {
 
 export interface SchedulerToolContext {
   credentialSubject?: AgentPluginCredentialSubject;
-  destination?: Destination;
-  requester?: Requester;
+  requester?: SlackRequester;
+  source?: SlackDestination;
   state: AgentPluginState;
   userText?: string;
 }
@@ -44,26 +45,32 @@ function throwToolInputError(error: string): never {
   throw new AgentPluginToolInputError(error);
 }
 
-function requireActiveDestination(context: SchedulerToolContext): Destination {
-  const parsed = destinationSchema.safeParse(context.destination);
+function requireActiveConversation(
+  context: SchedulerToolContext,
+): SlackDestination {
+  const parsed = destinationSchema.safeParse(context.source);
   if (!parsed.success) {
-    const destination = context.destination as Partial<Destination> | undefined;
+    const source = context.source as Partial<SlackDestination> | undefined;
     const issues = parsed.error.issues as readonly SchemaIssue[];
-    if (!destination || destination.platform !== "slack") {
-      throwToolInputError("No active Slack destination is available.");
+    if (!source || source.platform !== "slack") {
+      throwToolInputError("No active Slack conversation is available.");
     }
     if (issues.some((issue) => issue.code === "unrecognized_keys")) {
       throwToolInputError(
-        "Active Slack destination must not include unknown fields.",
+        "Active Slack conversation must not include unknown fields.",
       );
     }
     if (issues.some((issue) => issue.path[0] === "channelId")) {
-      throwToolInputError("Active Slack destination channel is invalid.");
+      throwToolInputError("Active Slack conversation channel is invalid.");
     }
     if (issues.some((issue) => issue.path[0] === "teamId")) {
-      throwToolInputError("Active Slack destination workspace is invalid.");
+      throwToolInputError("Active Slack conversation workspace is invalid.");
     }
-    throwToolInputError("No active Slack destination is available.");
+    throwToolInputError("No active Slack conversation is available.");
+  }
+
+  if (!isSlackDestination(parsed.data)) {
+    throwToolInputError("No active Slack conversation is available.");
   }
 
   return parsed.data;
@@ -72,6 +79,9 @@ function requireActiveDestination(context: SchedulerToolContext): Destination {
 function requireRequester(
   context: SchedulerToolContext,
 ): ScheduledTaskPrincipal {
+  if (context.requester?.platform !== "slack") {
+    throwToolInputError("No active Slack requester context is available.");
+  }
   const userId = context.requester?.userId?.trim();
   if (!userId || userId.toLowerCase() === "unknown") {
     throwToolInputError("No active Slack requester context is available.");
@@ -99,7 +109,7 @@ function isDmChannel(channelId: string): boolean {
 }
 
 function getConversationAccess(
-  destination: Destination,
+  destination: SlackDestination,
 ): ScheduledTaskConversationAccess {
   if (isDmChannel(destination.channelId)) {
     return { audience: "direct", visibility: "private" };
@@ -139,7 +149,7 @@ function getCredentialSubject(args: {
 
 function sameDestination(
   task: ScheduledTask,
-  destination: Destination,
+  destination: SlackDestination,
 ): boolean {
   const taskDestination = task.destination;
   return (
@@ -153,14 +163,14 @@ async function getWritableTask(args: {
   context: SchedulerToolContext;
   taskId: string;
 }): Promise<ScheduledTask> {
-  const destination = requireActiveDestination(args.context);
+  const destination = requireActiveConversation(args.context);
 
   const task = await createSchedulerStore(args.context.state).getTask(
     args.taskId,
   );
   if (!task || task.status === "deleted") {
     throwToolInputError(
-      "Scheduled task was not found in the active destination.",
+      "Scheduled task was not found in the active Slack conversation.",
     );
   }
 
@@ -369,7 +379,7 @@ export function createSlackScheduleCreateTaskTool(
       ),
     }),
     execute: async (input) => {
-      const destination = requireActiveDestination(context);
+      const destination = requireActiveConversation(context);
       const requester = requireRequester(context);
 
       const nowMs = Date.now();
@@ -426,7 +436,7 @@ export function createSlackScheduleCreateTaskTool(
   });
 }
 
-/** Create a tool that lists scheduled tasks for the active Slack destination. */
+/** Create a tool that lists scheduled tasks for the active Slack conversation. */
 export function createSlackScheduleListTasksTool(
   context: SchedulerToolContext,
 ) {
@@ -436,7 +446,7 @@ export function createSlackScheduleListTasksTool(
     annotations: { readOnlyHint: true, destructiveHint: false },
     inputSchema: Type.Object({}),
     execute: async () => {
-      const destination = requireActiveDestination(context);
+      const destination = requireActiveConversation(context);
 
       const tasks = await createSchedulerStore(context.state).listTasksForTeam(
         destination.teamId,
@@ -455,19 +465,19 @@ export function createSlackScheduleListTasksTool(
   });
 }
 
-/** Create a tool that edits a scheduled task in the active Slack destination. */
+/** Create a tool that edits a scheduled task in the active Slack conversation. */
 export function createSlackScheduleUpdateTaskTool(
   context: SchedulerToolContext,
 ) {
   return tool({
     description:
-      "Edit, pause, resume, or reschedule an existing Junior scheduled task in the active Slack conversation. Use only task IDs returned for this destination. Do not move scheduled tasks across conversations.",
+      "Edit, pause, resume, or reschedule an existing Junior scheduled task in the active Slack conversation. Use only task IDs returned for this conversation. Do not move scheduled tasks across conversations.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
         description:
-          "ID of the task to update. Must be from this active Slack destination.",
+          "ID of the task to update. Must be from this active Slack conversation.",
       }),
       task: Type.Optional(Type.String({ minLength: 1, maxLength: 4000 })),
       schedule: Type.Optional(Type.String({ minLength: 1, maxLength: 300 })),
@@ -573,19 +583,19 @@ export function createSlackScheduleUpdateTaskTool(
   });
 }
 
-/** Create a tool that removes a scheduled task from the active Slack destination. */
+/** Create a tool that removes a scheduled task from the active Slack conversation. */
 export function createSlackScheduleDeleteTaskTool(
   context: SchedulerToolContext,
 ) {
   return tool({
     description:
-      "Delete one scheduled Junior task from the active Slack conversation. Use only task IDs returned for this destination. Do not delete schedules from threads, other channels, or another user's DM.",
+      "Delete one scheduled Junior task from the active Slack conversation. Use only task IDs returned for this conversation. Do not delete schedules from threads, other channels, or another user's DM.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
         description:
-          "ID of the task to delete. Must be from this active Slack destination.",
+          "ID of the task to delete. Must be from this active Slack conversation.",
       }),
     }),
     execute: async ({ task_id }) => {
@@ -615,13 +625,13 @@ export function createSlackScheduleRunTaskNowTool(
 ) {
   return tool({
     description:
-      "Queue an existing active scheduled Junior task to run as soon as possible, without changing its cadence. Use when the user asks to run an existing scheduled task now. Use only task IDs returned for this destination.",
+      "Queue an existing active scheduled Junior task to run as soon as possible, without changing its cadence. Use when the user asks to run an existing scheduled task now. Use only task IDs returned for this conversation.",
     executionMode: "sequential",
     inputSchema: Type.Object({
       task_id: Type.String({
         minLength: 1,
         description:
-          "ID of the active task to run now. Must be from this active Slack destination.",
+          "ID of the active task to run now. Must be from this active Slack conversation.",
       }),
     }),
     execute: async ({ task_id }) => {

package/src/store.ts

@@ -1,6 +1,7 @@
 import {
   agentPluginCredentialSubjectSchema,
   destinationSchema,
+  isSlackDestination,
   type AgentPluginReadState,
   type AgentPluginState,
 } from "@sentry/junior-plugin-api";
@@ -363,7 +364,7 @@ function parseStoredTask(value: unknown): ScheduledTask | undefined {
   }
   const record = value as Partial<ScheduledTask>;
   const destination = destinationSchema.safeParse(record.destination);
-  if (!destination.success) {
+  if (!destination.success || !isSlackDestination(destination.data)) {
     return undefined;
   }
   const credentialSubject =

package/src/types.ts

@@ -1,6 +1,6 @@
 import type {
   AgentPluginCredentialSubject,
-  Destination,
+  SlackDestination,
 } from "@sentry/junior-plugin-api";
 
 export type ScheduledTaskStatus = "active" | "paused" | "blocked" | "deleted";
@@ -72,7 +72,7 @@ export interface ScheduledTask {
   createdBy: ScheduledTaskPrincipal;
   conversationAccess?: ScheduledTaskConversationAccess;
   credentialSubject?: AgentPluginCredentialSubject;
-  destination: Destination;
+  destination: SlackDestination;
   executionActor?: ScheduledTaskExecutionActor;
   lastRunAtMs?: number;
   nextRunAtMs?: number;