Altinity / altinity-mcp / 25512044591

82%

push

github

tools: isNullableCHType handles LowCardinality(Nullable(...))

Pre-merge review finding on the strict-mode-schema work
(b02ba21). isNullableCHType was a literal `strings.HasPrefix(s,
"nullable(")` check, so it missed the canonical CH column-type
wrap order `LowCardinality(Nullable(X))` and treated those
columns as non-nullable.

Effect: dynamic write tools advertised LowCardinality(Nullable(X))
columns as REQUIRED in the input schema even when the underlying
table accepts NULL — the model couldn't omit them on INSERT.

Fix: strip one leading `lowcardinality(` before the `nullable(`
prefix check. Test cases extended with the canonical-wrap form
(both casings) and a LowCardinality(String)-without-Nullable
counterexample to confirm we don't false-positive.

Extracted from worktree-tools-strict-schema-wt commit 88425e9
(other fixes there target PR #104 and are folded separately).

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

3 of 3 new or added lines in 1 file covered. (100.0%)

4509 of 5500 relevant lines covered (81.98%)

1.22 hits per line