resolved more integration test issues

Author: bashandbone

.roo/mcp.json

@@ -1,41 +1,51 @@
 {
   "mcpServers": {
     "context7": {
-      "alwaysAllow": [
-        "resolve-library-id",
-        "get-library-docs"
-      ],
-      "args": [
-        "--bun",
-        "@upstash/context7-mcp@latest"
-      ],
+      "alwaysAllow": ["resolve-library-id", "get-library-docs"],
+      "args": ["--bun", "@upstash/context7-mcp@latest"],
       "command": "bunx",
       "type": "stdio"
     },
     "fetch": {
-      "args": [
-        "mcp-server-fetch"
-      ],
-      "command": "uvx"
+      "args": ["mcp-server-fetch"],
+      "command": "uvx",
+      "alwaysAllow": ["fetch"]
     },
     "sequential-thinking": {
-      "alwaysAllow": [
-        "sequentialthinking"
-      ],
-      "args": [
-        "--bun",
-        "@modelcontextprotocol/server-sequential-thinking"
-      ],
+      "alwaysAllow": ["sequentialthinking"],
+      "args": ["--bun", "@modelcontextprotocol/server-sequential-thinking"],
       "command": "bunx",
       "type": "stdio"
     },
     "filesystem": {
       "command": "npx",
+      "args": ["-y", "@modelcontextprotocol/server-filesystem", "~/submod"],
+      "alwaysAllow": [
+        "read_file",
+        "read_multiple_files",
+        "write_file",
+        "edit_file",
+        "create_directory",
+        "list_directory",
+        "directory_tree",
+        "move_file",
+        "search_files",
+        "get_file_info",
+        "list_allowed_directories"
+      ]
+    },
+    "mem0-memory-mcp": {
+      "command": "bunx",
       "args": [
-        "-y",
-        "@modelcontextprotocol/server-filesystem",
-        "~/submod"
+        "--bun",
+        "@smithery/cli@latest",
+        "run",
+        "@mem0ai/mem0-memory-mcp",
+        "--key",
+        "${env:MEM0}",
+        "--profile",
+        "${env:SMITHERY_PROFILE}"
       ]
     }
   }
-}
+}

.vscode/mcp.json

@@ -28,7 +28,11 @@
       "command": "uvx"
     },
     "filesystem": {
-      "args": ["--bun", "@modelcontextprotocol/server-filesystem", "${workspaceFolder}"],
+      "args": [
+        "--bun",
+        "@modelcontextprotocol/server-filesystem",
+        "${workspaceFolder}"
+      ],
       "command": "bunx"
     },
     "github-remote": {
@@ -62,8 +66,15 @@
         "--key",
         "${env:MEM0}",
         "--profile",
-        "${env:SMITHERY_PROFILE}",
+        "${env:SMITHERY_PROFILE}"
       ]
+    },
+    "tavily": {
+      "command": "bunx",
+      "args": ["--bun", "tavily-mcp@latest"],
+      "env": {
+        "TAVILY_API_KEY": "${env:tavily_api_key}"
+      }
     }
   },
 

CONTRIBUTING.md

@@ -82,7 +82,7 @@ cargo build
 cargo test
 
 # Run the comprehensive test suite
-./scripts/run_tests.sh
+./scripts/run-tests.sh
 ```
 
 ### 4. Install Development Tools
@@ -172,13 +172,13 @@ My philosophy on testing is to "test what matters." Tests focus on integration a
 cargo test
 
 # Comprehensive test suite with reporting
-./scripts/run_tests.sh --verbose
+./scripts/run-tests.sh --verbose
 
 # Include performance tests
-./scripts/run_tests.sh --performance
+./scripts/run-tests.sh --performance
 
 # Filter specific tests
-./scripts/run_tests.sh --filter sparse_checkout
+./scripts/run-tests.sh --filter sparse_checkout
 ```
 
 ### Writing Tests
