Skip to content

meta(warden): Use Grok 4.5 through OpenRouter#199

Merged
dcramer merged 1 commit into
mainfrom
feat/grok-4.5
Jul 11, 2026
Merged

meta(warden): Use Grok 4.5 through OpenRouter#199
dcramer merged 1 commit into
mainfrom
feat/grok-4.5

Conversation

@dcramer

@dcramer dcramer commented Jul 11, 2026

Copy link
Copy Markdown
Member

Warden security reviews now use Grok 4.5 through OpenRouter instead of Claude Sonnet 4.6. The provider-qualified selector matches Pi's OpenRouter catalog and uses the existing WARDEN_OPENROUTER_API_KEY workflow credential.

Route Warden analysis through the OpenRouter Grok 4.5 model catalog entry.

Co-Authored-By: Codex <noreply@openai.com>
@dcramer

dcramer commented Jul 11, 2026

Copy link
Copy Markdown
Member Author

Once we merge ill delete WARDEN_MODEL in our github configs

@dcramer dcramer marked this pull request as ready for review July 11, 2026 13:49
Comment thread warden.toml
[defaults]
runtime = "pi"
model = "anthropic/claude-sonnet-4-6"
model = "openrouter/x-ai/grok-4.5"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Switching the LLM to Grok 4.5 may cause silent failures in the warden action if its output format differs from Claude Sonnet's, as parsing may fail silently.
Severity: MEDIUM

Suggested Fix

Before merging, verify that the getsentry/warden@v0 action can correctly parse outputs from the Grok 4.5 model. This may require testing in a separate environment or inspecting the action's source code to ensure it's model-agnostic. If incompatibilities exist, either revert to the previous model or update the parsing logic within the action.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: warden.toml#L5

Potential issue: The configuration in `warden.toml` is changing the LLM from Claude
Sonnet 4.6 to Grok 4.5. The external `getsentry/warden@v0` action, which processes the
model's output, may have parsing logic tailored to Claude's specific response structure.
Since different LLMs can have subtle variations in their output format even when
API-compatible, Grok's responses might not be correctly parsed. Due to
`continue-on-error: true` in the workflow, a parsing failure would not cause the CI to
fail but would likely result in the `warden` action silently producing no findings,
effectively disabling the analysis.

Did we get this right? 👍 / 👎 to inform future reviews.

@dcramer dcramer merged commit 68cb703 into main Jul 11, 2026
11 checks passed
@dcramer dcramer deleted the feat/grok-4.5 branch July 11, 2026 13:55
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