phone-a-friend-mcp-server 0.2.0__tar.gz → 0.3.0rc1__tar.gz

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: phone-a-friend-mcp-server
-Version: 0.2.0
+Version: 0.3.0rc1
 Summary: MCP Server for Phone-a-Friend assistance
 Project-URL: GitHub, https://github.com/abhishekbhakat/phone-a-friend-mcp-server
 Project-URL: Issues, https://github.com/abhishekbhakat/phone-a-friend-mcp-server/issues
@@ -47,7 +47,7 @@ This enables AI systems to leverage other AI models as "consultants" for complex
 ## Architecture 🏗️
 
 ```
-Primary AI → Phone-a-Friend MCP → OpenRouter → External AI (GPT-4, Claude, etc.) → Processed Response → Primary AI
+Primary AI → Phone-a-Friend MCP → OpenRouter → External AI (O3, Claude, etc.) → Processed Response → Primary AI
 ```
 
 **Sequential Workflow:**
@@ -71,22 +71,21 @@ The `uv` runner will automatically download and execute the server package if it
 
 Add the following JSON configuration to your MCP client and replace `<YOUR_API_KEY>` with your key:
 
-    ```json
-    {
-      "mcpServers": {
-        "phone-a-friend": {
-          "command": "uv",
-          "args": [
-            "run",
-            "phone-a-friend-mcp-server",
-            "--provider", "openai",
-            "--api-key", "<YOUR_API_KEY>"
-          ]
-        }
-      }
+```json
+{
+  "mcpServers": {
+    "phone-a-friend": {
+      "command": "uvx",
+      "args": [
+        "phone-a-friend-mcp-server",
+        "--provider", "openai",
+        "--api-key", "<YOUR_API_KEY>"
+      ]
     }
-    ```
-    > That's it! You can now use the `phone_a_friend` tool in any compatible client. For more options, see the Advanced Configuration section.
+  }
+}
+```
+> That's it! You can now use the `phone_a_friend` tool in any compatible client. For more options, see the Advanced Configuration section.
 
 ## Available Tools 🛠️
 
@@ -150,12 +149,12 @@ You can override the default model for each provider.
 
 **Override with CLI:**
 ```bash
-phone-a-friend-mcp-server --model "gpt-4-turbo"
+phone-a-friend-mcp-server --model "o3"
 ```
 
 **Override with Environment Variable:**
 ```bash
-export PHONE_A_FRIEND_MODEL="gpt-4-turbo"
+export PHONE_A_FRIEND_MODEL="o3"
 ```
 
 ### Additional Options

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/README.md

@@ -22,7 +22,7 @@ This enables AI systems to leverage other AI models as "consultants" for complex
 ## Architecture 🏗️
 
 ```
-Primary AI → Phone-a-Friend MCP → OpenRouter → External AI (GPT-4, Claude, etc.) → Processed Response → Primary AI
+Primary AI → Phone-a-Friend MCP → OpenRouter → External AI (O3, Claude, etc.) → Processed Response → Primary AI
 ```
 
 **Sequential Workflow:**
@@ -46,22 +46,21 @@ The `uv` runner will automatically download and execute the server package if it
 
 Add the following JSON configuration to your MCP client and replace `<YOUR_API_KEY>` with your key:
 
-    ```json
-    {
-      "mcpServers": {
-        "phone-a-friend": {
-          "command": "uv",
-          "args": [
-            "run",
-            "phone-a-friend-mcp-server",
-            "--provider", "openai",
-            "--api-key", "<YOUR_API_KEY>"
-          ]
-        }
-      }
+```json
+{
+  "mcpServers": {
+    "phone-a-friend": {
+      "command": "uvx",
+      "args": [
+        "phone-a-friend-mcp-server",
+        "--provider", "openai",
+        "--api-key", "<YOUR_API_KEY>"
+      ]
     }
-    ```
-    > That's it! You can now use the `phone_a_friend` tool in any compatible client. For more options, see the Advanced Configuration section.
+  }
+}
+```
+> That's it! You can now use the `phone_a_friend` tool in any compatible client. For more options, see the Advanced Configuration section.
 
 ## Available Tools 🛠️
 
@@ -125,12 +124,12 @@ You can override the default model for each provider.
 
 **Override with CLI:**
 ```bash
-phone-a-friend-mcp-server --model "gpt-4-turbo"
+phone-a-friend-mcp-server --model "o3"
 ```
 
 **Override with Environment Variable:**
 ```bash
-export PHONE_A_FRIEND_MODEL="gpt-4-turbo"
+export PHONE_A_FRIEND_MODEL="o3"
 ```
 
 ### Additional Options

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "phone-a-friend-mcp-server"
-version = "0.2.0"
+version = "0.3.0rc1"
 description = "MCP Server for Phone-a-Friend assistance"
 readme = "README.md"
 requires-python = ">=3.11"

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/src/phone_a_friend_mcp_server/tools/fax_tool.py

@@ -85,17 +85,18 @@ replacing <file="…"> blocks as needed. Commentary goes outside those tags."""
                 "all_related_context": {
                     "type": "string",
                     "description": (
-                        "MANDATORY. General, non-code context for the friend AI. "
-                        "Include known constraints (Python version, allowed deps, etc.), "
-                        "failing test output, or tracebacks. DO NOT include file contents here."
+                        "General context for the friend AI. Include known constraints "
+                        "(Python version, allowed deps, etc.), failing test output, tracebacks, "
+                        "or code snippets for reference. For complete files, use file_list instead."
                     ),
                 },
                 "file_list": {
                     "type": "array",
                     "items": {"type": "string"},
                     "description": (
-                        "MANDATORY. A list of file paths or glob patterns to be included in the code context. "
-                        "The tool will automatically read these files, filter them against .gitignore, and build the context."
+                        "Optional but recommended. A list of file paths or glob patterns to be included in the code context. "
+                        "The tool will automatically read these files, filter them against .gitignore, and build the context. "
+                        "Better and faster than including complete files in all_related_context."
                     ),
                 },
                 "task": {
@@ -119,7 +120,7 @@ replacing <file="…"> blocks as needed. Commentary goes outside those tags."""
                     ),
                 },
             },
-            "required": ["all_related_context", "file_list", "task", "output_directory"],
+            "required": ["all_related_context", "task", "output_directory"],
         }
 
     async def run(self, **kwargs) -> dict[str, Any]:

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/src/phone_a_friend_mcp_server/tools/phone_tool.py

