Skip to content

Commit 1ede40b

Browse files
PCfVWPCfVW
committed
refactor: migrate to unified add_index() API for python-arango 8.x compatibility
Replace deprecated add_*_index() methods with the unified add_index() API introduced in python-arango 8.x to eliminate deprecation warnings and future-proof the codebase. Changes: - Updated handle_create_index() in handlers.py to use col.add_index() - Replaced 6 deprecated methods (add_persistent_index, add_hash_index, add_skiplist_index, add_ttl_index, add_fulltext_index, add_geo_index) - Build index_data dictionary with type-specific parameters - Use formatter=True for backward compatibility with snake_case fields Benefits: - Eliminates DeprecationWarning for add_geo_index() - Uses python-arango 8.x recommended API - Cleaner, more maintainable code with single unified method - Future-proof against removal of deprecated methods in 9.x Testing: - All existing unit tests pass (171 passed) - Verified with live ArangoDB 3.11.14 - all index types work correctly - No deprecation warnings in production usage - Backward compatible - no breaking changes to MCP server API Related to python-arango 8.2.2 upgrade (see DEPENDENCY_UPGRADE_ANALYSIS)
1 parent c3595c6 commit 1ede40b

File tree

5 files changed

+257
-131
lines changed

5 files changed

+257
-131
lines changed

‎mcp_arangodb_async/entry.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ async def run_stdio() -> None:
439439
write_stream,
440440
InitializationOptions(
441441
server_name="mcp-arangodb-async",
442-
server_version="0.2.5",
442+
server_version="0.2.6",
443443
capabilities=server.get_capabilities(
444444
notification_options=NotificationOptions(),
445445
experimental_capabilities={},

0 commit comments

Comments
 (0)