Skip to content

Serve the TypeScript-style MCP docs page on Python GET /. - #38

Merged
manish-wekan merged 3 commits into
developfrom
feature/python-landing-page
Sep 28, 2026
Merged

manish-wekan merged 3 commits into
developfrom
feature/python-landing-page

Conversation

@abhijitt-code

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the Python stub landing page with the TypeScript documentation page (connection setup + tool catalog).
  • Vendor this SDK via [tool.uv.sources] so NitroCloud uv sync and Open URL use the new page without an invalid file: Requires-Dist.
  • Ignore vendor/ and stop tracking bytecode.

Test plan

  • GET / on python-starter, python-pizzaz, and python-oauth shows Connection Setup and tools
  • NitroCloud Python uv build succeeds (no file: wheel metadata error)
  • Open URL after deploy shows the new page, not "This is not a website"

abhijitt-code and others added 2 commits September 23, 2026 18:05
Stateless middleware stripped Mcp-Session-Id then forwarded GET to an open stream, which hung GitHub Actions for hours. Reject the header first, cap pytest at 60s, and align a few era/name assertions with the current wire contract.

Co-authored-by: Cursor <cursoragent@cursor.com>

@manish-wekan manish-wekan left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Verdict: request changes

GET / matches the TypeScript docs page. I booted it locally: Connection Setup, tool catalog (echo, search_records), not the old stub. Landing tests passed; CI is green.

Blocker: nitrostack-py install vendors the SDK and rewrites requirements.txt to ./vendor/nitrostack, which skips uv sync. Vendor on init/pack only, and keep the dep name as nitrostack so uv still resolves via [tool.uv.sources]. Also drop the hardcoded 0.3.2 fallback.

@manish-wekan

Copy link
Copy Markdown
Collaborator

Local GET / on this branch. This is the TypeScript-style docs page (Connection Setup + tools), not the old stub.

GET / landing page

install rewrote requirements.txt to a path pin, so uv never resolved nitrostack through tool.uv.sources. Leave the dependency name as nitrostack and stop inventing a 0.3.2 version when metadata is missing.

Co-authored-by: Cursor <cursoragent@cursor.com>
@manish-wekan
manish-wekan merged commit 07d45cb into develop Sep 28, 2026
3 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