style: apply ruff format

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

Author: neilgfoster

plugin/src/msgraph/client.py

@@ -919,7 +919,7 @@ def _humanize_value(value) -> str:
         parts = []
         for item in value:
             if isinstance(item, dict):  # recipient: {"emailAddress": {"name", "address"}}
-                addr = (item.get("emailAddress") or {})
+                addr = item.get("emailAddress") or {}
                 parts.append(addr.get("address") or addr.get("name") or json.dumps(item))
             else:
                 parts.append(str(item))
@@ -1023,7 +1023,7 @@ def cmd_rule_create(args) -> int:
     )
     print(
         f'Created rule "{args.name}" (id: {created.get("id", "?")}). For mail whose headers '
-        f'contain {args.header_contains}, it {" and ".join(summary)}. '
+        f"contain {args.header_contains}, it {' and '.join(summary)}. "
         f"Verified catch-set was {marker.get('count', '?')} message(s). "
         f"Reverse anytime with rule-remove."
     )
@@ -1084,7 +1084,7 @@ def cmd_searchfolder_list(args) -> int:
         n_src = len(f.get("sourceFolderIds") or [])
         print(
             f'- "{f.get("displayName", "(unnamed)")}"  (id: {f.get("id", "?")})'
-            f'\n    filter: {f.get("filterQuery", "(none)")}  [{scope} over {n_src} source folder(s)]'
+            f"\n    filter: {f.get('filterQuery', '(none)')}  [{scope} over {n_src} source folder(s)]"
         )
     print(f"{len(folders)} search folder(s). Pass --format detailed for full ids.")
     return 0
@@ -1107,8 +1107,7 @@ def cmd_searchfolder_create(args) -> int:
     # Well-known names (inbox, archive, …) are accepted verbatim by Graph; resolve any others to ids.
     _WELL_KNOWN = {"inbox", "archive", "drafts", "sentitems", "deleteditems", "junkemail", "msgfolderroot"}
     source_ids = [
-        n if n.casefold() in _WELL_KNOWN else _resolve_folder_id(tok["access_token"], n)
-        for n in source_names
+        n if n.casefold() in _WELL_KNOWN else _resolve_folder_id(tok["access_token"], n) for n in source_names
     ]
     body = {
         "@odata.type": "microsoft.graph.mailSearchFolder",
@@ -1201,9 +1200,7 @@ def _build_parser() -> argparse.ArgumentParser:
     sfc.add_argument("--name", required=True, help="search folder display name")
     sfc.add_argument("--category", help="build a category filter for this name")
     sfc.add_argument("--filter_query", help="explicit OData filter (overrides --category)")
-    sfc.add_argument(
-        "--source_folders", nargs="+", metavar="FOLDER", help="folders to mine (default: inbox)"
-    )
+    sfc.add_argument("--source_folders", nargs="+", metavar="FOLDER", help="folders to mine (default: inbox)")
     sfc.add_argument(
         "--include_nested",
         type=lambda v: str(v).lower() not in ("false", "0", "no"),

tests/test_client.py

@@ -121,7 +121,12 @@ def test_describe_unknown_verb_steers(self):
 
     def test_read_only_verbs_advertise_read_only(self):
         read_only = (
-            "mail-list", "mail-get", "rule-list", "rule-verify", "category-list", "searchfolder-list",
+            "mail-list",
+            "mail-get",
+            "rule-list",
+            "rule-verify",
+            "category-list",
+            "searchfolder-list",
         )
         for name in read_only:
             tool = next(t for t in client.TOOLS if t["name"] == name)
@@ -512,8 +517,12 @@ def responder(method, url, **kw):
         client._http = rec
         self._capture(
             client.cmd_rule_create,
-            _Args(name="Flag", header_contains=["List-Unsubscribe"], move_to_folder=None,
-                  assign_category=["Needs attention"]),
+            _Args(
+                name="Flag",
+                header_contains=["List-Unsubscribe"],
+                move_to_folder=None,
+                assign_category=["Needs attention"],
+            ),
         )
         rule_post = next(c for c in rec.calls if c[0] == "POST" and c[1].endswith("/messageRules"))
         actions = rule_post[3]["actions"]
@@ -540,8 +549,12 @@ def responder(method, url, **kw):
         client._http = rec
         self._capture(
             client.cmd_rule_create,
-            _Args(name="Both", header_contains=["List-Unsubscribe"], move_to_folder="News",
-                  assign_category=["Needs attention"]),
+            _Args(
+                name="Both",
+                header_contains=["List-Unsubscribe"],
+                move_to_folder="News",
+                assign_category=["Needs attention"],
+            ),
         )
         rule_post = next(c for c in rec.calls if c[0] == "POST" and c[1].endswith("/messageRules"))
         actions = rule_post[3]["actions"]
@@ -598,8 +611,13 @@ def test_create_refuses_without_mail_readwrite(self):
             self._sign_in(scope)
             with self.assertRaises(client.SteerError):
                 client.cmd_searchfolder_create(
-                    _Args(name="N", category="Needs attention", filter_query=None,
-                          source_folders=None, include_nested=True)
+                    _Args(
+                        name="N",
+                        category="Needs attention",
+                        filter_query=None,
+                        source_folders=None,
+                        include_nested=True,
+                    )
                 )
 
     def test_create_refuses_without_filter(self):
@@ -616,8 +634,13 @@ def test_create_shapes_body_with_odata_type(self):
         client._http = rec
         self._capture(
             client.cmd_searchfolder_create,
-            _Args(name="Needs attention", category="Needs attention", filter_query=None,
-                  source_folders=["inbox"], include_nested=True),
+            _Args(
+                name="Needs attention",
+                category="Needs attention",
+                filter_query=None,
+                source_folders=["inbox"],
+                include_nested=True,
+            ),
         )
         post = next(c for c in rec.calls if c[0] == "POST")
         self.assertTrue(post[1].endswith("/me/mailFolders/searchfolders/childFolders"))
@@ -633,8 +656,13 @@ def test_create_explicit_filter_overrides_category(self):
         client._http = rec
         self._capture(
             client.cmd_searchfolder_create,
-            _Args(name="Big", category="Ignored", filter_query="hasAttachments eq true",
-                  source_folders=None, include_nested=False),
+            _Args(
+                name="Big",
+                category="Ignored",
+                filter_query="hasAttachments eq true",
+                source_folders=None,
+                include_nested=False,
+            ),
         )
         body = next(c for c in rec.calls if c[0] == "POST")[3]
         self.assertEqual(body["filterQuery"], "hasAttachments eq true")
@@ -643,9 +671,17 @@ def test_create_explicit_filter_overrides_category(self):
 
     def test_list_read_scope_only(self):
         self._sign_in("Mail.Read MailboxSettings.Read offline_access")
-        payload = {"value": [{"id": "sf1", "displayName": "Needs attention",
-                              "filterQuery": "categories/any(c:c eq 'Needs attention')",
-                              "includeNestedFolders": True, "sourceFolderIds": ["inbox"]}]}
+        payload = {
+            "value": [
+                {
+                    "id": "sf1",
+                    "displayName": "Needs attention",
+                    "filterQuery": "categories/any(c:c eq 'Needs attention')",
+                    "includeNestedFolders": True,
+                    "sourceFolderIds": ["inbox"],
+                }
+            ]
+        }
         rec = _HttpRecorder(lambda method, url, **kw: payload)
         client._http = rec
         out = self._capture(client.cmd_searchfolder_list, _Args(format="concise"))

tests/test_url_construction.py

@@ -88,8 +88,7 @@ def setUp(self):
                 "access_token": "fake-access",
                 "refresh_token": "fake-refresh",
                 "scope": (
-                    "Mail.Read Mail.ReadWrite MailboxSettings.Read "
-                    "MailboxSettings.ReadWrite offline_access"
+                    "Mail.Read Mail.ReadWrite MailboxSettings.Read MailboxSettings.ReadWrite offline_access"
                 ),
                 "expires_at": time.time() + 3600,
             }
@@ -167,8 +166,13 @@ def test_searchfolder_create_url_with_category_filter_is_valid(self):
         # POST path itself must be valid; assert no query/path corruption regardless.
         self._run(
             client.cmd_searchfolder_create,
-            _Args(name="Needs attention", category="Needs attention", filter_query=None,
-                  source_folders=["inbox"], include_nested=True),
+            _Args(
+                name="Needs attention",
+                category="Needs attention",
+                filter_query=None,
+                source_folders=["inbox"],
+                include_nested=True,
+            ),
         )
         self.assertTrue(any(u.endswith("/me/mailFolders/searchfolders/childFolders") for u in self.urls))
         for url in self.urls: