@@ -2,6 +2,16 @@
 
 ## Unreleased
 
+- Updated M3 acceptance-gate E2E verification to address review feedback.

@@ -32,3 +32,2 @@
 _SRC = str(Path(__file__).resolve().parents[1] / "src")
-if _SRC not in sys.path:
-    sys.path.insert(0, _SRC)
+sys.path.insert(0, _SRC)

@@ -104,3 +90,2 @@
 
-    Exercises the real CLI rendering/serialization path with a mocked
-    lifecycle service that returns *plan*.
+    Some CLI paths emit structured logs before the JSON payload. This

@@ -316,2 +288,3 @@
 def decision_tree_view() -> None:
-    """View the decision tree structure and verify rendering.
+    """Invoke ``plan tree`` CLI and validate rendered hierarchy."""
+    decision_service = DecisionService()

@@ -544,1 +496,3 @@
-        _CHILD_DEC_ID: [_GRANDCHILD_DEC_ID],
+    """Validate dry-run correction impact analysis and CLI wiring."""
+    service = CorrectionService()
+    tree = {

@@ -10,3 +10,3 @@
 *** Variables ***
 ${PYTHON}        python
-${TIMEOUT}       30s
+# Normal duration: ~5-10s per test. Timeout raised from 30s to 120s for

@@ -121,0 +102,4 @@
+    except json.JSONDecodeError:
+        pass
+
+    for index, char in enumerate(text):

@@ -176,1 +127,3 @@
-    )
+def _make_settings(database_url: str = "sqlite:///:memory:") -> Any:
+    """Create a minimal Settings test double for service wiring."""
+    settings = create_autospec(Settings, instance=True)

@@ -307,0 +256,4 @@
+    strategize_decisions = decision_service.list_decisions(plan_id)
+    if not strategize_decisions:
+        _fail("no decisions recorded during Strategize")
+    if str(strategize_decisions[0].decision_type) != "strategy_choice":

@@ -333,0 +305,4 @@
+    ):
+        result = cli_runner.invoke(
+            plan_app,
+            ["tree", _PLAN_ULID, "--format", "json"],

@@ -438,0 +386,4 @@
+    if data.get("chosen") != child.chosen_option:
+        _fail(f"chosen option mismatch: {data}")
+    if data.get("rationale") != child.rationale:
+        _fail(f"rationale mismatch: {data}")

@@ -513,0 +461,4 @@
+        source_name=_PROJECT_NAME,
+    )
+
+    list_service = MagicMock()

@@ -542,3 +511,1 @@
-    tree: dict[str, list[str]] = {
-        _ROOT_DEC_ID: [_CHILD_DEC_ID],
-        _CHILD_DEC_ID: [_GRANDCHILD_DEC_ID],
+    service = CorrectionService()

@@ -626,0 +585,4 @@
+        guidance="Use session cookies instead of JWT",
+        dry_run=True,
+    )
+    mock_service.analyze_impact.assert_called_once_with(mock_request.correction_id)

@@ -666,2 +622,2 @@
-    if _GRANDCHILD_DEC_ID not in result.reverted_decisions:
-        _fail("grandchild not reverted")
+        _fail(f"expected applied status from execute_revert, got: {result.status}")
+    if (

@@ -862,1 +798,4 @@
+    tree = {
+        root.decision_id: [child.decision_id],
+        child.decision_id: ["leaf"],
     }

@@ -33,0 +39,4 @@
+    ...
+    ...                Validates: plan tree displays the decision tree correctly.
+    [Tags]    success_criteria    decision_tree
+    ${result}=    Run Process    ${PYTHON}    ${HELPER}    decision-tree-view    cwd=${WORKSPACE}    timeout=60s