@gscdump/analysis 0.25.6 → 0.25.7

package/dist/analyzer/index.mjs

@@ -2388,6 +2388,34 @@ const dataQueryAnalyzer = defineAnalyzer({
 		};
 	}
 });
+const DEFAULT_LIMIT = 1e3;
+const MAX_LIMIT = 5e4;
+function clampLimit(limit, fallback = DEFAULT_LIMIT) {
+	const n = Number(limit ?? fallback);
+	if (!Number.isFinite(n) || n <= 0) return fallback;
+	return Math.min(n, MAX_LIMIT);
+}
+function clampOffset(offset) {
+	const n = Number(offset ?? 0);
+	if (!Number.isFinite(n) || n < 0) return 0;
+	return Math.floor(n);
+}
+function paginateClause(input) {
+	const l = clampLimit(input.limit);
+	const o = clampOffset(input.offset);
+	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
+}
+function paginateInMemory(rows, input) {
+	const l = clampLimit(input.limit, rows.length);
+	const o = clampOffset(input.offset);
+	return rows.slice(o, o + l);
+}
+function resolveSort(input, allowed, defaults) {
+	return {
+		sortBy: input.sortBy && allowed.includes(input.sortBy) ? input.sortBy : defaults.sortBy,
+		sortDir: input.sortDir === "asc" || input.sortDir === "desc" ? input.sortDir : defaults.sortDir
+	};
+}
 const sortResults$1 = createMetricSorter("lostClicks", {
 	lostClicks: "desc",
 	declinePercent: "desc",
@@ -2441,7 +2469,6 @@ const decayAnalyzer = defineAnalyzer({
 		const { current: cur, previous: prev } = comparisonOf(params);
 		const minPreviousClicks = params.minPreviousClicks ?? 50;
 		const threshold = params.threshold ?? .2;
-		const limit = params.limit ?? 2e3;
 		return {
 			sql: `
       WITH cur AS (
@@ -2505,7 +2532,7 @@ const decayAnalyzer = defineAnalyzer({
       FROM joined
       WHERE declinePercent >= ? AND lostClicks > 0
       ORDER BY lostClicks DESC
-      LIMIT ${Number(limit)}
+      LIMIT 2000
     `,
 			params: [
 				cur.startDate,
@@ -2529,25 +2556,28 @@ const decayAnalyzer = defineAnalyzer({
 			}
 		};
 	},
-	reduceSql(rows) {
-		const arr = Array.isArray(rows) ? rows : [];
+	reduceSql(rows, params) {
+		const mapped = (Array.isArray(rows) ? rows : []).map((r) => ({
+			page: str$12(r.page),
+			currentClicks: num(r.currentClicks),
+			previousClicks: num(r.previousClicks),
+			lostClicks: num(r.lostClicks),
+			declinePercent: num(r.declinePercent),
+			currentPosition: num(r.currentPosition),
+			previousPosition: num(r.previousPosition),
+			positionDrop: num(r.positionDrop),
+			series: parseJsonList$8(r.seriesJson).map((s) => ({
+				week: str$12(s.week),
+				clicks: num(s.clicks),
+				impressions: num(s.impressions)
+			}))
+		}));
 		return {
-			results: arr.map((r) => ({
-				page: str$12(r.page),
-				currentClicks: num(r.currentClicks),
-				previousClicks: num(r.previousClicks),
-				lostClicks: num(r.lostClicks),
-				declinePercent: num(r.declinePercent),
-				currentPosition: num(r.currentPosition),
-				previousPosition: num(r.previousPosition),
-				positionDrop: num(r.positionDrop),
-				series: parseJsonList$8(r.seriesJson).map((s) => ({
-					week: str$12(s.week),
-					clicks: num(s.clicks),
-					impressions: num(s.impressions)
-				}))
-			})),
-			meta: { total: arr.length }
+			results: paginateInMemory(mapped, {
+				limit: params.limit ?? 2e3,
+				offset: params.offset
+			}),
+			meta: { total: mapped.length }
 		};
 	},
 	buildRows(params) {
@@ -3426,34 +3456,6 @@ const moversAnalyzer = defineAnalyzer({
 		})), [], params);
 	}
 });
-const DEFAULT_LIMIT = 1e3;
-const MAX_LIMIT = 5e4;
-function clampLimit(limit, fallback = DEFAULT_LIMIT) {
-	const n = Number(limit ?? fallback);
-	if (!Number.isFinite(n) || n <= 0) return fallback;
-	return Math.min(n, MAX_LIMIT);
-}
-function clampOffset(offset) {
-	const n = Number(offset ?? 0);
-	if (!Number.isFinite(n) || n < 0) return 0;
-	return Math.floor(n);
-}
-function paginateClause(input) {
-	const l = clampLimit(input.limit);
-	const o = clampOffset(input.offset);
-	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
-}
-function paginateInMemory(rows, input) {
-	const l = clampLimit(input.limit, rows.length);
-	const o = clampOffset(input.offset);
-	return rows.slice(o, o + l);
-}
-function resolveSort(input, allowed, defaults) {
-	return {
-		sortBy: input.sortBy && allowed.includes(input.sortBy) ? input.sortBy : defaults.sortBy,
-		sortDir: input.sortDir === "asc" || input.sortDir === "desc" ? input.sortDir : defaults.sortDir
-	};
-}
 const EXPECTED_CTR_BY_POSITION = {
 	1: .3,
 	2: .15,

package/dist/default-registry.mjs

@@ -2388,6 +2388,28 @@ const dataQueryAnalyzer = defineAnalyzer({
 		};
 	}
 });
+const DEFAULT_LIMIT = 1e3;
+const MAX_LIMIT = 5e4;
+function clampLimit(limit, fallback = DEFAULT_LIMIT) {
+	const n = Number(limit ?? fallback);
+	if (!Number.isFinite(n) || n <= 0) return fallback;
+	return Math.min(n, MAX_LIMIT);
+}
+function clampOffset(offset) {
+	const n = Number(offset ?? 0);
+	if (!Number.isFinite(n) || n < 0) return 0;
+	return Math.floor(n);
+}
+function paginateClause(input) {
+	const l = clampLimit(input.limit);
+	const o = clampOffset(input.offset);
+	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
+}
+function paginateInMemory(rows, input) {
+	const l = clampLimit(input.limit, rows.length);
+	const o = clampOffset(input.offset);
+	return rows.slice(o, o + l);
+}
 const sortResults$1 = createMetricSorter("lostClicks", {
 	lostClicks: "desc",
 	declinePercent: "desc",
@@ -2441,7 +2463,6 @@ const decayAnalyzer = defineAnalyzer({
 		const { current: cur, previous: prev } = comparisonOf(params);
 		const minPreviousClicks = params.minPreviousClicks ?? 50;
 		const threshold = params.threshold ?? .2;
-		const limit = params.limit ?? 2e3;
 		return {
 			sql: `
       WITH cur AS (
@@ -2505,7 +2526,7 @@ const decayAnalyzer = defineAnalyzer({
       FROM joined
       WHERE declinePercent >= ? AND lostClicks > 0
       ORDER BY lostClicks DESC
-      LIMIT ${Number(limit)}
+      LIMIT 2000
     `,
 			params: [
 				cur.startDate,
@@ -2529,25 +2550,28 @@ const decayAnalyzer = defineAnalyzer({
 			}
 		};
 	},
-	reduceSql(rows) {
-		const arr = Array.isArray(rows) ? rows : [];
+	reduceSql(rows, params) {
+		const mapped = (Array.isArray(rows) ? rows : []).map((r) => ({
+			page: str$12(r.page),
+			currentClicks: num(r.currentClicks),
+			previousClicks: num(r.previousClicks),
+			lostClicks: num(r.lostClicks),
+			declinePercent: num(r.declinePercent),
+			currentPosition: num(r.currentPosition),
+			previousPosition: num(r.previousPosition),
+			positionDrop: num(r.positionDrop),
+			series: parseJsonList$8(r.seriesJson).map((s) => ({
+				week: str$12(s.week),
+				clicks: num(s.clicks),
+				impressions: num(s.impressions)
+			}))
+		}));
 		return {
-			results: arr.map((r) => ({
-				page: str$12(r.page),
-				currentClicks: num(r.currentClicks),
-				previousClicks: num(r.previousClicks),
-				lostClicks: num(r.lostClicks),
-				declinePercent: num(r.declinePercent),
-				currentPosition: num(r.currentPosition),
-				previousPosition: num(r.previousPosition),
-				positionDrop: num(r.positionDrop),
-				series: parseJsonList$8(r.seriesJson).map((s) => ({
-					week: str$12(s.week),
-					clicks: num(s.clicks),
-					impressions: num(s.impressions)
-				}))
-			})),
-			meta: { total: arr.length }
+			results: paginateInMemory(mapped, {
+				limit: params.limit ?? 2e3,
+				offset: params.offset
+			}),
+			meta: { total: mapped.length }
 		};
 	},
 	buildRows(params) {
@@ -3426,28 +3450,6 @@ const moversAnalyzer = defineAnalyzer({
 		})), [], params);
 	}
 });
-const DEFAULT_LIMIT = 1e3;
-const MAX_LIMIT = 5e4;
-function clampLimit(limit, fallback = DEFAULT_LIMIT) {
-	const n = Number(limit ?? fallback);
-	if (!Number.isFinite(n) || n <= 0) return fallback;
-	return Math.min(n, MAX_LIMIT);
-}
-function clampOffset(offset) {
-	const n = Number(offset ?? 0);
-	if (!Number.isFinite(n) || n < 0) return 0;
-	return Math.floor(n);
-}
-function paginateClause(input) {
-	const l = clampLimit(input.limit);
-	const o = clampOffset(input.offset);
-	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
-}
-function paginateInMemory(rows, input) {
-	const l = clampLimit(input.limit, rows.length);
-	const o = clampOffset(input.offset);
-	return rows.slice(o, o + l);
-}
 const EXPECTED_CTR_BY_POSITION = {
 	1: .3,
 	2: .15,

package/dist/index.mjs

@@ -2754,6 +2754,28 @@ const dataQueryAnalyzer = defineAnalyzer$1({
 		};
 	}
 });
+const DEFAULT_LIMIT$1 = 1e3;
+const MAX_LIMIT = 5e4;
+function clampLimit(limit, fallback = DEFAULT_LIMIT$1) {
+	const n = Number(limit ?? fallback);
+	if (!Number.isFinite(n) || n <= 0) return fallback;
+	return Math.min(n, MAX_LIMIT);
+}
+function clampOffset(offset) {
+	const n = Number(offset ?? 0);
+	if (!Number.isFinite(n) || n < 0) return 0;
+	return Math.floor(n);
+}
+function paginateClause(input) {
+	const l = clampLimit(input.limit);
+	const o = clampOffset(input.offset);
+	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
+}
+function paginateInMemory(rows, input) {
+	const l = clampLimit(input.limit, rows.length);
+	const o = clampOffset(input.offset);
+	return rows.slice(o, o + l);
+}
 const sortResults$1 = createMetricSorter("lostClicks", {
 	lostClicks: "desc",
 	declinePercent: "desc",
@@ -2807,7 +2829,6 @@ const decayAnalyzer = defineAnalyzer$1({
 		const { current: cur, previous: prev } = comparisonOf$1(params);
 		const minPreviousClicks = params.minPreviousClicks ?? 50;
 		const threshold = params.threshold ?? .2;
-		const limit = params.limit ?? 2e3;
 		return {
 			sql: `
       WITH cur AS (
@@ -2871,7 +2892,7 @@ const decayAnalyzer = defineAnalyzer$1({
       FROM joined
       WHERE declinePercent >= ? AND lostClicks > 0
       ORDER BY lostClicks DESC
-      LIMIT ${Number(limit)}
+      LIMIT 2000
     `,
 			params: [
 				cur.startDate,
@@ -2895,25 +2916,28 @@ const decayAnalyzer = defineAnalyzer$1({
 			}
 		};
 	},
-	reduceSql(rows) {
-		const arr = Array.isArray(rows) ? rows : [];
+	reduceSql(rows, params) {
+		const mapped = (Array.isArray(rows) ? rows : []).map((r) => ({
+			page: str$12(r.page),
+			currentClicks: num$1(r.currentClicks),
+			previousClicks: num$1(r.previousClicks),
+			lostClicks: num$1(r.lostClicks),
+			declinePercent: num$1(r.declinePercent),
+			currentPosition: num$1(r.currentPosition),
+			previousPosition: num$1(r.previousPosition),
+			positionDrop: num$1(r.positionDrop),
+			series: parseJsonList$8(r.seriesJson).map((s) => ({
+				week: str$12(s.week),
+				clicks: num$1(s.clicks),
+				impressions: num$1(s.impressions)
+			}))
+		}));
 		return {
-			results: arr.map((r) => ({
-				page: str$12(r.page),
-				currentClicks: num$1(r.currentClicks),
-				previousClicks: num$1(r.previousClicks),
-				lostClicks: num$1(r.lostClicks),
-				declinePercent: num$1(r.declinePercent),
-				currentPosition: num$1(r.currentPosition),
-				previousPosition: num$1(r.previousPosition),
-				positionDrop: num$1(r.positionDrop),
-				series: parseJsonList$8(r.seriesJson).map((s) => ({
-					week: str$12(s.week),
-					clicks: num$1(s.clicks),
-					impressions: num$1(s.impressions)
-				}))
-			})),
-			meta: { total: arr.length }
+			results: paginateInMemory(mapped, {
+				limit: params.limit ?? 2e3,
+				offset: params.offset
+			}),
+			meta: { total: mapped.length }
 		};
 	},
 	buildRows(params) {
@@ -3788,28 +3812,6 @@ const moversAnalyzer = defineAnalyzer$1({
 		})), [], params);
 	}
 });
-const DEFAULT_LIMIT$1 = 1e3;
-const MAX_LIMIT = 5e4;
-function clampLimit(limit, fallback = DEFAULT_LIMIT$1) {
-	const n = Number(limit ?? fallback);
-	if (!Number.isFinite(n) || n <= 0) return fallback;
-	return Math.min(n, MAX_LIMIT);
-}
-function clampOffset(offset) {
-	const n = Number(offset ?? 0);
-	if (!Number.isFinite(n) || n < 0) return 0;
-	return Math.floor(n);
-}
-function paginateClause(input) {
-	const l = clampLimit(input.limit);
-	const o = clampOffset(input.offset);
-	return o > 0 ? `LIMIT ${l} OFFSET ${o}` : `LIMIT ${l}`;
-}
-function paginateInMemory(rows, input) {
-	const l = clampLimit(input.limit, rows.length);
-	const o = clampOffset(input.offset);
-	return rows.slice(o, o + l);
-}
 const EXPECTED_CTR_BY_POSITION = {
 	1: .3,
 	2: .15,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@gscdump/analysis",
   "type": "module",
-  "version": "0.25.6",
+  "version": "0.25.7",
   "description": "GSC analyzers — striking-distance, opportunity, movers, decay, brand, clustering, concentration, seasonality. Pure row-based + DuckDB-native.",
   "author": {
     "name": "Harlan Wilton",
@@ -70,9 +70,9 @@
   },
   "dependencies": {
     "drizzle-orm": "1.0.0-rc.3",
-    "@gscdump/engine": "0.25.6",
-    "@gscdump/engine-gsc-api": "0.25.6",
-    "gscdump": "0.25.6"
+    "@gscdump/engine": "0.25.7",
+    "@gscdump/engine-gsc-api": "0.25.7",
+    "gscdump": "0.25.7"
   },
   "devDependencies": {
     "vitest": "^4.1.7"