Skip to content

feat: quick review v2#17

Merged
AntoMontagneDev merged 6 commits into
mainfrom
feat/quick-review-v2
Mar 5, 2026
Merged

feat: quick review v2#17
AntoMontagneDev merged 6 commits into
mainfrom
feat/quick-review-v2

Conversation

@AntoMontagneDev

@AntoMontagneDev AntoMontagneDev commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

🔦 description

What change is being made?

Add a new list_directory_at_commit tool and integrate it into quick and agentic review workflows, introduce a hallucination-filter step for quick reviews, implement robust JSON parsing utilities for agent responses, update prompts and tooling descriptions, and add tests and changelog/version updates.

Why are these changes being made?

To improve context awareness for LLM reviews by enabling repository structure queries at specific commits and to reduce hallucinations by muting investigation-like comments, while also improving robustness of agent response parsing and aligning prompts/tooling descriptions; note that this introduces new dependencies (Git tooling) and potential environment considerations.

@lampe-ci

lampe-ci Bot commented Mar 5, 2026

Copy link
Copy Markdown

N/A (Line 0): Cannot perform a diff-based, line-level review yet because a base commit SHA is not provided. The PR shows numerous added and modified files across multiple packages, but to give precise, line-numbered feedback I need the base commit (or a specific commit SHA) to fetch the diffs. Please provide the base commit SHA (or confirm the base branch/commit) so I can retrieve the exact diffs and produce targeted comments.

@AntoMontagneDev AntoMontagneDev changed the title Feat/quick review v2 feat: quick review v2 Mar 5, 2026
@AntoMontagneDev AntoMontagneDev merged commit cb8bcbb into main Mar 5, 2026
6 of 7 checks passed
@AntoMontagneDev AntoMontagneDev deleted the feat/quick-review-v2 branch March 6, 2026 08:47
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