{
"event": "PreToolUse",
"tool_name": "Edit",
"tool_input": {
"file_path": "\/var\/www\/mcp-servers\/mcp-docs\/tools\/docs_tools\/docs_reader.py",
"old_string": " logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"success\",\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n\n return {\n \"success\": True,\n \"statistics\": stats\n }\n\n except Exception as e:\n logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"error\",\n error_message=str(e),\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n return {\"success\": False, \"error\": str(e)}",
"new_string": " _log_operation(logger, \"docs_statistics\", {}, start_time)\n return {\"success\": True, \"statistics\": stats}\n\n except Exception as e:\n _log_operation(\n logger, \"docs_statistics\", {}, start_time, \"error\", str(e)\n )\n return {\"success\": False, \"error\": str(e)}"
}
}
{
"tool_response": {
"filePath": "\/var\/www\/mcp-servers\/mcp-docs\/tools\/docs_tools\/docs_reader.py",
"oldString": " logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"success\",\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n\n return {\n \"success\": True,\n \"statistics\": stats\n }\n\n except Exception as e:\n logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"error\",\n error_message=str(e),\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n return {\"success\": False, \"error\": str(e)}",
"newString": " _log_operation(logger, \"docs_statistics\", {}, start_time)\n return {\"success\": True, \"statistics\": stats}\n\n except Exception as e:\n _log_operation(\n logger, \"docs_statistics\", {}, start_time, \"error\", str(e)\n )\n return {\"success\": False, \"error\": str(e)}",
"originalFile": "\"\"\"Read operations for documentation management.\"\"\"\n\nimport json\nimport sys\nimport time\nfrom typing import Optional\n\nsys.path.insert(0, \"\/opt\/mcp-servers\/mcp-docs\")\nfrom domain.dokumentation import LogEntry\nfrom infrastructure.docs_repository import get_repository\nfrom infrastructure.protokoll_logger import get_logger\n\nfrom .constants import DEFAULT_LIMIT, MS_PER_SECOND\n\n\ndef _log_operation(\n logger,\n tool_name: str,\n request_data: dict,\n start_time: float,\n status: str = \"success\",\n error: Optional[str] = None\n) -> None:\n \"\"\"Log an operation with timing information.\"\"\"\n logger.log(LogEntry(\n tool_name=tool_name,\n request=json.dumps(request_data),\n status=status,\n error_message=error,\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n\n\ndef register_reader_tools(mcp) -> None:\n \"\"\"Register read-only documentation tools with the MCP server.\"\"\"\n\n @mcp.tool()\n def docs_list(\n status: Optional[str] = None,\n parent_id: Optional[int] = None,\n search: Optional[str] = None,\n compact: bool = True,\n limit: int = DEFAULT_LIMIT\n ) -> dict:\n \"\"\"List all documents from the database.\"\"\"\n start_time = time.time()\n logger = get_logger()\n repo = get_repository()\n\n try:\n docs = repo.find_all(\n status=status,\n parent_id=parent_id,\n search=search,\n limit=limit\n )\n total = repo.count(status=status, parent_id=parent_id, search=search)\n\n result = {\n \"success\": True,\n \"docs\": [d.to_dict_compact() if compact else d.to_dict() for d in docs],\n \"total\": total,\n \"limit\": limit,\n \"compact\": compact\n }\n\n _log_operation(\n logger, \"docs_list\",\n {\"status\": status, \"parent_id\": parent_id, \"search\": search, \"limit\": limit},\n start_time\n )\n return result\n\n except Exception as e:\n _log_operation(\n logger, \"docs_list\",\n {\"status\": status, \"search\": search},\n start_time, \"error\", str(e)\n )\n return {\"success\": False, \"error\": str(e)}\n\n @mcp.tool()\n def docs_get(\n id: Optional[int] = None,\n path: Optional[str] = None,\n include_children: bool = False,\n include_breadcrumb: bool = False\n ) -> dict:\n \"\"\"Retrieve a document by ID or path.\"\"\"\n start_time = time.time()\n logger = get_logger()\n repo = get_repository()\n\n try:\n if id is None and path is None:\n return {\n \"success\": False,\n \"error\": \"Either id or path must be specified\"\n }\n\n doc = None\n if id is not None:\n doc = repo.find_by_id(id)\n elif path is not None:\n doc = repo.find_by_path(path)\n\n if not doc:\n return {\"success\": False, \"error\": \"Document not found\"}\n\n result = {\n \"success\": True,\n \"doc\": doc.to_dict()\n }\n\n if include_children and doc.id:\n children = repo.find_children(doc.id)\n result[\"children\"] = [c.to_dict_compact() for c in children]\n\n if include_breadcrumb and doc.id:\n result[\"breadcrumb\"] = repo.get_breadcrumb(doc.id)\n\n _log_operation(logger, \"docs_get\", {\"id\": id, \"path\": path}, start_time)\n return result\n\n except Exception as e:\n _log_operation(\n logger, \"docs_get\", {\"id\": id, \"path\": path},\n start_time, \"error\", str(e)\n )\n return {\"success\": False, \"error\": str(e)}\n\n @mcp.tool()\n def docs_search(query: str, limit: int = DEFAULT_LIMIT) -> dict:\n \"\"\"Full-text search in all documents.\"\"\"\n start_time = time.time()\n logger = get_logger()\n repo = get_repository()\n\n try:\n docs = repo.find_all(search=query, limit=limit)\n\n _log_operation(\n logger, \"docs_search\", {\"query\": query, \"limit\": limit}, start_time\n )\n return {\n \"success\": True,\n \"query\": query,\n \"results\": [d.to_dict_compact() for d in docs],\n \"count\": len(docs)\n }\n\n except Exception as e:\n _log_operation(\n logger, \"docs_search\", {\"query\": query},\n start_time, \"error\", str(e)\n )\n return {\"success\": False, \"error\": str(e)}\n\n @mcp.tool()\n def docs_statistics() -> dict:\n \"\"\"Retrieve statistics about all documents.\"\"\"\n start_time = time.time()\n logger = get_logger()\n repo = get_repository()\n\n try:\n stats = repo.get_statistics()\n\n logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"success\",\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n\n return {\n \"success\": True,\n \"statistics\": stats\n }\n\n except Exception as e:\n logger.log(LogEntry(\n tool_name=\"docs_statistics\",\n request=\"{}\",\n status=\"error\",\n error_message=str(e),\n duration_ms=int((time.time() - start_time) * MS_PER_SECOND)\n ))\n return {\"success\": False, \"error\": str(e)}\n",
"structuredPatch": [
{
"oldStart": 166,
"oldLines": 24,
"newStart": 166,
"newLines": 11,
"lines": [
" try:",
" stats = repo.get_statistics()",
" ",
"- logger.log(LogEntry(",
"- tool_name=\"docs_statistics\",",
"- request=\"{}\",",
"- status=\"success\",",
"- duration_ms=int((time.time() - start_time) * MS_PER_SECOND)",
"- ))",
"+ _log_operation(logger, \"docs_statistics\", {}, start_time)",
"+ return {\"success\": True, \"statistics\": stats}",
" ",
"- return {",
"- \"success\": True,",
"- \"statistics\": stats",
"- }",
"-",
" except Exception as e:",
"- logger.log(LogEntry(",
"- tool_name=\"docs_statistics\",",
"- request=\"{}\",",
"- status=\"error\",",
"- error_message=str(e),",
"- duration_ms=int((time.time() - start_time) * MS_PER_SECOND)",
"- ))",
"+ _log_operation(",
"+ logger, \"docs_statistics\", {}, start_time, \"error\", str(e)",
"+ )",
" return {\"success\": False, \"error\": str(e)}"
]
}
],
"userModified": false,
"replaceAll": false
}
}