@@ -87,17 +87,18 @@ replacing <file="…"> blocks as needed. Commentary goes outside those tags."""
                 "all_related_context": {
                     "type": "string",
                     "description": (
-                        "MANDATORY. General, non-code context for the friend AI. "
-                        "Include known constraints (Python version, allowed deps, etc.), "
-                        "failing test output, or tracebacks. DO NOT include file contents here."
+                        "General context for the friend AI. Include known constraints "
+                        "(Python version, allowed deps, etc.), failing test output, tracebacks, "
+                        "or code snippets for reference. For complete files, use file_list instead."
                     ),
                 },
                 "file_list": {
                     "type": "array",
                     "items": {"type": "string"},
                     "description": (
-                        "MANDATORY. A list of file paths or glob patterns to be included in the code context. "
-                        "The tool will automatically read these files, filter them against .gitignore, and build the context."
+                        "Optional but recommended. A list of file paths or glob patterns to be included in the code context. "
+                        "The tool will automatically read these files, filter them against .gitignore, and build the context. "
+                        "Better and faster than including complete files in all_related_context."
                     ),
                 },
                 "task": {
@@ -112,7 +113,7 @@ replacing <file="…"> blocks as needed. Commentary goes outside those tags."""
                     ),
                 },
             },
-            "required": ["all_related_context", "file_list", "task"],
+            "required": ["all_related_context", "task"],
         }
 
     async def run(self, **kwargs) -> dict[str, Any]:

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/src/phone_a_friend_mcp_server/utils/context_builder.py

@@ -14,6 +14,20 @@ def load_gitignore(base_dir: str) -> pathspec.PathSpec:
     return pathspec.PathSpec.from_lines("gitwildmatch", patterns)
 
 
