Skip to content

fix: Fix from_path absolute paths and git argument injection - #28

Merged
Pringled merged 2 commits into
mainfrom
fixes
Apr 22, 2026
Merged

fix: Fix from_path absolute paths and git argument injection#28
Pringled merged 2 commits into
mainfrom
fixes

Conversation

@Pringled

Copy link
Copy Markdown
Member

This PR fixes several issues:

  • from_path now stores repo-relative chunk paths instead of absolute paths
  • Add -- separator in from_git to prevent git option injection via URL
  • Validate path exists and is a directory before indexing
  • Improve _is_git_url to cover scp-like syntax and file://, git://, git+ssh://
  • Drop dead _index_root field

@Pringled
Pringled merged commit 0e71d89 into main Apr 22, 2026
8 checks passed
@Pringled
Pringled deleted the fixes branch April 22, 2026 06:17
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.

1 participant