fix bug where part of template annotation line was not stripped

In #2541, logic
was added to strip out the first template annotation line. Unfortunately,
the template annotations in Rails span multiple lines, causing
invalid Ruby to be eval'd.

This fix splits the compiled source on ";", which properly isolates
the template annotation for easy removal.

Author: joelhawksley

docs/CHANGELOG.md

@@ -10,9 +10,13 @@ nav_order: 6
 
 ## main
 
-* Fix segfault when Ruby coverage is enabled but template annotations are disabled.
+* Fix segfaults when Ruby coverage is enabled.
 
-    *George Holborn*
+    *George Holborn*, *Joel Hawksley*
+
+* Fix segfault when Ruby coverage is enabled and template annotations are enabled.
+
+    *Joel Hawksley*
 
 * Add `protocol` parameter to `with_request_url` test helper to enable testing with HTTPS protocol.
 

lib/view_component/template.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require "coverage"
 
 module ViewComponent
   class Template
@@ -65,7 +66,7 @@ def compiled_source
         result = super
         # Strip the annotation line to maintain correct line numbers when coverage
         # is running (avoids segfault from negative lineno)
-        result = result.sub(/\A[^\n]*\n/, "") if @strip_annotation_line
+        result = result.partition(";").last if @strip_annotation_line
         result
       end
     end