Skip to content

Ai review test- First pass#257

Draft
cj-radcliff wants to merge 5 commits intomainfrom
ai-review-test
Draft

Ai review test- First pass#257
cj-radcliff wants to merge 5 commits intomainfrom
ai-review-test

Conversation

@cj-radcliff
Copy link
Collaborator

@cj-radcliff cj-radcliff commented Feb 18, 2026

I am attempting to get an agentic review cycle happening, and this is how Gemini suggested I do it. There are three critical files here:

  • the action: currently focusing only on branches called experimental-agent, or branches which start with ai-test/
  • the standards: In doc/ai-standards.md, I had gemini generate some starting points for Intellij plugins using java and kotlin. It should be seen as a very basic beginner set of guidelines that needs to be honed.
  • The script. This is the basic driver that calls gemini. It depends on a secret GEMINI_API key, which is held in the repo's secret store (not in code). It also depends on a variable called "HUMAN_REVIEWER" to assign to review the output

@cj-radcliff
Copy link
Collaborator Author

@pq I cancelled the DAS run as I don't think its necessary to pass on this workstream

@cj-radcliff
Copy link
Collaborator Author

@helin24 if i want to keep this around but not ready to accept it, what should I do? Close and re-open or leave open until I'm ready to merge?

@helin24
Copy link
Member

helin24 commented Mar 3, 2026

@cj-radcliff can you convert to a draft PR?

@cj-radcliff cj-radcliff marked this pull request as draft March 3, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants