• fix: MCP roots take priority over CRB_WORKSPACE for multi-window support #164 👈 (View in Graphite)
• docs: simplify README installation, expand dev setup with all config options #163
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Greptile Summary

Flips workspace detection priority so MCP roots take precedence over CRB_WORKSPACE env var, fixing multi-window setups where a single shared MCP client config JSON would hardcode the wrong project path.

• Priority cascade changed: MCP roots → CRB_WORKSPACE → process cwd (previously CRB_WORKSPACE was chore(ci): bump astral-sh/setup-uv from 7.1.5 to 7.3.0 #1)
• server.py: Reordered _get_workspace_cwd logic to check MCP roots first, falling back to env var only when roots are unavailable or empty
• tests: Expanded from 6 to 8 test cases with comprehensive coverage of all fallback scenarios including empty roots and exception handling
• README: Removed CRB_WORKSPACE from default config examples and settings table — it's now positioned as an internal fallback for CI/scripts, not a user-facing setting

The change is well-motivated: MCP roots are sent per-window by the client and provide the correct workspace for multi-window scenarios, while CRB_WORKSPACE is static and shared across all windows. Test coverage is thorough and validates all code paths.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change is a simple priority reordering with clear motivation (multi-window support). All code paths are covered by thorough tests (8 test cases covering roots priority, env var fallback, empty roots, and exception handling). The logic is straightforward and the refactoring preserves backward compatibility by keeping CRB_WORKSPACE as a fallback.
• No files require special attention

Important Files Changed

Flowchart

flowchart TD
    Start[Tool call needs workspace] --> CheckCtx{Context available?}
    CheckCtx -->|Yes| CheckRoots[Try ctx.list_roots]
    CheckCtx -->|No| CheckEnv[Check CRB_WORKSPACE]
    
    CheckRoots --> RootsException{Exception?}
    RootsException -->|No| RootsEmpty{Roots empty?}
    RootsException -->|Yes| LogWarning[Log warning]
    
    RootsEmpty -->|No| ParseRoot[Parse first root URI]
    RootsEmpty -->|Yes| CheckEnv
    LogWarning --> CheckEnv
    
    ParseRoot --> ValidFile{Valid file:// URI?}
    ValidFile -->|Yes| ReturnRoot[Return root path]
    ValidFile -->|No| CheckEnv
    
    CheckEnv --> EnvSet{CRB_WORKSPACE set?}
    EnvSet -->|Yes| ReturnEnv[Return env var path]
    EnvSet -->|No| ReturnNone[Return None]
    
    ReturnRoot --> Success[Workspace detected]
    ReturnEnv --> Success
    ReturnNone --> UseCwd[Falls back to process cwd]
    
    style ReturnRoot fill:#90EE90
    style ReturnEnv fill:#FFD700
    style ReturnNone fill:#FFB6C1
    style Success fill:#90EE90

_{Last reviewed commit: 761dba5}

Merge activity

• Feb 16, 11:54 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 16, 11:55 AM UTC: Graphite couldn't merge this PR because it had merge conflicts.