flashforge-python-api 1.3.2__py3-none-any.whl → 1.3.4__py3-none-any.whl

flashforge/exceptions.py

@@ -0,0 +1,51 @@
+"""
+FlashForge Python API - Exceptions
+"""
+
+from __future__ import annotations
+
+
+class FlashForgeError(Exception):
+    """Base class for every error this library raises deliberately."""
+
+
+class FlashForgeResponseError(FlashForgeError):
+    """
+    The printer answered, but the payload could not be understood.
+
+    This is deliberately distinct from a ``None`` return, which every HTTP
+    control method uses for "could not reach the printer / the printer refused
+    us". Collapsing the two is what made issue #18 take three releases to
+    diagnose: a Creator 5 reporting ``chamberTemp: -108`` (the firmware's "no
+    chamber sensor" sentinel) surfaced in Home Assistant as ``cannot_connect``,
+    pointing the user at their network for a schema problem.
+
+    Raise this whenever the transport succeeded and the *content* was the
+    problem, so callers can tell the user something actionable.
+    """
+
+    def __init__(
+        self,
+        message: str,
+        *,
+        endpoint: str | None = None,
+        cause: Exception | None = None,
+    ) -> None:
+        """
+        Args:
+            message: Human-readable description of what could not be parsed.
+            endpoint: The endpoint whose response failed, e.g. "/detail".
+            cause: The underlying validation/decode error, kept for logs.
+        """
+        super().__init__(message)
+        self.message = message
+        self.endpoint = endpoint
+        self.cause = cause
+
+    def __str__(self) -> str:
+        parts = [self.message]
+        if self.endpoint:
+            parts.append(f"(endpoint: {self.endpoint})")
+        if self.cause:
+            parts.append(f"caused by {type(self.cause).__name__}: {self.cause}")
+        return " ".join(parts)

flashforge/models/machine_info.py

@@ -4,11 +4,76 @@ FlashForge Python API - Data Models
 
 from __future__ import annotations
 
+import re
 from datetime import datetime
 from enum import Enum
+from typing import Any
 
 from pydantic import BaseModel, ConfigDict, Field, field_validator, model_validator
 
+# ---------------------------------------------------------------------------
+# Validation policy for inbound firmware payloads
+# ---------------------------------------------------------------------------
+#
+# Models parsed FROM the printer carry types but NOT value ranges, and no
+# required fields beyond what we genuinely cannot work without. The reason is
+# blast radius: pydantic fails a model as a unit, and `Info.get_detail_response`
+# turns any failure into "no data", which Home Assistant shows as an offline
+# printer. So a `ge=0` on a field nobody reads could take the whole integration
+# down. That is exactly what happened in issue #18, where a Creator 5 reporting
+# `chamberTemp: -108` made every entity unavailable and the config flow report
+# `cannot_connect`.
+#
+# Range constraints belong on OUTBOUND command models (see responses.py), where
+# the values are ours and a bad one is our bug. Inbound, we take what we need
+# and tolerate the rest. This mirrors the TypeScript client (ff-5mp-api-ts),
+# which has never hit this class of failure.
+#
+# Firmware signals "this sensor does not exist" with an out-of-band negative
+# sentinel rather than by omitting the field: -108 on a chamber-less Creator 5,
+# and -100 is our own "heater off" value (TempControl.TEMP_OFF). Anything at or
+# below this floor is a marker, not a reading.
+TEMP_SENTINEL_FLOOR = -50.0
+
+_HEX_COLOR_RE = re.compile(r"^#[0-9A-Fa-f]{6}$")
+
+
+def sanitize_temperature(value: Any) -> Any:
+    """
+    Map firmware temperature sentinels to None, passing everything else through.
+
+    Non-numeric input is returned untouched so normal type validation still
+    reports it. Used as a `mode="before"` validator on every inbound
+    temperature field.
+    """
+    if isinstance(value, bool) or not isinstance(value, (int, float)):
+        return value
+    return None if value <= TEMP_SENTINEL_FLOOR else value
+
+
+def _normalize_hex_color(value: Any) -> Any:
+    """
+    Coerce a material color into `#RRGGBB`, or an empty string when it is not
+    recognizable.
+
+    Never raises. An unparseable color costs a swatch its color; it must not
+    cost the caller the entire /detail response.
+    """
+    if not isinstance(value, str):
+        return "" if value is None else value
+
+    candidate = value.strip()
+    if candidate == "" or _HEX_COLOR_RE.match(candidate):
+        return candidate
+
+    # Accept the shorthand `#RGB` form by expanding it, and a bare `RRGGBB`
+    # without the leading hash - both are plausible firmware variations.
+    if re.match(r"^#[0-9A-Fa-f]{3}$", candidate):
+        return "#" + "".join(char * 2 for char in candidate[1:])
+    if re.match(r"^[0-9A-Fa-f]{6}$", candidate):
+        return f"#{candidate}"
+    return ""
+
 
 class MachineState(Enum):
     """Enumerates the possible operational states of the FlashForge 3D printer."""
