feat: remove sync context support and related code

Author: JakkuSakura

docs/api/context.md

@@ -61,56 +61,3 @@ asyncio.run(run_all_hosts())
 `AsyncHostContext` scopes the helper to a single host (specified by name or by a
 `Host` instance) but otherwise behaves the same. Both helpers accept a
 `hosts=` override when you need to target a subset of hosts for a specific call.
-
-## SyncContext & SyncHostContext
-
-For synchronous code, `pyinfra.sync_context.SyncContext` and
-`pyinfra.sync_context.SyncHostContext` mirror the async API without any asyncio
-wrapping. The synchronous helpers still manage host connections for you and make
-host facts available via `host.get_fact(...)`.
-
-```python
-from pyinfra.api import Config, State, deploy
-from pyinfra.facts.server import Hostname
-from pyinfra.context import host
-from pyinfra.operations import server
-from pyinfra.sync_context import SyncContext, SyncHostContext
-
-
-def run_all_hosts():
-    inventory = ...
-    state = State(inventory, Config())
-
-    @deploy("Example sync deploy")
-    def my_sync_deploy():
-        server.shell(name="Sync deploy op", commands="echo sync deploy")
-
-    with SyncContext(state):
-        server.shell("echo hello from sync")
-        hostnames = {
-            host: host.get_fact(Hostname)
-            for host in state.inventory
-        }
-        print(f"Hostnames: {hostnames}")
-
-        my_sync_deploy()
-
-
-def run_single_host(state, host_name):
-    @deploy("Sync per-host deploy")
-    def my_sync_host_deploy():
-        server.shell(name="Sync host deploy op", commands="echo sync host deploy")
-
-    with SyncHostContext(state, host_name):
-        assert host is not None
-        server.shell("echo from sync single host")
-        hostname = host.get_fact(Hostname)
-        print(hostname)
-
-        my_sync_host_deploy()
-```
-
-`SyncHostContext` accepts either the host name or the `Host` object directly and
-ensures only that host is connected for the duration of the context. The sync
-and async helpers can be mixed within the same project—choose the one that fits
-the surrounding code.

src/pyinfra/__init__.py

@@ -17,7 +17,6 @@
 from .version import __version__  # noqa
 
 from .async_context import AsyncContext as AsyncContext, AsyncHostContext as AsyncHostContext  # noqa: E402,F401
-from .sync_context import SyncContext as SyncContext, SyncHostContext as SyncHostContext  # noqa: E402,F401
 
 # Initialise base classes - this sets the context modules to point at the underlying
 # class objects (Host, etc), which makes ipython/etc work as expected.

src/pyinfra/api/deploy.py

@@ -15,7 +15,7 @@
 
 from .arguments import pop_global_arguments
 from .arguments_typed import PyinfraOperation
-from .operation import get_async_context, get_sync_context
+from .operation import get_async_context
 from .exceptions import PyinfraError
 from .host import Host
 from .state import StateStage
@@ -91,10 +91,6 @@ def decorated_func(*args: P.args, **kwargs: P.kwargs) -> Any:
         if async_context is not None:
             return async_context._call_wrapped_deploy(decorated_func, args, kwargs)
 
-        sync_context = get_sync_context()
-        if sync_context is not None:
-            return sync_context._call_wrapped_deploy(decorated_func, args, kwargs)
-
         deploy_kwargs, _ = pop_global_arguments(context.state, context.host, kwargs)
 
         deploy_data = getattr(func, "deploy_data", None)

src/pyinfra/api/host.py

@@ -372,16 +372,12 @@ def get_fact(self, name_or_cls, *args, **kwargs):
         """
         Get a fact for this host, reading from the cache if present.
         """
-        from pyinfra.api.operation import get_async_context, get_sync_context
+        from pyinfra.api.operation import get_async_context
 
         async_context = get_async_context()
         if async_context is not None:
             return async_context._call_wrapped_fact(self, name_or_cls, args, kwargs)
 
-        sync_context = get_sync_context()
-        if sync_context is not None:
-            return sync_context._call_wrapped_fact(self, name_or_cls, args, kwargs)
-
         return _load_fact(self.state, self, name_or_cls, args=args, kwargs=kwargs)
 
     # Connector proxy

src/pyinfra/api/operation.py

@@ -42,12 +42,6 @@
     "pyinfra_current_async_context",
     default=None,
 )
-_current_sync_context: ContextVar[Any | None] = ContextVar(
-    "pyinfra_current_sync_context",
-    default=None,
-)
-
-
 def push_async_context(ctx: Any) -> Token:
     return _current_async_context.set(ctx)
 
@@ -64,22 +58,6 @@ def get_async_context() -> Any | None:
     return _current_async_context.get()
 
 
-def push_sync_context(ctx: Any) -> Token:
-    return _current_sync_context.set(ctx)
-
-
-def reset_sync_context(token: Token) -> None:
-    _current_sync_context.reset(token)
-
-
-def suspend_sync_context() -> Token:
-    return _current_sync_context.set(None)
-
-
-def get_sync_context() -> Any | None:
-    return _current_sync_context.get()
-
-
 if TYPE_CHECKING:
     from pyinfra.connectors.util import CommandOutput
 
@@ -314,10 +292,6 @@ def decorated_func(*args: P.args, **kwargs: P.kwargs) -> OperationMeta:
         if async_context is not None:
             return async_context._call_wrapped_operation(decorated_func, args, kwargs)
 
-        sync_context = get_sync_context()
-        if sync_context is not None:
-            return sync_context._call_wrapped_operation(decorated_func, args, kwargs)
-
         state = context.state
         host = context.host