t-bug-catcher 0.5.4__tar.gz → 0.5.6__tar.gz

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: t_bug_catcher
-Version: 0.5.4
+Version: 0.5.6
 Summary: Bug catcher
 Home-page: https://www.thoughtful.ai/
 Author: Thoughtful

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/setup.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 0.5.4
+current_version = 0.5.6
 commit = True
 tag = False
 

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/setup.py

@@ -26,7 +26,7 @@ setup(
     packages=find_packages(include=["t_bug_catcher", "t_bug_catcher.*"]),
     test_suite="tests",
     url="https://www.thoughtful.ai/",
-    version="0.5.4",
+    version="0.5.6",
     zip_safe=False,
     install_requires=install_requirements,
     include_package_data=True,

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher/__init__.py

@@ -3,7 +3,7 @@
 __author__ = """Thoughtful"""
 __email__ = "support@thoughtful.ai"
 # fmt: off
-__version__ = '0.5.4'
+__version__ = '0.5.6'
 # fmt: on
 
 from .bug_catcher import (
@@ -12,6 +12,7 @@ from .bug_catcher import (
     attach_file_to_exception,
     install_sys_hook,
     uninstall_sys_hook,
+    get_errors_count,
 )
 
 __all__ = [
@@ -20,4 +21,5 @@ __all__ = [
     "attach_file_to_exception",
     "install_sys_hook",
     "uninstall_sys_hook",
+    "get_errors_count",
 ]

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher/bug_catcher.py

@@ -12,7 +12,6 @@ from .jira import Jira
 from .stack_saver import StackSaver
 from .utils import logger
 from .utils.common import get_frames
-from .validation import PRE_RUN_VALIDATION
 
 
 class Configurator:
@@ -93,12 +92,17 @@ class BugCatcher:
         self.__configurator: Configurator = Configurator(self.__jira, self.__bug_snag)
         self.__sys_excepthook = None
         self.__stack_saver = StackSaver()
+        self.__errors_count = 0
 
     @property
     def configure(self):
         """Configures the JiraPoster and BugSnag classes."""
         return self.__configurator
 
+    def get_errors_count(self):
+        """Returns the number of exceptions reported."""
+        return self.__errors_count
+
     def report_error(
         self,
         exception: Optional[Exception] = None,
@@ -187,6 +191,8 @@ class BugCatcher:
         exc_info = f"{os.path.basename(frames[-1].filename)}:{frames[-1].name}:{frames[-1].lineno}"
         exception.handled_error = exc_info
 
+        self.__errors_count += 1
+
     def report_error_to_jira(
         self,
         exception: Optional[Exception] = None,
@@ -287,6 +293,8 @@ class BugCatcher:
         if self.__configurator.is_bugsnag_configured:
             self.__bug_snag.report_unhandled_error(exc_type, exc_value, exc_traceback)
 
+        self.__errors_count += 1
+
     def __get_sys_hook_attribute(self, attribute: str = "bug_catcher_client"):
         """Checks if the system hook is installed.
 
@@ -333,9 +341,4 @@ report_error_to_jira = __bug_catcher.report_error_to_jira
 report_error_to_bugsnag = __bug_catcher.report_error_to_bugsnag
 install_sys_hook = __bug_catcher.install_sys_hook
 uninstall_sys_hook = __bug_catcher.uninstall_sys_hook
-
-if not CONFIG.STAGE:
-    logger.warning("Implementation Stage is not configured. Please configure it before running.")
-else:
-    if CONFIG.STAGE.lower() == "delivery":
-        pre_run_validation = PRE_RUN_VALIDATION
+get_errors_count = __bug_catcher.get_errors_count

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher/bug_snag.py

@@ -1,5 +1,4 @@
 import sys
-from datetime import datetime
 from typing import Optional
 
 import bugsnag
@@ -44,7 +43,6 @@ class BugSnag:
                     "Content-Type": "application/json",
                     "Bugsnag-Api-Key": api_key,
                     "Bugsnag-Payload-Version": "4",
-                    "Bugsnag-Sent-At": f"{datetime.now().strftime('%Y-%m-%dT%H:%M:%S.%f')}",
                     "Bugsnag-Span-Sampling": "True",
                 },
                 data='{"message": "test"}',

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher/jira.py

@@ -476,11 +476,32 @@ class Jira:
             return []
         frames = get_frames(exc_traceback)
         file_name, line_no, _, _ = frames[-1]
-        path = Path.relative_to(Path(file_name), Path.cwd())
-        code_line = (
-            f"{self._build_info['repository_url']}/src/{self._build_info['last_commit']}"
-            f"/{path.as_posix()}?at={self._build_info['branch']}#lines-{line_no}"
-        )
+        try:
+            path = Path.relative_to(Path(file_name), Path.cwd())
+            code_line = (
+                f"{self._build_info['repository_url']}/src/{self._build_info['last_commit']}"
+                f"/{path.as_posix()}?at={self._build_info['branch']}#lines-{line_no}"
+            )
+            payload = [
+                {
+                    "type": "text",
+                    "text": f"{str(path.as_posix())}:{line_no}",
+                    "marks": [
+                        {
+                            "type": "link",
+                            "attrs": {"href": code_line},
+                        }
+                    ],
+                },
+            ]
+        except ValueError:
+            payload = [
+                {
+                    "type": "text",
+                    "text": "Package error.",
+                },
+            ]
+
         input_datetime = datetime.strptime(self._build_info["commit_datetime"], "%Y-%m-%d %H:%M:%S").strftime(
             "%d %B %Y %I:%M:%S %p"
         )
@@ -521,17 +542,8 @@ class Jira:
                         "type": "text",
                         "text": " > ",
                     },
-                    {
-                        "type": "text",
-                        "text": f"{str(path.as_posix())}:{line_no}",
-                        "marks": [
-                            {
-                                "type": "link",
-                                "attrs": {"href": code_line},
-                            }
-                        ],
-                    },
-                ],
+                ]
+                + payload,
             },
             {
                 "type": "paragraph",
@@ -1405,7 +1417,7 @@ class Jira:
                 labels=["bug_catcher", "fatal_error"],
                 priority=priority,
             )
-            if os.path.exists(stack_trace):
+            if stack_trace and os.path.exists(stack_trace):
                 os.remove(stack_trace)
             return response
         except Exception as ex:

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher/workitems.py

@@ -1,5 +1,3 @@
-from .utils import logger
-
 try:
     from RPA.Robocorp.WorkItems import WorkItems
 
@@ -9,6 +7,5 @@ try:
     variables = work_item.get("variables", dict())
     metadata = work_item.get("metadata", dict())
 except (ImportError, KeyError):
-    logger.warning("Workitems unavailable. Variables will be empty.")
     variables = {}
     metadata = {}

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: t_bug_catcher
-Version: 0.5.4
+Version: 0.5.6
 Summary: Bug catcher
 Home-page: https://www.thoughtful.ai/
 Author: Thoughtful

{t_bug_catcher-0.5.4 → t_bug_catcher-0.5.6}/t_bug_catcher.egg-info/SOURCES.txt

@@ -11,7 +11,6 @@ t_bug_catcher/config.py
 t_bug_catcher/exceptions.py
 t_bug_catcher/jira.py
 t_bug_catcher/stack_saver.py
-t_bug_catcher/validation.py
 t_bug_catcher/workitems.py
 t_bug_catcher.egg-info/PKG-INFO
 t_bug_catcher.egg-info/SOURCES.txt

t_bug_catcher-0.5.4/t_bug_catcher/validation.py

@@ -1,123 +0,0 @@
-import ast
-import logging
-import os
-
-from t_bug_catcher.utils.common import strip_path
-
-validation_logger = logging.getLogger("t_bug_catcher")
-
-validation_logger.setLevel(logging.DEBUG)
-console_handler = logging.StreamHandler()
-console_handler.setLevel(logging.DEBUG)
-
-formatter = logging.Formatter("%(levelname)s - %(name)s - %(message)s")
-validation_logger.addHandler(console_handler)
-
-EXCLUDED_DIRS = [".venv", "venv", "site-packages"]
-
-
-class IncorrectTryBlockVisitor(ast.NodeVisitor):
-    """A visitor that checks for incorrect try-except blocks."""
-
-    def __init__(self):
-        """Initializes the IncorrectTryBlockVisitor class."""
-        self.errors = {}
-
-    def visit_Try(self, node):
-        """Visits the try-except block in the AST."""
-        message = (
-            "Error not handled: specify the error type or re-raise exception or "
-            "report it with `t_bug_catcher.report_error()`"
-        )
-        for handler in node.handlers:
-            if isinstance(handler.type, ast.Name) and handler.type.id == "Exception":
-                if not self._contains_raise(handler.body) and not self._contains_correct_error_handling(handler.body):
-                    self.errors[handler.lineno] = message
-            elif handler.type is None:  # 'except:' that catches everything
-                if not any(isinstance(x, ast.Raise) for x in handler.body):
-                    self.errors[handler.lineno] = message
-        self.generic_visit(node)
-
-    @staticmethod
-    def _contains_correct_error_handling(statements):
-        for stmt in statements:
-            if isinstance(stmt, ast.Expr) and isinstance(stmt.value, ast.Call):
-                func = stmt.value.func
-                if (isinstance(func, ast.Name) and func.id == "report_error") or (
-                    isinstance(func, ast.Attribute) and func.attr == "report_error"
-                ):
-                    return True
-        return False
-
-    @staticmethod
-    def _contains_raise(statements):
-        return any(isinstance(stmt, ast.Raise) for stmt in statements)
-
-    def get_errors(self):
-        """Returns the errors found in the try-except blocks."""
-        return self.errors
-
-
-class BroadExceptionWarning:
-    """A class to represent a broad exception warning."""
-
-    def __init__(self, file_path: str, lineno: int, message: str, source_code: str):
-        """Initializes the BroadExceptionWarning class."""
-        self.lineno = lineno
-        self.message = message
-        self.__source_code_lines = source_code.split("\n")
-        self.code_line = self.__source_code_lines[lineno - 1]
-        self.code_lines = "\n".join(self.__source_code_lines[lineno - 3 : lineno + 2])
-        self.file_path = strip_path(file_path)
-
-
-class PreRunValidation:
-    """A class to perform pre-run validation checks."""
-
-    def __init__(self):
-        """Initializes the PreRunValidation class."""
-        self.broad_exception_warnings: list[BroadExceptionWarning] = []
-        self.analyze_project(os.getcwd())
-        self.log_warnings()
-
-    def analyze_project(self, project_dir: str):
-        """Analyzes the project directory for broad exception warnings."""
-        for root, dirs, files in os.walk(project_dir):
-            dirs[:] = [d for d in dirs if d not in EXCLUDED_DIRS]
-            for file in files:
-                if file.endswith(".py"):
-                    file_path = os.path.join(root, file)
-
-                    self.check_broad_exceptions(file_path)
-
-    def check_broad_exceptions(self, filename):
-        """Checks for broad exception warnings in the specified file."""
-        with open(filename, "r", encoding="utf8") as source:
-            source_code = source.read()
-            try:
-                tree = ast.parse(source_code, filename=filename)
-            except SyntaxError:
-                return
-            except Exception as ex:
-                validation_logger.error(f"Unable to validate: {filename} - {ex}")
-                return
-
-        visitor = IncorrectTryBlockVisitor()
-        visitor.visit(tree)
-
-        for line, message in visitor.get_errors().items():
-            be_warn = BroadExceptionWarning(filename, line, message, source_code=source_code)
-            self.broad_exception_warnings.append(be_warn)
-
-    def log_warnings(self):
-        """Logs the broad exception warnings."""
-        for be_warn in self.broad_exception_warnings:
-            validation_logger.warning(f"{be_warn.file_path}:{be_warn.lineno}: {be_warn.message}")
-
-    @property
-    def errors_count(self):
-        """Property to define the number of errors. Used for pre-commit hook."""
-        return len(self.broad_exception_warnings)
-
-
-PRE_RUN_VALIDATION = PreRunValidation()