@on-mission/sdk 0.1.8 → 0.1.11

package/dist/index.d.ts

@@ -607,7 +607,7 @@ type SendIntentsModule = {
  * a "tools.call" command, the registry dispatches to the right handler.
  *
  * Usage:
- *   mission.tools.register('gmail_search', async ({ params, context }) => { ... })
+ *   mission.tools.register('search', async ({ params, context }) => { ... })
  *   mission.tools.registerAll(handlers)
  */
 
@@ -713,7 +713,7 @@ type MissionSDK = {
      * Durable Outlook send-intent idempotency ledger. Claim a send's
      * content-identity before dispatching to Graph, mark it sent on a 2xx, release
      * it on a provable pre-dispatch failure — the exactly-once guard behind
-     * `outlook_mail_send`. Each call is a strongly-consistent round-trip to OUR
+     * `send`. Each call is a strongly-consistent round-trip to OUR
      * store via the gateway, never a read of Graph's Sent Items.
      */
     sendIntents: SendIntentsModule;
@@ -836,6 +836,27 @@ type MissionSDK = {
  */
 declare function createMissionSDK(options?: MissionSDKOptions): MissionSDK;
 
+/**
+ * A skill tool failure that carries the upstream HTTP status.
+ *
+ * Skill clients (Airtable, Gmail, …) call third-party REST APIs. When one of
+ * those APIs rejects a request — a 422 for a bad field name, a 404 for a
+ * missing record — the client throws. A plain `Error` loses the status: the
+ * skill runtime's tool dispatch (`POST /_mission/tools/:toolName`) then can't
+ * tell a client mistake from a server fault and blankets every throw to
+ * HTTP 500. That 500 propagates to the executor, which (a) stamps a misleading
+ * "HTTP 500" into the model-facing tool result and (b) treats a deterministic
+ * client error as a retryable server fault.
+ *
+ * Throw this instead of a bare `Error` so the real status survives the hop.
+ * The dispatch layer reads `statusCode` and returns it verbatim, so a 422
+ * stays a 422 all the way to the model.
+ */
+declare class SkillToolHttpError extends Error {
+    readonly statusCode: number;
+    constructor(statusCode: number, message: string);
+}
+
 /**
  * Mission SDK
  *
@@ -867,4 +888,4 @@ type WebhookResult = {
     response?: WebhookResponse;
 };
 
-export { type AgentMessage, type ApiClient, type AttachmentLimits, type AuthCredentials, type AuthModule, type ConfigModule, EmailAttachments, type EventHandler, type EventHandlerRegistry, FileTransferTimeoutError, type FilesModule, type IngressHandler, type IngressModule, type IngressRequest, type IngressResponse, type IngressStreamEvents, type IngressUrl, type LogModule, type MimeBodyParts, type MissionSDK, type MissionSDKOptions, type OAuthCredentials, type RedeemedAttachment, type SendIntentClaimOutcome, type SendIntentCompletion, type SendIntentRecord, type SendIntentStatus, type SendIntentsModule, type SendMessageId, type SendMessageIdInput, type SkillContext, type SkillEvent, type ToolHandler, type ToolsModule, type WebhookRequest, type WebhookResponse, type WebhookResult, clearDebugLog, createMissionSDK, debugLog };
+export { type AgentMessage, type ApiClient, type AttachmentLimits, type AuthCredentials, type AuthModule, type ConfigModule, EmailAttachments, type EventHandler, type EventHandlerRegistry, FileTransferTimeoutError, type FilesModule, type IngressHandler, type IngressModule, type IngressRequest, type IngressResponse, type IngressStreamEvents, type IngressUrl, type LogModule, type MimeBodyParts, type MissionSDK, type MissionSDKOptions, type OAuthCredentials, type RedeemedAttachment, type SendIntentClaimOutcome, type SendIntentCompletion, type SendIntentRecord, type SendIntentStatus, type SendIntentsModule, type SendMessageId, type SendMessageIdInput, type SkillContext, type SkillEvent, SkillToolHttpError, type ToolHandler, type ToolsModule, type WebhookRequest, type WebhookResponse, type WebhookResult, clearDebugLog, createMissionSDK, debugLog };

package/dist/index.js

@@ -13798,10 +13798,6 @@ var eventSubscriptionSchema = external_exports.object({
   eventType: external_exports.string().min(1).describe("Specific event type to subscribe to."),
   reason: external_exports.string().min(1).describe("Why this event matters for the mission.")
 });
-var appSubscriptionSchema = external_exports.object({
-  app: external_exports.string().min(1).describe("App slug to watch at the app level."),
-  reason: external_exports.string().min(1).describe("Why this app-level subscription matters.")
-});
 var toolSettingSchema = external_exports.enum(["disabled", "approval", "enabled"]).describe("Execution setting for the tool.");
 var promiseSchema = external_exports.object({
   text: external_exports.string().min(1).describe("Promise text to add to the plan."),
@@ -13829,40 +13825,37 @@ var agentPlanPatchSchema = external_exports.discriminatedUnion("op", [
     context: inputContextSchema.describe("New planner input context.")
   }),
   external_exports.object({
-    op: external_exports.literal("add_plan_schedule").describe("Append a new schedule to the plan."),
-    schedule: scheduleSchema.describe("Schedule to add. Omit the id field.")
+    op: external_exports.literal("add_plan_schedule").describe("Append a new schedule trigger to the agent."),
+    schedule: scheduleSchema.describe("Schedule trigger to add.")
   }),
   external_exports.object({
-    op: external_exports.literal("remove_plan_schedule").describe("Remove an existing schedule by id."),
-    id: external_exports.string().min(1).describe("Schedule id copied from read_agent_plan.")
+    op: external_exports.literal("remove_plan_schedule").describe("Remove an existing schedule trigger by label."),
+    label: external_exports.string().min(1).describe(
+      "Label of the schedule to remove. Schedule triggers carry no id \u2014 the label is the identity (it also derives the deterministic Temporal schedule handle)."
+    )
   }),
   external_exports.object({
-    op: external_exports.literal("update_plan_schedule").describe("Replace an existing schedule by id."),
-    id: external_exports.string().min(1).describe("Schedule id copied from read_agent_plan."),
-    schedule: scheduleSchema.describe(
-      "Full replacement schedule. Omit the id field."
-    )
+    op: external_exports.literal("update_plan_schedule").describe("Replace an existing schedule trigger by label."),
+    label: external_exports.string().min(1).describe(
+      "Label of the schedule to replace. Schedule triggers carry no id \u2014 the label is the identity."
+    ),
+    schedule: scheduleSchema.describe("Full replacement schedule trigger.")
   }),
   external_exports.object({
-    op: external_exports.literal("add_plan_event_subscription").describe("Append a new event subscription to the plan."),
+    op: external_exports.literal("add_plan_event_subscription").describe("Append a new event-subscription trigger to the agent."),
     eventSubscription: eventSubscriptionSchema.describe(
-      "Event subscription to add. Omit the id field."
+      "Event-subscription trigger to add."
     )
   }),
   external_exports.object({
-    op: external_exports.literal("remove_plan_event_subscription").describe("Remove an existing event subscription by id."),
-    id: external_exports.string().min(1).describe("Event subscription id copied from read_agent_plan.")
-  }),
-  external_exports.object({
-    op: external_exports.literal("add_plan_app_subscription").describe("Append a new app subscription to the plan."),
-    appSubscription: appSubscriptionSchema.describe(
-      "App subscription to add. Omit the id field."
+    op: external_exports.literal("remove_plan_event_subscription").describe(
+      "Remove an existing event-subscription trigger by (app, eventType)."
+    ),
+    app: external_exports.string().min(1).describe("App slug of the subscription to remove."),
+    eventType: external_exports.string().min(1).describe(
+      "Event type of the subscription to remove. Event triggers carry no id \u2014 (app, eventType) is the identity."
     )
   }),
-  external_exports.object({
-    op: external_exports.literal("remove_plan_app_subscription").describe("Remove an existing app subscription by id."),
-    id: external_exports.string().min(1).describe("App subscription id copied from read_agent_plan.")
-  }),
   external_exports.object({
     op: external_exports.literal("set_plan_tool_setting").describe("Replace the setting for an existing tool entry."),
     toolName: qualifiedToolNameSchema.describe(
@@ -13955,13 +13948,13 @@ var pageReadResultSchema = external_exports.object({
   framesDropped: external_exports.number().int().nonnegative()
 });
 var PAGE_MUTATING_TOOL_NAME_LIST = [
-  "browser_type",
-  "browser_click",
-  "browser_act",
-  "browser_navigate",
-  "browser_login",
-  "browser_back",
-  "browser_refresh"
+  "type",
+  "click",
+  "act",
+  "navigate",
+  "login",
+  "back",
+  "refresh"
 ];
 var PAGE_MUTATING_TOOL_NAMES = new Set(PAGE_MUTATING_TOOL_NAME_LIST);
 var mismatchSchema = external_exports.discriminatedUnion("kind", [
@@ -14131,8 +14124,8 @@ var tryParseId = (model, raw) => {
 
 // ../models/src/governance.ts
 var qualifiedToolNameSchema2 = external_exports.string().max(200, "qualified tool name must be \u2264200 chars").regex(
-  /^[A-Za-z0-9_.-]+__[A-Za-z0-9_.-]+$/,
-  "qualified tool name must be `namespace__toolName`"
+  /^(?:[A-Za-z0-9_.-]+__[A-Za-z0-9_.-]+|(?!.*__)[A-Za-z0-9_.-]+)$/,
+  "qualified tool name must be bare `toolName` or `namespace__toolName`"
 ).transform((name) => name);
 var brandedIdSchema = (model) => external_exports.string().max(64).superRefine((raw, ctx) => {
   if (tryParseId(model, raw) === null) {
@@ -14183,6 +14176,31 @@ var thresholdKinds = new Set(
   thresholdKindValues
 );
 
+// ../models/src/schedule-input.ts
+var scheduleInputItemSchema = external_exports.object({
+  label: external_exports.string().min(1).max(200).describe("Stable schedule label unique per agent."),
+  intent: external_exports.string().min(1).max(2e3).describe("What the schedule should make the agent do."),
+  cron: external_exports.string().min(1).max(200).describe("Cron expression in canonical storage format."),
+  timezone: external_exports.string().min(1).max(64).describe(
+    `IANA timezone for cron interpretation. Use the user's current timezone from the environment context unless they name a different one (e.g. "in London time"). If no timezone is in context and the user has not named one, you MUST ask the user \u2014 never supply a guessed or inferred zone, and never default to UTC. An unconfirmed guess is worse than asking.`
+  ),
+  status: external_exports.enum(["active", "paused"]).describe("Whether the schedule should exist in Temporal.")
+});
+var scheduleInputArraySchema = external_exports.array(scheduleInputItemSchema).max(50).superRefine((schedules, ctx) => {
+  const seen = /* @__PURE__ */ new Set();
+  for (const s of schedules) {
+    if (seen.has(s.label)) {
+      ctx.addIssue({
+        code: "custom",
+        message: `Duplicate schedule label: ${s.label}`,
+        path: ["label"]
+      });
+      return;
+    }
+    seen.add(s.label);
+  }
+});
+
 // ../models/src/skill-runtime/result.ts
 var changeSourceSchema = external_exports.union([
   external_exports.literal("broker"),
@@ -14394,10 +14412,83 @@ var stagingProvenanceSchema = zod_default.object({
 });
 var sha256HexSchema = zod_default.string().regex(/^[a-f0-9]{64}$/, "expected a lowercase sha256 hex").brand();
 
+// ../models/src/streaming.ts
+var DEVICE_TOOL_RESULT_BODY_LIMIT_BYTES = 4 * 1024 * 1024;
+
 // ../models/src/token-unit.ts
 var MAX = Number.MAX_SAFE_INTEGER;
 var MAX_BIGINT = BigInt(MAX);
 
+// ../models/src/tools.ts
+var DEVICE_WRITE_TOOL_NAME_LIST = [
+  "file_write",
+  "file_update",
+  "file_apply",
+  "pdf_create",
+  "pdf_edit",
+  "word_create",
+  "word_edit",
+  "excel_create",
+  "excel_edit"
+];
+var DEVICE_WRITE_TOOL_NAMES = new Set(
+  DEVICE_WRITE_TOOL_NAME_LIST
+);
+var DEVICE_DELIVERABLE_TOOL_NAME_LIST = [
+  ...DEVICE_WRITE_TOOL_NAME_LIST,
+  "file_move"
+];
+var DEVICE_READ_TOOL_NAME_LIST = [
+  "file_read",
+  "file_list",
+  "file_find",
+  "excel_read",
+  "word_read",
+  "pdf_read",
+  "pdf_render_page"
+];
+var DEVICE_READ_TOOL_NAMES = new Set(
+  DEVICE_READ_TOOL_NAME_LIST
+);
+var PLATFORM_READ_OR_GATHER_TOOL_NAME_LIST = [
+  // Search / discovery over the library, catalog, and activity store.
+  "library_search",
+  "library_expand",
+  "browse_library",
+  "describe_library",
+  "search_catalog",
+  // Doc / conversation-history & memory reads.
+  "conversation_history_search",
+  "conversation_history_read",
+  "memory_search",
+  // Agent / workspace / connection / inbound state reads.
+  "read_agent_state",
+  "read_agent_plan",
+  "read_agent_config",
+  "list_pending_items",
+  "list_connected_apps",
+  "list_devices",
+  "list_agents",
+  "get_workspace_settings",
+  "view_inbound_access",
+  // Activity (analytics) reads / queries.
+  "query_activity",
+  "describe_activity_schema",
+  "execute_activity_sql",
+  // Self / clock reads.
+  "self_read_mission",
+  "self_read_config",
+  "get_current_time",
+  // Sandbox reads.
+  "read_sandbox_file",
+  "list_sandbox_files",
+  // Library stage-out mints a single-use READ FileRef for a granted file.
+  "library_stage_out"
+];
+var PLATFORM_READ_OR_GATHER_TOOL_NAMES = new Set(
+  PLATFORM_READ_OR_GATHER_TOOL_NAME_LIST
+);
+
 // src/email-attachments.ts
 import { createHash, randomBytes } from "crypto";
 var GMAIL_ATTACHMENT_LIMITS = {
@@ -15294,6 +15385,16 @@ function createSendIntentsModule(api2) {
   return { claim, markSent, release };
 }
 
+// src/skill-tool-http-error.ts
+var SkillToolHttpError = class extends Error {
+  statusCode;
+  constructor(statusCode, message) {
+    super(message);
+    this.name = "SkillToolHttpError";
+    this.statusCode = statusCode;
+  }
+};
+
 // src/tool-registry.ts
 function createToolRegistry() {
   const handlers = /* @__PURE__ */ new Map();
@@ -15326,6 +15427,11 @@ function createToolRegistry() {
 }
 
 // src/sdk.ts
+var skillToolStatus = (err) => {
+  if (err instanceof SkillToolHttpError) return err.statusCode;
+  if (err instanceof Error && err.message.startsWith("Unknown tool")) return 404;
+  return 500;
+};
 var HEARTBEAT_INTERVAL_MS = (() => {
   const fromEnv = process.env.MISSION_HEARTBEAT_INTERVAL_MS;
   if (fromEnv) {
@@ -15558,7 +15664,12 @@ function createMissionSDK(options) {
         const message = err instanceof Error ? err.message : String(err);
         logStderr(`[SDK] Tool call '${toolName}' failed: ${message}`);
         return {
-          status: err instanceof Error && err.message.startsWith("Unknown tool") ? 404 : 500,
+          // Preserve the upstream status when the client knew it: a third-party
+          // 422 (bad field) must not masquerade as a 500 (server fault), or the
+          // executor mislabels the model-facing result and treats a
+          // deterministic client error as retryable. Fall back to 404 for an
+          // unknown tool, else 500 for a genuinely unexpected throw.
+          status: skillToolStatus(err),
           body: JSON.stringify({ error: message })
         };
       }
@@ -15724,6 +15835,7 @@ export {
   EmailAttachments,
   FileTransferTimeoutError,
   MAX_STAGING_UPLOAD_BYTES,
+  SkillToolHttpError,
   clearDebugLog,
   createMissionSDK,
   debugLog,