Skip to content

test(mcp): cover CLI index_repository name override#948

Open
blankanswer wants to merge 1 commit into
DeusData:mainfrom
blankanswer:agent/fix-cli-index-name
Open

test(mcp): cover CLI index_repository name override#948
blankanswer wants to merge 1 commit into
DeusData:mainfrom
blankanswer:agent/fix-cli-index-name

Conversation

@blankanswer

Copy link
Copy Markdown

What does this PR do?

Adds a regression test for #823 covering the index_repository name override when invoked through the CLI/supervised indexing path.

The test indexes a small temporary repository with a custom project name and verifies that:

  • the index_repository response preserves the requested project name
  • list_projects reports the custom name
  • search_graph can query the indexed project by that custom name

This matches the maintainer-requested coverage for repo_path + name, ensuring the response and stored project identity agree instead of silently falling back to the path-derived project name.

Closes #823.

Checklist

  • Every commit is signed off (git commit -s) — required, CI rejects unsigned commits (DCO, see CONTRIBUTING.md)
  • Tests pass locally (make -f Makefile.cbm test)
  • Lint passes locally (make -f Makefile.cbm lint-ci)
  • New behavior is covered by a test (reproduce-first for bug fixes)

Signed-off-by: Blank_Answer <97771966+blankanswer@users.noreply.github.com>
@blankanswer blankanswer requested a review from DeusData as a code owner July 8, 2026 03:25
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.

name parameter on index_repository is parsed but has no effect via cli mode

1 participant