Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

基本的なステージング機能の実装 #27

Merged
merged 5 commits into from
Oct 24, 2024
Merged

Conversation

kaitoyama
Copy link
Contributor

@kaitoyama kaitoyama commented Sep 1, 2024

User description

  • packageのURLをtraPtitechに変更
  • 環境変数STAGING_CHANNEL_IDで指定されたチャンネルからのリクエストに対して別の処理をする部分を実装

PR Type

enhancement, bug_fix


Description

  • モジュール名をtraPtitechに変更し、リポジトリの一貫性を確保。
  • ステージングチャンネルのリクエストを処理する機能を追加し、特定のチャンネルでの動作をカスタマイズ。
  • エラーハンドリングを追加し、ステージング機能の安定性を向上。
  • READMEの画像URLを更新し、ドキュメントの正確性を維持。

Changes walkthrough 📝

Relevant files
Enhancement
gpt.go
モジュール名の更新                                                                                               

internal/gpt/gpt.go

  • モジュール名をtraPtitechに変更
+2/-2     
MessageReceived.go
ステージング機能とエラーハンドリングの実装                                                                       

internal/handler/MessageReceived.go

  • ステージングチャンネルのリクエストを処理する機能を追加
  • エラーハンドリングを追加
+24/-2   
OnDirectMessageCreated.go
モジュール名の更新                                                                                               

internal/handler/OnDirectMessageCreated.go

  • モジュール名をtraPtitechに変更
+1/-1     
OnMessageCreated.go
モジュール名の更新                                                                                               

internal/handler/OnMessageCreated.go

  • モジュール名をtraPtitechに変更
+1/-1     
handler.go
モジュール名の更新                                                                                               

internal/handler/handler.go

  • モジュール名をtraPtitechに変更
+1/-1     
main.go
モジュール名の更新                                                                                               

main.go

  • モジュール名をtraPtitechに変更
+6/-6     
go.mod
モジュール名の更新                                                                                               

go.mod

  • モジュール名をtraPtitechに変更
+1/-1     
Documentation
README.md
画像URLの更新                                                                                                 

README.md

  • 画像URLをtraPtitechに変更
+1/-1     

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

@kaitoyama kaitoyama self-assigned this Sep 1, 2024
@kaitoyama kaitoyama marked this pull request as ready for review September 27, 2024 13:10
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review

エラーハンドリング
エラーが発生した場合にログを出力するだけでなく、適切なエラーハンドリングを行うべきです。現在の実装では、ステージング機能が有効な場合にエラーが発生しても、それ以上の処理が行われません。

環境変数の取得
os.LookupEnvを使用して環境変数STAGING_CHANNEL_IDを取得していますが、この環境変数が設定されていない場合のデフォルト動作が定義されていません。環境変数が存在しない場合の処理を追加することをお勧めします。

Copy link

@nokhnaton nokhnaton left a comment

Choose a reason for hiding this comment

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

一点だけコメントしました!
そこの挙動が想定通りなら他は問題なさそうです!

fmt.Println(err)
}

return

Choose a reason for hiding this comment

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

ステージングチャンネルだと普通のチャンネルにあるresetやchatの機能は使わないっていうことで大丈夫ですか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

とりあえず、このPRでは使わなくて、この後のRAGのPoCでは使うコード書いてます

Choose a reason for hiding this comment

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

なら良さそうです!

@kaitoyama kaitoyama merged commit 5b65fa2 into main Oct 24, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants