From 3db5aa001a27b48ce8011197ccec3f09131ae0e5 Mon Sep 17 00:00:00 2001 From: Jedrzej Kosinski Date: Mon, 20 Apr 2026 15:41:57 -0700 Subject: [PATCH] fix: remove dead code and redundant top-level keys from MultiCombo serialization Amp-Thread-ID: https://ampcode.com/threads/T-019dad04-a07a-724b-af4d-fbfe98654fdd Co-authored-by: Amp --- comfy_api/latest/_io.py | 4 +--- tests-unit/comfy_api_test/multicombo_serialization_test.py | 2 -- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/comfy_api/latest/_io.py b/comfy_api/latest/_io.py index 823874bde..4fd43008d 100644 --- a/comfy_api/latest/_io.py +++ b/comfy_api/latest/_io.py @@ -413,9 +413,7 @@ class MultiCombo(ComfyTypeI): "multi_select": prune_dict({ "placeholder": self.placeholder, "chip": self.chip, - }) if self.multiselect else None, - "placeholder": self.placeholder, - "chip": self.chip, + }), }) @comfytype(io_type="IMAGE") diff --git a/tests-unit/comfy_api_test/multicombo_serialization_test.py b/tests-unit/comfy_api_test/multicombo_serialization_test.py index 9e4fecdd0..df925ad19 100644 --- a/tests-unit/comfy_api_test/multicombo_serialization_test.py +++ b/tests-unit/comfy_api_test/multicombo_serialization_test.py @@ -31,5 +31,3 @@ def test_multicombo_serializes_multi_select_with_placeholder_and_chip(): "placeholder": "Select providers", "chip": True, } - assert serialized["placeholder"] == "Select providers" - assert serialized["chip"] is True