+def get_all_project_files(base_dir: str = ".") -> list[str]:
+    """Get all files in the project directory recursively."""
+    all_files = []
+    for root, dirs, files in os.walk(base_dir):
+        # Skip hidden directories like .git, .venv, etc.
+        dirs[:] = [d for d in dirs if not d.startswith('.')]
+
+        for file in files:
+            if not file.startswith('.'):  # Skip hidden files
+                rel_path = os.path.relpath(os.path.join(root, file), base_dir)
+                all_files.append(rel_path)
+    return sorted(all_files)
+
+
 def filter_paths(paths: list[str], spec: pathspec.PathSpec, base_dir: str = ".") -> list[str]:
     """Filters out paths that match the .gitignore spec and non-text files."""
     filtered_paths = []
@@ -76,18 +90,25 @@ def build_code_context(file_list: list[str], base_dir: str = ".") -> str:
     """
     spec = load_gitignore(base_dir)
 
-    all_files = []
-    for pattern in file_list:
-        all_files.extend(glob.glob(pattern, recursive=True))
-
-    unique_files = sorted(list(set(all_files)))
-
-    filtered_files = filter_paths(unique_files, spec, base_dir)
-
-    if not filtered_files:
-        return "No files to display. Check your `file_list` and `.gitignore`."
-
-    file_tree = build_file_tree(filtered_files, base_dir)
-    file_blocks = build_file_blocks(filtered_files, base_dir)
-
-    return f"<file_tree>\n{file_tree}\n</file_tree>\n\n{file_blocks}"
+    # Get complete project tree (like tree --gitignore)
+    all_project_files = get_all_project_files(base_dir)
+    filtered_all_files = filter_paths(all_project_files, spec, base_dir)
+    complete_tree = build_file_tree(filtered_all_files, base_dir)
+
+    # Handle selected files from file_list
+    if file_list:
+        selected_files = []
+        for pattern in file_list:
+            selected_files.extend(glob.glob(pattern, recursive=True))
+
+        unique_selected_files = sorted(list(set(selected_files)))
+        filtered_selected_files = filter_paths(unique_selected_files, spec, base_dir)
+
+        if filtered_selected_files:
+            file_blocks = build_file_blocks(filtered_selected_files, base_dir)
+            return f"<file_tree>\n{complete_tree}\n</file_tree>\n\n{file_blocks}"
+        else:
+            return f"<file_tree>\n{complete_tree}\n</file_tree>\n\nNo files to display from file_list. Check your patterns and .gitignore."
+    else:
+        # No file_list provided, just show the tree
+        return f"<file_tree>\n{complete_tree}\n</file_tree>\n\nNo specific files selected. Use file_list parameter to include file contents."

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/tests/test_context_builder.py

@@ -8,6 +8,7 @@ from phone_a_friend_mcp_server.utils.context_builder import (
     build_file_blocks,
     build_file_tree,
     filter_paths,
+    get_all_project_files,
     load_gitignore,
 )
 
@@ -103,3 +104,64 @@ def test_build_code_context(temp_project):
     assert "print('hello')" in context
     assert '<file="README.md">' in context
     assert "# Project" in context
+
+
+def test_get_all_project_files(temp_project):
+    """Test getting all project files."""
+    original_cwd = os.getcwd()
+    os.chdir(temp_project)
+    try:
+        all_files = get_all_project_files(".")
+    finally:
+        os.chdir(original_cwd)
+
+    # Should include all non-hidden files
+    assert "src/main.py" in all_files
+    assert "src/main.pyc" in all_files
+    assert "README.md" in all_files
+    assert "dist/package.tar.gz" in all_files
+    assert "app.log" in all_files
+    # Should not include hidden files or directories
+    assert ".gitignore" not in all_files
+
+
+def test_build_code_context_empty_file_list(temp_project):
+    """Test build_code_context with empty file_list."""
+    original_cwd = os.getcwd()
+    os.chdir(temp_project)
+    try:
+        context = build_code_context([], base_dir=".")
+    finally:
+        os.chdir(original_cwd)
+
+    # Should still show complete tree
+    assert "<file_tree>" in context
+    assert "main.py" in context
+    assert "README.md" in context
+    # Should not include ignored files in tree
+    assert "main.pyc" not in context
+    assert "dist" not in context
+    assert "app.log" not in context
+    # Should not include file contents
+    assert '<file="src/main.py">' not in context
+    assert "No specific files selected" in context
+
+
+def test_build_code_context_complete_tree_always_shown(temp_project):
+    """Test that complete tree is always shown regardless of file_list."""
+    original_cwd = os.getcwd()
+    os.chdir(temp_project)
+    try:
+        # Test with only one file in file_list
+        context = build_code_context(["README.md"], base_dir=".")
+    finally:
+        os.chdir(original_cwd)
+
+    # Should show complete tree (including files not in file_list)
+    assert "<file_tree>" in context
+    assert "main.py" in context  # This file is in tree but not in file_list
+    assert "README.md" in context
+    # Should only include content for files in file_list
+    assert '<file="README.md">' in context
+    assert "# Project" in context
+    assert '<file="src/main.py">' not in context  # Content not included

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/tests/test_tools.py

@@ -5,6 +5,7 @@ import pytest
 
 from phone_a_friend_mcp_server.config import PhoneAFriendConfig
 from phone_a_friend_mcp_server.tools.fax_tool import FaxAFriendTool
+from phone_a_friend_mcp_server.tools.phone_tool import PhoneAFriendTool
 
 
 @pytest.fixture
@@ -68,12 +69,58 @@ async def test_fax_a_friend_tool_with_context_builder(config, temp_project):
         assert "<file_tree>" in content
         assert "main.py" in content
         assert "README.md" in content
-        assert "main.pyc" not in content  # Should be ignored
-        assert '<file="src/main.py">' in content
-        assert "print('hello')" in content
-        assert "=== INSTRUCTIONS ===" in content
-        assert "Provide exhaustive, step-by-step reasoning." in content
-        assert "Never include files matching .gitignore patterns." in content
+
+
+@pytest.mark.asyncio
+async def test_fax_a_friend_tool_without_file_list(config, temp_project):
+    """Test the fax a friend tool without file_list parameter."""
+    tool = FaxAFriendTool(config)
+
+    result = await tool.run(
+        all_related_context="This is the general context with code snippets.",
+        task="Review the architecture.",
+        output_directory=".",
+    )
+
+    assert result["status"] == "success"
+    assert result["file_name"] == "fax_a_friend.md"
+
+    expected_file_path = os.path.join(temp_project, "fax_a_friend.md")
+    assert os.path.exists(expected_file_path)
+
+    with open(expected_file_path, encoding="utf-8") as f:
+        content = f.read()
+        assert "=== TASK ===" in content
+        assert "Review the architecture." in content
+        assert "=== GENERAL CONTEXT ===" in content
+        assert "This is the general context with code snippets." in content
+        assert "=== CODE CONTEXT ===" in content
+        assert "<file_tree>" in content
+        # Should show complete tree but no file contents
+        assert "No specific files selected" in content
+
+
+@pytest.mark.asyncio
+async def test_phone_a_friend_tool_without_file_list(config):
+    """Test the phone a friend tool without file_list parameter."""
+    tool = PhoneAFriendTool(config)
+
+    # Mock the agent to avoid actual API calls
+    class MockAgent:
+        async def run(self, prompt, model_settings=None):
+            class MockResult:
+                data = "Mock AI response"
+            return MockResult()
+
+    tool._create_agent = lambda: MockAgent()
+
+    result = await tool.run(
+        all_related_context="This is the general context with code snippets.",
+        task="Review the architecture.",
+    )
+
+    assert result["status"] == "success"
+    assert result["response"] == "Mock AI response"
 
 
 def test_config_default_temperature():

{phone_a_friend_mcp_server-0.2.0 → phone_a_friend_mcp_server-0.3.0rc1}/uv.lock

@@ -729,7 +729,7 @@ wheels = [
 
 [[package]]
 name = "phone-a-friend-mcp-server"
-version = "0.2.0"
+version = "0.3.0rc1"
 source = { editable = "." }
 dependencies = [
     { name = "aiofiles" },