Fixes secret scanning alerts migration - locations processing and matching

[theztefan]

This PR addresses a bug reported #1331 where Secret Scanning alerts with non-commit locations (e.g., pull_request_comment, issue_title, issue_body) were:

1. Locations were no being parsed correctly and NullReferenceException exceptions was thrown
2. The matching between non commit and wiki_commit locations was not correct. The issue was the strict URL comparison logic, which failed when the base URL differed between source and target repository.

Changes Made:

• Updated BuildSecretScanningAlertLocation method to make it more robust by safely accessing properties when building the object in src/Octoshift/Services/GithubApi.cs
• Enhanced AreLocationsEqual function in src/Octoshift/Services/SecretScanningAlertService.cs:
  • For commit and wiki_commit locations, the comparison remains strict, checking all relevant fields
  • For all other location types, the comparison now focuses on the unique identifier in the URL (e.g., the comment ID or issue number) by extracting and comparing the final path segment of the URLs.
• Added tests to ensure proper matching for all non-commit location types and that alerts with matching unique identifiers in URLs are correctly updated in the target repository.

• Did you write/update appropriate tests
• Release notes updated (if appropriate)
• Appropriate logging output
• Issue linked
• Docs updated (or issue created)
• New package licenses are added to ThirdPartyNotices.txt (if applicable)

[github-actions]

Unit Test Results

  1 files    1 suites   20s ⏱️
880 tests 880 ✅ 0 💤 0 ❌
881 runs  881 ✅ 0 💤 0 ❌

Results for commit 52f79bb.

♻️ This comment has been updated with latest results.

[ArinGhazarian]

Looks good, the only thing is test.output should be removed.

[github-actions]

Package	Line Rate	Branch Rate	Complexity	Health
gei	81%	73%	596	✔
ado2gh	84%	78%	631	✔
bbs2gh	82%	76%	669	✔
Octoshift	87%	76%	1402	✔
Summary	84% (7217 / 8558)	76% (1687 / 2234)	3298	✔

[ArinGhazarian]

🎉

[ArinGhazarian]

INT tests are all 🟢 merging now.