Skip to content

fix(mcp): normalize constrained project names - #1405

Open
mikemikimike wants to merge 1 commit into
basicmachines-co:mainfrom
mikemikimike:codex/fix-project-constraint-1388
Open

fix(mcp): normalize constrained project names#1405
mikemikimike wants to merge 1 commit into
basicmachines-co:mainfrom
mikemikimike:codex/fix-project-constraint-1388

Conversation

@mikemikimike

Copy link
Copy Markdown
Contributor

Summary

  • normalize BASIC_MEMORY_MCP_PROJECT before selecting startup indexing projects
  • apply the same normalized matching to the local watch service
  • add regression coverage for mixed-case constraints in both paths

Root cause

Project names are normalized to permalinks during configuration reconciliation, but the MCP constraint was compared directly with the persisted project name. A mixed-case constraint therefore selected no project, leaving startup indexing and file watching idle while reporting No projects to watch.

Validation

  • uv run --frozen pytest tests/index/test_watch_service.py tests/services/test_initialization.py -q
  • uv run ruff format --check ...
  • uv run ruff check ...
  • uv run pyright ...
  • git diff --check

Closes #1388.

Signed-off-by: mikemikimike <13286568797@163.com>
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.

[BUG] case sensitivity issue with --project

1 participant