feat(iterate): add generic iterate skill, replace iterative-eval

[baleen37]

Summary

Generic iterate skill 추가 및 iterative-eval 대체, create-pr auto-merge broken state 버그 수정.

Changes

• feat(iterate): add generic iterate skill, replace iterative-eval
• feat(iterative-eval): add iterative-eval skill for incremental A/B improvement
• fix(create-pr): stop auto-merge when verify reports broken state (exit 1 시 merge 중단)

Tests

• create-pr SKILL.md 로직 검토 완료

Breaking

• iterative-eval skill이 iterate로 대체됨

Summary by CodeRabbit

• Bug Fixes
  • Improved PR merge automation to properly halt when verification checks fail, preventing inadvertent merges on failed validations.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 95685e3e-b815-4bfc-8522-3e88b479101d

📥 Commits

Reviewing files that changed from the base of the PR and between de28307 and d6ff626.

📒 Files selected for processing (1)

• plugins/me/skills/create-pr/SKILL.md

---

📝 Walkthrough

Walkthrough

The script's post-gh pr checks --watch fallback path now captures the exit code from verify-pr-status.sh and conditionally halts before executing gh pr merge --squash. Previously, the code proceeded to merge regardless of verification status. The merge now only executes when the verifier returns exit code 0 or 2.

Changes

Cohort / File(s)	Summary
Create-PR Merge Control Flow plugins/me/skills/create-pr/SKILL.md	Added explicit exit code capture from verify-pr-status.sh and conditional branch to skip squash merge when verifier returns exit code 1, improving failure handling in the fallback merge path.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~15 minutes

Possibly related PRs

• fix(create-pr): add fallback for unreliable gh auto-merge #559 — Modifies the same create-pr/SKILL.md auto-merge fallback and verification flow, likely coordinating similar conditional merge logic.
• fix(pr-pass): handle UNKNOWN mergeStateStatus from already-merged PRs #562 — Alters verify-pr-status.sh to emit the exit codes (0, 1, 2) that this PR's logic now explicitly respects.
• feat(create-pr): split verify and sync workflows #489 — Updates the verification/merge orchestration by rewriting verify-pr-status.sh and restructuring sync logic alongside the merge flow.

Poem

🐰 A script that once charged ahead, now stops and thinks instead,
Respecting what the verifier said—no reckless merge to dread!
Exit codes guide the way, 0 or 2 save the day,
While 1 halts the play. ✨

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/iterate-skill

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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