execsql2 2.16.5__tar.gz → 2.16.7__tar.gz

{execsql2-2.16.5 → execsql2-2.16.7}/CHANGELOG.md

@@ -13,6 +13,24 @@ ______________________________________________________________________
 
 ______________________________________________________________________
 
+## [2.16.7] - 2026-04-30
+
+### Fixed
+
+- ANDIF/ORIF conditions now short-circuit: `IF (sub_defined(x)) ANDIF (not sub_empty(x))` no longer evaluates `sub_empty` when `sub_defined` returns false. Previously all modifiers were evaluated unconditionally, causing `sub_empty` to throw "Unrecognized substitution variable" on undefined variables.
+- Error reports for IF, LOOP, and INCLUDE nodes now show the correct source line and metacommand text instead of the previous command's location.
+
+______________________________________________________________________
+
+## [2.16.6] - 2026-04-30
+
+### Fixed
+
+- `execsql-format` no longer corrupts PL/pgSQL function bodies inside `$$`-delimited blocks. sqlglot does not understand PL/pgSQL and was rewriting `IF NOT EXISTS ... END IF`, `IF ... THEN RETURN ... END IF`, and similar constructs as `COMMIT;`. The formatter now tracks `$$` boundaries and skips sqlglot formatting for any SQL block containing dollar-quoted content.
+- Debug REPL `.vars` now shows `~` local and `#` param variables from the current stack frame, not just global variables. `.vars ~myvar` and `.set ~myvar value` also correctly read/write the stack frame's local scope instead of the global pool.
+
+______________________________________________________________________
+
 ## [2.16.5] - 2026-04-30
 
 ### Fixed

