Skip to content

ci: Set up Warden PR review#4

Merged
gricha merged 3 commits intomainfrom
ci/warden-setup
Feb 14, 2026
Merged

ci: Set up Warden PR review#4
gricha merged 3 commits intomainfrom
ci/warden-setup

Conversation

@gricha
Copy link
Member

@gricha gricha commented Feb 14, 2026

Add Warden automated code review for pull requests.

Rewrites warden.toml from the old [[triggers]] format to the current [[skills]] schema and adds a GitHub Actions workflow that runs getsentry/warden@v1 with the org GitHub App token for branded review comments.

Skills configured:

  • notseer — bug finding on src/**/*.ts (excluding tests)
  • security-review — security vulnerability scanning via getsentry/sentry-skills
  • warden-lint-judge — commented out pending feat: multi-pass skill pipeline warden#144 (multi-pass pipeline with scope = "report")

Relies on org-level secrets: WARDEN_APP_ID, WARDEN_PRIVATE_KEY, WARDEN_ANTHROPIC_API_KEY (via the app token flow), and optional WARDEN_MODEL / WARDEN_SENTRY_DSN.

gricha and others added 3 commits February 13, 2026 22:01
Add GitHub Actions workflow and update warden.toml to use the current
skills schema. Configures notseer and security-review skills on PR
events targeting src/**/*.ts. warden-lint-judge is commented out
pending getsentry/warden#144.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@gricha gricha marked this pull request as ready for review February 14, 2026 06:10
@gricha gricha merged commit 0030acf into main Feb 14, 2026
4 checks passed
@gricha gricha deleted the ci/warden-setup branch February 14, 2026 06:10
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.

name: Warden

permissions:
contents: write
Copy link

Choose a reason for hiding this comment

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

Workflow grants unnecessary contents: write permission

Medium Severity

The workflow declares contents: write at the top level, but a PR review workflow only needs to read repository contents and post review comments. Granting contents: write allows the workflow (and any actions it invokes, including the third-party getsentry/warden action) to push commits, create/delete branches, and modify repository contents. Narrowing this to contents: read follows the principle of least privilege and limits blast radius if any action is compromised.

Fix in Cursor Fix in Web

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