Skip to content

feat(mcp): add list_tags and get_tag_info MCP tools#12

Open
hbrooks wants to merge 6 commits into
masterfrom
demo/pr-40349
Open

feat(mcp): add list_tags and get_tag_info MCP tools#12
hbrooks wants to merge 6 commits into
masterfrom
demo/pr-40349

Conversation

@hbrooks

@hbrooks hbrooks commented May 28, 2026

Copy link
Copy Markdown

aminghadersohi and others added 6 commits May 21, 2026 21:14
Adds list_tags and get_tag_info MCP tools in a new superset/mcp_service/tag/
domain, following the existing database/dataset/dashboard patterns.
Registers both tools in app.py and covers them with unit tests.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…upport

TagFilter documents 'ct' as a valid operator but ColumnOperatorEnum was
missing it, causing ValidationError when constructing TagFilter with opr="ct".
Added ct = "ct" with ilike("%val%") mapping, updated TYPE_OPERATOR_MAP for
string columns, and updated base_dao_test to cover the new operator.
…tils

Remove duplicate _humanize_timestamp from 5 schema files (chart, dashboard,
database, dataset, tag) and expose a single canonical humanize_timestamp in
utils/response_utils. Use the timezone-aware implementation throughout.
…y interface

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Tag inherits AuditMixinNullable which provides both created_on and
changed_on. The schema included changed_on but omitted created_on,
leaving it inconsistent with all sibling schemas (chart, dashboard,
dataset, database) which expose both timestamps.

Adds created_on/created_on_humanized to TagInfo, serialize_tag_object,
ALL_TAG_COLUMNS, SORTABLE_TAG_COLUMNS, and the list_tags docstring.
…tency

All analogous item_serializer wrappers across chart, dashboard, dataset,
and database list tools use `cols` (not `_cols`) as the parameter name.
The underscore prefix is not needed — ruff ARG rules are not enabled, so
unused function parameters are not flagged.
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