feat:新規サークル作成エンドポイント #3

Mikuto0831 · 2025-12-12T03:10:33Z

やったこと\n- サービス作成を行うロジックの追加\n- 認証認可を使用した作成できるユーザの絞込

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Pull request overview

Copilot reviewed 17 out of 18 changed files in this pull request and generated 18 comments.

Comments suppressed due to low confidence (4)

backend/app/services/circle.py:2

Import of 'UUID' is not used.

from uuid import UUID

backend/app/core/security.py:2

Import of 'asyncio' is not used.

import asyncio

backend/app/core/security.py:3

Import of 'json' is not used.

import json

backend/app/core/security.py:4

Import of 'lru_cache' is not used.

from functools import lru_cache

Copilot · 2025-12-12T04:15:31Z

backend/app/api/v1/endpoints/circles.py

+    try:
+        user_info = await get_current_user(authorization)
+    except HTTPException as e:
+        raise e


The try-except block that re-raises the same HTTPException is redundant. The exception will automatically propagate without this wrapper. Consider removing this unnecessary error handling or adding meaningful additional handling if needed.

Suggested change

try:

user_info = await get_current_user(authorization)

except HTTPException as e:

raise e

user_info = await get_current_user(authorization)

Copilot · 2025-12-12T04:15:31Z

backend/app/services/circle.py

+        is_published=False,
+    )
+    session.add(new_circle)
+    await session.flush()  # ID を生成するため flush (commit ではなく)


The comment "ID を生成するため flush (commit ではなく)" is helpful, but consider expanding it to explain why flush is preferred over commit here (e.g., "トランザクション全体をコミットせずに ID を生成するため flush を使用").

Suggested change

await session.flush() # ID を生成するため flush (commit ではなく)

await session.flush() # トランザクション全体をコミットせずに ID を生成するため flush を使用（この後に他の操作もまとめてコミットするため）

Copilot · 2025-12-12T04:15:32Z

backend/app/services/circle.py

@@ -1,9 +1,12 @@
 """Circle service layer."""
+from uuid import UUID


The UUID import from the uuid module is not used in this file. The UUID type is used from type hints but is imported from elsewhere. This unused import should be removed.

Suggested change

from uuid import UUID

Copilot · 2025-12-12T04:15:32Z

backend/tests/conftest.py


-# テスト用データベースURL (環境変数で上書き可能)
-import os
+# .env ファイルを読み込む（プロジェクトルートから探す）


The .env file loading logic looks for the file in the backend directory. However, the root-level .env.example file (which was added in this PR) suggests there's also a project root .env file. Consider documenting which .env file takes precedence, or consolidating the configuration to avoid confusion about which environment file should be used for tests.

Suggested change

# .env ファイルを読み込む（プロジェクトルートから探す）

# .env ファイルの読み込みについて

# このテスト設定では、プロジェクトルート（backend ディレクトリの1つ上）の .env ファイルを読み込みます。

# バックエンド本体の .env 読み込みロジックが backend ディレクトリ内の .env を参照している場合、

# テストと本番で異なる .env ファイルが使われる可能性があります。

# どちらの .env ファイルが優先されるか、または .env ファイルの場所を統一することを推奨します。

Copilot · 2025-12-12T04:15:32Z

compose.dev.yml

+      KC_DB: postgres
+      KC_DB_URL: jdbc:postgresql://postgres:5432/circleportal
+      KC_DB_USERNAME: circleportal
+      KC_DB_PASSWORD: circleportal


The Keycloak admin credentials (admin/admin) are hardcoded in the docker-compose file. While this is acceptable for development, consider adding a comment warning that these credentials should never be used in production and should be changed immediately after initial setup, even in development environments with external access.

Suggested change

KC_DB_PASSWORD: circleportal

KC_DB_PASSWORD: circleportal

# 注意: 以下の管理者アカウント(admin/admin)は開発用途の初期値です。

# 本番環境では絶対に使用しないでください。

# また、外部からアクセス可能な開発環境の場合は、初期セットアップ後すぐに変更してください。

Copilot · 2025-12-12T04:15:35Z

docs/SETUP.md

+
+このテストは、実際の Keycloak から取得したトークンを使用してエンドポイントの認可を検証します。
+
+### 7. 開発サーバーの起動


Following the correction of section numbering, this section should be renumbered from 7 to 8 to maintain proper sequential ordering.

Suggested change

### 7. 開発サーバーの起動

### 8. 開発サーバーの起動

Copilot · 2025-12-12T04:15:35Z

backend/app/schemas/circle.py

+    class Config:
+        """Pydantic config."""
+
+        json_schema_extra = {
+            "example": {
+                "name": "LinuxClub",
+                "campus_id": 1,
+                "category": "culture",
+                "leader_email": "[email protected]",
+            }
+        }


The Pydantic Config class using json_schema_extra is the older Pydantic v2 style. While this still works, consider using model_config = ConfigDict(...) with the newer Pydantic v2 syntax for consistency with modern best practices. The same applies to other Config classes in this file.

Copilot · 2025-12-12T04:15:35Z

backend/app/core/security.py

+import asyncio
+import json


The asyncio and json imports are not used anywhere in this file. These unused imports should be removed.

Suggested change

import asyncio

import json

Copilot · 2025-12-12T04:15:36Z

backend/app/core/security.py

+            token,
+            public_key,
+            algorithms=["RS256"],
+            options={"verify_aud": False},  # audience 検証はスキップ (オプション)


The JWT verification has disabled audience verification with verify_aud: False. While this is marked as optional with a comment, skipping audience verification can be a security risk if the token is intended for a different service. Consider enabling audience verification and validating against the expected client_id, or document why this is intentionally disabled for your use case.

Suggested change

options={"verify_aud": False}, # audience 検証はスキップ (オプション)

audience=settings.keycloak_client_id, # audience 検証を有効化

Copilot · 2025-12-12T04:15:36Z

backend/app/core/security.py

+import httpx
+from fastapi import HTTPException, status
+from jose import JWTError, jwt, jwk
+from jose.exceptions import JWTClaimsError


Import of 'JWTClaimsError' is not used.

Suggested change

from jose.exceptions import JWTClaimsError

Mikuto0831 added 5 commits December 9, 2025 19:03

feat;要件定義にkeycloakの具体的記述を追加

2cd283e

feat:post /circle の作成

ef919a0

feat:keycloakによる認証テスト

e2e404b

compose.devを使用する

f84839a

chnage:要件定義書

3ef14b3

Mikuto0831 requested a review from Copilot December 12, 2025 03:14

Copilot started reviewing on behalf of Mikuto0831 December 12, 2025 03:14 View session

Copilot AI reviewed Dec 12, 2025

View reviewed changes

Mikuto0831 requested a review from Copilot December 12, 2025 04:08

Copilot started reviewing on behalf of Mikuto0831 December 12, 2025 04:08 View session

Copilot AI reviewed Dec 12, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat:新規サークル作成エンドポイント #3

feat:新規サークル作成エンドポイント #3

Uh oh!

Mikuto0831 commented Dec 12, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Copilot AI Dec 12, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	await session.flush() # ID を生成するため flush (commit ではなく)
	await session.flush() # トランザクション全体をコミットせずに ID を生成するため flush を使用（この後に他の操作もまとめてコミットするため）

		@@ -1,9 +1,12 @@
		"""Circle service layer."""
		from uuid import UUID

-# .env ファイルを読み込む（プロジェクトルートから探す）
+# .env ファイルの読み込みについて
+# このテスト設定では、プロジェクトルート（backend ディレクトリの1つ上）の .env ファイルを読み込みます。
+# バックエンド本体の .env 読み込みロジックが backend ディレクトリ内の .env を参照している場合、
+# テストと本番で異なる .env ファイルが使われる可能性があります。
+# どちらの .env ファイルが優先されるか、または .env ファイルの場所を統一することを推奨します。

-      KC_DB_PASSWORD: circleportal
+      KC_DB_PASSWORD: circleportal
+      # 注意: 以下の管理者アカウント(admin/admin)は開発用途の初期値です。
+      # 本番環境では絶対に使用しないでください。
+      # また、外部からアクセス可能な開発環境の場合は、初期セットアップ後すぐに変更してください。


		このテストは、実際の Keycloak から取得したトークンを使用してエンドポイントの認可を検証します。

		### 7. 開発サーバーの起動

	### 7. 開発サーバーの起動
	### 8. 開発サーバーの起動

	options={"verify_aud": False}, # audience 検証はスキップ (オプション)
	audience=settings.keycloak_client_id, # audience 検証を有効化

feat:新規サークル作成エンドポイント #3

Are you sure you want to change the base?

feat:新規サークル作成エンドポイント #3

Uh oh!

Conversation

Mikuto0831 commented Dec 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants