docs: mark Python MCP Client transport as supported#440
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Mark Python MCP Client as supported (MCP SDK transport) following tempoxyz/pympp#96 which adds the McpClient wrapper.
03fddc4 to
d074906
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 03fddc4fbf
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
src/components/ServicesPage.tsx
Outdated
| .svc-icon-img { | ||
| width: 28px; height: 28px; border-radius: 6px; | ||
| display: block; object-fit: contain; | ||
| background: light-dark(#e8e8ec, #2a2a2a); | ||
| filter: light-dark(invert(1), none); |
There was a problem hiding this comment.
Define
svc-icon-img outside the tablet media query
On screens wider than 1400px, this is the only rule that styles svc-icon-img, so the <img> in ServiceIcon loses the old object-fit/background/inversion treatment in the default desktop table. That matters because scripts/sync-logos.ts fetches greyscale theme=dark PNGs; many of those logos are white-on-transparent, so they become hard to see in light mode on wide desktops.
Useful? React with 👍 / 👎.
src/components/ServiceDiscovery.tsx
Outdated
| background: light-dark(#e8e8ec, #2a2a2a); | ||
| filter: light-dark(invert(1), none); |
There was a problem hiding this comment.
Use a valid
filter value for discovery icons
light-dark() only resolves color/image values, but filter accepts none, url(), or filter functions. Browsers therefore drop filter: light-dark(invert(1), none) here, so the discovery cards no longer invert white monochrome logos in light mode. The previous JS path handled that case explicitly, so this change makes a subset of service icons nearly invisible on the light theme.
Useful? React with 👍 / 👎.
Updates the SDK feature table — Python MCP Client changes from
—toMCP SDK transport.Follows tempoxyz/pympp#96 which adds the
McpClientwrapper.