fix(attempt): omit reverse_translation_outputs from report when not used

Fixes #1201

`Attempt.as_dict()` unconditionally included a `reverse_translation_outputs`
key in the serialized dict, even when no translation was configured. This
polluted every `report.jsonl` entry with a spurious empty list.

The field is now only emitted when `self.reverse_translation_outputs` is
non-empty, keeping reports clean for the common non-translation case.

Signed-off-by: Varun Nuthalapati <nuthalapativarun@gmail.com>

Author: nuthalapativarun

garak/attempt.py

@@ -288,10 +288,16 @@ def as_dict(self) -> dict:
             "conversations": [
                 asdict(conversation) for conversation in self.conversations
             ],
-            "reverse_translation_outputs": [
-                asdict(output) if output else None
-                for output in self.reverse_translation_outputs
-            ],
+            **(
+                {
+                    "reverse_translation_outputs": [
+                        asdict(output) if output else None
+                        for output in self.reverse_translation_outputs
+                    ]
+                }
+                if self.reverse_translation_outputs
+                else {}
+            ),
         }
 
     @property

tests/test_attempt.py

@@ -557,7 +557,6 @@ def test_json_serialize():
                 "notes": {},
             },
         ],
-        "reverse_translation_outputs": [],
     }
 
     json_serialised = json.dumps(att_dict, ensure_ascii=False)