bugbug 0.0.623__tar.gz → 0.0.650__tar.gz

bugbug-0.0.650/.gitignore

@@ -0,0 +1,62 @@
+*model
+*model.zst
+*model.zst.etag
+*data_X
+*data_y
+*data_X.zst
+*data_X.zst.etag
+*data_y.zst
+*data_y.zst.etag
+feature_importance.png
+importance.html
+importances.json
+metrics.json
+probs.json
+http_service/models/*model*
+
+data/
+sheets/
+
+.mypy_cache/
+.pytest_cache/
+*.pyc
+.coverage
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+pip-wheel-metadata/
+share/python-wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+cache/
+node_modules/
+
+# Logs
+*.log
+# Desktop Service Store
+*.DS_Store
+
+# Local env files
+.env
+/http_service/docker-compose.override.yml
+
+# Local files
+*.local.*
+
+# Claude Code local state
+.claude/worktrees/
+.claude/.cc-writes/

bugbug-0.0.650/PKG-INFO

@@ -0,0 +1,66 @@
+Metadata-Version: 2.5
+Name: bugbug
+Version: 0.0.650
+Summary: ML tools for Mozilla projects
+Author-email: Marco Castelluccio <mcastelluccio@mozilla.com>
+License-Expression: MPL-2.0
+License-File: LICENSE
+Classifier: License :: OSI Approved :: Mozilla Public License 2.0 (MPL 2.0)
+Classifier: Programming Language :: Python :: 3 :: Only
+Classifier: Programming Language :: Python :: 3.14
+Requires-Python: >=3.14
+Requires-Dist: amqp<5.5.0,>=5.3.1
+Requires-Dist: async-lru<2.5,>=2.3
+Requires-Dist: beautifulsoup4<4.16.0,>=4.14.3
+Requires-Dist: boto3<1.44.0,>=1.42.78
+Requires-Dist: httpx~=0.28.1
+Requires-Dist: imbalanced-learn~=0.14.1
+Requires-Dist: langchain-anthropic<1.8.0,>=1.3.4
+Requires-Dist: langchain-classic~=1.0.3
+Requires-Dist: langchain-community~=0.4.1
+Requires-Dist: langchain-google-genai<4.4.0,>=4.2.1
+Requires-Dist: langchain-mistralai~=1.1.2
+Requires-Dist: langchain-openai<1.8.0,>=1.1.14
+Requires-Dist: langchain~=1.3.15
+Requires-Dist: langgraph>=1.2.5
+Requires-Dist: libmozdata>=0.2.20
+Requires-Dist: lmdb<2.4.0,>=1.8.1
+Requires-Dist: lxml-html-clean~=0.4.4
+Requires-Dist: markdown2~=2.5.5
+Requires-Dist: matplotlib<3.12.0,>=3.10.8
+Requires-Dist: mercurial~=7.2
+Requires-Dist: microannotate~=0.0.24
+Requires-Dist: mozci~=2.4.10
+Requires-Dist: numpy~=2.4.3
+Requires-Dist: orjson<3.14.0,>=3.11.7
+Requires-Dist: ortools~=9.15.6755
+Requires-Dist: pandas~=3.0.1
+Requires-Dist: psutil~=7.2.2
+Requires-Dist: pyopenssl>=0.14
+Requires-Dist: python-dateutil~=2.9.0.post0
+Requires-Dist: python-hglib~=2.6.2
+Requires-Dist: qdrant-client~=1.15.1
+Requires-Dist: ratelimit~=2.2.1
+Requires-Dist: requests-html~=0.10.0
+Requires-Dist: requests<2.35,>=2.33
+Requires-Dist: rs-parsepatch~=0.4.6
+Requires-Dist: scikit-learn~=1.7.2
+Requires-Dist: scipy<1.19.0,>=1.17.1
+Requires-Dist: searchfox>=0.20.3
+Requires-Dist: sendgrid~=6.12.5
+Requires-Dist: shap[plots]<0.53,>=0.51
+Requires-Dist: tabulate~=0.10.0
+Requires-Dist: taskcluster~=114.0
+Requires-Dist: tenacity~=9.1.4
+Requires-Dist: tqdm<4.71.0,>=4.67.3
+Requires-Dist: unidiff<1.1.0,>=0.7.5
+Requires-Dist: weave>=0.53.12
+Requires-Dist: whatthepatch~=1.0.7
+Requires-Dist: xgboost<3.5,>=3.2
+Requires-Dist: zstandard~=0.25.0
+Provides-Extra: nlp
+Requires-Dist: spacy==3.8.16; extra == 'nlp'
+Provides-Extra: nn
+Provides-Extra: perf-regression-predictor
+Requires-Dist: torch==2.14.1; extra == 'perf-regression-predictor'
+Requires-Dist: transformers==5.20.1; extra == 'perf-regression-predictor'

bugbug-0.0.650/VERSION

@@ -0,0 +1 @@
+0.0.650

{bugbug-0.0.623 → bugbug-0.0.650}/bugbug/bug_features.py

@@ -54,7 +54,11 @@ class HasCrashSignature(SingleBugFeature):
 
 
 class Keywords(SingleBugFeature):
-    def __init__(self, to_ignore=set(), prefixes_to_ignore=set()):
+    def __init__(self, to_ignore=None, prefixes_to_ignore=None):
+        if prefixes_to_ignore is None:
+            prefixes_to_ignore = set()
+        if to_ignore is None:
+            to_ignore = set()
         self.to_ignore = to_ignore
         self.prefixes_to_ignore = prefixes_to_ignore
 
@@ -742,10 +746,7 @@ class IsPerformanceBug(SingleBugFeature):
             return True
 
         bug_whiteboard = bug["whiteboard"].lower()
-        if any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes):
-            return True
-
-        return False
+        return any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes)
 
 
 class IsMemoryBug(SingleBugFeature):
@@ -778,10 +779,7 @@ class IsMemoryBug(SingleBugFeature):
             return True
 
         bug_whiteboard = bug["whiteboard"].lower()
-        if any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes):
-            return True
-
-        return False
+        return any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes)
 
 
 class IsPowerBug(SingleBugFeature):
@@ -805,10 +803,7 @@ class IsPowerBug(SingleBugFeature):
             return True
 
         bug_whiteboard = bug["whiteboard"].lower()
-        if any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes):
-            return True
-
-        return False
+        return any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes)
 
 
 class IsSecurityBug(SingleBugFeature):
@@ -823,6 +818,7 @@ class IsSecurityBug(SingleBugFeature):
         self,
         bug: bugzilla.BugDict,
         bug_map: dict[int, bugzilla.BugDict] | None = None,
+        **kwargs,
     ) -> bool:
         if any(
             keyword.startswith(prefix)
@@ -832,10 +828,7 @@ class IsSecurityBug(SingleBugFeature):
             return True
 
         bug_whiteboard = bug["whiteboard"].lower()
-        if any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes):
-            return True
-
-        return False
+        return any(prefix in bug_whiteboard for prefix in self.whiteboard_prefixes)
 
 
 class IsCrashBug(SingleBugFeature):
@@ -849,20 +842,18 @@ class IsCrashBug(SingleBugFeature):
         self,
         bug: bugzilla.BugDict,
         bug_map: dict[int, bugzilla.BugDict] | None = None,
+        **kwargs,
     ) -> bool:
         # Checking for `[@` will exclude some bugs that do not have valid
         # signatures: https://mzl.la/46XAqRF
         if bug.get("cf_crash_signature") and "[@" in bug["cf_crash_signature"]:
             return True
 
-        if any(
+        return any(
             keyword.startswith(prefix)
             for keyword in bug["keywords"]
             for prefix in self.keyword_prefixes
-        ):
-            return True
-
-        return False
+        )
 
 
 class BugTypes(SingleBugFeature):
@@ -885,13 +876,12 @@ class BugTypes(SingleBugFeature):
         """Infer bug types based on various bug characteristics.
 
         Args:
-        - bug (bugzilla.BugDict): A dictionary containing bug data.
-        - bug_map (Optional[dict[int, bugzilla.BugDict]]): A mapping
-            of bug IDs to bug dictionaries. Default is None.
+            bug: A dictionary containing bug data.
+            bug_map: A mapping of bug IDs to bug dictionaries. Default is `None`.
 
         Returns:
-        - list[str]: A list of inferred bug types (e.g., "memory", "power",
-            "performance", "security", "crash").
+            A list of inferred bug types (e.g., `"memory"`, `"power"`,
+            `"performance"`, `"security"`, `"crash"`).
         """
         return [
             is_type.type_name

{bugbug-0.0.623 → bugbug-0.0.650}/bugbug/bug_snapshot.py

@@ -40,13 +40,13 @@ def keyword_mapping(keyword):
         "bug-quality": "bmo-bug-quality",
     }
 
-    return mapping[keyword] if keyword in mapping else keyword
+    return mapping.get(keyword, keyword)
 
 
 def group_mapping(group):
     mapping = {"release-core-security": "core-security-release"}
 
-    return mapping[group] if group in mapping else group
+    return mapping.get(group, group)
 
 
 def cf_rank(val):
@@ -96,7 +96,7 @@ def product(product):
         "Browser": "Core",
     }
 
-    return mapping[product] if product in mapping else product
+    return mapping.get(product, product)
 
 
 def target_milestone(target_milestone):
@@ -105,9 +105,7 @@ def target_milestone(target_milestone):
 
     mapping = {"6.2.2": "6.2.2.1", "Firefox 3.7": "Firefox 4.0"}
 
-    return (
-        mapping[target_milestone] if target_milestone in mapping else target_milestone
-    )
+    return mapping.get(target_milestone, target_milestone)
 
 
 def null_str(val):
@@ -539,7 +537,7 @@ def rollback(bug, when=None, do_assert=False):
     def assert_or_log(msg):
         msg = f"{msg}, in bug {bug['id']}"
         if do_assert:
-            assert False, msg
+            raise AssertionError(msg)
         else:
             logger.error(msg)
 
@@ -761,11 +759,12 @@ def rollback(bug, when=None, do_assert=False):
 
                 continue
 
-            if change["added"] != "---":
-                if field not in bug and not is_expected_inconsistent_field(
-                    field, last_product, bug["id"]
-                ):
-                    assert_or_log(f"{field} is not present")
+            if (
+                change["added"] != "---"
+                and field not in bug
+                and not is_expected_inconsistent_field(field, last_product, bug["id"])
+            ):
+                assert_or_log(f"{field} is not present")
 
             if field in bug and isinstance(bug[field], list):
                 if change["added"]:

{bugbug-0.0.623 → bugbug-0.0.650}/bugbug/bugzilla.py

@@ -5,6 +5,7 @@
 
 import collections
 import csv
+import itertools
 import math
 import re
 from datetime import datetime
@@ -78,6 +79,7 @@ ATTACHMENT_INCLUDE_FIELDS = [
 ]
 
 COMMENT_INCLUDE_FIELDS = [
+    "_collapsed_comments",
     "id",
     "count",
     "text",
@@ -196,6 +198,28 @@ def get(ids_or_query):
     return new_bugs
 
 
+def get_fields(
+    bug_ids: Iterable[int], include_fields: Iterable[str]
+) -> dict[int, BugDict]:
+    """Fetch only ``include_fields`` of the given bugs, keyed by id.
+
+    Unlike :func:`get`, no comments, attachments or history are requested and
+    ``BUGS_DB`` is left alone.
+    """
+    bugs: dict[int, BugDict] = {}
+    ids = sorted({int(bug_id) for bug_id in bug_ids})
+    if not ids:
+        return bugs
+
+    def bughandler(bug: BugDict) -> None:
+        bugs[int(bug["id"])] = bug
+
+    Bugzilla(
+        ids, include_fields=",".join(include_fields), bughandler=bughandler
+    ).get_data().wait()
+    return bugs
+
+
 def get_ids_between(date_from, date_to=None, security=False, resolution=None):
     params = {
         "f1": "creation_ts",
@@ -232,10 +256,7 @@ def download_bugs(bug_ids: Iterable[int], security: bool = False) -> list[BugDic
 
     new_bug_ids = sorted(list(new_bug_ids_set))
 
-    chunks = (
-        new_bug_ids[i : (i + Bugzilla.BUGZILLA_CHUNK_SIZE)]
-        for i in range(0, len(new_bug_ids), Bugzilla.BUGZILLA_CHUNK_SIZE)
-    )
+    chunks = itertools.batched(new_bug_ids, Bugzilla.BUGZILLA_CHUNK_SIZE, strict=False)
 
     @tenacity.retry(
         stop=tenacity.stop_after_attempt(7),
@@ -411,7 +432,9 @@ def get_product_component_count(months: int = 12) -> dict[str, int]:
     return bugs_number
 
 
-def get_active_product_components(products=[]) -> set[tuple[str, str]]:
+def get_active_product_components(products=None) -> set[tuple[str, str]]:
+    if products is None:
+        products = []
     active_components = set()
 
     def product_handler(product):
@@ -469,6 +492,56 @@ def get_groups_users(group_names: list[str]) -> list[str]:
     ]
 
 
+def fetch_comments(comment_ids: list[int]) -> dict[int, dict]:
+    if not comment_ids:
+        return {}
+
+    response = utils.get_session("bugzilla").get(
+        f"https://bugzilla.mozilla.org/rest/bug/comment/{comment_ids[0]}",
+        params={"comment_ids": comment_ids[1:]},
+        headers={
+            "X-Bugzilla-API-Key": Bugzilla.TOKEN,
+            "User-Agent": utils.get_user_agent(),
+        },
+    )
+
+    if response.status_code == 400 and len(comment_ids) > 1:
+        midpoint = len(comment_ids) // 2
+        return {
+            **fetch_comments(comment_ids[:midpoint]),
+            **fetch_comments(comment_ids[midpoint:]),
+        }
+
+    if response.status_code == 400 and response.json().get("code") in (110, 111):
+        return {}
+
+    response.raise_for_status()
+    return {
+        int(comment_id): comment
+        for comment_id, comment in response.json()["comments"].items()
+    }
+
+
+def get_comments(comment_ids: Iterable[int]) -> dict[int, tuple[BugDict, dict]]:
+    normalized_ids = list(dict.fromkeys(map(int, comment_ids)))
+    comments = {
+        comment_id: comment
+        for comment_ids_batch in itertools.batched(normalized_ids, 100, strict=False)
+        for comment_id, comment in fetch_comments(list(comment_ids_batch)).items()
+    }
+    bugs = get({comment["bug_id"] for comment in comments.values()})
+
+    return {
+        comment_id: (bugs[comment["bug_id"]], comment)
+        for comment_id, comment in comments.items()
+        if comment["bug_id"] in bugs
+    }
+
+
+def get_comment(comment_id: int) -> dict[int, tuple[BugDict, dict]]:
+    return get_comments([comment_id])
+
+
 def get_revision_ids(bug: BugDict) -> list[int]:
     revision_ids = []
 
@@ -560,8 +633,8 @@ def calculate_maintenance_effectiveness_indicator(
 
         return params
 
-    for severity in MAINTENANCE_EFFECTIVENESS_SEVERITY_WEIGHTS.keys():
-        for query_type in data.keys():
+    for severity in MAINTENANCE_EFFECTIVENESS_SEVERITY_WEIGHTS:
+        for query_type in data:
             params = build_query(severity, query_type)
 
             r = utils.get_session("bugzilla").get(
@@ -577,10 +650,10 @@ def calculate_maintenance_effectiveness_indicator(
             data[query_type][severity] = r.json()["bug_count"]
 
     # Calculate number of bugs without severity set.
-    for query_type in data.keys():
+    for query_type in data:
         data[query_type]["--"] = data[query_type]["--"] - sum(
             data[query_type][s]
-            for s in MAINTENANCE_EFFECTIVENESS_SEVERITY_WEIGHTS.keys()
+            for s in MAINTENANCE_EFFECTIVENESS_SEVERITY_WEIGHTS
             if s != "--"
         )
 
@@ -591,7 +664,7 @@ def calculate_maintenance_effectiveness_indicator(
     print("Before applying weights:")
     print(data)
 
-    for query_type in data.keys():
+    for query_type in data:
         # Apply weights.
         for (
             severity,

bugbug-0.0.650/bugbug/build_platforms.py

@@ -0,0 +1,195 @@
+# -*- coding: utf-8 -*-
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this file,
+# You can obtain one at http://mozilla.org/MPL/2.0/.
+
+"""Which platforms build the files of mozilla-central, according to the builds of Firefox's CI.
+
+Searchfox's indexing jobs build Firefox for each platform. Their index lists the source files
+(including headers) the build compiled, and the build's chrome map the files it packaged (JS, CSS,
+...): a file is built on the platforms whose builds list it. A file no build lists (a file added
+since the builds, tests, docs, Rust or Java files, ...) is built on the platforms that build files
+in its closest ancestor directory, or on all platforms if there's none.
+
+A patch only touching files built on some platforms can only break those (e.g. widget/cocoa only
+on macOS, gfx/layers/d3d11 only on Windows).
+"""
+
+import io
+import logging
+import os
+import zipfile
+from typing import Collection, Iterable, Iterator, Mapping
+
+import requests
+
+logger = logging.getLogger(__name__)
+
+# The OS families, named as test_scheduling.config_platform_family names them.
+PLATFORMS = ("android", "linux", "macosx", "windows")
+ALL_PLATFORMS = frozenset(PLATFORMS)
+
+# The Searchfox jobs of each platform (gecko.v2.mozilla-central.latest.firefox.<job>), for the
+# architectures tests run on.
+SEARCHFOX_JOBS = {
+    "android": ("android-aarch64-searchfox-debug",),
+    "linux": ("linux64-searchfox-debug",),
+    "macosx": ("macosx64-searchfox-debug", "macosx64-aarch64-searchfox-debug"),
+    "windows": ("win64-searchfox-debug",),
+}
+# Platforms tested on an architecture their jobs don't build (most Android tests run on x86_64
+# emulators, the job builds aarch64): a file built on other platforms (e.g. x86 SIMD code) isn't
+# excluded if they build files in its directory.
+INCOMPLETE_PLATFORMS = frozenset({"android"})
+# Directories of code that some platforms' jobs don't build, although their tests run it: the Android
+# job doesn't build gtests (4 files in gtest directories, against ~800 for the desktop jobs).
+UNBUILT_DIRECTORIES = {"android": frozenset({"gtest"})}
+
+ARTIFACT_URL = "https://firefox-ci-tc.services.mozilla.com/api/index/v1/task/gecko.v2.mozilla-central.latest.firefox.{job}/artifacts/public/build/{artifact}"
+
+
+class _RemoteFile(io.RawIOBase):
+    """A file read with HTTP range requests, to list the files of a remote zip without downloading it."""
+
+    def __init__(self, url: str) -> None:
+        response = requests.head(url, allow_redirects=True, timeout=60)
+        response.raise_for_status()
+        self.url = response.url
+        self.size = int(response.headers["Content-Length"])
+        self.position = 0
+
+    def readable(self) -> bool:
+        return True
+
+    def seekable(self) -> bool:
+        return True
+
+    def tell(self) -> int:
+        return self.position
+
+    def seek(self, offset: int, whence: int = io.SEEK_SET) -> int:
+        start = {io.SEEK_SET: 0, io.SEEK_CUR: self.position, io.SEEK_END: self.size}
+        self.position = start[whence] + offset
+        return self.position
+
+    def readinto(self, buffer) -> int:
+        end = min(self.size, self.position + len(buffer))
+        if end <= self.position:
+            return 0
+        response = requests.get(
+            self.url,
+            headers={
+                "Range": f"bytes={self.position}-{end - 1}",
+                "Accept-Encoding": "identity",
+            },
+            timeout=60,
+        )
+        response.raise_for_status()
+        data = response.content
+        buffer[: len(data)] = data
+        self.position += len(data)
+        return len(data)
+
+
+def _job_files(job: str) -> set[str]:
+    """The files compiled and packaged by a Searchfox job's build."""
+    url = ARTIFACT_URL.format(job=job, artifact="target.mozsearch-index.zip")
+    with zipfile.ZipFile(_RemoteFile(url)) as index:
+        files = {
+            name
+            for name in index.namelist()
+            if not name.endswith("/") and not name.startswith("__GENERATED__/")
+        }
+
+    response = requests.get(
+        ARTIFACT_URL.format(job=job, artifact="chrome-map.json"), timeout=60
+    )
+    response.raise_for_status()
+    # [chrome URLs, overrides, {installed path: [source path, ...]}, build config]: only the files
+    # packaged in the application, not test files installed for the harnesses (e.g. crashtests).
+    for installed, sources in response.json()[2].items():
+        if (
+            installed.startswith("dist/bin/")
+            and sources
+            and isinstance(sources[0], str)
+            and "*" not in sources[0]
+        ):
+            files.add(sources[0])
+
+    return files
+
+
+def retrieve_built_files() -> dict[str, list[str]]:
+    """The files built on each platform. Platforms whose files can't be retrieved are left out."""
+    built = {}
+    for platform, jobs in SEARCHFOX_JOBS.items():
+        try:
+            built[platform] = sorted(set().union(*(_job_files(job) for job in jobs)))
+        except Exception:
+            logger.exception("Failed to retrieve the files built on %s", platform)
+    return built
+
+
+def _ancestors(path: str) -> Iterator[str]:
+    directory = os.path.dirname(path)
+    while directory:
+        yield directory
+        directory = os.path.dirname(directory)
+
+
+class BuildPlatforms:
+    """The platforms building files, from the files built on each platform (see retrieve_built_files)."""
+
+    def __init__(self, built: Mapping[str, Iterable[str]]) -> None:
+        self.files = {
+            platform: frozenset(files)
+            for platform, files in built.items()
+            if platform in ALL_PLATFORMS and files
+        }
+        self.directories = {
+            platform: frozenset(d for f in files for d in _ancestors(f))
+            for platform, files in self.files.items()
+        }
+        # The types of the files the builds list: the directory of a file of another type (e.g. the
+        # inputs of code generation, like WebIDL or Glean YAML files, or build files) doesn't tell
+        # which platforms use it.
+        self.types = frozenset(
+            os.path.splitext(f)[1] for files in self.files.values() for f in files
+        )
+        # The platforms whose files are unknown might build anything.
+        self.unknown = ALL_PLATFORMS - set(self.files)
+
+    def platforms(self, path: str) -> frozenset[str]:
+        return self._platforms(path) | self.unknown | self._unbuilt(path)
+
+    def _platforms(self, path: str) -> frozenset[str]:
+        listed = {p for p, files in self.files.items() if path in files}
+        if listed:
+            directory = os.path.dirname(path)
+            listed |= {
+                p
+                for p in INCOMPLETE_PLATFORMS & set(self.directories)
+                if directory in self.directories[p]
+            }
+            return frozenset(listed)
+
+        if os.path.splitext(path)[1] not in self.types:
+            return ALL_PLATFORMS
+
+        for directory in _ancestors(path):
+            near = {p for p, dirs in self.directories.items() if directory in dirs}
+            if near:
+                return frozenset(near)
+
+        return ALL_PLATFORMS
+
+    def _unbuilt(self, path: str) -> frozenset[str]:
+        directories = set(path.split("/")[:-1])
+        return frozenset(
+            p for p, names in UNBUILT_DIRECTORIES.items() if directories & names
+        )
+
+    def find_platforms_for_paths(self, paths: Collection[str]) -> frozenset[str]:
+        """The platforms building any of `paths`."""
+        result: frozenset[str] = frozenset().union(*map(self.platforms, paths))
+        return result or ALL_PLATFORMS

{bugbug-0.0.623 → bugbug-0.0.650}/bugbug/code_search/parser.py

@@ -42,7 +42,9 @@ def search(repo_dir, commit_hash, symbol_name):
             return []
 
         logger.error(
-            f"Error running 'hg grep' command.\nstdout:\n{e.stdout.decode()}\n\nstderr:\n{e.stderr.decode()}"
+            "Error running 'hg grep' command.\nstdout:\n %s \n\nstderr:\n %s",
+            e.stdout.decode(),
+            e.stderr.decode(),
         )
         raise
 
@@ -60,7 +62,9 @@ def search(repo_dir, commit_hash, symbol_name):
             )
         except subprocess.CalledProcessError as e:
             logger.error(
-                f"Error running 'hg cat' command.\nstdout:\n{e.stdout.decode()}\n\nstderr:\n{e.stderr.decode()}"
+                "Error running 'hg cat' command.\nstdout:\n%s\n\nstderr:\n%s",
+                e.stdout.decode(),
+                e.stderr.decode(),
             )
             raise