anycubic-cloud-api 0.4.34__tar.gz → 0.4.36__tar.gz

{anycubic_cloud_api-0.4.34/src/anycubic_cloud_api.egg-info → anycubic_cloud_api-0.4.36}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: anycubic-cloud-api
-Version: 0.4.34
+Version: 0.4.36
 Summary: Async client for the Anycubic Cloud API and its MQTT stream
 Author: WaresWichall, Nino Bondonno
 Maintainer: Nino Bondonno

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "anycubic-cloud-api"
-version = "0.4.34"
+version = "0.4.36"
 description = "Async client for the Anycubic Cloud API and its MQTT stream"
 readme = "README.md"
 requires-python = ">=3.12"

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/src/anycubic_cloud_api/api/mqtt.py

@@ -42,6 +42,7 @@ from ..helpers.helpers import (
     get_mqtt_ssl_path_key,
     get_part_from_mqtt_topic,
     get_ssl_cert_directory,
+    redact_for_log,
     redact_part_from_mqtt_topic,
 )
 from .functions import AnycubicAPIFunctions
@@ -151,7 +152,7 @@ class AnycubicMQTTAPI(AnycubicAPIFunctions):
         if self._mqtt_topic_is_user_topic(topic):
             self._log_to_debug(
                 f"Anycubic MQTT USER Msg Received on `{topic}`:\n"
-                f"    {payload}"
+                f"    {redact_for_log(payload)}"
             )
 
         else:
@@ -182,7 +183,7 @@ class AnycubicMQTTAPI(AnycubicAPIFunctions):
                     f"  with type: {e.unhandled_mqtt_type}, "
                     f"action: {e.unhandled_mqtt_action}, "
                     f"state: {e.unhandled_mqtt_state}\n"
-                    f"    unhandled data: {e.unhandled_mqtt_data}"
+                    f"    unhandled data: {redact_for_log(e.unhandled_mqtt_data)}"
                 )
 
             except AnycubicMQTTUnknownUpdate as e:
@@ -193,7 +194,7 @@ class AnycubicMQTTAPI(AnycubicAPIFunctions):
                 self._log_to_debug(
                     f"Anycubic MQTT Message not understood: {e}\n"
                     f"  on MQTT topic: {redacted_topic}\n"
-                    f"    {payload}"
+                    f"    {redact_for_log(payload)}"
                 )
 
             except Exception as e:
@@ -203,7 +204,7 @@ class AnycubicMQTTAPI(AnycubicAPIFunctions):
                 self._log_to_error(
                     f"Anycubic MQTT Message error: {error_type}: {e}\n"
                     f"  on MQTT topic: {redacted_topic}\n"
-                    f"    {payload}\n"
+                    f"    {redact_for_log(payload)}\n"
                     f"{tb}"
                 )
 
@@ -230,7 +231,7 @@ class AnycubicMQTTAPI(AnycubicAPIFunctions):
                 self._log_to_debug(
                     f"Anycubic MQTT Message processed on topic: "
                     f"{redact_part_from_mqtt_topic(topic, 6)}\n"
-                    f"    {payload}"
+                    f"    {redact_for_log(payload)}"
                 )
 
     def _mqtt_publish_on_topic(

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/src/anycubic_cloud_api/data_models/printer.py

@@ -64,6 +64,20 @@ _PRINT_PROGRESS_KEYS = (
     'supplies_usage',
 )
 
+# The steps of a filament change, as a Kobra X with an ACE 2 Pro reports
+# them (#33). The ACE unfeed is the mirror of the feed, not yet seen.
+_FILAMENT_CHANGE_STATES = frozenset({
+    'extfilbox_filament_feeding',
+    'extfilbox_filament_unfeeding',
+    'ace_filament_feeding',
+    'ace_filament_unfeeding',
+    'nozzle_purging',
+})
+
+# Printers with a feeder of their own, which shares the box list with any
+# ACE fitted: the Kobra X.
+_PRINTERS_WITH_OWN_FEEDER = frozenset({20030})
+
 # How a printer words a refused order when it gives no code.
 _REFUSED_STATES = frozenset({'failed', 'fail', 'error', 'refused'})
 
@@ -438,6 +452,12 @@ class AnycubicPrinter:
                         updated.get(box.box_id, box) for box in known_boxes
                     ])
 
+                if known_boxes:
+                    previous = {box.box_id: box for box in known_boxes}
+                    for box in self._multi_color_box:
+                        if (known := previous.get(box.box_id)) is not None and known is not box:
+                            box.keep_printer_slot_from(known)
+
         except Exception as e:
             self._initialisation_error = True
             # Swallowing this leaves the ACE list empty, which presents as
@@ -1317,13 +1337,23 @@ class AnycubicPrinter:
             data.get('settings')
 
             if 'channel_no' in data:
-                self._apply_active_channel(data['channel_no'])
+                self._apply_active_channel(data['channel_no'], data.get('unit_no'))
 
             # A Kobra X reports progress this way between info reports, with
             # no task id. It was logged as unhandled and dropped (#34).
             if any(key in data for key in _PRINT_PROGRESS_KEYS):
                 self._apply_print_progress(project_id, data)
             return
+        elif action == 'start' and state in _FILAMENT_CHANGE_STATES:
+            # Each step of a filament change, naming the channel (and, for an
+            # ACE, the unit). Only a feed says what is now loaded (#33).
+            data = payload.get('data') or {}
+            # Read on every step, so the ones that change nothing are not
+            # reported as unhandled.
+            channel_no, unit_no = data.get('channel_no'), data.get('unit_no')
+            if state.endswith('_filament_feeding'):
+                self._apply_active_channel(channel_no, unit_no)
+            return
         elif action in ['start', 'stop'] and state in ['failed']:
             err_msg = payload.get('msg')
             self._is_printing = 1
@@ -1336,31 +1366,37 @@ class AnycubicPrinter:
         else:
             raise AnycubicMQTTUnknownUpdate(ErrorsMQTTUpdate.job_status)
 
-    def _apply_active_channel(self, channel_no: Any) -> None:
+    def _apply_active_channel(self, channel_no: Any, unit_no: Any = None) -> None:
         """Follow the filament channel the printer says it is printing from.
 
         A Kobra X reports each filament change as a print update carrying
-        only ``channel_no``, and stops sending the ACE's own ``loaded_slot``
-        after the first few changes of a print -- which is why the loaded
-        slot froze for the rest of the job (#34). The channel is numbered
-        from 1 across every box in order, so channel 5 on a printer whose
-        first box has four slots is the first slot of the second box.
+        ``channel_no``, and stops sending the box's own ``loaded_slot`` after
+        the first few changes of a print, which is why the loaded slot froze
+        (#34). Each box numbers its channels from 1; an ACE adds ``unit_no``
+        and the printer's own feeder sends none (#33, Kobra X + ACE 2 Pro).
+
+        With one box the channel can only mean that box. With several, an
+        ACE's ``unit_no`` picks it on a printer whose boxes are all ACEs.
+        On a Kobra X with an ACE, which list entry is the printer's own
+        feeder is not yet confirmed, so nothing is guessed there: a wrong
+        slot is worse than a stale one.
         """
         channel = as_int(channel_no, 0)
+        boxes = self._multi_color_box
 
-        if channel < 1 or not self._multi_color_box:
+        if channel < 1 or not boxes:
             return
 
-        index = channel - 1
-
-        for box in self._multi_color_box:
-            slot_count = len(box.slots)
-
-            if index < slot_count:
-                box.set_slot_loaded(index)
+        if len(boxes) == 1:
+            box = boxes[0]
+        else:
+            unit = as_int(unit_no, 0)
+            if unit < 1 or unit > len(boxes) or self._machine_type in _PRINTERS_WITH_OWN_FEEDER:
                 return
+            box = boxes[unit - 1]
 
