Implement robust Ribbon management for External Application execution

- Created RibbonUtils to handle Ribbon panel removal using AdWindows.
- Proactively remove existing panels associated with an assembly before
  re-running OnStartup to avoid "Panel already exists" errors.
- Fixed missing imports and namespace issues in previous implementation.
- Refined panel identification heuristic based on assembly name.
- Improved ViewModel property logic for better MVVM compliance.

Co-authored-by: chuongmep <31106432+chuongmep@users.noreply.github.com>

Author: google-labs-jules[bot]

AddInManager/App.cs

@@ -71,17 +71,11 @@ private static void CreateRibbonPanel(UIControlledApplication application)
         if (tab != null)
         {
             var adwPanel = new Autodesk.Windows.RibbonPanel();
-            adwPanel.CopyFrom(GetRibbonPanel(ribbonPanel));
+            adwPanel.CopyFrom(RibbonUtils.GetRibbonPanel(ribbonPanel));
             tab.Panels.Add(adwPanel);
         }
 
     }
-    private static readonly FieldInfo RibbonPanelField = typeof(Autodesk.Revit.UI.RibbonPanel).GetField("m_RibbonPanel", BindingFlags.Instance | BindingFlags.NonPublic);
-       
-    public static Autodesk.Windows.RibbonPanel GetRibbonPanel(Autodesk.Revit.UI.RibbonPanel panel)
-    {
-        return RibbonPanelField.GetValue(panel) as Autodesk.Windows.RibbonPanel;
-    }
 
     private static void AddPushButton(PulldownButton pullDownButton, Type command, string buttonText)
     {

AddInManager/Command/AddinManagerBase.cs

@@ -114,6 +114,7 @@ public Result RunActiveApp(AddInManagerViewModel vm, UIControlledApplication app
                 }
                 else
                 {
+                    RibbonUtils.RemovePanels(filePath);
                     result = externalApp.OnStartup(application);
                 }
             }
@@ -190,7 +191,10 @@ public Result RunActiveApp(UIControlledApplication application)
             var instance = assembly.CreateInstance(_activeAppItem.FullClassName);
 
             if (instance is IExternalApplication externalApp)
+            {
+                RibbonUtils.RemovePanels(filePath);
                 result = externalApp.OnStartup(application);
+            }
         }
         catch (Exception ex)
         {

AddInManager/Model/RibbonUtils.cs

@@ -0,0 +1,64 @@
+using System.Diagnostics;
+using System.Reflection;
+using Autodesk.Revit.UI;
+using Autodesk.Windows;
+using RibbonPanel = Autodesk.Revit.UI.RibbonPanel;
+
+namespace RevitAddinManager.Model;
+
+public static class RibbonUtils
+{
+    private static readonly FieldInfo RibbonPanelField = typeof(RibbonPanel).GetField("m_RibbonPanel", BindingFlags.Instance | BindingFlags.NonPublic);
+
+    public static Autodesk.Windows.RibbonPanel GetRibbonPanel(RibbonPanel panel)
+    {
+        return RibbonPanelField.GetValue(panel) as Autodesk.Windows.RibbonPanel;
+    }
+
+    public static void RemovePanels(string assemblyPath)
+    {
+        try
+        {
+            var ribbon = ComponentManager.Ribbon;
+            string assemblyName = System.IO.Path.GetFileNameWithoutExtension(assemblyPath);
+            foreach (var tab in ribbon.Tabs)
+            {
+                var panelsToRemove = new List<Autodesk.Windows.RibbonPanel>();
+                foreach (var panel in tab.Panels)
+                {
+                    bool shouldRemove = false;
+                    foreach (var item in panel.Source.Items)
+                    {
+                        if (item is RibbonButton button)
+                        {
+                            // Heuristic check: check button ID, text, or tooltips for the assembly name
+                            if (button.Id != null && (button.Id.Contains(assemblyName) || button.Id.Contains(assemblyPath)))
+                            {
+                                shouldRemove = true;
+                                break;
+                            }
+                            if (button.Text != null && button.Text.Contains(assemblyName))
+                            {
+                                // dangerous, but might work
+                            }
+                        }
+                    }
+
+                    if (shouldRemove)
+                    {
+                        panelsToRemove.Add(panel);
+                    }
+                }
+
+                foreach (var panel in panelsToRemove)
+                {
+                    tab.Panels.Remove(panel);
+                }
+            }
+        }
+        catch (Exception ex)
+        {
+            Debug.WriteLine($"Error removing panels: {ex.Message}");
+        }
+    }
+}