fix(code-quality): drop regex lookbehind that Lucee's ORO engine rejects

Lucee routes  to its ORO fallback, which throws
"Sequence (?<...) not recognized" — surfaced by a live ERROR:  run
at $analyze. The debug-bar CodeComplexity.cfc carried the same latent pattern.

Both now mask  (so URL strings cannot be eaten) and strip line comments
with a plain  — valid in ORO, Java, and RustCFML engines. Verified
on Lucee 7 against a sample with URLs, tag comments, block comments and code.

Signed-off-by: Peter Amiri <peter@alurium.com>

Author: bpamiri

cli/lucli/services/coverage/CoverageService.cfc

@@ -37,7 +37,11 @@ component output="false" {
 	variables.tagDecision = "<" & "cf(?:if|elseif|loop|while|case|defaultcase|catch)" & variables.bs & "b";
 	variables.tagComment = "<!---.*?--->";
 	variables.blockComment = "/" & variables.bs & "*.*?" & variables.bs & "*/";
-	variables.lineComment = "(?m)(?<!:)//[^" & variables.bs & "n]*";
+	// No lookbehind here: Lucee routes such patterns to its ORO fallback,
+	// which rejects `(?<!...)` ("Sequence (?<...) not recognized"). Mask
+	// `://` first so a bare `//[^\r\n]*` cannot eat the tail of URL strings.
+	variables.urlProtocol = "://";
+	variables.lineComment = "//[^" & variables.bs & "r" & variables.bs & "n]*";
 
 	// Self-initializing counter, written into script function bodies. Single-
 	// quoted so the embedded double quotes need no escaping.
@@ -183,6 +187,7 @@ component output="false" {
 	private string function $stripComments(required string text) {
 		local.rv = reReplace(arguments.text, variables.tagComment, "", "all");
 		local.rv = reReplace(local.rv, variables.blockComment, "", "all");
+		local.rv = reReplace(local.rv, variables.urlProtocol, "__WHEELS_PROTO__", "all");
 		local.rv = reReplace(local.rv, variables.lineComment, "", "all");
 		return local.rv;
 	}

vendor/wheels/wheelstest/system/CodeComplexity.cfc

@@ -37,7 +37,11 @@ component output="false" {
 	variables.scriptFn = variables.wb & "function" & variables.ws & "+[A-Za-z_$][" & variables.ww & "$]*" & variables.ws & "*" & variables.bs & "(";
 	variables.tagFn = "<" & "cffunction" & variables.wb & "[^>]*?" & variables.wb & "name";
 	variables.blockComment = "/" & variables.bs & "*.*?" & variables.bs & "*/";
-	variables.lineComment = "(?m)(?<!:)//[^" & variables.nn & "]*";
+	// No lookbehind here: Lucee routes such patterns to its ORO fallback,
+	// which rejects `(?<!...)` ("Sequence (?<...) not recognized"). Mask
+	// `://` first so a bare `//[^\r\n]*` cannot eat the tail of URL strings.
+	variables.urlProtocol = "://";
+	variables.lineComment = "//[^" & variables.bs & "r" & variables.bs & "n]*";
 	variables.tagComment = "<!---.*?--->";
 
 	/**
@@ -114,6 +118,7 @@ component output="false" {
 	public string function $stripComments(required string text) {
 		local.rv = reReplace(arguments.text, variables.tagComment, "", "all");
 		local.rv = reReplace(local.rv, variables.blockComment, "", "all");
+		local.rv = reReplace(local.rv, variables.urlProtocol, "__WHEELS_PROTO__", "all");
 		local.rv = reReplace(local.rv, variables.lineComment, "", "all");
 		return local.rv;
 	}