Skip to content

feat: refactor server tools and add health checks#2

Merged
endemics merged 4 commits into
endemics:mainfrom
gettingstuffdone:feat/server-refactor
Mar 29, 2026
Merged

feat: refactor server tools and add health checks#2
endemics merged 4 commits into
endemics:mainfrom
gettingstuffdone:feat/server-refactor

Conversation

@gettingstuffdone

Copy link
Copy Markdown

Description:
This PR refactors the Python server's tool organization and adds stability enhancements:

  • Tool Reorganization: Moved core tool logic from server/main.py into a structured server/tools/ package.
  • Formatting Logic: Extracted Markdown table and result formatting into server/tools/formatting.py for better reuse.
  • Health Check: Added a /health endpoint to the Python server to allow the Go gateway to monitor worker status.
  • Iceberg Bug Fix: Corrected S3 path parsing in catalog/iceberg.py to handle trailing slashes in table prefixes.
  • Simplified Workflow: Merged estimate_query into datalake_query via a new explain_only flag, streamlining the MCP toolset.
  • Test Coverage: Added unit tests for the Iceberg metadata reader (server/tests/test_iceberg_reader.py).
  • Cleanup: Updated .gitignore to exclude local config/config.yaml.

Comment thread server/tools/query.py
f"To proceed anyway, call `datalake_query` with `force=true`."
)

if estimate.warning and not params.force:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why removing the warning here?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be restored now

@endemics
endemics merged commit f3dd07b into endemics:main Mar 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants