-
Notifications
You must be signed in to change notification settings - Fork 1
Add comprehensive input/output schema for stdio and http MCP transports #7
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.
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
Conversation
Co-authored-by: pelikhan <[email protected]>
Co-authored-by: pelikhan <[email protected]>
|
|
||
| #### General: | ||
|
|
||
| - **`container-image`**: Container image to run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot this one should have been "container-version", and keep "container"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added container-version input in action.yml and removed the deprecated status from container. Both fields are now documented in the README. (commit 6295988)
Co-authored-by: pelikhan <[email protected]>
README.md
Outdated
| container-image: my-mcp-server:latest | ||
| ``` | ||
| ## Development |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot move development guide in CONTRIBUTING.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created CONTRIBUTING.md and moved all development instructions there. README now includes a link to the contributing guide. (commit 6295988)
…ogging Co-authored-by: pelikhan <[email protected]>
Co-authored-by: pelikhan <[email protected]>
Merged commits: - Add vitest test suites for TypeScript functions (#8) - Add comprehensive input/output schema for stdio and http MCP transports (#7) - Add agentic workflow for action metadata consistency checking (#11) - Add agentic workflow for automated super-linter analysis and reporting (#4) Resolved merge conflict in dist/index.js.map by rebuilding the package. Co-authored-by: pelikhan <[email protected]>
Merged commits from main: - Move test files to src/ directory alongside source files (#12) - Add daily test improver agentic workflow (#10) - Add agentic workflow for automated super-linter analysis and reporting (#4) - Add agentic workflow for action metadata consistency checking (#11) - Add comprehensive input/output schema for stdio and http MCP transports (#7) - Add vitest test suites for TypeScript functions (#8) Used merge=ours strategy for dist/index.js.map to keep our generated version. All 49 tests pass. Co-authored-by: pelikhan <[email protected]>
Extends the action to support both stdio and http MCP upstream connections with type-specific configuration parameters.
Changes
Action Interface (
action.yml)type(stdio/http),command,args,envfor stdio;url,headersfor http;container,container-image,container-version,logs-dirfor general configapi-key→tokenfor consistency; kepturl,portInput Marshaling (
src/main.ts)args(array),env/headers(string-keyed objects)commandrequired for stdio,urlrequired for httpcontainer,container-image, andcontainer-versioninputsUpstream Configuration (
src/server.ts)UpstreamConfiginterface withenvandheadersfieldsStdioClientTransportcore.warning()when headers provided (MCP SDK limitation)Documentation
Example Usage
Notes
container,container-image, andcontainer-versioninputs for container configurationOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.