An already-built pair set cannot be repaired by build_sft_pairs.py --source-entities;
that flag only works at generation time. Hemingway's and Yarros's sets both predate it.
The contamination is in the BEAT, so dropping the row removes it outright. Measured on
the Hemingway train pairs: 7,094 -> 7,024, 70 dropped, 0.99% of the training data. That
is cheaper and cleaner than regenerating 70 beats against a second generator session,
which would leave the set mixed-provenance for the sake of 1% more data.
Verified by read-back rather than by the write succeeding: re-auditing the filtered file
reports 0 of 7,024 on both columns, controls green, GATE PASS.
Two refusals rather than a best-effort write:
- a contaminated RESPONSE column aborts. That is a different fault -- pairs built
against an unrenamed corpus -- and dropping rows would hide it instead of fixing it.
- more than one --pairs input aborts, because the output is a single file and would
silently merge train and val into one.
Also cross-validated the detector against the lv-bronte pair sets on real data, where the
answer is already on the record:
pairs-full + pairs-val (post-fix) 0 of 3,858 matches the recorded "0 leaks across
3,858 pairs" exactly
pairs-full.CONTAMINATED 15 of 792 = 1.89%, Rochester x6, Jane, Brocklehurst
x2, Beck, Fairfax, Burns, Helen, Eyre -- against a
record of "13 of the first 714 beats (1.8%)" with
the same names
An independently written instrument reproducing a documented finding at the right
magnitude, on the right names, is the control that says its zeroes mean absent and not
blind.
211 lines
11 KiB
Python
211 lines
11 KiB
Python
"""Do the GENERATED BEATS name characters the rename removed?
|
||
|
||
`leak_gate.py` reads the corpus and the renamed copies. It never reads the pairs,
|
||
so it is structurally blind to the leak found on lv-bronte (2026-09-16): the beat
|
||
is written by an LLM that read the passage, and if it recognises the book it
|
||
supplies the canonical names from its own memory. The rename can be perfect and
|
||
the instruction half of every pair still carry `Rochester`.
|
||
|
||
`build_sft_pairs.py --source-entities` closes that at BUILD time. This closes it
|
||
for pair sets already built — Hemingway's and Yarros's both predate the flag, and
|
||
a clean corpus gate is not evidence about them either way.
|
||
|
||
WHAT COUNTS AS A LEAK, and why the distinction matters. A source surface the beat
|
||
names is only a leak if the rename actually took it away — so every matched surface
|
||
is classified against the renamed copies first, and only the removed ones count
|
||
against the gate. Real-world names the pipeline deliberately keeps (`Paris` 173
|
||
occurrences, `Madrid` 108, `Spain` 87, all still present in the renamed copies) are
|
||
held back by the stoplist BEFORE the entity map is built, so on Hemingway the map
|
||
turns out to be exactly the removed set: 941 surfaces, 941 removed, 0 kept. Do not
|
||
assume that holds on another corpus — the classification is measured per run, and a
|
||
pipeline that instead carries kept surfaces INTO the map would report every mention
|
||
of `Paris` as a leak if this step were skipped.
|
||
|
||
THE RESPONSE SIDE IS THE DIAGNOSTIC. Beats and responses are scanned separately.
|
||
Leaks in the beats with a clean response column is the lv-bronte signature: the
|
||
rename worked and the generator undid it on the instruction side. Hits in BOTH
|
||
columns mean something upstream is wrong — the pairs were built against an
|
||
unrenamed corpus — and that is a different, larger problem.
|
||
|
||
CONTROLS, every run, because a scanner that only ever sees beats cannot tell
|
||
`absent` from `blind`:
|
||
* POSITIVE -- the same pattern over the UNRENAMED source works. Every surface
|
||
must be found there, or the zeroes downstream are worthless.
|
||
* NEGATIVE -- a nonce that appears in no tree. A hit means manufactured signal.
|
||
|
||
Exit code is the gate: 0 iff the controls pass AND no beat names a removed surface.
|
||
"""
|
||
from __future__ import annotations
|
||
import argparse, json, re, sys
|
||
from collections import Counter
|
||
from pathlib import Path
|
||
|
||
sys.path.insert(0, str(Path(__file__).resolve().parent))
|
||
from leak_gate import NONCE, load_works, load_copies, scan # noqa: E402
|
||
|
||
|
||
def load_pairs(paths: list[Path]) -> list[dict]:
|
||
rows = []
|
||
for p in paths:
|
||
for line in p.read_text(encoding="utf-8").splitlines():
|
||
if line.strip():
|
||
r = json.loads(line)
|
||
r["_src"] = p.name
|
||
rows.append(r)
|
||
return rows
|
||
|
||
|
||
def main() -> int:
|
||
ap = argparse.ArgumentParser()
|
||
ap.add_argument("--pairs", required=True, nargs="+", help="pairs jsonl (train and/or val)")
|
||
ap.add_argument("--entities", required=True, help="entities json for the UNRENAMED source")
|
||
ap.add_argument("--corpus", required=True, help="source corpus dir (manifest.json + works/)")
|
||
ap.add_argument("--renamed", required=True, help="rename.py --out dir, to classify kept vs removed")
|
||
ap.add_argument("--min-cap", type=int, default=8,
|
||
help="rename.py's renameable threshold; mirrored from leak_gate.py")
|
||
ap.add_argument("--report", default=None, help="write the full JSON breakdown here")
|
||
ap.add_argument("--show", type=int, default=25, help="example beats to print")
|
||
ap.add_argument("--filter-out", default=None, metavar="PATH",
|
||
help="write a copy of the pairs with every contaminated ROW removed. "
|
||
"Turns this detector into the fix for an already-built pair set: "
|
||
"the contamination is in the beat, so dropping the row removes it "
|
||
"outright, and on Hemingway that costs 0.96%% of the training data. "
|
||
"Cheaper and cleaner than regenerating 70 beats against a second "
|
||
"generator session, which would leave the set mixed-provenance. "
|
||
"Refuses to write when a RESPONSE is contaminated -- that is a "
|
||
"different fault (pairs built against an unrenamed corpus) and "
|
||
"dropping rows would hide it rather than fix it.")
|
||
a = ap.parse_args()
|
||
|
||
ents_all = json.loads(Path(a.entities).read_text())
|
||
# Mirror build_sft_pairs.py's --source-entities surface set exactly, so this
|
||
# audit answers "would that flag have rejected it", not a near-miss variant.
|
||
surfaces = sorted({(e.get("surface") or key)
|
||
for w in ents_all.values() for key, e in w["entities"].items()
|
||
if "’" not in key and "'" not in key})
|
||
if not surfaces:
|
||
print("== entity map yields no surfaces"); return 1
|
||
|
||
source = load_works(Path(a.corpus))
|
||
copies = load_copies(Path(a.renamed))
|
||
if not copies:
|
||
print("== no renamed copies found -- cannot tell a removed name from a kept one"); return 1
|
||
|
||
# ---- controls --------------------------------------------------------
|
||
src_hits = scan(source, surfaces + [NONCE])
|
||
missing = [s for s in surfaces if s not in src_hits]
|
||
pos_ok = not missing
|
||
neg_ok = NONCE not in src_hits
|
||
print(f" {len(surfaces)} source surfaces · {len(source)} works · {len(copies)} renamed copies")
|
||
print(f" [{'PASS' if pos_ok else 'FAIL'}] positive control: every surface found in the "
|
||
f"unrenamed source ({len(surfaces) - len(missing)}/{len(surfaces)})"
|
||
+ ("" if pos_ok else f" -- MISSING {missing[:10]}"))
|
||
|
||
# ---- which surfaces did the rename actually remove? ------------------
|
||
copy_hits = scan(copies, surfaces + [NONCE])
|
||
neg_ok = neg_ok and NONCE not in copy_hits
|
||
print(f" [{'PASS' if neg_ok else 'FAIL'}] negative control: nonce `{NONCE}` absent from both trees")
|
||
kept = {s for s in surfaces if s in copy_hits}
|
||
removed = [s for s in surfaces if s not in copy_hits]
|
||
print(f" rename KEPT {len(kept)} surfaces (real places, allow-listed, sub-threshold) · "
|
||
f"REMOVED {len(removed)}")
|
||
if not removed:
|
||
print("== the rename removed nothing -- this audit has no leak to look for"); return 1
|
||
|
||
# ---- the measurement -------------------------------------------------
|
||
rows = load_pairs([Path(p) for p in a.pairs])
|
||
print(f" {len(rows)} pairs from {len({r['_src'] for r in rows})} file(s)")
|
||
removed_pat = re.compile(r"\b(" + "|".join(re.escape(s) for s in
|
||
sorted(removed, key=len, reverse=True)) + r")\b")
|
||
|
||
cols = {"beat": Counter(), "response": Counter()}
|
||
hit_rows: dict[str, list] = {"beat": [], "response": []}
|
||
for r in rows:
|
||
for col in cols:
|
||
text = r.get(col) or ""
|
||
found = sorted(set(removed_pat.findall(text)))
|
||
if found:
|
||
cols[col].update(found)
|
||
hit_rows[col].append({"src": r["_src"], "work": r.get("work"),
|
||
"names": found, "text": text})
|
||
|
||
print()
|
||
for col in ("beat", "response"):
|
||
n = len(hit_rows[col])
|
||
print(f" {col.upper():<9} naming a REMOVED surface: {n} of {len(rows)} "
|
||
f"({n / len(rows):.2%}) · {len(cols[col])} distinct names")
|
||
for s, c in cols[col].most_common(15):
|
||
print(f" {s:<20} x{c}")
|
||
|
||
# The lv-bronte signature, stated rather than left to be inferred.
|
||
nb, nr = len(hit_rows["beat"]), len(hit_rows["response"])
|
||
print()
|
||
if nb and not nr:
|
||
print(" ⭐ BEAT-ONLY leak — the rename held and the beat generator undid it on the "
|
||
"instruction side. Regenerate the pairs with --source-entities.")
|
||
elif nb and nr:
|
||
print(" ⚠⚠ BOTH columns leak — this is NOT the beat-generator class. The pairs were "
|
||
"probably built against an unrenamed corpus; check the provenance `corpus` path.")
|
||
elif nr:
|
||
print(" ⚠⚠ RESPONSE-only leak — the response is copied from the corpus, so a hit here "
|
||
"means the renamed copies are not what the pairs were built from.")
|
||
|
||
for col in ("beat", "response"):
|
||
for h in hit_rows[col][:a.show]:
|
||
print(f"\n [{col}] {h['src']} · {h['work']} · {h['names']}")
|
||
print(f" {h['text'][:300]}")
|
||
|
||
if a.report:
|
||
Path(a.report).write_text(json.dumps({
|
||
"pairs": [str(p) for p in a.pairs],
|
||
"surfaces_total": len(surfaces), "kept": len(kept), "removed": len(removed),
|
||
"controls": {"positive_pass": pos_ok, "negative_pass": neg_ok, "missing": missing[:50]},
|
||
"rows": len(rows),
|
||
"beat_hits": nb, "beat_names": dict(cols["beat"]),
|
||
"response_hits": nr, "response_names": dict(cols["response"]),
|
||
"examples": {c: hit_rows[c][:50] for c in hit_rows},
|
||
}, ensure_ascii=False, indent=2), encoding="utf-8")
|
||
print(f"\n wrote {a.report}")
|
||
|
||
if a.filter_out:
|
||
if nr:
|
||
print("\n== REFUSING to write a filtered copy: the RESPONSE column is contaminated.")
|
||
print(" That is not the beat-generator fault and dropping rows would hide it.")
|
||
print(" Check the provenance `corpus` path -- the pairs were probably built")
|
||
print(" against an unrenamed corpus, and the fix is upstream of this file.")
|
||
return 2
|
||
if not (pos_ok and neg_ok):
|
||
print("\n== REFUSING to write a filtered copy: the controls did not pass, so the")
|
||
print(" set of rows to drop is not trustworthy.")
|
||
return 2
|
||
if len({r["_src"] for r in rows}) != 1:
|
||
print("\n== REFUSING to write a filtered copy from more than one --pairs file:")
|
||
print(" the output is a single file and would silently merge train and val.")
|
||
print(" Filter each input separately.")
|
||
return 2
|
||
drop = set()
|
||
for i, r in enumerate(rows):
|
||
if removed_pat.search(r.get("beat") or ""):
|
||
drop.add(i)
|
||
out = Path(a.filter_out)
|
||
kept = 0
|
||
with out.open("w", encoding="utf-8") as fh:
|
||
for i, r in enumerate(rows):
|
||
if i in drop:
|
||
continue
|
||
r = {k: v for k, v in r.items() if k != "_src"}
|
||
fh.write(json.dumps(r, ensure_ascii=False) + "\n")
|
||
kept += 1
|
||
print(f"\n wrote {out}: {kept} pairs kept, {len(drop)} dropped "
|
||
f"({len(drop) / len(rows):.2%})")
|
||
print(" ⚠ Re-run this audit against the filtered file before training on it. A fix")
|
||
print(" that is not read back is a claim, not a result.")
|
||
|
||
ok = pos_ok and neg_ok and nb == 0 and nr == 0
|
||
print(f"\n GATE: {'PASS' if ok else 'FAIL'}")
|
||
return 0 if ok else 2
|
||
|
||
|
||
if __name__ == "__main__":
|
||
raise SystemExit(main())
|