fix(aggregator): rename labellers table via forward migration (#2111)

Migration 0001 was edited in place to rename `labellers` -> `labelers`
after shipping on main. Databases that already applied 0001 never see the
edit, so they keep `labellers` while every query in src/ reads `labelers`,
failing with `no such table`.

Restore 0001 to main's byte-for-byte schema and add 0004 to rename the
table forward. Both fresh installs and main upgrades now converge on
`labelers`. Adds an upgrade test that provisions main's frozen 0001, applies
the live set, and asserts the table is queryable.

Author: ascorbic

apps/aggregator/migrations/0001_init.sql

@@ -178,14 +178,14 @@ CREATE TABLE mirrored_artifacts (
 );
 
 ------------------------------------------------------------------------------
--- Labels (populated when the labeler integration lands)
+-- Labels (populated when the labeller integration lands)
 ------------------------------------------------------------------------------
 
 -- Append-only label history. Every label received is written here, including
 -- negations. Current state is derived from latest cts per (src, uri, val) and
 -- projected into label_state below for hot-path lookups.
 CREATE TABLE labels (
-	src TEXT NOT NULL,                          -- labeler DID
+	src TEXT NOT NULL,                          -- labeller DID
 	uri TEXT NOT NULL,                          -- AT URI of subject
 	cid TEXT,                                   -- optional version-specific CID
 	val TEXT NOT NULL,                          -- e.g. 'security:yanked', '!takedown'
@@ -229,8 +229,8 @@ CREATE TABLE label_state (
 CREATE INDEX idx_label_state_enforce ON label_state(uri, val, trusted)
 	WHERE neg = 0 AND trusted = 1;
 
--- Trusted/known labelers (operator config, edited via deployment).
-CREATE TABLE labelers (
+-- Trusted/known labellers (operator config, edited via deployment).
+CREATE TABLE labellers (
 	did TEXT PRIMARY KEY,
 	endpoint TEXT NOT NULL,                     -- subscribeLabels URL
 	signing_key TEXT NOT NULL,                  -- cached #atproto_label key
@@ -278,9 +278,9 @@ END;
 ------------------------------------------------------------------------------
 
 -- Cursor state for ingest sources (Jetstream microsecond timestamp,
--- subscribeLabels seq cursors per labeler, etc.).
+-- subscribeLabels seq cursors per labeller, etc.).
 CREATE TABLE ingest_state (
-	source TEXT PRIMARY KEY,                    -- 'jetstream', 'labeler:did:web:labels.example.com', etc.
+	source TEXT PRIMARY KEY,                    -- 'jetstream', 'labeller:did:web:labels.example.com', etc.
 	cursor TEXT NOT NULL,
 	updated_at TEXT NOT NULL
 );

apps/aggregator/migrations/0004_rename_labellers_to_labelers.sql

@@ -0,0 +1,12 @@
+-- Rename `labellers` -> `labelers` to match the spelling every query in
+-- `src/` uses (`labeler-resolver.ts`, `labels-consumer.ts`,
+-- `request-policy.ts`, `index.ts`). `0001_init.sql` shipped the table as
+-- `labellers` and is immutable once applied, so the correction has to land as
+-- a forward migration: databases provisioned by `0001` keep `labellers` and
+-- only this migration brings them to `labelers`.
+--
+-- No indexes, triggers, or views reference the table, so SQLite's
+-- `ALTER TABLE ... RENAME TO` is sufficient — there are no explicitly-named
+-- objects embedding the old spelling to recreate.
+
+ALTER TABLE labellers RENAME TO labelers;