From a465e6d1d45c1764a8eb9cf7b148f6e124e49580 Mon Sep 17 00:00:00 2001
From: JiangNan <1394485448@qq.com>
Date: Sun, 8 Mar 2026 12:49:02 +0800
Subject: [PATCH] fix: allow disabling autocommit via init()

The previous code used `config.autocommit = autocommit or config.autocommit`,
which means once autocommit is set to True it can never be turned off
again since `False or True` evaluates to `True`.

Use direct assignment so `init(autocommit=False)` works as expected.

Signed-off-by: JiangNan <1394485448@qq.com>
---
 src/ell/configurator.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/ell/configurator.py b/src/ell/configurator.py
index 5b79d1169..2defb95ca 100644
--- a/src/ell/configurator.py
+++ b/src/ell/configurator.py
@@ -230,7 +230,7 @@ def init(
                 'Failed importing SQLiteStore. Install with `pip install -U "ell-ai[all]"`. More info: https://docs.ell.so/installation')
     else:
         config.store = store
-    config.autocommit = autocommit or config.autocommit
+    config.autocommit = autocommit
 
     if default_api_params is not None:
         config.default_api_params.update(default_api_params)
@@ -261,3 +261,4 @@ def register_provider(provider: Provider, client_type: Type[Any]) -> None:
 def set_store(*args, **kwargs) -> None:
     raise DeprecationWarning(
         "The set_store function is deprecated and will be removed in a future version. Use ell.init(store=...) instead.")
+