Add REPETITION_CACHE_TTL and dedupe regexpCache

Introduce REPETITION_CACHE_TTL (300s) and replace magic timeout literals with this constant for repetition cache lookups. Move the regexpCache Map declaration to the class top-level to avoid a duplicate declaration and add JSDoc comments for both the TTL and regexp cache to clarify intent and improve maintainability.

Author: talyguryn

workers/grouper/src/index.ts

@@ -53,6 +53,11 @@ const DB_DUPLICATE_KEY_ERROR = '11000';
  */
 const MAX_CODE_LINE_LENGTH = 140;
 
+/**
+ * TTL for repetition cache lookups in seconds
+ */
+const REPETITION_CACHE_TTL = 300;
+
 /**
  * Worker for handling Javascript events
  */
@@ -87,6 +92,11 @@ export default class GrouperWorker extends Worker {
    */
   private cacheCleanupInterval: NodeJS.Timeout | null = null;
 
+  /**
+   * Cache for compiled RegExp patterns to avoid repeated compilation
+   */
+  private regexpCache = new Map<string, RegExp>();
+
   /**
    * Start consuming messages
    */
@@ -429,11 +439,6 @@ export default class GrouperWorker extends Worker {
     return undefined;
   }
 
-  /**
-   * Cache for compiled RegExp patterns to avoid repeated compilation
-   */
-  private regexpCache = new Map<string, RegExp>();
-
   /**
    * Method that returns matched pattern for event, if event do not match any of patterns return null
    *
@@ -525,7 +530,7 @@ export default class GrouperWorker extends Worker {
             },
             { projection: { _id: 1 } }
           );
-      }, 300);
+      }, REPETITION_CACHE_TTL);
 
       if (repetition) {
         shouldIncrementRepetitionAffectedUsers = false;
@@ -566,7 +571,7 @@ export default class GrouperWorker extends Worker {
             },
             { projection: { _id: 1 } }
           );
-      }, 300);
+      }, REPETITION_CACHE_TTL);
 
       /**
        * If daily repetition exists, don't increment daily affected users