@@ -35,12 +100,11 @@ class Temperature(BaseModel):
     # `extra="allow"` would silently absorb.
     model_config = ConfigDict(extra="forbid")
 
-    current: float = Field(
-        default=0.0, ge=-50, le=500, description="The current temperature in Celsius"
-    )
-    set: float = Field(
-        default=0.0, ge=-50, le=500, description="The target (set) temperature in Celsius"
-    )
+    # No range constraints: MachineInfoParser feeds these straight from firmware
+    # values, so a range here fails `from_detail` for the same reason a range on
+    # FFPrinterDetail fails the response - one odd reading, no data at all.
+    current: float = Field(default=0.0, description="The current temperature in Celsius")
+    set: float = Field(default=0.0, description="The target (set) temperature in Celsius")
 
 
 class SlotInfo(BaseModel):
@@ -53,26 +117,32 @@ class SlotInfo(BaseModel):
     # integration reports as "could not retrieve printer information".
     model_config = ConfigDict(extra="allow", populate_by_name=True)
 
+    # Every field defaulted: a slot missing one attribute must cost that
+    # attribute, not the whole /detail response. `slot_id` carries no 1-4 range
+    # for the same reason - a station reporting a fifth slot should show up as a
+    # slot we ignore, not as an offline printer.
     has_filament: bool = Field(
-        alias="hasFilament", description="Indicates if filament is present in this slot"
+        default=False,
+        alias="hasFilament",
+        description="Indicates if filament is present in this slot",
     )
     material_color: str = Field(
-        alias="materialColor", description="Color of the material in this slot (e.g., '#FFFFFF')"
+        default="",
+        alias="materialColor",
+        description="Color of the material in this slot (e.g., '#FFFFFF')",
     )
     material_name: str = Field(
-        alias="materialName", description="Name of the material in this slot (e.g., 'PLA')"
+        default="",
+        alias="materialName",
+        description="Name of the material in this slot (e.g., 'PLA')",
     )
-    slot_id: int = Field(alias="slotId", ge=1, le=4, description="Identifier for this slot (1-4)")
+    slot_id: int = Field(default=0, alias="slotId", description="Identifier for this slot (1-4)")
 
-    @field_validator("material_color")
+    @field_validator("material_color", mode="before")
     @classmethod
-    def validate_hex_color(cls, v: str) -> str:
-        """Validate that the color is a valid hex color code or empty string."""
-        import re
-
-        if v == "" or re.match(r"^#[0-9A-Fa-f]{6}$", v):
-            return v
-        raise ValueError(f"Invalid hex color format: {v}")
+    def normalize_material_color(cls, v: Any) -> Any:
+        """Normalize the material color, falling back to '' rather than raising."""
+        return _normalize_hex_color(v)
 
 
 class MatlStationInfo(BaseModel):
@@ -81,27 +151,29 @@ class MatlStationInfo(BaseModel):
     # `extra="allow"`: inbound, see SlotInfo.
     model_config = ConfigDict(extra="allow", populate_by_name=True)
 
+    # Defaulted and unconstrained throughout, per the inbound policy at the top
+    # of this module. `slot_infos` in particular had `min_length=1`, which made
+    # an explicitly empty `slotInfos: []` - a station with nothing loaded - fail
+    # the entire /detail response.
     current_load_slot: int = Field(
-        alias="currentLoadSlot", ge=0, le=4, description="Currently loading slot ID (0 if none)"
+        default=0, alias="currentLoadSlot", description="Currently loading slot ID (0 if none)"
     )
     current_slot: int = Field(
-        alias="currentSlot", ge=0, le=4, description="Currently active/printing slot ID (0 if none)"
+        default=0, alias="currentSlot", description="Currently active/printing slot ID (0 if none)"
     )
     slot_cnt: int = Field(
-        alias="slotCnt", ge=1, le=4, description="Total number of slots in the station"
+        default=0, alias="slotCnt", description="Total number of slots in the station"
     )
     slot_infos: list[SlotInfo] = Field(
-        min_length=1,
-        max_length=4,
         default_factory=list,
         alias="slotInfos",
         description="Array of information for each slot",
     )
     state_action: int = Field(
-        alias="stateAction", ge=0, description="Current action state of the material station"
+        default=0, alias="stateAction", description="Current action state of the material station"
     )
     state_step: int = Field(
-        alias="stateStep", ge=0, description="Current step within the state action"
+        default=0, alias="stateStep", description="Current step within the state action"
     )
 
 
@@ -111,24 +183,24 @@ class IndepMatlInfo(BaseModel):
     # `extra="allow"`: inbound, see SlotInfo.
     model_config = ConfigDict(extra="allow", populate_by_name=True)
 
-    material_color: str = Field(alias="materialColor", description="Color of the material")
+    material_color: str = Field(
+        default="", alias="materialColor", description="Color of the material"
+    )
     material_name: str = Field(
-        alias="materialName", description="Name of the material (can be '?' if unknown)"
+        default="",
+        alias="materialName",
+        description="Name of the material (can be '?' if unknown)",
     )
-    state_action: int = Field(alias="stateAction", ge=0, description="Current action state")
+    state_action: int = Field(default=0, alias="stateAction", description="Current action state")
     state_step: int = Field(
-        alias="stateStep", ge=0, description="Current step within the state action"
+        default=0, alias="stateStep", description="Current step within the state action"
     )
 
-    @field_validator("material_color")
+    @field_validator("material_color", mode="before")
     @classmethod
-    def validate_hex_color(cls, v: str) -> str:
-        """Validate that the color is a valid hex color code or empty string."""
-        import re
-
-        if v == "" or re.match(r"^#[0-9A-Fa-f]{6}$", v):
-            return v
-        raise ValueError(f"Invalid hex color format: {v}")
+    def normalize_material_color(cls, v: Any) -> Any:
+        """Normalize the material color, falling back to '' rather than raising."""
+        return _normalize_hex_color(v)
 
 
 class FFGcodeToolData(BaseModel):
@@ -140,33 +212,28 @@ class FFGcodeToolData(BaseModel):
     model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     filament_weight: float = Field(
+        default=0.0,
         alias="filamentWeight",
-        ge=0,
         description="Calculated filament weight for this tool/material in the print",
     )
     material_color: str = Field(
-        alias="materialColor", description="Material color hex string (e.g., '#FFFF00')"
+        default="", alias="materialColor", description="Material color hex string (e.g., '#FFFF00')"
     )
     material_name: str = Field(
-        alias="materialName", description="Name of the material (e.g., 'PLA')"
+        default="", alias="materialName", description="Name of the material (e.g., 'PLA')"
     )
     slot_id: int = Field(
+        default=0,
         alias="slotId",
-        ge=0,
-        le=4,
         description="Slot ID from the material station, if applicable (0 if not or direct)",
     )
-    tool_id: int = Field(alias="toolId", ge=0, le=3, description="Tool ID or extruder number (0-3)")
+    tool_id: int = Field(default=0, alias="toolId", description="Tool ID or extruder number (0-3)")
 
