Skip to content

Conversation

@Taiga2022
Copy link
Collaborator

@Taiga2022 Taiga2022 commented Oct 1, 2025

📝 変更内容

何を変更したか

backend用のGithubActionsの作成

なぜ変更したか

CI環境の作成のため

🧪 テスト・確認項目

動作確認

  • プルリクエストにラベルを追加したか
  • アサインに自分を追加したか
  • ローカル環境で動作確認済み
  • 既存機能に影響がないことを確認
  • モバイル表示を確認(該当する場合)

📸 スクリーンショット(UI変更がある場合)

Before After
Before After

💡 補足事項


🔗 関連Issue

Closes #


Note

Adds a backend CI workflow (lint/format/build) and improves server websocket cleanup/error handling while fixing DynamoDB table config to honor env var.

  • CI:
    • Add Server CI workflow .github/workflows/backend-ci.yml for PRs touching server/**: setup Go 1.24.4, run golangci-lint, enforce gofmt, and build server/src/cmd/route/main.go.
  • Backend:
    • Database: NewDBConnection now honors DYNAMO_TABLE env with default "quiz" (removed hard-coded override).
    • WebSocket:
      • Client: safer deadline setting, error-checked close/write paths in ReadPump/WritePump.
      • Hub: refine unregisterClient flow (host detection, room close broadcast, conditional room deletion, cleanup of client send channels).
    • Quiz:
      • Ensure unique question selection and minor cleanup; remove unused getRandomQuestion.
    • Minor formatting/ordering adjustments (no functional impact).

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

@Taiga2022 Taiga2022 requested a review from nkoji21 October 1, 2025 14:14
@Taiga2022 Taiga2022 self-assigned this Oct 1, 2025
@Taiga2022 Taiga2022 added 🔧 環境 開発環境の構築や設定に関するタスク ⚙️ バックエンド サーバーやデータベースなど、裏側の処理に関するタスク labels Oct 1, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@nkoji21 nkoji21 self-requested a review October 1, 2025 14:53
@nkoji21 nkoji21 requested a review from Copilot October 1, 2025 14:54
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

Backend用のGitHub Actions CI環境を構築し、併せてWebSocketクライアントのエラーハンドリング改善やコードクリーンアップを実施しました。

  • Backend CI/CD パイプラインの構築(Go lint、format、build)
  • WebSocketクライアントの接続安定性向上とエラーハンドリング強化
  • DynamoDBテーブル名のハードコード削除と環境変数対応

Reviewed Changes

Copilot reviewed 9 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/backend-ci.yml Go 1.24.4を使用したCI/CDパイプライン新規作成
server/src/internal/feature/quiz/websocket/client.go WebSocket接続のエラーハンドリング強化とデッドライン設定の安全化
server/src/internal/feature/quiz/websocket/hub.go ホスト判定ロジックの改善と変数宣言スタイル統一
server/src/internal/database/database.go DynamoDBテーブル名のハードコード削除とimport順序修正
server/src/internal/feature/quiz/service/quizService.go 未使用関数の削除と空行整理
その他 import順序統一と不要な空行削除のマイナークリーンアップ

@nkoji21 nkoji21 marked this pull request as draft October 1, 2025 14:57
@nkoji21 nkoji21 marked this pull request as ready for review October 1, 2025 14:59
@Taiga2022 Taiga2022 merged commit 8ff693e into main Oct 1, 2025
1 of 2 checks passed
@Taiga2022 Taiga2022 deleted the feature/129/Actions_BE branch October 1, 2025 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️ バックエンド サーバーやデータベースなど、裏側の処理に関するタスク 🔧 環境 開発環境の構築や設定に関するタスク

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants