Skip to content

fix(mcp): accept the JSON type alias in SCAN - #1340

Draft
joshrotenberg wants to merge 2 commits into
mainfrom
fix/mcp-scan-json-alias-1218
Draft

joshrotenberg wants to merge 2 commits into
mainfrom
fix/mcp-scan-json-alias-1218

Conversation

@joshrotenberg

@joshrotenberg joshrotenberg commented Nov 12, 2026 •

Copy link
Copy Markdown
Collaborator

redis_scan now maps ASCII case variants of json to the native RedisJSON type ReJSON-RL in its SCAN TYPE argument. Other type strings retain their bytes, and the output keeps the caller's original type label. The tool and field descriptions document both spellings.

Bounded coverage for #1218; leaves #1218 open. Schema, defaults, read-only annotations and traversal are unchanged. A non-checkout merge-tree check is clean against #1325 at 718c99c and retains both changes; that contributor branch is unchanged, and no combined runtime result is claimed.

Validation:

  • Same alias test corpus: old handler 3 passed / 3 failed; updated handler 6 passed.
  • Minimal and all-feature configurations: 6 alias tests plus 2 existing limit-guard tests passed; none ignored or filtered out.
  • Pinned Redis 8.6.7 / RedisJSON 80600: JSON aliases and native type return the same JSON keys under RESP2 and RESP3, with string/unfiltered controls and stored values checked.
  • Targeted catalog baseline contract, formatting and strict all-target/all-feature Clippy pass.

Cargo.lock is unchanged. The PR remains in draft for independent review, the separate security dependency update and hosted validation after refresh.

This branch has not been deployed

No deployments
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.

1 participant