-    @field_validator("material_color")
+    @field_validator("material_color", mode="before")
     @classmethod
-    def validate_hex_color(cls, v: str) -> str:
-        """Validate that the color is a valid hex color code or empty string."""
-        import re
-
-        if v == "" or re.match(r"^#[0-9A-Fa-f]{6}$", v):
-            return v
-        raise ValueError(f"Invalid hex color format: {v}")
+    def normalize_material_color(cls, v: Any) -> Any:
+        """Normalize the material color, falling back to '' rather than raising."""
+        return _normalize_hex_color(v)
 
 
 class FFGcodeFileEntry(BaseModel):
@@ -178,28 +245,27 @@ class FFGcodeFileEntry(BaseModel):
     # filament weight, and the per-tool material data for EVERY file.
     model_config = ConfigDict(extra="allow", populate_by_name=True)
 
+    # `gcode_file_name` stays required - it is the one field without which the
+    # entry means nothing, so failing here is honest. Everything else is
+    # metadata and gets a default.
     gcode_file_name: str = Field(
         alias="gcodeFileName", description="The name of the G-code file (e.g., 'FISH_PLA.3mf')"
     )
     gcode_tool_cnt: int | None = Field(
         default=None,
-        ge=0,
-        le=4,
         alias="gcodeToolCnt",
         description="Number of tools/materials used in this G-code file",
     )
     gcode_tool_datas: list[FFGcodeToolData] | None = Field(
         default=None,
-        max_length=4,
         alias="gcodeToolDatas",
         description="Array of detailed information for each tool/material",
     )
     printing_time: int = Field(
-        alias="printingTime", ge=0, description="Estimated printing time in seconds"
+        default=0, alias="printingTime", description="Estimated printing time in seconds"
     )
     total_filament_weight: float | None = Field(
         default=None,
-        ge=0,
         alias="totalFilamentWeight",
         description="Total estimated filament weight for the print",
     )
@@ -220,33 +286,29 @@ class FFPrinterDetail(BaseModel):
     model_config = ConfigDict(extra="allow")
 
     auto_shutdown: str | None = Field(default=None, alias="autoShutdown")
-    auto_shutdown_time: int | None = Field(default=None, ge=0, alias="autoShutdownTime")
-    camera: int | None = Field(default=None, ge=0, alias="camera")
+    auto_shutdown_time: int | None = Field(default=None, alias="autoShutdownTime")
+    camera: int | None = Field(default=None, alias="camera")
     camera_stream_url: str | None = Field(default=None, alias="cameraStreamUrl")
-    chamber_fan_speed: int | None = Field(default=None, ge=0, le=100, alias="chamberFanSpeed")
-    chamber_target_temp: float | None = Field(
-        default=None, ge=-50, le=500, alias="chamberTargetTemp"
-    )
-    chamber_temp: float | None = Field(default=None, ge=-50, le=500, alias="chamberTemp")
+    chamber_fan_speed: int | None = Field(default=None, alias="chamberFanSpeed")
+    chamber_target_temp: float | None = Field(default=None, alias="chamberTargetTemp")
+    chamber_temp: float | None = Field(default=None, alias="chamberTemp")
     clear_fan_status: str | None = Field(default=None, alias="clearFanStatus")
-    cooling_fan_speed: int | None = Field(default=None, ge=0, le=100, alias="coolingFanSpeed")
-    cooling_fan_left_speed: int | None = Field(
-        default=None, ge=0, le=100, alias="coolingFanLeftSpeed"
-    )
+    cooling_fan_speed: int | None = Field(default=None, alias="coolingFanSpeed")
+    cooling_fan_left_speed: int | None = Field(default=None, alias="coolingFanLeftSpeed")
     coordinate: list[float] | None = Field(default=None, alias="coordinate")
-    cumulative_filament: float | None = Field(default=None, ge=0, alias="cumulativeFilament")
-    cumulative_print_time: int | None = Field(default=None, ge=0, alias="cumulativePrintTime")
-    current_print_speed: int | None = Field(default=None, ge=0, le=1000, alias="currentPrintSpeed")
+    cumulative_filament: float | None = Field(default=None, alias="cumulativeFilament")
+    cumulative_print_time: int | None = Field(default=None, alias="cumulativePrintTime")
+    current_print_speed: int | None = Field(default=None, alias="currentPrintSpeed")
     door_status: str | None = Field(default=None, alias="doorStatus")
     error_code: str | None = Field(default=None, alias="errorCode")
-    estimated_left_len: float | None = Field(default=None, ge=0, alias="estimatedLeftLen")
-    estimated_left_weight: float | None = Field(default=None, ge=0, alias="estimatedLeftWeight")
-    estimated_right_len: float | None = Field(default=None, ge=0, alias="estimatedRightLen")
-    estimated_right_weight: float | None = Field(default=None, ge=0, alias="estimatedRightWeight")
-    estimated_time: float | None = Field(default=None, ge=0, alias="estimatedTime")
-    extrude_ctrl: int | None = Field(default=None, ge=0, alias="extrudeCtrl")
+    estimated_left_len: float | None = Field(default=None, alias="estimatedLeftLen")
+    estimated_left_weight: float | None = Field(default=None, alias="estimatedLeftWeight")
+    estimated_right_len: float | None = Field(default=None, alias="estimatedRightLen")
+    estimated_right_weight: float | None = Field(default=None, alias="estimatedRightWeight")
+    estimated_time: float | None = Field(default=None, alias="estimatedTime")
+    extrude_ctrl: int | None = Field(default=None, alias="extrudeCtrl")
     external_fan_status: str | None = Field(default=None, alias="externalFanStatus")
-    fill_amount: float | None = Field(default=None, ge=0, le=100, alias="fillAmount")
+    fill_amount: float | None = Field(default=None, alias="fillAmount")
     firmware_version: str | None = Field(default=None, alias="firmwareVersion")
     flash_register_code: str | None = Field(default=None, alias="flashRegisterCode")
     # AD5X-only, and firmware omits what does not apply: the Creator 5 series
@@ -261,48 +323,78 @@ class FFPrinterDetail(BaseModel):
     internal_fan_status: str | None = Field(default=None, alias="internalFanStatus")
     ip_addr: str | None = Field(default=None, alias="ipAddr")
     left_filament_type: str | None = Field(default=None, alias="leftFilamentType")
-    left_target_temp: float | None = Field(default=None, ge=-50, le=500, alias="leftTargetTemp")
-    left_temp: float | None = Field(default=None, ge=-50, le=500, alias="leftTemp")
+    left_target_temp: float | None = Field(default=None, alias="leftTargetTemp")
+    left_temp: float | None = Field(default=None, alias="leftTemp")
     light_status: str | None = Field(default=None, alias="lightStatus")
     location: str | None = Field(default=None, alias="location")
     mac_addr: str | None = Field(default=None, alias="macAddr")
     measure: str | None = Field(default=None, alias="measure")
-    move_ctrl: int | None = Field(default=None, ge=0, alias="moveCtrl")
+    move_ctrl: int | None = Field(default=None, alias="moveCtrl")
     name: str | None = Field(default=None, alias="name")
-    nozzle_cnt: int | None = Field(default=None, ge=1, le=4, alias="nozzleCnt")
+    nozzle_cnt: int | None = Field(default=None, alias="nozzleCnt")
     nozzle_model: str | None = Field(default=None, alias="nozzleModel")
-    nozzle_style: int | None = Field(default=None, ge=0, alias="nozzleStyle")
+    nozzle_style: int | None = Field(default=None, alias="nozzleStyle")
     # --- Creator 5 series raw fields ---
     # Immutable factory model name (e.g. "Creator 5 Pro"); unlike `name` this is
     # not user-editable. May be absent on older firmware.
     model: str | None = Field(default=None, alias="model")
     # Per-tool current nozzle temperatures (one entry per nozzle). Multi-nozzle
     # Creator 5 series report these; single-nozzle models use rightTemp/leftTemp.
