From 9f200c15b74f7787eb4412845ab6890663ef2870 Mon Sep 17 00:00:00 2001
From: Esteban Zimanyi <estebanzimanyi@gmail.com>
Date: Fri, 10 Jul 2026 12:11:23 +0200
Subject: [PATCH] Make the streaming-relevance baseline reproducible in-repo
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The generator codegen_facades.py projects the JMEOS FFI surface against
tools/baseline/streaming-relevance-baseline.json, but that baseline was a
frozen snapshot produced by an out-of-repo classifier and recorded a
dangling absolute source_catalog path, so it could not be regenerated
here — the last non-reproducible input in the generation chain.

Vendor the deterministic classifier as tools/classify_streaming_relevance.py
(the tier of a function is decided purely by its name, object-model role,
and temporal-parameter count) plus a tools/regen_baseline.sh wrapper, and
document the full input chain in GENERATION.md:

  MobilityDB @ tools/meos-source-commit.txt
    -> MEOS-API/run.py            -> meos-idl.json
    -> classify_streaming_relevance.py -> streaming-relevance-baseline.json
    -> codegen_facades.py         -> org.mobilitydb.meos.MeosOps* facades

Regenerate the committed baseline through the vendored classifier so its
provenance is portable. The function classification is byte-identical to
the previous baseline, so the 57 generated facade classes are unchanged;
only the metadata (source_catalog, generated_at) differs.
---
 GENERATION.md                                 |  29 ++
 .../streaming-relevance-baseline.json         |   5 +-
 tools/classify_streaming_relevance.py         | 321 ++++++++++++++++++
 tools/regen_baseline.sh                       |  30 ++
 4 files changed, 382 insertions(+), 3 deletions(-)
 create mode 100755 tools/classify_streaming_relevance.py
 create mode 100755 tools/regen_baseline.sh

diff --git a/GENERATION.md b/GENERATION.md
index 2c9131c..2beb3d7 100644
--- a/GENERATION.md
+++ b/GENERATION.md
@@ -25,6 +25,35 @@ emits the `org.mobilitydb.meos.MeosOps*` 1:1 forwarder facades the Flink process
 The facades are a *consumer* projection (they live here, not in JMEOS, so the JMEOS FFI line
 and the facade line do not diverge).
 
+## The streaming-relevance baseline (generator input)
+
+`tools/codegen_facades.py` emits a facade only for functions in the
+**streaming-relevant** tiers, read from `tools/baseline/streaming-relevance-baseline.json`.
+That baseline is itself **generated and reproducible** — it is not a hand-maintained
+list. It is produced by **`tools/classify_streaming_relevance.py`**, a deterministic
+classifier: the tier of a function is decided purely by its name, its object-model role,
+and its number of temporal parameters (zero per-function judgement), so the same MEOS
+catalog always yields the same baseline.
+
+The full input chain, from the tracked MobilityDB commit:
+
+```
+MobilityDB @ tools/meos-source-commit.txt
+  → MEOS-API/run.py over the MEOS headers        → meos-idl.json  (the catalog)
+  → tools/classify_streaming_relevance.py        → tools/baseline/streaming-relevance-baseline.json
+  → tools/codegen_facades.py  (jar-surface ∩ baseline)   → org.mobilitydb.meos.MeosOps* facades
+```
+
+To refresh the baseline (e.g. after bumping `tools/meos-source-commit.txt`): rebuild the
+catalog at the tracked commit with MEOS-API, then
+
+```
+tools/regen_baseline.sh <path-to-meos-idl.json>
+```
+
+and commit the diff. Because the classifier is deterministic, an unchanged catalog
+regenerates the baseline byte-for-byte.
+
 ## Generate-then-retire — the green-CI version is the probe
 
 Hand-written facades/glue are replaced by the generated forwarders **family by family,
diff --git a/tools/baseline/streaming-relevance-baseline.json b/tools/baseline/streaming-relevance-baseline.json
index 4055e12..fd5342d 100644
--- a/tools/baseline/streaming-relevance-baseline.json
+++ b/tools/baseline/streaming-relevance-baseline.json
@@ -1,7 +1,6 @@
 {
   "schema_version": "0.1.0-draft",
-  "generated_at": "2026-05-21T07:46:14.181946Z",
-  "source_catalog": "/home/esteban/src/_streaming_relevance/meos-api/output/meos-idl.json",
+  "source_catalog": "meos-idl.json (MobilityDB master, MEOS-API run.py)",
   "source_catalog_function_count": 3548,
   "classifier": "v4",
   "vocabulary": [
@@ -41647,4 +41646,4 @@
       "n_params": 3
     }
   ]
-}
\ No newline at end of file
+}
diff --git a/tools/classify_streaming_relevance.py b/tools/classify_streaming_relevance.py
new file mode 100755
index 0000000..3690fff
--- /dev/null
+++ b/tools/classify_streaming_relevance.py
@@ -0,0 +1,321 @@
+#!/usr/bin/env python3
+"""Classify the MEOS public API into streaming tiers — the reproducible
+producer of ``tools/baseline/streaming-relevance-baseline.json``.
+
+The baseline is the generator input consumed by ``codegen_facades.py``
+(which emits a facade only for the streaming-relevant tiers). This script
+is deterministic: the tier of a function is decided purely by its name,
+its object-model role, and the number of temporal parameters — there is
+zero per-function judgement, so the same MEOS catalog always yields the
+same baseline.
+
+Usage:
+    classify_streaming_relevance.py <meos-idl.json> [-o <out.json>]
+                                    [--source-ref <label>]
+
+``<meos-idl.json>`` is the MEOS-API catalog produced by running
+``MEOS-API/run.py`` over the MEOS headers at the commit recorded in
+``tools/meos-source-commit.txt`` (see GENERATION.md for the full chain).
+The output defaults to stdout. ``--source-ref`` records portable
+provenance in the artifact instead of an absolute local path.
+"""
+import argparse
+import json
+import re
+import sys
+from collections import Counter, defaultdict
+from os.path import basename
+
+TIERS = ['stateless', 'bounded-state', 'windowed', 'cross-stream',
+         'sequence-only', 'io-meta', 'internal', 'ambiguous']
+
+PUBLIC_HEADERS = {
+    'meos.h', 'meos_geo.h', 'meos_cbuffer.h', 'meos_npoint.h',
+    'meos_pose.h', 'meos_rgeo.h', 'meos_transform.h', 'meos_catalog.h',
+}
+
+TEMPORAL_TYPES = (
+    'Temporal', 'TInstant', 'TSequence', 'TSequenceSet',
+    'TInt', 'TFloat', 'TBool', 'TText',
+    'TGeo', 'TGeoPoint', 'TGeompoint', 'TGeogpoint',
+    'TGeometry', 'TGeography',
+    'TCbuffer', 'TNpoint', 'TPose', 'TRGeometry',
+)
+
+
+def is_temporal(t):
+    s = t.get('canonical', t.get('c', ''))
+    return any(re.search(rf'\b{tt}\b', s) for tt in TEMPORAL_TYPES)
+
+
+def is_sequence_typed(t):
+    s = t.get('canonical', t.get('c', ''))
+    return 'TSequence' in s or 'SeqSet' in s
+
+
+def classify(fn, fn_role):
+    name = fn['name']
+    header = fn['file']
+    if header not in PUBLIC_HEADERS:
+        return ('internal', 'high', f'header {header} not public')
+
+    role = fn_role.get(name)
+    ret = fn['returnType']
+    params = fn['params']
+    n_temporal_params = sum(1 for p in params if is_temporal(p))
+    ret_is_temporal = is_temporal(ret)
+    ret_is_seq = is_sequence_typed(ret)
+
+    # --- IO / parsing / serialization ---
+    if role == 'output' or re.search(
+        r'_(in|out|recv|send|as_text|as_hexwkb|as_wkb|as_mfjson|as_ewkt|as_geojson|from_(wkb|hexwkb|text|mfjson|ewkt|geojson|wkt))$', name):
+        return ('io-meta', 'high', 'IO/serialization')
+    if 'mfjson' in name or 'wkb' in name or 'ewkt' in name or '_to_string' in name:
+        return ('io-meta', 'high', 'name has IO token')
+    # MEOS lifecycle / errno / array container helpers / rtree / ensure_
+    if re.match(r'^(meos_|ensure_|rtree_|skiplist_|meostype_|temptype_)', name):
+        return ('io-meta', 'high', 'MEOS infra / catalog / utility')
+
+    # --- Type conversions ---
+    if role == 'conversion':
+        return ('stateless', 'high', 'role=conversion')
+    if re.search(r'_to_(timestamptz|date|float|int|bigint|text|bool|tstzspan|span|stbox|tbox|tgeompoint|tgeogpoint|geo|geom|geog|set)$', name):
+        return ('stateless', 'high', 'name pattern is _to_<type>')
+
+    # --- Accessor ---
+    if role == 'accessor':
+        return ('bounded-state', 'high', 'role=accessor')
+
+    # --- Restriction (at/minus/delete) ---
+    if role == 'restriction':
+        return ('bounded-state', 'high', 'role=restriction')
+    if re.match(r'^(minus|at|delete)_', name) or '_at_' in name or '_minus_' in name or '_delete_' in name:
+        return ('bounded-state', 'high', 'restriction name pattern')
+
+    # --- Aggregate (windowed by definition) ---
+    if role == 'aggregate':
+        return ('windowed', 'high', 'role=aggregate')
+
+    # --- Ever / Always (e* / a* AND ever_/always_) ---
+    if name.startswith('ever_') or name.startswith('always_'):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'ever/always over 2 temporals')
+        return ('windowed', 'high', 'ever/always over 1 temporal')
+    if re.match(r'^[ea](intersects|contains|disjoint|touches|within|covers|crosses|overlaps|dwithin)_', name):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'ever/always spatial-rel on 2 temporals')
+        return ('bounded-state', 'high', 'ever/always spatial-rel on 1 temporal')
+
+    # --- Predicate (other) ---
+    if role == 'predicate':
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'predicate on 2 temporals')
+        return ('bounded-state', 'high', 'predicate on 1 temporal')
+
+    # --- Constructor ---
+    if role == 'constructor':
+        if ret_is_seq:
+            return ('sequence-only', 'high', 'constructor of TSequence/TSeqSet')
+        return ('stateless', 'high', 'constructor of instant/scalar')
+
+    # === Non-OO public functions, name-pattern fallbacks ===
+
+    # Set / span / spanset / tbox / stbox algebra (union, intersection,
+    # contains, overlaps, adjacent, position) - pure box/set algebra
+    if re.match(
+        r'^(union|intersection|difference|minus|contains|contained|overlaps|adjacent|'
+        r'left|right|overleft|overright|below|above|overbelow|overabove|front|back|'
+        r'overfront|overback|before|after|overbefore|overafter)_'
+        r'(set|span|spanset|tbox|stbox|date|timestamptz|int|bigint|float|text|bool|geo|geom|geog)',
+        name):
+        return ('stateless', 'high', 'set/span/box algebra (pure)')
+
+    # Numeric / text / bool ops on temporals (per-instant)
+    if re.match(
+        r'^t(int|float|number|text|bool)_(add|sub|mul|div|mod|abs|sqrt|round|ceil|floor|sign|neg|'
+        r'pow|exp|ln|log10|sin|cos|tan|degrees|radians|upper|lower|concat|and|or|not|xor|reverse|length)$',
+        name):
+        return ('stateless', 'high', 'temporal numeric/text/bool op')
+
+    # Temporal compare (teq, tne, tlt, tle, tgt, tge) - per-instant
+    if re.match(r'^t(eq|ne|lt|le|gt|ge)_', name):
+        return ('stateless', 'high', 'temporal comparison (per-instant)')
+
+    # Per-value comparison / hash on base / set / span / tbox
+    if re.match(
+        r'^(temporal|tfloat|tint|tbool|ttext|set|span|spanset|tbox|stbox|cbuffer|npoint|pose|'
+        r'date|geo|geom|geog|float|int|bigint|text|bool|timestamptz)_(eq|ne|cmp|lt|le|gt|ge|hash)',
+        name):
+        return ('stateless', 'high', 'scalar comparison/hash')
+
+    # Sequence-derived metrics (windowed)
+    if re.search(
+        r'_(length|cumulative_length|speed|integral|derivative|twavg|twmin|twmax|twsum|stops|'
+        r'min_dist|max_dist|nearest_approach|nearestapproach|granger)', name):
+        return ('windowed', 'high', 'sequence-derived metric')
+
+    # Trajectory / timespan / valueset / *_n / num_* — derived from full sequence
+    if re.search(r'_(trajectory|timespan|periods|periodset|valueset|num_(instants|sequences|values|timestamps))$', name):
+        return ('windowed', 'medium', 'derived from full sequence')
+
+    # Distance / NAD / NAI / shortestline
+    if re.match(r'^(nad|nai|shortestline|tdistance|distance)_', name):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'distance on 2 temporals')
+        return ('bounded-state', 'high', 'distance op')
+
+    # Constructor-like make / from_base / from_mfjson on instants
+    if re.search(r'(_make|_from_base|_make_array)$', name) or re.match(r'^geo_make', name) or re.match(r'^geom_make', name):
+        if ret_is_seq:
+            return ('sequence-only', 'medium', 'make-of-sequence')
+        return ('stateless', 'medium', 'make/from_base of instant/scalar')
+
+    # Shift / scale / round / fix / canonicalize / expand
+    if re.search(r'_(shift|scale|shift_scale|round|fix|canonicalize|expand|expandspace|expandtime|setbbox|setprecision)$', name) or \
+       re.search(r'_(shift|scale|shift_scale|round|fix|canonicalize|expand|expandspace|expandtime)_', name):
+        return ('stateless', 'high', 'transform/normalize (pure)')
+
+    # Geometry transformers (geom_to_*, geog_to_*, geo_*)
+    if re.match(r'^(geo|geom|geog|pose|cbuffer|npoint|stbox|tbox|set|span|spanset|float|int|bigint|text|bool|date|timestamptz)_', name):
+        # Default for base-type funcs without a more specific rule
+        return ('stateless', 'medium', 'base-type fn, default pure')
+
+    # Utility: copy / free / hash / compress / decompress
+    if re.search(r'_(copy|free|hash|compress|decompress|fix|valid|set_|setbbox)$', name):
+        return ('stateless', 'medium', 'utility')
+
+    # Topology "same_" predicate on box/temporal
+    if re.match(r'^same_', name):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'same predicate on 2 temporals')
+        return ('stateless', 'high', 'same predicate (pure box equality)')
+
+    # Temporal spatial relation lifted (returns a tbool)
+    if re.match(r'^t(intersects|contains|covers|disjoint|dwithin|touches|crosses|overlaps|within|equals)_', name):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'temporal spatial-rel lift on 2 temporals')
+        return ('bounded-state', 'high', 'temporal spatial-rel lift on 1 temporal')
+
+    # textcat / text-concat operator on temporals
+    if re.match(r'^textcat_', name):
+        return ('stateless', 'high', 'text concatenation (per-instant)')
+
+    # Temporal arithmetic operators (different name shapes: add_<X>_<Y>, sub_*, mul_*, div_*)
+    if re.match(r'^(add|sub|mul|div|mod)_(t(int|float|number)|int|float)', name):
+        return ('stateless', 'high', 'arithmetic op on temporal number (per-instant)')
+
+    # Position / topology relations — type-suffix-agnostic, signature-driven.
+    # Catches adjacent_numspan_tnumber, contained_tnumber_tbox, left_X_Y, etc.
+    if re.match(r'^(adjacent|contained|contains|overlaps|same|left|right|overleft|overright|'
+                r'before|after|overbefore|overafter|below|above|overbelow|overabove|'
+                r'front|back|overfront|overback)_', name):
+        if n_temporal_params >= 2:
+            return ('cross-stream', 'high', 'topology/position rel on 2 temporals')
+        if n_temporal_params == 1:
+            return ('bounded-state', 'high', 'topology/position rel on 1 temporal + scalar')
+        return ('stateless', 'high', 'topology/position rel on 2 scalars (box/span algebra)')
+
+    # Scalar arithmetic (add_date_int, add_timestamptz_interval, sub_*, mul_*)
+    if re.match(r'^(add|sub|mul|div|mod)_(date|timestamptz|int|bigint|float|interval)', name):
+        return ('stateless', 'high', 'scalar arithmetic')
+
+    # int32_cmp / int64_cmp / other scalar comparators with width suffix
+    if re.match(r'^(int32|int64|uint32|uint64|float32|float64)_(eq|ne|cmp|lt|le|gt|ge|hash)', name):
+        return ('stateless', 'high', 'scalar comparison with width suffix')
+
+    # cstring2text / text2cstring / pg_* casts / cstring_to_text
+    if re.match(r'^(cstring|text)2(cstring|text)|^cstring_to_|^text_to_cstring|^pg_', name):
+        return ('io-meta', 'high', 'string/CString conversion (low-level)')
+
+    # Last-resort: anything that returns a Datum / void / int errno-style without a temporal param = io-meta
+    # (numeric utility wrappers, MEOS internals exposed for codegen, error handling)
+    if n_temporal_params == 0 and not ret_is_temporal:
+        rc = ret.get('canonical', '') or ret.get('c', '')
+        if rc in ('int', 'bool', 'void', 'char *', 'const char *', 'Datum'):
+            return ('stateless', 'low', 'scalar in/out (default-stateless catch-all)')
+
+    return ('ambiguous', 'low', f'no rule (role={role}, hdr={header}, ntemp={n_temporal_params})')
+
+
+def build(cat, source_ref):
+    fn_role, fn_class = {}, {}
+    for cls_name, cls in cat['objectModel']['classes'].items():
+        for m in cls.get('methods', []):
+            fn = m['function']
+            fn_role[fn] = m.get('role')
+            fn_class.setdefault(fn, []).append(cls_name)
+
+    results = []
+    for fn in cat['functions']:
+        tier, conf, reason = classify(fn, fn_role)
+        results.append({
+            'name': fn['name'], 'file': fn['file'],
+            'role': fn_role.get(fn['name']),
+            'class': fn_class.get(fn['name'], []),
+            'tier': tier, 'confidence': conf, 'reason': reason,
+            'return_c': fn['returnType'].get('canonical', fn['returnType'].get('c')),
+            'n_params': len(fn['params']),
+        })
+
+    total = len(results)
+    public_rows = [r for r in results if r['tier'] != 'internal']
+    tot = Counter(r['tier'] for r in results)
+    pub_tot = Counter(r['tier'] for r in public_rows)
+    by_h = defaultdict(Counter)
+    for r in public_rows:
+        by_h[r['file']][r['tier']] += 1
+    streaming_relevant = sum(pub_tot[t] for t in ['stateless', 'bounded-state', 'windowed', 'cross-stream'])
+
+    return {
+        'schema_version': '0.1.0-draft',
+        'source_catalog': source_ref,
+        'source_catalog_function_count': total,
+        'classifier': 'v4',
+        'vocabulary': TIERS,
+        'public_headers': sorted(PUBLIC_HEADERS),
+        'rollup': {
+            'total': dict(tot),
+            'public_only': dict(pub_tot),
+            'public_by_header': {h: dict(c) for h, c in by_h.items()},
+            'headline_public': {
+                'streaming_relevant': streaming_relevant,
+                'sequence_only': pub_tot['sequence-only'],
+                'io_meta': pub_tot['io-meta'],
+                'ambiguous': pub_tot['ambiguous'],
+                'public_total': len(public_rows),
+            },
+        },
+        'functions': results,
+    }
+
+
+def main():
+    ap = argparse.ArgumentParser(description=__doc__,
+                                 formatter_class=argparse.RawDescriptionHelpFormatter)
+    ap.add_argument('catalog', help='meos-idl.json produced by MEOS-API/run.py')
+    ap.add_argument('-o', '--out', default='-',
+                    help='output baseline path (default: stdout)')
+    ap.add_argument('--source-ref', default=None,
+                    help='portable provenance recorded in the artifact '
+                         '(default: the catalog basename)')
+    a = ap.parse_args()
+
+    with open(a.catalog) as f:
+        cat = json.load(f)
+    artifact = build(cat, a.source_ref or basename(a.catalog))
+
+    text = json.dumps(artifact, indent=2) + '\n'
+    if a.out == '-':
+        sys.stdout.write(text)
+    else:
+        with open(a.out, 'w') as f:
+            f.write(text)
+        pub = artifact['rollup']['headline_public']
+        sys.stderr.write(
+            f"wrote {a.out}: {artifact['source_catalog_function_count']} fns, "
+            f"{pub['streaming_relevant']} streaming-relevant, "
+            f"{pub['ambiguous']} ambiguous\n")
+
+
+if __name__ == '__main__':
+    main()
diff --git a/tools/regen_baseline.sh b/tools/regen_baseline.sh
new file mode 100755
index 0000000..2f87c4b
--- /dev/null
+++ b/tools/regen_baseline.sh
@@ -0,0 +1,30 @@
+#!/usr/bin/env bash
+# Regenerate tools/baseline/streaming-relevance-baseline.json — the generator
+# input consumed by codegen_facades.py.
+#
+# The baseline is a deterministic classification of the MEOS public API into
+# streaming tiers. It is derived from the MEOS-API catalog (meos-idl.json),
+# which is itself produced by running MEOS-API/run.py over the MEOS headers at
+# the commit recorded in tools/meos-source-commit.txt. See GENERATION.md.
+#
+# Usage:
+#   tools/regen_baseline.sh <meos-idl.json> [source-ref]
+#
+#   <meos-idl.json>  the MEOS-API catalog (from MEOS-API/run.py over the MEOS
+#                    headers at tools/meos-source-commit.txt)
+#   [source-ref]     optional portable provenance label recorded in the artifact
+#
+# The classifier is deterministic: the same catalog always yields the same
+# baseline, so a refresh is: rebuild the catalog at the tracked commit, re-run
+# this script, commit the diff.
+set -euo pipefail
+
+here="$(cd "$(dirname "$0")" && pwd)"
+catalog="${1:?usage: regen_baseline.sh <meos-idl.json> [source-ref]}"
+source_ref="${2:-meos-idl.json (MEOS-API run.py over tools/meos-source-commit.txt)}"
+out="$here/baseline/streaming-relevance-baseline.json"
+
+python3 "$here/classify_streaming_relevance.py" "$catalog" \
+  --source-ref "$source_ref" -o "$out"
+
+echo "regenerated $out"