Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 3, 2025

  • Modify .github/workflows/check.yml to also run on push events to the main branch
  • Fix TypeScript build by adding skipLibCheck: true to tsconfig.json (fixes @modelcontextprotocol/[email protected] missing @cfworker/json-schema type declarations)
  • Verify build passes locally
  • Run code review
Original prompt

Can you make sure that the tests are running on commits on the main branch as well?


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add tests execution on main branch commits Run tests on commits to main branch Dec 3, 2025
Copilot AI requested a review from geelen December 3, 2025 04:59
@geelen geelen marked this pull request as ready for review December 3, 2025 05:00
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 3, 2025

Open in StackBlitz

npx https://pkg.pr.new/mcp-remote@198

commit: f82a067

@geelen
Copy link
Owner

geelen commented Dec 3, 2025

@copilot fix the build pls\

Copy link
Contributor Author

Copilot AI commented Dec 3, 2025

@copilot fix the build pls\

Fixed in f82a067. The build was failing because @modelcontextprotocol/[email protected] references @cfworker/json-schema types that aren't installed. Added skipLibCheck: true to tsconfig.json to skip type checking in library .d.ts files.

@geelen geelen merged commit 85508f2 into main Dec 3, 2025
8 checks passed
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