Skip to content

[WIP] chore: add frontend code review skill for cursor#21379

Draft
koji wants to merge 8 commits into
edgefrom
chore_update-cursor-skill-for-code-review
Draft

[WIP] chore: add frontend code review skill for cursor#21379
koji wants to merge 8 commits into
edgefrom
chore_update-cursor-skill-for-code-review

Conversation

@koji

@koji koji commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Overview

add frontend code review skill for cursor

close AUTH-

Test Plan and Hands on Testing

Changelog

Review requests

Risk assessment

low

add frontend code review skill for cursor

close AUTH-

Copilot AI 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.

Pull request overview

Adds a new Cursor skill (grill-frontend-code) intended to guide frontend code reviews with a short checklist and a prescribed review output format.

Changes:

  • Introduces a new skill definition file with review scope, perspectives, and expected output format.
  • Adds a frontend review reference checklist covering TypeScript and React checks (incl. React Doctor command).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 9 comments.

File Description
.cursor/skills/grill-frontend-code/SKILL.md Defines the new Cursor skill workflow, review perspectives, and output format.
.cursor/skills/grill-frontend-code/reference-check-list.md Provides the reference checklist and example snippets for reviewers to follow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .cursor/skills/grill-frontend-code/SKILL.md Outdated
Comment thread .cursor/skills/grill-frontend-code/SKILL.md
Comment thread .cursor/skills/grill-frontend-code/reference-check-list.md Outdated
Comment thread .cursor/skills/grill-frontend-code/reference-check-list.md Outdated
Comment thread .cursor/skills/grill-frontend-code/reference-check-list.md Outdated
Comment thread .cursor/skills/grill-frontend-code/SKILL.md Outdated
Comment thread .cursor/skills/grill-frontend-code/reference-check-list.md Outdated
Comment thread .cursor/skills/grill-frontend-code/reference-check-list.md
Comment thread .cursor/skills/grill-frontend-code/SKILL.md Outdated
koji and others added 7 commits April 28, 2026 21:03
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@koji

koji commented Apr 29, 2026

Copy link
Copy Markdown
Contributor Author
  • update checklist
  • [ ]

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.

2 participants