Skip to content

[WIP] Fix code based on review comments#238

Closed
Copilot wants to merge 1 commit intonextfrom
copilot/fix-review-comments-code
Closed

[WIP] Fix code based on review comments#238
Copilot wants to merge 1 commit intonextfrom
copilot/fix-review-comments-code

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 11, 2026

  • Fix fingerprint fallback logic in server/src/lib/sarif-utils.ts — apply the reviewer's suggestion to only fall back to full-path when fingerprints are absent, not when both results have fingerprints that don't match (keeps fingerprint mode strict)
  • Add unit test for "fingerprints present but mismatch, locations overlap" case in server/test/src/lib/sarif-utils.test.ts
  • Rebuild server dist bundle
  • Run tests to verify all changes pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants