level: preserve args through as_dict round-trip when only one slot is populated

Level.as_dict() used all(self.args.values()), which dropped the entire
args entry whenever either 'keyword' or 'block' was empty -- the common
case for users who pass only keyword args. The scheduler reconstructs
the level via Level(repr=level) before handing it to the job adapter,
and re-feeds job.level into run_job during trsh, so the loss surfaced
as user-supplied keywords (e.g. opt=(verytight)) silently disappearing
on every trsh re-run while the initial run worked correctly.

Switch to any() so args is preserved when at least one slot has content;
still dropped when both are empty.

Author: calvinp0

arc/level.py

@@ -187,7 +187,7 @@ def as_dict(self) -> dict:
         original_dict = self.__dict__
         clean_dict = {}
         for key, val in original_dict.items():
-            if val is not None and key != 'args' or key == 'args' and all([v for v in self.args.values()]):
+            if (val is not None and key != 'args') or (key == 'args' and any(v for v in self.args.values())):
                 clean_dict[key] = val
         return clean_dict
 

arc/level_test.py

@@ -119,6 +119,26 @@ def test_build(self):
                          "dlpno-ccsd(t)/def2-tzvp, auxiliary_basis: def2-tzvp/c, solvation_method: smd, "
                          "solvent: water, solvation_scheme_level: 'apfd/def2-tzvp, software: gaussian', software: orca")
 
+    def test_as_dict_preserves_args_when_one_slot_empty(self):
+        """args must round-trip through as_dict() even when only 'keyword' (or only 'block') is populated.
+
+        Regression: as_dict() previously used ``all(self.args.values())`` which dropped args
+        whenever either slot was empty (the common case). Trsh re-runs lost user-supplied
+        keyword args because the scheduler reconstructs the level via ``Level(repr=level)``.
+        """
+        keyword_only = Level(repr={'method': 'wb97xd', 'basis': 'def2tzvp',
+                                   'args': {'keyword': {'opt': 'opt=(verytight)'}}})
+        self.assertIn('args', keyword_only.as_dict())
+        self.assertEqual(Level(repr=keyword_only).args, keyword_only.args)
+
+        block_only = Level(repr={'method': 'wb97xd', 'basis': 'def2tzvp',
+                                 'args': {'block': {'1': 'extra block'}}})
+        self.assertIn('args', block_only.as_dict())
+        self.assertEqual(Level(repr=block_only).args, block_only.args)
+
+        empty_args = Level(method='wb97xd', basis='def2tzvp')
+        self.assertNotIn('args', empty_args.as_dict())
+
     def test_year_validation(self):
         """Test year validation for Level"""
         with self.assertRaises(ValueError):