Skip to content

codediff: use new agentic client instead of gemini-cli#534

Merged
academo merged 3 commits intomainfrom
academo/use-agentic-client-for-code-diff
Mar 10, 2026
Merged

codediff: use new agentic client instead of gemini-cli#534
academo merged 3 commits intomainfrom
academo/use-agentic-client-for-code-diff

Conversation

@academo
Copy link
Collaborator

@academo academo commented Mar 9, 2026

It uses the recently introduced agentic client instead of the gemini-cli to perform the code-diff

@academo academo requested review from a team as code owners March 9, 2026 12:43
@academo academo requested review from Ukochka, andresmgot and sunker March 9, 2026 12:43
@academo academo requested a review from s4kh March 9, 2026 12:43
@academo academo self-assigned this Mar 9, 2026
@academo academo requested review from tolzhabayev and xnyo March 9, 2026 12:43
@academo academo moved this from 📬 Triage to 🔬 In review in Grafana Catalog Team Mar 9, 2026
Copy link
Contributor

@mckn mckn left a comment

Choose a reason for hiding this comment

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

LGTM! Great work on this

// so the agent doesn't lose track of what it's comparing across sequential questions.
func buildQuestionWithContext(question, currentCommit, newCommit string) string {
return fmt.Sprintf(
"You are comparing changes between commits %s and %s. Feel free to use git diff and git show to understand what changed. If a change looks significant, compare full file versions to understand context. %s.\n First verify in which current git ref you are",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"You are comparing changes between commits %s and %s. Feel free to use git diff and git show to understand what changed. If a change looks significant, compare full file versions to understand context. %s.\n First verify in which current git ref you are",
"You are comparing changes between commits %s and %s. Feel free to use git diff and git show to understand what changed. If a change looks significant, compare full file versions to understand context. %s.\n First verify in which current git ref you are on.",

@academo academo merged commit c36a624 into main Mar 10, 2026
9 checks passed
@academo academo deleted the academo/use-agentic-client-for-code-diff branch March 10, 2026 08:09
@github-project-automation github-project-automation bot moved this from 🔬 In review to 🚀 Shipped in Grafana Catalog Team Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚀 Shipped

Development

Successfully merging this pull request may close these issues.

2 participants