fix(autopilot): sticky コメントのマーカー統一で重複投稿を防ぐ (#809)

[smalruby3-editor-bot]

Summary

PR #761（甲子園 Epic）で autopilot status sticky が 2 つ投稿された不具合を修正する。
原因は sticky コメントの識別マーカーが 2 種類に分裂していたこと:

• <!-- autopilot-sticky-status -->（コントラクト docs/autopilot/autonomous-contract.md §2/§7 の正準マーカー）
• <!-- autopilot:sticky -->（daemon feat(autopilot): daemon が PR ラベル/Draft/sticky を自動同期 #794 の PR 同期実装が使っていたマーカー）

daemon が既存 sticky を自分のマーカーでしか検索しないため、別マーカーの既存 sticky を
見つけられず新規投稿し、重複していた。

Changes Made

• マーカーをコントラクト仕様 autopilot-sticky-status に統一（実装を spec に合わせる）。
  renderSticky は正準マーカーのみを出力する。
• 旧マーカー autopilot:sticky は legacy として検出対象に残し、両マーカーにマッチさせて
  既存 sticky を in-place 更新する（STICKY_MARKERS / isStickyComment / selectStickyCommentIds）。
• upsertStickyComment を、両マーカーで検出 → 複数あれば先頭を PATCH・残りを DELETE して
  1 つに集約する実装に変更（重複を作らない・既存を吸収）。
• コメント本文取得を listIssueComments（@base64 経由で改行安全）に切り出した。

Test Coverage

• isStickyComment / selectStickyCommentIds が両マーカーにマッチする unit test を追加。
• STICKY_MARKER が正準マーカーであること、renderSticky が legacy を出力しないことを検証。
• 既存テスト（daemon の sticky マーカー検証）を新マーカーに更新。
• node --test: 75 passed / 0 failed。

Related Issues

Closes #809

🤖 Generated with Claude Code

[smalruby3-editor-bot]

🤖 autopilot status

field	value
Status	Review
AI Status	—
HITL	No
Size	small

Linked issue #809. Maintained by autopilot (single writer); do not edit.

[smalruby3-editor-bot]

🤖 autopilot 敵対的レビュー結果

別レビュアー視点で差分を検証しました。自動修正が必要な明白な問題はなし。人間レビューへ渡します。

確認した観点と結果

観点	結果
正確性	✅ listIssueComments の base64 パースは空ボディ (id のみの行) でもクラッシュしない。selectStickyCommentIds は入力順（= GitHub の作成日時昇順）を保持し、upsertStickyComment は最古を残して PATCH・残りを DELETE するため重複が 1 つに収束する。
セキュリティ	✅ gh() は execFileSync('gh', args[])（シェル非経由）で引数配列、body も -f body=... で 1 引数として渡すため injection なし。jq フィルタは静的文字列。
マーカー統一	✅ 正準 autopilot-sticky-status に統一。旧 autopilot:sticky は legacy として吸収対象に残置。リポジトリ内に未更新の旧マーカー参照は無し（contract は marker 文字列リテラルを規定しておらず同期不要）。
テスト	✅ 検出系の純粋関数（isStickyComment / selectStickyCommentIds / マーカー定数 / renderSticky）を網羅。75 passed / 0 failed。

軽微な指摘（非ブロッカー・対応不要）

• upsertStickyComment の集約ループ（PATCH 先頭 + DELETE 残り）に直接の unit test が無い。 ただし判定ロジックは純粋関数 selectStickyCommentIds に切り出し済みでテスト済み、daemon.test.js 側は本関数をモックしている。project.js は module レベルの gh を直接呼ぶ構造で注入テストには refactor が要るため、レビュー段階での強制修正は見送り。
• legacy sticky が人手投稿だった場合 daemon が DELETE するが、これは Issue fix(autopilot): sticky コメントのマーカー不一致で重複投稿される #809 が意図する「1 つに集約」の挙動であり想定内。

LGTM。