Remove new exposure of plugins

Author: yuandrew

temporalio/client.py

@@ -350,11 +350,6 @@ def api_key(self, value: Optional[str]) -> None:
         self.service_client.config.api_key = value
         self.service_client.update_api_key(value)
 
-    @property
-    def plugins(self) -> Sequence[Plugin]:
-        """Plugins used by this client."""
-        return self._config["plugins"]
-
     # Overload for no-param workflow
     @overload
     async def start_workflow(

temporalio/worker/_worker.py

@@ -376,7 +376,6 @@ def __init__(
                     f"The same plugin type {type(client_plugin)} is present from both client and worker. It may run twice and may not be the intended behavior."
                 )
         plugins = plugins_from_client + list(plugins)
-        config["plugins"] = plugins
 
         self.plugins = plugins
         for plugin in plugins:
@@ -557,7 +556,9 @@ def check_activity(activity):
             )
 
         worker_plugins = [plugin.name() for plugin in config.get("plugins", [])]
-        client_plugins = [plugin.name() for plugin in config["client"].plugins]
+        client_plugins = [
+            plugin.name() for plugin in config["client"].config()["plugins"]
+        ]
         plugins = list(set(worker_plugins + client_plugins))
 
         # Create bridge worker last. We have empirically observed that if it is

tests/test_plugins.py

@@ -152,9 +152,6 @@ async def test_worker_plugin_basic_config(client: Client) -> None:
     )
     task_queue = worker.config().get("task_queue")
     assert task_queue is not None and task_queue.startswith("replaced_queue")
-    assert [p.name() for p in worker.config().get("plugins", [])] == [
-        MyWorkerPlugin().name()
-    ]
 
     # Test client plugin propagation to worker plugins
     new_config = client.config()
@@ -165,9 +162,6 @@ async def test_worker_plugin_basic_config(client: Client) -> None:
     )
     task_queue = worker.config().get("task_queue")
     assert task_queue is not None and task_queue.startswith("combined")
-    assert [p.name() for p in worker.config().get("plugins", [])] == [
-        MyCombinedPlugin().name()
-    ]
 
     # Test both. Client propagated plugins are called first, so the worker plugin overrides in this case
     worker = Worker(
@@ -178,10 +172,6 @@ async def test_worker_plugin_basic_config(client: Client) -> None:
     )
     task_queue = worker.config().get("task_queue")
     assert task_queue is not None and task_queue.startswith("replaced_queue")
-    assert [p.name() for p in worker.config().get("plugins", [])] == [
-        MyCombinedPlugin().name(),
-        MyWorkerPlugin().name(),
-    ]
 
 
 async def test_worker_duplicated_plugin(client: Client) -> None: