Fix stateful regex bug in URL pattern matching

[Copilot]

The #URL_REGEX class property was defined with the g flag, making it stateful. Reusing it across multiple .test() and .exec() calls caused lastIndex to persist, leading to unpredictable matching behavior.

Changes:

• Removed g flag from #URL_REGEX class property definition
• Create fresh global regex instance when needed for #addMatchedRanges()
• .test() now safely uses non-global regex

// Before: stateful, unpredictable
#URL_REGEX = /[a-zA-Z0-9+.-]*:\/\/[^\s]+/g;
this.#addMatchedRanges(content, this.#URL_REGEX, ranges);      // mutates lastIndex
if (this.#URL_REGEX.test(beforePath)) { ... }                   // uses stale lastIndex

// After: stateless, predictable
#URL_REGEX = /[a-zA-Z0-9+.-]*:\/\/[^\s]+/;
this.#addMatchedRanges(content, new RegExp(this.#URL_REGEX, 'g'), ranges);
if (this.#URL_REGEX.test(beforePath)) { ... }                   // fresh state

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

Note

Free review on us!

CodeRabbit is offering free reviews until Thu Nov 06 2025 to showcase some of the refinements we've made.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}