From e53dc0a0debcfc1ad8f98ba65c8af17981abaa0c Mon Sep 17 00:00:00 2001
From: BrandonPacewic <92102436+BrandonPacewic@users.noreply.github.com>
Date: Wed, 21 Aug 2024 10:31:42 -0700
Subject: [PATCH 1/4] Remove unused code

---
 exporter/SynthesisFusionAddin/src/Types.py    |   3 +
 .../SynthesisFusionAddin/src/UI/Camera.py     |   2 +-
 .../src/UI/ConfigCommand.py                   | 258 ++----------------
 .../src/UI/Configuration/SerialCommand.py     | 104 -------
 .../SynthesisFusionAddin/src/UI/Handlers.py   | 107 --------
 5 files changed, 21 insertions(+), 453 deletions(-)
 delete mode 100644 exporter/SynthesisFusionAddin/src/UI/Configuration/SerialCommand.py

diff --git a/exporter/SynthesisFusionAddin/src/Types.py b/exporter/SynthesisFusionAddin/src/Types.py
index 5e2d7c967..82169e0f5 100644
--- a/exporter/SynthesisFusionAddin/src/Types.py
+++ b/exporter/SynthesisFusionAddin/src/Types.py
@@ -5,6 +5,8 @@
 from enum import Enum, EnumType
 from typing import Any, TypeAlias, get_args, get_origin
 
+import adsk.fusion
+
 # Not 100% sure what this is for - Brandon
 JointParentType = Enum("JointParentType", ["ROOT", "END"])
 
@@ -85,6 +87,7 @@ def toKg(pounds: LBS) -> KG:
 
 
 PRIMITIVES = (bool, str, int, float, type(None))
+SELECTABLE_JOINT_TYPES = (adsk.fusion.JointTypes.RevoluteJointType, adsk.fusion.JointTypes.SliderJointType)
 
 
 def encodeNestedObjects(obj: Any) -> Any:
diff --git a/exporter/SynthesisFusionAddin/src/UI/Camera.py b/exporter/SynthesisFusionAddin/src/UI/Camera.py
index 53168fd24..4b8306268 100644
--- a/exporter/SynthesisFusionAddin/src/UI/Camera.py
+++ b/exporter/SynthesisFusionAddin/src/UI/Camera.py
@@ -9,7 +9,7 @@
 
 
 @logFailure
-def captureThumbnail(size: int = 250) -> str | os.PathLike[str]:
+def captureThumbnail(size: int = 250) -> str:
     """
     ## Captures Thumbnail and saves it to a temporary path - needs to be cleared after or on startup
     - Size: int (Default: 200) : (width & height)
diff --git a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
index cbdd5f188..b95d6ecbb 100644
--- a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
+++ b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
@@ -2,9 +2,6 @@
     links the Configuration Command seen when pressing the Synthesis button in the Addins Panel
 """
 
-import os
-import pathlib
-from enum import Enum
 from typing import Any
 
 import adsk.core
@@ -15,70 +12,22 @@
 from src.Logging import getLogger, logFailure
 from src.Parser.ExporterOptions import ExporterOptions
 from src.Parser.SynthesisParser.Parser import Parser
-from src.Types import ExportLocation, ExportMode
+from src.Types import SELECTABLE_JOINT_TYPES, ExportLocation, ExportMode
 from src.UI import FileDialogConfig
-from src.UI.Configuration.SerialCommand import SerialCommand
 from src.UI.GamepieceConfigTab import GamepieceConfigTab
 from src.UI.GeneralConfigTab import GeneralConfigTab
 from src.UI.JointConfigTab import JointConfigTab
 
-# ====================================== CONFIG COMMAND ======================================
-
 generalConfigTab: GeneralConfigTab
 jointConfigTab: JointConfigTab
 gamepieceConfigTab: GamepieceConfigTab
 
 logger = getLogger()
 
-"""
-INPUTS_ROOT (adsk.fusion.CommandInputs):
-    - Provides access to the set of all commandInput UI elements in the panel
-"""
-INPUTS_ROOT = None
-
-
-# Transition: AARD-1765
-# This should be removed in the config command refactor. Seemingly impossible to type.
-def GUID(arg: str | adsk.core.Base) -> str | adsk.core.Base:
-    """### Will return command object when given a string GUID, or the string GUID of an object (depending on arg value)
-
-    Args:
-        arg str | object: Either a command input object or command input GUID
-
-    Returns:
-        str | object: Either a command input object or command input GUID
-    """
-    if type(arg) == str:
-        object = gm.app.activeDocument.design.findEntityByToken(arg)[0]
-        return object
-    else:  # type(obj)
-        return arg.entityToken  # type: ignore[union-attr]
-
-
-class JointMotions(Enum):
-    """### Corresponds to the API JointMotions enum
-
-    Args:
-        Enum (enum.Enum)
-    """
-
-    RIGID = 0
-    REVOLUTE = 1
-    SLIDER = 2
-    CYLINDRICAL = 3
-    PIN_SLOT = 4
-    PLANAR = 5
-    BALL = 6
+INPUTS_ROOT: adsk.core.CommandInputs
 
 
 class ConfigureCommandCreatedHandler(adsk.core.CommandCreatedEventHandler):
-    """### Start the Command Input Object and define all of the input groups to create our ParserOptions object.
-
-    Notes:
-        - linked and called from (@ref HButton) and linked
-        - will be called from (@ref Events.py)
-    """
-
     def __init__(self, configure: Any) -> None:
         super().__init__()
 
@@ -87,22 +36,16 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         exporterOptions = ExporterOptions().readFromDesign() or ExporterOptions()
         cmd = args.command
 
-        # Set to false so won't automatically export on switch context
         cmd.isAutoExecute = False
         cmd.isExecutedWhenPreEmpted = False
-        cmd.okButtonText = "Export"  # replace default OK text with "export"
-        cmd.setDialogInitialSize(400, 350)  # these aren't working for some reason...
-        cmd.setDialogMinimumSize(400, 350)  # these aren't working for some reason...
+        cmd.okButtonText = "Export"
+        cmd.setDialogSize(800, 350)
 
-        global INPUTS_ROOT  # Global CommandInputs arg
+        global INPUTS_ROOT
         INPUTS_ROOT = cmd.commandInputs
 
-        # ~~~~~~~~~~~~~~~~ HELP FILE ~~~~~~~~~~~~~~~~
-        """
-        Sets the small "i" icon in bottom left of the panel.
-            - This is an HTML file that has a script to redirect to exporter workflow tutorial.
-        """
-        cmd.helpFile = os.path.join(".", "src", "Resources", "HTML", "info.html")
+        # TODO?
+        # cmd.helpFile = os.path.join(".", "src", "Resources", "HTML", "info.html")
 
         global generalConfigTab
         generalConfigTab = GeneralConfigTab(args, exporterOptions)
@@ -139,10 +82,7 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
                 *gm.app.activeDocument.design.rootComponent.allJoints,
                 *gm.app.activeDocument.design.rootComponent.allAsBuiltJoints,
             ]:
-                if (
-                    joint.jointMotion.jointType in (JointMotions.REVOLUTE.value, JointMotions.SLIDER.value)
-                    and not joint.isSuppressed
-                ):
+                if joint.jointMotion.jointType in SELECTABLE_JOINT_TYPES and not joint.isSuppressed:
                     jointConfigTab.addJoint(joint)
 
         # Adding saved wheels must take place after joints are added as a result of how the two types are connected.
@@ -154,75 +94,6 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
                 if len(fusionJoints):
                     jointConfigTab.addWheel(fusionJoints[0], wheel)
 
-        # ~~~~~~~~~~~~~~~~ JOINT SETTINGS ~~~~~~~~~~~~~~~~
-        """
-        Joint settings group command
-        """
-
-        # Transition: AARD-1689
-        # Should possibly be implemented later?
-
-        # jointsSettings = a_input.addGroupCommandInput(
-        #     "joints_settings", "Joints Settings"
-        # )
-        # jointsSettings.isExpanded = False
-        # jointsSettings.isEnabled = True
-        # jointsSettings.tooltip = "tooltip"  # TODO: update tooltip
-        # joints_settings = jointsSettings.children
-
-        # self.createBooleanInput(
-        #     "kinematic_only",
-        #     "Kinematic Only",
-        #     joints_settings,
-        #     checked=False,
-        #     tooltip="tooltip",  # TODO: update tooltip
-        #     enabled=True,
-        # )
-
-        # self.createBooleanInput(
-        #     "calculate_limits",
-        #     "Calculate Limits",
-        #     joints_settings,
-        #     checked=True,
-        #     tooltip="tooltip",  # TODO: update tooltip
-        #     enabled=True,
-        # )
-
-        # self.createBooleanInput(
-        #     "auto_assign_ids",
-        #     "Auto-Assign ID's",
-        #     joints_settings,
-        #     checked=True,
-        #     tooltip="tooltip",  # TODO: update tooltip
-        #     enabled=True,
-        # )
-
-        # ~~~~~~~~~~~~~~~~ CONTROLLER SETTINGS ~~~~~~~~~~~~~~~~
-        """
-        Controller settings group command
-        """
-
-        # Transition: AARD-1689
-        # Should possibly be implemented later?
-
-        # controllerSettings = a_input.addGroupCommandInput(
-        #     "controller_settings", "Controller Settings"
-        # )
-
-        # controllerSettings.isExpanded = False
-        # controllerSettings.isEnabled = True
-        # controllerSettings.tooltip = "tooltip"  # TODO: update tooltip
-        # controller_settings = controllerSettings.children
-
-        # self.createBooleanInput(  # export signals checkbox
-        #     "export_signals",
-        #     "Export Signals",
-        #     controller_settings,
-        #     checked=True,
-        #     tooltip="tooltip",
-        #     enabled=True,
-        # )
-
         getAuth()
         user_info = getUserInfo()
         apsSettings = INPUTS_ROOT.addTabCommandInput(
@@ -230,66 +101,37 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         )
         apsSettings.tooltip = "Configuration settings for Autodesk Platform Services."
 
-        # clear all selections before instantiating handlers.
         gm.ui.activeSelections.clear()
-
-        # ====================================== EVENT HANDLERS ======================================
-        """
-        Instantiating all the event handlers
-        """
-
         onExecute = ConfigureCommandExecuteHandler()
         cmd.execute.add(onExecute)
-        gm.handlers.append(onExecute)  # 0
+        gm.handlers.append(onExecute)
 
         onInputChanged = ConfigureCommandInputChanged(cmd)
         cmd.inputChanged.add(onInputChanged)
-        gm.handlers.append(onInputChanged)  # 1
+        gm.handlers.append(onInputChanged)
 
-        onExecutePreview = CommandExecutePreviewHandler(cmd)
+        onExecutePreview = CommandExecutePreviewHandler()
         cmd.executePreview.add(onExecutePreview)
-        gm.handlers.append(onExecutePreview)  # 2
+        gm.handlers.append(onExecutePreview)
 
         onSelect = MySelectHandler(cmd)
         cmd.select.add(onSelect)
-        gm.handlers.append(onSelect)  # 3
+        gm.handlers.append(onSelect)
 
         onPreSelect = MyPreSelectHandler(cmd)
         cmd.preSelect.add(onPreSelect)
-        gm.handlers.append(onPreSelect)  # 4
+        gm.handlers.append(onPreSelect)
 
         onPreSelectEnd = MyPreselectEndHandler(cmd)
         cmd.preSelectEnd.add(onPreSelectEnd)
-        gm.handlers.append(onPreSelectEnd)  # 5
+        gm.handlers.append(onPreSelectEnd)
 
         onDestroy = MyCommandDestroyHandler()
         cmd.destroy.add(onDestroy)
-        gm.handlers.append(onDestroy)  # 8
+        gm.handlers.append(onDestroy)
 
 
 class ConfigureCommandExecuteHandler(adsk.core.CommandEventHandler):
-    """### Called when Ok is pressed confirming the export
-
-    Process Steps:
-
-        1. Check for process open in explorer
-
-        1.5. Open file dialog to allow file location save
-            - Not always optimal if sending over socket for parse
-
-        2. Check Socket bind
-
-        3. Check Socket recv
-            - if true send data about file location in temp path
-
-        4. Parse file and focus on unity window
-
-    """
-
-    def __init__(self) -> None:
-        super().__init__()
-        self.current = SerialCommand()
-
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.CommandEventArgs) -> None:
         exporterOptions = ExporterOptions().readFromDesign()
@@ -305,14 +147,6 @@ def notify(self, args: adsk.core.CommandEventArgs) -> None:
             if not savepath:
                 # save was canceled
                 return
-
-            # Transition: AARD-1742
-            # With the addition of a 'release' build the fusion exporter will not have permissions within the sourced
-            # folder. Because of this we cannot use this kind of tmp path anymore. This code was already unused and
-            # should be removed.
-            # updatedPath = pathlib.Path(savepath).parent
-            # if updatedPath != self.current.filePath:
-            #     self.current.filePath = str(updatedPath)
         else:
             savepath = processedFileName
 
@@ -361,51 +195,17 @@ def notify(self, args: adsk.core.CommandEventArgs) -> None:
 
 
 class CommandExecutePreviewHandler(adsk.core.CommandEventHandler):
-    """### Gets an event that is fired when the command has completed gathering the required input and now needs to perform a preview.
-
-    Args:
-        adsk (CommandEventHandler): Command event handler that a client derives from to handle events triggered by a CommandEvent.
-    """
-
-    def __init__(self, cmd: adsk.core.Command) -> None:
-        super().__init__()
-        self.cmd = cmd
-
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.CommandEventArgs) -> None:
-        """Notify member called when a command event is triggered
-
-        Args:
-            args (CommandEventArgs): command event argument
-        """
         jointConfigTab.handlePreviewEvent(args)
         gamepieceConfigTab.handlePreviewEvent(args)
 
 
 class MySelectHandler(adsk.core.SelectionEventHandler):
-    """### Event fires when the user selects an entity.
-    ##### This is different from a preselection where an entity is shown as being available for selection as the mouse passes over the entity. This is the actual selection where the user has clicked the mouse on the entity.
-
-    Args: SelectionEventHandler
-    """
-
-    lastInputCmd = None
-
     def __init__(self, cmd: adsk.core.Command) -> None:
         super().__init__()
         self.cmd = cmd
 
