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
@DeusData DeusData added bug Something isn't working ux/behavior Display bugs, docs, adoption UX priority/normal Standard review queue; useful PR with ordinary maintainer urgency. labels Jul 8, 2026
@DeusData

DeusData commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Thanks for adding coverage around CLI project naming. Triage: test-focused PR for #823, normal priority. Review focus is that this captures the regression boundary rather than only asserting current plumbing.

@DeusData DeusData added this to the 0.9.1-rc milestone Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority/normal Standard review queue; useful PR with ordinary maintainer urgency. ux/behavior Display bugs, docs, adoption UX

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

2 participants