Skip to content

feat: adding ability to specify different mcp server types - #4307

Merged
ArakTaiRoth merged 1 commit into
GovAlta:devfrom
hwen3:http_transport_mcp
Oct 14, 2026
Merged

ArakTaiRoth merged 1 commit into
GovAlta:devfrom
hwen3:http_transport_mcp

Conversation

@hwen3

@hwen3 hwen3 commented Sep 24, 2026

Copy link
Copy Markdown

Before (the change)

  • mcp server only supports stdio which makes it harder to host for multi user traffic usage

After (the change)

  • allowing support for both stdio and http (defaulting to stdio)

Make sure that you've checked the boxes below before you submit the PR

  • I have read and followed the setup steps
  • I have created necessary unit tests
  • I have tested the functionality in both React and Angular.

Steps needed to test

@ArakTaiRoth

Copy link
Copy Markdown
Collaborator

Closing as I don't know who contributed this and I can't view their past history, was never spoken to about this, it's not attached to a story, and it's failing.

@ArakTaiRoth ArakTaiRoth closed this Oct 8, 2026
@ArakTaiRoth

Copy link
Copy Markdown
Collaborator

Reopening as the author reached out to me, and this is for a valid reason and use case.

@twjeffery twjeffery left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@hwen3 Looks good to me

@hwen3

hwen3 commented Oct 14, 2026

Copy link
Copy Markdown
Author

thanks @twjeffery. looks like i dont have permission to merge and the build is failing due to permissions as well. also let me know if yall need additional context.

@ArakTaiRoth

Copy link
Copy Markdown
Collaborator

thanks @twjeffery. looks like i dont have permission to merge and the build is failing due to permissions as well. also let me know if yall need additional context.

@hwen3 I'm going to look at fixing the build, and then I'll merge. If you can fix the build as well before me, be my guest and then I'll merge.

@ArakTaiRoth
ArakTaiRoth merged commit 8e83cff into GovAlta:dev Oct 14, 2026
4 of 8 checks passed
@ArakTaiRoth

Copy link
Copy Markdown
Collaborator

thanks @twjeffery. looks like i dont have permission to merge and the build is failing due to permissions as well. also let me know if yall need additional context.

@hwen3 I'm going to look at fixing the build, and then I'll merge. If you can fix the build as well before me, be my guest and then I'll merge.

@hwen3 Actually, I just merged. The only thing that was failing was our preview build, which doesn't even apply for this PR anyways. And it was failing because the push came from a fork, not from the repository. This work should be merged now, and will be released to production nearer the end of the month.

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.

4 participants