@@ -235,7 +235,7 @@ fn test_submod_init_command() {
 Before submitting your PR, ensure:
 
 - [ ] **Code compiles** without warnings
-- [ ] **All tests pass** (`./scripts/run_tests.sh`)
+- [ ] **All tests pass** (`./scripts/run-tests.sh`)
 - [ ] **Code is formatted** (`cargo fmt`)
 - [ ] **Lints pass** (`cargo clippy`)
 - [ ] **Documentation is updated** if needed
@@ -266,7 +266,7 @@ Brief description of the changes and motivation.
 ## Testing
 - [ ] I have added tests that prove my fix is effective or that my feature works
 - [ ] New and existing unit tests pass locally with my changes
-- [ ] I have run the full test suite (`./scripts/run_tests.sh`)
+- [ ] I have run the full test suite (`./scripts/run-tests.sh`)
 
 ## Checklist
 - [ ] My code follows the project's style guidelines

Cargo.lock

@@ -5,6 +5,8 @@ edition = "2024"
 rust-version = "1.87"
 description = "Git submodule manager with sparse checkout support using gitoxide"
 authors = ["Adam Poulemanos <[email protected]>"]
+license = "MIT" # Plain MIT license: plainlicense.org/licenses/permissive/mit/
+
 resolver = "3"
 
 [dependencies]
@@ -31,6 +33,10 @@ thiserror = "2.0.12"
 git2 = { version = "0.20.2", optional = true }
 
 
+[lib]
+name = "submod"
+path = "src/lib.rs"
+
 [[bin]]
 name = "submod"
 path = "src/main.rs"

Cargo.toml

@@ -254,7 +254,7 @@ cargo build
 cargo test
 
 # Or use the comprehensive test runner
-./scripts/run_tests.sh --verbose
+./scripts/run-tests.sh --verbose
 ```
 
 ### Running Tests
@@ -267,13 +267,13 @@ cargo test
 cargo test --test integration_tests
 
 # Run with the test script for better reporting
-./scripts/run_tests.sh
+./scripts/run-tests.sh
 
 # Run performance tests
-./scripts/run_tests.sh --performance
+./scripts/run-tests.sh --performance
 
 # Filter specific tests
-./scripts/run_tests.sh --filter sparse_checkout
+./scripts/run-tests.sh --filter sparse_checkout
 ```
 
 ### Project Structure
@@ -300,7 +300,7 @@ We welcome contributions! Please see our [Contributing Guide](CONTRIBUTING.md) f
 1. **Fork the repository**
 2. **Create a feature branch**: `git checkout -b feature/amazing-feature`
 3. **Make your changes** and add tests if applicable (test anything that has a result)
-4. **Run the test suite**: `./scripts/run_tests.sh`
+4. **Run the test suite**: `./scripts/run-tests.sh`
 5. **Commit your changes**: `git commit -m 'Add amazing feature'`
 6. **Push to your branch**: `git push origin feature/amazing-feature`
 7. **Open a Pull Request**

README.md

@@ -1,4 +1,14 @@
 [tools]
-bun = "latest"
-cargo-binstall = "latest"
-rust = "1.87"
+bun = "latest"   # We use bun to run mcp tool development servers. Totally optional, but helpful.
+cargo-binstall = "latest"  # For installing binaries from crates.io
+"cargo:cargo-audit" = "latest"
+"cargo:cargo-deny" = "latest"
+"cargo:cargo-nextest" = "latest"
+"cargo:cargo-watch" = "latest"
+rust = "1.87"   # The minimum Rust version we support; mise just makes sure it's there.
+uv = "latest"   # Another runner for MCP servers.
+
+[env]
+CARGO_TARGET_DIR = "target"
+RUST_BACKTRACE = "1"
+RUST_TEST_THREADS=1

mise.toml

@@ -87,6 +87,9 @@ else
     cargo build --bin submod > /dev/null 2>&1
 fi
 
+# Force Rust tests to run serially to avoid git submodule race conditions
+export RUST_TEST_THREADS=1
+
 if [[ $? -eq 0 ]]; then
     print_success "Build completed successfully"
 else

scripts/run_tests.sh scripts/run-tests.shscripts/run_tests.sh renamed to scripts/run-tests.sh

@@ -42,6 +42,7 @@ Reset {
     #[arg(short, long)]
     all: bool,
     /// Specific submodule names to reset
+    #[arg(required_unless_present = "all")]
     names: Vec<String>,
 },
 /// Run full sync: check, init, update