Skip to content

Conversation

scholtz
Copy link

@scholtz scholtz commented Jun 27, 2025

Sample code how to access the session id.

Motivation and Context

Showcase #564

When this issue will be solved I believe this may be useful information for everybody how to obtain session id in the tool.

How Has This Been Tested?

Through local mcp tool in vscode.

{
  "servers": {
    "Local1": {
      "command": "npx",
      "cwd": "${input:cwd}",
      "args": ["mcp-remote", "http://localhost:5002/mcp/", "--transport", "http-first"]
    }
  },
  "inputs": [
    {
      "id": "cwd",
      "type": "promptString",
      "description": "Working Directory"
    },
    {
      "id": "debugType",
      "type": "promptString",
      "description": "Debug Type"
    }
  ]
}

Breaking Changes

This is just a sample how i believe it should work.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

This is example code on how to replicate issue #564

When this issue will be solved I believe this may be useful information for everybody how to obtain session id in the tool.

@scholtz
Copy link
Author

scholtz commented Jun 29, 2025

is there anything i can help so that this sample code can be merged into the samples?

}

[McpServerTool(Name = "get_session"), Description("Returns current session id")]
public async Task<string?> GetSession(IMcpServer mcpServer)
Copy link
Member

Choose a reason for hiding this comment

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

This sample seems somewhat artificial to me -- the session id is mostly intended as a key for session state so it would be nice to have an example showcasing that.

@halter73
Copy link
Contributor

Hi @scholtz. Thanks for your contribution. I agree with @eiriktsarpalis that a little contrived and not substantial enough to warrant an entire sample. If there were some useful way to use the session id to key into session state in the new AspNetCoreMcpPerSessionTools added by #724, that might be interesting. I could see the AspNetCoreMcpPerSessionTools being the sample we end up using to demo more advanced features like querying the session ID.

@halter73 halter73 closed this Aug 27, 2025
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.

3 participants