Mcp undefined filename bug

[msukkari]

Fixes MCP results returning undefined filenames for local repositories by adding fallbacks for webUrl.

For local repositories, the webUrl field is inherently undefined. The MCP search_code and list_repos tools were directly using file.webUrl and repo.webUrl respectively, leading to "undefined" being displayed. This PR introduces fallbacks to file.fileName.text for search results and repo.repoCloneUrl for repo listings when webUrl is not available.

Fixes #717

---

Linear Issue: SOU-193

 

[cursor]

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]

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.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[msukkari]

Heads up @brendan-kellam this PR will slightly change behavior of MCP code search tool to not include web url in file field. IMO this makes sense since this should really just be the path to the file. The result includes a repository field which includes the full repo info

[brendan-kellam]

to clarify, will file just be the path now?