@automagik/omni 2.260422.5 → 2.260422.6

package/db/drizzle/0028_drop_trigger_rate_limit.sql

@@ -0,0 +1,15 @@
+-- #384: Drop inbound rate limiter on the dispatcher.
+--
+-- The `trigger_rate_limit` column gated raw inbound `message.received` events
+-- (default 5 msgs / 60s per user-channel-instance) before they reached the
+-- `MessageDebouncer`. Fast-typed conversational bursts — natural on WhatsApp —
+-- routinely exceeded the cap and the limiter silently dropped messages with no
+-- feedback, retry, or DLQ, corrupting agent context downstream.
+--
+-- The debouncer already collapses bursts into a single dispatch (messageCount:N),
+-- so this cap was both redundant and actively harmful. The `RateLimiter` class,
+-- inbound+reaction rate gates, and `trigger_rate_limit` column are all removed.
+-- If backend-protection caps are later needed, they must be dispatch-level
+-- (counting completed agent runs), not message-level.
+
+ALTER TABLE "instances" DROP COLUMN "trigger_rate_limit";

package/db/drizzle/meta/_journal.json

@@ -197,6 +197,13 @@
       "when": 1776766900000,
       "tag": "0027_bridge_tmux_session",
       "breakpoints": true
+    },
+    {
+      "idx": 28,
+      "version": "7",
+      "when": 1777000000000,
+      "tag": "0028_drop_trigger_rate_limit",
+      "breakpoints": true
     }
   ]
 }

package/dist/index.js