-        # Transition: AARD-1765
-        # self.allWheelPreselections = []  # all child occurrences of selections
-        # self.allGamepiecePreselections = []  # all child gamepiece occurrences of selections
-
-        self.selectedOcc = None  # selected occurrence (if there is one)
-        self.selectedJoint = None  # selected joint (if there is one)
-
-        # Transition: AARD-1765
-        # self.wheelJointList = []
-        self.algorithmicSelection = True
-
     @logFailure(messageBox=True)
     def traverseAssembly(
         self, child_occurrences: adsk.fusion.OccurrenceList, jointedOcc: dict[adsk.fusion.Joint, adsk.fusion.Occurrence]
@@ -543,11 +343,6 @@ def notify(self, args: adsk.core.SelectionEventArgs) -> None:
 
 
 class MyPreselectEndHandler(adsk.core.SelectionEventHandler):
-    """### Event fires when the mouse is moved away from an entity that was in a preselect state.
-
-    Args: SelectionEventArgs
-    """
-
     def __init__(self, cmd: adsk.core.Command) -> None:
         super().__init__()
         self.cmd = cmd
@@ -563,25 +358,12 @@ def notify(self, args: adsk.core.SelectionEventArgs) -> None:
 
 
 class ConfigureCommandInputChanged(adsk.core.InputChangedEventHandler):
-    """### Gets an event that is fired whenever an input value is changed.
-        - Button pressed, selection made, switching tabs, etc...
-
-    Args: InputChangedEventHandler
-    """
-
     def __init__(self, cmd: adsk.core.Command) -> None:
         super().__init__()
         self.cmd = cmd
-        self.allWeights = [None, None]  # [lbs, kg]
-        self.isLbs = True
-        self.isLbs_f = True
 
     @logFailure
     def reset(self) -> None:
-        """### Process:
-        - Reset the mouse icon to default
-        - Clear active selections
-        """
         self.cmd.setCursor("", 0, 0)
         gm.ui.activeSelections.clear()
 
@@ -597,14 +379,8 @@ def notify(self, args: adsk.core.InputChangedEventArgs) -> None:
 
 
 class MyCommandDestroyHandler(adsk.core.CommandEventHandler):
-    """### Gets the event that is fired when the command is destroyed. Globals lists are released and active selections are cleared (when exiting the panel).
-        - In other words, when the OK or Cancel button is pressed...
-
-    Args: CommandEventHandler
-    """
-
     @logFailure(messageBox=True)
-    def notify(self, args: adsk.core.CommandEventArgs) -> None:
+    def notify(self, _: adsk.core.CommandEventArgs) -> None:
         jointConfigTab.reset()
         gamepieceConfigTab.reset()
 
diff --git a/exporter/SynthesisFusionAddin/src/UI/Configuration/SerialCommand.py b/exporter/SynthesisFusionAddin/src/UI/Configuration/SerialCommand.py
deleted file mode 100644
index 61c1e3010..000000000
--- a/exporter/SynthesisFusionAddin/src/UI/Configuration/SerialCommand.py
+++ /dev/null
@@ -1,104 +0,0 @@
-""" SerialCommand module
-
-Defines what is generated by the Fusion Command.
-Use to be protobuf but can only de-serialize from a filestream strangely.
-This is used to store in the userdata.
-"""
-
-import json
-
-from src.Types import OString
-
-
-# Transition: AARD-1765
-# Will likely be removed later as this is no longer used. Avoiding adding typing for now.
-def generateFilePath() -> str:
-    """Generates a temporary file path that can be used to save the file for exporting
-
-    Example:
-     - "%appdata%/Roaming/Temp/HellionFiles"
-
-    Returns:
-        str: file path
-    """
-    tempPath = OString.TempPath("").getPath()  # type: ignore
-    return str(tempPath)
-
-
-class Struct:
-    """For decoding the dict values into named values"""
-
-    def __init__(self, **entries):  # type: ignore
-        self.__dict__.update(entries)
-
-
-class SerialCommand:
-    """All of the command inputs combined"""
-
-    def __init__(self):  # type: ignore
-        self.general = General()
-        self.advanced = Advanced()
-
-        # Transition: AARD-1742
-        # With the addition of a 'release' build the fusion exporter will not have permissions within the sourced
-        # folder. Because of this we cannot use this kind of tmp path anymore. This code was already unused and
-        # should be removed.
-        # self.filePath = generateFilePath()
-
-    def toJSON(self) -> str:
-        """Converts this class into a json object that can be written to the object data
-
-        Returns:
-            str: json version of this object
-        """
-        return json.dumps(self, default=lambda o: o.__dict__, sort_keys=True, indent=1)
-
-
-class General:
-    """General Options"""
-
-    def __init__(self):  # type: ignore
-        # This is the overall export decision point
-        self.exportMode = ExportMode.standard
-        self.RenderType = RenderType.basic3D
-        self.material = BooleanInput("material", True)
-        self.joints = BooleanInput("joints", False)
-        self.rigidGroups = BooleanInput("rigidgroup", False)
-        # self.wheelType =
-        self.simpleWheelExport = BooleanInput("simplewheelexport", False)
-
-
-class Advanced:
-    """Advanced settings in the command input"""
-
-    def __init__(self):  # type: ignore
-        self.friction = BooleanInput("friction", True)
-        self.density = BooleanInput("density", True)
-        self.mass = BooleanInput("mass", True)
-        self.volume = BooleanInput("volume", True)
-        self.surfaceArea = BooleanInput("surfaceArea", True)
-        self.com = BooleanInput("com", True)
-
-
-class BooleanInput:
-    """Class to store the value of a boolean input"""
-
-    def __init__(self, name: str, default: bool):
-        self.name = name
-        self.checked = default
-
-
-class ExportMode:
-    """Export Mode defines the type of export"""
-
-    standard = 0
-    VR = 1
-    Simulation = 2
-
-
-class RenderType:
-    """This will modify the type of material shaders used"""
-
-    basic3D = 0
-    URP = 1
-    HDRP = 2
diff --git a/exporter/SynthesisFusionAddin/src/UI/Handlers.py b/exporter/SynthesisFusionAddin/src/UI/Handlers.py
index 0e60ba36f..b3f11f516 100644
--- a/exporter/SynthesisFusionAddin/src/UI/Handlers.py
+++ b/exporter/SynthesisFusionAddin/src/UI/Handlers.py
@@ -41,110 +41,3 @@ def __init__(self, button: Any) -> None:
 
     def notify(self, _: adsk.core.CommandEventArgs) -> None:
         self.button.exec_func()
-
-
-""" OLD PALETTE COMMANDS
-class HPaletteHTMLEventHandler(adsk.core.HTMLEventHandler):
-    def __init__(self, palette):
-        super().__init__()
-        self.palette = palette
-
-    def notify(self, args) -> None:
-        ui = adsk.core.Application.get().userInterface
-        try:
-            htmlArgs = adsk.core.HTMLEventArgs.cast(args)
-
-            for event in self.palette.events[0]:
-                if event[0] == htmlArgs.action:
-                    # if Helper.check_solid_open() :
-                    val = event[1](htmlArgs.data)
-                    if val is not None:
-                        # logging.getLogger("HellionFusion.HUI.Handlers").debug(
-                        #    f"{htmlArgs.action}: response: {val}"
-                        # )
-                        htmlArgs.returnData = val
-                        return
-                else:
-                    htmlArgs.returnData = ""
-        except:
-            ui.messageBox("Failed:\n{}".format(traceback.format_exc()))
-            
-"""
-
-""" These are old functions that mapped the palette commands
-
-class CustomDocumentSavedHandler(adsk.core.DocumentEventHandler):
-    def __init__(self):
-        super().__init__()
-
-    def notify(self, args) -> bool:
-        eventArgs = adsk.core.DocumentEventArgs.cast(args)
-        name = Helper.getDocName()
-
-        if name in gm.queue:
-            connected = Helper.checkAttribute()
-            if connected and (connected == "True"):
-                try:
-                    # req = DesignModificationLookup(gm.app.activeDocument.design)
-                    # if req is not None:
-                    #    sent = nm.send(req)
-                    #    logging.getLogger('HellionFusion.HUI.Handlers.DocumentSaved').debug(f'Sending update with data: {req}')
-
-                    design = gm.app.activeDocument.design
-                    name = design.rootComponent.name.rsplit(" ", 1)[0]
-                    version = design.rootComponent.name.rsplit(" ", 1)[1]
-
-                    # version comes back the same - this is terrible but it will do
-                    version = int(version[1:])
-                    version += 1
-                    version = f"v{version}"
-
-                    Helper.addUnityAttribute()
-                    req = Parser(parseOptions=ParseOptions()).parseUpdated(version)
-                    if req is not None:
-                        sent = nm.send(req)
-                    else:
-                        logging.getLogger(
-                            "HellionFusion.HUI.Handlers.DocumentSave"
-                        ).error(
-                            f"Failed to Parse Update or generate request ----- \n {req}"
-                        )
-                    return True
-
-                except:
-                    gm.ui.messageBox()
-                    logging.getLogger("HellionFusion.HUI.Handlers.DocumentSave").error(
-                        "Failed:\n{}".format(traceback.format_exc())
-                    )
-
-                # TODO: add item to queue here
-                # let the Network manager send them and control how they are sent
-                # if len(gm.palettes) >= 1:
-                #    palette = gm.ui.palettes.itemById(gm.palettes[0].uid)
-                #    if palette:
-                #        name = Helper.getDocName()
-        else:
-            return False
-
-
-class ConnectionPaletteHandler(adsk.core.CustomEventHandler):
-    def __init__(self):
-        super().__init__()
-
-    def notify(self, args):
-        ui = adsk.core.Application.get().userInterface
-        try:
-            if ui.activeCommand != "SelectCommand":
-                ui.commandDefinitions.itemById("SelectCommand").execute()
-
-            if len(gm.palettes) >= 1:
-                palette = ui.palettes.itemById(gm.palettes[0].uid)
-                if palette:
-                    res = palette.sendInfoToHTML(
-                        "updateConnection", nm.NetCommand.connected()
-                    )
-        except:
-            if ui:
-                ui.messageBox("Failed:\n{}".format(traceback.format_exc()))
-
-"""

From db954a401717606ed04089ef2a106bbbc9af1f00 Mon Sep 17 00:00:00 2001
From: BrandonPacewic <92102436+BrandonPacewic@users.noreply.github.com>
Date: Thu, 22 Aug 2024 12:00:01 -0700
Subject: [PATCH 2/4] Final changes to `ConfigCommand.py`

---
 .../src/Parser/ExporterOptions.py             |   4 +-
 .../src/Resources/HTML/info.html              |   4 +-
 exporter/SynthesisFusionAddin/src/Types.py    |   2 +
 .../SynthesisFusionAddin/src/UI/Camera.py     |   2 +-
 .../src/UI/ConfigCommand.py                   | 290 ++++--------------
 .../src/UI/GamepieceConfigTab.py              |   2 +-
 .../SynthesisFusionAddin/src/UI/Handlers.py   |  12 +
 .../src/UI/JointConfigTab.py                  |  37 ++-
 8 files changed, 111 insertions(+), 242 deletions(-)

diff --git a/exporter/SynthesisFusionAddin/src/Parser/ExporterOptions.py b/exporter/SynthesisFusionAddin/src/Parser/ExporterOptions.py
index 1eb34e9e5..39f6c276b 100644
--- a/exporter/SynthesisFusionAddin/src/Parser/ExporterOptions.py
+++ b/exporter/SynthesisFusionAddin/src/Parser/ExporterOptions.py
@@ -32,7 +32,7 @@ class ExporterOptions:
     # Python's `os` module can return `None` when attempting to find the home directory if the
     # user's computer has conflicting configs of some sort. This has happened and should be accounted
     # for accordingly.
-    fileLocation: str | None = field(
+    fileLocation: str | os.PathLike[str] | None = field(
         default=(os.getenv("HOME") if platform.system() == "Windows" else os.path.expanduser("~"))
     )
     name: str | None = field(default=None)
@@ -52,7 +52,7 @@ class ExporterOptions:
     compressOutput: bool = field(default=True)
     exportAsPart: bool = field(default=False)
 
-    exportLocation: ExportLocation = field(default=ExportLocation.UPLOAD)
+    exportLocation: ExportLocation = field(default=ExportLocation.DOWNLOAD)
 
     hierarchy: ModelHierarchy = field(default=ModelHierarchy.FusionAssembly)
     visualQuality: TriangleMeshQualityOptions = field(default=TriangleMeshQualityOptions.LowQualityTriangleMesh)
diff --git a/exporter/SynthesisFusionAddin/src/Resources/HTML/info.html b/exporter/SynthesisFusionAddin/src/Resources/HTML/info.html
index a44b45f11..759ed2a48 100644
--- a/exporter/SynthesisFusionAddin/src/Resources/HTML/info.html
+++ b/exporter/SynthesisFusionAddin/src/Resources/HTML/info.html
@@ -2,7 +2,7 @@
 <html>
     <head>
         <script>
-            window.location.replace("https://synthesis.autodesk.com/tutorials.html")
+            window.location.replace("https://synthesis.autodesk.com/codelab/FusionExporterCodelab/index.html#0")
         </script>
         <style>
             .redirecting {
@@ -18,4 +18,4 @@
             <h1>Redirecting...</h1>
         </div>
     </body>
-</html>
\ No newline at end of file
+</html>
diff --git a/exporter/SynthesisFusionAddin/src/Types.py b/exporter/SynthesisFusionAddin/src/Types.py
index bb714b27f..8efa7695d 100644
--- a/exporter/SynthesisFusionAddin/src/Types.py
+++ b/exporter/SynthesisFusionAddin/src/Types.py
@@ -85,6 +85,8 @@ class ModelHierarchy(Enum):
 KG: TypeAlias = float
 LBS: TypeAlias = float
 PRIMITIVES = (bool, str, int, float, type(None))
+
+# All currently supported Fusion joints
 SELECTABLE_JOINT_TYPES = (adsk.fusion.JointTypes.RevoluteJointType, adsk.fusion.JointTypes.SliderJointType)
 
 
diff --git a/exporter/SynthesisFusionAddin/src/UI/Camera.py b/exporter/SynthesisFusionAddin/src/UI/Camera.py
index 4b8306268..53168fd24 100644
--- a/exporter/SynthesisFusionAddin/src/UI/Camera.py
+++ b/exporter/SynthesisFusionAddin/src/UI/Camera.py
@@ -9,7 +9,7 @@
 
 
 @logFailure
-def captureThumbnail(size: int = 250) -> str:
+def captureThumbnail(size: int = 250) -> str | os.PathLike[str]:
     """
     ## Captures Thumbnail and saves it to a temporary path - needs to be cleared after or on startup
     - Size: int (Default: 200) : (width & height)
diff --git a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
index 8f73d0a68..7d9d97f71 100644
--- a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
+++ b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
@@ -1,7 +1,8 @@
-""" Generate all the front-end command inputs and GUI elements.
-    links the Configuration Command seen when pressing the Synthesis button in the Addins Panel
+"""
+Central location for which all UI is generated and handled for the main configuration panel.
 """
 
+import os
 from typing import Any
 
 import adsk.core
@@ -16,6 +17,7 @@
 from src.UI import FileDialogConfig
 from src.UI.GamepieceConfigTab import GamepieceConfigTab
 from src.UI.GeneralConfigTab import GeneralConfigTab
+from src.UI.Handlers import PersistentEventHandler
 from src.UI.JointConfigTab import JointConfigTab
 
 generalConfigTab: GeneralConfigTab
@@ -28,13 +30,34 @@
 
 
 class ConfigureCommandCreatedHandler(adsk.core.CommandCreatedEventHandler):
+    """Called when the panel is initially created."""
+
     def __init__(self, configure: Any) -> None:
         super().__init__()
 
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
-        exporterOptions = ExporterOptions().readFromDesign() or ExporterOptions()
         cmd = args.command
+        gm.ui.activeSelections.clear()
+        onExecute = ConfigureCommandExecuteHandler()
+        cmd.execute.add(onExecute)
+
+        onInputChanged = ConfigureCommandInputChanged()
+        cmd.inputChanged.add(onInputChanged)
+
+        onExecutePreview = CommandExecutePreviewHandler()
+        cmd.executePreview.add(onExecutePreview)
+
+        onSelect = MySelectHandler()
+        cmd.select.add(onSelect)
+
+        onPreSelectEnd = MyPreselectEndHandler(cmd)
+        cmd.preSelectEnd.add(onPreSelectEnd)
+
+        onDestroy = MyCommandDestroyHandler()
+        cmd.destroy.add(onDestroy)
+
+        exporterOptions = ExporterOptions().readFromDesign() or ExporterOptions()
 
         cmd.isAutoExecute = False
         cmd.isExecutedWhenPreEmpted = False
@@ -44,8 +67,7 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         global INPUTS_ROOT
         INPUTS_ROOT = cmd.commandInputs
 
-        # TODO?
-        # cmd.helpFile = os.path.join(".", "src", "Resources", "HTML", "info.html")
+        cmd.helpFile = os.path.join(".", "src", "Resources", "HTML", "info.html")
 
         global generalConfigTab
         generalConfigTab = GeneralConfigTab(args, exporterOptions)
@@ -54,45 +76,36 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         gamepieceConfigTab = GamepieceConfigTab(args, exporterOptions)
         generalConfigTab.gamepieceConfigTab = gamepieceConfigTab
 
-        if not exporterOptions.exportMode == ExportMode.FIELD:
-            gamepieceConfigTab.isVisible = False
-
-        if exporterOptions.gamepieces:
-            for synGamepiece in exporterOptions.gamepieces:
-                fusionOccurrence = gm.app.activeDocument.design.findEntityByToken(synGamepiece.occurrenceToken)[0]
-                gamepieceConfigTab.addGamepiece(fusionOccurrence, synGamepiece)
-
         global jointConfigTab
         jointConfigTab = JointConfigTab(args)
         generalConfigTab.jointConfigTab = jointConfigTab
 
+        if not exporterOptions.exportMode == ExportMode.FIELD:
+            gamepieceConfigTab.isVisible = False
+
         if not exporterOptions.exportMode == ExportMode.ROBOT:
             jointConfigTab.isVisible = False
 
-        # Transition: AARD-1685
-        # There remains some overlap between adding joints as wheels.
-        # Should investigate changes to improve performance.
-        if exporterOptions.joints:
+        design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
+        for synGamepiece in exporterOptions.gamepieces:
+            fusionOccurrence = design.findEntityByToken(synGamepiece.occurrenceToken)[0]
+            gamepieceConfigTab.addGamepiece(fusionOccurrence, synGamepiece)
+
+        if len(exporterOptions.joints):
             for synJoint in exporterOptions.joints:
-                fusionJoints = gm.app.activeDocument.design.findEntityByToken(synJoint.jointToken)
+                fusionJoints = design.findEntityByToken(synJoint.jointToken)
                 if len(fusionJoints):
                     jointConfigTab.addJoint(fusionJoints[0], synJoint)
         else:
-            for joint in [
-                *gm.app.activeDocument.design.rootComponent.allJoints,
-                *gm.app.activeDocument.design.rootComponent.allAsBuiltJoints,
-            ]:
+            for joint in [*design.rootComponent.allJoints, *design.rootComponent.allAsBuiltJoints]:
                 if joint.jointMotion.jointType in SELECTABLE_JOINT_TYPES and not joint.isSuppressed:
                     jointConfigTab.addJoint(joint)
 
         # Adding saved wheels must take place after joints are added as a result of how the two types are connected.
-        # Transition: AARD-1685
-        # Should consider changing how the parser handles wheels and joints to avoid overlap
-        if exporterOptions.wheels:
-            for wheel in exporterOptions.wheels:
-                fusionJoints = gm.app.activeDocument.design.findEntityByToken(wheel.jointToken)
-                if len(fusionJoints):
-                    jointConfigTab.addWheel(fusionJoints[0], wheel)
+        for wheel in exporterOptions.wheels:
+            fusionJoints = design.findEntityByToken(wheel.jointToken)
+            if len(fusionJoints):
+                jointConfigTab.addWheel(fusionJoints[0], wheel)
 
         getAuth()
         user_info = getUserInfo()
@@ -101,45 +114,13 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         )
         apsSettings.tooltip = "Configuration settings for Autodesk Platform Services."
 
-        gm.ui.activeSelections.clear()
-        onExecute = ConfigureCommandExecuteHandler()
-        cmd.execute.add(onExecute)
-        gm.handlers.append(onExecute)
-
-        onInputChanged = ConfigureCommandInputChanged(cmd)
-        cmd.inputChanged.add(onInputChanged)
-        gm.handlers.append(onInputChanged)
-
-        onExecutePreview = CommandExecutePreviewHandler()
-        cmd.executePreview.add(onExecutePreview)
-        gm.handlers.append(onExecutePreview)
-
-        onSelect = MySelectHandler(cmd)
-        cmd.select.add(onSelect)
-        gm.handlers.append(onSelect)
-
-        onPreSelect = MyPreSelectHandler(cmd)
-        cmd.preSelect.add(onPreSelect)
-        gm.handlers.append(onPreSelect)
-
-        onPreSelectEnd = MyPreselectEndHandler(cmd)
-        cmd.preSelectEnd.add(onPreSelectEnd)
-        gm.handlers.append(onPreSelectEnd)
-
-        onDestroy = MyCommandDestroyHandler()
-        cmd.destroy.add(onDestroy)
-        gm.handlers.append(onDestroy)
 
+class ConfigureCommandExecuteHandler(PersistentEventHandler, adsk.core.CommandEventHandler):
+    """Called when the `Export` button is clicked from the main configuration panel."""
 
-class ConfigureCommandExecuteHandler(adsk.core.CommandEventHandler):
     @logFailure(messageBox=True)
-    def notify(self, args: adsk.core.CommandEventArgs) -> None:
+    def notify(self, _: adsk.core.CommandEventArgs) -> None:
         exporterOptions = ExporterOptions().readFromDesign()
-
-        if args.executeFailed:
-            logger.error("Could not execute configuration due to failure")
-            return
-
         processedFileName = gm.app.activeDocument.name.replace(" ", "_")
         if generalConfigTab.exportLocation == ExportLocation.DOWNLOAD:
             savepath = FileDialogConfig.saveFileDialog(defaultPath=exporterOptions.fileLocation)
@@ -152,21 +133,21 @@ def notify(self, args: adsk.core.CommandEventArgs) -> None:
 
         adsk.doEvents()
 
-        design = gm.app.activeDocument.design
+        design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
 
-        name_split: list[str] = design.rootComponent.name.split(" ")
-        if len(name_split) < 2:
+        nameSplit = design.rootComponent.name.split(" ")
+        if len(nameSplit) < 2:
             gm.ui.messageBox("Please open the robot design you would like to export", "Synthesis: Error")
             return
 
-        name = name_split[0]
-        version = name_split[1]
+        name = nameSplit[0]
+        version = nameSplit[1]
 
         selectedJoints, selectedWheels = jointConfigTab.getSelectedJointsAndWheels()
         selectedGamepieces = gamepieceConfigTab.getGamepieces()
 
         exporterOptions = ExporterOptions(
-            str(savepath),
+            savepath,
             name,
             version,
             materials=0,
@@ -186,187 +167,47 @@ def notify(self, args: adsk.core.CommandEventArgs) -> None:
 
         Parser(exporterOptions).export()
         exporterOptions.writeToDesign()
-
-        # All selections should be reset AFTER a successful export and save.
-        # If we run into an exporting error we should return back to the panel with all current options
-        # still in tact. Even if they did not save.
         jointConfigTab.reset()
         gamepieceConfigTab.reset()
 
 
-class CommandExecutePreviewHandler(adsk.core.CommandEventHandler):
+class CommandExecutePreviewHandler(PersistentEventHandler, adsk.core.CommandEventHandler):
+    """Called when an execute command is ready to be previewed."""
+
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.CommandEventArgs) -> None:
         jointConfigTab.handlePreviewEvent(args)
         gamepieceConfigTab.handlePreviewEvent(args)
 
 
-class MySelectHandler(adsk.core.SelectionEventHandler):
-    def __init__(self, cmd: adsk.core.Command) -> None:
-        super().__init__()
-        self.cmd = cmd
-
-    @logFailure(messageBox=True)
-    def traverseAssembly(
-        self, child_occurrences: adsk.fusion.OccurrenceList, jointedOcc: dict[adsk.fusion.Joint, adsk.fusion.Occurrence]
-    ) -> (
-        list[adsk.fusion.Joint | adsk.fusion.Occurrence] | None
-    ):  # recursive traversal to check if children are jointed
-        """### Traverses the entire occurrence hierarchy to find a match (jointed occurrence) in self.occurrence
-
-        Args:
-            child_occurrences (adsk.fusion.OccurrenceList): a list of child occurrences
-
-        Returns:
-            occ (Occurrence): if a match is found, return the jointed occurrence
-            None: if no match is found
-        """
-        for occ in child_occurrences:
-            for joint, value in jointedOcc.items():
-                if occ in value:
-                    return [joint, occ]  # occurrence that is jointed
-
-            if occ.childOccurrences:  # if occurrence has children, traverse sub-tree
-                self.traverseAssembly(occ.childOccurrences, jointedOcc)
-        return None  # no jointed occurrence found
-
-    @logFailure(messageBox=True)
-    def wheelParent(self, occ: adsk.fusion.Occurrence) -> list[str | adsk.fusion.Occurrence | None]:
-        """### Identify an occurrence that encompasses the entire wheel component.
-
-        Process:
-
-            1. if the selection has no parent, return the selection.
-
-            2. if the selection is directly jointed, return the selection.
-
-            3. else keep climbing the occurrence tree until no parent is found.
-
-            - if a jointed occurrence is in the tree, return the selection parent.
-
-            - if no jointed occurrence was found, return the selection.
-
-        Args:
-            occ (Occurrence): The selected child occurrence
-
-        Returns:
-            occ (Occurrence): Wheel parent
-        """
-        parent = occ.assemblyContext
-        jointedOcc = {}  # dictionary with all jointed occurrences
-
-        try:
-            for joint in occ.joints:
-                if joint.jointMotion.jointType == adsk.fusion.JointTypes.RevoluteJointType:
-                    # gm.ui.messageBox("Selection is directly jointed.\nReturning selection.\n\n" + "Occurrence:\n--> " + occ.name + "\nJoint:\n--> " + joint.name)
-                    return [joint.entityToken, occ]
-        except:
-            for joint in occ.component.joints:
-                if joint.jointMotion.jointType == adsk.fusion.JointTypes.RevoluteJointType:
-                    # gm.ui.messageBox("Selection is directly jointed.\nReturning selection.\n\n" + "Occurrence:\n--> " + occ.name + "\nJoint:\n--> " + joint.name)
-                    return [joint.entityToken, occ]
-
-        if parent == None:  # no parent occurrence
-            # gm.ui.messageBox("Selection has no parent occurrence.\nReturning selection.\n\n" + "Occurrence:\n--> " + occ.name + "\nJoint:\n--> NONE")
-            return [None, occ]  # return what is selected
-
-        for joint in gm.app.activeDocument.design.rootComponent.allJoints:
-            if joint.jointMotion.jointType != adsk.fusion.JointTypes.RevoluteJointType:
-                continue
-            jointedOcc[joint.entityToken] = [
-                joint.occurrenceOne,
-                joint.occurrenceTwo,
-            ]
-
-        parentLevel = 1  # the number of nodes above the one selected
-        returned = None  # the returned value of traverseAssembly()
-        parentOccurrence = occ  # the parent occurrence that will be returned
-        treeParent = parent  # each parent that will traverse up in algorithm.
-
-        while treeParent != None:  # loops until reaches top-level component
-            returned = self.traverseAssembly(treeParent.childOccurrences, jointedOcc)
-
-            if returned != None:
-                for i in range(parentLevel):
-                    parentOccurrence = parentOccurrence.assemblyContext
-
-                # gm.ui.messageBox("Joint found.\nReturning parent occurrence.\n\n" + "Selected occurrence:\n--> " + occ.name + "\nParent:\n--> " + parentOccurrence.name + "\nJoint:\n--> " + returned[0] + "\nNodes above selection:\n--> " + str(parentLevel))
-                return [returned[0], parentOccurrence]
-
-            parentLevel += 1
-            treeParent = treeParent.assemblyContext
-        # gm.ui.messageBox("No jointed occurrence found.\nReturning selection.\n\n" + "Occurrence:\n--> " + occ.name + "\nJoint:\n--> NONE")
-        return [None, occ]  # no jointed occurrence found, return what is selected
+class MySelectHandler(PersistentEventHandler, adsk.core.SelectionEventHandler):
+    """Called when a selection in the current design is made."""
 
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.SelectionEventArgs) -> None:
-        """### Notify member is called when a selection event is triggered.
-
-        Args:
-            args (SelectionEventArgs): A selection event argument
-        """
         if gamepieceConfigTab.isVisible:
-            self.cmd.setCursor("", 0, 0)  # Reset select cursor back to normal cursor.
             gamepieceConfigTab.handleSelectionEvent(args, args.selection.entity)
 
         if jointConfigTab.isVisible:
-            self.cmd.setCursor("", 0, 0)  # Reset select cursor back to normal cursor.
             jointConfigTab.handleSelectionEvent(args, args.selection.entity)
 
 
-class MyPreSelectHandler(adsk.core.SelectionEventHandler):
-    """### Event fires when a entity preselection is made (mouse hovering).
-    ##### When a user is selecting geometry, they move the mouse over the model and if the entity the mouse is currently over is valid for selection it will highlight indicating that it can be selected. This process of determining what is available for selection and highlighting it is referred to as the "preselect" behavior.
-
-    Args: SelectionEventHandler
-    """
+class MyPreselectEndHandler(PersistentEventHandler, adsk.core.SelectionEventHandler):
+    """Called upon a pre-selection end in the current design. (Mouse hover off)"""
 
     def __init__(self, cmd: adsk.core.Command) -> None:
         super().__init__()
         self.cmd = cmd
 
     @logFailure(messageBox=True)
-    def notify(self, args: adsk.core.SelectionEventArgs) -> None:
-        design = adsk.fusion.Design.cast(gm.app.activeProduct)
-        preSelectedOcc = adsk.fusion.Occurrence.cast(args.selection.entity)
-        preSelectedJoint = adsk.fusion.Joint.cast(args.selection.entity)
+    def notify(self, _: adsk.core.SelectionEventArgs) -> None:
+        self.cmd.setCursor("", 0, 0)  # Reset mouse icon to default
 
-        onSelect = gm.handlers[3]  # select handler
 
-        if (not preSelectedOcc and not preSelectedJoint) or not design:
-            self.cmd.setCursor("", 0, 0)
-            return
-
-        preSelected = preSelectedOcc if preSelectedOcc else preSelectedJoint
-        if not preSelected:
-            self.cmd.setCursor("", 0, 0)
-
-
-class MyPreselectEndHandler(adsk.core.SelectionEventHandler):
-    def __init__(self, cmd: adsk.core.Command) -> None:
-        super().__init__()
-        self.cmd = cmd
+class ConfigureCommandInputChanged(PersistentEventHandler, adsk.core.InputChangedEventHandler):
+    """Called when an input field in the configuration panel has been updated."""
 
     @logFailure(messageBox=True)
-    def notify(self, args: adsk.core.SelectionEventArgs) -> None:
-        design = adsk.fusion.Design.cast(gm.app.activeProduct)
-        preSelectedOcc = adsk.fusion.Occurrence.cast(args.selection.entity)
-        preSelectedJoint = adsk.fusion.Joint.cast(args.selection.entity)
-
-        if (preSelectedOcc or preSelectedJoint) and design:
-            self.cmd.setCursor("", 0, 0)  # if preselection ends (mouse off of design), reset the mouse icon to default
-
-
-class ConfigureCommandInputChanged(adsk.core.InputChangedEventHandler):
-    def __init__(self, cmd: adsk.core.Command) -> None:
-        super().__init__()
-        self.cmd = cmd
-
-    @logFailure
-    def reset(self) -> None:
-        self.cmd.setCursor("", 0, 0)
-        gm.ui.activeSelections.clear()
-
     def notify(self, args: adsk.core.InputChangedEventArgs) -> None:
         if generalConfigTab.isActive:
             generalConfigTab.handleInputChanged(args)
@@ -378,11 +219,14 @@ def notify(self, args: adsk.core.InputChangedEventArgs) -> None:
             gamepieceConfigTab.handleInputChanged(args, INPUTS_ROOT)
 
 
-class MyCommandDestroyHandler(adsk.core.CommandEventHandler):
+class MyCommandDestroyHandler(PersistentEventHandler, adsk.core.CommandEventHandler):
+    """Called when the configuration panel is destroyed."""
+
     @logFailure(messageBox=True)
     def notify(self, _: adsk.core.CommandEventArgs) -> None:
         jointConfigTab.reset()
         gamepieceConfigTab.reset()
 
-        for group in gm.app.activeDocument.design.rootComponent.customGraphicsGroups:
+        design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
+        for group in design.rootComponent.customGraphicsGroups:
             group.deleteMe()
diff --git a/exporter/SynthesisFusionAddin/src/UI/GamepieceConfigTab.py b/exporter/SynthesisFusionAddin/src/UI/GamepieceConfigTab.py
index c28f6c60f..16aab484f 100644
--- a/exporter/SynthesisFusionAddin/src/UI/GamepieceConfigTab.py
+++ b/exporter/SynthesisFusionAddin/src/UI/GamepieceConfigTab.py
@@ -273,7 +273,7 @@ def handleInputChanged(
     @logFailure
     def handleSelectionEvent(self, args: adsk.core.SelectionEventArgs, selectedOcc: adsk.fusion.Occurrence) -> None:
         selectionInput = args.activeInput
-        rootComponent = adsk.core.Application.get().activeDocument.design.rootComponent
+        rootComponent = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct).rootComponent
         occurrenceList: list[adsk.fusion.Occurrence] = rootComponent.allOccurrencesByComponent(selectedOcc.component)
         for occ in occurrenceList:
             if not self.addGamepiece(occ):
diff --git a/exporter/SynthesisFusionAddin/src/UI/Handlers.py b/exporter/SynthesisFusionAddin/src/UI/Handlers.py
index b3f11f516..2f838f196 100644
--- a/exporter/SynthesisFusionAddin/src/UI/Handlers.py
+++ b/exporter/SynthesisFusionAddin/src/UI/Handlers.py
@@ -2,6 +2,18 @@
 
 import adsk.core
 
+from src import gm
+
+
+class PersistentEventHandler:
+    """Auto adds itself to the global handler list on creation."""
+
+    # Inherit order matters here, `PersistentEventHandler` should come first
+    # ex: class HButtonCommandCreatedEvent(PersistentEventHandler, adsk.core.CommandCreatedEventHandler)
+    def __init__(self) -> None:
+        super().__init__()
+        gm.handlers.append(self)
+
 
 class HButtonCommandCreatedEvent(adsk.core.CommandCreatedEventHandler):
     """## Abstraction of CreatedEvent as its mostly useless in this context
diff --git a/exporter/SynthesisFusionAddin/src/UI/JointConfigTab.py b/exporter/SynthesisFusionAddin/src/UI/JointConfigTab.py
index 217d40932..bf9badb32 100644
--- a/exporter/SynthesisFusionAddin/src/UI/JointConfigTab.py
+++ b/exporter/SynthesisFusionAddin/src/UI/JointConfigTab.py
@@ -2,7 +2,14 @@
 import adsk.fusion
 
 from src.Logging import logFailure
-from src.Types import Joint, JointParentType, SignalType, Wheel, WheelType
+from src.Types import (
+    SELECTABLE_JOINT_TYPES,
+    Joint,
+    JointParentType,
+    SignalType,
+    Wheel,
+    WheelType,
+)
 from src.UI import IconPaths
 from src.UI.CreateCommandInputsHelper import (
     createBooleanInput,
@@ -479,20 +486,24 @@ def handleInputChanged(
     def handleSelectionEvent(self, args: adsk.core.SelectionEventArgs, selectedJoint: adsk.fusion.Joint) -> None:
         selectionInput = args.activeInput
         jointType = selectedJoint.jointMotion.jointType
-        if jointType == adsk.fusion.JointTypes.RevoluteJointType or jointType == adsk.fusion.JointTypes.SliderJointType:
-            if not self.addJoint(selectedJoint):
-                ui = adsk.core.Application.get().userInterface
-                result = ui.messageBox(
-                    "You have already selected this joint.\n" "Would you like to remove it?",
-                    "Synthesis: Remove Joint Confirmation",
-                    adsk.core.MessageBoxButtonTypes.YesNoButtonType,
-                    adsk.core.MessageBoxIconTypes.QuestionIconType,
-                )
+        if not jointType in SELECTABLE_JOINT_TYPES:
+            ui = adsk.core.Application.get().userInterface
+            ui.messageBox("The selected joint is not currently supported by Synthesis.")
+            return
+
+        if not self.addJoint(selectedJoint):
+            ui = adsk.core.Application.get().userInterface
+            result = ui.messageBox(
+                "You have already selected this joint.\n" "Would you like to remove it?",
+                "Synthesis: Remove Joint Confirmation",
+                adsk.core.MessageBoxButtonTypes.YesNoButtonType,
+                adsk.core.MessageBoxIconTypes.QuestionIconType,
+            )
 
-                if result == adsk.core.DialogResults.DialogYes:
-                    self.removeJoint(selectedJoint)
+            if result == adsk.core.DialogResults.DialogYes:
+                self.removeJoint(selectedJoint)
 
-            selectionInput.isEnabled = selectionInput.isVisible = False
+        selectionInput.isEnabled = selectionInput.isVisible = False
 
     @logFailure
     def handlePreviewEvent(self, args: adsk.core.CommandEventArgs) -> None:

From a3685b47e1f5a9870da627c66865e96007618da2 Mon Sep 17 00:00:00 2001
From: BrandonPacewic <92102436+BrandonPacewic@users.noreply.github.com>
Date: Fri, 23 Aug 2024 09:24:55 -0700
Subject: [PATCH 3/4] Expanded fix from GH-1099

---
 .../src/UI/ConfigCommand.py                   | 38 +++++++++----------
 1 file changed, 18 insertions(+), 20 deletions(-)

diff --git a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
index 7d9d97f71..44d06aabd 100644
--- a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
+++ b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
@@ -38,6 +38,9 @@ def __init__(self, configure: Any) -> None:
     @logFailure(messageBox=True)
     def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         cmd = args.command
+        global INPUTS_ROOT
+        INPUTS_ROOT = cmd.commandInputs
+
         gm.ui.activeSelections.clear()
         onExecute = ConfigureCommandExecuteHandler()
         cmd.execute.add(onExecute)
@@ -63,10 +66,6 @@ def notify(self, args: adsk.core.CommandCreatedEventArgs) -> None:
         cmd.isExecutedWhenPreEmpted = False
         cmd.okButtonText = "Export"
         cmd.setDialogSize(800, 350)
-
-        global INPUTS_ROOT
-        INPUTS_ROOT = cmd.commandInputs
-
         cmd.helpFile = os.path.join(".", "src", "Resources", "HTML", "info.html")
 
         global generalConfigTab
@@ -120,36 +119,35 @@ class ConfigureCommandExecuteHandler(PersistentEventHandler, adsk.core.CommandEv
 
     @logFailure(messageBox=True)
     def notify(self, _: adsk.core.CommandEventArgs) -> None:
+        design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
         exporterOptions = ExporterOptions().readFromDesign()
+
+        designNameSplit = design.rootComponent.name.split(" ")
+        if len(designNameSplit) > 1:
+            docName, docVersion = designNameSplit[:2]
+        else:
+            docName = designNameSplit[0]
+            docVersion = ""
+
         processedFileName = gm.app.activeDocument.name.replace(" ", "_")
+        defaultFileName = "_".join([docName, docVersion]) + ".mira" if docVersion else f"{docName}.mira"
         if generalConfigTab.exportLocation == ExportLocation.DOWNLOAD:
-            savepath = FileDialogConfig.saveFileDialog(defaultPath=exporterOptions.fileLocation)
-
-            if not savepath:
-                # save was canceled
-                return
+            savepath = FileDialogConfig.saveFileDialog(exporterOptions.fileLocation, defaultFileName)
         else:
             savepath = processedFileName
 
-        adsk.doEvents()
-
-        design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
-
-        nameSplit = design.rootComponent.name.split(" ")
-        if len(nameSplit) < 2:
-            gm.ui.messageBox("Please open the robot design you would like to export", "Synthesis: Error")
+        if not savepath:  # User cancelled the save dialog
             return
 
-        name = nameSplit[0]
-        version = nameSplit[1]
+        adsk.doEvents()
 
         selectedJoints, selectedWheels = jointConfigTab.getSelectedJointsAndWheels()
         selectedGamepieces = gamepieceConfigTab.getGamepieces()
 
         exporterOptions = ExporterOptions(
             savepath,
-            name,
-            version,
+            docName,
+            docVersion,
             materials=0,
             joints=selectedJoints,
             wheels=selectedWheels,

From 8c1d9858936cb05e0fc9be9f80f9307ddfd0778d Mon Sep 17 00:00:00 2001
From: BrandonPacewic <92102436+BrandonPacewic@users.noreply.github.com>
Date: Wed, 28 Aug 2024 20:23:51 -0700
Subject: [PATCH 4/4] Updated file naming to account for multi word design
 names

---
 exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
index 1474c4090..5f4704b84 100644
--- a/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
+++ b/exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py
@@ -3,6 +3,7 @@
 """
 
 import os
+import re
 import webbrowser
 from typing import Any
 
@@ -123,12 +124,10 @@ def notify(self, _: adsk.core.CommandEventArgs) -> None:
         design = adsk.fusion.Design.cast(adsk.core.Application.get().activeProduct)
         exporterOptions = ExporterOptions().readFromDesign()
 
-        designNameSplit = design.rootComponent.name.split(" ")
-        if len(designNameSplit) > 1:
-            docName, docVersion = designNameSplit[:2]
-        else:
-            docName = designNameSplit[0]
-            docVersion = "v0"
+        fullName = design.rootComponent.name
+        versionMatch = re.search(r"v\d+", fullName)
+        docName = (fullName[: versionMatch.start()].strip() if versionMatch else fullName).replace(" ", "_")
+        docVersion = versionMatch.group() if versionMatch else "v0"
 
         processedFileName = gm.app.activeDocument.name.replace(" ", "_")
         defaultFileName = f"{'_'.join([docName, docVersion])}.mira"