feat: レビューガイドラインをCursorRulesに追加してみる #5549
Draft
Conversation
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
commit: |
schktjm
commented
Apr 14, 2025
| * **【避けるべきこと】** `new String()`, `new Boolean()`, `new Number()` のように `new` キーワードをプリミティブ型の生成に使用すると、意図せずオブジェクトが生成され、予期せぬ挙動や混乱を招くため避けてください。 | ||
|
|
||
| * **コンポーネントのスタイリングと状態制御:** | ||
| * 【推奨】CSS クラスの適用は、CSS Modules やそれに類する仕組み（例: `styles` オブジェクト）を介して行ってください。 |
schktjm
commented
Apr 14, 2025
| * **【避けるべきこと】** コンポーネント内で直接 `className` 属性を文字列操作したり、クラス名の追加/削除によって `hidden` や `disabled` などの状態を制御したりすることは、外部 CSS の影響や意図しない副作用を招く可能性があるため避けてください。 | ||
|
|
||
| * **メモ化 (Memoization):** | ||
| * 【推奨】不要な再レンダリングを避けるため、`React.memo`, `useCallback`, `useMemo` を適切に使用し、パフォーマンスを最適化してください。 |
There was a problem hiding this comment.
過度な最適化はやり過ぎではという会話が入ってない
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.
関連URL
概要
過去3年間のPRをコメント数が多い順に90件集めてNotebookLMに取り込んで、ガイドラインを吐かせてみました。
https://notebooklm.google.com/notebook/cbdfddc8-1530-4eaf-8891-bf9824fcfff7?pli=1
実際に取り込んだPR一覧はこちら
https://gist.github.com/schktjm/f921219197762d7a7ebdaa9d38671228
変更内容
確認方法