{execsql2-2.16.5 → execsql2-2.16.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: execsql2
-Version: 2.16.5
+Version: 2.16.7
 Summary: Runs a SQL script against a PostgreSQL, SQLite, MariaDB/MySQL, DuckDB, Firebird, MS-Access, MS-SQL-Server, or Oracle database, or an ODBC DSN. Provides metacommands to import and export data, copy data between databases, conditionally execute SQL and metacommands, and dynamically alter SQL and metacommands with substitution variables.
 Project-URL: Homepage, https://execsql2.readthedocs.io
 Project-URL: Repository, https://github.com/geocoug/execsql

{execsql2-2.16.5 → execsql2-2.16.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "execsql2"
-version = "2.16.5"
+version = "2.16.7"
 description = "Runs a SQL script against a PostgreSQL, SQLite, MariaDB/MySQL, DuckDB, Firebird, MS-Access, MS-SQL-Server, or Oracle database, or an ODBC DSN. Provides metacommands to import and export data, copy data between databases, conditionally execute SQL and metacommands, and dynamically alter SQL and metacommands with substitution variables."
 readme = { file = "README.md", content-type = "text/markdown" }
 license = { file = "LICENSE.txt" }
@@ -165,7 +165,7 @@ skip-magic-trailing-comma = false
 line-ending = "auto"
 
 [tool.bumpversion]
-current_version = "2.16.5"
+current_version = "2.16.7"
 commit = true
 tag = true
 tag_name = "v{new_version}"

{execsql2-2.16.5 → execsql2-2.16.7}/src/execsql/format.py

@@ -242,16 +242,27 @@ def format_file(source: str, indent: int = 4, use_sql: bool = True) -> str:
     sql_acc: list[str] = []
     output: list[str] = []
 
+    in_dollar_quote = False
+
     def flush_sql() -> None:
+        nonlocal in_dollar_quote
         if sql_acc:
-            output.extend(format_sql_block(sql_acc, depth, indent, use_sql))
+            # If any line in the accumulated block is inside a $$-delimited
+            # region, skip sqlglot formatting entirely.  PL/pgSQL function
+            # bodies contain IF/END IF, LOOP, RETURN, etc. that sqlglot does
+            # not understand and will corrupt (e.g., rewriting to COMMIT).
+            safe_for_sqlglot = use_sql and not in_dollar_quote
+            output.extend(format_sql_block(sql_acc, depth, indent, safe_for_sqlglot))
             sql_acc.clear()
 
     for raw_line in source.expandtabs(4).splitlines():
         m = METACOMMAND_RE.match(raw_line)
 
         if not raw_line.strip():
-            flush_sql()
+            if not in_dollar_quote:
+                flush_sql()
+            else:
+                sql_acc.append(raw_line)
             output.append("")
 
         elif m:
@@ -279,6 +290,9 @@ def format_file(source: str, indent: int = 4, use_sql: bool = True) -> str:
                 output.append(format_metacommand(payload, depth, indent))
 
         else:
+            # Track $$ boundaries to prevent sqlglot from mangling PL/pgSQL
+            if "$$" in raw_line and raw_line.count("$$") % 2 == 1:
+                in_dollar_quote = not in_dollar_quote
             sql_acc.append(raw_line)
 
     flush_sql()

{execsql2-2.16.5 → execsql2-2.16.7}/src/execsql/script/executor.py

@@ -139,13 +139,24 @@ def _eval_condition(
     condition: str,
     modifiers: list[ConditionModifier] | None = None,
 ) -> bool:
-    """Evaluate a condition string with optional ANDIF/ORIF modifiers."""
+    """Evaluate a condition string with optional ANDIF/ORIF modifiers.
+
+    Short-circuits ANDIF (stops on first False) and ORIF (stops on first True)
+    so that patterns like ``IF (sub_defined(x)) ANDIF (not sub_empty(x))``
+    don't evaluate ``sub_empty`` when ``x`` is undefined.
+    """
     effective_locals = _stack_localvars(ctx)
     expanded = substitute_vars(condition, effective_locals, ctx=ctx)
     result = xcmd_test(expanded)
 
     if modifiers:
         for mod in modifiers:
+            # Short-circuit: AND with False can't become True,
+            # OR with True can't become False.
+            if mod.kind == "AND" and not result:
+                continue
+            if mod.kind == "OR" and result:
+                continue
             mod_expanded = substitute_vars(mod.condition, effective_locals, ctx=ctx)
             mod_result = xcmd_test(mod_expanded)
             if mod.kind == "AND":
@@ -353,9 +364,11 @@ def _execute_node(
         _exec_metacommand(ctx, command, node.span.file, node.span.start_line, localvars)
 
     elif isinstance(node, IfBlock):
+        ctx.last_command = _FakeScriptCmd(node)
         _execute_if(ctx, node, localvars, in_loop=in_loop)
 
     elif isinstance(node, LoopBlock):
+        ctx.last_command = _FakeScriptCmd(node)
         _execute_loop(ctx, node, localvars)
 
     elif isinstance(node, BatchBlock):
@@ -368,6 +381,7 @@ def _execute_node(
         _execute_sql_block(ctx, node, localvars, in_loop=in_loop)
 
     elif isinstance(node, IncludeDirective):
+        ctx.last_command = _FakeScriptCmd(node)
         _execute_include(ctx, node, localvars)
 
 

{execsql2-2.16.5 → execsql2-2.16.7}/tests/test_executor.py

@@ -434,6 +434,41 @@ class TestAssert:
         assert result.returncode != 0
 
 
+# ---------------------------------------------------------------------------
+# Short-circuit ANDIF/ORIF
+# ---------------------------------------------------------------------------
+
+
+class TestShortCircuit:
+    def test_andif_short_circuits(self, tmp_path):
+        """ANDIF does not evaluate sub_empty when sub_defined is false."""
+        script = (
+            "CREATE TABLE t (x INT);\n"
+            "-- !x! IF (sub_defined(undefined_var))\n"
+            "-- !x! ANDIF (not sub_empty(undefined_var))\n"
+            "INSERT INTO t VALUES (1);\n"
+            "-- !x! ELSE\n"
+            "INSERT INTO t VALUES (2);\n"
+            "-- !x! ENDIF\n"
+        )
+        result = _run_ast(script, tmp_path)
+        assert result.returncode == 0, result.stderr
+        assert _query_db(tmp_path, "SELECT x FROM t") == [(2,)]
+
+    def test_orif_short_circuits(self, tmp_path):
+        """ORIF does not evaluate when first condition is already true."""
+        script = (
+            "CREATE TABLE t (x INT);\n"
+            "-- !x! IF (TRUE)\n"
+            "-- !x! ORIF (not sub_empty(undefined_var))\n"
+            "INSERT INTO t VALUES (1);\n"
+            "-- !x! ENDIF\n"
+        )
+        result = _run_ast(script, tmp_path)
+        assert result.returncode == 0, result.stderr
+        assert _query_db(tmp_path, "SELECT x FROM t") == [(1,)]
+
+
 # ---------------------------------------------------------------------------
 # Profiling
 # ---------------------------------------------------------------------------

{execsql2-2.16.5 → execsql2-2.16.7}/uv.lock

@@ -648,7 +648,7 @@ wheels = [
 
 [[package]]
 name = "execsql2"
-version = "2.16.5"
+version = "2.16.7"
 source = { editable = "." }
 dependencies = [
     { name = "python-dateutil" },