fix(sdk): prevent ORB scheduler config dict from being absorbed as string override

SDKConfig.from_dict maps known dataclass field names from the input dict.
The top-level "scheduler" key in an ORB config.json is a nested config
object ({"type": "hostfactory", "config_root": "..."}), not the plain
string that SDKConfig.scheduler expects (e.g. "hostfactory").

When from_file() parsed a real config.json the dict was stored in
SDKConfig.scheduler, the truthy dict triggered the override branch in
ORBClient.initialize(), and ConfigurationManager.override_scheduler_strategy
stored the dict. Downstream, _create_scheduler_strategy() retrieved the
dict as scheduler_type and passed it to SchedulerRegistry.create_strategy()
which attempted to use it as a dict key, raising "unhashable type: 'dict'",
surfacing as FactoryError on TemplateConfigurationPort in all 32
test_sdk_full_cycle_default and 7 test_cleanup_e2e live tests.

Fix: when from_dict encounters a "scheduler" value that is a dict, treat it
as an opaque ORB config object (move to custom_config) rather than setting
the string-override field. Plain string values ("default", "hostfactory")
continue to work unchanged.

Adds two regression tests to test_sdk_config.py.

Author: fgogolli

src/orb/sdk/config.py

@@ -70,7 +70,19 @@ def from_dict(cls, config: dict[str, Any]) -> "SDKConfig":
 
         for key, value in config.items():
             if key in known_fields:
-                sdk_config[key] = value
+                # ``scheduler`` is a plain string override in SDKConfig (e.g. "default"
+                # or "hostfactory").  When loading from an ORB config.json the top-level
+                # ``scheduler`` key is the full scheduler sub-config dict
+                # ({"type": "hostfactory", "config_root": "..."}).  Ingesting that dict
+                # as the string override causes ConfigurationManager.override_scheduler_strategy
+                # to store a dict, which then propagates as the scheduler_type into the
+                # registry lookup and fails with "unhashable type: 'dict'".
+                if key == "scheduler" and isinstance(value, dict):
+                    # The ORB config scheduler object is not an SDK string override;
+                    # skip it so the scheduler type is resolved from the config file.
+                    custom_config[key] = value
+                else:
+                    sdk_config[key] = value
             else:
                 custom_config[key] = value
 

tests/unit/sdk/test_sdk_config.py

@@ -102,6 +102,30 @@ def test_empty_dict_uses_defaults(self):
         config = SDKConfig.from_dict({})
         assert config.provider == "aws"
 
+    def test_scheduler_dict_not_absorbed_as_string_override(self):
+        # Regression: when loading from an ORB config.json the top-level "scheduler" key
+        # is a nested config dict {"type": "hostfactory", "config_root": "..."}.
+        # from_dict must NOT ingest that dict as the SDKConfig.scheduler string override
+        # because that propagates a dict into ConfigurationManager.override_scheduler_strategy
+        # which then surfaces as "unhashable type: 'dict'" deep in the DI factory chain.
+        orb_config = {
+            "provider": {"type": "aws"},
+            "scheduler": {"type": "hostfactory", "config_root": "$ORB_CONFIG_DIR"},
+        }
+        config = SDKConfig.from_dict(orb_config)
+        # scheduler field must remain None (no string override set)
+        assert config.scheduler is None
+        # the original dict is preserved in custom_config so nothing is silently dropped
+        assert config.custom_config.get("scheduler") == {
+            "type": "hostfactory",
+            "config_root": "$ORB_CONFIG_DIR",
+        }
+
+    def test_scheduler_string_override_is_still_accepted(self):
+        # Explicit string scheduler overrides (e.g. passed programmatically) must still work.
+        config = SDKConfig.from_dict({"provider": "aws", "scheduler": "default"})
+        assert config.scheduler == "default"
+
 
 class TestSDKConfigFromFile:
     def test_loads_json_file(self, tmp_path):