@@ -58796,7 +58796,6 @@ var init_schema2 = __esm(() => {
     triggerReactions: jsonb("trigger_reactions").$type(),
     triggerMentionPatterns: jsonb("trigger_mention_patterns").$type(),
     triggerMode: varchar("trigger_mode", { length: 20 }).notNull().default("round-trip"),
-    triggerRateLimit: integer("trigger_rate_limit").notNull().default(5),
     inboundMaxAgeMinutes: integer("inbound_max_age_minutes").notNull().default(10),
     profileName: varchar("profile_name", { length: 255 }),
     profilePicUrl: text("profile_pic_url"),
@@ -113804,7 +113803,7 @@ import { fileURLToPath } from "url";
 // package.json
 var package_default = {
   name: "@automagik/omni",
-  version: "2.260422.5",
+  version: "2.260422.6",
   description: "LLM-optimized CLI for Omni",
   type: "module",
   bin: {

package/dist/server/index.js

@@ -224451,7 +224451,7 @@ var init_sentry_scrub = __esm(() => {
 var require_package8 = __commonJS((exports, module) => {
   module.exports = {
     name: "@omni/api",
-    version: "2.260422.5",
+    version: "2.260422.6",
     type: "module",
     exports: {
       ".": {
@@ -230193,7 +230193,6 @@ var init_schema2 = __esm(() => {
     triggerReactions: jsonb("trigger_reactions").$type(),
     triggerMentionPatterns: jsonb("trigger_mention_patterns").$type(),
     triggerMode: varchar("trigger_mode", { length: 20 }).notNull().default("round-trip"),
-    triggerRateLimit: integer("trigger_rate_limit").notNull().default(5),
     inboundMaxAgeMinutes: integer("inbound_max_age_minutes").notNull().default(10),
     profileName: varchar("profile_name", { length: 255 }),
     profilePicUrl: text("profile_pic_url"),
@@ -282871,38 +282870,6 @@ function createPluginAckProvider(plugin6) {
   };
 }
 
-class RateLimiter {
-  counters = new Map;
-  windowMs;
-  constructor(windowMs = DEFAULT_RATE_WINDOW_MS) {
-    this.windowMs = windowMs;
-  }
-  isAllowed(userId, channelType, instanceId, maxPerMinute) {
-    const key = `${userId}:${channelType}:${instanceId}`;
-    const now = Date.now();
-    let timestamps = this.counters.get(key) ?? [];
-    timestamps = timestamps.filter((ts) => now - ts < this.windowMs);
-    if (timestamps.length >= maxPerMinute) {
-      log90.debug("Rate limit exceeded", { key, count: timestamps.length, limit: maxPerMinute });
-      return false;
-    }
-    timestamps.push(now);
-    this.counters.set(key, timestamps);
-    return true;
-  }
-  cleanup() {
-    const now = Date.now();
-    for (const [key, timestamps] of this.counters.entries()) {
-      const active = timestamps.filter((ts) => now - ts < this.windowMs);
-      if (active.length === 0) {
-        this.counters.delete(key);
-      } else {
-        this.counters.set(key, active);
-      }
-    }
-  }
-}
-
 class ReactionDedup {
   seen = new Map;
   maxEntries = 1e4;
@@ -284949,7 +284916,7 @@ async function checkAccessWithFallback(accessService, instance4, payload, channe
   }
   return true;
 }
-async function shouldProcessReaction(agentRunner, accessService, rateLimiter, reactionDedup, payload, metadata, eventType = "reaction.received") {
+async function shouldProcessReaction(agentRunner, accessService, reactionDedup, payload, metadata, eventType = "reaction.received") {
   if (!metadata.instanceId)
     return null;
   const instance4 = await agentRunner.getInstanceWithProvider(metadata.instanceId);
@@ -284962,11 +284929,6 @@ async function shouldProcessReaction(agentRunner, accessService, rateLimiter, re
     return null;
   }
   const channel4 = metadata.channelType ?? instance4.channel;
-  const rateLimit = instance4.triggerRateLimit;
-  if (!rateLimiter.isAllowed(payload.from, channel4, instance4.id, rateLimit ?? DEFAULT_RATE_LIMIT)) {
-    log90.info("Rate limited reaction trigger", { instanceId: instance4.id, from: payload.from });
-    return null;
-  }
   const accessDenied = await checkAccessWithFallback(accessService, instance4, payload, channel4);
   if (accessDenied)
     return null;
@@ -285072,9 +285034,7 @@ async function shouldSkipViaGate(triggerType, firstMsg, instance4, messages4, se
 async function setupAgentDispatcher(eventBus, services, db2) {
   const agentRunner = services.agentRunner;
   const accessService = services.access;
-  const rateLimiter = new RateLimiter;
   const reactionDedup = new ReactionDedup;
-  const cleanupInterval2 = setInterval(() => rateLimiter.cleanup(), 60000);
   const mediaCleanupInterval = setInterval(() => {
     const now = Date.now();
     const leakThreshold = 10 * 60000;
@@ -285109,17 +285069,6 @@ async function setupAgentDispatcher(eventBus, services, db2) {
     const triggerType = classifyMessageTrigger(msgContext);
     if (await shouldSkipViaGate(triggerType, firstMsg, instance4, messages4, services))
       return;
-    const channel4 = firstMsg.metadata.channelType ?? instance4.channel;
-    const rateLimit = instance4.triggerRateLimit;
-    if (!rateLimiter.isAllowed(firstMsg.payload.from, channel4, instance4.id, rateLimit ?? DEFAULT_RATE_LIMIT)) {
-      log90.info("Rate limited (debounced trigger)", {
-        instanceId: instance4.id,
-        from: firstMsg.payload.from,
-        channel: channel4,
-        bufferedCount: messages4.length
-      });
-      return;
-    }
     if (firstMsg.metadata.journeyTracked && firstMsg.metadata.correlationId) {
       const tracker = getJourneyTracker();
       tracker.recordCheckpoint(firstMsg.metadata.correlationId, "T5", JOURNEY_STAGES.T5);
@@ -285179,7 +285128,7 @@ async function setupAgentDispatcher(eventBus, services, db2) {
       const payload = event.payload;
       const metadata = event.metadata;
       try {
-        const instance4 = await shouldProcessReaction(agentRunner, accessService, rateLimiter, reactionDedup, payload, metadata);
+        const instance4 = await shouldProcessReaction(agentRunner, accessService, reactionDedup, payload, metadata);
         if (!instance4)
           return;
         const traceId = metadata.traceId ?? generateCorrelationId("trc");
@@ -285208,7 +285157,7 @@ async function setupAgentDispatcher(eventBus, services, db2) {
       const payload = event.payload;
       const metadata = event.metadata;
       try {
-        const instance4 = await shouldProcessReaction(agentRunner, accessService, rateLimiter, reactionDedup, payload, metadata, "reaction.removed");
+        const instance4 = await shouldProcessReaction(agentRunner, accessService, reactionDedup, payload, metadata, "reaction.removed");
         if (!instance4)
           return;
         const traceId = metadata.traceId ?? generateCorrelationId("trc");
@@ -285278,14 +285227,12 @@ async function setupAgentDispatcher(eventBus, services, db2) {
     log90.info("Agent dispatcher initialized (message + reaction + reaction-removed + media triggers)");
   } catch (error2) {
     log90.error("Failed to set up agent dispatcher", { error: String(error2) });
-    clearInterval(cleanupInterval2);
     clearInterval(mediaCleanupInterval);
     debouncer.clear();
     throw error2;
   }
   return async () => {
     log90.info("Shutting down agent dispatcher");
-    clearInterval(cleanupInterval2);
     clearInterval(mediaCleanupInterval);
     debouncer.clear();
     for (const [mediaId, pending] of mediaCompletions.entries()) {
@@ -285331,7 +285278,7 @@ async function setupAgentDispatcher(eventBus, services, db2) {
     log90.info("Agent dispatcher shutdown complete");
   };
 }
-var log90, _natsGenieProviderCtor, QUOTED_MESSAGE_MAX_CHARS = 4000, DM_HISTORY_LIMIT = 20, DEFAULT_RATE_LIMIT = 5, DEFAULT_RATE_WINDOW_MS = 60000, CHANNEL_MESSAGE_LIMITS, DEFAULT_MESSAGE_LIMIT = 4000, MEDIA_BASE_PATH3, MEDIA_ICONS, MEDIA_WAIT_NULL, mediaCompletions, mediaResultCache, DEFAULT_SEND_MEDIA_PATH_TYPES, BOT_PREFIX = "\uD83E\uDD16 ", activeStreams, sessionActivityStore, PROC_REACT_START, PROC_REACT_DONE = "\u2705", providerCache, openclawClientPool, nullFilterWarnedInstances, DEFAULT_GATE_MODEL = "gemini-3-flash-preview", GATE_TIMEOUT_MS = 3000, setupAgentResponder;
+var log90, _natsGenieProviderCtor, QUOTED_MESSAGE_MAX_CHARS = 4000, DM_HISTORY_LIMIT = 20, CHANNEL_MESSAGE_LIMITS, DEFAULT_MESSAGE_LIMIT = 4000, MEDIA_BASE_PATH3, MEDIA_ICONS, MEDIA_WAIT_NULL, mediaCompletions, mediaResultCache, DEFAULT_SEND_MEDIA_PATH_TYPES, BOT_PREFIX = "\uD83E\uDD16 ", activeStreams, sessionActivityStore, PROC_REACT_START, PROC_REACT_DONE = "\u2705", providerCache, openclawClientPool, nullFilterWarnedInstances, DEFAULT_GATE_MODEL = "gemini-3-flash-preview", GATE_TIMEOUT_MS = 3000, setupAgentResponder;
 var init_agent_dispatcher = __esm(() => {
   init_src2();
   init_src();
@@ -292993,6 +292940,39 @@ var init_dist6 = __esm(() => {
   init_zod();
 });
 
+// ../api/src/schemas/date-query.ts
+function parseDateOrIssue(paramName, v2, ctx) {
+  const parsed = new Date(v2);
+  if (Number.isNaN(parsed.getTime())) {
+    ctx.addIssue({
+      code: exports_external.ZodIssueCode.custom,
+      message: `invalid ${paramName} parameter: expected ISO 8601 date string, got "${v2}"`
+    });
+    return exports_external.NEVER;
+  }
+  return parsed;
+}
+var optionalDateParam = (paramName) => exports_external.string().optional().transform((v2, ctx) => {
+  if (v2 === undefined || v2 === "")
+    return;
+  return parseDateOrIssue(paramName, v2, ctx);
+}), requiredDateParam = (paramName) => exports_external.string({
+  required_error: `${paramName} is required`,
+  invalid_type_error: `${paramName} must be a string`
+}).transform((v2, ctx) => {
+  if (v2 === "") {
+    ctx.addIssue({
+      code: exports_external.ZodIssueCode.custom,
+      message: `${paramName} is required`
+    });
+    return exports_external.NEVER;
+  }
+  return parseDateOrIssue(paramName, v2, ctx);
+});
+var init_date_query = __esm(() => {
+  init_zod();
+});
+
 // ../api/src/routes/v2/access.ts
 var accessRoutes, listQuerySchema, createRuleSchema, updateRuleSchema, checkAccessSchema;
 var init_access3 = __esm(() => {
@@ -293000,6 +292980,7 @@ var init_access3 = __esm(() => {
   init_src();
   init_dist2();
   init_zod();
+  init_date_query();
   accessRoutes = new Hono2;
   listQuerySchema = exports_external.object({
     instanceId: exports_external.string().uuid().optional(),
@@ -293014,7 +292995,7 @@ var init_access3 = __esm(() => {
     priority: exports_external.number().int().default(0).describe("Rule priority (higher = checked first)"),
     enabled: exports_external.boolean().default(true).describe("Whether rule is active"),
     reason: exports_external.string().optional().describe("Human-readable reason"),
-    expiresAt: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined).describe("Optional expiration"),
+    expiresAt: optionalDateParam("expiresAt").describe("Optional expiration (ISO 8601 date string)"),
     action: exports_external.enum(["block", "allow", "silent_block"]).default("block").describe("Action to take"),
     blockMessage: exports_external.string().optional().describe("Custom block message")
   });
@@ -294177,25 +294158,14 @@ function resolveContactName2(chat2, contactNames) {
   }
   return;
 }
-var chatsRoutes, ChatTypeSchema, listQuerySchema6, createChatSchema, updateChatSchema, addParticipantSchema, updateParticipantRoleSchema, chatChannelActionSchema, muteActionSchema, labelBodySchema, optionalDateParam = (paramName) => exports_external.string().optional().transform((v2, ctx) => {
-  if (v2 === undefined || v2 === "")
-    return;
-  const parsed = new Date(v2);
-  if (Number.isNaN(parsed.getTime())) {
-    ctx.addIssue({
-      code: exports_external.ZodIssueCode.custom,
-      message: `invalid ${paramName} parameter: expected ISO 8601 date string, got "${v2}"`
-    });
-    return exports_external.NEVER;
-  }
-  return parsed;
-}), listMessagesQuerySchema, markChatReadSchema, disappearingSchema, DISAPPEARING_DURATIONS, syncNamesSchema, clearSessionSchema;
+var chatsRoutes, ChatTypeSchema, listQuerySchema6, createChatSchema, updateChatSchema, addParticipantSchema, updateParticipantRoleSchema, chatChannelActionSchema, muteActionSchema, labelBodySchema, listMessagesQuerySchema, markChatReadSchema, disappearingSchema, DISAPPEARING_DURATIONS, syncNamesSchema, clearSessionSchema;
 var init_chats2 = __esm(() => {
   init_dist6();
   init_src();
   init_dist2();
   init_zod();
   init_session_cleaner();
+  init_date_query();
   init_api_keys();
   chatsRoutes = new Hono2;
   ChatTypeSchema = exports_external.enum([
@@ -294843,12 +294813,13 @@ var init_dead_letters3 = __esm(() => {
   init_dist6();
   init_dist2();
   init_zod();
+  init_date_query();
   deadLettersRoutes = new Hono2;
   listQuerySchema8 = exports_external.object({
     status: exports_external.string().optional().transform((v2) => v2?.split(",")),
     eventType: exports_external.string().optional().transform((v2) => v2?.split(",") ?? undefined),
-    since: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
-    until: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
+    since: optionalDateParam("since"),
+    until: optionalDateParam("until"),
     limit: exports_external.coerce.number().int().min(1).max(100).default(50),
     cursor: exports_external.string().optional()
   });
@@ -294908,11 +294879,12 @@ var init_event_ops3 = __esm(() => {
   init_dist6();
   init_dist2();
   init_zod();
+  init_date_query();
   init_api_keys();
   eventOpsRoutes = new Hono2;
   replayOptionsSchema = exports_external.object({
-    since: exports_external.string().datetime().transform((v2) => new Date(v2)),
-    until: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
+    since: requiredDateParam("since"),
+    until: optionalDateParam("until"),
     eventTypes: exports_external.array(exports_external.string()).optional(),
     instanceId: exports_external.string().uuid().optional(),
     limit: exports_external.number().int().positive().max(1e5).optional(),
@@ -294995,6 +294967,7 @@ var init_events4 = __esm(() => {
   init_src();
   init_dist2();
   init_zod();
+  init_date_query();
   init_api_keys();
   eventsRoutes = new Hono2;
   listQuerySchema9 = exports_external.object({
@@ -295004,8 +294977,8 @@ var init_events4 = __esm(() => {
     eventType: exports_external.string().optional().transform((v2) => v2?.split(",")),
     contentType: exports_external.string().optional().transform((v2) => v2?.split(",")),
     direction: exports_external.enum(["inbound", "outbound"]).optional(),
-    since: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
-    until: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
+    since: optionalDateParam("since"),
+    until: optionalDateParam("until"),
     search: exports_external.string().optional(),
     limit: exports_external.coerce.number().int().min(1).max(100).default(50),
     cursor: exports_external.string().optional()
@@ -295026,16 +294999,16 @@ var init_events4 = __esm(() => {
     limit: exports_external.number().int().min(1).max(100).default(50)
   });
   analyticsQuerySchema = exports_external.object({
-    since: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
-    until: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
+    since: optionalDateParam("since"),
+    until: optionalDateParam("until"),
     instanceId: exports_external.string().uuid().optional(),
     granularity: exports_external.enum(["hourly", "daily"]).optional(),
     allTime: exports_external.coerce.boolean().optional()
   });
   timelineQuerySchema = exports_external.object({
     channels: exports_external.string().optional().transform((v2) => v2?.split(",")),
-    since: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
-    until: exports_external.string().datetime().optional().transform((v2) => v2 ? new Date(v2) : undefined),
+    since: optionalDateParam("since"),
+    until: optionalDateParam("until"),
     limit: exports_external.coerce.number().int().min(1).max(100).default(50),
     cursor: exports_external.string().optional()
   });
@@ -303099,7 +303072,7 @@ function validateContactPhone(phone, platformUserId) {
 }
 
 // ../api/src/plugins/sync-worker.ts
-class RateLimiter2 {
+class RateLimiter {
   lastRequest = 0;
   minIntervalMs;
   constructor(requestsPerMinute) {
@@ -303116,7 +303089,7 @@ class RateLimiter2 {
 }
 function getRateLimiter(channelType) {
   const rpm = RATE_LIMITS2[channelType] ?? RATE_LIMITS2.default ?? 20;
-  return new RateLimiter2(rpm);
+  return new RateLimiter(rpm);
 }
 function parseSyncDepth(depth) {
   if (!depth)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@automagik/omni",
-  "version": "2.260422.5",
+  "version": "2.260422.6",
   "description": "LLM-optimized CLI for Omni",
   "type": "module",
   "bin": {
@@ -36,14 +36,14 @@
     "qrcode-terminal": "^0.12.0"
   },
   "devDependencies": {
-    "@omni/api": "2.260422.4",
-    "@omni/channel-discord": "2.260422.4",
-    "@omni/channel-sdk": "2.260422.4",
-    "@omni/channel-slack": "2.260422.4",
-    "@omni/channel-telegram": "2.260422.4",
-    "@omni/channel-whatsapp": "2.260422.4",
-    "@omni/core": "2.260422.4",
-    "@omni/sdk": "2.260422.4",
+    "@omni/api": "2.260422.5",
+    "@omni/channel-discord": "2.260422.5",
+    "@omni/channel-sdk": "2.260422.5",
+    "@omni/channel-slack": "2.260422.5",
+    "@omni/channel-telegram": "2.260422.5",
+    "@omni/channel-whatsapp": "2.260422.5",
+    "@omni/core": "2.260422.5",
+    "@omni/sdk": "2.260422.5",
     "@types/node": "^22.10.3",
     "@types/qrcode-terminal": "^0.12.2",
     "typescript": "^5.7.3"