Files
beads/integrations/beads-mcp/src/beads_mcp/config.py
Baishampayan Ghose 4353592fe6 test(mcp): Fix two failing integration tests and linting errors
1. Fix `test_default_beads_path_auto_detection`
    - Changed beads_path to use `Field(default_factory=_default_beads_path)` so the default is evaluated at instance
    creation time, not class definition time
    - Updated test to mock both `shutil.which` and `os.access`
2. Fix `test_init_creates_beads_directory`
    - Fixed test to pass `working_dir=temp_dir` to `BdClient` instead of using `os.chdir()`
    - The `_get_working_dir()` method checks `PWD` env var first, which isn't updated by `os.chdir()`
3. Fix minor linting errors reported by `ruff` tool
4. Update `beads` version to `0.9.6` in `uv.lock` file

MCP Server test coverage is now excellent, at 92% overall maintaining our high-standards of production level quality.

```
Name                         Stmts   Miss  Cover
------------------------------------------------
src/beads_mcp/__init__.py        1      0   100%
src/beads_mcp/__main__.py        3      3     0%
src/beads_mcp/bd_client.py     214     14    93%
src/beads_mcp/config.py         51      2    96%
src/beads_mcp/models.py         92      1    99%
src/beads_mcp/server.py         58     16    72%
src/beads_mcp/tools.py          59      0   100%
------------------------------------------------
TOTAL                          478     36    92%
```
2025-10-15 16:59:16 +05:30

140 lines
4.5 KiB
Python

"""Configuration for beads MCP server."""
import os
import shutil
import sys
from pathlib import Path
from pydantic import Field, field_validator
from pydantic_settings import BaseSettings, SettingsConfigDict
def _default_beads_path() -> str:
"""Get default bd executable path.
First tries to find bd in PATH, falls back to ~/.local/bin/bd.
Returns:
Default path to bd executable
"""
# Try to find bd in PATH first
bd_in_path = shutil.which("bd")
if bd_in_path:
return bd_in_path
# Fall back to common install location
return str(Path.home() / ".local" / "bin" / "bd")
class Config(BaseSettings):
"""Server configuration loaded from environment variables."""
model_config = SettingsConfigDict(env_prefix="")
beads_path: str = Field(default_factory=_default_beads_path)
beads_db: str | None = None
beads_actor: str | None = None
beads_no_auto_flush: bool = False
beads_no_auto_import: bool = False
beads_working_dir: str | None = None
@field_validator("beads_path")
@classmethod
def validate_beads_path(cls, v: str) -> str:
"""Validate BEADS_PATH points to an executable bd binary.
Args:
v: Path to bd executable (can be command name or absolute path)
Returns:
Validated absolute path
Raises:
ValueError: If path is invalid or not executable
"""
path = Path(v)
# If not an absolute/existing path, try to find it in PATH
if not path.exists():
found = shutil.which(v)
if found:
v = found
path = Path(v)
else:
raise ValueError(
f"bd executable not found at: {v}\n\n"
+ "The beads Claude Code plugin requires the bd CLI to be installed.\n\n"
+ "Install bd CLI:\n"
+ " curl -fsSL https://raw.githubusercontent.com/steveyegge/beads/main/install.sh | bash\n\n"
+ "Or visit: https://github.com/steveyegge/beads#installation\n\n"
+ "After installation, restart Claude Code to reload the MCP server."
)
if not os.access(v, os.X_OK):
raise ValueError(f"bd executable at {v} is not executable.\nPlease check file permissions.")
return v
@field_validator("beads_db")
@classmethod
def validate_beads_db(cls, v: str | None) -> str | None:
"""Validate BEADS_DB points to an existing database file.
Args:
v: Path to database file or None
Returns:
Validated path or None
Raises:
ValueError: If path is set but file doesn't exist
"""
if v is None:
return v
path = Path(v)
if not path.exists():
raise ValueError(
f"BEADS_DB points to non-existent file: {v}\n" + "Please verify the database path is correct."
)
return v
class ConfigError(Exception):
"""Configuration error with helpful message."""
pass
def load_config() -> Config:
"""Load and validate configuration from environment variables.
Returns:
Validated configuration
Raises:
ConfigError: If configuration is invalid
"""
try:
return Config()
except Exception as e:
default_path = _default_beads_path()
error_msg = (
"Beads MCP Server Configuration Error\n\n"
+ f"{e}\n\n"
+ "Common fix: Install the bd CLI first:\n"
+ " curl -fsSL https://raw.githubusercontent.com/steveyegge/beads/main/install.sh | bash\n\n"
+ "Or visit: https://github.com/steveyegge/beads#installation\n\n"
+ "After installation, restart Claude Code.\n\n"
+ "Advanced configuration (optional):\n"
+ f" BEADS_PATH - Path to bd executable (default: {default_path})\n"
+ " BEADS_DB - Path to beads database file (default: auto-discover)\n"
+ " BEADS_WORKING_DIR - Working directory for bd commands (default: $PWD or cwd)\n"
+ " BEADS_ACTOR - Actor name for audit trail (default: $USER)\n"
+ " BEADS_NO_AUTO_FLUSH - Disable automatic JSONL sync (default: false)\n"
+ " BEADS_NO_AUTO_IMPORT - Disable automatic JSONL import (default: false)"
)
print(error_msg, file=sys.stderr)
raise ConfigError(error_msg) from e