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

[douhashi]

期待する動作

• soba:todo:high / soba:todo / soba:todo:low ラベルがついているIssueの一覧が soba:queued にいれる候補
• soba:queuedに入れる際に、優先順位を soba:todo:high > soba:todo > soba:todo:low とする
• soba:todo:high に複数のIssueがある場合は、Issue番号の若いものを優先 (soba:todo / soba:todo:low も同様)
• 現状の仕様通り、 soba:queued に入るのは 常に1件のみ (シリアル開発)

注意事項:

• 複数のIssueが soba:queued にならないようにする (現状の仕様通りの動きを担保)

[douhashi]

Implementation Plan: soba:todo優先度ラベル機能の実装

Requirements Overview

• 目的: soba:todo:high / soba:todo / soba:todo:low ラベルによるIssueの優先度管理を実装する
• 背景: 現状、すべてのsoba:todoラベルが同じ優先度で処理されており、重要度の高いIssueを先に処理できない
• 機能要件:
  • soba:todo:high > soba:todo > soba:todo:low の優先順位でIssueをキューイングする
  • 同じ優先度のIssue間では、Issue番号の小さいものを優先する
  • 常に1件のみをsoba:queuedに入れる（シリアル処理を維持）
• 受入基準:
  • 高優先度のIssueが存在する場合、通常・低優先度のIssueより先に処理される
  • 複数のIssueが同時にsoba:queuedになることがない
  • 既存のsoba:todoラベルによる処理と互換性がある

Design Policy

• 技術選択:
  • 既存のQueueManager実装を拡張（queue_manager.go）
  • 優先度判定は別メソッドとして実装（テスタビリティ向上）
  • ラベル定義は既存のドメインモデルを活用（domain/phase.go）
• アーキテクチャ考慮点:
  • 現行のシリアル処理を維持
  • 後方互換性を確保（soba:todoラベルのみの場合も正常動作）
  • 将来的な優先度追加に対応しやすい設計

Implementation Steps

1. 優先度ラベル検出機能の修正

• 作業内容: collectTodoIssuesメソッドを修正し、soba:todo:highとsoba:todo:lowも対象に含める
• 関連ファイル:
  • internal/service/queue_manager.go (collectTodoIssuesメソッド)
• 注意点: soba:todo:highやsoba:todo:lowラベルが付いているIssueも「todoIssue」として扱う

2. アクティブタスク判定の修正

• 作業内容: hasActiveTaskメソッドの条件を調整し、優先度ラベルも考慮する
• 関連ファイル:
  • internal/service/queue_manager.go (hasActiveTaskメソッド)
• 注意点: soba:todo:high/soba:todo:lowがついていてもtodo状態として扱う

3. ラベル更新処理の修正

• 作業内容: updateLabelsメソッドを修正し、優先度ラベルも削除対象に含める
• 関連ファイル:
  • internal/service/queue_manager.go (EnqueueNextIssueメソッド内のupdateLabels呼び出し)
• 注意点: soba:todo:high/soba:todo:lowがある場合も適切に削除する

4. テストケースの追加・修正

• 作業内容: 優先度機能の動作確認テストを追加
• 関連ファイル:
  • internal/service/queue_manager_priority_test.go (既存のテストを拡張)
  • internal/service/queue_manager_test.go (必要に応じて修正)

Test Plan

Unit Tests

• 優先度判定テスト: getPriorityメソッドが正しく優先度を返すことを確認
• ソートテスト: sortByPriorityが優先度順にソートすることを確認
• 選択テスト: selectPriorityIssueが最適なIssueを選択することを確認
• 統合テスト: EnqueueNextIssueが優先度を考慮してキューイングすることを確認

Integration Tests

• 高優先度優先処理: soba:todo:highがsoba:todoより先に処理される
• 複数優先度混在: 異なる優先度が混在しても正しく処理される
• 後方互換性: soba:todoのみの既存動作が維持される
• ラベル削除: 優先度ラベルも含めて正しく削除される

Risks and Countermeasures

Risk 1: 既存のワークフローへの影響

• 内容: 優先度ラベルの導入により既存の処理が破綻する可能性
• 対策:
  • soba:todoラベルのみの場合の動作を変更しない
  • 優先度ラベルを「soba:todo」の拡張として扱う
  • 十分なテストカバレッジを確保

Risk 2: ラベル削除の不整合

• 内容: 優先度ラベルの削除漏れによる状態不整合
• 対策:
  • すべての優先度ラベル（high/low）を明示的に削除する処理を追加
  • エラーハンドリングを強化（ラベルが存在しない場合も正常処理）

Risk 3: 複数優先度ラベルの競合

• 内容: 1つのIssueにsoba:todo:highとsoba:todo:lowが両方付く場合
• 対策: 最高優先度を採用する（既存実装で対応済み）

Schedule

• 見積もり: 総計 4時間
• 内訳:
  • Step 1-2（ラベル検出・判定修正）: 1時間
  • Step 3（ラベル更新処理）: 1時間
  • Step 4（テスト実装）: 2時間

[douhashi]

Created PR #162