Skip to content

Conversation

@nkoji21
Copy link
Owner

@nkoji21 nkoji21 commented Oct 1, 2025

やったこと

  • prettier -> biome に移行

  • tailwindcssのclassソートする機能も追加

  • CI用にチェック専用のスクリプトを追加

  • pnpm format pnpm lint で、リントエラーを解消

    • 主にこれが原因で、変更ファイル数が多い
    • 解消することで、影響が出そうな過剰なリントエラーに関しては、無効化コメントを書いて対応してる

なぜやったか

  • biomeのOSSを僕がしたいくて、biomeを使いたいから
  • eslint prettierの環境から、biomeのみにすることで環境が綺麗になる

Note

Replaces Prettier/ESLint with Biome and applies automated formatting plus minor a11y/typing/react fixes across the client.

  • Tooling:
    • Add client/biome.json; update package.json scripts to use Biome; remove Prettier/ESLint configs (.prettier*, eslint.config.mjs) and related deps.
  • Codebase-wide updates:
    • Reorder Tailwind classes and general formatting; adjust decorative elements to aria-hidden, remove unnecessary roles, add type="button", and improve list keys.
    • Small React/TS fixes: memoize callbacks, refine useEffect deps, use parseInt(..., 10), prefer indexOf over findIndex in comparisons, and improve error typing/logic (e.g., exponentiation operator).

Written by Cursor Bugbot for commit 2076626. This will update automatically on new commits. Configure here.

Copilot AI review requested due to automatic review settings October 1, 2025 14:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

フロントエンドのフォーマットツールをPrettier + ESLintからBiomeに移行する変更です。

  • ESLintとPrettierの設定ファイルを削除し、Biome設定を追加
  • package.jsonのスクリプトとdevDependenciesをBiome用に更新
  • TailwindCSSクラスの並び順やTypeScriptコードの改善を実施

Reviewed Changes

Copilot reviewed 24 out of 25 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
client/package.json Biome関連のスクリプトと依存関係に更新
client/biome.json Biomeの設定ファイルを新規追加
client/eslint.config.mjs ESLint設定ファイルを削除
client/.prettierrc Prettier設定ファイルを削除
client/.prettierignore Prettierの除外設定ファイルを削除
client/hooks/useWebSocket.ts error型をunknownに変更、指数計算を**演算子に変更
client/hooks/useTypingAnimation.ts useEffectの依存配列を修正、インポート順序を調整
client/hooks/api/useJoinRoom.ts インポート文の並び順をアルファベット順に変更
client/hooks/api/useCreateRoom.ts インポート文の並び順をアルファベット順に変更
client/components/*.tsx TailwindCSSクラスの並び順、aria属性、React import等を修正
client/app/room/**/*.tsx React/Hooks使用方法、key属性、TailwindCSSクラス順序等を改善

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@Taiga2022 Taiga2022 merged commit 6fedcc5 into main Oct 1, 2025
2 checks passed
@Taiga2022 Taiga2022 deleted the chore/front-biome branch October 1, 2025 15:01
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.

3 participants