-            index -= slot_count
+        if channel <= len(box.slots):
+            box.set_slot_loaded(channel - 1)
 
     def _apply_print_progress(
         self,
@@ -1466,8 +1502,12 @@ class AnycubicPrinter:
                     continue
 
                 assert self._multi_color_box
-                self._multi_color_box[box_id].set_current_temperature(box['temp'])
-                self._multi_color_box[box_id].set_drying_status(box['drying_status'])
+                # An ACE 2 Pro's periodic drying report carries no `temp`, and
+                # reading it raised every six minutes, so drying never updated
+                # (#33). Keep the last reading rather than inventing one.
+                if 'temp' in box:
+                    self._multi_color_box[box_id].set_current_temperature(box['temp'])
+                self._multi_color_box[box_id].set_drying_status(box.get('drying_status'))
             return
         elif action == 'setDry' and self._report_refused_drying(state, payload):
             return
@@ -1508,9 +1548,32 @@ class AnycubicPrinter:
             if self.external_shelves:
                 self.external_shelves.update_with_mqtt_data(data)
             return
+        elif action == 'auto' and state == 'done':
+            # Sent as a filament change completes, naming the feeding slot as
+            # "<box>-<slot>", both counted from 0 ("0-3" while the screen shows
+            # slot 4 on a Kobra X, #34); empty while the filament is retracted.
+            data = payload.get('data') or {}
+            self._apply_current_filament(data.get('current_filament'))
+            return
         else:
             raise AnycubicMQTTUnknownUpdate(ErrorsMQTTUpdate.shelves)
 
+    def _apply_current_filament(self, current_filament: Any) -> None:
+        """Follow the printer's own "<box>-<slot>" reading, where the box is certain.
+
+        Only with a single box: which list entry the printer's box number
+        means beside an ACE is not confirmed (#33). An empty reading is a
+        retract for calibration or wiping, not a change of slot, so the last
+        slot is kept.
+        """
+        boxes = self._multi_color_box
+        if not isinstance(current_filament, str) or not boxes or len(boxes) != 1:
+            return
+        box_no, _, slot_no = current_filament.partition('-')
+        box_index, slot_index = as_int(box_no, -1), as_int(slot_no, -1)
+        if box_index == 0 and 0 <= slot_index < len(boxes[0].slots):
+            boxes[0].set_slot_loaded(slot_index)
+
     def _process_mqtt_update_file(
         self,
         action: str,

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/src/anycubic_cloud_api/data_models/printer_properties.py

@@ -702,6 +702,7 @@ class AnycubicMultiColorBox:
         "_model_id",
         "_auto_feed",
         "_loaded_slot",
+        "_slot_from_printer",
         "_feed_status",
         "_temp",
         "_humidity",
@@ -734,6 +735,9 @@ class AnycubicMultiColorBox:
         self._model_id: int = as_int(model_id)
         self._auto_feed: int = as_int(auto_feed)
         self._loaded_slot: int = as_int(loaded_slot, -1)
+        # Whether the printer itself has reported the loaded slot, as opposed
+        # to a cloud record that only knew it at some earlier point.
+        self._slot_from_printer = False
         self.set_feed_status(feed_status)
         self.set_current_temperature(temp)
         self._humidity: float | None = (
@@ -759,7 +763,21 @@ class AnycubicMultiColorBox:
         self._feed_status = AnycubicFeedStatus.from_json(feed_status)
 
     def set_slot_loaded(self, slot_num: int) -> None:
+        """A loaded slot reported by the printer (MQTT or LAN)."""
         self._loaded_slot = as_int(slot_num, -1)
+        self._slot_from_printer = True
+
+    def keep_printer_slot_from(self, previous: AnycubicMultiColorBox) -> None:
+        """Carry over a slot the printer reported, past a cloud refresh.
+
+        The cloud's record of the loaded slot goes stale during a print: a
+        Kobra X kept reporting slot 4 while each cloud poll said slot 2, and
+        the sensor flipped to 2 mid-print (#34). What the printer said last
+        wins until it says something else.
+        """
+        if previous._slot_from_printer:
+            self._loaded_slot = previous._loaded_slot
+            self._slot_from_printer = True
 
     def set_current_temperature(self, temp: int) -> None:
         # Reached from the constructor as well as from MQTT, so a null

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/src/anycubic_cloud_api/helpers/helpers.py

@@ -164,6 +164,34 @@ def get_part_from_mqtt_topic(topic: str, part: int) -> str | None:
     return split_topic[part]
 
 
+# Identifiers a printer puts in its own reports. A debug log gets pasted into
+# public issues, and only the topic's printer key was being redacted: a
+# Kobra X's PrintStart report carried its cn_code in the clear (#34).
+_LOG_REDACT_KEYS = frozenset({
+    'cn_code', 'cn', 'sn', 'serial', 'machine_sn', 'mac', 'machine_mac',
+    'ip', 'key', 'printer_key', 'device_id', 'deviceId', 'user_id', 'email',
+})
+_LOG_REDACT_PARTS = ('token', 'secret', 'password')
+
+
+def redact_for_log(data: Any) -> Any:
+    """A copy of an MQTT payload that is safe to put in a log."""
+    if isinstance(data, dict):
+        return {
+            key: (
+                "**REDACTED**"
+                if isinstance(key, str)
+                and data[key] not in (None, '')
+                and (key in _LOG_REDACT_KEYS or any(part in key.lower() for part in _LOG_REDACT_PARTS))
+                else redact_for_log(data[key])
+            )
+            for key in data
+        }
+    if isinstance(data, list):
+        return [redact_for_log(item) for item in data]
+    return data
+
+
 def redact_part_from_mqtt_topic(topic: str, part: int) -> str:
     split_topic = topic.split("/")
     new_chunk = list()

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36/src/anycubic_cloud_api.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: anycubic-cloud-api
-Version: 0.4.34
+Version: 0.4.36
 Summary: Async client for the Anycubic Cloud API and its MQTT stream
 Author: WaresWichall, Nino Bondonno
 Maintainer: Nino Bondonno

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/src/anycubic_cloud_api.egg-info/SOURCES.txt

@@ -47,6 +47,7 @@ tests/test_auth_mode_fallback.py
 tests/test_axis_position.py
 tests/test_endpoint_urls.py
 tests/test_exchange_rate_limit.py
+tests/test_filament_change_0435.py
 tests/test_lan_client.py
 tests/test_lan_commands.py
 tests/test_lan_handshake.py
@@ -55,6 +56,7 @@ tests/test_null_tolerant_parsing.py
 tests/test_partial_reports.py
 tests/test_print_reports_0434.py
 tests/test_printer_error_codes.py
+tests/test_printer_slot_0436.py
 tests/test_project_null_status.py
 tests/test_regions.py
 tests/test_secondary_ace.py

anycubic_cloud_api-0.4.36/tests/test_filament_change_0435.py

@@ -0,0 +1,133 @@
+"""Filament-change reports and the ACE 2 Pro's drying report (0.4.35).
+
+From a Kobra X with an ACE 2 Pro (#33). Each step of a filament change
+arrives as a print ``start`` naming ``channel_no``, counted from 1 within
+its box, plus ``unit_no`` when the feed comes from an ACE. These were
+dropped as unhandled. The ACE 2 Pro's periodic drying report has no
+``temp``, and reading it raised every six minutes.
+"""
+
+from unittest.mock import MagicMock
+
+import pytest
+
+from anycubic_cloud_api.data_models.consumable import AnycubicConsumableData
+from anycubic_cloud_api.data_models.printer import AnycubicPrinter
+
+KOBRA_X = 20030
+KOBRA_S1 = 20025
+
+
+def slot(index):
+    return {"index": index, "sku": "", "type": "PLA", "color": [1, 2, 3], "status": 5, "edit_status": 1}
+
+
+def box(box_id, slots=4):
+    return {"id": box_id, "loaded_slot": -1, "temp": 21, "slots": [slot(i) for i in range(slots)]}
+
+
+def make_printer(boxes, machine_type=KOBRA_X):
+    return AnycubicPrinter(
+        api_parent=MagicMock(),
+        machine_type=machine_type,
+        machine_name="Printer",
+        id=1,
+        multi_color_box=boxes,
+        ignore_init_errors=True,
+    )
+
+
+def message(msg_type, action, state, data):
+    return AnycubicConsumableData({
+        "type": msg_type, "action": action, "state": state,
+        "code": 200, "msg": "done", "msgid": "m", "timestamp": 1, "data": data,
+    })
+
+
+def topic(machine_type, kind):
+    return f"anycubic/anycubicCloud/v1/printer/public/{machine_type}/key/{kind}/report"
+
+
+def feed(printer, state, channel_no, unit_no=None, machine_type=KOBRA_X):
+    data = {"channel_no": channel_no, "taskid": "t"}
+    if unit_no is not None:
+        data["unit_no"] = unit_no
+    printer.process_mqtt_update(topic(machine_type, "print"), message("print", "start", state, data))
+
+
+def loaded(printer):
+    return [b.loaded_slot for b in printer.multi_color_box]
+
+
+class TestFilamentChange:
+    @pytest.mark.parametrize(("state", "unit_no"), [
+        ("extfilbox_filament_feeding", None),
+        ("ace_filament_feeding", 1),
+    ])
+    def test_one_box_follows_the_feed(self, state, unit_no):
+        printer = make_printer([box(0)])
+
+        feed(printer, state, 3, unit_no)
+
+        assert loaded(printer) == [2]
+
+    @pytest.mark.parametrize("state", [
+        "extfilbox_filament_unfeeding", "ace_filament_unfeeding", "nozzle_purging",
+    ])
+    def test_other_steps_are_read_but_change_nothing(self, state):
+        """Consumed, so not logged as unhandled; only a feed says what is loaded."""
+        printer = make_printer([box(0)])
+        printer.multi_color_box[0].set_slot_loaded(1)
+
+        feed(printer, state, 3, 1)
+
+        assert loaded(printer) == [1]
+
+    @pytest.mark.parametrize(("unit_no", "expected"), [(1, [1, -1]), (2, [-1, 1])])
+    def test_an_ace_unit_picks_its_box_on_a_printer_of_aces(self, unit_no, expected):
+        """A Kobra S1 has no feeder of its own: unit 1 is the first ACE."""
+        printer = make_printer([box(0), box(1)], machine_type=KOBRA_S1)
+
+        feed(printer, "ace_filament_feeding", 2, unit_no, machine_type=KOBRA_S1)
+
+        assert loaded(printer) == expected
+
+    @pytest.mark.parametrize(("state", "unit_no"), [
+        ("extfilbox_filament_feeding", None),
+        ("ace_filament_feeding", 1),
+    ])
+    def test_a_kobra_x_with_an_ace_guesses_nothing(self, state, unit_no):
+        """Which list entry is the Kobra X's own feeder is unconfirmed (#33)."""
+        printer = make_printer([box(0), box(1)])
+
+        feed(printer, state, 1, unit_no)
+
+        assert loaded(printer) == [-1, -1]
+
+    @pytest.mark.parametrize(("channel_no", "unit_no"), [(5, 1), (0, 1), (2, 3), ("x", 1), (2, None)])
+    def test_a_channel_or_unit_that_fits_no_box_changes_nothing(self, channel_no, unit_no):
+        printer = make_printer([box(0), box(1)], machine_type=KOBRA_S1)
+
+        feed(printer, "ace_filament_feeding", channel_no, unit_no, machine_type=KOBRA_S1)
+
+        assert loaded(printer) == [-1, -1]
+
+
+class TestAce2ProDryingReport:
+    REPORT = {"multi_color_box": [{
+        "drying_status": {"duration": 14400, "humidity": 28, "remain_time": 162, "status": 1, "target_temp": 50},
+        "id": 0,
+    }]}
+
+    def test_a_report_without_temp_still_updates_drying(self):
+        printer = make_printer([box(0)])
+
+        printer.process_mqtt_update(
+            topic(KOBRA_X, "multiColorBox"),
+            message("multiColorBox", "autoUpdateDryStatus", "success", self.REPORT),
+        )
+
+        ace = printer.multi_color_box[0]
+        assert ace.drying_status.is_drying
+        assert ace.drying_status.target_temperature == 50
+        assert ace.current_temperature == 21  # the last reading is kept

{anycubic_cloud_api-0.4.34 → anycubic_cloud_api-0.4.36}/tests/test_print_reports_0434.py

@@ -90,13 +90,18 @@ class TestActiveChannel:
 
         assert printer.primary_multi_color_box_loaded_slot == slot_index
 
-    def test_channels_beyond_the_first_box_land_in_the_next(self):
-        """Channel 5 with four slots in the first box is the second box's first."""
+    @pytest.mark.parametrize("channel_no", [1, 5])
+    def test_with_two_boxes_and_no_unit_nothing_is_guessed(self, channel_no):
+        """Channels are numbered per box (#33), so without a unit the box is unknown.
+
+        0.4.34 counted across boxes and put channel 5 in the second box.
+        """
         printer = make_printer([box(0), box(1)])
+        before = [b.loaded_slot for b in printer.multi_color_box]
 
-        self._channel(printer, 5)
+        self._channel(printer, channel_no)
 
-        assert printer.secondary_multi_color_box_loaded_slot == 0
+        assert [b.loaded_slot for b in printer.multi_color_box] == before
 
     @pytest.mark.parametrize("channel_no", [0, -1, 9, None, "x"])
     def test_a_channel_with_no_slot_changes_nothing(self, channel_no):

anycubic_cloud_api-0.4.36/tests/test_printer_slot_0436.py

@@ -0,0 +1,149 @@
+"""The printer's own loaded slot, and logs safe to paste (0.4.36).
+
+From a Kobra X printing from slot 4 (#34): the sensor flipped to 2 mid-print
+with no message from the printer. Each cloud poll rebuilt the box from the
+cloud's record, whose loaded slot was stale. The printer also names the
+feeding slot itself in its extfilbox report, and its PrintStart report put
+the printer's cn_code into the debug log.
+"""
+
+import pytest
+from test_filament_change_0435 import (
+    KOBRA_S1,
+    KOBRA_X,
+    box,
+    make_printer,
+    message,
+    topic,
+)
+
+from anycubic_cloud_api.helpers.helpers import redact_for_log
+
+
+def cloud_info(boxes, machine_type=KOBRA_X):
+    return {
+        "id": 1, "name": "Printer", "key": "k", "machine_type": machine_type, "model": "Printer",
+        "img": "", "device_status": 1, "is_printing": 2, "type_function_ids": [], "version": None,
+        "multi_color_box": boxes,
+    }
+
+
+def stale_box(box_id, loaded_slot):
+    b = box(box_id)
+    b["loaded_slot"] = loaded_slot
+    return b
+
+
+def loaded(printer):
+    return [b.loaded_slot for b in printer.multi_color_box]
+
+
+class TestCloudPollKeepsThePrintersSlot:
+    def test_a_slot_the_printer_reported_survives_a_cloud_poll(self):
+        printer = make_printer([box(0)])
+        printer.multi_color_box[0].set_slot_loaded(3)
+
+        printer.update_from_info_json(cloud_info([stale_box(0, 1)]))
+
+        assert loaded(printer) == [3]
+
+    def test_without_a_printer_report_the_cloud_value_is_used(self):
+        printer = make_printer([box(0)])
+
+        printer.update_from_info_json(cloud_info([stale_box(0, 1)]))
+
+        assert loaded(printer) == [1]
+
+    def test_the_next_printer_report_still_wins(self):
+        printer = make_printer([box(0)])
+        printer.multi_color_box[0].set_slot_loaded(3)
+        printer.update_from_info_json(cloud_info([stale_box(0, 1)]))
+
+        printer.multi_color_box[0].set_slot_loaded(0)
+
+        assert loaded(printer) == [0]
+
+    def test_each_box_keeps_its_own(self):
+        printer = make_printer([box(0), box(1)], machine_type=KOBRA_S1)
+        printer.multi_color_box[1].set_slot_loaded(2)
+
+        printer.update_from_info_json(cloud_info([stale_box(0, 1), stale_box(1, 0)], machine_type=KOBRA_S1))
+
+        assert loaded(printer) == [1, 2]
+
+    def test_other_cloud_fields_still_update(self):
+        printer = make_printer([box(0)])
+        printer.multi_color_box[0].set_slot_loaded(3)
+        refreshed = stale_box(0, 1)
+        refreshed["temp"] = 33
+
+        printer.update_from_info_json(cloud_info([refreshed]))
+
+        assert printer.multi_color_box[0].current_temperature == 33
+
+
+def extfilbox(printer, current_filament, machine_type=KOBRA_X):
+    printer.process_mqtt_update(
+        topic(machine_type, "extfilbox"),
+        message("extfilbox", "auto", "done", {"current_filament": current_filament}),
+    )
+
+
+class TestCurrentFilament:
+    @pytest.mark.parametrize(("reading", "slot"), [("0-3", 3), ("0-0", 0)])
+    def test_one_box_follows_the_printer(self, reading, slot):
+        printer = make_printer([box(0)])
+
+        extfilbox(printer, reading)
+
+        assert loaded(printer) == [slot]
+
+    @pytest.mark.parametrize("reading", ["", None, "0-9", "1-2", "x", "0-", "-1"])
+    def test_a_retract_or_unknown_reading_keeps_the_slot(self, reading):
+        printer = make_printer([box(0)])
+        printer.multi_color_box[0].set_slot_loaded(3)
+
+        extfilbox(printer, reading)
+
+        assert loaded(printer) == [3]
+
+    def test_with_two_boxes_nothing_is_guessed(self):
+        printer = make_printer([box(0), box(1)])
+
+        extfilbox(printer, "0-3")
+
+        assert loaded(printer) == [-1, -1]
+
+    def test_it_survives_the_next_cloud_poll(self):
+        printer = make_printer([box(0)])
+        extfilbox(printer, "0-3")
+
+        printer.update_from_info_json(cloud_info([stale_box(0, 1)]))
+
+        assert loaded(printer) == [3]
+
+
+class TestRedactForLog:
+    def test_printer_identifiers_are_redacted_wherever_they_are(self):
+        payload = {"type": "buried", "data": {"cn_code": "CN123", "nested": [{"machine_mac": "AA:BB"}], "ip": "10.0.0.2"}}
+
+        assert redact_for_log(payload) == {
+            "type": "buried",
+            "data": {"cn_code": "**REDACTED**", "nested": [{"machine_mac": "**REDACTED**"}], "ip": "**REDACTED**"},
+        }
+
+    @pytest.mark.parametrize("key", ["token", "casdoor_user_token", "rtc_token", "app_secret", "password"])
+    def test_credentials_are_redacted(self, key):
+        assert redact_for_log({key: "secret-value"}) == {key: "**REDACTED**"}
+
+    def test_ordinary_fields_and_empty_values_pass_through(self):
+        payload = {"progress": 42, "taskid": "131448158", "cn_code": "", "state": "printing", "slots": [1, 2]}
+
+        assert redact_for_log(payload) == payload
+
+    def test_the_input_is_not_changed(self):
+        payload = {"data": {"cn_code": "CN123"}}
+
+        redact_for_log(payload)
+
+        assert payload == {"data": {"cn_code": "CN123"}}