Skip to content

feat: soba:todoラベルの優先順位機能を追加 (#159)#160

Merged
douhashi merged 1 commit into
mainfrom
soba/159
Nov 4, 2025
Merged

feat: soba:todoラベルの優先順位機能を追加 (#159)#160
douhashi merged 1 commit into
mainfrom
soba/159

Conversation

@douhashi
Copy link
Copy Markdown
Owner

@douhashi douhashi commented Nov 4, 2025

Implementation Complete

fixes #159

Changes

  • GetSobaLabels()に優先度ラベル(soba:todo:highsoba:todo:low)を追加
  • QueueManagerに優先度判定ロジック(getPriority())を実装
  • 優先度順ソート機能(sortByPriority())を実装
  • Issue選択をselectMinimumIssue()からselectPriorityIssue()に変更
  • 完全なテストカバレッジを実現(TDD)

Priority Rules

  1. 最優先: soba:todo:high ラベルのIssue(Issue番号が若いものから)
  2. 通常: soba:todo ラベルのIssue(Issue番号が若いものから)
  3. 低優先: soba:todo:low ラベルのIssue(Issue番号が若いものから)

複数の優先度ラベルが付いている場合は最高優先度を採用します。

Test Results

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

Checklist

  • Implementation follows the plan
  • Test coverage ensured (TDD approach)
  • No impact on existing features
  • Backward compatibility maintained (soba:todoは通常優先度として動作)

- 新しい優先度ラベル(soba:todo:high、soba:todo:low)を追加
- 優先度判定ロジック(getPriority)を実装
- 優先度順ソート機能(sortByPriority)を実装
- selectMinimumIssueをselectPriorityIssueに置き換え
- TDDアプローチで完全なテストカバレッジを確保

優先順位ルール:
1. 高優先度(soba:todo:high)
2. 通常優先度(soba:todo)
3. 低優先度(soba:todo:low)

同一優先度内ではIssue番号順を保持
@douhashi
Copy link
Copy Markdown
Owner Author

douhashi commented Nov 4, 2025

Review Results

✅ Decision

  • Approve (LGTM)
  • Request changes

🔄 Merge Status

  • No conflicts
  • Conflicts exist (rebase required)

👍 Good Points

  • 要件に従い soba:todo:highsoba:todo:low ラベルを追加
  • 優先度判定ロジック(getPriority())が明確で理解しやすい実装
  • 優先度順ソート(sortByPriority())が正しく実装されている
  • 複数の優先度ラベルが付いている場合、最高優先度を採用する適切な仕様
  • 後方互換性を維持(既存の soba:todo は通常優先度として動作)
  • TDDアプローチによる完全なテストカバレッジ
  • 各優先度レベル(高、通常、低)のテストケース網羅
  • エッジケースのテスト(複数優先度ラベル、空配列など)
  • 統合テスト(EnqueueNextIssue_WithPriority)も実装
  • 既存の selectMinimumIssue() から selectPriorityIssue() への変更が適切
  • CIチェックが全てPass

🔧 Improvement Suggestions

なし - 実装は計画通りで、全ての要件を満たしています。

@douhashi douhashi added the soba:lgtm PR approved for auto-merge label Nov 4, 2025
@douhashi douhashi merged commit 52d6121 into main Nov 4, 2025
3 checks passed
@douhashi douhashi deleted the soba/159 branch November 4, 2025 12:35
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.

feat: soba:todoラベルの優先順位機能を追加

1 participant