Skip to content

feat(claudebox): pass mode=public to the v2 /run webhook#23607

Merged
ludamad merged 1 commit into
nextfrom
cb/4b53f2dd8370
May 27, 2026
Merged

feat(claudebox): pass mode=public to the v2 /run webhook#23607
ludamad merged 1 commit into
nextfrom
cb/4b53f2dd8370

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Follow-up to #23600 (merged), which pointed CI at the ClaudeBox v2 /run webhook but did not pass a mode.

A v2 session's GitHub repo access is derived from its mode (slack_route.effective_mode in session meta). The slackbot group's repos are entirely mode-scoped, so a /run session without a mode only resolves the group chain's unscoped repos (claudebox + honk-ai) and cannot write aztec-packages — the same "no GitHub access" failure #23600 set out to fix. Prompt text has no effect; /run never parses mode flags.

This passes mode: "public" in both dispatch payloads (overridable via repo var CLAUDEBOX_MODE), so kickoff//claudebox/review sessions resolve the public read-write repo set and can open PRs against aztec-packages.

Depends on AztecProtocol/claudebox#625, which adds mode support to the v2 /run endpoint (it stamps effective_mode into the new session's meta). Without #625 deployed, mode is accepted but ignored, so land/deploy #625 first or together.

Testing

  • bash -n on every run: body (7 steps) and YAML parse — pass.
  • jq payload validated with mode: "public" for the issue-comment, workflow_dispatch, and review shapes — valid JSON matching the v2 RunRequest.

Created by claudebox · group: slackbot

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 27, 2026
@AztecBot AztecBot marked this pull request as ready for review May 27, 2026 20:44
@AztecBot AztecBot requested a review from charlielye as a code owner May 27, 2026 20:44
@AztecBot AztecBot changed the title ci(claudebox): pass mode=public to the v2 /run webhook feat(claudebox): pass mode=public to the v2 /run webhook May 27, 2026
@ludamad ludamad enabled auto-merge May 27, 2026 20:46
@ludamad ludamad added this pull request to the merge queue May 27, 2026
Merged via the queue into next with commit c80f287 May 27, 2026
46 of 48 checks passed
@ludamad ludamad deleted the cb/4b53f2dd8370 branch May 27, 2026 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants