Open
Conversation
…ved performance in repository and usecase files
Summary of ChangesHello @hikahana, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、バックエンドのGo言語バージョンを1.26にアップグレードし、Google Wireによる依存性注入を導入することで、システムの基盤を強化することを目的としています。これにより、コードの保守性と拡張性が大幅に向上し、開発効率の改善が期待されます。また、フロントエンドの主要ライブラリも更新され、全体的な技術スタックの健全性が保たれています。 Highlights
Changelog
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
このプルリクエストはGoのバージョンアップを目的としていますが、実際にはgoogle/wireを使用したDI導入や、コントローラからoapi-codegenベースのハンドラへの移行など、大規模なアーキテクチャリファクタリングが含まれています。さらに、予算、支出、購入申請・物品・報告、領収書、財源といった多くの機能がAPIから削除されています。これらの重要な変更がPRのタイトルや説明に記載されていないのは、コードレビューやプロジェクトの履歴管理において重大な懸念点です。リファクタリング自体はアーキテクチャを改善する良いものですが、ドキュメント不足は問題です。また、削除された機能のコード（リポジトリ、ユースケース）がデッドコードとして多く残っており、保守性に影響を与えるため、今後の対応が望まれます。
| time.Local = time.FixedZone("JST", 9*60*60) | ||
|
|
||
| client, echo := di.InitializeServer() | ||
| serverComponents, err := di.InitializeServer() |
There was a problem hiding this comment.
新しいDI設定 (di.InitializeServer()) では、このPRでAPIから削除されたbudgets、expenses、purchase_ordersなどの機能に関連するいくつかのユースケースとリポジトリが使用されなくなりました。しかし、対応するファイル（例: budget_usecase.go, budget_repository.go）はコードベースに残っています。これは大量のデッドコードとなります。保守性を向上させ、コードをクリーンに保つために、これらの未使用ファイルを後続のPRで削除することを強く推奨します。
| docker compose build | ||
| docker compose run --rm view npm install | ||
| make fix-perms | ||
| docker compose run --rm view pnpm install |
There was a problem hiding this comment.
pnpm installがbuildターゲットで実行されていますが、compose.ymlのviewサービスのcommandでも実行されるため、冗長になっています。make setupフローを使用すると、このコマンドが2回実行されることになります。セットアッププロセスを高速化し、冗長性をなくすために、この行を削除することをお勧めします。compose.ymlのcommandがコンテナ起動時にインストールを処理します。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
対応Issue
Go 1.26へのアップデート対応
概要
prod.api.Dockerfile) のGoバージョンを1.25.5から1.26-alpine3.23に更新api/dev.Dockerfile) のGoバージョンを1.25.5から1.26-alpine3.23に更新go fixを使用するためのターゲットを追加go fixを使用してコードの自動最適化を実施（strings.Builder等の性能改善）画面スクリーンショット等
N/A（インフラ・環境設定の変更のため）
テスト項目
make run-dbでデータベースが正常に起動することmake runでアプリケーションが正常に起動することcurl -s http://localhost:1323/でヘルスチェックが正常に応答することmake run-testでテストが正常に通ること備考
go fixにより、コードの自動最適化が実施されています