Skip to content

fix: Issue駆動CIプロンプトを書籍 ch07 7.6 節に整合させPR作成まで完走させる#24

Merged
laiso merged 1 commit into
mainfrom
fix/issue-driven-pr-creation-prompt
May 1, 2026
Merged

fix: Issue駆動CIプロンプトを書籍 ch07 7.6 節に整合させPR作成まで完走させる#24
laiso merged 1 commit into
mainfrom
fix/issue-driven-pr-creation-prompt

Conversation

@laiso
Copy link
Copy Markdown
Owner

@laiso laiso commented May 1, 2026

Summary

  • bin/cli.tsissueDrivenInstructions を書籍 ch07 7.6 節の記述に合わせて強化
  • 7 項目の TODO チェックボックステンプレ・「重要: ファイル修正だけで終わるな」の強調・createIssueComment の名指し指示を導入
  • ch07-7 補足の ## Issue本文(参照用) 埋め込みは維持

背景

従来の CI 向け追加指示は 1 行のステップ羅列(Issue理解 → ファイル読込 → 修正 → テスト → Gitコミット/プッシュ → PR作成 → Issueへ結果コメント)で、ベース指示 (src/core/prompt.md) の「必ず以下の形式で TODO リストを作成してください」+ 4 項目テンプレートに押し負けていた。

結果として gpt-5-nano 等の小型モデルでは 4 項目テンプレ(タスク理解/ファイル読込/変更/動作確認)をそのまま採用し、ファイル修正とテスト実行を終えた段階で「正常終了」してしまい、createBranch / commitChanges / pushBranch / createPullRequest / createIssueComment ツールが一度も呼ばれずに完了する事象が発生していた(参考: TM-DataScientist/nano-code の Actions ログ)。

書籍 ch07 7.6 節 (manuscript/ch07_github_actions/ch07_GitHub Actionsで開発ワークフローを自動化する.md:378-400) には PR 作成まで完走する強いプロンプトが既に掲載されており、実装を書籍に整合させる修正。

Test plan

  • ローカルで `LLM_PROVIDER=openai LLM_MODEL=gpt-5-nano LLM_API_KEY=… ISSUE_TEXT="calculator.ts にテストを追加" ISSUE_NUMBER=999 bun run bin/cli.ts --yolo` を実行し、TODO が 7 項目で出力されることを確認
  • `gh workflow run nano-code.yml -f task="calculator.ts にテストを追加して"` で Actions 上の手動トリガーから PR 作成まで到達することを確認
  • Issue を新規作成して自動トリガー経由でも PR 作成 + Issue コメント投稿まで完走することを確認

🤖 Generated with Claude Code

書籍 ch07 7.6 節の `issueDrivenInstructions` で示されている 7 項目 TODO テンプレートと
「ファイル修正だけで終わるな」の重要強調、`createIssueComment` の名指し指示を採用。
従来の 1 行ステップ羅列ではベース指示の 4 項目 TODO テンプレに押し負け、gpt-5-nano
等の小型モデルがファイル修正/テスト実行のみで終了し PR 作成まで到達しなかった。

ch07-7 補足の `## Issue本文(参照用)` ブロックは維持。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@laiso laiso mentioned this pull request May 1, 2026
@laiso laiso merged commit 2a82b27 into main May 1, 2026
2 checks passed
@laiso laiso deleted the fix/issue-driven-pr-creation-prompt branch May 1, 2026 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant