Copilot code review: Previously missed findings should be posted as inline comments #208330
Replies: 5 comments 1 reply
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
|
I agree with the above, and would also include "Suppressed findings". I would like to be able to promote any/all feedback that Copilot has to individual feedback thread comments. As it is currently, trying to find all feedback is difficult. |
|
I agree that the distinction between “finding exists” and “finding is surfaced where the developer is already working” matters a lot. For automated PR review, I’d treat findings as part of a lifecycle: detected → surfaced → acknowledged → resolved/suppressed If a finding only exists in an overview summary, it can easily disappear from the developer’s normal review flow. Inline comments make the finding actionable because they preserve the connection between the issue and the exact code location. It would also be useful to distinguish new findings, previously missed findings, and suppressed findings while keeping all three accessible from the PR review. |
|
I've been experiencing this as well. Copilot reviews after the first iteration typically contain multiple items being introduced in "Previously missed" (they were never previously reported), and none of them include links to a comment that I can address/decline. This causes an accumulation of findings lacking means to resolve them; effectively paralyzing the Copilot review. |
|
I’d also like to share an observation from working with external AI agents and Copilot. The way these issues are currently presented is particularly confusing to AI agents. Claude (Opus and Fable) and Grok have all repeatedly overlooked review comments and even pushed back on them, incorrectly asserting that the issues had already been resolved. Given how heavily engineers now rely on AI agents in their development workflows, I think it’s increasingly important to make review feedback as clear, unambiguous, and agent-friendly as possible. |
Uh oh!
There was an error while loading. Please reload this page.
🏷️ Discussion Type
Product Feedback
💬 Feature/Topic Area
Copilot in GitHub
Body
The new Previously missed section is easy to overlook because those findings are shown only in the overview comment and are not posted as inline review comments.
These are still actionable, unresolved findings, so developers who review unresolved conversations or inline comments can easily miss them.
Please either:
At minimum, Previously missed findings should be surfaced as prominently as Open findings.
All reactions