-    nozzle_temps: list[float] | None = Field(default=None, alias="nozzleTemps")
+    # Entries are nullable because a sentinel reading is normalized to None by
+    # `drop_nozzle_temperature_sentinels` below.
+    nozzle_temps: list[float | None] | None = Field(default=None, alias="nozzleTemps")
     # Per-tool target nozzle temperatures (one entry per nozzle).
-    nozzle_target_temps: list[float] | None = Field(default=None, alias="nozzleTargetTemps")
+    nozzle_target_temps: list[float | None] | None = Field(
+        default=None, alias="nozzleTargetTemps"
+    )
     # Lidar / first-layer scanner presence flag (1 = present, 0 = absent).
-    lidar: int | None = Field(default=None, ge=0, alias="lidar")
-    pid: int | None = Field(default=None, ge=0, alias="pid")
-    plat_target_temp: float | None = Field(default=None, ge=-50, le=500, alias="platTargetTemp")
-    plat_temp: float | None = Field(default=None, ge=-50, le=500, alias="platTemp")
+    lidar: int | None = Field(default=None, alias="lidar")
+    pid: int | None = Field(default=None, alias="pid")
+    plat_target_temp: float | None = Field(default=None, alias="platTargetTemp")
+    plat_temp: float | None = Field(default=None, alias="platTemp")
     polar_register_code: str | None = Field(default=None, alias="polarRegisterCode")
-    print_duration: int | None = Field(default=None, ge=0, alias="printDuration")
+    print_duration: int | None = Field(default=None, alias="printDuration")
     print_file_name: str | None = Field(default=None, alias="printFileName")
     print_file_thumb_url: str | None = Field(default=None, alias="printFileThumbUrl")
-    print_layer: int | None = Field(default=None, ge=0, alias="printLayer")
-    print_progress: float | None = Field(default=None, ge=0, le=100, alias="printProgress")
-    print_speed_adjust: int | None = Field(default=None, ge=0, le=1000, alias="printSpeedAdjust")
-    remaining_disk_space: float | None = Field(default=None, ge=0, alias="remainingDiskSpace")
+    print_layer: int | None = Field(default=None, alias="printLayer")
+    print_progress: float | None = Field(default=None, alias="printProgress")
+    print_speed_adjust: int | None = Field(default=None, alias="printSpeedAdjust")
+    remaining_disk_space: float | None = Field(default=None, alias="remainingDiskSpace")
     right_filament_type: str | None = Field(default=None, alias="rightFilamentType")
-    right_target_temp: float | None = Field(default=None, ge=-50, le=500, alias="rightTargetTemp")
-    right_temp: float | None = Field(default=None, ge=-50, le=500, alias="rightTemp")
+    right_target_temp: float | None = Field(default=None, alias="rightTargetTemp")
+    right_temp: float | None = Field(default=None, alias="rightTemp")
     status: str | None = Field(default=None, alias="status")
-    target_print_layer: int | None = Field(default=None, ge=0, alias="targetPrintLayer")
-    tvoc: float | None = Field(default=None, ge=0, alias="tvoc")
-    z_axis_compensation: float | None = Field(
-        default=None, ge=-10, le=10, alias="zAxisCompensation"
+    target_print_layer: int | None = Field(default=None, alias="targetPrintLayer")
+    tvoc: float | None = Field(default=None, alias="tvoc")
+    z_axis_compensation: float | None = Field(default=None, alias="zAxisCompensation")
+
+    # Firmware reports absent temperature hardware with a negative sentinel
+    # (-108 on a chamber-less Creator 5, issue #18) rather than by omitting the
+    # field. Map those to None here so "no sensor" reads as "not reported"
+    # everywhere downstream, instead of as a -108 C reading or a hard failure.
+    @field_validator(
+        "chamber_temp",
+        "chamber_target_temp",
+        "left_temp",
+        "left_target_temp",
+        "plat_temp",
+        "plat_target_temp",
+        "right_temp",
+        "right_target_temp",
+        mode="before",
     )
+    @classmethod
+    def drop_temperature_sentinels(cls, v: Any) -> Any:
+        """Replace out-of-band 'no sensor' temperature markers with None."""
+        return sanitize_temperature(v)
+
+    @field_validator("nozzle_temps", "nozzle_target_temps", mode="before")
+    @classmethod
+    def drop_nozzle_temperature_sentinels(cls, v: Any) -> Any:
+        """Apply the same sentinel handling to the per-tool temperature arrays."""
+        if not isinstance(v, list):
+            return v
+        return [sanitize_temperature(entry) for entry in v]
 
 
 class FFMachineInfo(BaseModel):
@@ -393,6 +485,11 @@ class FFMachineInfo(BaseModel):
     # Only true on models with confirmed hardware (Creator 5 Pro). When false,
     # `door_open` is cosmetic and should not be surfaced.
     has_door_sensor: bool = False
+    # True only when the printer actually reported a chamber temperature. The
+    # heated chamber is a Creator 5 series *option*, not a family trait: units
+    # without it report the -108 sentinel, which FFPrinterDetail normalizes to
+    # None. Gate chamber entities on this, never on `is_creator5`.
+    has_chamber_sensor: bool = False
 
     # Current print stats
     print_duration: int = 0

flashforge/tcp/parsers/endstop_status.py

@@ -4,10 +4,13 @@ FlashForge Python API - Endstop Status Parser
 Parses endstop and machine status information from M119 command responses.
 """
 
+import logging
 import re
 from enum import Enum
 from typing import Optional
 
+logger = logging.getLogger(__name__)
+
 # Pre-compiled regex to match key:value pairs where value is an integer
 KV_PATTERN = re.compile(r"([A-Za-z0-9-]+):\s*(\d+)")
 
@@ -144,7 +147,7 @@ class EndstopStatus:
                     elif "BUSY" in machine_status:
                         self.machine_status = MachineStatus.BUSY
                     else:
-                        print(f"EndstopStatus: Encountered unknown MachineStatus: {machine_status}")
+                        logger.warning("EndstopStatus: unknown MachineStatus: %s", machine_status)
                         self.machine_status = MachineStatus.DEFAULT
                 elif line.startswith("MoveMode:"):
                     move_mode = line.replace("MoveMode:", "", 1).strip().upper()
@@ -159,7 +162,7 @@ class EndstopStatus:
                     elif "HOMING" in move_mode:
                         self.move_mode = MoveMode.HOMING
                     else:
-                        print(f"EndstopStatus: Encountered unknown MoveMode: {move_mode}")
+                        logger.warning("EndstopStatus: unknown MoveMode: %s", move_mode)
                         self.move_mode = MoveMode.DEFAULT
                 elif line.startswith("Status "):
                     self.status = Status(line)
@@ -184,9 +187,8 @@ class EndstopStatus:
             return self
 
         except Exception as e:
-            print("Unable to create EndstopStatus instance from replay")
-            print(f"Replay: {replay}")
-            print(f"Error: {e}")
+            logger.warning("Unable to create an EndstopStatus from the reply: %s", e)
+            logger.debug("Reply that failed to parse: %s", replay)
             return None
 
     def is_print_complete(self) -> bool:

flashforge/tcp/parsers/print_status.py

@@ -4,8 +4,11 @@ FlashForge Python API - Print Status Parser
 Parses print progress information from M27 command responses.
 """
 
+import logging
 from typing import Optional
 
+logger = logging.getLogger(__name__)
+
 
 class PrintStatus:
     """
@@ -69,13 +72,13 @@ class PrintStatus:
                         return None
 
             if not self.sd_current or not self.sd_total:
-                print("PrintStatus: Invalid SD progress format")
+                logger.warning("PrintStatus: invalid SD progress format.")
                 return None
 
             return self
 
         except Exception as e:
-            print(f"Error parsing print status: {e}")
+            logger.warning("Error parsing the print status: %s", e)
             return None
 
     def get_print_percent(self) -> float: