lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

docs(claude): formalize Auto-review passed as approval signal #85

Merged
lekss361 merged 2 commits from chore/auto-review-approval-rule into main 2026-05-11 21:43:26 +00:00
Conversation 6 Commits 2 Files changed 1 +35 -3

2 commits

Author SHA1 Message Date
lekss361
1682d9f432 fix(claude): address PR #85 auto-review feedback 
- Fix cyrillic/latin typo (rule 5 second sub-bullet)
- Add SHA marker validation to polling loop (race condition on stale LGTM after fixup-push)
- Add explicit "Auto-merge scope" rule 6 — allowlist (docs/UI/.claude) vs blocklist (migrations/API/infra/secrets)
- Sync "Когда auto-mode" section with new merge approval signals
- Lower cap to 5 iters for scope-blocked PRs (human required anyway)
- Self-modifying CLAUDE.md rules → always require human

Per auto-review on e66e1ea.
 2026-05-12 00:27:50 +03:00
lekss361
e66e1ea981 docs(claude): formalize "Auto-review passed" as approval signal + polling loop 
Per user feedback after PR #84 merge:
- Critical rule 5: "Auto-review passed" / "LGTM" от auto-review bot = approval
  (наряду с явным "merge it" / "approved" от human user)
- Add "Polling loop для PR" section: 60s ScheduleWakeup pattern,
  decisive states (merged / approved / changes / no-news), cap 30 iters

This formalizes what main session already did: after fixup commit, poll
comments, detect auto-review verdict, and auto-merge if passed.
 2026-05-12 00:18:21 +03:00