Skip to content

Conversation

@robertlestak
Copy link

No description provided.

@Laksh-star
Copy link
Owner

Thanks so much for the contribution and the detailed implementation! Before I review and merge, could you please:

  • Add a brief PR description and quick testing steps (e.g., how to run: npm run start:http and how to test /mcp).
  • Add tests covering tool handlers and the streamable HTTP session lifecycle (unit tests with mocked TMDB, or a small integration test).
  • Run npm audit and either fix or document any high/critical findings. Please confirm if package-lock.json was added intentionally.
  • Ensure CI runs a build (npm ci && npm run build) and tests — add a workflow if one's missing.

Once you've pushed those, I'll re-run CI and continue the review. If you have any questions or context to share (e.g., example requests), please include those too. Thanks again!

Repository owner deleted a comment from claude bot Nov 25, 2025
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