Skip to content

Conversation

@msukkari
Copy link
Contributor

Add detailed logging and warnings to compileGenericGitHostConfig_file to prevent silent failures during local repository indexing.

The original bug (SOU-181) reported that when an invalid local repository path was provided, the system would fail to index it without any log messages, making debugging difficult. This PR addresses that by introducing specific warnings for various failure scenarios (no paths matched, path not a git repo, no origin URL) and providing clear info logs for successful operations.


Linear Issue: SOU-181

Open in Cursor Open in Web

When a local git repository connection fails to find any valid
repositories, the system now logs warnings instead of failing silently.
This helps users diagnose issues with their local repository paths.

- Added warning when glob pattern matches no paths
- Added warning when paths match but no valid git repos are found
- Added info logs to show progress during local repo compilation
- Added tests to verify the new warning behavior
@cursor
Copy link

cursor bot commented Dec 30, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@coderabbitai
Copy link

coderabbitai bot commented Dec 30, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

3 participants