Update streamlit_app.py

streamlit_app.py

@@ -30,7 +30,6 @@ except Exception:
 # google.generativeai SDK (try both legacy and newer patterns)
 try:
     import google.generativeai as genai
-    # some installs expose a top-level `responses` object, others require attribute access
     genai_responses = getattr(genai, "responses", None) or getattr(genai, "Responses", None)
     upload_file = getattr(genai, "upload_file", None)
     get_file = getattr(genai, "get_file", None)
@@ -388,9 +387,6 @@ def generate_via_responses_api(prompt_text: str, processed, model_used: str, max
     user_msg = {"role": "user", "content": "Please summarize the attached video."}
     call_variants = []
 
-    # Two common payload shapes: `genai.responses.generate(model=..., **payload)`
-    # and legacy `genai.Responses.create(...)` or model/chat wrappers.
-    # Build payloads for both styles.
     call_variants.append({"method": "responses.generate", "payload": {"model": model_used, "messages": [system_msg, user_msg], "files": [{"name": fname}], "safety_settings": safety_settings, "max_output_tokens": max_tokens}})
     call_variants.append({"method": "responses.generate_alt", "payload": {"model": model_used, "input": [{"text": prompt_text, "files": [{"name": fname}]}], "safety_settings": safety_settings, "max_output_tokens": max_tokens}})
     call_variants.append({"method": "legacy_responses_create", "payload": {"model": model_used, "input": prompt_text, "file": fname, "max_output_tokens": max_tokens}})
@@ -411,25 +407,21 @@ def generate_via_responses_api(prompt_text: str, processed, model_used: str, max
             try:
                 if progress_callback:
                     progress_callback("starting", int(time.time() - start), {"model": model_used, "attempt": attempts, "method": method})
-                # Preferred new API style if available
                 if genai_responses is not None and hasattr(genai_responses, "generate"):
-                    response = genai_responses.generate(**payload) if method.startswith("responses.generate") else genai_responses.generate(**payload)
+                    response = genai_responses.generate(**payload)
                     text = _normalize_genai_response(response)
                     if progress_callback:
                         progress_callback("done", int(time.time() - start), {"model": model_used, "attempt": attempts, "method": method})
                     return text
-                # Some versions expose a top-level Responses class/or function `genai.Responses.create`
                 if hasattr(genai, "Responses") and hasattr(genai.Responses, "create"):
                     response = genai.Responses.create(**payload)  # type: ignore
                     text = _normalize_genai_response(response)
                     if progress_callback:
                         progress_callback("done", int(time.time() - start), {"model": model_used, "attempt": attempts, "method": method})
                     return text
-                # Legacy model object style (older gemini SDK wrappers)
                 if hasattr(genai, "GenerativeModel"):
                     try:
                         model_obj = genai.GenerativeModel(model_name=model_used)
-                        # try chat pattern
                         if hasattr(model_obj, "start_chat"):
                             chat = model_obj.start_chat()
                             resp = chat.send_message(prompt_text, timeout=timeout)
@@ -440,14 +432,12 @@ def generate_via_responses_api(prompt_text: str, processed, model_used: str, max
                             return text
                     except Exception:
                         pass
-                # If none matched, raise to be caught below and trigger helpful error
                 raise RuntimeError("No supported response generation method available in installed google-generativeai package.")
             except Exception as e:
                 last_exc = e
                 msg = str(e)
                 logger.warning("Responses.generate error (model=%s attempt=%s method=%s): %s", model_used, attempts, method, msg)
                 if not is_transient_error(msg):
-                    # non-transient -> surface meaningful hint for common misconfig issues
                     if "No supported response generation method" in msg or "has no attribute" in msg or "module 'google.generativeai' has no attribute" in msg:
                         raise RuntimeError(
                             "Installed google-generativeai package does not expose a compatible Responses API. "