Script help: Parenting and keeping transforms.

I have an idea for an add-on that I think would be very useful to my workflow, but this is my first interaction with python so I wanted to have an extra set of eyes to make sure I'm not doing anything blatantly wrong or inefficient. The goal is to eventually make it an extension, so I'm trying to make the code proper.

The first step is to add an empty in the same location as the selected object, rename it to match the object, and parent the object to it. I added for now the ability to choose a few different empty types and set the radius relative to the size of the object. 

I ran into all sorts of trouble with parenting, with the object inheriting the scale of the empty (which is why I went simpler to its radius instead), and keeping offset when the object was not in the world origin. My solution is the code below. It's all very annotated because I'm still getting used to python's syntax and it helps me remember what my intention was with each line, so I apologize if it's annoying.

I'm wondering if there's anything you would do differently to make it simpler or more straight-forward?

Let me know! And thank you!

import bpy

class EmptyAssistantOperator(bpy.types.Operator):
    """Add an empty at each selected object's origin and parent the object to it"""
    bl_idname = "object.empty_assist"
    bl_label = "Empty Assistant"
    bl_options = {'REGISTER', 'UNDO'}
    
    empty_type: bpy.props.EnumProperty(
        default='PLAIN_AXES',
        name="Empty Type",
        description="Choose the type of empty",
        items=[
        ('PLAIN_AXES', "Plain Axes", ""),
        ('CUBE', "Cube", ""),
        ('SPHERE', "Sphere", ""),
        ('CIRCLE', "Circle", "")
        ]
    )
    
    scale_factor: bpy.props.FloatProperty(
        name="Empty Scale",
        description="Multiplier for empty size relative to object",
        default=1.0,
        min=0.1,
        max=5.0
    )
    
    @classmethod
    def poll(cls, context):
        return context.selected_objects
    
    def invoke(self, context, event):
        # Opens a popup dialog for user input
        return context.window_manager.invoke_props_dialog(self)
    
    
    def execute(self, context):
        selected_objects = context.selected_objects
        
        for obj in selected_objects:
            # Get the current world location of the object
            world_location = obj.matrix_world.translation
            
            # Calculate object's bounding box dimensions
            bbox = (max(obj.dimensions)) * self.scale_factor
            
            # Create an empty at the object's world location
            bpy.ops.object.empty_add(type=self.empty_type, radius=bbox, location=world_location)
            empty = context.object  # The newly created empty becomes the active object
            
            # Rename the empty to match the object's name
            empty.name = f"{obj.name}_Empty"
            
            # Parent the object to the empty
            obj.parent = empty
            
            # Update the object's location to maintain its world position (relative to the empty)
            obj.location = empty.matrix_world.inverted() @ world_location
        return {'FINISHED'}

def menu_func(self, context):
    self.layout.separator()
    self.layout.operator(EmptyAssistantOperator.bl_idname)

# Register and add to the "Object" menu
def register():
    bpy.utils.register_class(EmptyAssistantOperator)
    bpy.types.VIEW3D_MT_object.append(menu_func)

def unregister():
    bpy.utils.unregister_class(EmptyAssistantOperator)
    bpy.types.VIEW3D_MT_object.remove(menu_func)

if __name__ == "__main__":
    register()