velocity-python 0.1.67__tar.gz → 0.1.68__tar.gz

{velocity_python-0.1.67 → velocity_python-0.1.68}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.67
+Version: 0.1.68
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.67 → velocity_python-0.1.68}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "velocity-python"
-version = "0.1.67"
+version = "0.1.68"
 authors = [
   { name="Velocity Team", email="info@codeclubs.org" },
 ]

{velocity_python-0.1.67 → velocity_python-0.1.68}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.67"
+__version__ = version = "0.1.68"
 
 import importlib as _importlib
 

{velocity_python-0.1.67 → velocity_python-0.1.68}/src/velocity/payment/stripe_adapter.py

@@ -521,6 +521,10 @@ class StripeAdapter(PaymentProcessorAdapter):
                     "platform_fee": platform_fee_cents,
                     "application_fee": application_fee_cents,
                     "client_amount": amount_cents - application_fee_cents,
+                    # Real Stripe processing cost from the settled BalanceTransaction
+                    # (None for an uncaptured manual auth); callers freeze it as the
+                    # 'stripe_settled' processor fee, else keep their estimate (M6).
+                    "actual_processor_fee": self._settled_processor_fee_cents(charge),
                 },
             }
 
@@ -621,6 +625,10 @@ class StripeAdapter(PaymentProcessorAdapter):
                 "platform_fee": platform_fee_cents,
                 "application_fee": payment_intent.application_fee_amount or application_fee_cents,
                 "client_amount": payment_intent.amount - (payment_intent.application_fee_amount or application_fee_cents),
+                # Real Stripe processing cost from the settled BalanceTransaction (None
+                # when not yet available); callers freeze it as the 'stripe_settled'
+                # processor fee, otherwise keep their formula estimate (FeeModel.md M6).
+                "actual_processor_fee": self._settled_processor_fee_cents(charge),
                 "error_message": None,
                 "metadata": {
                     "payment_intent": payment_intent,
@@ -849,13 +857,43 @@ class StripeAdapter(PaymentProcessorAdapter):
             latest_charge_id = getattr(payment_intent, "latest_charge", None)
             if latest_charge_id:
                 try:
-                    charge = stripe.Charge.retrieve(latest_charge_id, api_key=self._api_key)
+                    charge = stripe.Charge.retrieve(
+                        latest_charge_id,
+                        expand=["balance_transaction"],
+                        api_key=self._api_key,
+                    )
                 except stripe.error.StripeError:
                     charge = None
         if charge:
             upsert_stripe_charge_record(tx, charge, source=source)
         return charge
 
+    def _settled_processor_fee_cents(self, charge):
+        """Return the processor's actual fee (cents) from the charge's BalanceTransaction.
+
+        Stripe exposes the real processing cost on the charge's BalanceTransaction
+        (``balance_transaction.fee``). Returns ``None`` when it is not yet available
+        (e.g. an uncaptured manual authorization) or cannot be fetched — callers then
+        keep their formula estimate rather than over-reporting. Never raises. This is the
+        ``stripe_settled`` actual-cost source in CaringCent's FeeModel.md (M6).
+        """
+        if not charge:
+            return None
+        bt = getattr(charge, "balance_transaction", None)
+        if bt is None and isinstance(charge, dict):
+            bt = charge.get("balance_transaction")
+        try:
+            if bt is None:
+                return None
+            if isinstance(bt, str):
+                bt = stripe.BalanceTransaction.retrieve(bt, api_key=self._api_key)
+            fee = getattr(bt, "fee", None)
+            if fee is None and isinstance(bt, dict):
+                fee = bt.get("fee")
+            return int(fee) if fee is not None else None
+        except stripe.error.StripeError:
+            return None
+
     def _determine_account_status(self, account) -> str:
         """
         Determine human-readable account status from Stripe Account object.

{velocity_python-0.1.67 → velocity_python-0.1.68}/src/velocity_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.67
+Version: 0.1.68
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.67 → velocity_python-0.1.68}/tests/test_payment_stripe_adapter.py

@@ -238,6 +238,79 @@ def test_authorize_payment_upserts_payment_intent_mirror(adapter, monkeypatch):
     ]
 
 
+def test_settled_processor_fee_from_expanded_balance_transaction(adapter):
+    # Stripe exposes the real processing cost on the charge's BalanceTransaction.
+    charge = SimpleNamespace(
+        id="ch_123",
+        balance_transaction=SimpleNamespace(id="txn_1", fee=332),
+    )
+    assert adapter._settled_processor_fee_cents(charge) == 332
+
+
+def test_settled_processor_fee_retrieves_unexpanded_balance_transaction(adapter, monkeypatch):
+    monkeypatch.setattr(
+        "velocity.payment.stripe_adapter.stripe.BalanceTransaction.retrieve",
+        lambda txn_id, api_key=None: SimpleNamespace(id=txn_id, fee=288),
+    )
+    charge = SimpleNamespace(id="ch_123", balance_transaction="txn_1")
+    assert adapter._settled_processor_fee_cents(charge) == 288
+
+
+def test_settled_processor_fee_none_when_unavailable(adapter):
+    # No charge, or no balance transaction yet (uncaptured auth) -> None, never raises.
+    assert adapter._settled_processor_fee_cents(None) is None
+    assert (
+        adapter._settled_processor_fee_cents(
+            SimpleNamespace(id="ch_1", balance_transaction=None)
+        )
+        is None
+    )
+
+
+def test_charge_stored_payment_method_reports_actual_processor_fee(adapter, monkeypatch):
+    monkeypatch.setattr(
+        "velocity.payment.stripe_adapter.stripe.PaymentIntent.create",
+        lambda **kwargs: SimpleNamespace(
+            id="pi_123",
+            latest_charge="ch_123",
+            status="succeeded",
+            amount=10420,
+            currency="usd",
+            application_fee_amount=820,
+        ),
+    )
+    monkeypatch.setattr(
+        "velocity.payment.stripe_adapter.stripe.Charge.retrieve",
+        lambda charge_id, expand=None, api_key=None: SimpleNamespace(
+            id=charge_id,
+            balance_transaction=SimpleNamespace(id="txn_1", fee=332),
+        ),
+    )
+    monkeypatch.setattr(
+        "velocity.payment.stripe_adapter.upsert_stripe_payment_intent_record",
+        lambda *a, **k: None,
+    )
+    monkeypatch.setattr(
+        "velocity.payment.stripe_adapter.upsert_stripe_charge_record",
+        lambda *a, **k: None,
+    )
+
+    result = adapter.charge_stored_payment_method(
+        None,
+        {
+            "amount": 10420,
+            "payment_method": "pm_123",
+            "customer_profile_id": "cus_123",
+            "processor_account_id": "acct_123",
+            "revenue_split_percentage": 8,
+            "client_id": "client_1",
+        },
+    )
+
+    assert result["success"] is True
+    assert result["actual_processor_fee"] == 332
+
+
 def test_authorize_payment_uses_explicit_application_fee(adapter, monkeypatch):
     captured = {}