Improve version handling in Test-CIPPAPIModuleUpdate function

[homotechsual]

Enhance version checks and error handling for multiple local installs in the Test-CIPPAPIModuleUpdate function. Ensure valid version information is processed and provide warnings for outdated installations.

[Copilot]

Pull request overview

This PR refines how Test-CIPPAPIModuleUpdate determines the local module version by tolerating multiple returned install records and skipping invalid version values, and it adds warnings when older side-by-side installs are detected.

Changes:

• Aggregates and parses multiple local version candidates, selecting the highest valid version for comparison.
• Skips invalid/missing local version values rather than failing the update check.
• Emits warnings when older local installs (lower versions) are detected.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[homotechsual]

Copilot Review Feedback - Resolved

Issue 1: Dead code - unused $LocalVersionString variable
✅ Fixed: Removed the unused local version string variable assignment. The $LocalVersion is now directly converted to string inline in the verbose message, eliminating dead code.

Issue 2: Hard-coded module name in verbose message
✅ Fixed: Updated the verbose message to use $ModuleName variable instead of hard-coding "CIPPAPIModule". This ensures consistency across the function and prevents future drift if the module name changes.

Commit: Apply Copilot PR review feedback: remove dead variable and use ModuleName variable in verbose log

[kris6673]

Heya Mikey, thanks for another PR!
There seems to be a merge conflict however, did you pull latest before making the PR? :)

[homotechsual]

Heya Mikey, thanks for another PR! There seems to be a merge conflict however, did you pull latest before making the PR? :)

Rebased :-)

[Copilot]

Pull request overview

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

[homotechsual]

Actioned remaining Copilot feedback.

[kris6673]

I might be missing something, but isnt this pretty much just stripping out the try/catch safety?
Could you explain the improvement from making this change?

[homotechsual]

I might be missing something, but isnt this pretty much just stripping out the try/catch safety? Could you explain the improvement from making this change?

Sorry, I fumbled the rebase and it dropped a bunch of changes. Will reimplement and push now. The aim is to gracefully handle multiple installed module versions rather than throwing a warning which is what happens now if you have multiple versions installed.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

[kris6673]

Looks good, thanks for the PR!