moar pylint

Author: jedgarpark

Fruit_Jam/Larsio_Paint_Music/code.py

@@ -7,7 +7,7 @@
 or Metro RP2350 with EYESPI DVI breakout and TLV320DAC3100 breakout on STEMMA_I2C,
 pin D7 reset, 9/10/11 = BCLC/WSEL/DIN
 """
-# pylint: disable=invalid-name,too-few-public-methods,broad-exception-caught,redefined-outer-name
+# pylint: disable=invalid-name,too-few-public-methods,broad-except,redefined-outer-name
 
 # Main application file for Larsio Paint Music
 
@@ -101,7 +101,7 @@ def run(self):
     try:
         app = MusicStaffApp(audio_output=AUDIO_OUTPUT)
         app.run()
-    except Exception as e:
+    except Exception as e:  # pylint: disable=broad-except
         print(f"Error with I2S audio: {e}")
 
         # Force garbage collection
@@ -112,5 +112,5 @@ def run(self):
         try:
             app = MusicStaffApp(audio_output="pwm")
             app.run()
-        except Exception as e2:
+        except Exception as e2:  # pylint: disable=broad-except
             print(f"Fatal error: {e2}")

Fruit_Jam/Larsio_Paint_Music/control_panel.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # control_panel.py: CircuitPython Music Staff Application component
 """

Fruit_Jam/Larsio_Paint_Music/cursor_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # cursor_manager.py: CircuitPython Music Staff Application component
 """

Fruit_Jam/Larsio_Paint_Music/display_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # display_manager.py: CircuitPython Music Staff Application component
 """

Fruit_Jam/Larsio_Paint_Music/input_handler.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # input_handler.py: CircuitPython Music Staff Application component
 """
@@ -11,8 +14,8 @@
 
 
 # pylint: disable=invalid-name,no-member,too-many-instance-attributes,too-many-arguments
-# pylint: disable=too-many-branches,too-many-statements,broad-exception-caught
-# pylint: disable=too-many-nested-blocks,too-many-locals,too-many-positional-arguments
+# pylint: disable=too-many-branches,too-many-statements,broad-except
+# pylint: disable=too-many-nested-blocks,too-many-locals,no-self-use
 class InputHandler:
     """Handles user input through mouse and interactions with UI elements"""
 
@@ -62,7 +65,7 @@ def find_mouse(self):
                         try:
                             manufacturer = device.manufacturer
                             product = device.product
-                        except Exception:  # Specify exception type
+                        except Exception:  # pylint: disable=broad-except
                             manufacturer = "Unknown"
                             product = "Unknown"
 
@@ -74,13 +77,13 @@ def find_mouse(self):
                             has_kernel_driver = hasattr(device, 'is_kernel_driver_active')
                             if has_kernel_driver and device.is_kernel_driver_active(0):
                                 device.detach_kernel_driver(0)
-                        except Exception as e:
+                        except Exception as e:  # pylint: disable=broad-except
                             print(f"Error detaching kernel driver: {e}")
 
                         # Set configuration
                         try:
                             device.set_configuration()
-                        except Exception as e:
+                        except Exception as e:  # pylint: disable=broad-except
                             print(f"Error setting configuration: {e}")
                             continue  # Try next device
 
@@ -98,7 +101,7 @@ def find_mouse(self):
                             buf = bytearray(1)
                             device.ctrl_transfer(bmRequestType, bRequest, wValue, wIndex, buf)
                             print("Set to report protocol mode")
-                        except Exception as e:
+                        except Exception as e:  # pylint: disable=broad-except
                             print(f"Could not set protocol: {e}")
 
                         # Buffer for reading data
@@ -115,14 +118,14 @@ def find_mouse(self):
                             # Timeout is normal if mouse isn't moving
                             print("Mouse connected but not sending data (normal)")
                             return True
-                        except Exception as e:
+                        except Exception as e:  # pylint: disable=broad-except
                             print(f"Mouse test read failed: {e}")
                             # Continue to try next device or retry
                             self.mouse = None
                             self.in_endpoint = None
                             continue
 
-                    except Exception as e:
+                    except Exception as e:  # pylint: disable=broad-except
                         print(f"Error initializing device: {e}")
                         continue
 
@@ -134,7 +137,7 @@ def find_mouse(self):
                 gc.collect()
                 time.sleep(RETRY_DELAY)
 
-            except Exception as e:
+            except Exception as e:  # pylint: disable=broad-except
                 print(f"Error during mouse detection: {e}")
                 gc.collect()
                 time.sleep(RETRY_DELAY)
@@ -204,7 +207,7 @@ def process_mouse_input(self):
                 gc.collect()
 
             return False
-        except Exception as e:
+        except Exception as e:  # pylint: disable=broad-except
             print(f"Error reading mouse: {type(e).__name__}")
             return False
 

Fruit_Jam/Larsio_Paint_Music/note_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # note_manager.py: CircuitPython Music Staff Application component
 """
@@ -8,7 +11,7 @@
 
 # pylint: disable=invalid-name,no-member,too-many-instance-attributes,too-many-arguments
 # pylint: disable=too-many-branches,too-many-statements,protected-access,too-many-locals
-# pylint: disable=too-many-positional-arguments, trailing-whitespace
+# pylint: disable=trailing-whitespace
 class NoteManager:
     """Manages notes, their positions, and related data"""
 

Fruit_Jam/Larsio_Paint_Music/playback_controller.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 Playback controller for CircuitPython Music Staff Application.
 Manages the playback state, button displays, and sound triggering.

Fruit_Jam/Larsio_Paint_Music/sound_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # sound_manager.py: CircuitPython Music Staff Application component
 """

Fruit_Jam/Larsio_Paint_Music/sprite_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 Sprite manager for CircuitPython Music Staff Application.
 Handles loading and managing sprite images and palettes.
@@ -8,7 +11,7 @@
 from displayio import Palette, TileGrid
 
 
-# pylint: disable=too-many-instance-attributes,invalid-name,broad-exception-caught
+# pylint: disable=too-many-instance-attributes,invalid-name,broad-except
 class SpriteManager:
     """Manages sprites and palettes for note display"""
 

Fruit_Jam/Larsio_Paint_Music/staff_view.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # staff_view.py: Larsio Paint Music component
 """

Fruit_Jam/Larsio_Paint_Music/ui_manager.py

@@ -1,3 +1,6 @@
+# SPDX-FileCopyrightText: 2025 John Park and Claude AI for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
 """
 # ui_manager.py: CircuitPython Music Staff Application component
 """
@@ -21,7 +24,7 @@
 
 # pylint: disable=invalid-name,no-member,too-many-instance-attributes,too-many-arguments
 # pylint: disable=too-many-branches,too-many-statements,too-many-public-methods
-# pylint: disable=too-many-locals,attribute-defined-outside-init,possibly-used-before-assignment
+# pylint: disable=too-many-locals,attribute-defined-outside-init
 # pylint: disable=consider-using-in,too-many-return-statements,no-else-return
 class UIManager:
     """Manages the UI elements, input, and user interaction"""