sql-error-categorizer 0.1.9__tar.gz → 0.1.10__tar.gz

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/Makefile

@@ -34,7 +34,7 @@ uninstall: $(VENV)
 	$(VENV_BIN)/python -m pip uninstall -y $(NAME)
 
 documentation:
-	make html -C docs/
+	make html SPHINXBUILD="../$(VENV_BIN)/sphinx-build" -C docs/
 
 test: install
 	$(VENV_BIN)/python -m pytest

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sql_error_categorizer
-Version: 0.1.9
+Version: 0.1.10
 Summary: This project analyses SQL statements and labels possible errors or complications.
 Project-URL: Repository, https://github.com/DavidePonzini/sql_error_categorizer
 Project-URL: Documentation, https://sql-error-categorizer.readthedocs.io/en/latest/index.html

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/docs/conf.py

@@ -29,7 +29,9 @@ exclude_patterns = ['_build', 'Thumbs.db', '.DS_Store']
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-html-output
 
 html_theme = 'alabaster'
-html_static_path = ['_static']
+html_static_path = [
+    # '_static',
+]
 
 
 # -- Autoapi -----------------------------------------------------------------

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sql_error_categorizer"
-version = "0.1.9"
+version = "0.1.10"
 authors = [
   { name="Davide Ponzini", email="davide.ponzini95@gmail.com" },
 ]

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/detectors/logical.py

@@ -711,14 +711,14 @@ class LogicalErrorDetector(BaseDetector):
     
     def _selects_star(self, ast: dict) -> bool:
         '''
-        Checks if a 'SELECT *' is used in the query by looking for a 'Star'
+        Checks if a `SELECT *` is used in the query by looking for a 'Star'
         node in the AST's expression list.
 
         Args:
             ast: The Abstract Syntax Tree of the query.
 
         Returns:
-            True if 'SELECT *' is found, otherwise False.
+            True if `SELECT *` is found, otherwise False.
         '''
         if not ast:
             return False

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/set_operations/__init__.py

@@ -78,11 +78,9 @@ def create_set_operation_tree(sql: str, catalog: Catalog = Catalog(), search_pat
 def parse_op_token(tok: sqlparse.sql.Token) -> tuple[str, bool | None] | None:
     '''
     Parse "UNION", "INTERSECT", "EXCEPT" with optional inline ALL/DISTINCT.
+
     Returns:
-        tuple: `(op, all_flag)` where all_flag is:
-        - True  if ALL inline (e.g., "UNION ALL")
-        - False if DISTINCT inline (e.g., "EXCEPT DISTINCT")
-        - None  if no modifier inline (so caller may look right).
+        tuple: `(op, all_flag)` where all_flag is: True  if ALL inline (e.g., "UNION ALL"); False if DISTINCT inline (e.g., "EXCEPT DISTINCT"); None  if no modifier inline (so caller may look right).
     '''
     if tok.ttype is not Keyword:
         return None
@@ -106,11 +104,9 @@ def split_on(tokens: list[sqlparse.sql.Token], idx: int, all_in_token: bool | No
     '''
     Splits around the operator at idx. If the modifier wasn't inline,
     consume a single immediate ALL/DISTINCT to the right.
+    
     Returns:
-        tuple: A tuple containing:
-            - left_tokens (list[sqlparse.sql.Token]): Tokens to the left of the operator.
-            - right_tokens (list[sqlparse.sql.Token]): Tokens to the right of the operator
-            - all_flag (bool | None): True if ALL, False if DISTINCT, None if unspecified.
+        tuple: A tuple containing: left_tokens (list[sqlparse.sql.Token]): Tokens to the left of the operator; right_tokens (list[sqlparse.sql.Token]): Tokens to the right of the operator; all_flag (bool | None): True if ALL, False if DISTINCT, None if unspecified.
     '''
     left_tokens = tokens[:idx]
     right_tokens = tokens[idx + 1:]

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/set_operations/binary_set_operation.py

@@ -3,6 +3,7 @@ from ...catalog import Table, Constraint, ConstraintType, ConstraintColumn
 
 from abc import ABC
 from copy import deepcopy
+import sqlglot
 from sqlglot import exp
 
 from typing import TYPE_CHECKING
@@ -44,6 +45,18 @@ class BinarySetOperation(SetOperation, ABC):
 
         return result
 
+    @property
+    def trailing_ast(self) -> exp.Expression | None:
+        '''Parses and returns the AST of the trailing SQL clauses (e.g., ORDER BY, LIMIT) if present, with a fake `SELECT 1` prefix.'''
+        if self.trailing_sql is None:
+            return None
+        if self._trailing_ast is None:
+            # Parse trailing SQL with a fake SELECT to get valid AST
+            fake_sql = f'SELECT 1 {self.trailing_sql}'
+            parsed = sqlglot.parse_one(fake_sql)
+            self._trailing_ast = parsed
+        return self._trailing_ast
+
     @property
     def output(self) -> Table:
         # Assume the output schema is the same as the left input

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/set_operations/set_operation.py

@@ -40,18 +40,6 @@ class SetOperation(ABC):
     @abstractmethod
     def print_tree(self, pre: str = '') -> None:
         pass
-
-    @property
-    def trailing_ast(self) -> exp.Expression | None:
-        '''Parses and returns the AST of the trailing SQL clauses (e.g., ORDER BY, LIMIT) if present, with a fake `SELECT 1` prefix.'''
-        if self.trailing_sql is None:
-            return None
-        if self._trailing_ast is None:
-            # Parse trailing SQL with a fake SELECT to get valid AST
-            fake_sql = f'SELECT 1 {self.trailing_sql}'
-            parsed = sqlglot.parse_one(fake_sql)
-            self._trailing_ast = parsed
-        return self._trailing_ast
     
     @property
     @abstractmethod

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/binary_ops.py

@@ -1,7 +1,8 @@
 from .base import get_type
 from ...catalog import Catalog
 from sqlglot import exp
-from .types import ResultType, AtomicType, DataType
+from .types import ResultType, AtomicType
+from sqlglot.expressions import DataType
 from .util import is_number, to_number, to_date, error_message
 
 @get_type.register

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/functions.py

@@ -1,7 +1,8 @@
 from .base import get_type
 from ...catalog import Catalog
 from sqlglot import exp
-from .types import ResultType, AtomicType, DataType
+from .types import ResultType, AtomicType
+from sqlglot.expressions import DataType
 from .util import is_number, error_message
 
 @get_type.register

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/predicates.py

@@ -1,7 +1,8 @@
 from .base import get_type
 from ...catalog import Catalog
 from sqlglot import exp
-from .types import ResultType, AtomicType, DataType
+from .types import ResultType, AtomicType
+from sqlglot.expressions import DataType
 from .util import is_string, to_number, to_date, error_message
 
 @get_type.register

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/primitives.py

@@ -1,7 +1,8 @@
 from .base import get_type
 from ...catalog import Catalog
 from sqlglot import exp
-from .types import ResultType, AtomicType, DataType, TupleType
+from .types import ResultType, AtomicType, TupleType
+from sqlglot.expressions import DataType
 from .util import is_number, is_date, to_number, to_date, error_message
 from ...util.ast.column import get_real_name, get_schema
 

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/unary_ops.py

@@ -1,7 +1,8 @@
 from .base import get_type
 from ...catalog import Catalog
 from sqlglot import exp
-from .types import ResultType, AtomicType, DataType
+from .types import ResultType, AtomicType
+from sqlglot.expressions import DataType
 from .util import is_number, error_message
 
 @get_type.register

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/query/typechecking/util.py

@@ -1,4 +1,5 @@
-from .types import ResultType, DataType
+from .types import ResultType
+from sqlglot.expressions import DataType
 from dateutil.parser import parse
 from sqlglot import exp
 

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/sql_errors.py

@@ -1,7 +1,7 @@
-from enum import Enum
+from enum import IntEnum
 
 
-class SqlErrors(Enum):
+class SqlErrors(IntEnum):
     '''Enumeration of SQL error types with unique identifiers.'''
     SYN_1_OMITTING_CORRELATION_NAMES                                    = 1
     SYN_2_AMBIGUOUS_COLUMN                                              = 2

{sql_error_categorizer-0.1.9 → sql_error_categorizer-0.1.10}/src/sql_error_categorizer/util/ast/__init__.py

@@ -1,9 +1,6 @@
 '''Utility functions for processing SQL ASTs made with sqlglot.'''
 
-from .column import *
-from .function import *
-from .subquery import *
-from .table import *
+from . import column, function, subquery, table
 
 import sqlglot.optimizer.normalize
 from sqlglot import exp