@mailwoman/corpus 9.0.0 → 9.2.0

package/src/shard-recipes/street-affix.ts

@@ -27,8 +27,6 @@
  *   FR/DE postcode (the v0.9.8 blemish).
  */
 
-import { spawnSync } from "node:child_process"
-
 import {
 	DirectionalAbbreviation,
 	lookupDirectional,
@@ -41,10 +39,12 @@ import {
 import type { ComponentTag } from "@mailwoman/core/types"
 import { dataRootPath } from "@mailwoman/core/utils"
 
-import { stableSourceID } from "../adapter.ts"
-import { alignRow } from "../align.ts"
-import type { CanonicalRow } from "../types.ts"
-import { makeMulberry32, readCSVRecords, type ShardRecipe } from "./scaffold.ts"
+import { stableSourceID } from "#adapters/utils"
+import type { CanonicalRow } from "#types"
+import { alignRow } from "#utils"
+
+import { NAME_PRONE_US_SUFFIXES } from "../name-prone-us-suffixes.ts"
+import { makeMulberry32, readCSVRecords, readZippedCSVRecords, shardSourceID, type ShardRecipe } from "./scaffold.ts"
 
 // Same OA cache as the unit shard. Train = every NON-Vermont state; eval = Vermont (the holdout).
 /* oxlint-disable sister-software/no-unnamed-threshold -- the bare decimals below are weighted-sampler
@@ -140,6 +140,7 @@ interface USTuple {
 	locality: string
 	region: string
 	postcode: string
+	base_source_id: string
 }
 
 /**
@@ -163,19 +164,11 @@ type Prefix = Pick<NonNullable<ReturnType<typeof matchLeadingDirectional>>, "can
 /**
  * Stream real US tuples (number/street/city/postcode) out of a cached OA zip.
  */
-function readTuples(source: USSource): USTuple[] {
-	const r = spawnSync("unzip", ["-p", source.zip, source.csv], { maxBuffer: 1024 * 1024 * 1024, encoding: "buffer" })
-
-	if (r.status !== 0) {
-		console.error(`  WARN: unzip failed for ${source.zip} (status ${r.status})`)
-
-		return []
-	}
-
+async function readTuples(source: USSource): Promise<USTuple[]> {
 	const tuples: USTuple[] = []
 	const seen = new Set<string>()
 
-	for (const row of readCSVRecords(r.stdout)) {
+	for await (const row of readZippedCSVRecords(source.zip, source.csv)) {
 		const street = row.street ?? ""
 		const locality = row.city ?? ""
 		const house_number = row.number ?? ""
@@ -185,7 +178,15 @@ function readTuples(source: USSource): USTuple[] {
 
 		if (seen.has(key)) continue
 		seen.add(key)
-		tuples.push({ house_number, street, locality, region: source.region, postcode: row.postcode ?? "" })
+
+		tuples.push({
+			house_number,
+			street,
+			locality,
+			region: source.region,
+			postcode: row.postcode ?? "",
+			base_source_id: `openaddresses:${source.csv}:${key}`,
+		})
 	}
 
 	return tuples
@@ -205,7 +206,10 @@ const isSuffixOrDirectional = (word: string): boolean =>
  * Split an OA street into { prefix?, name, suffix } using the codex. Requires a trailing suffix and a non-empty name
  * that isn't itself an affix token. Returns null when the street has no usable suffix.
  */
-function parseStreet(street: string): { prefix: Prefix | null; name: string; suffix: string } | null {
+function parseStreet(
+	street: string,
+	opts: { allowNameProneTail?: boolean } = {}
+): { prefix: Prefix | null; name: string; suffix: string } | null {
 	let words = street.trim().split(/\s+/)
 
 	if (words.length < 2) return null
@@ -225,11 +229,42 @@ function parseStreet(street: string): { prefix: Prefix | null; name: string; suf
 	const suffix = trail.canonical
 	const name = words.slice(0, -1).join(" ")
 
-	if (!name || isSuffixOrDirectional(name)) return null
+	if (!name) return null
+
+	if (isSuffixOrDirectional(name)) {
+		const nameTail = matchTrailingSuffix(name)
+
+		if (!opts.allowNameProneTail || !nameTail || !NAME_PRONE_US_SUFFIXES.has(nameTail.canonical)) return null
+	}
 
 	return { prefix, name, suffix }
 }
 
+export type SuffixBoundaryClass = "terminal-only" | "terminal-contrast"
+
+/**
+ * Classify a real street surface for #1569. `terminal-only` has an ambiguous suffix-eligible name word immediately
+ * before a different terminal type (`Blue Hill Rd`); `terminal-contrast` ends at the ambiguous word itself (`Sutton
+ * Hollow`). The contrast check intentionally wins when both final words are name-prone: only the LAST token is the
+ * suffix under the canonical rule.
+ */
+export function classifySuffixBoundaryStreet(street: string): SuffixBoundaryClass | null {
+	const words = street.trim().split(/\s+/)
+
+	if (words.length < 2) return null
+	const terminal = matchTrailingSuffix(words.at(-1)!)
+
+	if (!terminal) return null
+
+	if (NAME_PRONE_US_SUFFIXES.has(terminal.canonical)) return "terminal-contrast"
+
+	if (words.length < 3) return null
+
+	const penultimate = matchTrailingSuffix(words.at(-2)!)
+
+	return penultimate && NAME_PRONE_US_SUFFIXES.has(penultimate.canonical) ? "terminal-only" : null
+}
+
 /**
  * Render the affix-split street in random surface forms (abbrev vs expanded per affix), Title-cased.
  */
@@ -279,18 +314,65 @@ function renderStreet(
  */
 const VENUES = ["John Doe", "Jane Smith", "Acme Inc", "Wayne Enterprises", "Maria Garcia", "Riverside Clinic"]
 
+/**
+ * Real-venue pool for the suffix-boundary v2 venue shell (corpus 0.19.0). The 2026-08-10 recipe review found the
+ * v0.18.x shard's venue-led rows detectable as templates (six fixed venue strings), and the frozen B1 board's failures
+ * concentrate in exactly that shell (rich rows 5/43 vs bare 53/65 at v4.3.3 step-40k). HRSA's health-center site file
+ * supplies thousands of REAL US facility names ('Alburg Health Center'-register), US-government public domain, already
+ * a corpus source (`usgov-hrsa-fqhc`). Kept verbatim (including the ~11% all-caps names — real register diversity);
+ * comma-carrying names are dropped because the venue layout uses commas as its field delimiter.
+ */
+const VENUE_POOL_CSV = dataRootPath(
+	"corpus",
+	"sources",
+	"usgov-hrsa-fqhc",
+	"Health_Center_Service_Delivery_and_LookAlike_Sites.csv"
+)
+
+async function readVenuePool(csvPath: string): Promise<string[]> {
+	const seen = new Set<string>()
+	const pool: string[] = []
+
+	for await (const record of readCSVRecords(csvPath)) {
+		const name = (record["site name"] ?? record["Site Name"] ?? "").trim().replaceAll(/\s+/gu, " ")
+
+		if (name.length < 3 || name.length > 60 || name.includes(",") || !/\p{L}/u.test(name)) continue
+		const key = name.toLowerCase()
+
+		if (seen.has(key)) continue
+		seen.add(key)
+		pool.push(name)
+	}
+
+	return pool
+}
+
 const tail = (loc: string, reg: string, pc: string): string => (pc ? `${loc}, ${reg} ${pc}` : `${loc}, ${reg}`)
 
+/**
+ * Layout-shell options for {@link renderRow}. `cuts` are the cumulative random() cutoffs for [full, bare, street-only] —
+ * the remainder is the venue shell. Defaults reproduce the original street-affix distribution (40/25/20/15) with the
+ * six template venues.
+ */
+interface RenderRowOpts {
+	venues?: readonly string[]
+	cuts?: readonly [number, number, number]
+}
+
 /**
  * Embed the rendered street in a RANDOM layout so the model recognizes affixes wherever the street sits: full address,
  * bare house+street, street-only (pure affix parse), or venue-prefixed.
  */
-function renderRow(
+export function renderRow(
 	random: () => number,
 	base: USTuple,
 	street: string,
-	streetComponents: Partial<Record<ComponentTag, string>>
+	streetComponents: Partial<Record<ComponentTag, string>>,
+	opts: RenderRowOpts = {}
 ): { fmt: string; raw: string; components: Partial<Record<ComponentTag, string>> } {
+	const venues = opts.venues ?? VENUES
+	const [fullCut, bareCut, streetOnlyCut] = opts.cuts ?? [0.4, 0.65, 0.85]
+
 	const hn = base.house_number,
 		loc = base.locality,
 		reg = base.region,
@@ -300,17 +382,17 @@ function renderRow(
 	const withRoad: Partial<Record<ComponentTag, string>> = { house_number: hn, ...streetComponents }
 	const r = random()
 
-	if (r < 0.4)
+	if (r < fullCut)
 		return {
 			fmt: "full",
 			raw: `${road}, ${tail(loc, reg, pc)}`,
 			components: { ...withRoad, locality: loc, region: reg, ...(pc ? { postcode: pc } : {}) },
 		}
 
-	if (r < 0.65) return { fmt: "bare", raw: road, components: withRoad }
+	if (r < bareCut) return { fmt: "bare", raw: road, components: withRoad }
 
-	if (r < 0.85) return { fmt: "street-only", raw: street, components: { ...streetComponents } }
-	const v = VENUES[Math.floor(random() * VENUES.length)]!
+	if (r < streetOnlyCut) return { fmt: "street-only", raw: street, components: { ...streetComponents } }
+	const v = venues[Math.floor(random() * venues.length)]!
 
 	return {
 		fmt: "venue",
@@ -320,28 +402,15 @@ function renderRow(
 }
 
 /**
- * Capped reader for the multi-locale BALANCE sources. The FR/IT/NL countrywide extracts are GB-scale, so cap the bytes
- * with `head` (mirrors build-country-shard-balanced.mjs) rather than holding a whole extract in memory. Only keeps
- * tuples that carry a POSTCODE.
+ * Capped reader for the multi-locale BALANCE sources. The FR/IT/NL countrywide extracts are GB-scale, so this reads
+ * only as far as `limit` distinct tuples — the `break` closes the reader and releases the archive. Only keeps tuples
+ * that carry a POSTCODE.
  */
-function readBalanceTuples(source: BalanceSource, limit: number): BalanceTuple[] {
-	const maxLines = Math.max(limit * 8, 20_000) + 1
-
-	const r = spawnSync("bash", ["-c", `unzip -p "${source.zip}" "${source.csv}" | head -n ${maxLines}`], {
-		maxBuffer: 1024 * 1024 * 1024,
-		encoding: "buffer",
-	})
-
-	if (r.status !== 0) {
-		console.error(`  WARN: unzip failed for ${source.zip} (status ${r.status})`)
-
-		return []
-	}
-
+async function readBalanceTuples(source: BalanceSource, limit: number): Promise<BalanceTuple[]> {
 	const tuples: BalanceTuple[] = []
 	const seen = new Set<string>()
 
-	for (const row of readCSVRecords(r.stdout)) {
+	for await (const row of readZippedCSVRecords(source.zip, source.csv)) {
 		if (tuples.length >= limit) break
 
 		const street = row.street ?? "",
@@ -412,7 +481,7 @@ export const streetAffixRecipe: ShardRecipe = {
 		const pool: USTuple[] = []
 
 		for (const s of sources) {
-			const t = readTuples(s)
+			const t = await readTuples(s)
 
 			console.error(`  ${s.csv}: ${t.length} unique tuples`)
 
@@ -513,7 +582,7 @@ export const streetAffixRecipe: ShardRecipe = {
 			const mlPool: BalanceTuple[] = []
 
 			for (const s of mlSources) {
-				const t = readBalanceTuples(s, perSource)
+				const t = await readBalanceTuples(s, perSource)
 
 				console.error(`  balance ${s.csv} (${s.iso2}): ${t.length} tuples`)
 
@@ -585,3 +654,173 @@ export const streetAffixRecipe: ShardRecipe = {
 		return { emitted: emitted + balanceEmitted, skipped: skipped + balanceSkipped }
 	},
 }
+
+/**
+ * #1569 root-fix shard. Both classes come from real non-Vermont OA streets and use the affix recipe's existing layout
+ * diversity. v4.3.1 makes terminal-only 80% of the mix: the first 40/60 run moved a 100-row TRAIN sample only 4→11
+ * while contrast was already 95/100 before training (93/100 after). Post-run audit found that the global affix relabel
+ * pass corrupts many already-decomposed target rows into double suffixes; do not retrain this recipe until relabel is
+ * idempotent over a decomposed street family. The 20% contrast leg remains explicit, additive to the already-strong
+ * base distribution, and B2 still gates it unchanged.
+ *
+ * V2 (corpus 0.19.0, 2026-08-10 recipe review): the v4.3.3 board split (rich venue-led rows 5/43 vs bare 53/65) showed
+ * the model separating template rows from real ones, and the venue shell was the giveaway — six fixed venue strings. v2
+ * draws the venue shell from thousands of REAL HRSA facility names and raises its share (venue 30%, full 35%, bare 20%,
+ * street-only 15%). Dose policy moved to the recipe's config side: weight ≤4 effective passes per run (Muennighoff
+ * 2023's repetition knee) and the source is excluded from the augmentation pool — see the v4.4.0 config.
+ */
+export const suffixBoundaryRecipe: ShardRecipe = {
+	name: "suffix-boundary",
+	description: "US #1569 suffix boundary: OA terminal-only positives + over-sampled terminal contrasts",
+	mode: "generate",
+	async run(opts, write) {
+		const random = makeMulberry32(opts.seed)
+		const count = opts.count ?? 30_000
+		const source = opts.sourceName ?? "synth-suffix-boundary"
+		const sources = opts.golden ? [EVAL_SOURCE] : TRAIN_SOURCES
+
+		// v2 venue shell: real facility names, loud when the source file is missing or thin.
+		const venuePool = await readVenuePool(VENUE_POOL_CSV)
+
+		if (venuePool.length < 500) {
+			throw new Error(
+				`suffix-boundary v2 venue pool too thin: ${venuePool.length} names from ${VENUE_POOL_CSV} — ` +
+					"is the usgov-hrsa-fqhc source present?"
+			)
+		}
+
+		const shell: RenderRowOpts = { venues: venuePool, cuts: [0.35, 0.55, 0.7] }
+
+		const pool: Record<SuffixBoundaryClass, USTuple[]> = {
+			"terminal-only": [],
+			"terminal-contrast": [],
+		}
+
+		for (const s of sources) {
+			for (const tuple of await readTuples(s)) {
+				const rowClass = classifySuffixBoundaryStreet(tuple.street)
+
+				if (rowClass) {
+					pool[rowClass].push(tuple)
+				}
+			}
+		}
+
+		if (!pool["terminal-only"].length || !pool["terminal-contrast"].length) {
+			throw new Error(
+				`Suffix-boundary source class missing: terminal-only=${pool["terminal-only"].length}, ` +
+					`terminal-contrast=${pool["terminal-contrast"].length}`
+			)
+		}
+
+		let emitted = 0
+		let skipped = 0
+		const classCounts: Record<SuffixBoundaryClass, number> = { "terminal-only": 0, "terminal-contrast": 0 }
+		const terminalOnlyTarget = Math.round(count * 0.8)
+
+		const classTargets: Record<SuffixBoundaryClass, number> = {
+			"terminal-only": terminalOnlyTarget,
+			"terminal-contrast": count - terminalOnlyTarget,
+		}
+
+		const formatCounts: Record<string, number> = {}
+		let stemPairs = 0
+
+		const emitOne = (
+			rowClass: SuffixBoundaryClass,
+			base: USTuple,
+			parsed: NonNullable<ReturnType<typeof parseStreet>>,
+			method: string
+		): boolean => {
+			const { street, components: streetComponents } = renderStreet(random, parsed)
+			const { fmt, raw, components } = renderRow(random, base, street, streetComponents, shell)
+
+			const canonical: CanonicalRow = {
+				raw,
+				components,
+				country: "US",
+				locale: "en-US",
+				source,
+				source_id: shardSourceID(source, { ...components, class: rowClass, n: String(emitted) }),
+				corpus_version: "0.19.0",
+				license:
+					"OpenAddresses US (non-VT) skeletons; terminal suffix labels via USPS Pub-28 codex; venue shell from HRSA site names (US public domain)",
+			}
+
+			const aligned = alignRow(canonical)
+
+			if (aligned.kind !== "labeled" || !aligned.row) {
+				skipped++
+
+				return false
+			}
+
+			write(JSON.stringify({ ...aligned.row, synth_method: method, synth_base_id: base.base_source_id }) + "\n")
+
+			classCounts[rowClass]++
+			formatCounts[fmt] = (formatCounts[fmt] ?? 0) + 1
+
+			emitted++
+
+			return true
+		}
+
+		// oxlint-disable-next-line eslint/no-unmodified-loop-condition -- emitted/skipped advance inside emitOne (a closure); the rule cannot see through the call
+		while (emitted < count && emitted + skipped < count * 10) {
+			const terminalOnlyOpen = classCounts["terminal-only"] < classTargets["terminal-only"]
+			const terminalContrastOpen = classCounts["terminal-contrast"] < classTargets["terminal-contrast"]
+
+			const rowClass: SuffixBoundaryClass =
+				terminalOnlyOpen && terminalContrastOpen
+					? random() < 0.8
+						? "terminal-only"
+						: "terminal-contrast"
+					: terminalContrastOpen
+						? "terminal-contrast"
+						: "terminal-only"
+
+			const classPool = pool[rowClass]
+			const base = classPool[Math.floor(random() * classPool.length)]!
+			const parsed = parseStreet(base.street, { allowNameProneTail: rowClass === "terminal-only" })
+
+			if (!parsed) {
+				skipped++
+
+				continue
+			}
+
+			if (!emitOne(rowClass, base, parsed, `suffix-boundary:${rowClass}`)) continue
+
+			// Stem-pair hard negative (v2, 2026-08-10 recipe review): after a terminal-only row
+			// ('Menlo Park' + 'Road'), also emit the SAME stem without its true suffix as a
+			// contrast row ('Menlo' + 'Park' under the canonical last-token rule). Sharing the stem
+			// forces the model to key on the licensing evidence — the trailing true suffix — rather
+			// than on name identity. Dose-neutral: these fill the existing 20% contrast target.
+			if (
+				rowClass === "terminal-only" &&
+				classCounts["terminal-contrast"] < classTargets["terminal-contrast"] &&
+				emitted < count &&
+				random() < 0.5
+			) {
+				const stemParsed = parseStreet(parsed.name)
+
+				if (
+					stemParsed &&
+					emitOne("terminal-contrast", base, stemParsed, "suffix-boundary:terminal-contrast:stem-pair")
+				) {
+					stemPairs++
+				}
+			}
+		}
+
+		console.error(
+			`Done: emitted ${emitted}, skipped ${skipped}; source pools ${JSON.stringify({
+				terminalOnly: pool["terminal-only"].length,
+				terminalContrast: pool["terminal-contrast"].length,
+			})}; classes ${JSON.stringify(classCounts)}; stem pairs ${stemPairs}; ` +
+				`venue pool ${venuePool.length}; formats ${JSON.stringify(formatCounts)}`
+		)
+
+		return { emitted, skipped }
+	},
+}

package/src/shard-recipes/street-bare.ts

@@ -17,9 +17,10 @@
  *   reproduces the legacy run byte-for-byte.
  */
 
-import { alignRow } from "../align.ts"
-import { DEFAULT_US_BASES } from "../synthesize-intersection.ts"
-import { synthesizeStreetRow, type StreetBaseTuple } from "../synthesize-street.ts"
+import { DEFAULT_US_BASES } from "#synthesizers/intersection"
+import { synthesizeStreetRow, type StreetBaseTuple } from "#synthesizers/street"
+import { alignRow } from "#utils"
+
 import { makeMulberry32, shardSourceID, type CanonicalShardRow, type ShardRecipe } from "./scaffold.ts"
 
 /**

package/src/shard-recipes/street.ts

@@ -7,8 +7,9 @@
  *   {@link synthesizeStreetRow} → aligned LabeledRow. Ported from scripts/build-street-shard.mjs.
  */
 
-import { stableSourceID } from "../adapter.ts"
-import { synthesizeStreetRow, type StreetBaseTuple } from "../synthesize-street.ts"
+import { stableSourceID } from "#adapters/utils"
+import { synthesizeStreetRow, type StreetBaseTuple } from "#synthesizers/street"
+
 import { alignAndWrite, makeLcg, readTuples, type ShardRecipe } from "./scaffold.ts"
 
 /**

package/src/shard-recipes/sub-venue-sources.ts

@@ -26,30 +26,31 @@
  */
 
 import { readFileSync } from "node:fs"
-import { createRequire } from "node:module"
-import { dirname, resolve } from "node:path"
 import { DatabaseSync } from "node:sqlite"
+import { fileURLToPath } from "node:url"
 
 import { parseJSONStrict } from "@mailwoman/core/objects"
 import { dataRootPath } from "@mailwoman/core/utils"
 
-import type { LocaleBaseTuple } from "../synthesize-german.ts"
-import { classifyIdentifier, readSubVenueJSONL, type SubVenueLexiconTable } from "../tools/sub-venue-lexicon.ts"
-import { SUBVENUE_PROMOTIONS, type SubVenuePromotion } from "../tools/sub-venue-promotions.ts"
+import type { LocaleBaseTuple } from "#synthesizers/german"
+import {
+	classifyIdentifier,
+	readSubVenueJSONL,
+	type SubVenueLexiconTable,
+	SUBVENUE_PROMOTIONS,
+	type SubVenuePromotion,
+} from "#tools"
+
 import { readTuples as readLocaleTuples, type LocalePart } from "./locale.ts"
 import { makeMulberry32, readTuples as readShardTuples } from "./scaffold.ts"
 
 //#region Lexicon
 
-const require_ = createRequire(import.meta.url)
-
 /**
- * The committed lexicon, resolved through the package manifest rather than by counting `..` segments — the same file
- * sits at `corpus/data/` from both the source tree and the compiled `out/` tree, and `createRequire().resolve` finds it
- * from either without a `__isCompiledTree` flag of its own.
+ * The committed lexicon, resolved through the package manifest.
  */
 export function defaultLexiconPath(): string {
-	return resolve(dirname(require_.resolve("@mailwoman/corpus/package.json")), "data", "sub-venue-lexicon.json")
+	return fileURLToPath(import.meta.resolve("@mailwoman/corpus/data/sub-venue-lexicon.json"))
 }
 
 /**
@@ -568,17 +569,6 @@ export const EMPTY_NAME_POOLS: NamePools = {
 	unpromotedShapes: [],
 }
 
-/**
- * An extract row, as `sub-venue-extract` writes it. `SubVenueHarvestRow` plus the `tier` the harvest does not need and
- * this recipe does: `venue` rows (station, airport, campus) fill the venue slot.
- */
-interface SubVenueExtractRow {
-	designatorID: string
-	tier?: string
-	name?: string | null
-	ref?: string | null
-}
-
 /**
  * What every pool reader needs to know about the leg it is reading for.
  */
@@ -602,7 +592,7 @@ function isLongerProperName(low: string, name: string, designatorPhrases: readon
  * Read one OSM extract and split it into name pools.
  */
 export function readExtractPools(path: string, query: PoolQuery): NamePools {
-	const rows = readSubVenueJSONL(path) as unknown as SubVenueExtractRow[]
+	const rows = readSubVenueJSONL(path)
 	const venues = new Set<string>()
 	const attested = new Set<string>()
 	const rejectedVenues = new Set<string>()

package/src/shard-recipes/sub-venue.ts

@@ -60,12 +60,14 @@
  *        `JP` key. The JP extract is harvested and ready; the leg belongs to the JP shard.
  */
 
+import { isPresent } from "@mailwoman/core/objects"
 import type { ComponentTag } from "@mailwoman/core/types"
 import { dataRootPath } from "@mailwoman/core/utils"
 
-import { alignRow } from "../align.ts"
-import type { LocaleBaseTuple } from "../synthesize-german.ts"
-import type { SubVenueLexiconTable } from "../tools/sub-venue-lexicon.ts"
+import type { LocaleBaseTuple } from "#synthesizers/german"
+import type { SubVenueLexiconTable } from "#tools"
+import { alignRow } from "#utils"
+
 import { makeMulberry32, shardSourceID, type ShardRecipe } from "./scaffold.ts"
 import {
 	buildIdentifierModel,
@@ -374,7 +376,7 @@ export function renderGroups(
 
 	const raw = groups
 		.map((group) => group.map((piece) => fold(piece.text)).join(" "))
-		.filter(Boolean)
+		.filter(isPresent)
 		.join(separator)
 
 	const components: Partial<Record<ComponentTag, string>> = {}
@@ -394,7 +396,7 @@ export function renderGroups(
  * The street + tail groups for a country, in that country's own order.
  *
  * DE/ES/FR put the postcode before the locality and DE/ES put the house number after the street; GB and US keep the
- * anglophone order and US carries a region. These are the same orders `synthesize-german.ts` renders, restated here
+ * anglophone order and US carries a region. These are the same orders `synthesizers/german.ts` renders, restated here
  * because this recipe assembles its groups piece-by-piece (it has to, to place a sub-venue group in front of them).
  */
 export function addressGroups(country: string, tuple: LocaleBaseTuple, withStreet: boolean): Group[] {
@@ -826,13 +828,13 @@ function buildLegPools(
 	leg: SubVenueLeg,
 	query: PoolQuery,
 	contextByCountry: ReadonlyMap<string, LocaleBaseTuple[]>,
-	paths: { extractsDir: string; poiDb: string }
+	paths: { extractsDir: string; poiDB: string }
 ): LegPools {
 	const extractPools = leg.extract ? readExtractPools(`${paths.extractsDir}/${leg.extract}`, query) : EMPTY_NAME_POOLS
 
 	// poi.db holds four countries; only these two legs are inside it.
 	const poiPools =
-		leg.country === "US" || leg.country === "FR" ? readPOIPools(paths.poiDb, leg.country, query) : EMPTY_NAME_POOLS
+		leg.country === "US" || leg.country === "FR" ? readPOIPools(paths.poiDB, leg.country, query) : EMPTY_NAME_POOLS
 
 	const names = mergeNamePools(extractPools, poiPools)
 	let context = contextByCountry.get(leg.country) ?? []
@@ -906,7 +908,7 @@ export const subVenueRecipe: ShardRecipe = {
 		const count = opts.count
 		const negativeFraction = opts.negativeFraction ?? DEFAULT_NEGATIVE_FRACTION
 		const extractsDir = opts.extractsDir ?? dataRootPath("sub-venue", "extracts")
-		const poiDb = opts.poiDb ?? dataRootPath("poi", "poi.db")
+		const poiDB = opts.poiDB ?? dataRootPath("poi", "poi.db")
 		const tuplesPath = opts.subVenueTuples ?? dataRootPath("corpus", "intermediate", "house-venue-tuples-v3.jsonl")
 		const lexicon: SubVenueLexiconTable = readSubVenueLexicon(opts.lexicon ?? defaultLexiconPath())
 
@@ -937,7 +939,7 @@ export const subVenueRecipe: ShardRecipe = {
 				english: leg.english,
 			}
 
-			const pools = buildLegPools(leg, query, contextByCountry, { extractsDir, poiDb })
+			const pools = buildLegPools(leg, query, contextByCountry, { extractsDir, poiDB })
 
 			legPromoted.set(leg.locale, promoted)
 			legPools.set(leg.locale, pools)