Skip to content

fix: unblock reviewer swarm permissions - #7

Merged
cgaravitoq merged 1 commit into
mainfrom
reviewer-replacement-seq
Jul 5, 2026
Merged

fix: unblock reviewer swarm permissions#7
cgaravitoq merged 1 commit into
mainfrom
reviewer-replacement-seq

Conversation

@cgaravitoq

Copy link
Copy Markdown
Owner

Summary

Fix reviewer swarm failures caused by missing review-state directories and overly narrow bash allowlists.

Changes

  • Create the review-state directory before the guardrail plugin writes swarm budget state.
  • Expand reviewer bash permissions for git/repo inspection and read-only shell commands.
  • Keep subreviewers read-only by preserving write/edit/patch/task restrictions.
  • Add a regression test for first reviewer task budget write in an empty state directory.

Verify

  • bun test - pass
  • git diff --check - pass

@cgaravitoq
cgaravitoq merged commit ac2e099 into main Jul 5, 2026
2 checks passed
@cgaravitoq
cgaravitoq deleted the reviewer-replacement-seq branch July 5, 2026 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant