fix(check_batch_cost): persist managed file before mutating response and propagate team_id

- Move setattr after store_unified_file_id so the response only receives the
  managed ID once the DB record is successfully written. Avoids serializing
  an orphaned managed ID into file_object when the store call fails.
- Populate team_id on the minimal UserAPIKeyAuth from job.team_id so the
  managed file record is created with the correct team ownership, allowing
  other team members to access the batch output file via /files/{id}/content.

Co-authored-by: Yassin Kortam <yassin@berri.ai>

Author: cursoragent

enterprise/litellm_enterprise/proxy/common_utils/check_batch_cost.py

@@ -306,7 +306,8 @@ async def check_batch_cost(self):
                 if managed_files_hook is not None:
                     from litellm.proxy._types import UserAPIKeyAuth
                     _minimal_auth = UserAPIKeyAuth(
-                        user_id=job.created_by or "default-user-id"
+                        user_id=job.created_by or "default-user-id",
+                        team_id=getattr(job, "team_id", None),
                     )
                     for _file_attr in ["output_file_id", "error_file_id"]:
                         _raw_file_id = getattr(response, _file_attr, None)
@@ -317,14 +318,14 @@ async def check_batch_cost(self):
                                     model_id=model_id,
                                     model_name=str(model_name) if model_name else deployment_info.model_name or None,
                                 )
-                                setattr(response, _file_attr, _unified_file_id)
                                 await managed_files_hook.store_unified_file_id(
                                     file_id=_unified_file_id,
                                     file_object=None,
                                     litellm_parent_otel_span=None,
                                     model_mappings={model_id: _raw_file_id},
                                     user_api_key_dict=_minimal_auth,
                                 )
+                                setattr(response, _file_attr, _unified_file_id)
                                 verbose_proxy_logger.info(
                                     f"CheckBatchCost: converted {_file_attr} "
                                     f"{_raw_file_id!r} -> managed ID for batch {batch_id}"