aflib 0.0.7__tar.gz → 0.0.8__tar.gz

aflib-0.0.7/README.md → aflib-0.0.8/PKG-INFO

@@ -1,3 +1,12 @@
+Metadata-Version: 2.5
+Name: aflib
+Version: 0.0.8
+Summary: Helper library for business logic inside Airflows plpython function bodies
+License: MIT
+License-File: LICENSE
+Requires-Python: >=3.11
+Description-Content-Type: text/markdown
+
 # aflib
 
 Helper library for business logic inside [Airflows](https://flows.ninja) `plpython3u`
@@ -18,16 +27,24 @@ so low.
 Airflows resolves Python packages from public PyPI by name and version only; there is no
 local-import, git or private-index route. Declare it in
 `models/pythonPackages/pythonPackages.airflows` by pinning **name and version in
-separate fields** — never as a single `aflib==0.0.7` string (that form produced
+separate fields** — never as a single `aflib==0.0.8` string (that form produced
 malformed `name==ver==ver` entries in a real instance's package list):
 
 ```
 name:    aflib
-version: 0.0.7
+version: 0.0.8
 ```
 
 Then import it from a function body.
 
+## What 0.0.8 fixes
+
+`Batch.run` accepts a predicate whose column name contains the letters `_t`.
+`id_tuition IS NOT NULL`, `start_time < now()` and `is_test = false` are
+predicates. 0.0.7 refused them, because the guard searched for `_t` as a
+substring. `_t.col` and `"_t".col` are still refused: that alias exists only on
+the outer `UPDATE`.
+
 ## What 0.0.7 gives you
 
 Everything in 0.0.6, plus three capabilities that ran on a real instance on
@@ -82,10 +99,10 @@ the 2026-09 platform probes showed, plus two verified fixes:
   resolve.
 - **`identity.current_user(plpy)`** — `SELECT current_user`. Not a User id.
 
-Pin **name and version in separate fields** (`aflib` / `0.0.7`), not
-`aflib==0.0.7` as one string.
+Pin **name and version in separate fields** (`aflib` / `0.0.8`), not
+`aflib==0.0.8` as one string.
 
-## Core API (0.0.1–0.0.3, still in 0.0.7)
+## Core API (0.0.1–0.0.3, still in 0.0.8)
 
 ```python
 from aflib import Db, errors, fail, logging, ok, params

aflib-0.0.7/PKG-INFO → aflib-0.0.8/README.md

@@ -1,12 +1,3 @@
-Metadata-Version: 2.5
-Name: aflib
-Version: 0.0.7
-Summary: Helper library for business logic inside Airflows plpython function bodies
-License: MIT
-License-File: LICENSE
-Requires-Python: >=3.11
-Description-Content-Type: text/markdown
-
 # aflib
 
 Helper library for business logic inside [Airflows](https://flows.ninja) `plpython3u`
@@ -27,16 +18,24 @@ so low.
 Airflows resolves Python packages from public PyPI by name and version only; there is no
 local-import, git or private-index route. Declare it in
 `models/pythonPackages/pythonPackages.airflows` by pinning **name and version in
-separate fields** — never as a single `aflib==0.0.7` string (that form produced
+separate fields** — never as a single `aflib==0.0.8` string (that form produced
 malformed `name==ver==ver` entries in a real instance's package list):
 
 ```
 name:    aflib
-version: 0.0.7
+version: 0.0.8
 ```
 
 Then import it from a function body.
 
+## What 0.0.8 fixes
+
+`Batch.run` accepts a predicate whose column name contains the letters `_t`.
+`id_tuition IS NOT NULL`, `start_time < now()` and `is_test = false` are
+predicates. 0.0.7 refused them, because the guard searched for `_t` as a
+substring. `_t.col` and `"_t".col` are still refused: that alias exists only on
+the outer `UPDATE`.
+
 ## What 0.0.7 gives you
 
 Everything in 0.0.6, plus three capabilities that ran on a real instance on
@@ -91,10 +90,10 @@ the 2026-09 platform probes showed, plus two verified fixes:
   resolve.
 - **`identity.current_user(plpy)`** — `SELECT current_user`. Not a User id.
 
-Pin **name and version in separate fields** (`aflib` / `0.0.7`), not
-`aflib==0.0.7` as one string.
+Pin **name and version in separate fields** (`aflib` / `0.0.8`), not
+`aflib==0.0.8` as one string.
 
-## Core API (0.0.1–0.0.3, still in 0.0.7)
+## Core API (0.0.1–0.0.3, still in 0.0.8)
 
 ```python
 from aflib import Db, errors, fail, logging, ok, params

{aflib-0.0.7 → aflib-0.0.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "aflib"
-version = "0.0.7"
+version = "0.0.8"
 description = "Helper library for business logic inside Airflows plpython function bodies"
 readme = "README.md"
 # The platform runs Python 3.11.2 (observed, not assumed — see docs/platform-constraints.md).

{aflib-0.0.7 → aflib-0.0.8}/src/aflib/__init__.py

@@ -56,7 +56,7 @@ from aflib.result import fail, ok
 # Kept in step with `pyproject.toml` by hand. A deployed body returns this so a run can
 # prove which aflib it actually imported — a stale package otherwise looks exactly like a
 # working one.
-__version__ = "0.0.7"
+__version__ = "0.0.8"
 
 __all__ = [
     "BadParameter",

{aflib-0.0.7 → aflib-0.0.8}/src/aflib/batch.py

@@ -54,6 +54,7 @@ inside the environment's `statement_timeout` — 180 s on PRO, no limit on PRE,
 cannot tell you whether a chunk fits**.
 """
 
+import re
 import time
 
 from aflib.db import Db
@@ -88,6 +89,18 @@ class Progress(object):
             self.done, self.remaining, self.complete)
 
 
+def _alias_as_qualifier(alias, pending):
+    """True when `pending` uses `alias` as a qualifier, not as letters inside a name.
+
+    Searching for the substring `_t` refuses `id_tuition IS NOT NULL`,
+    `start_time < now()` and `is_test = false`. Observed on PRE 2026-10-02,
+    acceptance probe `aflib-007-accept-r1`, case `batch`. The alias is a problem
+    only as `_t.col` or `"_t".col`, with optional space before the dot.
+    """
+    pattern = r'(?<![\w"])"?{0}"?\s*\.'.format(re.escape(alias))
+    return re.search(pattern, pending) is not None
+
+
 class Batch(object):
     """One committed chunk per call. `plpy` is injected, as everywhere in aflib."""
 
@@ -105,14 +118,16 @@ class Batch(object):
         the outer `WHERE`, where Postgres resolves it against the update target, and
         inside the subquery, where it resolves against the subquery's own `FROM`. Both
         resolutions are what we want, and either qualification breaks one of them. A
-        predicate mentioning the alias is rejected rather than silently misplaced.
+        predicate that qualifies a column with the alias (`_t.col` or `"_t".col`) is
+        rejected rather than silently misplaced. A column whose name merely contains
+        those letters — `id_tuition`, `start_time`, `is_test` — is not a qualification.
 
         `order` defaults to `key`, which makes the work deterministic and the chunks
         disjoint. Ordering by something non-unique would let two concurrent callers pick
         overlapping sets; the row-level guard still keeps them correct, but they would
         waste each other's work.
         """
-        if self.ALIAS in pending:
+        if _alias_as_qualifier(self.ALIAS, pending):
             raise ValueError(
                 "`pending` must be unqualified: it is placed both in the outer WHERE and "
                 "inside the subquery, and %r resolves in only one of them." % self.ALIAS)

{aflib-0.0.7 → aflib-0.0.8}/tests/test_batch.py

@@ -66,13 +66,31 @@ class TestThePredicateIsPlacedTwice:
         where = sql.split("SET")[1].split("AND")[0]
         assert PENDING in where, "the first predicate must sit in the UPDATE's own WHERE"
 
-    def test_a_qualified_predicate_is_refused_rather_than_misplaced(self, make_plpy):
+    @pytest.mark.parametrize("pending", [
+        "_t.id_status = $1",
+        '"_t".estado = \'pending\'',
+        '"_t" . estado = \'pending\'',
+        "_t . id = 1",
+    ])
+    def test_a_qualified_predicate_is_refused_rather_than_misplaced(self, make_plpy, pending):
         # It would resolve in the outer WHERE and break inside the subquery, so the
         # helper must not accept it and quietly produce a statement that fails at runtime.
+        # The quoted form is what the PRE acceptance probe sends.
         with pytest.raises(ValueError) as exc:
-            Batch(make_plpy()).run(TABLE, SET, "_t.id_status = $1")
+            Batch(make_plpy()).run(TABLE, SET, pending)
         assert "unqualified" in str(exc.value)
 
+    @pytest.mark.parametrize("pending", [
+        "id_tuition IS NOT NULL", "start_time < now()", "is_test = false",
+    ])
+    def test_a_column_containing_the_alias_text_is_not_a_qualification(self, make_plpy, pending):
+        # Failing on 0.0.7: the guard looks for the substring "_t" anywhere, so every
+        # column containing it is refused. Observed on PRE 2026-10-02, probe
+        # aflib-007-accept-r1, case `batch`: `id_tuition IS NOT NULL` raised ValueError.
+        fake = _plpy(make_plpy, 0, 0)
+        Batch(fake).run(TABLE, SET, pending)
+        assert _sql(fake.executed[0]).count(pending) == 2
+
 
 class TestChunkBoundaries:
     def test_limit_bounds_the_subquery_not_the_update(self, make_plpy):