Skip to content

fix: accept .svn directories in repo root validation#456

Open
zxyDEDIRE wants to merge 1 commit intotirth8205:mainfrom
zxyDEDIRE:fix/svn-validate-repo-root
Open

fix: accept .svn directories in repo root validation#456
zxyDEDIRE wants to merge 1 commit intotirth8205:mainfrom
zxyDEDIRE:fix/svn-validate-repo-root

Conversation

@zxyDEDIRE
Copy link
Copy Markdown

The repo root validation in _validate_repo_root() and related functions only checked for .git and .code-review-graph directories. This caused all MCP tools to reject SVN working copies at the validation gate, even though the core SVN support (incremental.py, changes.py) was already implemented in v2.3.3 via PR #255.

Add .svn directory checks to:

  • tools/_common.py:_validate_repo_root() (MCP tool entry point)
  • daemon.py (daemon skip logic and register validation)
  • registry.py (register command validation)

The repo root validation in _validate_repo_root() and related functions
only checked for .git and .code-review-graph directories. This caused
all MCP tools to reject SVN working copies at the validation gate, even
though the core SVN support (incremental.py, changes.py) was already
implemented in v2.3.3 via PR tirth8205#255.

Add .svn directory checks to:
- tools/_common.py:_validate_repo_root() (MCP tool entry point)
- daemon.py (daemon skip logic and register validation)
- registry.py (register command validation)

Co-Authored-By: Claude Opus 4.7 <[email protected]>
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.

1 participant