fix: handle None/relative db_path in state tracker

Author: TPEmist

pop_pay/client.py

@@ -5,10 +5,11 @@
 from pop_pay.core.state import PopStateTracker
 
 class PopClient:
-    def __init__(self, provider: VirtualCardProvider, policy: GuardrailPolicy, engine: GuardrailEngine = None, db_path: str = "pop_state.db"):
+    def __init__(self, provider: VirtualCardProvider, policy: GuardrailPolicy, engine: GuardrailEngine = None, db_path: str = None):
         self.provider = provider
         self.policy = policy
-        self.state_tracker = PopStateTracker(db_path=db_path)
+        from pop_pay.core.state import DEFAULT_DB_PATH
+        self.state_tracker = PopStateTracker(db_path=db_path or DEFAULT_DB_PATH)
         self.engine = engine if engine is not None else GuardrailEngine()
 
     async def process_payment(self, intent: PaymentIntent) -> VirtualSeal:

pop_pay/core/state.py

@@ -11,7 +11,9 @@
 
 class PopStateTracker:
     def __init__(self, db_path: str = DEFAULT_DB_PATH):
-        os.makedirs(os.path.dirname(db_path), exist_ok=True)
+        parent = os.path.dirname(db_path)
+        if parent:
+            os.makedirs(parent, exist_ok=True)
         self.db_path = db_path
         # We keep the connection open for the lifetime of the tracker
         # This is especially important for :memory: databases