Should task and variant names be optional in event handlers?

[ginoperrotta]

In our agent class, the event handlers look like:

    def task_variant_start(
        self,
        task_name: typing.Optional[str],
        variant_name: typing.Optional[str],
    ) -> None:

Why are the names optional? Do we use that? If not, should we remove it?

[coreylowman]

Oh yeah that's from the very beginning when we were thinking it could be configurable whether agents receive task/variant information. It's currently not used at all! If we add a configuration option to turn it on/off then we can keep it, otherwise can probably delete it