Skip to content

fix(publisher): support --file flag in publish command #868

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.

Sign up for GitHub

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

Jump to bottom
Open
majiayu000 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(publisher): support --file flag in publish command #868

majiayu000 wants to merge 1 commit into from
+21 −5

Conversation

@majiayu000
Copy link

@majiayu000 majiayu000 commented Dec 28, 2025

Summary

  • Added proper flag parsing using Go's flag package
  • mcp-publisher publish --file path/to/server.json now works correctly
  • Maintains backward compatibility with positional arguments
  • Improved error messages to show actual file path when custom file not found

Test

# Before: Error: server.json not found
mcp-publisher publish --file .registry/server.json

# After: reads .registry/server.json correctly

Fixes #767

@majiayu000
fix(publisher): support --file flag in publish command
56f36c7 
Added proper flag parsing using the flag package so that
'mcp-publisher publish --file path/to/server.json' works correctly.

Also improved error messages to show the actual file path when
a custom file is specified but not found.

Fixes modelcontextprotocol#767

Signed-off-by: majiayu000 <1835304752@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

publish subcommand's --file flag not working

1 participant

@majiayu000