fix: resolve all flake8 lint errors across vultron/ and test/

P1 (runtime crash) fixes:
- extractor.py: move VultronEvent/EVENT_CLASS_MAP/VultronTypes imports
  to module level; remove forward-reference string annotation
- helpers.py: remove spurious f-prefix from non-interpolating f-string
- robot.py: add missing _parse_args() function
- mcp_server.py: fix 9 use-case call signatures (SvcXxx(dl, req).execute())

P3/P4 (unused imports) fixes:
- Remove unused imports across 14 test files, 12 demo scripts, 5 vocab
  example files, and various adapter/use-case modules
- Restore BASE_URL in demo modules with # noqa: F401 (needed for test
  monkeypatching via mp.setattr)
- Fix F811 (logging redefinition) in test_embargo_use_cases.py
- Fix F841 (assigned-but-unused vars) in test_cli.py, test_create_tree.py,
  test_create_activity.py, and triggers/embargo.py

E402 (imports not at top) fixes:
- test_trigger_services.py: move FUTURE_DATETIME constant after all imports
- test_performance.py: move logger = ... after all imports; remove unused
  DataLayer import

E203 (.flake8):
- Add E203 to extend-ignore (Black-style slice whitespace)

All # noqa placement fixed to appear on the opening 'from X import (' line,
not on the closing ')' line (where flake8 ignores them).

Result: flake8 clean, 985 tests pass, Black unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: ahouseholder

.flake8

@@ -1,5 +1,5 @@
 [flake8]
-extend-ignore = E501
+extend-ignore = E203,E501
 exclude =
     .git,
     __pycache__,

test/api/v2/backend/test_outbox.py

@@ -37,7 +37,6 @@
 from types import SimpleNamespace
 from unittest.mock import MagicMock
 
-import pytest
 
 from vultron.adapters.driving.fastapi import outbox_handler as oh
 

test/api/v2/backend/test_trigger_services.py

@@ -26,9 +26,6 @@
 
 import pytest
 from fastapi import HTTPException
-
-FUTURE_DATETIME = datetime(2099, 12, 1, tzinfo=timezone.utc)
-
 from vultron.adapters.driven.db_record import object_to_record
 from vultron.adapters.driving.fastapi._trigger_adapter import (
     close_report_trigger,
@@ -56,6 +53,8 @@
     VulnerabilityReport,
 )
 
+FUTURE_DATETIME = datetime(2099, 12, 1, tzinfo=timezone.utc)
+
 # ---------------------------------------------------------------------------
 # Fixtures
 # ---------------------------------------------------------------------------

test/conftest.py

@@ -17,7 +17,6 @@
 This file provides test session hooks for cleanup of test artifacts.
 """
 
-import os
 from pathlib import Path
 
 import pytest

test/core/behaviors/case/conftest.py

@@ -9,6 +9,6 @@
 """
 
 # noqa: F401 — imported for vocabulary registration side-effect
-from vultron.wire.as2.vocab.objects.vulnerability_case import (
+from vultron.wire.as2.vocab.objects.vulnerability_case import (  # noqa: F401
     VulnerabilityCase,
-)  # noqa: F401
+)

test/core/behaviors/case/test_create_tree.py

@@ -131,13 +131,6 @@ def test_create_case_tree_creates_case_actor(
 ):
     tree = create_create_case_tree(case_obj=case_obj, actor_id=actor.as_id)
     bridge.execute_with_setup(tree=tree, actor_id=actor.as_id, activity=None)
-    all_objects = datalayer.get_all("Service")
-    case_actors = [
-        r
-        for r in all_objects
-        if isinstance(r, dict)
-        and r.get("data_", {}).get("context") == case_obj.as_id
-    ]
     # Verify at least one CaseActor exists for this case
     # (also checking via read fallback if stored under different table)
     found = False

test/core/behaviors/report/conftest.py

@@ -9,6 +9,6 @@
 """
 
 # noqa: F401 — imported for vocabulary registration side-effect
-from vultron.wire.as2.vocab.objects.vulnerability_case import (
+from vultron.wire.as2.vocab.objects.vulnerability_case import (  # noqa: F401
     VulnerabilityCase,
-)  # noqa: F401
+)

test/core/behaviors/report/test_policy.py

@@ -21,7 +21,6 @@
 """
 
 import logging
-from unittest.mock import MagicMock
 
 import pytest
 

test/core/behaviors/test_performance.py

@@ -24,23 +24,22 @@
 import time
 from unittest.mock import MagicMock
 
-logger = logging.getLogger(__name__)
-
 import pytest
 from py_trees.common import Status
 
-from vultron.core.ports.datalayer import DataLayer
+from vultron.core.behaviors.bridge import BTBridge
+from vultron.core.behaviors.report.validate_tree import (
+    create_validate_report_tree,
+)
 from vultron.core.models.vultron_types import (
     VultronAccept,
     VultronCase,
     VultronCaseActor,
     VultronOffer,
     VultronReport,
 )
-from vultron.core.behaviors.bridge import BTBridge
-from vultron.core.behaviors.report.validate_tree import (
-    create_validate_report_tree,
-)
+
+logger = logging.getLogger(__name__)
 
 
 @pytest.fixture

test/core/use_cases/test_actor_use_cases.py

@@ -15,7 +15,6 @@
 import logging
 from unittest.mock import MagicMock
 
-import pytest
 
 from vultron.wire.as2.vocab.objects.vulnerability_case import VulnerabilityCase
 from vultron.core.use_cases.actor import (