Skip to content

[WIP] Use mcp-apps-testing from npm instead of embedding#29

Merged
aviveldan merged 5 commits intomainfrom
copilot/use-mcp-apps-testing-from-npm
Feb 14, 2026
Merged

[WIP] Use mcp-apps-testing from npm instead of embedding#29
aviveldan merged 5 commits intomainfrom
copilot/use-mcp-apps-testing-from-npm

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 14, 2026

Plan to use mcp-apps-testing from npm

  • Add mcp-apps-testing npm package as a dev dependency in ui/package.json
  • Update the import in ui/mcp-apps.spec.ts to use the npm package instead of local directory
  • Remove the "Build mcp-apps-testing framework" step from .github/workflows/test.yml
  • Remove the local ui/mcp-apps-testing directory from the repository
  • Run tests to verify the changes work correctly
  • Update documentation to reflect the changes
  • Clean up .gitignore to remove obsolete references
Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature]: use mcp-apps-testing from npm instead of locally embedding</issue_title>
<issue_description>### Is your feature request related to a problem?

use mcp-apps-testing from npm instead of locally embedding

https://www.npmjs.com/package/mcp-apps-testing

Describe the solution you'd like

Once it's available on npm, use it.

Describe alternatives you've considered

No response

Additional context

No response</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@github-actions github-actions bot added documentation Improvements or additions to documentation github-actions dependencies ui labels Feb 14, 2026
@aviveldan aviveldan marked this pull request as ready for review February 14, 2026 12:10
@aviveldan aviveldan self-requested a review as a code owner February 14, 2026 12:10
@aviveldan aviveldan merged commit 0d242e0 into main Feb 14, 2026
10 checks passed
Copilot stopped work on behalf of aviveldan due to an error February 14, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies documentation Improvements or additions to documentation github-actions ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: use mcp-apps-testing from npm instead of locally embedding

2 participants