Fix the problem where extension point might not be obtained when the plugin was started

[JohnNiang]

What type of PR is this?

/kind bug
/area plugin
/area core

What this PR does / why we need it:

This PR refactors SpringComponentsFinder to prevent unexpected cleanup, which might lead to not be able to obtain plugin extension when the plugin was started.

The real reason is that entries is initialized by AttachmentReconciler during initialization of some plugins.

Please note the problem cannot be reproduced stably.

Refs:

• https://github.com/pf4j/pf4j/blob/e3125f2998197d6f866abd1eb77a922103c1cace/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java#L229-L249
• https://github.com/pf4j/pf4j/blob/e3125f2998197d6f866abd1eb77a922103c1cace/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java#L312-L316

Which issue(s) this PR fixes:

Fixes #5999

Does this PR introduce a user-facing change?

修复插件启动成功但仍然可能无法获取扩展点导致页面无法访问的问题

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.3% Duplication on New Code

See analysis details on SonarCloud

[codecov]

Codecov Report

Attention: Patch coverage is 77.27273% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 56.13%. Comparing base (5fdf6c0) to head (0c49766).
Report is 214 commits behind head on main.

Files	Patch %	Lines
...va/run/halo/app/plugin/SpringComponentsFinder.java	77.27%	3 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #6006      +/-   ##
============================================
- Coverage     56.91%   56.13%   -0.78%     
- Complexity     3319     3513     +194     
============================================
  Files           587      621      +34     
  Lines         18968    20914    +1946     
  Branches       1401     1460      +59     
============================================
+ Hits          10795    11740     +945     
- Misses         7594     8582     +988     
- Partials        579      592      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[guqing]

Note That:

此问题应该与这个重构有关 https://github.com/halo-dev/halo/pull/5251/files#diff-a66b8608c4a9c737903ad1182ab8a33a73b2ebfce26352f8205db82be28667bdL284-L291

相关片段

 private void checkExtensionFinderReady(PluginWrapper pluginWrapper) {
    if (extensionFinder instanceof SpringComponentsFinder springComponentsFinder) {
        springComponentsFinder.readPluginStorageToMemory(pluginWrapper);
        return;
    }
    // should never happen
    throw new PluginRuntimeException("Plugin component classes may not loaded yet.");
}
private void removePluginComponentsCache(String pluginId) {
    if (extensionFinder instanceof SpringComponentsFinder springComponentsFinder) {
        springComponentsFinder.removeComponentsStorage(pluginId);
    }
}

[guqing]

/approve

[ruibaby]

/lgtm

[f2c-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: guqing, ruibaby

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [guqing,ruibaby]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment