Add file exclusion patterns for AI Assistant PR review #2703
Václav Kozák opened 4 months ago

Problem

When using AI Assistant to review pull requests, it currently reviews all changed files including auto-generated files like:

  • pnpm-lock.yaml / package-lock.json / yarn.lock
  • EF Core migration snapshots (**/ApplicationContextModelSnapshot.cs)
  • EF Core migration Designer files (**/*Designer.cs)
  • Other generated code

This wastes AI tokens and produces unhelpful review comments on files that shouldn't be manually reviewed.

Proposed Solution

Add a configuration option (similar to "Code Analysis Setting" → "Analysis Files") that allows specifying file patterns to exclude from AI Assistant review.

Example patterns:

-**/pnpm-lock.yaml
-**/package-lock.json
-**/*Designer.cs
-**/Migrations/*.cs
-**/*.generated.cs

Expected Behavior

Files matching exclusion patterns should be skipped (not sent to the model provider at all) when AI Assistant reviews commits or pull requests.

  • OneDev commented 4 months ago

    State changed as build OD-7134 is successful

  • OneDev changed state to 'Closed' 4 months ago
    Previous Value Current Value
    Open
    Closed
1/1
Type
Improvement
Priority
Major
Assignee
Labels
No labels
Issue Votes (0)
Watchers (3)
Reference
OD-2703
Please wait...
Connection lost or session expired, reload to recover
Page is in error, reload to recover