S5: delete AgentScheduleClient — no backward compatibility

User decision supersedes the deprecated-shim compromise: SchedulerClient is
the ONE schedule client. Removed schedule_client.py, the
conductor.ai.agents.schedule.client re-export shim, the ScheduleClient alias,
and OrkesClients.get_agent_schedule_client(). Runtime and control-plane
accessors now hand out get_scheduler_client() (same shared instance).
Tests migrate to a SchedulerClient double; a removal guard keeps the
symbols from creeping back.

Gates: unit 2568 passed / 1 skipped; suite21 11/11 live vs :8085 on a bare
OrkesSchedulerClient fixture; suite24 5/5 (1 flaky rerun).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: Kowser

e2e/test_suite21_scheduling.py

@@ -32,8 +32,8 @@
     Schedule,
     ScheduleNameConflict,
 )
-from conductor.ai.agents.schedule.client import ScheduleClient
 from conductor.client.ai.schedule import _from_workflow_schedule
+from conductor.client.scheduler_client import SchedulerClient
 
 pytestmark = [pytest.mark.e2e]
 
@@ -125,15 +125,12 @@ def agent_name(conductor_clients) -> Iterator[str]:
 
 
 @pytest.fixture()
-def schedule_client(conductor_clients) -> ScheduleClient:
-    return ScheduleClient(
-        conductor_clients.get_scheduler_client(),
-        conductor_clients.get_workflow_client(),
-    )
+def schedule_client(conductor_clients) -> SchedulerClient:
+    return conductor_clients.get_scheduler_client()
 
 
 @pytest.fixture(autouse=True)
-def clean_schedules(schedule_client: ScheduleClient, agent_name: str):
+def clean_schedules(schedule_client: SchedulerClient, agent_name: str):
     """Purge any leftover schedules for this agent before each test."""
     schedule_client.reconcile(agent_name, [])
     yield

examples/agents/hello_world_schedule.py

@@ -107,10 +107,10 @@ def main() -> None:
     # 1. Register the workflow that the schedule will fire.
     register_hello_world_workflow(agent_name)
 
-    # 2. Build the agent schedule client against the scheduler-enabled
+    # 2. Build the scheduler client against the scheduler-enabled
     #    Conductor instance.
     clients = OrkesClients(configuration=Configuration(base_url="http://localhost:8080"))
-    sched_client = clients.get_agent_schedule_client()
+    sched_client = clients.get_scheduler_client()
 
     # 3. Declarative deploy: one schedule, fires every 2 seconds.
     #    Quartz 6-field cron: 'sec min hour day month day-of-week'.

src/conductor/ai/agents/runtime/http_client.py

@@ -203,18 +203,19 @@ def _workflow_client(self) -> Any:
 
     @property
     def schedules(self) -> Any:
-        """Cron schedule lifecycle client (:class:`AgentScheduleClient`).
+        """Cron schedule lifecycle client (:class:`SchedulerClient`).
 
-        Exposes ``save/get/list_for_agent/pause/resume/delete/run_now/
-        preview_next/reconcile``. When bound to a runtime, this is the
-        *same* :class:`AgentScheduleClient` instance the runtime uses — there
-        is one shared schedule surface, not two.
+        Exposes ``get_schedule/save_schedule/get_all_schedules`` plus the
+        lifecycle methods ``pause/resume/delete/run_now/preview_next/
+        reconcile``. When bound to a runtime, this is the *same*
+        :class:`SchedulerClient` instance the runtime uses — there is one
+        shared schedule surface, not two.
         """
         if self._runtime is not None:
             return self._runtime.schedules_client()
         if self._schedule_client_instance is None:
             self._schedule_client_instance = (
-                self._get_orkes_clients().get_agent_schedule_client()
+                self._get_orkes_clients().get_scheduler_client()
             )
         return self._schedule_client_instance
 

src/conductor/ai/agents/runtime/runtime.py

@@ -2231,13 +2231,13 @@ def client(self) -> Any:
         return self._http
 
     def schedules_client(self) -> Any:
-        """Return the shared :class:`ScheduleClient` for this runtime.
+        """Return the shared :class:`SchedulerClient` for this runtime.
 
         Delegates to :attr:`client` so the runtime and the control-plane
         client expose the *same* schedule surface (one instance, not two).
         """
         if self._schedule_client_instance is None:
