fix(ci): address lint violations and unit test updates for plan tree envelope (#9313)

features/steps/plan_explain_steps.py

@@ -402,7 +402,12 @@ def step_roots_no_children(context: Context) -> None:
 @then("the json tree output should be valid json")
 def step_tree_json_valid(context: Context) -> None:
     parsed = json.loads(context.pe_tree_json)
-    assert isinstance(parsed, list), "Expected a JSON array"
+    assert isinstance(parsed, dict), (
+        f"Expected JSON envelope (dict), got {type(parsed).__name__}"
+    )
+    _ENVELOPE_KEYS = {"command", "status", "exit_code", "data", "timing", "messages"}
+    missing = _ENVELOPE_KEYS - set(parsed.keys())
+    assert not missing, f"Envelope missing required fields: {missing}"
 
 
 @then('the json tree output should contain "{text}"')

src/cleveragents/cli/commands/plan.py

@@ -4154,9 +4154,7 @@ def tree_decisions_cmd(
 
     if fmt in (OutputFormat.JSON, OutputFormat.YAML):
         console.print(
-            format_output(
-                tree_data, fmt, command="plan tree", status="ok", exit_code=0
-            )
+            format_output(tree_data, fmt, command="plan tree", status="ok", exit_code=0)
         )
     elif fmt == OutputFormat.TABLE:
         # Flatten for table view