Add community skills search, pagination, and an "Add your skill" section#34
Open
kaankacar wants to merge 4 commits into
Open
Add community skills search, pagination, and an "Add your skill" section#34kaankacar wants to merge 4 commits into
kaankacar wants to merge 4 commits into
Conversation
- CommunitySearch client island filters the server-rendered community cards by title or any part of the description, and paginates them at 8 per page. Cards stay in the static HTML for agents and crawlers. - "Add your skill" block explains how to get listed via a PR against ECOSYSTEM_CARDS in site/src/data/skills.ts, entry format inlined. - The llms.txt community blurb now tells agents how to submit a skill.
3 tasks
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR enhances the landing page’s Community skills section to better scale with more entries by adding a client-side search + pagination island while keeping community card content server-rendered in the static HTML, and it documents a clear contribution path via an “Add your skill” section and updated llms.txt blurb.
Changes:
- Added
CommunitySearchclient island to provide case-insensitive title/description search and pagination (8 cards/page) over pre-rendered community cards. - Added an “Add your skill” contribution block linking to GitHub’s edit flow and showing the required
ECOSYSTEM_CARDSentry format. - Updated
/llms.txtgeneration and site docs to instruct contributors how to submit new community skills.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| site/src/app/styles.scss | Adds styling for community search input, per-card wrappers, pagination, and the “Add your skill” block. |
| site/src/app/page.tsx | Wraps community cards in CommunitySearch and adds the “Add your skill” section/snippet and contribution links. |
| site/src/app/_components/CommunitySearch.tsx | New client component implementing search + pagination by toggling wrapper visibility around server-rendered children. |
| site/scripts/generate-llms-txt.mjs | Extends the Community Built blurb to include contribution instructions while still listing all community skills. |
| site/CLAUDE.md | Documents the new CommunitySearch island and notes ecosystem entries require no extra wiring. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+69
to
+78
| <div className="SkillsLanding__ecosystemGrid"> | ||
| {Children.map(children, (child, index) => ( | ||
| <div | ||
| className="SkillsLanding__communityItem" | ||
| data-hidden={!visible.has(index)} | ||
| > | ||
| {child} | ||
| </div> | ||
| ))} | ||
| </div> |
Nothing reads it: the community grid renders outside SkillsFilter, the search haystack is title + description only, and llms.txt lists the community section flat. Requiring contributors to copy a field that does nothing (and silently accepts any FilterType value) just invites confusion, so remove it from the type, the four entries, the on-site snippet, and the docs. The standards skill's "Ecosystem" category in SKILL_CARD_SOURCES stays; that one drives the main grid's filter tab.
- Search input now spans exactly one grid column so it lines up with the card below it (full width on narrow screens) - "Showing x of y skills" line renders under the community grid, above the Add your skill block, and doubles as the aria-live status for search results (replaces the screen-reader-only span)
|
|
I noticed that |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #33.
CommunitySearchclient island for the community section: search matches the card title and any part of the description (searching "DEX" surfaces the Soroswap skill), with pagination at 8 cards per page. Cards are still server-rendered and passed in as children, same slot pattern asSkillsFilter, so every entry stays in the static HTML and nothing changes for agents and crawlers.ECOSYSTEM_CARDSinsite/src/data/skills.ts. The entry format is inlined and the link goes to GitHub's edit view, which starts the fork + PR flow for non-collaborators.llms.txtcommunity blurb now ends with the same instruction, so agents reading/llms.txtknow how to submit too. Every community skill keeps appearing there regardless of what the UI shows.With only 4 community cards today nothing visibly changes except the search box and the new section; pagination kicks in past 8 cards.
pnpm lint,pnpm lint:ts, andpnpm buildpass, andout/index.htmlwas checked to confirm all card data remains in the server HTML.