Skip to content
This repository was archived by the owner on Sep 23, 2025. It is now read-only.

feat: デフォルトのログレベルをINFOに、verboseオプション時のみDEBUGに (#144)#150

Merged
douhashi merged 1 commit intomainfrom
soba/144
Sep 20, 2025
Merged

feat: デフォルトのログレベルをINFOに、verboseオプション時のみDEBUGに (#144)#150
douhashi merged 1 commit intomainfrom
soba/144

Conversation

@douhashi
Copy link
Owner

実装完了

fixes #144

変更内容

  • デフォルトのログレベルをINFOに設定(既に実装済みだったため確認のみ)
  • bin/sobaで--verboseまたは-vフラグ指定時にDEBUGレベルに変更(既に実装済みだったため確認のみ)
  • Logger.newを使用していた箇所をSemanticLoggerに統一
    • WorkflowBlockingChecker
    • WorkflowIntegrityChecker
    • QueueingService
    • SlackNotifier
    • ClosedIssueWindowCleaner用のログ設定
  • 不要なrequire "logger"文を削除
  • ログフォーマットをSemanticLogger形式で統一

テスト結果

  • 単体テスト: ✅ パス
  • 統合テスト: ✅ パス
  • 全体テスト: ✅ パス(667 examples, 0 failures, 2 pending)

確認事項

  • 実装計画に沿った実装
  • テストカバレッジ確保
  • 既存機能への影響なし
  • Rubocopチェック通過

動作確認

  • 通常起動時(verboseオプションなし): INFOレベル以上のログのみ表示
  • soba start -vまたはsoba start --verbose起動時: DEBUGレベル以上のログも表示
  • すべてのログが統一されたSemanticLogger形式で出力されることを確認

- デフォルトのログレベルをINFOに設定(既に実装済み)
- bin/sobaで--verboseまたは-vフラグ指定時にDEBUGレベルに変更(既に実装済み)
- Logger.newを使用していた箇所をSemanticLoggerに統一
  - WorkflowBlockingChecker
  - WorkflowIntegrityChecker
  - QueueingService
  - SlackNotifier
  - ClosedIssueWindowCleaner用のログ設定
- ログフォーマットをSemanticLogger形式で統一
- 単体テストと統合テストを追加

fixes #144

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@douhashi
Copy link
Owner Author

レビュー結果

✅ 判定

  • 承認(LGTM)
  • 修正要求

🔄 マージ状態

  • コンフリクトなし
  • コンフリクトあり(要リベース)

👍 良い点

  • 実装計画に沿った適切な実装がされている
  • Logger.newを使用していた全ての箇所がSemanticLoggerに統一され、ログフォーマットが一貫性のあるものになった
  • 新規テストが追加され、ログレベルの切り替えと統合動作が検証されている
  • Rubocopチェックが通過し、コーディング規約に準拠している
  • CIが全て正常にパスしている(lint, security, test)
  • 不要なrequire "logger"文が削除され、依存関係がクリーンになっている

🛠 改善提案

  • 特に改善が必要な点はありません。実装内容は要件を満たしており、コード品質も良好です。

📝 詳細確認事項

  • デフォルトログレベルがINFOに設定されている
  • --verboseまたは-vフラグでDEBUGレベルに切り替わる
  • 全てのロガーがSemanticLogger形式で統一されている
  • 667テストが全てパス(0 failures, 2 pending)
  • 既存機能への影響がないことが確認されている

💬 コメント

素晴らしい実装です。計画通りに実装が進められ、テストカバレッジも確保されています。ログの統一により、デバッグやトラブルシューティングが容易になることが期待できます。

@douhashi douhashi added the soba:lgtm PR approved for auto-merge label Sep 20, 2025
@douhashi douhashi merged commit 99db2ba into main Sep 20, 2025
4 checks passed
@douhashi douhashi deleted the soba/144 branch September 20, 2025 13:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

soba:lgtm PR approved for auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

デフォルトのログレベルを INFO に soba start時にverbose optionが指定されているときだけDEBUGに

1 participant