From 8c3461a1d66d8c30284e8d35afea6a408f04816e Mon Sep 17 00:00:00 2001 From: Mike Dalessio Date: Fri, 20 Mar 2026 19:53:02 -0400 Subject: [PATCH] Remove @regexp_timeout_reported from AutoLinkScrubber No longer needed since the scrubber is not memoized across calls and the text node length guard prevents the performance issue. Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/auto_link_scrubber.rb | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/auto_link_scrubber.rb b/lib/auto_link_scrubber.rb index e36039be9..17191c2ea 100644 --- a/lib/auto_link_scrubber.rb +++ b/lib/auto_link_scrubber.rb @@ -22,13 +22,12 @@ class AutoLinkScrubber < Loofah::Scrubber def initialize @direction = :top_down - @regexp_timeout_reported = false end def scrub(node) return Loofah::Scrubber::STOP if EXCLUDED_ELEMENTS.include?(node.name) - if node.text? && !@regexp_timeout_reported + if node.text? replacement = autolink_text_node(node) node.replace(replacement) if replacement end @@ -78,7 +77,6 @@ class AutoLinkScrubber < Loofah::Scrubber links rescue Regexp::TimeoutError => error Sentry.capture_exception error if Fizzy.saas? - @regexp_timeout_reported = true [] end