Skip to content
This repository was archived by the owner on Jul 24, 2026. It is now read-only.

Add hostname and server type to mcp req#8

Merged
dimitriosGX merged 1 commit into
masterfrom
dk-stdio
Jul 16, 2026
Merged

Add hostname and server type to mcp req#8
dimitriosGX merged 1 commit into
masterfrom
dk-stdio

Conversation

@dimitriosGX

@dimitriosGX dimitriosGX commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Add hostname to MCP submit requests and set explicit stdio type for local servers. This lets the backend approve stdio servers per machine and removes the need to infer transport.

  • New Features
    • SubmitRequest includes hostname; request body sends "hostname".
    • Daemon populates hostname via platform::hostname() (env vars, then hostname command).
    • For stdio configs, request body sets "type": "stdio"; tests updated accordingly.

Written for commit b2f524f. Summary will update on new commits.

Review in cubic

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 3 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@dimitriosGX
dimitriosGX merged commit 389ac47 into master Jul 16, 2026
11 checks passed
@dimitriosGX
dimitriosGX deleted the dk-stdio branch July 16, 2026 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant