Add Emdash Skills to Community Resources#328
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility | ||
|
|
||
|
|
There was a problem hiding this comment.
The diff adds an extra blank line after the new entry, leaving two consecutive blank lines before ## Community Prompts. The existing section ends with a single blank line; this adds an unnecessary second one.
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility | |
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility | |
| ## Community Prompts |
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 53-55
Comment:
**Extra blank line introduced**
The diff adds an extra blank line after the new entry, leaving two consecutive blank lines before `## Community Prompts`. The existing section ends with a single blank line; this adds an unnecessary second one.
```suggestion
- [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility
## Community Prompts
```
How can I resolve this? If you propose a fix, please make it concise.
|
|
||
| - [windsurfrules by kinopeee](https://github.com/kinopeee/windsurfrules) | ||
| - [windsurfrules_by_kamusis](https://github.com/kamusis/windsurf_memories) | ||
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility |
There was a problem hiding this comment.
The PR description states this "Matches existing format," but the two existing Community Resources entries (windsurfrules by kinopeee, windsurfrules_by_kamusis) have no description text. Adding a - … description is inconsistent with the current pattern in this section.
Should descriptions be added uniformly to existing entries, or should this entry drop the description to match the current no-description convention?
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 53
Comment:
**Description absent from existing entries**
The PR description states this "Matches existing format," but the two existing Community Resources entries (`windsurfrules by kinopeee`, `windsurfrules_by_kamusis`) have no description text. Adding a ` - …` description is inconsistent with the current pattern in this section.
Should descriptions be added uniformly to existing entries, or should this entry drop the description to match the current no-description convention?
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
|
|
||
| - [windsurfrules by kinopeee](https://github.com/kinopeee/windsurfrules) | ||
| - [windsurfrules_by_kamusis](https://github.com/kamusis/windsurf_memories) | ||
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility |
There was a problem hiding this comment.
The linked repository is named claude-skills and the PR description references "18 agents" — this appears to be a skill set targeting Anthropic's Claude rather than Windsurf/Codeium. The entry does mention .windsurfrules compatibility, but it is worth confirming that the resource is primarily a Windsurf community resource rather than a general Claude/LLM tool.
Is this resource primarily built for Windsurf, or is Windsurf compatibility secondary to its Claude-focused purpose?
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 53
Comment:
**Repo name suggests Claude (Anthropic) focus, not Windsurf**
The linked repository is named `claude-skills` and the PR description references "18 agents" — this appears to be a skill set targeting Anthropic's Claude rather than Windsurf/Codeium. The entry does mention `.windsurfrules compatibility`, but it is worth confirming that the resource is primarily a Windsurf community resource rather than a general Claude/LLM tool.
Is this resource primarily built for Windsurf, or is Windsurf compatibility secondary to its Claude-focused purpose?
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
There was a problem hiding this comment.
🚩 Commit message uses non-standard type prefix
The commit message is Add Emdash Skills to Awesome Windsurf community resources. Per .windsurfrules and CONTRIBUTING.md, the commit type must be one of [add, update, remove, fix, meta, release] with format type: description. The commit uses Add (capitalized, no colon separator) rather than the required add: ... format. This would normally be caught by commitlint (commitlint.config.js), but if the hook was bypassed, it violates the repo conventions. This is noted as an analysis rather than a bug since it's a commit metadata issue rather than a file content issue in the diff.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
|
||
|
|
There was a problem hiding this comment.
🟡 Double blank line violates MD012 markdownlint rule
The PR introduces two consecutive blank lines (lines 54-55) between the new list entry and the ## Community Prompts heading. The repository's .markdownlint-cli2.jsonc:4 explicitly enables "MD012": true (no multiple consecutive blank lines), so this will fail the markdownlint CI check. The previous version of the file had only a single blank line separating the list from the next section heading.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
|
||
| - [windsurfrules by kinopeee](https://github.com/kinopeee/windsurfrules) | ||
| - [windsurfrules_by_kamusis](https://github.com/kamusis/windsurf_memories) | ||
| - [Emdash Skills by megabytespace](https://github.com/megabytespace/claude-skills) - 14-category autonomous product-building OS with 18 agents and .windsurfrules compatibility |
There was a problem hiding this comment.
🚩 Linked repository is Claude-focused, not Windsurf-specific
The new entry links to https://github.com/megabytespace/claude-skills — a repository with "claude" in its name — but is listed in the Windsurf community resources. The description claims ".windsurfrules compatibility," which may justify its inclusion, but a reviewer should verify the linked repo actually contains meaningful Windsurf integration rather than being a primarily Claude-oriented project with only tangential Windsurf relevance.
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
Entry format
Matches existing format:
- [name by author](url) - descriptionGreptile Summary
This PR adds a single entry — Emdash Skills by megabytespace — to the Community Resources section of the README. All findings are P2 style/clarification items around a trailing blank line, description consistency, and the resource's primary focus (Claude vs. Windsurf).
Confidence Score: 5/5
Safe to merge; all findings are minor style/clarification concerns with no functional impact.
Only P2 findings present — a superfluous blank line, a description that differs from existing entry conventions, and a relevance question. None of these block merging.
README.md — minor formatting and relevance clarification needed.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[README.md Community Resources] --> B[windsurfrules by kinopeee] A --> C[windsurfrules_by_kamusis] A --> D["Emdash Skills by megabytespace NEW"] D --> E[github.com/megabytespace/claude-skills]Prompt To Fix All With AI
Reviews (1): Last reviewed commit: "Add Emdash Skills to Awesome Windsurf co..." | Re-trigger Greptile