chore(skills): unify PR stop condition to 3 passes

[PierreBrisorgueil]

Aligns Node with Vue — both now require 3 consecutive clean passes with grace periods before declaring convergence.

Summary by CodeRabbit

• Documentation
  • Updated internal documentation of monitoring processes and thresholds.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 44aefe8a-7ee7-4d78-a1a5-1fbb63fe8917

📥 Commits

Reviewing files that changed from the base of the PR and between 8b69820 and c74fc9e.

📒 Files selected for processing (1)

• .claude/skills/pull-request/SKILL.md

---

Walkthrough

Documentation update to the pull-request skill that adjusts the monitor loop stop-condition threshold from 2 to 3 consecutive iterations with zero unresolved review threads before checking branch protection status.

Changes

Cohort / File(s)	Summary
Monitor Loop Documentation .claude/skills/pull-request/SKILL.md	Updated stop-condition threshold for consecutive zero-unresolved-thread iterations from 2 to 3 in the monitor loop control flow documentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• fix(pull-request skill): reply and resolve all threads including non-actionable #3167: Modifies the same pull-request skill documentation file with changes to the monitor loop stop-condition logic.
• chore(skill): align pull-request skill with Vue stack #3190: Updates the same skill documentation, including threshold adjustments for zero unresolved threads detection.
• chore(claude): rename pr skill to pull-request and fix automation gaps #3160: Changes the monitor-loop stop-condition behavior in the pull-request skill documentation.

Suggested labels

Chore

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/unify-pr-stop-condition

---

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.}

[Copilot]

Pull request overview

Updates the pull-request skill’s convergence logic to match the Vue workflow by requiring 3 consecutive clean passes (with grace periods) before stopping.

Changes:

• Increase the monitor loop stop threshold from 2 to 3 consecutive “zero unresolved threads” passes.

[Copilot]

The stop condition is updated here to require 3 consecutive clean passes, but later in the same document (section 6f “Stop condition”) it still states “2 consecutive passes”. Please update that section to match the new 3-pass convergence rule so the instructions are consistent.

Suggested change

	if consecutive_zero >= 3 → check branch protection (see 6f), then STOP ✓
	if consecutive_zero >= 2 → check branch protection (see 6f), then STOP ✓