Fix NullReferenceException in NUIGadgetManager

[upple]

Description of Change

• Wrap UIGadgetInfo.CreateUIGadgetInfo with try/catch and log packageId on exception
• Add try/catch in UIGadgetManager static constructor to prevent TypeInitializationException
• Add try/catch in foreach loop to continue loading other packages on failure
• Add try/catch in UIGadgetManager.Refresh() with packageId in error logs
• Add try/catch in NUIGadgetManager static constructor, LoadGadgetInfos, and Refresh

API Changes

• ACR: N/A

[gemini-code-assist]

Code Review

This pull request enhances the robustness of the UIGadget components by implementing extensive try-catch blocks, improved null safety for application paths, and more detailed logging during initialization and resource loading. Feedback focuses on preventing a potential native resource leak in CreateUIGadgetInfo using a finally block, adopting safer casting for CoreApplication.Current, and removing several redundant null checks and exception handlers to streamline the implementation.

[upple]

@hjhun
Thank you for the reveiw.
Please review again.

[JoonghyunCho]

🤖 [AI Review]

Reviewed — no findings.

Scope checked:

• UIGadgetInfo.CreateUIGadgetInfo refactored to try/finally — fixes the native-handle leak when any of the SetResourceType / SetResourceVersion / SetMetadata calls throw. PackageInfoDestroy(handle) now runs unconditionally; handle remains valid until finally because the unmanaged calls in the try body don't depend on it being already destroyed.
• Moving SetExecutableFile / SetResourceFile / SetResourceClassName / SetResourcePath / SetUIGadgetResourcePath inside the try block is benign — they take only info, not handle, so timing relative to PackageInfoDestroy is irrelevant.
• UIGadgetManager widens catch (ArgumentNullException || OverflowException) to catch (Exception) around per-package CreateUIGadgetInfo calls — defensible in a foreach-over-packages loop where one bad package should not stop the others; each failure is logged with packageId for diagnosis. No callers above rely on specific exception types here.
• NUIGadgetManager.LoadGadgetInfos drops the redundant if (gadgetInfo != null) check — new NUIGadgetInfo(info) either throws or returns non-null, so the guard was unreachable.
• No public API changes; all touched types/members are internal or non-public-surface — no XML-doc obligations.

No 🔴 critical issues, no 🟡 suggestions to flag.

---

Automated review — final merge decision rests with human reviewers.