From 720150616d6743f93eae13882c736e0d03b7855f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20NEDJAR?= <sebastien@nedjar.com>
Date: Sat, 28 Mar 2026 18:39:44 +0100
Subject: [PATCH 1/2] tooling: Enable ruff A, B, E722, and PERF rules.

---
 lib/apds9960/apds9960/exceptions.py              | 4 ++--
 lib/bq27441/bq27441/device.py                    | 4 ++--
 lib/ism330dl/examples/static_orientation.py      | 4 ++--
 lib/lis2mdl/examples/tilt_compensated_heading.py | 2 +-
 lib/ssd1327/examples/framebuf_scroll.py          | 2 +-
 lib/ssd1327/examples/random_pixels.py            | 2 +-
 pyproject.toml                                   | 6 ++++--
 tests/runner/mpremote_bridge.py                  | 4 ++--
 8 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/lib/apds9960/apds9960/exceptions.py b/lib/apds9960/apds9960/exceptions.py
index b297e3b1..8e98e1ae 100644
--- a/lib/apds9960/apds9960/exceptions.py
+++ b/lib/apds9960/apds9960/exceptions.py
@@ -1,9 +1,9 @@
 class APDS9960InvalidDevId(ValueError):
-    def __init__(self, id, valid_ids):
+    def __init__(self, device_id, valid_ids):
         Exception.__init__(
             self,
             "Device id 0x{} is not a valid one (valid: {})!".format(
-                format(id, "02x"),
+                format(device_id, "02x"),
                 ", ".join(["0x{}".format(format(i, "02x")) for i in valid_ids]),
             ),
         )
diff --git a/lib/bq27441/bq27441/device.py b/lib/bq27441/bq27441/device.py
index 83a5150e..74b97dec 100644
--- a/lib/bq27441/bq27441/device.py
+++ b/lib/bq27441/bq27441/device.py
@@ -470,9 +470,9 @@ def soft_reset(self):
         return self.execute_control_word(BQ27441_CONTROL_SOFT_RESET)
 
     # Read a 16 - bit command word from the BQ27441-G1A, def format = little endian int16
-    def read_word(self, sub_address, format="<h"):
+    def read_word(self, sub_address, fmt="<h"):
         data = bytes(self._read_reg(sub_address, 2))
-        return struct.unpack(format, data)[0]
+        return struct.unpack(fmt, data)[0]
 
     # Read a 16 - bit subcommand() from the BQ27441-G1A's control()
     def read_control_word(self, function):
diff --git a/lib/ism330dl/examples/static_orientation.py b/lib/ism330dl/examples/static_orientation.py
index c2490fd2..46976078 100644
--- a/lib/ism330dl/examples/static_orientation.py
+++ b/lib/ism330dl/examples/static_orientation.py
@@ -16,8 +16,8 @@
 # -------------------------------------------------
 
 while True:
-    dir = imu.orientation()
+    direction = imu.orientation()
 
-    print("Orientation: {}".format(dir))
+    print("Orientation: {}".format(direction))
 
     sleep_ms(500)
diff --git a/lib/lis2mdl/examples/tilt_compensated_heading.py b/lib/lis2mdl/examples/tilt_compensated_heading.py
index ed59d222..5617874e 100644
--- a/lib/lis2mdl/examples/tilt_compensated_heading.py
+++ b/lib/lis2mdl/examples/tilt_compensated_heading.py
@@ -38,7 +38,7 @@ def pitch_roll_from_accel(ax, ay, az):
     accel = imu.acceleration_g()
     ax, ay, az = accel
 
-    heading = mag.heading_with_tilt_compensation(lambda: accel)
+    heading = mag.heading_with_tilt_compensation(lambda a=accel: a)
     pitch_deg, roll_deg = pitch_roll_from_accel(ax, ay, az)
     direction = mag.direction_label(heading)
 
diff --git a/lib/ssd1327/examples/framebuf_scroll.py b/lib/ssd1327/examples/framebuf_scroll.py
index d5ba75c2..621580ec 100644
--- a/lib/ssd1327/examples/framebuf_scroll.py
+++ b/lib/ssd1327/examples/framebuf_scroll.py
@@ -19,7 +19,7 @@
 display.show()
 sleep(1)
 
-for i in range(2):
+for _ in range(2):
     display.scroll(0, 16)  # framebuf.scroll
     display.show()
     sleep(1)
diff --git a/lib/ssd1327/examples/random_pixels.py b/lib/ssd1327/examples/random_pixels.py
index c04eced0..0acf1a78 100644
--- a/lib/ssd1327/examples/random_pixels.py
+++ b/lib/ssd1327/examples/random_pixels.py
@@ -12,7 +12,7 @@
 # random pixels (slow)
 # note: urandom not available on all devices
 
-for i in range(256):
+for _ in range(256):
     x = uos.urandom(1)[0] // 2
     y = uos.urandom(1)[0] // 2
     display.pixel(x, y, 15)
diff --git a/pyproject.toml b/pyproject.toml
index 1f2b07a9..ecbef757 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -38,7 +38,9 @@ target-version = "py37"
 
 [tool.ruff.lint]
 select = [
+  "A",      # flake8-builtins: prevent shadowing builtins
   "ASYNC",  # flake8-async
+  "B",      # flake8-bugbear: common bug patterns
   "C4",     # flake8-comprehensions
   "C90",    # McCabe cyclomatic complexity
   "DTZ",    # flake8-datetimez
@@ -60,6 +62,7 @@ select = [
   "RSE",    # flake8-raise
   "RUF",    # Ruff-specific rules
   "T10",    # flake8-debugger
+  "PERF",   # Perflint
   "T20",    # flake8-print: no print() in production code
   "TCH",    # flake8-type-checking
   "W",      # pycodestyle
@@ -67,7 +70,6 @@ select = [
 ]
 ignore = [
   "E501",    # line length, recommended to disable
-  "E722",    # bare except
   "E741",    # 'l' is currently widely used
   "F401",    # unused import (wildcard re-exports)
   "F403",    # wildcard import
@@ -104,7 +106,7 @@ max-statements = 166
 [tool.ruff.lint.per-file-ignores]
 # manifest.py files are evaluated with some global names pre-defined
 "**/manifest.py" = ["F821"]
-"**/examples/*.py" = ["T20", "N806"]
+"**/examples/*.py" = ["T20", "N806", "PERF203"]
 "tests/**/*.py" = ["T20"]
 
 [tool.ruff.format]
diff --git a/tests/runner/mpremote_bridge.py b/tests/runner/mpremote_bridge.py
index 8bd52303..cd03de06 100644
--- a/tests/runner/mpremote_bridge.py
+++ b/tests/runner/mpremote_bridge.py
@@ -173,11 +173,11 @@ def run_raw_script(self, script, mount_dir=None):
         last_line = lines[-1] if lines else ""
         try:
             return json.loads(last_line)
-        except json.JSONDecodeError:
+        except json.JSONDecodeError as err:
             full = "\n".join(lines)
             raise RuntimeError(
                 f"Script did not produce valid JSON result.\nFull output:\n{full}"
-            )
+            ) from err
 
     def scan_bus(self, i2c_config):
         """Scan I2C bus and return list of addresses."""

From 0735ba921b0b30580e400c868b6d6b9b6e6ba297 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20NEDJAR?= <sebastien@nedjar.com>
Date: Sat, 28 Mar 2026 18:45:17 +0100
Subject: [PATCH 2/2] tooling: Fix Copilot review comments on ruff rules PR.

---
 lib/bq27441/bq27441/device.py | 2 +-
 pyproject.toml                | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/bq27441/bq27441/device.py b/lib/bq27441/bq27441/device.py
index 74b97dec..cba16a54 100644
--- a/lib/bq27441/bq27441/device.py
+++ b/lib/bq27441/bq27441/device.py
@@ -469,7 +469,7 @@ def reset(self):
     def soft_reset(self):
         return self.execute_control_word(BQ27441_CONTROL_SOFT_RESET)
 
-    # Read a 16 - bit command word from the BQ27441-G1A, def format = little endian int16
+    # Read a 16-bit command word from the BQ27441-G1A, default fmt = little-endian int16
     def read_word(self, sub_address, fmt="<h"):
         data = bytes(self._read_reg(sub_address, 2))
         return struct.unpack(fmt, data)[0]
diff --git a/pyproject.toml b/pyproject.toml
index ecbef757..b7051582 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -106,7 +106,8 @@ max-statements = 166
 [tool.ruff.lint.per-file-ignores]
 # manifest.py files are evaluated with some global names pre-defined
 "**/manifest.py" = ["F821"]
-"**/examples/*.py" = ["T20", "N806", "PERF203"]
+"**/examples/*.py" = ["T20", "N806"]
+"lib/mcp23009e/examples/i2c_scan.py" = ["PERF203"]
 "tests/**/*.py" = ["T20"]
 
 [tool.ruff.format]