Move Spark enums to dedicated enums.py module

Extracts SparkDevice, SparkEndpoint, and VENDOR_ID from
spark_reader_async.py to a separate enums.py file for better
organization. These are USB protocol constants, not control logic.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: rickwierenga

pylabrobot/plate_reading/tecan/spark20m/enums.py

@@ -0,0 +1,23 @@
+from enum import Enum
+
+
+# Tecan Spark USB Vendor ID
+VENDOR_ID = 0x0C47
+
+
+class SparkDevice(Enum):
+  """USB product IDs for Spark device modules."""
+
+  FLUORESCENCE = 0x8027
+  ABSORPTION = 0x8026
+  LUMINESCENCE = 0x8022
+  PLATE_TRANSPORT = 0x8028
+
+
+class SparkEndpoint(Enum):
+  """USB endpoint addresses for Spark communication."""
+
+  BULK_IN = 0x82
+  BULK_IN1 = 0x81
+  BULK_OUT = 0x01
+  INTERRUPT_IN = 0x83

pylabrobot/plate_reading/tecan/spark20m/spark_backend.py

@@ -14,8 +14,9 @@
 from .controls.plate_transport_control import MovementSpeed, PlateControl, PlatePosition
 from .controls.sensor_control import InstrumentMessageType, SensorControl
 from .controls.system_control import SystemControl
+from .enums import SparkDevice, SparkEndpoint
 from .spark_processor import AbsorbanceProcessor, FluorescenceProcessor
-from .spark_reader_async import SparkDevice, SparkEndpoint, SparkReaderAsync
+from .spark_reader_async import SparkReaderAsync
 
 logger = logging.getLogger(__name__)
 

pylabrobot/plate_reading/tecan/spark20m/spark_reader_async.py

@@ -1,31 +1,14 @@
 import asyncio
 import logging
-from enum import Enum
 from typing import Optional
 
 import usb.core
 
 from pylabrobot.io.usb import USB
 
+from .enums import VENDOR_ID, SparkDevice, SparkEndpoint
 from .spark_packet_parser import parse_single_spark_packet
 
-# Tecan Spark
-VENDOR_ID = 0x0C47
-
-
-class SparkDevice(Enum):
-  FLUORESCENCE = 0x8027
-  ABSORPTION = 0x8026
-  LUMINESCENCE = 0x8022
-  PLATE_TRANSPORT = 0x8028
-
-
-class SparkEndpoint(Enum):
-  BULK_IN = 0x82
-  BULK_IN1 = 0x81
-  BULK_OUT = 0x01
-  INTERRUPT_IN = 0x83
-
 
 class SparkReaderAsync:
   def __init__(self, vid=VENDOR_ID):

pylabrobot/plate_reading/tecan/spark20m/spark_reader_async_tests.py

@@ -3,11 +3,8 @@
 from unittest.mock import AsyncMock, patch
 
 # Import the module under test
-from pylabrobot.plate_reading.tecan.spark20m.spark_reader_async import (
-  SparkDevice,
-  SparkEndpoint,
-  SparkReaderAsync,
-)
+from pylabrobot.plate_reading.tecan.spark20m.enums import SparkDevice, SparkEndpoint
+from pylabrobot.plate_reading.tecan.spark20m.spark_reader_async import SparkReaderAsync
 
 
 class TestSparkReaderAsync(unittest.IsolatedAsyncioTestCase):