From 4ccadcfd3fb3a08666aa4c808a123500bb14ac46 Mon Sep 17 00:00:00 2001 From: Trevin Chow Date: Thu, 16 Apr 2026 22:28:52 -0700 Subject: [PATCH] fix(resolve-pr-feedback): unblock /loop scheduling (#582) --- plugins/compound-engineering/AGENTS.md | 2 ++ .../compound-engineering/skills/resolve-pr-feedback/SKILL.md | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/compound-engineering/AGENTS.md b/plugins/compound-engineering/AGENTS.md index 2bc55cc..a137c35 100644 --- a/plugins/compound-engineering/AGENTS.md +++ b/plugins/compound-engineering/AGENTS.md @@ -182,6 +182,8 @@ grep -E '^description:' skills/*/SKILL.md Beta skills use a `-beta` suffix and `disable-model-invocation: true` to prevent accidental auto-triggering. See `docs/solutions/skill-design/beta-skills-framework.md` for naming, validation, and promotion rules. +**Caveat on non-beta use of `disable-model-invocation`:** The flag blocks all model-initiated invocations via the Skill tool, which includes scheduled re-entry from `/loop`. Only a user typing a slash command directly bypasses it. If a skill is intended to be schedulable (e.g., `resolve-pr-feedback`), do not set this flag — rely on description specificity and argument requirements to prevent accidental auto-fire instead. + ### Stable/Beta Sync When modifying a skill that has a `-beta` counterpart (or vice versa), always check the other version and **state your sync decision explicitly** before committing — e.g., "Propagated to beta — shared test guidance" or "Not propagating — this is the experimental delegate mode beta exists to test." Syncing to both, stable-only, and beta-only are all valid outcomes. The goal is deliberate reasoning, not a default rule. diff --git a/plugins/compound-engineering/skills/resolve-pr-feedback/SKILL.md b/plugins/compound-engineering/skills/resolve-pr-feedback/SKILL.md index e79b973..2ef6983 100644 --- a/plugins/compound-engineering/skills/resolve-pr-feedback/SKILL.md +++ b/plugins/compound-engineering/skills/resolve-pr-feedback/SKILL.md @@ -2,7 +2,6 @@ name: resolve-pr-feedback description: Resolve PR review feedback by evaluating validity and fixing issues in parallel. Use when addressing PR review comments, resolving review threads, or fixing code review feedback. argument-hint: "[PR number, comment URL, or blank for current branch's PR]" -disable-model-invocation: true allowed-tools: Bash(gh *), Bash(git *), Read ---