From 49427cfcd2cf13366b2a27bd0069d58566b9c8fc Mon Sep 17 00:00:00 2001
From: ADITYA TIWARI <142050150+ADITYATIWARI342005@users.noreply.github.com>
Date: Thu, 27 Nov 2025 16:40:33 +0530
Subject: [PATCH] Refactor duration regex and remove RegExp.escape polyfill

Removed polyfill for RegExp.escape and updated duration regex.

Signed-off-by: ADITYA TIWARI <142050150+ADITYATIWARI342005@users.noreply.github.com>
---
 .../codemirror-promql/src/complete/hybrid.ts  | 19 +++----------------
 1 file changed, 3 insertions(+), 16 deletions(-)

diff --git a/web/ui/module/codemirror-promql/src/complete/hybrid.ts b/web/ui/module/codemirror-promql/src/complete/hybrid.ts
index 11d18adcef..814147e532 100644
--- a/web/ui/module/codemirror-promql/src/complete/hybrid.ts
+++ b/web/ui/module/codemirror-promql/src/complete/hybrid.ts
@@ -166,22 +166,9 @@ function arrayToCompletionResult(data: Completion[], from: number, to: number, i
   } as CompletionResult;
 }
 
-// Polyfill RegExp.escape for compatibility with ES2024 and TypeScript.
-// Ensures safe, standards-based regex escaping in all environments.
-declare global {
-  interface RegExpConstructor {
-    escape?: (s: string) => string;
-  }
-}
-
-if (!RegExp.escape) {
-  RegExp.escape = function (s: string): string {
-    return s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-  };
-}
-
-// Matches complete PromQL durations, including compound units (e.g., 5m, 1d2h, 1h30m, etc.)
-export const durationWithUnitRegexp = new RegExp(`^(\\d+(${durationTerms.map((term) => RegExp.escape!(term.label)).join('|')}))+$`);
+// Matches complete PromQL durations, including compound units (e.g., 5m, 1d2h, 1h30m, etc.).
+// Duration units are a fixed, safe set (no regex metacharacters), so no escaping is needed.
+export const durationWithUnitRegexp = new RegExp(`^(\\d+(${durationTerms.map((term) => term.label).join('|')}))+$`);
 
 // Determines if a duration already has a complete time unit to prevent autocomplete insertion (issue #15452)
 function hasCompleteDurationUnit(state: EditorState, node: SyntaxNode): boolean {