fix: Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @tests/test_workspace_db_special_paths.py at line 66, The test unpacks a second value from the context manager returned by open_global_db(ws_root) into the variable name path but never uses it, causing a Ruff RUF059 lint error; update the unpacked name in the with statement (the tuple unpack from open_global_db in tests/test_workspace_db_special_paths.py) to a discard name like _path or _ to mark it intentionally unused while leaving conn unchanged.

Author: bradjin8

tests/test_workspace_db_special_paths.py

@@ -63,7 +63,7 @@ def test_build_composer_id_to_workspace_id_handles_spaces(self):
     def test_open_global_db_handles_spaces(self):
         with tempfile.TemporaryDirectory() as tmp:
             ws_root = self._build_fixture(tmp)
-            with open_global_db(ws_root) as (conn, path):
+            with open_global_db(ws_root) as (conn, _):
                 self.assertIsNotNone(conn)
                 row = conn.execute(
                     "SELECT key FROM cursorDiskKV WHERE key = 'composerData:probe'"