Greptile Summary

Replaces complex IPv6 regex checking with a simpler colon-counting approach to prevent port stripping from corrupting compressed IPv6 addresses like ::1 and fe80::1.

Key improvements:

• Uses colonCount == 1 as the guard for port stripping instead of !IPV6_REGEX.matches()
• Simpler invariant: IPv6 addresses always have 2+ colons, while hostname:port has exactly one
• Fixes bug where compressed IPv6 forms weren't matched by the existing regex and were being corrupted
• Adds comprehensive test coverage for compressed IPv6 forms

Impact:

• Port stripping continues to work correctly for hostnames and IPv4 addresses with ports
• Compressed IPv6 addresses are no longer corrupted by the port-stripping logic
• Code is simpler and more maintainable than regex-based approach

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The change is a clear improvement over the previous implementation. The colon-counting approach is simpler, more robust, and correctly handles all edge cases including compressed IPv6 forms, bracketed IPv6 with ports, and standard hostname:port patterns. The test coverage is appropriate and the change fixes a real bug without introducing new issues.
• No files require special attention

Important Files Changed

_{Last reviewed commit: acc5324}

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!