Fix case-insensitive URL scheme matching, strip trailing punctuation from URLs, and update closableLogger comment

Author: Copilot

internal/logger/global_helpers.go

@@ -52,7 +52,7 @@ func withMutexLock(mu *sync.Mutex, fn func() error) error {
 }
 
 // closableLogger is a constraint for types that have a Close method.
-// This is satisfied by *FileLogger, *JSONLLogger, *MarkdownLogger, *ServerFileLogger, and *ToolsLogger.
+// This is satisfied by *FileLogger, *JSONLLogger, *MarkdownLogger, *ObservedURLDomainsLogger, *ServerFileLogger, and *ToolsLogger.
 type closableLogger interface {
 	*FileLogger | *JSONLLogger | *MarkdownLogger | *ServerFileLogger | *ToolsLogger | *ObservedURLDomainsLogger
 	Close() error

internal/middleware/jqschema_test.go

@@ -1886,6 +1886,16 @@ func TestExtractURLDomains(t *testing.T) {
 		[]string{"docs.example.com", "example.com"},
 		urlutil.ExtractURLDomains(`See https://Example.com/a and http://docs.example.com:8080/x plus https://example.com/b`),
 	)
+	// Case-insensitive scheme matching.
+	assert.Equal(t,
+		[]string{"example.com"},
+		urlutil.ExtractURLDomains(`HTTPS://example.com/path`),
+	)
+	// Trailing punctuation from prose must be stripped.
+	assert.Equal(t,
+		[]string{"example.com"},
+		urlutil.ExtractURLDomains(`See https://example.com, and (https://example.com) for details.`),
+	)
 }
 
 func TestExtractURLDomainsFromValue(t *testing.T) {

internal/urlutil/domains.go

@@ -8,9 +8,10 @@ import (
 )
 
 // urlPattern requires a non-empty hostname candidate and then captures the rest
-// of the URL until common delimiter characters. Matches are still validated with
+// of the URL until common delimiter characters. The (?i) flag makes the scheme
+// match case-insensitive (e.g. "HTTPS://"). Matches are still validated with
 // url.Parse before hostname extraction.
-var urlPattern = regexp.MustCompile(`https?://[^\s/"'<>]+[^\s"'<>]*`)
+var urlPattern = regexp.MustCompile(`(?i)https?://[^\s/"'<>]+[^\s"'<>]*`)
 
 // ExtractURLDomainsFromValue recursively extracts unique URL hostnames from string leaves.
 func ExtractURLDomainsFromValue(value any) []string {
@@ -58,6 +59,12 @@ func ExtractURLDomains(text string) []string {
 
 	domainSet := make(map[string]struct{})
 	for _, match := range matches {
+		// Strip trailing punctuation that may appear when a URL is embedded in
+		// prose (e.g. "https://example.com," or "https://example.com)"). These
+		// characters are valid inside a URL so the regex cannot exclude them
+		// blindly; trimming them from the tail of each candidate is the safest
+		// heuristic.
+		match = strings.TrimRight(match, ".,;:!?)]}\"'")
 		parsed, err := url.Parse(match)
 		if err != nil {
 			continue