Skip to content

Conversation

souvikghosh04
Copy link
Contributor

@souvikghosh04 souvikghosh04 commented Sep 26, 2025

Why make this change?

Note: This PR is work in progress. Description and other details will be updated with more details.

What is this change?

This PR implements built-int tools delete-entity and execute-entity as part of built-in tools to support delete operation and execution of stored procedures.

How was this tested?

Functional testing using Insomnia client by running DAB in localhost and local SQL DB database

  • MCP endpoint: http://localhost:5000/mcp
  • JSON payload (details below)
  • Querying and validating data in local database
  • TODO: Test in MCP clients and MCP inspector

Sample Request(s)

  • delete-record
POST: http://localhost:5000/mcp

{
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "name": "delete-record",
    "arguments": {
      "entityName": "Book",
      "primaryKey": {
        "id": 5003
      }
    }
  },
  "id": 1
}


  • execute-entity
{
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "name": "execute-entity",
    "arguments": {
      "entityName": "GetBook",
      "parameters": {
				"id": 5
			}
    }
  },
  "id": 2
}


{
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "name": "execute-entity",
    "arguments": {
      "entityName": "UpdateBookTitle",
      "parameters": {
				"id": 5,
				"title": "Vampire Diaries"
			}
    }
  },
  "id": 5
}

Jerry Nixon and others added 30 commits August 27, 2025 15:06
This reverts commit 04b19f9, reversing
changes made to 3948822.
…a-api-builder into jerry-mcp-core"

This reverts commit 0b649dbbc877a8ad009ac31eb4a082664d49bce3, reversing
changes made to 2240172.
## Why make this change?

- Introduces a dedicated runtime layer for MCP

## What is this change?

- Adds a runtime initialization module that centralizes configuration
load, validation, DI registration, and service/materialization order.
- Refactors existing startup code into clearer phases; removes
duplicated wiring.
- Improves logging structure and prepares for future feature flags /
layered config overrides.
- Cleans up deprecated code paths (overall net +901 / −130 LOC).
- No intentional breaking changes (verify before merge).

## How was this tested?

- [ ] Integration Tests
- [ ] Unit Tests

## Sample Request(s)

- Discover tools: POST `http://localhost:5000/mcp`

JSON body

```
{
  "jsonrpc": "2.0",
  "id": "1",
  "method": "tools/list",
  "params": {}
}
```

- List entities: POST `http://localhost:5000/mcp`

JSON body

```
{
  "jsonrpc": "2.0",
  "id": 1,
  "method": "tools/call",
  "params": {
    "name": "list_entities"
  }
}
```

---------

Co-authored-by: Rahul Nishant <[email protected]>
Co-authored-by: RubenCerna2079 <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Aniruddh Munde <[email protected]>
souvikghosh04 and others added 16 commits September 24, 2025 16:35
function parameters should be camelCased.
## Why make this change?
- Fixes service tests for mcp-core PR

## What is this change?
- It creates the JsonConverterFactory that is needed to serialize and
deserialize the MCP properties correctly.
- Fixes tests to ensure that the serialization/deserialization works as
intended.

## How was this tested?

- [ ] Integration Tests
- [X] Unit Tests
@souvikghosh04 souvikghosh04 changed the title Usr/sogh/mcpdev Adding tools- delete-entity and execute-entity Sep 26, 2025
@souvikghosh04 souvikghosh04 changed the title Adding tools- delete-entity and execute-entity Adding tools: delete-entity and execute-entity Sep 26, 2025
@souvikghosh04 souvikghosh04 self-assigned this Sep 26, 2025
@souvikghosh04 souvikghosh04 added mssql an issue thats specific to mssql mcp-server labels Sep 26, 2025
@souvikghosh04 souvikghosh04 added this to the 1.7 milestone Sep 26, 2025
@souvikghosh04 souvikghosh04 changed the title Adding tools: delete-entity and execute-entity Adding tools: delete-entity Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mcp-server mssql an issue thats specific to mssql
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants