style: apply ruff-format to entire codebase

First-time run of ruff-format via pre-commit hook normalises quote
style, trailing commas, and whitespace across 188 Python files.
No logic changes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 2 people

CLAUDE.md

@@ -187,3 +187,56 @@ Each signal type has its own Flask blueprint:
 ## Testing Notes
 
 Tests use pytest with extensive mocking of external tools. Key fixtures in `tests/conftest.py`. Mock subprocess calls when testing decoder integration.
+
+### Think Before Coding
+
+Don't assume. Don't hide confusion. Surface tradeoffs.
+
+Before implementing:
+
+State your assumptions explicitly. If uncertain, ask.
+If multiple interpretations exist, present them - don't pick silently.
+If a simpler approach exists, say so. Push back when warranted.
+If something is unclear, stop. Name what's confusing. Ask.
+2. Simplicity First
+
+Minimum code that solves the problem. Nothing speculative.
+
+No features beyond what was asked.
+No abstractions for single-use code.
+No "flexibility" or "configurability" that wasn't requested.
+No error handling for impossible scenarios.
+If you write 200 lines and it could be 50, rewrite it.
+Ask yourself: "Would a senior engineer say this is overcomplicated?" If yes, simplify.
+
+### Surgical Changes
+
+Touch only what you must. Clean up only your own mess.
+
+When editing existing code:
+
+Don't "improve" adjacent code, comments, or formatting.
+Don't refactor things that aren't broken.
+Match existing style, even if you'd do it differently.
+If you notice unrelated dead code, mention it - don't delete it.
+When your changes create orphans:
+
+Remove imports/variables/functions that YOUR changes made unused.
+Don't remove pre-existing dead code unless asked.
+The test: Every changed line should trace directly to the user's request.
+
+### Goal-Driven Execution
+
+Define success criteria. Loop until verified.
+
+Transform tasks into verifiable goals:
+
+"Add validation" → "Write tests for invalid inputs, then make them pass"
+"Fix the bug" → "Write a test that reproduces it, then make it pass"
+"Refactor X" → "Ensure tests pass before and after"
+For multi-step tasks, state a brief plan:
+
+1. [Step] → verify: [check]
+2. [Step] → verify: [check]
+3. [Step] → verify: [check]
+

bin/import_artemis.py

@@ -38,9 +38,7 @@
 REPO_ROOT = Path(__file__).resolve().parent.parent
 SIGNALS_JSON = REPO_ROOT / "data" / "signals.json"
 
-RELEASE_INFO_URL = (
-    "https://raw.githubusercontent.com/AresValley/Artemis/master/config/release-info.json"
-)
+RELEASE_INFO_URL = "https://raw.githubusercontent.com/AresValley/Artemis/master/config/release-info.json"
 
 _LOCATION_MAP: dict[str, str] = {
     "worldwide": "GLOBAL",
@@ -65,6 +63,7 @@
 # Download helpers
 # ---------------------------------------------------------------------------
 
+
 def _fetch_json(url: str) -> dict:
     with urllib.request.urlopen(url, timeout=15) as r:
         return json.loads(r.read())
@@ -97,18 +96,14 @@ def _download_db(dest_dir: Path) -> Path:
     extract_dir.mkdir()
     with tarfile.open(tar_path) as tf:
         # Safety: strip any absolute paths or '..' traversal
-        members = [
-            m for m in tf.getmembers()
-            if not os.path.isabs(m.name) and ".." not in m.name
-        ]
+        members = [m for m in tf.getmembers() if not os.path.isabs(m.name) and ".." not in m.name]
         tf.extractall(extract_dir, members=members)
 
     # Find data.sqlite anywhere in the extracted tree
     matches = list(extract_dir.rglob("data.sqlite"))
     if not matches:
         raise FileNotFoundError(
-            f"No data.sqlite found after extracting {tar_path}. "
-            "The Artemis-DB tar structure may have changed."
+            f"No data.sqlite found after extracting {tar_path}. The Artemis-DB tar structure may have changed."
         )
 
     return matches[0]
@@ -118,6 +113,7 @@ def _download_db(dest_dir: Path) -> Path:
 # Discovery (for users who have Artemis installed)
 # ---------------------------------------------------------------------------
 
+
 def _find_installed_db() -> Path | None:
     home = Path.home()
     candidates = [
@@ -140,6 +136,7 @@ def _find_installed_db() -> Path | None:
 # Conversion helpers
 # ---------------------------------------------------------------------------
 
+
 def _slugify(name: str) -> str:
     slug = re.sub(r"[^a-z0-9]+", "-", name.lower()).strip("-")
     return slug[:60]
@@ -206,6 +203,7 @@ def _bandwidth_range(bw_values: list[int]) -> dict[str, int] | None:
 # Database loading
 # ---------------------------------------------------------------------------
 
+
 def load_artemis(db_path: Path, limit: int = 0, existing_ids: set[str] | None = None) -> list[dict]:
     """Read signals from an Artemis data.sqlite and return them in Intercept's schema."""
     with closing(sqlite3.connect(str(db_path))) as conn:
@@ -281,17 +279,19 @@ def load_artemis(db_path: Path, limit: int = 0, existing_ids: set[str] | None =
         slug = _unique_slug(_slugify(name.strip()), used_ids)
         used_ids.add(slug)
 
-        results.append({
-            "id": slug,
-            "name": name.strip(),
-            "description": (description or "").strip(),
-            "categories": cats[sig_id],
-            "frequency_ranges": freq_ranges,
-            "bandwidth_range": _bandwidth_range(bws[sig_id]),
-            "modulations": mods[sig_id],
-            "regions": locs[sig_id] or ["GLOBAL"],
-            "sigidwiki_url": sigidwiki_url,
-        })
+        results.append(
+            {
+                "id": slug,
+                "name": name.strip(),
+                "description": (description or "").strip(),
+                "categories": cats[sig_id],
+                "frequency_ranges": freq_ranges,
+                "bandwidth_range": _bandwidth_range(bws[sig_id]),
+                "modulations": mods[sig_id],
+                "regions": locs[sig_id] or ["GLOBAL"],
+                "sigidwiki_url": sigidwiki_url,
+            }
+        )
 
     if skipped_no_freq:
         print(f"  Skipped {skipped_no_freq} signals with no frequency data")
@@ -302,6 +302,7 @@ def load_artemis(db_path: Path, limit: int = 0, existing_ids: set[str] | None =
 # Merge
 # ---------------------------------------------------------------------------
 
+
 def merge(existing: list[dict], imported: list[dict]) -> tuple[list[dict], int, int]:
     existing_names = {s["name"].lower() for s in existing}
     merged = list(existing)
@@ -320,30 +321,38 @@ def merge(existing: list[dict], imported: list[dict]) -> tuple[list[dict], int,
 # Main
 # ---------------------------------------------------------------------------
 
+
 def main() -> None:
     parser = argparse.ArgumentParser(
         description="Import Artemis signal database into data/signals.json",
         formatter_class=argparse.RawDescriptionHelpFormatter,
         epilog=__doc__,
     )
     parser.add_argument(
-        "db_path", nargs="?",
+        "db_path",
+        nargs="?",
         help="Path to data.sqlite (omit to auto-detect installed Artemis DB)",
     )
     parser.add_argument(
-        "--download", action="store_true",
+        "--download",
+        action="store_true",
         help="Download the latest Artemis-DB tar (~290 MB) and import automatically",
     )
     parser.add_argument(
-        "--dry-run", action="store_true",
+        "--dry-run",
+        action="store_true",
         help="Show import stats without writing data/signals.json",
     )
     parser.add_argument(
-        "--no-merge", action="store_true",
+        "--no-merge",
+        action="store_true",
         help="Replace data/signals.json entirely instead of merging",
     )
     parser.add_argument(
-        "--limit", type=int, default=0, metavar="N",
+        "--limit",
+        type=int,
+        default=0,
+        metavar="N",
         help="Only process the first N signals (for testing)",
     )
     args = parser.parse_args()

data/patterns.py

@@ -1,39 +1,83 @@
 # Detection patterns for various device types
 
 # Known beacon prefixes for tracker detection
-AIRTAG_PREFIXES = ['4C:00']  # Apple continuity
-TILE_PREFIXES = ['C4:E7', 'DC:54', 'E4:B0', 'F8:8A']
-SAMSUNG_TRACKER = ['58:4D', 'A0:75']
+AIRTAG_PREFIXES = ["4C:00"]  # Apple continuity
+TILE_PREFIXES = ["C4:E7", "DC:54", "E4:B0", "F8:8A"]
+SAMSUNG_TRACKER = ["58:4D", "A0:75"]
 
 # Drone detection patterns (SSID patterns)
 DRONE_SSID_PATTERNS = [
     # DJI
-    'DJI-', 'DJI_', 'Mavic', 'Phantom', 'Spark-', 'Mini-', 'Air-', 'Inspire',
-    'Matrice', 'Avata', 'FPV-', 'Osmo', 'RoboMaster', 'Tello',
+    "DJI-",
+    "DJI_",
+    "Mavic",
+    "Phantom",
+    "Spark-",
+    "Mini-",
+    "Air-",
+    "Inspire",
+    "Matrice",
+    "Avata",
+    "FPV-",
+    "Osmo",
+    "RoboMaster",
+    "Tello",
     # Parrot
-    'Parrot', 'Bebop', 'Anafi', 'Disco-', 'Mambo', 'Swing',
+    "Parrot",
+    "Bebop",
+    "Anafi",
+    "Disco-",
+    "Mambo",
+    "Swing",
     # Autel
-    'Autel', 'EVO-', 'Dragonfish', 'Lite+', 'Nano',
+    "Autel",
+    "EVO-",
+    "Dragonfish",
+    "Lite+",
+    "Nano",
     # Skydio
-    'Skydio',
+    "Skydio",
     # Other brands
-    'Holy Stone', 'Potensic', 'SYMA', 'Hubsan', 'Eachine', 'FIMI',
-    'Xiaomi_FIMI', 'Yuneec', 'Typhoon', 'PowerVision', 'PowerEgg',
+    "Holy Stone",
+    "Potensic",
+    "SYMA",
+    "Hubsan",
+    "Eachine",
+    "FIMI",
+    "Xiaomi_FIMI",
+    "Yuneec",
+    "Typhoon",
+    "PowerVision",
+    "PowerEgg",
     # Generic drone patterns
-    'Drone', 'UAV-', 'Quadcopter', 'FPV_', 'RC-Drone'
+    "Drone",
+    "UAV-",
+    "Quadcopter",
+    "FPV_",
+    "RC-Drone",
 ]
 
 # Drone OUI prefixes (MAC address prefixes for drone manufacturers)
 DRONE_OUI_PREFIXES = {
     # DJI
-    '60:60:1F': 'DJI', '48:1C:B9': 'DJI', '34:D2:62': 'DJI', 'E0:DB:55': 'DJI',
-    'C8:6C:87': 'DJI', 'A0:14:3D': 'DJI', '70:D7:11': 'DJI', '98:3A:56': 'DJI',
+    "60:60:1F": "DJI",
+    "48:1C:B9": "DJI",
+    "34:D2:62": "DJI",
+    "E0:DB:55": "DJI",
+    "C8:6C:87": "DJI",
+    "A0:14:3D": "DJI",
+    "70:D7:11": "DJI",
+    "98:3A:56": "DJI",
     # Parrot
-    '90:03:B7': 'Parrot', 'A0:14:3D': 'Parrot', '00:12:1C': 'Parrot', '00:26:7E': 'Parrot',
+    "90:03:B7": "Parrot",
+    "A0:14:3D": "Parrot",
+    "00:12:1C": "Parrot",
+    "00:26:7E": "Parrot",
     # Autel
-    '8C:F5:A3': 'Autel', 'D8:E0:E1': 'Autel',
+    "8C:F5:A3": "Autel",
+    "D8:E0:E1": "Autel",
     # Yuneec
-    '60:60:1F': 'Yuneec',
+    "60:60:1F": "Yuneec",
     # Skydio
-    'F8:0F:6F': 'Skydio',
+    "F8:0F:6F": "Skydio",
 }