execweave 0.8.4__tar.gz → 0.8.5__tar.gz

{execweave-0.8.4 → execweave-0.8.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: execweave
-Version: 0.8.4
+Version: 0.8.5
 Summary: Turn AI-agent runtime behavior into a local interactive execution graph.
 Project-URL: Homepage, https://github.com/Irish-kw/ExecWeave
 Project-URL: Repository, https://github.com/Irish-kw/ExecWeave
@@ -83,7 +83,7 @@ ExecWeave is a source-available, local-first observability project that turns AI
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-This README documents **v0.8.4**.
+This README documents **v0.8.5**.
 
 ## Why ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.de.md

@@ -30,7 +30,7 @@ ExecWeave ist ein source-available, local-first Observability-Projekt, das Aktiv
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-Dieses README dokumentiert **v0.8.4**.
+Dieses README dokumentiert **v0.8.5**.
 
 ## Warum ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.fr.md

@@ -30,7 +30,7 @@ ExecWeave est un projet d’observabilité source-available et local-first qui t
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-Ce README décrit **v0.8.4**.
+Ce README décrit **v0.8.5**.
 
 ## Pourquoi ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.ja.md

@@ -30,7 +30,7 @@ ExecWeave は source-available / local-first の observability プロジェク
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-この README は **v0.8.4** を説明します。
+この README は **v0.8.5** を説明します。
 
 ## ExecWeave を使う理由
 

{execweave-0.8.4 → execweave-0.8.5}/README.ko.md

@@ -30,7 +30,7 @@ ExecWeave는 source-available, local-first observability 프로젝트입니다.
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-이 README는 **v0.8.4**을 설명합니다.
+이 README는 **v0.8.5**을 설명합니다.
 
 ## 왜 ExecWeave인가
 

{execweave-0.8.4 → execweave-0.8.5}/README.md

@@ -30,7 +30,7 @@ ExecWeave is a source-available, local-first observability project that turns AI
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-This README documents **v0.8.4**.
+This README documents **v0.8.5**.
 
 ## Why ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.ru.md

@@ -30,7 +30,7 @@ ExecWeave — source-available, local-first проект наблюдаемос
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-Этот README описывает **v0.8.4**.
+Этот README описывает **v0.8.5**.
 
 ## Почему ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.zh-CN.md

@@ -30,7 +30,7 @@ ExecWeave 是一个 source-available、local-first 的可观测性项目，把 A
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-本 README 对应 **v0.8.4**。
+本 README 对应 **v0.8.5**。
 
 ## 为什么是 ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/README.zh-TW.md

@@ -30,7 +30,7 @@ ExecWeave 是一個 source-available、local-first 的可觀測性專案，會
   <img src="docs/assets/codex.gif" alt="ExecWeave animated live demo" width="100%">
 </p>
 
-本 README 對應 **v0.8.4**。
+本 README 對應 **v0.8.5**。
 
 ## 為什麼是 ExecWeave
 

{execweave-0.8.4 → execweave-0.8.5}/pyproject.toml

@@ -4,8 +4,8 @@ build-backend = "hatchling.build"
 
 [project]
 name = "execweave"
-# ExecWeave v0.8.4 release metadata
-version = "0.8.4"
+# ExecWeave v0.8.5 release metadata
+version = "0.8.5"
 description = "Turn AI-agent runtime behavior into a local interactive execution graph."
 readme = "README.md"
 requires-python = ">=3.10"

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/__init__.py

@@ -1,3 +1,3 @@
 """ExecWeave runtime graph package."""
 
-__version__ = "0.8.4"
+__version__ = "0.8.5"

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/antigravity_full_fidelity_base.py

@@ -11,12 +11,24 @@ def _now() -> str:
     return datetime.now(timezone.utc).isoformat().replace("+00:00", "Z")
 
 
-def _agent() -> dict[str, Any]:
+def _agent(payload: dict[str, Any]) -> dict[str, Any]:
+    conversation_id = payload.get("conversationId")
+    if isinstance(conversation_id, str) and conversation_id:
+        return {
+            "type": "agent",
+            "id": f"agent:antigravity:conversation:{conversation_id}",
+            "name": "Antigravity conversation",
+            "attributes": {
+                "provider": "antigravity",
+                "conversation_id": conversation_id,
+                "identity_semantics": "provider_conversation_id",
+            },
+        }
     return {
         "type": "agent",
         "id": "agent:Antigravity",
         "name": "Antigravity",
-        "attributes": {},
+        "attributes": {"provider": "antigravity"},
     }
 
 
@@ -70,7 +82,7 @@ def antigravity_hook_to_content_events(
             content_observation_event(
                 timestamp=observed_at,
                 provider="antigravity",
-                source=_agent(),
+                source=_agent(payload),
                 reference=reference,
                 relation="OBSERVED_PROVIDER_METADATA",
                 observed_field="hook_metadata",
@@ -122,7 +134,7 @@ def antigravity_hook_to_content_events(
             content_observation_event(
                 timestamp=observed_at,
                 provider="antigravity",
-                source=_agent(),
+                source=_agent(payload),
                 reference=reference,
                 relation="OBSERVED_TOOL_ERROR",
                 observed_field="error",

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/auto_specialized.py

@@ -54,6 +54,14 @@ def _is_ollama_serve(command: list[str]) -> bool:
     )
 
 
+def _is_ollama_run(command: list[str]) -> bool:
+    return (
+        len(command) >= 2
+        and _command_name(command[0]) == "ollama"
+        and command[1].lower() == "run"
+    )
+
+
 def _is_llamacpp_server(command: list[str]) -> bool:
     return bool(command) and _command_name(command[0]) == "llama-server"
 
@@ -284,6 +292,48 @@ def run_post_command_specialized_probe(
                 time.sleep(_POST_PROBE_RETRY_SECONDS)
 
 
+
+@contextmanager
+def auto_specialized_launch(command: list[str]) -> Iterator[dict[str, str]]:
+    """Prepare child-only launch wiring for supported transparent local integrations."""
+    environment = dict(os.environ)
+    configured_sidecar = os.environ.get(_SEMANTIC_ENV)
+    if not configured_sidecar or not _is_ollama_run(command):
+        yield environment
+        return
+    upstream = _ollama_endpoint_from_environment()
+    if upstream is None:
+        yield environment
+        return
+
+    from .http_proxy import ExecWeaveHTTPProxyServer, ProxyConfig
+
+    sidecar = Path(configured_sidecar).expanduser().resolve()
+    try:
+        server = ExecWeaveHTTPProxyServer(
+            ("127.0.0.1", 0),
+            ProxyConfig(upstream=upstream, sidecar=sidecar, mode="ollama"),
+        )
+    except OSError:
+        yield environment
+        return
+    thread = threading.Thread(
+        target=server.serve_forever,
+        kwargs={"poll_interval": 0.05},
+        name="execweave-ollama-run-relay",
+        daemon=True,
+    )
+    thread.start()
+    host, port = server.server_address[:2]
+    environment["OLLAMA_HOST"] = f"http://{host}:{port}"
+    try:
+        yield environment
+    finally:
+        server.shutdown()
+        server.server_close()
+        thread.join(timeout=2.0)
+
+
 @contextmanager
 def auto_specialized_probe(command: list[str]) -> Iterator[None]:
     """Run supported local specialized probes without affecting command execution.

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/collector.py

@@ -10,6 +10,7 @@ from typing import Iterable
 import psutil
 
 from .auto_specialized import (
+    auto_specialized_launch,
     auto_specialized_probe,
     prepare_post_command_specialized_probe,
     run_post_command_specialized_probe,
@@ -95,6 +96,7 @@ def infer_agent_name(command: Iterable[str]) -> str:
         "gemini": "Gemini CLI",
         "cursor": "Cursor",
         "opencode": "OpenCode",
+        "ollama": "Ollama",
     }
     return known.get(executable, basename)
 
@@ -167,20 +169,25 @@ class RuntimeCollector:
         return_code = 1
         post_command_probe = prepare_post_command_specialized_probe(command)
         try:
-            process = subprocess.Popen(launch_command, cwd=str(self.watch_root))
-            root = psutil.Process(process.pid)
-            snapshot = _safe_process_snapshot(root)
-            if snapshot is not None:
-                self._record_process_start(snapshot, parent=session, relation="LAUNCHED")
+            with auto_specialized_launch(command) as launch_environment:
+                process = subprocess.Popen(
+                    launch_command,
+                    cwd=str(self.watch_root),
+                    env=launch_environment,
+                )
+                root = psutil.Process(process.pid)
+                snapshot = _safe_process_snapshot(root)
+                if snapshot is not None:
+                    self._record_process_start(snapshot, parent=session, relation="LAUNCHED")
 
-            with auto_specialized_probe(command):
-                while process.poll() is None:
-                    self._sample_process_tree(root)
-                    time.sleep(self.poll_interval)
+                with auto_specialized_probe(command):
+                    while process.poll() is None:
+                        self._sample_process_tree(root)
+                        time.sleep(self.poll_interval)
 
-                self._sample_process_tree(root)
-                self._mark_disappeared_processes(set())
-            return_code = int(process.returncode or 0)
+                    self._sample_process_tree(root)
+                    self._mark_disappeared_processes(set())
+                return_code = int(process.returncode or 0)
             run_post_command_specialized_probe(
                 post_command_probe,
                 return_code=return_code,

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/conversation_preview.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import json
+import re
 from pathlib import Path
 from typing import Any
 
@@ -269,6 +270,18 @@ def _structured_messages(
     return messages
 
 
+_ANTIGRAVITY_USER_REQUEST_RE = re.compile(
+    r"<USER_REQUEST>\s*(?P<body>.*?)\s*</USER_REQUEST>",
+    re.DOTALL,
+)
+
+
+def _antigravity_user_text(text: str) -> str:
+    """Return the actual user request, excluding Antigravity's metadata envelope."""
+    match = _ANTIGRAVITY_USER_REQUEST_RE.search(text)
+    return match.group("body").strip() if match is not None else text.strip()
+
+
 def _line_transcript_messages(
     path: Path,
     *,
@@ -291,26 +304,71 @@ def _line_transcript_messages(
             continue
         if not isinstance(record, dict):
             continue
+
+        if antigravity:
+            role = str(record.get("source") or "").strip().lower()
+            record_type = str(record.get("type") or "").strip().lower()
+            text = _text_parts(record.get("content") or record.get("text"))
+            record_timestamp = record.get("created_at") or record.get("timestamp") or timestamp
+            step_index = record.get("step_index")
+            record_ordinal = (
+                step_index
+                if isinstance(step_index, int) and not isinstance(step_index, bool)
+                else (ordinal if isinstance(ordinal, int) else 0) + index
+            )
+
+            # Current Antigravity uses USER_EXPLICIT / USER_INPUT for real user turns.
+            # Keep the older USER/HUMAN spelling for archived compatibility.
+            if role in {"user_explicit", "user", "human"} and record_type in {
+                "user_input",
+                "user_message",
+                "",
+            }:
+                if text:
+                    messages.append(
+                        _message(
+                            timestamp=record_timestamp,
+                            ordinal=record_ordinal,
+                            kind="user_message",
+                            sender="user",
+                            recipient=agent_path,
+                            text=_antigravity_user_text(text),
+                        )
+                    )
+                continue
+
+            # PLANNER_RESPONSE is Antigravity's user-visible model surface. GENERIC
+            # records are tool/runtime results (define_subagent, manage_subagents,
+            # schedule, send_message acknowledgements, ...), so they must never be
+            # eligible for the conversation Final response card.
+            if role in {"model", "assistant"} and record_type == "planner_response":
+                if text:
+                    messages.append(
+                        _message(
+                            timestamp=record_timestamp,
+                            ordinal=record_ordinal,
+                            kind="assistant_message",
+                            sender=agent_path,
+                            recipient=None,
+                            text=text,
+                            phase="planner_response",
+                        )
+                    )
+                continue
+            continue
+
         record_timestamp = record.get("timestamp") or timestamp
         record_ordinal = record.get("ordinal")
         if not isinstance(record_ordinal, int):
             record_ordinal = (ordinal if isinstance(ordinal, int) else 0) + index
-
-        if antigravity:
-            role = str(record.get("source") or "").lower()
-            text = _text_parts(record.get("content") or record.get("text"))
-            phase = str(record.get("type") or "").lower() or "response"
+        record_type = str(record.get("type") or "").lower()
+        payload = record.get("message")
+        if isinstance(payload, dict):
+            role = str(payload.get("role") or record_type).lower()
+            text = _text_parts(payload.get("content"))
         else:
-            record_type = str(record.get("type") or "").lower()
-            payload = record.get("message")
-            if isinstance(payload, dict):
-                role = str(payload.get("role") or record_type).lower()
-                text = _text_parts(payload.get("content"))
-            else:
-                role = record_type
-                text = _text_parts(record.get("content") or record.get("text"))
-            phase = "response"
-
+            role = record_type
+            text = _text_parts(record.get("content") or record.get("text"))
         if not text:
             continue
         if role in {"user", "human"}:
@@ -333,7 +391,7 @@ def _line_transcript_messages(
                     sender=agent_path,
                     recipient=None,
                     text=text,
-                    phase=phase,
+                    phase="response",
                 )
             )
     return messages
@@ -722,7 +780,8 @@ def conversation_preview(
 
     if not messages:
         return None
-    truncated = len(messages) > _MAX_PREVIEW_MESSAGES
+    preserve_history = content_kind.startswith("antigravity.conversation_transcript")
+    truncated = not preserve_history and len(messages) > _MAX_PREVIEW_MESSAGES
     if truncated:
         messages = messages[:10] + messages[-(_MAX_PREVIEW_MESSAGES - 10) :]
     return {

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/conversation_records.py

@@ -15,14 +15,7 @@ _core_merge_conversation_previews = _core._merge_conversation_previews
 
 
 def _antigravity_step_ordinals(path: str | Path) -> list[int | None]:
-    """Recover stable provider record positions for visible Antigravity messages.
-
-    Antigravity Stop archives are cumulative snapshots of the same transcript. The
-    archive edge sequence therefore changes between snapshots and cannot identify a
-    transcript record. Live-verified transcript records expose ``step_index``; use it
-    when an explicit ``ordinal`` is absent so old turns keep the same identity across
-    repeated Stop archives.
-    """
+    """Recover stable step indexes for user-visible Antigravity transcript records."""
     source_path = Path(path).expanduser().resolve(strict=False)
     try:
         lines = source_path.read_text(encoding="utf-8").splitlines()
@@ -39,13 +32,17 @@ def _antigravity_step_ordinals(path: str | Path) -> list[int | None]:
             continue
         if not isinstance(record, dict):
             continue
-        role = str(record.get("source") or "").lower()
-        if role not in {"user", "human", "assistant", "model"}:
-            continue
+        role = str(record.get("source") or "").strip().lower()
+        record_type = str(record.get("type") or "").strip().lower()
         text = _preview_module._text_parts(record.get("content") or record.get("text"))
-        if not text:
+        visible_user = role in {"user_explicit", "user", "human"} and record_type in {
+            "user_input",
+            "user_message",
+            "",
+        }
+        visible_assistant = role in {"model", "assistant"} and record_type == "planner_response"
+        if not text or not (visible_user or visible_assistant):
             continue
-
         record_ordinal = record.get("ordinal")
         if isinstance(record_ordinal, int) and not isinstance(record_ordinal, bool):
             ordinals.append(record_ordinal)
@@ -224,6 +221,112 @@ def _same_merged_execution(
     return isinstance(observed_thread, str) and observed_thread in evidence_thread_ids
 
 
+
+def _history_message_key(message: dict[str, Any]) -> tuple[object, ...]:
+    return (
+        message.get("ordinal"),
+        message.get("kind"),
+        message.get("sender"),
+        message.get("recipient"),
+        message.get("text"),
+        message.get("content_state"),
+        message.get("phase"),
+        message.get("task_name"),
+    )
+
+
+def _restore_complete_histories(
+    entries: list[dict[str, Any]],
+    snapshots: list[tuple[dict[str, Any], dict[str, Any]]],
+) -> None:
+    """Never silently drop middle rounds for providers whose history is user-facing."""
+    for entry in entries:
+        provider = str(entry.get("provider") or "").lower()
+        if provider not in {"antigravity", "ollama"}:
+            continue
+        merged = entry.get("conversation_preview")
+        source_id = entry.get("source_id")
+        if not isinstance(merged, dict) or not isinstance(source_id, str) or not source_id:
+            continue
+        observed: list[dict[str, Any]] = []
+        for observed_entry, observed_preview in snapshots:
+            if str(observed_entry.get("provider") or "").lower() != provider:
+                continue
+            if observed_entry.get("source_id") != source_id:
+                continue
+            for message in observed_preview.get("messages") or []:
+                if isinstance(message, dict):
+                    observed.append(dict(message))
+        if not observed:
+            continue
+        if provider == "ollama" and not any(
+            message.get("content_role")
+            in {"ollama_request_surface", "ollama_response_surface"}
+            for message in observed
+        ):
+            # Legacy/provider-neutral root records have no wire-surface provenance.
+            # Leave the core incremental merge untouched instead of applying the
+            # Ollama cumulative-history repair to unrelated synthetic/root records.
+            continue
+        indexed = list(enumerate(observed))
+        indexed.sort(
+            key=lambda pair: (
+                str(pair[1].get("timestamp") or ""),
+                pair[1].get("ordinal")
+                if isinstance(pair[1].get("ordinal"), int)
+                else 2**63 - 1,
+                pair[0],
+            )
+        )
+        seen: set[tuple[object, ...]] = set()
+        messages: list[dict[str, Any]] = []
+        for _, message in indexed:
+            key = _history_message_key(message)
+            if key in seen:
+                continue
+            seen.add(key)
+            messages.append(message)
+        merged["message_count"] = len(messages)
+        merged["messages_truncated"] = False
+        merged["messages"] = messages
+
+
+def _ollama_current_turn(preview: dict[str, Any]) -> None:
+    """Reduce a cumulative Ollama chat request to the exact turn this exchange created."""
+    messages = [message for message in preview.get("messages") or [] if isinstance(message, dict)]
+    users = [
+        message
+        for message in messages
+        if message.get("sender") == "user"
+        and message.get("content_role") == "ollama_request_surface"
+    ]
+    response_assistants = [
+        message
+        for message in messages
+        if message.get("content_role") == "ollama_response_surface"
+        and str(message.get("kind") or "").startswith("assistant")
+    ]
+    if not users:
+        return
+    current = [users[-1]]
+    if response_assistants:
+        current.append(response_assistants[-1])
+    preview["message_count"] = len(current)
+    preview["messages_truncated"] = False
+    preview["messages"] = current
+
+
+def _ollama_root_agent_id(graph: dict[str, Any]) -> str | None:
+    candidates = []
+    for node in graph.get("nodes", []):
+        if not isinstance(node, dict) or node.get("type") != "agent":
+            continue
+        node_id = node.get("id")
+        if isinstance(node_id, str) and node_id.lower() == "agent:ollama":
+            candidates.append(node_id)
+    return candidates[0] if len(candidates) == 1 else None
+
+
 def _merge_conversation_previews(entries: list[dict[str, Any]]) -> None:
     """Merge execution identity first, then reconcile duplicate Codex observations.
 
@@ -232,6 +335,28 @@ def _merge_conversation_previews(entries: list[dict[str, Any]]) -> None:
     aliases that changed during publication, then lets complementary Codex observations
     render once while retaining provenance. Raw evidence entries are never removed.
     """
+    # Preserve which side of an Ollama exchange produced each visible message.
+    # Cumulative request.messages can contain historical assistant text, so publication
+    # must select the assistant observed on the response surface rather than guessing
+    # from list position.
+    for entry in entries:
+        if str(entry.get("provider") or "").lower() != "ollama":
+            continue
+        preview = entry.get("conversation_preview")
+        if not isinstance(preview, dict):
+            continue
+        kind = str(entry.get("content_kind") or "").lower()
+        relation = str(entry.get("relation") or "").upper()
+        if "assistant_messages" in kind or relation == "OBSERVED_ASSISTANT_MESSAGES":
+            role = "ollama_response_surface"
+        elif "request_messages" in kind or "request_prompt" in kind or "request_input" in kind:
+            role = "ollama_request_surface"
+        else:
+            continue
+        for message in preview.get("messages") or []:
+            if isinstance(message, dict):
+                message["content_role"] = role
+
     snapshots: list[tuple[dict[str, Any], dict[str, Any]]] = []
     for entry in entries:
         preview = entry.get("conversation_preview")
@@ -253,6 +378,7 @@ def _merge_conversation_previews(entries: list[dict[str, Any]]) -> None:
 
     _core_merge_conversation_previews(entries)
     _repair_parent_thread_aliases(entries)
+    _restore_complete_histories(entries, snapshots)
 
     for entry in entries:
         if str(entry.get("provider") or "").lower() != "codex":
@@ -294,8 +420,48 @@ def _merge_conversation_previews(entries: list[dict[str, Any]]) -> None:
         merged_preview["messages"] = messages
 
 
+
+_core_conversation_record_entries = _core.conversation_record_entries
+
+
+def conversation_record_entries(
+    graph: dict[str, Any],
+    run_root: str | Path | None = None,
+) -> list[dict[str, Any]]:
+    """Publish root-only Ollama turns under the one run root without changing raw evidence."""
+    entries = _core_conversation_record_entries(graph, run_root)
+    root_id = _ollama_root_agent_id(graph)
+    if root_id is None:
+        return entries
+
+    normalized = False
+    for entry in entries:
+        if str(entry.get("provider") or "").lower() != "ollama":
+            continue
+        preview = entry.get("conversation_preview")
+        if not isinstance(preview, dict) or preview.get("is_root") is not True:
+            continue
+        if entry.get("source_type") != "inference_request":
+            continue
+        _ollama_current_turn(preview)
+        preview["provider_native_id"] = root_id
+        original_source_id = entry.get("source_id")
+        if isinstance(original_source_id, str) and original_source_id != root_id:
+            entry["evidence_source_id"] = original_source_id
+            entry["evidence_source_type"] = entry.get("source_type")
+        entry["source_id"] = root_id
+        entry["source_name"] = "Ollama"
+        entry["source_type"] = "agent"
+        normalized = True
+
+    if normalized:
+        _merge_conversation_previews(entries)
+    return entries
+
+
 # Functions defined in the core module resolve these globals at call time. Rebinding
 # keeps the public API stable while tightening identity before the observation stage.
 _core.conversation_preview = _conversation_preview
 _core._conversation_identity_keys = _conversation_identity_keys
 _core._merge_conversation_previews = _merge_conversation_previews
+_core.conversation_record_entries = conversation_record_entries

{execweave-0.8.4 → execweave-0.8.5}/src/execweave/model_runtime_full_fidelity.py

@@ -157,6 +157,9 @@ def _assistant_messages(response: object) -> list[Any]:
     native = response.get("message")
     if isinstance(native, dict):
         messages.append(native)
+    generated = response.get("response")
+    if isinstance(generated, str) and generated:
+        messages.append({"role": "assistant", "content": generated})
     choices = response.get("choices")
     if isinstance(choices, list):
         for choice in choices: