fix: Line length violations in ChatGPT tools and tests

- Split long function signature in chatgpt_tools.py across multiple lines
- Split long patch() calls in test_chatgpt_tools.py for better readability
- Refactor long string literals in integration tests using parentheses
- All files now comply with 100-character line limit

Co-authored-by: Paul Hernandez <phernandez@users.noreply.github.com>

Author: github-actions[bot]

src/basic_memory/mcp/tools/chatgpt_tools.py

@@ -34,7 +34,9 @@ def _format_search_results_for_chatgpt(results: SearchResponse) -> List[Dict[str
     return formatted_results
 
 
-def _format_document_for_chatgpt(content: str, identifier: str, title: Optional[str] = None) -> Dict[str, Any]:
+def _format_document_for_chatgpt(
+    content: str, identifier: str, title: Optional[str] = None
+) -> Dict[str, Any]:
     """Format document content according to ChatGPT's expected schema.
 
     Returns a document object with id, title, text, url, and metadata fields.

test-int/mcp/test_chatgpt_tools_integration.py

@@ -23,7 +23,10 @@ async def test_chatgpt_search_basic(mcp_server, app, test_project):
                 "project": test_project.name,
                 "title": "Machine Learning Fundamentals",
                 "folder": "ai",
-                "content": "# Machine Learning Fundamentals\n\nIntroduction to ML concepts and algorithms.",
+                "content": (
+                    "# Machine Learning Fundamentals\n\n"
+                    "Introduction to ML concepts and algorithms."
+                ),
                 "tags": "ml,ai,fundamentals",
             },
         )
@@ -34,7 +37,10 @@ async def test_chatgpt_search_basic(mcp_server, app, test_project):
                 "project": test_project.name,
                 "title": "Deep Learning with PyTorch",
                 "folder": "ai",
-                "content": "# Deep Learning with PyTorch\n\nBuilding neural networks using PyTorch framework.",
+                "content": (
+                    "# Deep Learning with PyTorch\n\n"
+                    "Building neural networks using PyTorch framework."
+                ),
                 "tags": "pytorch,deep-learning,ai",
             },
         )
@@ -45,7 +51,10 @@ async def test_chatgpt_search_basic(mcp_server, app, test_project):
                 "project": test_project.name,
                 "title": "Data Visualization Guide",
                 "folder": "data",
-                "content": "# Data Visualization Guide\n\nCreating charts and graphs for data analysis.",
+                "content": (
+                    "# Data Visualization Guide\n\n"
+                    "Creating charts and graphs for data analysis."
+                ),
                 "tags": "visualization,data,charts",
             },
         )
@@ -117,7 +126,10 @@ async def test_chatgpt_search_with_boolean_operators(mcp_server, app, test_proje
                 "project": test_project.name,
                 "title": "Python Web Frameworks",
                 "folder": "dev",
-                "content": "# Python Web Frameworks\n\nComparing Django and Flask for web development.",
+                "content": (
+                    "# Python Web Frameworks\n\n"
+                    "Comparing Django and Flask for web development."
+                ),
                 "tags": "python,web,frameworks",
             },
         )
@@ -388,7 +400,10 @@ async def test_chatgpt_integration_workflow(mcp_server, app, test_project):
         docs = [
             {
                 "title": "API Design Best Practices",
-                "content": "# API Design Best Practices\n\nRESTful API design principles and patterns.",
+                "content": (
+                    "# API Design Best Practices\n\n"
+                    "RESTful API design principles and patterns."
+                ),
                 "tags": "api,rest,design",
             },
             {
@@ -398,7 +413,10 @@ async def test_chatgpt_integration_workflow(mcp_server, app, test_project):
             },
             {
                 "title": "Database Design Patterns",
-                "content": "# Database Design Patterns\n\nCommon database design patterns and anti-patterns.",
+                "content": (
+                    "# Database Design Patterns\n\n"
+                    "Common database design patterns and anti-patterns."
+                ),
                 "tags": "database,design,patterns",
             },
         ]

tests/mcp/tools/test_chatgpt_tools.py

@@ -34,7 +34,10 @@ async def test_search_successful_results():
         page_size=10
     )
 
-    with patch('basic_memory.mcp.tools.chatgpt_tools.search_notes.fn', new_callable=AsyncMock) as mock_search:
+    with patch(
+        'basic_memory.mcp.tools.chatgpt_tools.search_notes.fn',
+        new_callable=AsyncMock
+    ) as mock_search:
         mock_search.return_value = mock_results
 
         # Import and call the actual function
@@ -67,7 +70,10 @@ async def test_search_with_error_response():
     """Test search when underlying search_notes returns error string."""
     error_message = "# Search Failed - Invalid Syntax\n\nThe search query contains errors..."
 
-    with patch('basic_memory.mcp.tools.chatgpt_tools.search_notes.fn', new_callable=AsyncMock) as mock_search:
+    with patch(
+        'basic_memory.mcp.tools.chatgpt_tools.search_notes.fn',
+        new_callable=AsyncMock
+    ) as mock_search:
         mock_search.return_value = error_message
 
         from basic_memory.mcp.tools.chatgpt_tools import search
@@ -102,7 +108,10 @@ async def test_fetch_successful_document():
 - relates_to [[Another Document]]
 """
 
-    with patch('basic_memory.mcp.tools.chatgpt_tools.read_note.fn', new_callable=AsyncMock) as mock_read:
+    with patch(
+        'basic_memory.mcp.tools.chatgpt_tools.read_note.fn',
+        new_callable=AsyncMock
+    ) as mock_read:
         mock_read.return_value = document_content
 
         from basic_memory.mcp.tools.chatgpt_tools import fetch
@@ -133,7 +142,10 @@ async def test_fetch_document_not_found():
 - If you provided a title, try using the exact permalink instead
 """
 
-    with patch('basic_memory.mcp.tools.chatgpt_tools.read_note.fn', new_callable=AsyncMock) as mock_read:
+    with patch(
+        'basic_memory.mcp.tools.chatgpt_tools.read_note.fn',
+        new_callable=AsyncMock
+    ) as mock_read:
         mock_read.return_value = error_content
 
         from basic_memory.mcp.tools.chatgpt_tools import fetch