no longer show slack link in error messages (#1051)

* errro messages

* lint

* tests

* f

Author: edublancas

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## 0.11.1dev
 
+* [Fix] No longer showing the Slack link in error messages
+
 ## 0.11.0 (2025-03-03)
 
 * [API Change] Disabled `%sql` and `%%sql` on Databricks ([#1047](https://github.com/ploomber/jupysql/issues/1047))

src/sql/connection/connection.py

@@ -22,7 +22,6 @@
 from IPython.core.error import UsageError
 import sqlglot
 import sqlparse
-from ploomber_core.exceptions import modify_exceptions
 
 
 from sql.store import store
@@ -951,7 +950,6 @@ def close(self):
         self._connection.engine.dispose()
 
     @classmethod
-    @modify_exceptions
     def _start_sqlalchemy_connection(cls, engine, connect_str):
         try:
             connection = engine.connect()

src/sql/exceptions.py

@@ -18,9 +18,6 @@ def exception_factory(name):
     def _error(message):
         exc = error.UsageError(message)
         exc.error_type = name
-        # this attribute will allow the @modify_exceptions decorator to add the
-        # community link
-        exc.modify_exception = True
         return exc
 
     return _error

src/sql/inspect.py

@@ -1,6 +1,5 @@
 from sqlalchemy import inspect
 from prettytable import PrettyTable
-from ploomber_core.exceptions import modify_exceptions
 from sql.connection import ConnectionManager
 from sql import exceptions
 import math
@@ -165,7 +164,6 @@ def _assign_column_specific_stats(col_stats, is_numeric):
     return col_stats
 
 
-@modify_exceptions
 class Columns(DatabaseInspection):
     """
     Represents the columns in a database table
@@ -195,7 +193,6 @@ def __init__(self, name, schema, conn=None) -> None:
         self._table_txt = self._table.get_string()
 
 
-@modify_exceptions
 class TableDescription(DatabaseInspection):
     """
      Generates descriptive statistics.

src/sql/magic.py

@@ -9,7 +9,6 @@
     from ipywidgets import interact
 except ModuleNotFoundError:
     interact = None
-from ploomber_core.exceptions import modify_exceptions
 from IPython.core.magic import (
     Magics,
     cell_magic,
@@ -367,7 +366,6 @@ def execute(self, line="", cell="", local_ns=None):
             line=line, cell=cell, local_ns=local_ns, is_interactive_mode=False
         )
 
-    @modify_exceptions
     def _execute(self, line, cell, local_ns, is_interactive_mode=False):
         """
         This function implements the cell logic; we create this private
@@ -626,7 +624,6 @@ def interactive_execute_wrapper(**kwargs):
 
     legal_sql_identifier = re.compile(r"^[A-Za-z0-9#_$]+")
 
-    @modify_exceptions
     def _persist_dataframe(
         self, raw, conn, user_ns, append=False, index=True, replace=False
     ):

src/sql/magic_plot.py

@@ -1,6 +1,5 @@
 from IPython.core.magic import Magics, line_magic, magics_class, no_var_expand
 from IPython.core.magic_arguments import argument, magic_arguments
-from ploomber_core.exceptions import modify_exceptions
 
 try:
     from traitlets.config.configurable import Configurable
@@ -76,7 +75,6 @@ class SqlPlotMagic(Magics, Configurable):
         type=float,
         help="Histogram binwidth",
     )
-    @modify_exceptions
     def execute(self, line="", cell="", local_ns=None):
         """
         Plot magic

src/sql/plot.py

@@ -3,7 +3,6 @@
 """
 
 from ploomber_core.dependencies import requires
-from ploomber_core.exceptions import modify_exceptions
 from jinja2 import Template
 
 from sql import exceptions, display
@@ -106,7 +105,6 @@ def _between(conn, table, column, whislo, whishi, with_=None):
 
 
 # https://github.com/matplotlib/matplotlib/blob/b5ac96a8980fdb9e59c9fb649e0714d776e26701/lib/matplotlib/cbook/__init__.py
-@modify_exceptions
 def _boxplot_stats(conn, table, column, whis=1.5, autorange=False, with_=None):
     """Compute statistics required to create a boxplot"""
     if not conn:
@@ -549,7 +547,6 @@ def histogram(
     return ax
 
 
-@modify_exceptions
 def _histogram(
     table, column, bins, with_=None, conn=None, facet=None, breaks=None, binwidth=None
 ):
@@ -688,7 +685,6 @@ def _histogram(
     return bin_, height, bin_size
 
 
-@modify_exceptions
 def _histogram_stacked(
     table,
     column,
@@ -765,7 +761,6 @@ def _histogram_stacked(
     return data
 
 
-@modify_exceptions
 def _filter_aggregate(*filter_queries):
     """Return a single filter query based on multiple queries.
 
@@ -797,7 +792,6 @@ def _filter_aggregate(*filter_queries):
     return final_filter
 
 
-@modify_exceptions
 def _bar(table, column, with_=None, conn=None):
     """get x and height for bar plot"""
     if not conn:
@@ -985,7 +979,6 @@ def bar(
     return ax
 
 
-@modify_exceptions
 def _pie(table, column, with_=None, conn=None):
     """get x and height for pie chart"""
     if not conn:

src/sql/store.py

@@ -2,7 +2,6 @@
 from typing import Iterator, Iterable
 from collections.abc import MutableMapping
 from jinja2 import Template
-from ploomber_core.exceptions import modify_exceptions
 import sql.connection
 import difflib
 
@@ -80,7 +79,6 @@ def infer_dependencies(self, query, key):
                     dependencies.append(table)
         return dependencies
 
-    @modify_exceptions
     def store(self, key, query, with_=None):
         if "-" in key:
             raise exceptions.UsageError(

src/tests/test_magic.py

@@ -29,10 +29,8 @@
 
 from conftest import runsql
 from sql.connection import PLOOMBER_DOCS_LINK_STR
-from ploomber_core.exceptions import COMMUNITY
 import psutil
 
-COMMUNITY = COMMUNITY.strip()
 
 DISPLAYLIMIT_LINK = (
     '<a href="https://jupysql.ploomber.io/en/'
@@ -1163,7 +1161,6 @@ def test_autolimit(ip):
 Set the environment variable $DATABASE_URL
 
 For more details, see: {PLOOMBER_DOCS_LINK_STR}
-{COMMUNITY}
 """
 
 
@@ -1181,7 +1178,6 @@ def test_error_on_invalid_connection_string(ip_empty, clean_conns):
 Ref: https://docs.sqlalchemy.org/en/20/core/engines.html#database-urls
 
 For more details, see: {PLOOMBER_DOCS_LINK_STR}
-{COMMUNITY}
 """  # noqa
 
 
@@ -1207,7 +1203,6 @@ def test_error_on_invalid_connection_string_with_existing_conns(ip_empty, clean_
 Perhaps you meant to use driver the dialect: "sqlite"
 
 For more details, see: {PLOOMBER_DOCS_LINK_STR}
-{COMMUNITY}
 """  # noqa
 
 
@@ -1235,7 +1230,6 @@ def test_error_on_invalid_connection_string_with_possible_typo(ip_empty, clean_c
     Example: %sql postgresql://username:password@hostname/dbname
 
 For more details, see: {PLOOMBER_DOCS_LINK_STR}
-{COMMUNITY}
 """  # noqa
 
 

src/tests/test_magic_cmd.py

@@ -38,23 +38,23 @@ def ip_snippets(ip):
     ip.run_cell("%sql sqlite://")
     ip.run_cell(
         """
-        %%sql --save high_price --no-execute
+%%sql --save high_price --no-execute
 SELECT *
 FROM "test_store"
 WHERE price >= 1.50
 """
     )
     ip.run_cell(
         """
-        %%sql --save high_price_a --no-execute
+%%sql --save high_price_a --no-execute
 SELECT *
 FROM "high_price"
 WHERE symbol == 'a'
 """
     )
     ip.run_cell(
         """
-        %%sql --save high_price_b --no-execute
+%%sql --save high_price_b --no-execute
 SELECT *
 FROM "high_price"
 WHERE symbol == 'b'
@@ -986,7 +986,7 @@ def test_force_delete_all_with_variable_substitution(ip_snippets):
 def test_force_delete_all_child_query(ip_snippets, arg):
     ip_snippets.run_cell(
         """
-        %%sql --save high_price_b_child --no-execute
+%%sql --save high_price_b_child --no-execute
 SELECT *
 FROM "high_price_b"
 WHERE symbol == 'b'