Git Repo URL telemetry link fix: only Https

[liwilliam2021]

I've implemented a solution to convert git repository URLs to HTTPS format in telemetry. Here's a summary of the changes made:

Added a new convertGitUrlToHttps function in src/utils/git.ts that:

Leaves HTTPS URLs unchanged
Converts SSH URLs (like [email protected]:user/repo.git) to HTTPS format (https://github.com/user/repo.git)
Converts SSH URLs with ssh:// prefix to HTTPS format
Handles invalid URLs gracefully
Modified the getGitRepositoryInfo function to use this new function, converting repository URLs to HTTPS format before storing them in telemetry.

Added comprehensive tests for the new functionality.

These changes ensure that repository URLs in telemetry will work correctly regardless of whether they're in git or HTTPS format. The implementation maintains backward compatibility while adding the new conversion functionality.

If you'd like to test this implementation, you'll need to switch back to "Code" mode to run the tests. The tests should verify that:

HTTPS URLs remain unchanged
SSH URLs are properly converted to HTTPS format
The repository information extraction still works correctly
This solution addresses the issue you mentioned where git version URLs don't work in telemetry by automatically converting them to HTTPS format.

---

Important

Adds convertGitUrlToHttps() to convert SSH URLs to HTTPS in telemetry, with tests for URL conversion.

• Behavior:
  • Adds convertGitUrlToHttps() in git.ts to convert SSH URLs to HTTPS, leaving HTTPS URLs unchanged.
  • Updates getGitRepositoryInfo() in git.ts to use convertGitUrlToHttps() for telemetry.
  • Handles invalid URLs gracefully, returning the original URL if conversion fails.
• Tests:
  • Adds tests for convertGitUrlToHttps() in git.spec.ts to verify URL conversion.
  • Updates tests for getGitRepositoryInfo() in git.spec.ts to check URL conversion to HTTPS.
• Misc:
  • Ensures backward compatibility by maintaining original URL format if conversion is not possible.

^{This description was created by for c4c8a82. You can customize this summary. It will automatically update as commits are pushed.}

[delve-auditor]

✅ No security or compliance issues detected. Reviewed everything up to c4c8a82.

Security Overview

• 🔎 Scanned files: 2 changed file(s)

Detected Code Changes

The diff is too large to display a summary of code changes.

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

[daniel-lxs]

LGTM