feedback-autofixを更新し準備が失敗した場合も結果を報告するようにした

[TinyKitten]

概要

TrainLCD/feedback-autofix の固定先を 406de875（feedback-autofix#1 の時点）から c71e2b4（master の先端）へ上げます。あわせて、report の呼び出し条件に failed を足します。

406de875 の prepare は、管理チケットの番号を特定したあとで落ちると eligible を空のまま返します。すると always() を付けていても report は走らず、ジョブが赤くなるだけで管理チケットには何も残りません。ISSUES_REPO_TOKEN の期限切れ、gh pr list の失敗、スクリプトのエラーがこれに当たります。

TrainLCD/feedback-autofix#3 で prepare 側を直し、#4 で master へ反映しました。呼び出し側の条件も揃えないと、この手当ては効きません。

変更内容

.github/workflows/auto_fix_from_feedback.yml だけです。Worker のコードには触れていません。

固定先の SHA（2 か所）

-- uses: TrainLCD/feedback-autofix/prepare@406de875a89dd1e640f8b66b71d4de9516a952ca # feedback-autofix#1
+- uses: TrainLCD/feedback-autofix/prepare@c71e2b4abf4149b4b83e9a87cd185b1558ed8237 # feedback-autofix#4

report も同じ SHA へ上げています。

report の呼び出し条件

-  if: ${{ always() && steps.prepare.outputs.eligible == 'true' }}
+  if: ${{ always() && (steps.prepare.outputs.eligible == 'true' || steps.prepare.outputs.failed == 'true') }}

failed は #4 で入った prepare の新しい出力です。管理チケットの番号が分かったあとで準備が失敗すると true になります。このとき eligible は false になるので、npm ci と Claude Code Action は従来どおり飛ばされ、report だけが走って「準備の途中で止まりました」を管理チケットへ書き、最後にジョブを落とします。

外さないための理由をコメントとして残しました。

動きの変化

準備の結果	これまで	この PR のあと
対象で、最後まで成功	エージェントが動き、report が結果を書く	変わりません
対象外	何も書かずに終わる	変わりません
対象と分かったあとで準備が失敗	ジョブが赤くなるだけで、管理チケットには何も残らない	report が止まった箇所を書いてから、ジョブを落とす
管理チケットの番号が分かる前に失敗	ジョブが赤くなる	変わりません（書き込む先が決まらないため）

設定の変更

バインディング・シークレット・キュー・KV・R2・Cron のいずれも変えていません。

テスト

npm run lint・npm test・npm run typecheck は実行していません。この PR は GitHub Actions のワークフローファイル 1 つだけを変えていて、src/ にも test/ にも package.json にも触れていないためです。CI 側の判断に任せます。

かわりに次の 2 つを確認しました。

• .github/workflows/auto_fix_from_feedback.yml をパースし、構文が壊れていないこと
• 固定先の c71e2b4abf4149b4b83e9a87cd185b1558ed8237 を直接取得し、prepare/action.yml に failed 出力と 4 つの continue-on-error: true、Summarize preparation（id: state）が入っていること

git fetch --depth 1 https://github.com/TrainLCD/feedback-autofix \
  c71e2b4abf4149b4b83e9a87cd185b1558ed8237
git show FETCH_HEAD:prepare/action.yml

関連Issue

• 準備が対象確定後に落ちてもreportまで進むようにした feedback-autofix#3
• devをmasterへ反映する feedback-autofix#4
• フィードバックから自動で修正PRを出すワークフローを追加 #36

🤖 Generated with Claude Code

https://claude.ai/code/session_018jUHQb3vwnKPNYL2Cm1mxw

---

Generated by Claude Code

Summary by CodeRabbit

• 改善
  • 自動フィードバック対応の処理を更新しました。
  • 対応対象と判定された場合、または処理中に失敗が発生した場合でも、結果の報告が実行されるようになりました。
  • 自動修正に関する処理結果を、より安定して確認できるようになりました。

[coderabbitai]

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 65c46bd5-ff13-4563-8561-b438afa621ca

📥 Commits

Reviewing files that changed from the base of the PR and between a81fe64 and 0c034ec.

📒 Files selected for processing (1)

• .github/workflows/auto_fix_from_feedback.yml

Limit details: You’ve used all 5 included reviews currently available. Your 16 included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

---

📝 Walkthrough

Walkthrough

自動修正ワークフローが feedback-autofix#4 の prepare と report を使用します。report は、対象判定済み、または処理失敗時にも実行されます。

Changes

自動修正フィードバック処理

Layer / File(s)	Summary
アクション参照と報告条件の更新 .github/workflows/auto_fix_from_feedback.yml	prepare と report の参照を feedback-autofix#4 に更新しました。report は常時評価され、prepare の eligible または failed が true の場合に実行されます。

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	タイトルは、feedback-autofix の更新と、準備失敗時にも結果を報告する変更を明確に示しています。プルリクエストの主な変更内容と一致しています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

Usage-based review receipt

• Mode: Continue automatically
• Reviewed files: 1
• Charged: $0.25
• View usage details

Note

This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. View usage-based billing.

---

_{うさぎが新しい流れを駆ける
prepare が合図を送る
eligible なら耳を立てる
failed でも report が続く
feedback-autofix#4 が道を照らす}

_{Comment @coderabbitai help to get the list of available commands.}