WIP (expected no progress): CBG-5479: Add droid as available reviewer#8369
Draft
torcolvin wants to merge 6 commits into
Draft
WIP (expected no progress): CBG-5479: Add droid as available reviewer#8369torcolvin wants to merge 6 commits into
torcolvin wants to merge 6 commits into
Conversation
Collaborator
Author
|
@droid perform review |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Factory-AI “Droid” integration to the repository via a new GitHub Actions workflow and a .droid.yaml configuration file, enabling on-demand reviews when @droid is mentioned.
Changes:
- Introduces a new
.github/workflows/droid.ymlworkflow that triggers Droid runs based on issue/PR comment and review events. - Adds
.droid.yamlto configure review output and guideline/context behavior for Droid.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/droid.yml | Adds the GitHub Actions workflow that runs Droid on comment/review triggers, with concurrency + permissions. |
| .droid.yaml | Adds Droid configuration for review behavior and guideline/context documents. |
Comment on lines
+53
to
+55
| - name: Checkout repository | ||
| uses: actions/checkout@v5 | ||
| with: |
Comment on lines
+28
to
+44
| if: | | ||
| ( | ||
| (github.event_name == 'issue_comment' | ||
| && contains(github.event.comment.body, '@droid') | ||
| && !contains(github.event.comment.body, '@droid security')) || | ||
| (github.event_name == 'pull_request_review_comment' | ||
| && contains(github.event.comment.body, '@droid') | ||
| && !contains(github.event.comment.body, '@droid security')) || | ||
| (github.event_name == 'pull_request_review' | ||
| && contains(github.event.review.body, '@droid') | ||
| && !contains(github.event.review.body, '@droid security')) || | ||
| (github.event_name == 'issues' | ||
| && (contains(github.event.issue.body, '@droid') | ||
| || contains(github.event.issue.title, '@droid')) | ||
| && !contains(github.event.issue.body, '@droid security') | ||
| && !contains(github.event.issue.title, '@droid security')) | ||
| ) |
Comment on lines
+46
to
+51
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
| issues: write | ||
| id-token: write | ||
| actions: read |
Comment on lines
+23
to
+26
| guidelines: | ||
| - path: "*" | ||
| guideline: "Follow our ui-review.instructions.md style guide." | ||
| context_document_path: ".github/copilot-instructions.md" |
Collaborator
Author
|
Summoning @droid again |
Collaborator
Author
|
@droid is this present? |
Collaborator
Author
|
@droid do a review |
torcolvin
changed the title
torcolvin
changed the title
torcolvin
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CBG-0000
Describe your PR here...
Pre-review checklist
fmt.Print,log.Print, ...)base.UD(docID),base.MD(dbName))docs/apiDependencies (if applicable)
Integration Tests