Skip to content

fix(fullsend): allow Claude Code Vertex authentication - #13

Merged
kadel merged 1 commit into
mainfrom
fix/fullsend-claude-exe-policy
Sep 1, 2026
Merged

kadel merged 1 commit into
mainfrom
fix/fullsend-claude-exe-policy

Conversation

@kadel

@kadel kadel commented Sep 1, 2026 •

Copy link
Copy Markdown

Allow Claude Code 2.1+’s claude.exe binary to reach Vertex STS.

Configure Review to use the repository-owned RHDH image and policy, and update the code/fix and e2e-triage policies with the same allowlist.

This fixes the policy_denied failure confirmed in the OpenShell logs for run 33504512709.

This fix follows the Fullsend policy configuration introduced in redhat-developer/rhdh-agentic#124.

Allow the claude.exe binary to reach Vertex STS and run Review with the repository-owned image and policy.

Assisted-by: Codex
Signed-off-by: Tomas Kral <tkral@redhat.com>
@kadel
kadel requested a review from durandom as a code owner September 1, 2026 13:20
@fullsend-ai-review

fullsend-ai-review Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ❌ Failure (validation failed after 2 iteration(s)) · Started 1:25 PM UTC · Completed 1:27 PM UTC

Commit: d493020 · View workflow run →

Runtime: claude · Model: claude-opus-4-8

@kadel
kadel merged commit da17a79 into main Sep 1, 2026
9 of 10 checks passed
@fullsend-ai-retro

fullsend-ai-retro Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

🤖 Finished Retro · ✅ Success · Started 1:37 PM UTC · Completed 2:00 PM UTC

Commit: d493020 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $6.14

@kadel
kadel deleted the fix/fullsend-claude-exe-policy branch September 1, 2026 13:49
@fullsend-ai-retro

Copy link
Copy Markdown

Retro: PR fullsend-ai/fullsend#13 — fix(fullsend): allow Claude Code Vertex authentication

What happened

PR fullsend-ai/fullsend#13 on this newly-created repo (Aug 18, 2026) fixed a policy_denied failure that had been blocking all review agent runs since the repo was set up. Claude Code 2.1+ ships as a bun-compiled binary named claude.exe, but the upstream network policies from fullsend-ai/agents (pinned at 4bbe4f5) only allowlisted **/claude — causing the sandbox to block claude.exe from reaching Vertex AI STS for authentication.

The review agent dispatched on PR fullsend-ai/fullsend#13 itself also hit policy_denied (chicken-and-egg: the fix wasn't in the base branch yet). The agent consumed zero tokens across 2 iterations, produced no output, and reported "validation failed after 2 iteration(s)." The PR was correctly merged by kadel ~10 minutes later. A follow-up PR fullsend-ai/fullsend#14 (merged 7 minutes after) switched the model pin from Opus 4.8 (unavailable on Vertex) to Opus 4.6.

No review agent has ever completed successfully on this repo. All 4+ review dispatches failed due to policy or model issues before the fixes in PRs fullsend-ai/fullsend#13 and fullsend-ai/fullsend#14 landed.

Evidence for existing issues

Proposals skipped (target repo not allowed)

File manually or update create_issues.allow_targets in config.yaml:

  • **Add /claude.exe to upstream network policy binary allowlists (fullsend-ai/agents)

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