chore: code cleanup and test fixture updates
- Apply ruff formatting to workspace_commands.py - Remove unnecessary f-string in workspace.py - Remove unused import get_phase_config from spec_runner.py - Add phase_name parameter to mock_run_agent_fn fixture 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -31,7 +31,9 @@ from workspace import (
|
||||
from .utils import print_banner
|
||||
|
||||
|
||||
def _get_changed_files_from_git(worktree_path: Path, base_branch: str = "main") -> list[str]:
|
||||
def _get_changed_files_from_git(
|
||||
worktree_path: Path, base_branch: str = "main"
|
||||
) -> list[str]:
|
||||
"""
|
||||
Get list of changed files from git diff between base branch and HEAD.
|
||||
|
||||
|
||||
@@ -189,7 +189,7 @@ def merge_existing_build(
|
||||
)
|
||||
print()
|
||||
print("Example:")
|
||||
print(highlight(f" git checkout main # or your feature branch"))
|
||||
print(highlight(" git checkout main # or your feature branch"))
|
||||
print(highlight(f" python auto-claude/run.py --spec {spec_name} --merge"))
|
||||
return False
|
||||
|
||||
|
||||
@@ -92,7 +92,7 @@ elif dev_env_file.exists():
|
||||
load_dotenv(dev_env_file)
|
||||
|
||||
from debug import debug, debug_error, debug_section, debug_success
|
||||
from phase_config import get_phase_config, resolve_model_id
|
||||
from phase_config import resolve_model_id
|
||||
from review import ReviewState
|
||||
from spec import SpecOrchestrator
|
||||
from ui import Icons, highlight, icon, muted, print_section, print_status
|
||||
|
||||
@@ -602,6 +602,7 @@ def mock_run_agent_fn():
|
||||
async def _mock_agent(
|
||||
prompt_file: str,
|
||||
additional_context: str = None,
|
||||
phase_name: str = None,
|
||||
) -> tuple[bool, str]:
|
||||
nonlocal call_count
|
||||
if side_effect is not None:
|
||||
|
||||
Reference in New Issue
Block a user