-            self._schedule_client_instance = self._clients.get_agent_schedule_client()
+            self._schedule_client_instance = self._clients.get_scheduler_client()
         return self._schedule_client_instance
 
     def _deploy_via_server(self, agent: Any, *, framework: Optional[str] = None) -> str:

src/conductor/ai/agents/schedule/client.py

@@ -4,9 +4,9 @@
 """Agent-facing clients and models for the Conductor SDK.
 
 Canonical home of the ``/agent/*`` transport (:class:`AgentApiClient`), the
-agent cron-schedule client (:class:`AgentScheduleClient`), their models, and
-the agent exception hierarchy. Build the clients via
-``OrkesClients.get_agent_client()`` / ``get_agent_schedule_client()``.
+agent schedule models (:class:`Schedule`/:class:`ScheduleInfo`), and the agent
+exception hierarchy. Build the transport via ``OrkesClients.get_agent_client()``;
+the schedule lifecycle lives on ``OrkesClients.get_scheduler_client()``.
 
 Everything here must stay importable without the ``[agents]`` extra and must
 not import ``conductor.ai`` (the agents authoring layer composes these
@@ -20,7 +20,6 @@
     AgentspanError,
 )
 from conductor.client.ai.schedule import Schedule, ScheduleInfo
-from conductor.client.ai.schedule_client import AgentScheduleClient, ScheduleClient
 from conductor.client.ai.schedule_errors import (
     InvalidCronExpression,
     ScheduleError,
@@ -32,11 +31,9 @@
     "AgentApiClient",
     "AgentAPIError",
     "AgentNotFoundError",
-    "AgentScheduleClient",
     "AgentspanError",
     "InvalidCronExpression",
     "Schedule",
-    "ScheduleClient",
     "ScheduleError",
     "ScheduleInfo",
     "ScheduleNameConflict",

src/conductor/client/ai/__init__.py

@@ -9,11 +9,11 @@
 prefixed wire name plus server-computed fields like ``next_run``.
 
 The private helpers below (payload mapping, wire-name prefixing, typed error
-translation) are shared by ``SchedulerClient``'s schedule-lifecycle methods,
-the module-level ``schedules.*`` API, and the deprecated ``AgentScheduleClient``
-shim. ``SchedulerClient``'s native ``get_schedule``/``save_schedule``/
-``get_all_schedules`` are the source of truth for reads/writes/lists; the
-``ScheduleInfo`` view exists for the module-level API.
+translation) are shared by ``SchedulerClient``'s schedule-lifecycle methods
+and the module-level ``schedules.*`` API. ``SchedulerClient``'s native
+``get_schedule``/``save_schedule``/``get_all_schedules`` are the source of
+truth for reads/writes/lists; the ``ScheduleInfo`` view exists for the
+module-level API.
 """
 
 from __future__ import annotations

src/conductor/client/ai/schedule.py

@@ -77,21 +77,5 @@ def get_agent_client(self) -> "AgentApiClient":  # noqa: F821
 
         return AgentApiClient(self.configuration)
 
-    def get_agent_schedule_client(self) -> "AgentScheduleClient":  # noqa: F821
-        """DEPRECATED — use :meth:`get_scheduler_client`; removal planned for the
-        next major release.
-
-        ``SchedulerClient`` now carries the schedule lifecycle itself
-        (``pause``/``resume``/``delete``/``run_now``/``preview_next``/``reconcile``),
-        with ``get_schedule``/``save_schedule``/``get_all_schedules`` as the
-        source of truth for reads/writes/lists. This returns a pure delegation
-        shim kept for backward compatibility.
-        """
-        # Imported lazily: this module is on virtually every SDK program's import
-        # path and must not grow import-time weight for the agent surface.
-        from conductor.client.ai.schedule_client import AgentScheduleClient
-
-        return AgentScheduleClient(self.get_scheduler_client(), self.get_workflow_client())
-
 
 ConductorClients = OrkesClients