Skip to content

Fix false positive warnings from parsing inline code/URLs as file paths (#68)#69

Merged
devrimcavusoglu merged 2 commits intomainfrom
fix/false-positive-file-path-warnings
Mar 17, 2026
Merged

Fix false positive warnings from parsing inline code/URLs as file paths (#68)#69
devrimcavusoglu merged 2 commits intomainfrom
fix/false-positive-file-path-warnings

Conversation

@devrimcavusoglu
Copy link
Copy Markdown
Owner

Summary

  • Add looksLikeURL helper to detect protocol-prefixed URLs (http://, https://, ftp://) and domain-like patterns (e.g., example.com/path, pkg.go.dev/encoding/json)
  • Apply URL filtering to backtick-enclosed references, which previously had no URL detection
  • Unify markdown link URL filtering to use the same looksLikeURL helper
  • Relative paths (./, ../) are correctly preserved as file references

Closes #68

Test plan

  • Added TestLooksLikeURL with 10 cases covering protocols, domains, and real file paths
  • Added 7 new TestExtractFileReferences cases: backtick URLs with/without protocols, domain-like paths, mixed URLs with real paths, relative dot-paths, FTP URLs
  • Added TestValidateFolder case verifying URLs in backticks produce no warnings
  • All existing tests continue to pass
  • make test and make lint pass clean

🤖 Generated with Claude Code

devrimcavusoglu and others added 2 commits March 17, 2026 18:48
…hs (#68)

Add looksLikeURL helper to detect protocol-prefixed URLs and domain-like
patterns (e.g., example.com/path) so they are skipped during file reference
extraction. Apply filtering to both backtick matches and markdown link matches.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@devrimcavusoglu devrimcavusoglu merged commit 4f6770a into main Mar 17, 2026
4 checks passed
@devrimcavusoglu devrimcavusoglu deleted the fix/false-positive-file-path-warnings branch March 18, 2026 05:43
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.

Fix false positive warnings from parsing inline code/URLs as file paths

1 participant