Skip to content

docs: Recommend org-level secrets for Warden configuration#18

Merged
dcramer merged 4 commits intomainfrom
feat/cleanup-config
Jan 29, 2026
Merged

docs: Recommend org-level secrets for Warden configuration#18
dcramer merged 4 commits intomainfrom
feat/cleanup-config

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Jan 29, 2026

Update setup guide to be more prescriptive about organization-wide configuration. Recommend setting WARDEN_ANTHROPIC_API_KEY, WARDEN_MODEL, and GitHub App secrets at the org level so all repos share them.

Workflow examples now use env vars instead of action inputs for API key and model, making the configuration cleaner.

Also tightened up language following brand guidelines.

Update setup guide to be more prescriptive about organization-wide
configuration. Recommend setting WARDEN_ANTHROPIC_API_KEY, WARDEN_MODEL,
and GitHub App secrets at the org level so all repos share them.

Workflow examples now use env vars instead of action inputs for API key
and model, making the configuration cleaner.

Also tightened up language following brand guidelines.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
warden Ready Ready Preview, Comment Jan 29, 2026 11:06pm

Request Review

Copy link
Contributor

@sentry-warden sentry-warden bot left a comment

Choose a reason for hiding this comment

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

security-review

security-review: Found 1 issue (1 medium)

Co-authored-by: sentry-warden[bot] <258096371+sentry-warden[bot]@users.noreply.github.com>
The env block was incorrectly nested under on.pull_request where it
would be silently ignored. Move it to the job level so the environment
variables are properly available to all steps.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@sentry-warden
Copy link
Contributor

sentry-warden bot commented Jan 29, 2026

security-review

security-review: No issues found

No findings to report.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

GitHub Actions substitutes unconfigured secrets with empty strings.
The nullish coalescing operator only treats null/undefined as "not set",
so '' was being passed to the SDK as the model name, causing API errors.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@sentry-warden
Copy link
Contributor

sentry-warden bot commented Jan 29, 2026

security-review

security-review: Found 1 issue (1 info)

No findings to report.

@dcramer dcramer merged commit 597ace8 into main Jan 29, 2026
12 checks passed
@dcramer dcramer deleted the feat/cleanup-config branch January 29, 2026 23:15
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