Skip to content

Add Skills Hub to Directories section#199

Open
charlie-cao wants to merge 1 commit intoPatrickJS:mainfrom
charlie-cao:add-skills-hub-directory
Open

Add Skills Hub to Directories section#199
charlie-cao wants to merge 1 commit intoPatrickJS:mainfrom
charlie-cao:add-skills-hub-directory

Conversation

@charlie-cao
Copy link

@charlie-cao charlie-cao commented Mar 20, 2026

What this adds

Adds Skills Hub to the Directories section.

Skills Hub is an open directory of community-rated AI coding skill packs for GitHub Copilot, Cursor, and Claude — with ratings, version history, and one-click copy.

URL: https://skillshub.io/skills

Why it fits here

  • Directly complements the .cursorrules files listed in this repo
  • Multi-tool support: Copilot + Cursor + Claude (not just Cursor)
  • Community ratings and reviews (quality signal, not just a flat list)
  • Version history with rollback — skills evolve without breaking users
  • Free and open

Disclosure: I'm part of the Skills Hub team. Happy to adjust the description if needed.

Summary by CodeRabbit

  • Documentation
    • Revised sponsorship descriptions for clarity.
    • Enhanced Knative integration information in related entries.
    • Added new Skills Hub link to the Directories section.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 20, 2026

📝 Walkthrough

Walkthrough

README.md updated with minor content changes: "Unblocked MCP" sponsorship headings clarified, Knative integration entry expanded to include "Knative" in the services list, and a new "Skills Hub" directory link added with accompanying description.

Changes

Cohort / File(s) Summary
Documentation
README.md
Updated sponsorship headings for clarity, expanded Knative integration description to include "Knative" alongside other services, and added new "Skills Hub" link under Directories section with descriptive text.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Poem

🐰 A hop through the docs, what a treat!
New directories to make it complete,
Skills Hub joins the line,
Knative shines divine,
Our readme's now perfectly neat! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding Skills Hub to the Directories section. It is concise, specific, and clearly summarizes the primary modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can approve the review once all CodeRabbit's comments are resolved.

Enable the reviews.request_changes_workflow setting to automatically approve the review once all CodeRabbit's comments are resolved.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (2)
README.md (2)

292-294: Inconsistent formatting in the Directories section.

The "Skills Hub" entry includes a description, while the existing "CursorList" and "CursorDirectory" entries do not. This creates formatting inconsistency within the Directories section.

Consider either:

  1. Adding brief descriptions to the existing directory entries, or
  2. Removing the description from Skills Hub to match the existing format
Option 1: Add descriptions to maintain consistency (recommended)
-- [CursorList](https://cursorlist.com)
-- [CursorDirectory](https://cursor.directory/)
+- [CursorList](https://cursorlist.com) — Curated list of cursor rules and configurations.
+- [CursorDirectory](https://cursor.directory/) — Directory of cursor rules for various frameworks and tools.
 - [Skills Hub](https://skillshub.io/skills) — Community-rated AI coding skill packs for Copilot, Cursor & Claude. Ratings, version history, and one-click copy.

As per coding guidelines, use consistent formatting for list items in the README.md file.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 292 - 294, The Directories list in README.md is
inconsistent: the "Skills Hub" item contains a descriptive trailing clause while
"CursorList" and "CursorDirectory" do not; update the list for consistency by
either adding brief descriptions to "CursorList" and "CursorDirectory" to match
"Skills Hub" or removing the description from "Skills Hub" so all three entries
follow the same format — locate the list entries containing the strings
"CursorList", "CursorDirectory", and "Skills Hub" and make the chosen change so
each list item uses the same style.

137-137: Redundant mention of "Knative" in the description.

The description reads "Knative development with Knative, Istio..." which redundantly mentions "Knative" twice. Other entries in the README follow the pattern of listing only complementary technologies in the integration list, not the primary technology itself.

Suggested rewording
-- [Knative (Istio, Typesense, GPU)](./rules/knative-istio-typesense-gpu-cursorrules-prompt-fil/.cursorrules) - Cursor rules for Knative development with Knative, Istio, Typesense, and GPU integration.
+- [Knative (Istio, Typesense, GPU)](./rules/knative-istio-typesense-gpu-cursorrules-prompt-fil/.cursorrules) - Cursor rules for Knative development with Istio, Typesense, and GPU integration.

As per coding guidelines, use clear and concise language in all documentation files.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 137, The README entry currently repeats "Knative" in its
description; update the line for the link "[Knative (Istio, Typesense, GPU)]" so
the description lists only complementary technologies (e.g., change "Cursor
rules for Knative development with Knative, Istio, Typesense, and GPU
integration." to "Cursor rules for Knative development with Istio, Typesense,
and GPU integration.")—edit the markdown entry referencing .cursorrules to
remove the redundant "Knative" from the description.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@README.md`:
- Around line 292-294: The Directories list in README.md is inconsistent: the
"Skills Hub" item contains a descriptive trailing clause while "CursorList" and
"CursorDirectory" do not; update the list for consistency by either adding brief
descriptions to "CursorList" and "CursorDirectory" to match "Skills Hub" or
removing the description from "Skills Hub" so all three entries follow the same
format — locate the list entries containing the strings "CursorList",
"CursorDirectory", and "Skills Hub" and make the chosen change so each list item
uses the same style.
- Line 137: The README entry currently repeats "Knative" in its description;
update the line for the link "[Knative (Istio, Typesense, GPU)]" so the
description lists only complementary technologies (e.g., change "Cursor rules
for Knative development with Knative, Istio, Typesense, and GPU integration." to
"Cursor rules for Knative development with Istio, Typesense, and GPU
integration.")—edit the markdown entry referencing .cursorrules to remove the
redundant "Knative" from the description.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6e6ddb3a-04c5-46b5-9817-cf7ee51ea799

📥 Commits

Reviewing files that changed from the base of the PR and between fc2ce04 and 91fa09d.

📒 Files selected for processing (1)
  • README.md

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.

1 participant