Skip to content

Add coverage for hooks-only plugin discovery and Local inventory refresh - #347186

Draft
Martin Aeschlimann (aeschli) with Copilot wants to merge 2 commits into
mainfrom
copilot/local-hooks-empty-issue
Draft

Add coverage for hooks-only plugin discovery and Local inventory refresh#347186
Martin Aeschlimann (aeschli) with Copilot wants to merge 2 commits into
mainfrom
copilot/local-hooks-empty-issue

Conversation

Copilot AI commented Nov 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Local Hooks was reported empty for an enabled hooks-only plugin until installing another plugin refreshed discovery.

  • Discovery coverage: Check that asynchronously loaded plugin hooks invalidate the cached inventory.
  • Local inventory coverage: Check that hook-only change notifications refresh rows and counts without another contribution changing.
  • Scope: Test-only changes. The reported failure was not reproduced; no production fix is included.

Copilot AI balanced review requested due to automatic review settings November 9, 2026 21:31
Copilot AI reviewed Nov 9, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

Copilot AI linked an issue Nov 9, 2026 that may be closed by this pull request
Local Hooks list stays empty for a hooks-only plugin until another plugin refreshes discovery #344813
Open
@aeschli
test: cover hooks-only plugin discovery and Local inventory refresh
17076ae 
Co-authored-by: aeschli <6461412+aeschli@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix empty Local Hooks list for hooks-only plugin Add coverage for hooks-only plugin discovery and Local inventory refresh Nov 9, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@aeschli Martin Aeschlimann (aeschli) Awaiting requested review from aeschli

At least 1 approving review is required to merge this pull request.

Assignees

@aeschli Martin Aeschlimann (aeschli)

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Local Hooks list stays empty for a hooks-only plugin until another plugin refreshes discovery

3 participants

@aeschli