Altinity / altinity-mcp
77%
main: 77%

LAST BUILD BRANCH: dependabot/go_modules/golang.org/x/sync-0.21.0
DEFAULT BRANCH: main
Repo Added
Token 563hba5bwFOyvxaiv1nGu7hzJBfrENnzl regen
Build 426 Last
Files 22
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH v0.1.0-issue35
branch: v0.1.0-issue35
CHANGE BRANCH
x
Sync Branches

coverage: 76.846% (-2.4%) from 79.281%
24557856044

push

github

BorisTyshkevich 
Fix tools-management issues from app-team review

1. getArgumentsMap now returns (map, error) instead of silently swallowing
   JSON parse errors. All 5 call sites updated to surface the error to the
   MCP client instead of producing cryptic downstream errors.

2. Add clickhouse.max_query_length (default 10MB) and max_parameter_size
   (default 10MB) limits. Oversize inputs rejected before reaching CH.

3. New truncateErrForClient helper caps error messages sent to clients
   at 2000 chars. Full error still logged server-side. 5 call sites updated.

4. Unit tests for buildInsertQuery (unicode, null bytes, backslash,
   quote escaping, required-param validation, optional-param omission,
   no-columns) and buildDynamicWriteQuery (mode dispatch, unsupported
   mode rejection). New TestRegisterTools_RejectsUnsupportedWriteModes
   covers config-time rejection of update/upsert modes.

5. EnsureDynamicTools now uses RWMutex fast path + TryLock so concurrent
   requests don't block during initial slow discovery — they see static
   tools immediately and get updated via notifications/tools/list_changed
   when discovery completes.

Cleanup:
- Remove dead dynamicToolMeta.IsStatic field
- Reject update/upsert write modes at config-parse time in RegisterTools
  instead of at call time (the tool no longer registers if it can't work)
- Simplify buildDynamicWriteQuery dispatcher (only insert is valid)
- Update TestGetArgumentsMap_ErrorPath to match new error-returning behavior

Declined from review spec:
- 1MB query limit default (too restrictive for OLAP; using 10MB)
- 1000-char error truncation (too aggressive; using 2000)

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

50 of 105 new or added lines in 2 files covered. (47.62%)

252 existing lines in 2 files now uncovered.

4361 of 5675 relevant lines covered (76.85%)

1.1 hits per line

Relevant lines Covered
5675 RELEVANT LINES 4361 COVERED LINES
1.1 HITS PER LINE
Source Files on v0.1.0-issue35

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
24557856044 v0.1.0-issue35 Fix tools-management issues from app-team review 1. getArgumentsMap now returns (map, error) instead of silently swallowing JSON parse errors. All 5 call sites updated to surface the error to the MCP client instead of producing cryptic down... push BorisTyshkevich github
76.85
24508576960 v0.1.0-issue35 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... push BorisTyshkevich github
79.28
24507656808 v0.1.0-issue35 Defer dynamic tool discovery until credentials are available In forward-OAuth mode with blank static credentials, EnsureDynamicTools failed during the tools/list handshake because no Bearer token was present yet. Discovery now checks for availabl... push BorisTyshkevich github
79.77
24505715835 v0.1.0-issue35 Fix column_type compat: use default_kind only for column filtering The column_type field doesn't exist in system.columns on all ClickHouse versions (e.g., 26.1.x Altinity Antalya). This caused dynamic write tool discovery to fail silently. The de... push BorisTyshkevich github
83.81
24503086106 v0.1.0-issue35 Serve HTTP transport at root path and fix Issue #35 bugs - Change HTTP transport endpoint from /http to / (root path) - SSE transport unchanged at /sse, OAuth unchanged at /oauth/* - Remove /http-based OAuth .well-known aliases (now redundant) - ... push BorisTyshkevich github
83.78
See All Builds (425)

Badge your Repo: altinity-mcp

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh