Skip to content

Conversation

@jqnatividad
Copy link
Collaborator

resolves #3079

qsv now has MCP support.

qsv pro's MCP server will still have value-added functionality - e.g. CKAN support, Interactive FAIRification UI/UX, etc.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds Model Context Protocol (MCP) server support to qsv, enabling natural language interaction with qsv's 66 CSV data-wrangling commands through Claude Desktop.

Key changes:

  • Implements MCP server with 22 tools (20 common commands + generic fallback + pipeline tool)
  • Exposes 1,279 test examples as browsable MCP resources
  • Adds installation helper script and comprehensive documentation

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.claude/skills/src/types.ts Adds MCP-specific TypeScript interfaces for tools, resources, and pipeline steps
.claude/skills/src/mcp-tools.ts Implements MCP tool definitions and execution handlers for qsv commands
.claude/skills/src/mcp-server.ts Main MCP server implementation with tool and resource handlers
.claude/skills/src/mcp-resources.ts Resource provider exposing test examples as browsable MCP resources
.claude/skills/src/mcp-pipeline.ts Pipeline tool for chaining multiple qsv operations
.claude/skills/src/index.ts Exports MCP components for external use
.claude/skills/scripts/install-mcp.js Interactive installer for Claude Desktop configuration
.claude/skills/package.json Adds MCP SDK dependency and new npm scripts
.claude/skills/package-lock.json Locks MCP SDK and its dependencies
.claude/skills/README.md Documents MCP integration with quick start guide
.claude/skills/README-MCP.md Comprehensive MCP server documentation with examples and troubleshooting
Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 9 comments.

Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 10 comments.

Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

jqnatividad and others added 2 commits January 4, 2026 06:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 10 comments.

Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • .claude/skills/package-lock.json: Language not supported

@jqnatividad jqnatividad merged commit c683fbb into master Jan 4, 2026
23 checks passed
@jqnatividad jqnatividad deleted the 3079-mcp-server branch January 4, 2026 13:03
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.

feat: qsv pro MCP server

2 participants