Skip to content

Sort community skills alphabetically - #111

Open
mariaelisaaraya wants to merge 1 commit into
stellar:mainfrom
mariaelisaaraya:sort-ecosystem-cards-alphabetically
Open

Sort community skills alphabetically#111
mariaelisaaraya wants to merge 1 commit into
stellar:mainfrom
mariaelisaaraya:sort-ecosystem-cards-alphabetically

Conversation

@mariaelisaaraya

Copy link
Copy Markdown
Contributor

Community skills were ordered by when each PR got merged, so new entries always landed on the last page. This sorts them alphabetically by title instead, so a skill's position depends on its name, not on when it was added.

Also updated CLAUDE.md so future entries get inserted in the right spot instead of appended at the end.

Test plan:

  • pnpm lint:ts
  • pnpm lint
  • pnpm build
  • Verified locally with pnpm dev

Community skills were ordered by when each PR got merged, so newer
entries always landed on the last page. This sorts them alphabetically
by title instead, so a skill's position depends on its name, not on
when it was added.

Also updated CLAUDE.md so new entries get inserted in the right spot
going forward, instead of appended at the end.
Copilot AI balanced review requested due to automatic review settings August 21, 2026 04:09
@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

👀 Picked this up — a review will follow shortly.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Sorts community skills alphabetically for stable display ordering.

Changes:

  • Reorders ECOSYSTEM_CARDS by title.
  • Documents the ordering requirement.
  • Cleans stray characters from a description.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
site/src/data/skills.ts Alphabetizes community skill cards.
site/CLAUDE.md Adds alphabetical insertion guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread site/CLAUDE.md
Comment on lines +88 to +90
other wiring needed. Insert the entry alphabetically by `title`
(case-insensitive) in `ECOSYSTEM_CARDS`, not appended at the end, so a
card's position reflects its name rather than merge order.

@devodii devodii left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

how do we make sure newer submissions follow this convention?

@drQedwards

Copy link
Copy Markdown
Contributor

you would make sure like in contract-metadata to denote in the documentation not to append your skills to end of the list.

you would code a flag check that would check to see if the developer of the new skill follow alphabetical instructions by if they put their ecosystem card code in the correct slot in the JSON or just put in the first or last position.

alphanumeric-check.yml I recommend adding

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.

5 participants