Repository navigation
refactor(integrations): stop listing the community designation - #1252
samkujovich wants to merge 6 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
e5a4d97 to
0134a6a
Compare
samkujovich
changed the title
No toolkit is assigned the community type any more: Bright Data, Clickhouse, MongoDB and Postgres are Arcade toolkits. The type stays in the design system, so the index keeps handling it, but it no longer lists Community as a designation or offers it as a type filter. A ?type=community URL falls back to all types. Part of GRO-615
0134a6a to
7983382
Compare
| * The designations the index presents. `community` is still a design-system | ||
| * ToolkitType, but no toolkit is assigned it. | ||
| */ | ||
| export type ListedToolkitType = Exclude<ToolkitType, "community">; |
There was a problem hiding this comment.
We should probably remove community from the design system as well, so we don't need to do this here. We can merge that PR first and update this one if that is ok with you, @samkujovich
There was a problem hiding this comment.
Agreed. We're doing it your way. ArcadeAI/monorepo#5987 removes community from design-system (8.0.0) and from the monorepo copy of the docs site; no toolkit uses it since ArcadeAI/monorepo#5732. Once 8.0.0 is on npm, I'll cut this PR down to bumping design-system and deleting the community entries, without the ListedToolkitType plumbing.
~ 🤖 sam-i-am, Sam’s Agent
There was a problem hiding this comment.
I approved #5987. You can refactor this one when that one is merged!
| community: true, | ||
| auth: true, | ||
| } satisfies Record<ToolkitType, true>; | ||
| } satisfies Record<ListedToolkitType, true>; |
There was a problem hiding this comment.
We can avoid all these changes by renaming it in the DS.
There was a problem hiding this comment.
Same fix as the thread above: once ArcadeAI/monorepo#5987 publishes 8.0.0, these changes collapse to plain deletions.
~ 🤖 sam-i-am, Sam’s Agent
Part of GRO-615
What/why
The integrations index lists "Community" as one of five MCP server designations, offers it as a type filter, and labels Clickhouse, MongoDB, Postgres and Bright Data with it. Arcade builds and maintains all four, and ArcadeAI/monorepo#5732 reclassifies them as Arcade toolkits, so nothing is Community any more. After this PR the index lists four designations, and the type filter no longer offers Community. Once the nightly picks up #5732's metadata, the four cards show as Arcade Optimized and sort with the other optimized servers.
Stacked on #1249.
Codebase changes
communitystays a design-systemToolkitType, so docs keeps handling it: toolkit pages and cards still have a label for it. Only the index's list of designations and its type filter leave it out. They come from aListedToolkitTypethat excludes it.?type=communityURL is no longer a valid filter and falls back to all types, rather than filtering to an empty list.Proof
"type": "arcade", which the nightly writes after ArcadeAI/monorepo#5732. Not committed; the nightly owns those filescommunityis not a valid type filterfilter-params.test.tsnow rejects itpnpm test(901 passed),pnpm lint,tsc --noEmit,pnpm buildAdditional notes
Merge order: #1249, then this. It works with or without ArcadeAI/monorepo#5732; the cards change once that merges and the nightly runs.
~ 🤖 sam-i-am, Sam’s Agent
Note
Low Risk
UI and filter-param behavior on the integrations index only; no auth or data-path changes, with unit test coverage for rejected
communitytype URLs.Overview
Removes Community from the integrations MCP index: the intro now shows four designations (dynamic count), the type filter and designation grid use
ListedToolkitType/LISTED_TYPE_CONFIG(everything exceptcommunity), and?type=communityis ignored like other invalid types. FullToolkitTypeconfig (including community labels on cards/pages) is unchanged.Copy updates the Verified blurb to “built outside Arcade” and drops the Community description. CrewAI docs tweak a next-steps bullet to mention adding MCP servers by name.
Reviewed by Cursor Bugbot for commit a1af357. Bugbot is set up for automated code reviews on this repo. Configure here.