Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add Azure Table list back #743

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 13 commits into
base: main
Choose a base branch
Loading
from
Draft

Add Azure Table list back #743

wants to merge 13 commits into from

Conversation

alzimmermsft
Copy link
Contributor

@alzimmermsft alzimmermsft commented Oct 8, 2025 •
edited
Loading

What does this PR do?

Adds back Azure Table list tool under Azure.Mcp.Tools.Table, azmcp_table_list.

GitHub issue number?

Fixes #675

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages (cleanup guide)
    • Added comprehensive tests for new/modified functionality
    • Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
    • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • Extra steps for Azure MCP Server tool changes:
    • Updated test prompts in /servers/Azure.Mcp.Server/docs/e2eTestPrompts.md

@alzimmermsft alzimmermsft self-assigned this Oct 8, 2025
@github-project-automation github-project-automation bot moved this to Untriaged in Azure MCP Server Oct 8, 2025
@alzimmermsft alzimmermsft marked this pull request as ready for review October 8, 2025 19:37
@alzimmermsft alzimmermsft requested review from a team as code owners October 8, 2025 19:37
Copilot
Copilot AI reviewed Oct 8, 2025
View reviewed changes
Copy link
Contributor

@Copilot 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 back the Azure Table list tool functionality under Azure.Mcp.Tools.Table, providing the azmcp_table_list command to list all tables in an Azure Storage account.

Key changes include:

  • Implementation of a new Azure Table tool with list functionality
  • Complete test coverage with both unit and live tests
  • Integration into the Azure MCP Server with proper documentation

Reviewed Changes

Copilot reviewed 28 out of 29 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tools/Azure.Mcp.Tools.Table/src/* Core implementation of the Table tool including commands, services, and options
tools/Azure.Mcp.Tools.Table/tests/* Comprehensive test coverage with unit tests and live tests
servers/Azure.Mcp.Server/src/Program.cs Integration of Table tool into the server
servers/Azure.Mcp.Server/docs/* Documentation updates for the new table commands
Directory.Packages.props Addition of Azure.Data.Tables package dependency
AzureMcp.sln Solution file updates to include new Table tool projects

tools/Azure.Mcp.Tools.Table/tests/test-resources-post.ps1 Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/src/TableSetup.cs Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/tests/Azure.Mcp.Tools.Table.LiveTests/TableCommandTests.cs Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/tests/Azure.Mcp.Tools.Table.LiveTests/TableCommandTests.cs Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/tests/Azure.Mcp.Tools.Table.LiveTests/TableCommandTests.cs Outdated Show resolved Hide resolved
vcolin7
vcolin7 reviewed Oct 8, 2025
View reviewed changes
.github/CODEOWNERS Outdated Show resolved Hide resolved
servers/Azure.Mcp.Server/docs/e2eTestPrompts.md Outdated Show resolved Hide resolved
servers/Azure.Mcp.Server/CHANGELOG.md Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Tables/tests/Azure.Mcp.Tools.Tables.LiveTests/TablesCommandTests.cs Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/tests/Azure.Mcp.Tools.Table.UnitTests/TableListCommandTests.cs Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Table/src/TableSetup.cs Outdated Show resolved Hide resolved
jongio
jongio reviewed Oct 9, 2025
View reviewed changes
servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated Show resolved Hide resolved
vcolin7
vcolin7 reviewed Oct 9, 2025
View reviewed changes
tools/Azure.Mcp.Tools.Tables/src/TablesSetup.cs Show resolved Hide resolved
tools/Azure.Mcp.Tools.Tables/src/Commands/TablesListCommand.cs Outdated Show resolved Hide resolved
tools/Azure.Mcp.Tools.Tables/tests/test-resources.bicep Show resolved Hide resolved
@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure MCP Server Oct 9, 2025
vcolin7
vcolin7 requested changes Oct 9, 2025
View reviewed changes
Copy link
Contributor

@vcolin7 vcolin7 left a comment

Choose a reason for hiding this comment

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

Holding off until a Cosmos conversation is resolved.

@alzimmermsft alzimmermsft added the Do Not Merge Do Not Merge / WIP PRs label Oct 10, 2025
@alzimmermsft alzimmermsft marked this pull request as draft October 10, 2025 20:24
@alzimmermsft alzimmermsft added this to the 2025-11 milestone Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jongio jongio jongio left review comments

Copilot code review Copilot Copilot left review comments

@vcolin7 vcolin7 vcolin7 requested changes

@hallipr hallipr Awaiting requested review from hallipr hallipr is a code owner automatically assigned from microsoft/azure-mcp

@joshfree joshfree Awaiting requested review from joshfree joshfree is a code owner automatically assigned from microsoft/azure-mcp

@wbreza wbreza Awaiting requested review from wbreza wbreza is a code owner automatically assigned from microsoft/azure-mcp

@conniey conniey Awaiting requested review from conniey conniey is a code owner automatically assigned from microsoft/azure-mcp

@tmeschter tmeschter Awaiting requested review from tmeschter tmeschter is a code owner automatically assigned from microsoft/azure-mcp

@sandeep-sen sandeep-sen Awaiting requested review from sandeep-sen sandeep-sen is a code owner automatically assigned from microsoft/azure-mcp

@JasonYeMSFT JasonYeMSFT Awaiting requested review from JasonYeMSFT JasonYeMSFT is a code owner automatically assigned from microsoft/azure-mcp

Requested changes must be addressed to merge this pull request.

Assignees

@alzimmermsft alzimmermsft

Labels

Do Not Merge Do Not Merge / WIP PRs

Projects

Azure MCP Server
Status: In Progress

Milestone

2025-11

Development

Successfully merging this pull request may close these issues.

[FEATURE] Support for searching Azure Storage Tables

3 participants

@alzimmermsft @jongio @vcolin7
Morty Proxy This is a proxified and sanitized view of the page, visit original site.