Skip to content

Commit 7909e34

Browse files
Bruce Haumanclaude
andcommitted
Remove unused other_tools directory and associated tests
Deleted the other_tools directory containing deprecated tools (create_directory, list_directory, move_file, namespace, and symbol tools) that were not actively used in the main MCP server. Updated PROJECT_SUMMARY.md to remove all references to these deprecated tools and simplified the Extension Points section. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 767b148 commit 7909e34

File tree

23 files changed

+18
-2368
lines changed

23 files changed

+18
-2368
lines changed

PROJECT_SUMMARY.md

Lines changed: 2 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -64,19 +64,6 @@ The project allows AI assistants to:
6464
- `config.clj`: Configuration file management for persistence
6565
- `truncate.clj`: Pretty-printing with depth truncation
6666

67-
#### Unused Tools (moved to other_tools/)
68-
69-
**Note**: These tools have been moved to `/src/clojure_mcp/other_tools/` to clearly separate them from actively used tools. They remain fully functional with passing tests but are not registered in `main.clj`. This organizational change helps maintain a cleaner codebase by distinguishing between essential tools and potentially unnecessary ones.
70-
71-
- `/src/clojure_mcp/other_tools/create_directory/`: Tool for creating directories
72-
- `/src/clojure_mcp/other_tools/list_directory/`: Tool for listing directory contents
73-
- `/src/clojure_mcp/other_tools/move_file/`: Tool for moving/renaming files
74-
- `/src/clojure_mcp/other_tools/namespace/`: Clojure namespace exploration tools
75-
- Includes: `current_namespace`, `clojure_list_namespaces`, `clojure_list_vars_in_namespace`
76-
- `/src/clojure_mcp/other_tools/symbol/`: Symbol information and documentation tools
77-
- Includes: `symbol_completions`, `symbol_metadata`, `symbol_documentation`, `source_code`, `symbol_search`
78-
79-
All unused tools have corresponding test files moved to `/test/clojure_mcp/other_tools/` with updated namespace declarations.
8067

8168
### Example Main Files
8269

@@ -503,10 +490,9 @@ See `/doc/custom-mcp-server.md` for comprehensive documentation on creating cust
503490
## Extension Points
504491

505492
1. **Adding New Tools**:
506-
- Create a new tool namespace in `/src/clojure_mcp/tools/` for active tools
493+
- Create a new tool namespace in `/src/clojure_mcp/tools/`
507494
- Implement the required multimethods from `tool-system`
508495
- Register the tool in `main.clj` within the `make-tools` function
509-
- Note: Tools in `/src/clojure_mcp/other_tools/` are not automatically registered
510496

511497
2. **Creating Custom Servers**:
512498
- Define factory functions for tools, prompts, and resources
@@ -517,14 +503,7 @@ See `/doc/custom-mcp-server.md` for comprehensive documentation on creating cust
517503
- Most tools follow a pipeline architecture that can be modified by adding new steps
518504
- Pipeline steps follow a thread-first pattern with error short-circuiting
519505

520-
4. **Re-activating Unused Tools**:
521-
- Tools in `/src/clojure_mcp/other_tools/` can be re-activated by:
522-
- Moving them back to `/src/clojure_mcp/tools/`
523-
- Updating namespace declarations
524-
- Adding them to the imports and `make-tools` function in `main.clj`
525-
- Alternatively, create custom MCP servers using these tools via the core API
526-
527-
5. **Alternative Transports**:
506+
4. **Alternative Transports**:
528507
- Use `sse-core/build-and-start-mcp-server` for SSE transport
529508
- See `sse-main.clj` for an example implementation
530509

@@ -558,6 +537,4 @@ See `/doc/custom-mcp-server.md` for comprehensive documentation on creating cust
558537
- Path-based data manipulation using `set_path`/`get_path`/`delete_path` operations
559538
- Direct storage of JSON-compatible values
560539

561-
**Tool Reorganization**: To improve codebase maintainability, unused tools have been moved to `/src/clojure_mcp/other_tools/`. This separation clarifies which tools are actively used in the main MCP server (`main.clj`) versus those that remain available but are not currently essential.
562-
563540
This project summary is designed to provide AI assistants with a quick understanding of the Clojure MCP project structure and capabilities, enabling more effective assistance with minimal additional context. The project continues to evolve with improvements focused on making it easier to create custom MCP servers while maintaining compatibility with a wide range of LLMs.

doc/component-filtering.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -30,45 +30,45 @@ Control which tools are available to the AI assistant.
3030
### Configuration Keys
3131

3232
```edn
33-
{:enable-tools [:clojure-eval :read-file :file-write] ; Only these tools
34-
:disable-tools [:dispatch-agent :architect]} ; Remove these tools
33+
{:enable-tools [:clojure_eval :read_file :file_write] ; Only these tools
34+
:disable-tools [:dispatch_agent :architect]} ; Remove these tools
3535
```
3636

3737
### Tool Identifiers
3838

3939
Tools can be specified using keywords or strings:
40-
- `:clojure-eval` or `"clojure-eval"`
41-
- `:read-file` or `"read_file"`
42-
- `:file-write` or `"file_write"`
40+
- `:clojure_eval` or `"clojure_eval"`
41+
- `:read_file` or `"read_file"`
42+
- `:file_write` or `"file_write"`
4343

4444
Common tool IDs include:
45-
- `:clojure-eval` - Evaluate Clojure code
46-
- `:read-file` - Read file contents
47-
- `:file-edit` - Edit files
48-
- `:file-write` - Write files
45+
- `:clojure_eval` - Evaluate Clojure code
46+
- `:read_file` - Read file contents
47+
- `:file_edit` - Edit files
48+
- `:file_write` - Write files
4949
- `:bash` - Execute shell commands
5050
- `:grep` - Search file contents
51-
- `:glob-files` - Find files by pattern
52-
- `:dispatch-agent` - Launch sub-agents
51+
- `:glob_files` - Find files by pattern
52+
- `:dispatch_agent` - Launch sub-agents
5353
- `:architect` - Technical planning
54-
- `:code-critique` - Code review
55-
- `:scratch-pad` - Persistent storage
54+
- `:code_critique` - Code review
55+
- `:scratch_pad` - Persistent storage
5656

5757
### Examples
5858

5959
**Minimal REPL-only server:**
6060
```edn
61-
{:enable-tools [:clojure-eval]}
61+
{:enable-tools [:clojure_eval]}
6262
```
6363

6464
**Read-only exploration server:**
6565
```edn
66-
{:enable-tools [:read-file :grep :glob-files :LS :clojure-inspect-project]}
66+
{:enable-tools [:read_file :grep :glob_files :LS :clojure_inspect_project]}
6767
```
6868

6969
**Full access except agents:**
7070
```edn
71-
{:disable-tools [:dispatch-agent :architect :code-critique]}
71+
{:disable-tools [:dispatch_agent :architect :code_critique]}
7272
```
7373

7474
## Prompts Filtering

src/clojure_mcp/other_tools/README.md

Lines changed: 0 additions & 97 deletions
This file was deleted.

src/clojure_mcp/other_tools/create_directory/core.clj

Lines changed: 0 additions & 75 deletions
This file was deleted.

src/clojure_mcp/other_tools/create_directory/tool.clj

Lines changed: 0 additions & 86 deletions
This file was deleted.

0 commit comments

Comments
 (0)