[BACKLOG-46631] Ensured that the list of plugin directories is sorted

[peterrinehart]

before processing to ensure consistent behavior and that plugin folders with datestamps in the name (new plugin installs/updates) will be processed after those without (previous installs to be deleted). Also added an extra check after processing the plugin folder to ensure a plugin with the same ID hasn't already been processed. The second check should not be necessary after sorting the folder names, but this will prevent duplicate plugins from loading.

[Copilot]

Pull Request Overview

This PR implements sorted processing of plugin directories to ensure consistent behavior and prevent duplicate plugin loading. The main goal is to process plugin folders with datestamps after those without, while adding safeguards against duplicate plugin IDs.

• Sorts plugin directories before processing to handle datestamped folders consistently
• Adds duplicate plugin ID detection to prevent loading plugins with the same ID
• Enhanced logging for better debugging and monitoring of plugin processing

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
SystemPathXmlPluginProvider.java	Implements directory sorting, duplicate plugin detection, and enhanced logging
SystemPathXmlPluginProviderTest.java	Adds test coverage for duplicate plugin ID handling scenario

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[hitachivantarasonarqube]

Analysis Details

0 Issues

• 0 Bugs
• 0 Vulnerabilities
• 0 Code Smells

Coverage and Duplications

• 95.00% Coverage (35.00% Estimated after merge)
• 0.00% Duplicated Code (3.10% Estimated after merge)

Project ID: pentaho:pentaho-platform-ce-parent

View in SonarQube

[buildguy]

Note:

---

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

---

🐸 JFrog Frogbot

[buildguy]

✅ Build finished in 24m 16s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl extensions

---

👌 All tests passed!

Tests run: 1736, Failures: 0, Skipped: 1    Test Results

---

ℹ️ This is an automatic message