Skip to content

Conversation

@niclim
Copy link
Contributor

@niclim niclim commented Dec 17, 2025

Motivation and Context

When prettier is defined in dependencies, it gets pulled into when installed in other packages. Specifically, when we install modelcontextprotocol/ext-apps -> prettier also gets installed, which is different to our prettier package. This can change the conflict with downstream repo's prettier version and change how code is formatted

How Has This Been Tested?

Should have no user facing changes (dev only)

  • pulled in by installing "@modelcontextprotocol/ext-apps": "git://github.com/niclim/ext-apps.git#nl/move-prettier-to-dev-deps ", and confirmed that we're not longer pulling in the prettier defined in @modelcontextprotocol/ext-apps
  • ran npm run prettier -> formats with no changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 18, 2025

Open in StackBlitz

npm i https://pkg.pr.new/modelcontextprotocol/ext-apps/@modelcontextprotocol/ext-apps@179

commit: 037c37d

Copy link
Collaborator

@ochafik ochafik left a comment

Choose a reason for hiding this comment

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

Thanks!

@ochafik
Copy link
Collaborator

ochafik commented Dec 18, 2025

Thanks @niclim ! Would you mind updating your branch? CI should turn green now that #180 (comment) got in.

@niclim
Copy link
Contributor Author

niclim commented Dec 18, 2025

@ochafik should be updated now!

@ochafik ochafik merged commit 9963d0f into modelcontextprotocol:main Dec 18, 2025
14 checks passed
@ochafik ochafik mentioned this pull request Jan 7, 2026
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