add: アプリを持っていない人でもQRコードを読み込めば、相手のユーザーの視聴履歴を閲覧できるサイトを実装。 by AliceWonerfulWorld · Pull Request #136 · KOU050223/Animeishi

AliceWonerfulWorld · 2025-08-15T17:44:22Z

Pull Request

概要

変更内容

テスト

単体テスト実行済み
結合テスト実行済み
手動テスト実行済み

スクリーンショット（該当する場合）

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

Copilotレビューガイドライン

レビュー方針

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

レビューレベル

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

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

チェック項目

コード品質

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

TypeScript/JavaScript固有

セキュリティ

パフォーマンス

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

テスト（しばらくは不要）

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

React・Next.js固有（該当する場合）

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

Git/GitHub関連

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

レビュー観点の優先順位

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

レビュー時の注意事項

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

除外項目

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

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

チェックリスト

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

Copilot

Pull Request Overview

QRコードを使用してアプリ非ユーザーでもユーザーの視聴履歴を閲覧できるWebサイトを実装するPull Requestです。

Firebase Hostingを使用したWebビューアーの追加
QRコードからWebサイトへのリダイレクト機能
ユーザープロフィールとアニメ視聴履歴の表示機能

Reviewed Changes

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

Show a summary per file

File	Description
viewer/public/user.html	ユーザープロフィール表示用のHTMLページ
viewer/public/index.html	ランディングページのHTML
viewer/public/assets/js/firebase-config.js	Firebase設定ファイル
viewer/public/assets/js/app.js	メインアプリケーションロジック
viewer/public/assets/css/style.css	レスポンシブデザインのCSS
viewer/firebase.json	Firebase Hostingの設定
viewer/README.md	プロジェクトのドキュメント
viewer/.firebaserc	Firebaseプロジェクト設定
lib/ui/home/view/home_page.dart	QRコード生成時のURL短縮機能追加
lib/ui/animes/view_model/anime_list_view_model.dart	アニメリストのデフォルトソート順変更

Copilot · 2025-08-15T17:45:20Z

viewer/public/assets/js/firebase-config.js

+firebase.initializeApp(firebaseConfig);
+
+// Firestore初期化
+const db = firebase.firestore();


FirebaseのAPIキーなどの機密情報がハードコードされています。環境変数や設定ファイルを使用して機密情報を管理し、リポジトリにコミットしないようにしてください。

Suggested change

const db = firebase.firestore();

// Firebase設定を外部ファイルから読み込む

// 必ず 'viewer/public/assets/js/firebase-config.json' を作成し、機密情報を管理してください。

// このファイルは .gitignore に追加し、リポジトリにコミットしないでください。

fetch('assets/js/firebase-config.json')

.then(response => {

if (!response.ok) {

throw new Error('Firebase設定ファイルが見つかりません。firebase-config.json を作成してください。');

}

return response.json();

})

.then(firebaseConfig => {

// Firebase初期化

firebase.initializeApp(firebaseConfig);

// Firestore初期化

window.db = firebase.firestore();

})

.catch(error => {

console.error('Firebaseの初期化に失敗しました:', error);

});

Copilot · 2025-08-15T17:45:20Z

viewer/public/assets/js/app.js

+    this.shortId = this.getUserIdFromUrl();
+
+    if (shortId) {
+        this.currentUserId = await this.getUserIdFromShortId(shortId);


変数 shortId が定義されていません。正しくは this.shortId であるべきです。

Suggested change

this.currentUserId = await this.getUserIdFromShortId(shortId);

if (this.shortId) {

this.currentUserId = await this.getUserIdFromShortId(this.shortId);

この修正入れましょう

Copilot · 2025-08-15T17:45:20Z

viewer/public/assets/js/app.js

+    this.shortId = this.getUserIdFromUrl();
+
+    if (shortId) {
+        this.currentUserId = await this.getUserIdFromShortId(shortId);


変数 shortId が定義されていません。正しくは this.shortId であるべきです。

Suggested change

this.currentUserId = await this.getUserIdFromShortId(shortId);

if (this.shortId) {

this.currentUserId = await this.getUserIdFromShortId(this.shortId);

Copilot · 2025-08-15T17:45:21Z

viewer/public/assets/js/app.js

+        const uid = doc.id;
+        if (uid.startsWith(shortId)) {
+          return uid;
+        }


全ユーザーのコレクションを取得してからフィルタリングするのは非効率的です。Firestoreのwhere句を使用して、より効率的なクエリを実装することを検討してください。

Suggested change

}

// Firestoreで短縮IDに一致するユーザーを検索（shortIdフィールドで検索）

const usersSnapshot = await db.collection('users').where('shortId', '==', shortId).get();

if (!usersSnapshot.empty) {

// 最初の一致したユーザーのIDを返す

return usersSnapshot.docs[0].id;

これはして欲しい

…ear-desc

KOU050223 · 2025-08-15T18:14:25Z

修正まとめ

ハードコーディングをやめる
shortIdにしなくても良いかも
- _generateShortIdの場所を移動する
- if (shortId) {をthis.shortIdに修正する

KOU050223

まとめた修正点を修正しましょう

viewer/public/assets/js/firebase-config.js

KOU050223 · 2025-08-15T18:03:04Z

viewer/public/assets/js/app.js

+    this.shortId = this.getUserIdFromUrl();
+
+    if (shortId) {
+        this.currentUserId = await this.getUserIdFromShortId(shortId);


この修正入れましょう

KOU050223 · 2025-08-15T18:09:55Z

viewer/public/assets/js/app.js

+  async getUserIdFromShortId(shortId) {
+    try {
+      // Firestoreで短縮IDに一致するユーザーを検索
+      const usersSnapshot = await db.collection('users').get();


全ユーザーデータをとってくるのをやめたいです

KOU050223 · 2025-08-15T18:12:25Z

lib/ui/home/view/home_page.dart

 class HomeTabPage extends StatefulWidget {
  const HomeTabPage({super.key});

+  String _generateShortId(String uid) {


この関数を下の_HomeTabPageStateに移動してください
ビルドが落ちています

…ttps://github.com/KOU050223/Animeishi into AliceWonerfulWorld/feature/default-sort-year-desc

github-actions · 2025-08-16T11:20:23Z

Visit the preview URL for this PR (updated for commit 6bf28eb):

https://animeishi-73560--pr136-alicewonerfulworld-f-nk0dnmbd.web.app

_{(expires Sat, 23 Aug 2025 14:14:43 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: f1f494a67df8ccbb2232f73b6ccab4934a51e505}

KOU050223 · 2025-08-16T11:22:17Z

.envが上がってきています

KOU050223 · 2025-08-16T11:28:25Z

結局shortを使用する感じですか？

…し、404ページを追加。

KOU050223

LGTM

AliceWonerfulWorld added 3 commits August 16, 2025 01:09

feat: アニメリストのソート順を年代順に変更し、昇順をKOU順に設定。

00bc88b

fix: コメントの修正

12a87cf

feat: ゲストモードで視聴履歴を閲覧できるサイトを実装。ユーザーUIDを短縮し、QRコード用のURLを生成する機能を追加

6ce35d2

AliceWonerfulWorld requested review from KOU050223, NazonoKansatugata and Copilot August 15, 2025 17:44

github-project-automation bot added this to アニ名刺Project Aug 15, 2025

Copilot AI reviewed Aug 15, 2025

View reviewed changes

Merge branch 'develop' into AliceWonerfulWorld/feature/default-sort-y…

c3883bf

…ear-desc

KOU050223 requested changes Aug 15, 2025

View reviewed changes

KOU050223 assigned AliceWonerfulWorld Aug 15, 2025

KOU050223 moved this to レビュー待ち in アニ名刺Project Aug 15, 2025

AliceWonerfulWorld added 2 commits August 16, 2025 20:11

feat: ユーザーIDの短縮生成関数をホームページに移動し、Firebase設定を環境変数に変更、その他微修正

20ba78c

Merge branch 'AliceWonerfulWorld/feature/default-sort-year-desc' of h…

2ebf5ca

…ttps://github.com/KOU050223/Animeishi into AliceWonerfulWorld/feature/default-sort-year-desc

feat: ユーザー個人データのアクセスルールを更新し、ゲストサイトのデプロイ手順を追加。Firebase設定を環境変数から直接指定に変更…

6bf28eb

…し、404ページを追加。

KOU050223 approved these changes Aug 16, 2025

View reviewed changes

KOU050223 merged commit 2e25cb2 into develop Aug 16, 2025
2 checks passed

github-project-automation bot moved this from レビュー待ち to Done in アニ名刺Project Aug 16, 2025

KOU050223 deleted the AliceWonerfulWorld/feature/default-sort-year-desc branch August 16, 2025 14:21

-const db = firebase.firestore();
+// Firebase設定を外部ファイルから読み込む
+// 必ず 'viewer/public/assets/js/firebase-config.json' を作成し、機密情報を管理してください。
+// このファイルは .gitignore に追加し、リポジトリにコミットしないでください。
+fetch('assets/js/firebase-config.json')
+  .then(response => {
+    if (!response.ok) {
+      throw new Error('Firebase設定ファイルが見つかりません。firebase-config.json を作成してください。');
+    }
+    return response.json();
+  })
+  .then(firebaseConfig => {
+    // Firebase初期化
+    firebase.initializeApp(firebaseConfig);
+    // Firestore初期化
+    window.db = firebase.firestore();
+  })
+  .catch(error => {
+    console.error('Firebaseの初期化に失敗しました:', error);
+  });

	this.currentUserId = await this.getUserIdFromShortId(shortId);
	if (this.shortId) {
	this.currentUserId = await this.getUserIdFromShortId(this.shortId);

-        }
+      // Firestoreで短縮IDに一致するユーザーを検索（shortIdフィールドで検索）
+      const usersSnapshot = await db.collection('users').where('shortId', '==', shortId).get();
+      if (!usersSnapshot.empty) {
+        // 最初の一致したユーザーのIDを返す
+        return usersSnapshot.docs[0].id;

Conversation

AliceWonerfulWorld commented Aug 15, 2025

Pull Request

概要

変更内容

関連Issue

テスト

スクリーンショット（該当する場合）

Copilotレビューガイドライン

レビュー方針

レビューレベル

チェック項目

コード品質

TypeScript/JavaScript固有

セキュリティ

パフォーマンス

テスト （しばらくは不要）

React・Next.js固有（該当する場合）

Git/GitHub関連

レビュー観点の優先順位

レビュー時の注意事項

除外項目

チェックリスト

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

KOU050223 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

KOU050223 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

KOU050223 commented Aug 15, 2025

Uh oh!

KOU050223 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

KOU050223 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

KOU050223 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

KOU050223 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Aug 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

KOU050223 commented Aug 16, 2025

Uh oh!

KOU050223 commented Aug 16, 2025

Uh oh!

KOU050223 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

テスト（しばらくは不要）

github-actions bot commented Aug 16, 2025 •

edited

Loading