Skip to content

Develop#151

Merged
KOU050223 merged 67 commits intomainfrom
develop
Aug 16, 2025
Merged

Develop#151
KOU050223 merged 67 commits intomainfrom
develop

Conversation

@KOU050223
Copy link
Owner

いろいろ

AI レビューガイドライン(クリックして展開)

Copilotレビューガイドライン

レビュー方針

必ず日本語でレビューコメントを記載すること

レビューレベル

レビューコメントには、以下のプレフィックスを必ず付けること:

  • [ask] - 回答必須:実装の意図や設計について確認が必要
  • [must] - 修正必須:セキュリティ、バグ、重大な設計問題
  • [imo] - 修正任意:パフォーマンス改善、可読性向上
  • [nits] - 細かな修正:typo、フォーマット、命名規則など
  • [next] - 今後の改善:今回のPRでは対応不要だが将来的な改善点
  • [good] - 良い実装:評価すべき実装や設計
  • [suggestion] - 提案:より良い実装方法の提案

チェック項目

コード品質

  • 命名規則の統一性(変数、関数、クラス名がキャメルケースやスネークケースで適切)
  • コードの可読性と保守性
  • DRY原則の遵守(重複コードの排除)
  • SOLID原則の適用
  • 適切なコメントの記載

TypeScript/JavaScript固有

  • any型の使用を避ける
  • 適切な型定義
  • 非同期処理の適切な実装
  • ES6+の適切な活用
  • エラーハンドリングの実装

セキュリティ

  • 入力値検証の実装
  • SQLインジェクション対策
  • XSS対策
  • 機密情報のハードコード回避
  • 適切な権限制御

パフォーマンス

  • 効率的なアルゴリズムの選択
  • メモリリークの回避
  • 不要な処理の排除
  • データベースクエリの最適化

テスト (しばらくは不要)

  • 単体テストの網羅性
  • テストケースの妥当性
  • モックの適切な使用
  • エッジケースのテスト

React・Next.js固有(該当する場合)

  • コンポーネントの適切な分割
  • 状態管理の最適化
  • 不要な再レンダリングの防止
  • Hooksの適切な使用

Git/GitHub関連

  • コミットメッセージの明確性
  • 適切なブランチ戦略
  • コンフリクトの解決
  • PRの説明の充実

レビュー観点の優先順位

  1. セキュリティ問題 - 最優先で指摘
  2. バグの可能性 - 高優先度で指摘
  3. パフォーマンス問題 - 重要度に応じて指摘
  4. コード品質 - 建設的にフィードバック
  5. スタイル・フォーマット - 必要に応じて指摘

レビュー時の注意事項

  • 修正提案を行う際は、具体的なコード例を提示する
  • 問題の理由と影響範囲を明確に説明する
  • 良い実装についても積極的に評価する
  • 建設的で学習につながるフィードバックを心がける
  • チーム全体のスキル向上につながる指摘を行う

除外項目

以下の場合は簡潔な指摘または省略:

  • 自動フォーマット可能な問題
  • 明らかなタイポ
  • 既存コードとの整合性のみの問題

チェックリスト

  • ブランチは最新のmain/developから作成されている
  • コンフリクトが解決されている
  • CIが通っている
  • 必要なドキュメントが更新されている

KOU050223 and others added 30 commits August 10, 2025 02:07
snsをリファクタリングしました
refactor: デスクトップ向けの不要なプラットフォームコードを削除
* refactor: テスト系機能をFeatureFlagで制御するように改修

- FeatureFlag管理システムを新規作成
- テストログイン機能をFeatureFlagで本番環境では無効化
- アニメテストデータ作成機能をFeatureFlagで制御
- デバッグログもFeatureFlagで制御

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* wip

* fix: デザインを戻した

* feat: デバッグログとテストログイン機能を追加

---------

Co-authored-by: Claude <noreply@anthropic.com>
FriendWatchListPageを同様に分離しました
feat: CI/CDワークフローとフォーマッター設定を追加 (issue #103)
* feat: 新しいアイコンとロゴ画像を追加

* feat: アプリ名と説明を日本語に更新
* feat: アニメ画像取得サービスを追加し、UIを更新

* feat: タイムアウト設定を定数化し、ログ出力を改善
* feat: アニメ画像取得サービスを追加し、UIを更新

* feat: タイムアウト設定を定数化し、ログ出力を改善

* feat: アニメ画像表示のレイアウトを改善
AI分析機能追加
まだターミナルのみ
セキュリティ強化
分析ボタンを追加し、メッセージをリザルト画面に表示できるようにしました

友達追加とともにメッセージを表示しようとするとmessageがまだ返答されず空欄になる問題を解決しました
* feat: アニメ画像表示のレイアウトを改善

* feat: アニメリストヘッダーからアクションボタンを削除

* feat: フローティングアクションボタンのスタイルを改善

* fix: constをつけパフォーマンスに考慮
* feat: アニメリストに削除機能を追加

- 個別アニメの削除ボタンを実装
- 削除確認ダイアログを追加
- 削除成功時のスナックバー通知を実装
- プロフィール画面のエラー修正

* fix: 不要な生成ファイルを削除し、.gitignoreに環境変数ファイルを追加

- 不要な生成ファイル(macOS、Windows、Linux関連)を削除

- anime_list_view_model.dartのコードを微修正
KOU050223 and others added 25 commits August 16, 2025 19:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ault-sort-year-desc

add: アプリを持っていない人でもQRコードを読み込めば、相手のユーザーの視聴履歴を閲覧できるサイトを実装。
…i-image

Revert "feat: 名刺画像機能の実装 (#126)"
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
fix: QRデータ生成の統一とFirestoreセキュリティルール修正
* feat: QRコードからのユーザーID抽出ロジックを更新

* feat: QRコードからユーザーIDを抽出するロジックを改善

* feat: QRコードからユーザーIDを抽出するテストを追加

* feat: サブコレクションの読み取りルールを改善
* add functions

* Add Geminiのコードを複製

デプロイ用にとりあえず
実装やコード削除はまだです

* update function

* change functionからたたく

ローカルだとうまくいかない

* coss改変

* fix cors

* fix APIkeyの収納

* fix json
@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 310192a):

https://animeishi-73560--pr151-develop-uft61d0i.web.app

(expires Sat, 23 Aug 2025 21:20:52 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f1f494a67df8ccbb2232f73b6ccab4934a51e505

Copy link
Collaborator

@NazonoKansatugata NazonoKansatugata left a comment

Choose a reason for hiding this comment

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

でかすぎるっぴ

@KOU050223 KOU050223 merged commit f1ee6ad into main Aug 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants