From 81c190c9c58abd498e63629e173506b8839a0e83 Mon Sep 17 00:00:00 2001 From: Chris Coutinho Date: Wed, 6 May 2026 23:40:53 +0200 Subject: [PATCH] fix(webdav): finish lazy-logging conversion in get_tag_by_name The two debug calls in get_tag_by_name were left as f-strings when the method was migrated to _make_request in round 1. Convert to lazy %-style formatting per repo convention (PR #764 review round 5). Co-Authored-By: Claude Opus 4.7 (1M context) --- nextcloud_mcp_server/client/webdav.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nextcloud_mcp_server/client/webdav.py b/nextcloud_mcp_server/client/webdav.py index cff493a5..25d004e2 100644 --- a/nextcloud_mcp_server/client/webdav.py +++ b/nextcloud_mcp_server/client/webdav.py @@ -1190,10 +1190,10 @@ class WebDAVClient(BaseNextcloudClient): and user_assignable_elem.text is not None else True, } - logger.debug(f"Found tag '{tag_name}' with ID {tag_info['id']}") + logger.debug("Found tag %r with ID %s", tag_name, tag_info["id"]) return tag_info - logger.debug(f"Tag '{tag_name}' not found") + logger.debug("Tag %r not found", tag_name) return None async def get_files_by_tag(self, tag_id: int) -> list[dict[str, Any]]: