Skip to content

chore: update author metadata to LF Projects#1992

Merged
maxisbey merged 1 commit intomainfrom
update-author-metadata
Feb 4, 2026
Merged

chore: update author metadata to LF Projects#1992
maxisbey merged 1 commit intomainfrom
update-author-metadata

Conversation

@maxisbey
Copy link
Contributor

@maxisbey maxisbey commented Feb 4, 2026

Update authors across all pyproject.toml files to Model Context Protocol a Series of LF Projects, LLC., matching the TypeScript SDK. Also removes stale maintainers from example pyproject.toml files.

@maxisbey maxisbey requested a review from dsp-ant February 4, 2026 10:50
@claude
Copy link

claude bot commented Feb 4, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

description = "Model Context Protocol SDK"
readme = "README.md"
requires-python = ">=3.10"
authors = [{ name = "Anthropic, PBC." }]
Copy link
Member

Choose a reason for hiding this comment

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

Well, Anthropic did authored... 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha true... is it normal to keep previous authors as well in something like this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

according to claude it's not haha, so gonna merge

Copy link
Member

@Kludex Kludex Feb 4, 2026

Choose a reason for hiding this comment

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

Since you are asking... I'm not sure how I feel about removing a maintainer from the maintainer list (Justin). It feels like he is just forgotten from the project.

Haha true... is it normal to keep previous authors as well in something like this?

You can... I don't think there are strict rules, but I don't think MCP org is the author of this project, it's Anthropic.

Copy link
Member

Choose a reason for hiding this comment

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

MCP may have ownership, but author doesn't change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since you are asking... I'm not sure how I feel about removing a maintainer from the maintainer list (Justin). It feels like he is just forgotten from the project

Yea definitely not wanting to remove credit from someone at all.

I was talking to @dsp-ant about it and the thinking the list should mainly contain active maintainers since it has contact details. But I'm not super fussed either way.

Justin is still kept as lead maintainer here https://github.com/modelcontextprotocol/modelcontextprotocol/blob/main/MAINTAINERS.md, although that's ofc not in this repo.

@maxisbey maxisbey merged commit d3133ae into main Feb 4, 2026
33 checks passed
@maxisbey maxisbey deleted the update-author-metadata branch February 4, 2026 11:01
@Kludex
Copy link
Member

Kludex commented Feb 4, 2026

Instead of author on the package metadata, what needs to be changed is this: https://github.com/modelcontextprotocol/python-sdk/blob/main/LICENSE

@Kludex
Copy link
Member

Kludex commented Feb 4, 2026

...and by change, I don't mean replace Anthropic by MCP org, but adding a new copyright line.

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.

2 participants