Skip to content

feat: soba:todo優先度ラベル機能の実装 (#161)#162

Merged
douhashi merged 1 commit into
mainfrom
soba/161
Nov 9, 2025
Merged

feat: soba:todo優先度ラベル機能の実装 (#161)#162
douhashi merged 1 commit into
mainfrom
soba/161

Conversation

@douhashi
Copy link
Copy Markdown
Owner

@douhashi douhashi commented Nov 9, 2025

Implementation Complete

fixes #161

Changes

  • collectTodoIssuesメソッドを修正し、soba:todo:highsoba:todo:lowラベルも収集対象に含めるように変更
  • hasActiveTaskメソッドを修正し、優先度ラベルをtodo系ラベルとして適切に認識するように変更
  • 新規メソッドupdateTodoLabelsToQueuedを追加し、すべてのtodo系ラベル(優先度ラベル含む)を削除してからsoba:queuedを追加するように実装
  • 優先度機能の包括的なテストケースを追加(queue_manager_collect_test.go, queue_manager_update_test.go
  • 既存のテストケースを更新し、新しいラベル削除処理に対応

Test Results

  • Unit tests: ✅ Pass
  • Full test suite: ✅ Pass

Checklist

  • Implementation follows the plan
  • Test coverage ensured
  • No impact on existing features
  • Backward compatibility maintained

- collectTodoIssuesメソッドを修正し、soba:todo:high/lowも収集対象に
- hasActiveTaskメソッドを修正し、優先度ラベルを考慮
- updateTodoLabelsToQueuedメソッドを追加し、すべてのtodo系ラベルを削除
- 優先度機能の包括的なテストケースを追加
- 後方互換性を維持しながら優先度ベースのキューイングを実現

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

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

douhashi commented Nov 9, 2025

Review Results

✅ Decision

  • Approve (LGTM)
  • Request changes

🔄 Merge Status

  • No conflicts
  • Conflicts exist (rebase required)

👍 Good Points

  • 計画書に沿った丁寧な実装がされている
  • collectTodoIssuesメソッドがsoba:todo:highsoba:todo:lowラベルも適切に収集するようになっている
  • hasActiveTaskメソッドが優先度ラベルをtodo系として正しく認識している
  • 新規追加されたupdateTodoLabelsToQueuedメソッドにより、全てのtodo系ラベルが適切に削除される仕組みになっている
  • 既存のテストケースの更新と新規テストケースの追加により、テストカバレッジが十分確保されている
  • 後方互換性が維持されており、既存のsoba:todoラベルのみの動作に影響がない

🔧 Improvement Suggestions

  • 特になし(実装が適切でテストも充実している)

@douhashi douhashi added the soba:lgtm PR approved for auto-merge label Nov 9, 2025
@douhashi douhashi merged commit acea1d4 into main Nov 9, 2025
3 checks passed
@douhashi douhashi deleted the soba/161 branch November 9, 2025 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

soba:lgtm PR approved for auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

不具合: soba:todo:high / soba:todo:low ラベルが機能しない

1 participant