cursedbelt-server 4.28.0 → 4.29.0

package/dist/server/metrics/metricsBuffer.d.ts

@@ -27,8 +27,6 @@ export interface MetricRow {
     region?: string | null;
     city?: string | null;
 }
-/** The optional place columns, in insert order. Each is written only when the table HAS it. */
-export declare const PLACE_COLUMNS: readonly ["country", "region", "city"];
 export interface MetricsBufferOptions {
     db: Database;
     /** Flush cadence in ms. Default: 1500. */

package/dist/server/metrics/metricsBuffer.js

@@ -1,5 +1,8 @@
-/** The optional place columns, in insert order. Each is written only when the table HAS it. */
-export const PLACE_COLUMNS = ['country', 'region', 'city'];
+import { PLACE_COLUMNS } from './requestPlace.js';
+// Every place column is a `MetricRow` field — a column declared in `requestPlace.ts` that this
+// row cannot carry fails to compile here rather than inserting `undefined` for ever.
+const _placeColumnsAreRowFields = PLACE_COLUMNS;
+void _placeColumnsAreRowFields;
 export function createMetricsBuffer(opts) {
     const { db, flushIntervalMs = 1500, maxSize = 5000 } = opts;
     let buffer = [];

package/dist/server/metrics/requestPlace.d.ts

@@ -30,6 +30,14 @@ export interface RequestPlace {
 }
 /** Place one request, or `null` when it cannot. Must not throw — but a throw is caught and read as `null`. */
 export type RequestLocator = (request: Request) => RequestPlace | null;
+/**
+ * The place columns, in insert order — the ONE declaration every reader and migration uses
+ * (`metricsBuffer` writes them, an app's reader allows them, an app's migration adds them).
+ * Exported through `cursedbelt-server/telemetry` since 4.29.0 (task 2141): until then it was
+ * private to `metricsBuffer.ts`, so flix, station and desk each declared their own copy, and a
+ * fourth column added here would have been written by the buffer and dropped by all three.
+ */
+export declare const PLACE_COLUMNS: readonly ["country", "region", "city"];
 /** Each place field is capped so three of them can never crowd the 5120-byte Analytics Engine blob budget. */
 export declare const PLACE_FIELD_MAX_CHARS = 96;
 /**

package/dist/server/metrics/requestPlace.js

@@ -22,6 +22,14 @@
  *
  * Worker-safe: no `bun:` import (`telemetryIsWorkerSafe.spec.ts` walks this graph).
  */
+/**
+ * The place columns, in insert order — the ONE declaration every reader and migration uses
+ * (`metricsBuffer` writes them, an app's reader allows them, an app's migration adds them).
+ * Exported through `cursedbelt-server/telemetry` since 4.29.0 (task 2141): until then it was
+ * private to `metricsBuffer.ts`, so flix, station and desk each declared their own copy, and a
+ * fourth column added here would have been written by the buffer and dropped by all three.
+ */
+export const PLACE_COLUMNS = ['country', 'region', 'city'];
 /** Each place field is capped so three of them can never crowd the 5120-byte Analytics Engine blob budget. */
 export const PLACE_FIELD_MAX_CHARS = 96;
 /** Cloudflare's "no country could be determined" code — an unknown, not a place. */

package/dist/server/requestLog/requestMetrics.js

@@ -62,7 +62,7 @@ import { Database } from "bun:sqlite";
 import { closeSync, existsSync, mkdirSync, openSync, readSync, renameSync, statSync } from "node:fs";
 import { join } from "node:path";
 import { applyCcPragmas } from "cwip/sqlite";
-import { PLACE_COLUMNS } from "../metrics/metricsBuffer.js";
+import { PLACE_COLUMNS } from "../metrics/requestPlace.js";
 import { createTelemetrySink } from "../metrics/telemetrySink.js";
 import { requestLogger } from "../middleware/requestLogger.js";
 import { runMigrations } from "../migration/runner.js";

package/dist/server/telemetry.d.ts

@@ -17,4 +17,4 @@
  */
 export { type RequestLoggerOpts, requestLogger } from './middleware/requestLogger.js';
 export { type AnalyticsEngineDataset, createTelemetrySink, TELEMETRY_POINT_VERSION, type TelemetryEvent, type TelemetrySink, type TelemetrySinkOptions, toDataPoint, } from './metrics/telemetrySink.js';
-export { locateFromCloudflare, type RequestLocator, type RequestPlace, } from './metrics/requestPlace.js';
+export { locateFromCloudflare, PLACE_COLUMNS, type RequestLocator, type RequestPlace, } from './metrics/requestPlace.js';

package/dist/server/telemetry.js

@@ -17,4 +17,4 @@
  */
 export { requestLogger } from './middleware/requestLogger.js';
 export { createTelemetrySink, TELEMETRY_POINT_VERSION, toDataPoint, } from './metrics/telemetrySink.js';
-export { locateFromCloudflare, } from './metrics/requestPlace.js';
+export { locateFromCloudflare, PLACE_COLUMNS, } from './metrics/requestPlace.js';

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "cursedbelt-server",
-	"version": "4.28.0",
+	"version": "4.29.0",
 	"license": "ISC",
 	"type": "module",
 	"description": "The app-facing Bun/Hono server tier of the cursedbelt split — storage, sharing, activity, guard, sync. React-free; cursedbelt-core below it.",

package/src/server/metrics/metricsBuffer.ts

@@ -1,4 +1,5 @@
 import type { Database } from 'bun:sqlite';
+import { PLACE_COLUMNS } from './requestPlace.js';
 
 /**
  * In-memory, batched writer for `request_metrics`. To keep request latency
@@ -30,8 +31,10 @@ export interface MetricRow {
   city?: string | null;
 }
 
-/** The optional place columns, in insert order. Each is written only when the table HAS it. */
-export const PLACE_COLUMNS = ['country', 'region', 'city'] as const;
+// Every place column is a `MetricRow` field — a column declared in `requestPlace.ts` that this
+// row cannot carry fails to compile here rather than inserting `undefined` for ever.
+const _placeColumnsAreRowFields: readonly (keyof MetricRow)[] = PLACE_COLUMNS;
+void _placeColumnsAreRowFields;
 
 export interface MetricsBufferOptions {
   db: Database;

package/src/server/metrics/requestPlace.ts

@@ -33,6 +33,15 @@ export interface RequestPlace {
 /** Place one request, or `null` when it cannot. Must not throw — but a throw is caught and read as `null`. */
 export type RequestLocator = (request: Request) => RequestPlace | null;
 
+/**
+ * The place columns, in insert order — the ONE declaration every reader and migration uses
+ * (`metricsBuffer` writes them, an app's reader allows them, an app's migration adds them).
+ * Exported through `cursedbelt-server/telemetry` since 4.29.0 (task 2141): until then it was
+ * private to `metricsBuffer.ts`, so flix, station and desk each declared their own copy, and a
+ * fourth column added here would have been written by the buffer and dropped by all three.
+ */
+export const PLACE_COLUMNS = ['country', 'region', 'city'] as const;
+
 /** Each place field is capped so three of them can never crowd the 5120-byte Analytics Engine blob budget. */
 export const PLACE_FIELD_MAX_CHARS = 96;
 

package/src/server/requestLog/requestMetrics.ts

@@ -63,7 +63,7 @@ import { closeSync, existsSync, mkdirSync, openSync, readSync, renameSync, statS
 import { join } from "node:path";
 import { applyCcPragmas } from "cwip/sqlite";
 import type { MiddlewareHandler } from "hono";
-import { PLACE_COLUMNS } from "../metrics/metricsBuffer.js";
+import { PLACE_COLUMNS } from "../metrics/requestPlace.js";
 import type { RequestLocator } from "../metrics/requestPlace.js";
 import { createTelemetrySink, type TelemetrySink } from "../metrics/telemetrySink.js";
 import { requestLogger } from "../middleware/requestLogger.js";

package/src/server/telemetry.ts

@@ -27,6 +27,7 @@ export {
 } from './metrics/telemetrySink.js';
 export {
 	locateFromCloudflare,
+	PLACE_COLUMNS,
 	type RequestLocator,
 	type RequestPlace,
 } from './metrics/requestPlace.js';

package/src/server/telemetryIsWorkerSafe.spec.ts

@@ -88,6 +88,14 @@ describe('cursedbelt-server/telemetry', () => {
 		const found = [..."import { Database } from 'bun:sqlite';\nimport type { X } from 'bun:sqlite';".matchAll(regex)].map((m) => m[1]);
 		expect(found).toEqual(['bun:sqlite']);
 	});
+
+	test('publishes PLACE_COLUMNS — the one declaration the buffer writes and every app reads (task 2141)', async () => {
+		const telemetry = await import('./telemetry.ts');
+		expect(telemetry.PLACE_COLUMNS).toEqual(['country', 'region', 'city']);
+		// The same array the Bun buffer inserts from, not a second copy of it.
+		const { PLACE_COLUMNS } = await import('./metrics/requestPlace.ts');
+		expect(telemetry.PLACE_COLUMNS).toBe(PLACE_COLUMNS);
+	});
 });
 
 describe('🔴 cursedbelt-server/engagement/d1 — a Worker mounts it (task 2096)', () => {