Altinity / altinity-mcp / 24508576960

77%

main: 77%

Build Type

push

github

Committed by BorisTyshkevich

Commit Message

Make execute_query always read-only, reject write statements

execute_query is the READ tool — it should always have ReadOnlyHint: true
regardless of whether the server allows writes. Previously, when
readOnly=false, execute_query was annotated as destructive, which was
incorrect now that read and write tools are separate.

Changes:
- execute_query always annotated ReadOnlyHint:true, DestructiveHint:false
- New HandleReadOnlyQuery handler rejects non-SELECT statements with a
  clear error directing users to write_query
- Retire makeExecuteQueryAnnotations() — annotations are now tool-specific
- write_query retains HandleExecuteQuery (no read-only enforcement)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

16 of 22 new or added lines in 1 file covered. (72.73%)

97 existing lines in 1 file now uncovered.

4301 of 5425 relevant lines covered (79.28%)

1.13 hits per line