pylint fix

Author: Ayaz-Microsoft

src/backend/api/router.py

@@ -1466,4 +1466,4 @@ async def get_generated_image(blob_name: str):
         return Response(content=data, media_type="image/png")
     except Exception as exc:
         logging.error(f"Error retrieving image '{blob_name}': {exc}")
-        raise HTTPException(status_code=404, detail="Image not found")
+        raise HTTPException(status_code=404, detail="Image not found")

src/backend/orchestration/orchestration_manager.py

@@ -20,10 +20,8 @@
 from common.config.app_config import config
 from common.database.database_base import DatabaseBase
 from common.models.messages import TeamConfiguration
-from common.utils.markdown_utils import (
-    normalize_markdown_tables as _normalize_markdown_tables,
-    reflow_collapsed_table_line as _reflow_collapsed_table_line,
-)
+from common.utils.markdown_utils import \
+    normalize_markdown_tables as _normalize_markdown_tables
 from models.messages import AgentMessageStreaming, WebsocketMessageType
 from orchestration.connection_config import (connection_config,
                                              orchestration_config)
@@ -39,18 +37,18 @@
 apply_tool_history_leak_patch()
 
 _BARE_IMAGE_URL_RE = re.compile(
-   r"(?<![\(\]])"
-   r"(?<!\]\()"
-   r"("
-   # Absolute image URL (any host, or a backend /api/v4/images path)
-   r"https?://[^\s)]+?(?:/api/v4/images/[^\s)]+?|[^\s)]+?\.(?:png|jpe?g|gif|webp))"
-   # Bare relative backend image path (emitted by the MCP/backend image tools).
-   # The (?<![^\s]) guard requires the path to start at whitespace/string-start so
-   # it never matches the same substring inside an absolute URL.
-   r"|(?<![^\s])/api/v4/images/[^\s)]+?\.(?:png|jpe?g|gif|webp)"
-   r")"
-   r"(?=[\s)\]]|$)",
-   re.IGNORECASE,
+    r"(?<![\(\]])"
+    r"(?<!\]\()"
+    r"("
+    # Absolute image URL (any host, or a backend /api/v4/images path)
+    r"https?://[^\s)]+?(?:/api/v4/images/[^\s)]+?|[^\s)]+?\.(?:png|jpe?g|gif|webp))"
+    # Bare relative backend image path (emitted by the MCP/backend image tools).
+    # The (?<![^\s]) guard requires the path to start at whitespace/string-start so
+    # it never matches the same substring inside an absolute URL.
+    r"|(?<![^\s])/api/v4/images/[^\s)]+?\.(?:png|jpe?g|gif|webp)"
+    r")"
+    r"(?=[\s)\]]|$)",
+    re.IGNORECASE,
 )
 
 
@@ -222,25 +220,20 @@ async def get_current_or_new_orchestration(
             current is not None and current_team_id != team_config.team_id
         )
 
-
         cls.logger.info(
             "get_current_or_new_orchestration: user='%s' selected_team='%s' "
             "cached_team='%s' team_switched=%s team_changed=%s current_is_none=%s",
             user_id, team_config.team_id, current_team_id,
             team_switched, team_changed, current is None,
         )
 
-
         # Full rebuild: no workflow exists, team explicitly switched, or the
         # cached workflow belongs to a different team than the selected one.
         needs_full_rebuild = current is None or team_switched or team_changed
 
-
         # Lightweight reset: workflow finished but agents are still valid for the
         # same team (a team change always routes to full rebuild above so we
         # never reuse the previous team's agents here).
-
-
         needs_workflow_reset = not needs_full_rebuild and workflow_terminated
 
         if needs_full_rebuild:
@@ -859,4 +852,4 @@ async def _process_event_stream(
             if tool_approvals:
                 result["tool_approvals"] = tool_approvals
             return result
-        return None
+        return None

src/tests/backend/orchestration/test_orchestration_manager.py

@@ -1006,12 +1006,12 @@ def test_given_new_instance_when_init_then_logger_is_set(self):
         assert isinstance(manager.logger, logging.Logger)
 
 
-# =========================================================================
 # _normalize_markdown_tables (Bug 47810)
-# =========================================================================
 from backend.orchestration.orchestration_manager import (  # noqa: E402
     _normalize_markdown_tables,
-    _reflow_collapsed_table_line,
+)
+from common.utils.markdown_utils import (  # noqa: E402
+    reflow_collapsed_table_line as _reflow_collapsed_table_line,
 )