Skip to content

ci: Integrate Warden for AI-powered PR code review#3651

Merged
buenaflor merged 3 commits intomainfrom
buenaflor/ci/integrate-warden
Apr 21, 2026
Merged

ci: Integrate Warden for AI-powered PR code review#3651
buenaflor merged 3 commits intomainfrom
buenaflor/ci/integrate-warden

Conversation

@buenaflor
Copy link
Copy Markdown
Contributor

📜 Description

Adds warden.toml at the repo root to enable Warden AI-powered PR reviews. The org-wide workflow in getsentry/.github already runs Warden on all repos — this config file activates it for sentry-dart.

Skills enabled:

  • code-review — AI code review
  • find-bugs — bug detection
  • security-review — security analysis
  • gha-security-review — GitHub Actions security review

Also adds the warden skill to agents.toml for local agent usage.

💡 Motivation and Context

Per the SDK Review and CI standards, SDK repositories should have Warden configured in CI.

See reference rollout in sentry-react-native#6003.

💚 How did you test it?

Warden will run on this PR via the org-wide workflow. Org-level secrets (WARDEN_APP_ID, WARDEN_PRIVATE_KEY, WARDEN_ANTHROPIC_API_KEY, WARDEN_MODEL, WARDEN_SENTRY_DSN) are picked up automatically — no repo-level configuration needed.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
  • I updated the docs if needed
  • All tests passing
  • No breaking changes

🔮 Next steps

Add warden.toml at the repo root to enable Warden AI-powered PR reviews.
The org-wide workflow in getsentry/.github already runs Warden on all
repos — this config activates it for sentry-dart.

Enabled skills: code-review, find-bugs, security-review,
gha-security-review.

Also add the warden skill to agents.toml for local agent usage.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 20, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Internal Changes

Deps

  • Bump actions/create-github-app-token from 3.0.0 to 3.1.1 by dependabot in #3652
  • Bump getsentry/craft/.github/workflows/changelog-preview.yml from 2.25.2 to 2.25.4 by dependabot in #3655
  • Bump actions/cache from 5.0.4 to 5.0.5 by dependabot in #3656

Other

  • Integrate Warden for AI-powered PR code review by buenaflor in #3651

🤖 This preview updates automatically when you update the PR.

@buenaflor buenaflor marked this pull request as ready for review April 20, 2026 14:38
@buenaflor buenaflor requested a review from denrase as a code owner April 20, 2026 14:38
Copilot AI review requested due to automatic review settings April 20, 2026 14:38
Comment thread agents.toml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds repository-level configuration to enable Warden’s org-wide AI PR review workflow for sentry-dart, and wires the Warden skill into local agent tooling.

Changes:

  • Add warden.toml at the repo root to activate Warden and configure review thresholds + PR triggers.
  • Add the warden skill to agents.toml for local agent usage.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
warden.toml Enables Warden with configured defaults and PR-triggered skills (code-review, find-bugs, security-review, gha-security-review).
agents.toml Adds the warden skill (getsentry/warden) to the local agent skills list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

The warden skill sources from getsentry/warden, so add the repo to the
trust list alongside the other skill source repos.

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@denrase denrase left a comment

Choose a reason for hiding this comment

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

One question/suggestion

Comment thread warden.toml
actions = ["opened", "synchronize", "reopened"]

[[skills]]
name = "gha-security-review"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should we scope this so it only runs when gha change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Copy Markdown

@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.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 95e3c99. Configure here.

Comment thread warden.toml
@buenaflor buenaflor merged commit 107a26b into main Apr 21, 2026
13 checks passed
@buenaflor buenaflor deleted the buenaflor/ci/integrate-warden branch April 21, 2026 12:10
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.

3 participants