Skip to content

Add opt-in vendor exclusion to class-name collision inspection - #231

Draft
funivan with Copilot wants to merge 13 commits into
masterfrom
copilot/add-option-to-ignore-vendor-classes
Draft

Add opt-in vendor exclusion to class-name collision inspection#231
funivan with Copilot wants to merge 13 commits into
masterfrom
copilot/add-option-to-ignore-vendor-classes

Conversation

Copilot AI commented Aug 30, 2026

Copy link
Copy Markdown

Class name collision inspection currently treats vendor-defined classes as equal collision candidates, which creates noise for application code. This change adds an opt-in mode to ignore collisions originating from the project’s vendor tree.

  • Inspection configuration

    • Added ignoreVendorClasses boolean option to ClassNameCollisionInspection (default false).
    • Exposed in inspection UI via a checkbox: Ignore vendor classes.
  • Collision filtering behavior

    • Collision lookup now excludes vendor classes only when the option is enabled.
    • Vendor detection is VFS-based and scoped to the project’s vendor directory (strict ancestry check), avoiding raw-path heuristics.
  • Coverage updates

    • Added focused tests for:
      • vendor collision ignored when option is enabled
      • vendor collision still reported when option is disabled
  • Inspection docs

    • Updated inspection help text to describe the new option and its effect.
@JvmField
var ignoreVendorClasses = false

override fun createOptionsPanel(): JComponent =
    SingleCheckboxOptionsPanel("Ignore vendor classes", this, "ignoreVendorClasses")

Copilot AI linked an issue Aug 30, 2026 that may be closed by this pull request
Copilot AI and others added 12 commits August 30, 2026 15:57
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Co-authored-by: funivan <425208+funivan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add option to ignore vendor classes for name collision inspection Add opt-in vendor exclusion to class-name collision inspection Aug 30, 2026
Copilot AI requested a review from funivan August 30, 2026 16:04
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.

[name collision] do not check vendor classes

2 participants