What's Changed
Security review is on by default with automatic review
- GitHub: an omitted or empty
automatic_security_reviewnow followsautomatic_review(#172)- Workflows with
automatic_review: truethat don't setautomatic_security_reviewnow get a security review on every PR. Setautomatic_security_review: falseto opt out - Workflows that set neither input are unchanged and stay mention-driven (
@droid review,@droid security)
- Workflows with
- GitLab:
automatic_security_reviewin the template now defaults to"true"(#173)- Projects that include
templates/droid-review.ymlwithout setting the input now run the security-reviewer subagent on every MR pipeline.[security]findings are posted alongside code-review comments, and the tracking note shows the security badge - Security review only runs when
automatic_reviewis enabled. Ifautomatic_reviewis"false", the job still skips - To keep code-only review, set
automatic_security_review: "false"
- Projects that include
- The GitLab example, the GitLab setup guide and the
install-code-reviewinstaller now recommend security review (Factory-AI/factory-mono#27295)
GitHub App tokens carry only the permissions each run needs
- The action now asks the token exchange for a narrowed Factory Droid App token instead of every permission the app holds (#181)
- Reviews, security reviews and fills on a PR get
contents: readandpull_requests: write.@droidon an issue getscontents: readandissues: write.@droid security --fulland CI Steward getcontents: writeonly when they need it - Nothing asks for
workflows,checks,statuses,discussionsoractions: write - Unchanged: the
prepare/andreview-guidelines/sub-actions still get the full token, and a token you pass ingithub_tokenis used as is - The backend side (Factory-AI/factory-mono#28048) is rolling out. Until it reaches production, the exchange ignores the request and keeps returning a full token, so runs are not affected either way
- Reviews, security reviews and fills on a PR get
New: Azure DevOps pull request review
- Droid code review for Azure DevOps Services and Azure DevOps Server 2020+, with the same two-pass review, inline comments, tracking thread and optional security review as GitLab (#171)
- Credentials are redacted from debug artifacts, threads and errors, reusing the protections from #174 (#177)
- New
REVIEW_DRAFTSinput (#171) andFORK_PULL_REQUESTSinput. SetFORK_PULL_REQUESTS: failso fork PRs fail the build instead of passing a Required build policy without a review (#179)
New: Bitbucket Cloud and Data Center review
- Droid code review for Bitbucket Cloud and Bitbucket Data Center / Server, with inline comments, a sticky tracking comment, optional security review, review depth and an optional
droid-reviewbuild status (BUILD_STATUS=true) for merge checks (#170) - A skipped draft PR no longer posts a successful build status that could satisfy a merge check (#180)
Review depth
review_depth: shallownow usesoss-latest-premiuminstead ofoss-latest-balanced(#184).deepis unchanged. CLIs that predate tier aliases fall back tokimi-k3
Fixes
- GitLab: the review template installs
procps, so deep reviews on large MRs no longer crash withExecutable not found in $PATH: "ps"(#178)
Full Changelog: v11...v12