Handle symlinks from breaking the tool

[DMarinhoCodacy]

No description provided.

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

_{TIP This summary will be updated as you push new changes.}

[codacy-production]

Pull Request Overview

The new filtering may silently discard all pathless Semgrep errors, including unrelated configuration, invocation, or runtime failures—not just symlink-related errors. This should be restricted to the known symlink condition or otherwise preserve project-level failures before merging.

Both required behaviors lack automated tests: skipping empty-path errors and retaining valid-path errors.

About this PR

• Add automated tests covering both the empty-path filtering behavior and preservation of valid-path errors.

Test suggestions

• appendErrorToResult skips an error whose location path is empty.
• appendErrorToResult preserves a valid-path error in the result.

Prompt proposal for missing tests

Consider implementing these tests if applicable:
1. appendErrorToResult skips an error whose location path is empty.
2. appendErrorToResult preserves a valid-path error in the result.

_{TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback}