dev-loops 0.2.2 → 0.2.3

package/.claude/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "dev-loops",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Agent-harness-agnostic dev-loop: agents, skills, and hooks for GitHub/Copilot-driven development loops.",
   "author": {
     "name": "Manuel Fittko",

package/.claude/skills/copilot-pr-followup/SKILL.md

@@ -225,6 +225,7 @@ When unresolved feedback exists, use a narrow follow-up loop:
     - if the refreshed snapshot reports unresolved threads, re-enter the reply/resolve loop for the missed threads
 12. only after GitHub-side reply/resolve work is done for the addressed threads and the refreshed thread snapshot proves zero unresolved threads remain, decide whether another Copilot pass is desired
     - resolve the review-round cap from config via `resolveRefinementConfig(config, "maxCopilotRounds")` from `@dev-loops/core/config`; default config ships `maxCopilotRounds: 5`
+    - **Opt out entirely:** `maxCopilotRounds: 0` disables the external Copilot review gate for the repo — the loop runs `draft_gate → pre_approval_gate` with the local harness only, never requesting or waiting on Copilot. Use this when the repo has no Copilot reviewer configured or prefers local-harness-only review.
     - use the completed Copilot review-round count from `detect-copilot-loop-state.mjs` / `copilot-pr-handoff.mjs` as the current PR's review-round count
     - if completed review rounds have reached the maximum (default: 5), do **not** re-request Copilot review
     - when the round limit is reached **and** the refreshed thread snapshot proves zero unresolved threads **and** current-head CI is green or credibly green, treat that clean state as eligible for `pre_approval_gate` fallback instead of deadlocking on another Copilot rerequest

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented in this file.
 
+## 0.2.3
+
+### Added
+
+- **Opt out of the Copilot review gate via `refinement.maxCopilotRounds: 0`** (#832). For repos
+  without a Copilot reviewer configured (or that prefer local-harness-only review), setting
+  `maxCopilotRounds: 0` disables the external Copilot review cycle entirely — the loop runs
+  `draft_gate → pre_approval_gate` with no Copilot request or wait. The config schema now accepts
+  `0` (`nonnegative`; negative still rejected); `evaluatePrGateCoordination` routes `0` through the
+  existing `internal_only` path, `shouldGuardCopilotReviewRequest` never forces a request at `0`,
+  and the watch-cycle handoff (`copilot-pr-handoff`) skips the request too. Default (`5`) unchanged.
+  Documented in the README, extension config docs, and the `copilot-pr-followup` skill.
+
 ## 0.2.2
 
 ### Fixed

package/README.md

@@ -193,7 +193,7 @@ npx dev-loops gates   # see what reviewers will check
 Key surfaces:
 - **Gate angles** — which review lenses run at draft and pre-approval gates
 - **Persona prompts** — focused instructions per angle (DRY, KISS, YAGNI, SRP, SoC, and more)
-- **Refinement** — fan-out count and mode for parallel review variants
+- **Refinement** — fan-out count and mode for parallel review variants; `refinement.maxCopilotRounds` caps Copilot re-review rounds (default `5`), and **`maxCopilotRounds: 0` disables the Copilot review gate entirely** for local-harness-only review (`draft_gate → pre_approval_gate`, no Copilot) — useful when the repo has no Copilot reviewer configured
 - **Autonomy** — which gates require operator confirmation
 - **Workflow defaults** — retrospective enforcement, draft-first posture, dev-mode policy
 

package/extension/README.md

@@ -133,7 +133,9 @@ personas:
 
 # Override gate requirements
 refinement:
-  fanOut: 5      # run 5 parallel review variants instead of 3
+  fanOut: 5            # run 5 parallel review variants instead of 3
+  maxCopilotRounds: 0  # 0 disables the Copilot review gate entirely (local-harness-only
+                       # review: draft_gate → pre_approval_gate, no Copilot). Default: 5.
 
 autonomy:
   stopAt:

package/package.json

@@ -68,7 +68,7 @@
   },
   "dependencies": {
     "mermaid": "11.15.0",
-    "@dev-loops/core": "^0.2.2"
+    "@dev-loops/core": "^0.2.3"
   },
   "repository": {
     "type": "git",
@@ -78,7 +78,7 @@
     "url": "https://github.com/mfittko/dev-loops/issues"
   },
   "homepage": "https://github.com/mfittko/dev-loops#readme",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "files": [
     "cli/",
     "lib/",

package/scripts/loop/copilot-pr-handoff.mjs

@@ -366,12 +366,31 @@ export async function runHandoff(options, { env = process.env, ghCommand = "gh"
   // Detect internal tooling PRs — suppress Copilot review request step entirely.
   // Internal-only PRs (scripts/docs/tests/config) skip the request, not just the wait.
   let internalOnlySkipCopilot = false;
-  // Skip internal detection in sequential stub/test mode to avoid consuming stub entries.
-  // Claims-mode stubs handle interleaved calls; detection runs normally.
-  if (!env.GH_SEQUENCE_PATH || env.GH_STUB_MODE === "claims") {
-  if (options.watchStatus === undefined &&
+  // Config opt-out (#832): maxCopilotRounds: 0 disables the external Copilot review
+  // gate for the repo (local-harness-only review). Treat it like an internal-only PR
+  // — skip the request, not just the wait — regardless of stub/sequence mode. This
+  // mirrors the gate-coordination side, which routes maxCopilotRounds: 0 to internal_only.
+  if (refinementConfig?.maxCopilotRounds === 0 &&
+      options.watchStatus === undefined &&
       (interpretation.state === STATE.PR_READY_NO_FEEDBACK ||
        interpretation.state === STATE.READY_TO_REREQUEST_REVIEW)) {
+    internalOnlySkipCopilot = true;
+    interpretation = {
+      ...interpretation,
+      state: STATE.INTERNAL_TOOLING_DIRECT_GATE,
+      nextAction: NEXT_ACTIONS[STATE.INTERNAL_TOOLING_DIRECT_GATE],
+      allowedTransitions: TRANSITIONS[STATE.INTERNAL_TOOLING_DIRECT_GATE] || [STATE.DONE],
+    };
+  }
+  // Skip internal detection in sequential stub/test mode to avoid consuming stub entries.
+  // Claims-mode stubs handle interleaved calls; detection runs normally.
+  if (
+    !internalOnlySkipCopilot &&
+    (!env.GH_SEQUENCE_PATH || env.GH_STUB_MODE === "claims") &&
+    options.watchStatus === undefined &&
+    (interpretation.state === STATE.PR_READY_NO_FEEDBACK ||
+      interpretation.state === STATE.READY_TO_REREQUEST_REVIEW)
+  ) {
     try {
       const internalCheck = await detectPrInternalOnly(options, { env, ghCommand });
       if (internalCheck.ok && internalCheck.internalOnly) {
@@ -387,7 +406,6 @@ export async function runHandoff(options, { env = process.env, ghCommand = "gh"
       // Best-effort: if detection fails, fall through to normal request behavior
     }
   }
-  }
 
   let reviewRequestStatus;
   const shouldRequestReview = !internalOnlySkipCopilot && options.watchStatus === undefined

package/skills/copilot-pr-followup/SKILL.md

@@ -229,6 +229,7 @@ When unresolved feedback exists, use a narrow follow-up loop:
     - if the refreshed snapshot reports unresolved threads, re-enter the reply/resolve loop for the missed threads
 12. only after GitHub-side reply/resolve work is done for the addressed threads and the refreshed thread snapshot proves zero unresolved threads remain, decide whether another Copilot pass is desired
     - resolve the review-round cap from config via `resolveRefinementConfig(config, "maxCopilotRounds")` from `@dev-loops/core/config`; default config ships `maxCopilotRounds: 5`
+    - **Opt out entirely:** `maxCopilotRounds: 0` disables the external Copilot review gate for the repo — the loop runs `draft_gate → pre_approval_gate` with the local harness only, never requesting or waiting on Copilot. Use this when the repo has no Copilot reviewer configured or prefers local-harness-only review.
     - use the completed Copilot review-round count from `detect-copilot-loop-state.mjs` / `copilot-pr-handoff.mjs` as the current PR's review-round count
     - if completed review rounds have reached the maximum (default: 5), do **not** re-request Copilot review
     - when the round limit is reached **and** the refreshed thread snapshot proves zero unresolved threads **and** current-head CI is green or credibly green, treat that clean state as eligible for `pre_approval_gate` fallback instead of deadlocking on another Copilot rerequest