@gscdump/sdk 0.36.0 → 0.36.2

package/dist/index.d.mts

@@ -36,6 +36,8 @@ interface HostedFetchOptions {
   headers?: HeadersInit;
   query?: Record<string, unknown>;
   body?: unknown;
+  /** Opt into in-flight de-dupe for safe read-style POST requests. */
+  dedupe?: boolean;
   [key: string]: unknown;
 }
 interface HostedClientOptions {
@@ -43,6 +45,10 @@ interface HostedClientOptions {
   fetch?: HostedFetch;
   headers?: HostedHeaders;
   validate?: boolean | 'request' | 'response';
+  /** Coalesce concurrent identical GET/HEAD requests within this client instance. */
+  dedupe?: boolean;
+  /** Optional shared in-flight map for de-duping across related client instances. */
+  dedupeScope?: Map<string, Promise<Result<unknown, PartnerApiError>>>;
 }
 type AnalyticsFetch = HostedFetch;
 type AnalyticsHeaders = HostedHeaders;

package/dist/index.mjs

@@ -22,6 +22,11 @@ function withDefaultSearchType(value, searchType) {
 function searchTypeQuery(searchType) {
 	return { searchType: searchType ?? "web" };
 }
+function stableJson$1(value) {
+	if (value == null || typeof value !== "object") return JSON.stringify(value) ?? "null";
+	if (Array.isArray(value)) return `[${value.map((item) => item === void 0 ? "null" : stableJson$1(item)).join(",")}]`;
+	return `{${Object.entries(value).filter(([, item]) => item !== void 0).sort(([a], [b]) => a.localeCompare(b)).map(([key, item]) => `${JSON.stringify(key)}:${stableJson$1(item)}`).join(",")}}`;
+}
 function dateRangeOptionsQuery(options) {
 	const query = {};
 	const start = options?.start ?? options?.startDate;
@@ -43,17 +48,17 @@ function tablesQuery(tablesOrOptions, options) {
 		...searchTypeQuery(source?.searchType),
 		...dateRangeOptionsQuery(source)
 	};
-	if (tables) query.tables = Array.isArray(tables) ? tables.join(",") : tables;
+	if (tables) query.tables = Array.isArray(tables) ? [...new Set(tables.filter(Boolean))].sort().join(",") : [...new Set(tables.split(",").map((t) => t.trim()).filter(Boolean))].sort().join(",");
 	return query;
 }
 function dataQuery(state, options) {
 	const opts = options;
 	const scoped = withDefaultSearchType(state, opts?.searchType);
 	const query = {
-		q: JSON.stringify(scoped),
+		q: stableJson$1(scoped),
 		searchType: scoped.searchType ?? "web"
 	};
-	if (opts?.comparison) query.qc = JSON.stringify(withDefaultSearchType(opts.comparison, scoped.searchType));
+	if (opts?.comparison) query.qc = stableJson$1(withDefaultSearchType(opts.comparison, scoped.searchType));
 	if (opts?.filter) query.filter = opts.filter;
 	return query;
 }
@@ -61,10 +66,10 @@ function dataDetailQuery(state, options) {
 	const opts = options;
 	const scoped = withDefaultSearchType(state, opts?.searchType);
 	const query = {
-		q: JSON.stringify(scoped),
+		q: stableJson$1(scoped),
 		searchType: scoped.searchType ?? "web"
 	};
-	if (opts?.comparison) query.qc = JSON.stringify(withDefaultSearchType(opts.comparison, scoped.searchType));
+	if (opts?.comparison) query.qc = stableJson$1(withDefaultSearchType(opts.comparison, scoped.searchType));
 	return query;
 }
 function analysisQuery(params) {
@@ -98,7 +103,7 @@ function indexingUrlsQuery(params = {}) {
 }
 function indexingDiagnosticsQuery(params = {}) {
 	const query = {};
-	if (params.sampleIssues) query.sampleIssues = Array.isArray(params.sampleIssues) ? params.sampleIssues.join(",") : params.sampleIssues;
+	if (params.sampleIssues) query.sampleIssues = Array.isArray(params.sampleIssues) ? [...new Set(params.sampleIssues.map((item) => String(item).trim()).filter(Boolean))].sort().join(",") : params.sampleIssues;
 	if (params.sampleLimit != null) query.sampleLimit = params.sampleLimit;
 	return query;
 }
@@ -206,20 +211,51 @@ function shouldValidate(options, phase) {
 function parseWith(schema, value) {
 	return schema ? schema.parse(value) : value;
 }
+function stableJson(value) {
+	if (value == null || typeof value !== "object") return JSON.stringify(value) ?? "undefined";
+	if (Array.isArray(value)) return `[${value.map((item) => item === void 0 ? "undefined" : stableJson(item)).join(",")}]`;
+	return `{${Object.entries(value).filter(([, item]) => item !== void 0).sort(([a], [b]) => a.localeCompare(b)).map(([key, item]) => `${JSON.stringify(key)}:${stableJson(item)}`).join(",")}}`;
+}
+function headersKey(headers) {
+	return [...headers.entries()].sort(([a], [b]) => a.localeCompare(b)).map(([key, value]) => `${key}:${value}`).join("\n");
+}
+function isDedupeable(init) {
+	if ("signal" in init && init.signal) return false;
+	const method = (init.method ?? "GET").toUpperCase();
+	return method === "GET" || method === "HEAD" || init.dedupe === true;
+}
 function createHostedRequester(options, defaults) {
 	const fetchImpl = options.fetch ?? ofetch;
 	const apiBase = trimApiBase(options.apiBase, defaults.apiBase);
+	const dedupe = options.dedupe !== false;
+	const inflight = options.dedupeScope ?? /* @__PURE__ */ new Map();
 	async function requestResult(path, init = {}, responseSchema) {
+		const { dedupe: _dedupe, ...fetchInit } = init;
 		const headers = mergeHeaders(await resolveHeaders(options), init.headers);
-		try {
-			const out = await fetchImpl(buildPath(apiBase, path), {
-				...init,
-				headers
+		const fullPath = buildPath(apiBase, path);
+		const dedupeKey = dedupe && isDedupeable(init) ? `${(init.method ?? "GET").toUpperCase()} ${fullPath}\nq=${stableJson(init.query)}\nb=${stableJson(init.body)}\nh=${headersKey(headers)}` : null;
+		if (dedupeKey) {
+			const existing = inflight.get(dedupeKey);
+			if (existing) return existing;
+		}
+		const run = (async () => {
+			try {
+				const out = await fetchImpl(fullPath, {
+					...fetchInit,
+					headers
+				});
+				return ok(shouldValidate(options, "response") ? parseWith(responseSchema, out) : out);
+			} catch (error) {
+				return err(toPartnerError(error));
+			}
+		})();
+		if (dedupeKey) {
+			inflight.set(dedupeKey, run);
+			run.finally(() => {
+				if (inflight.get(dedupeKey) === run) inflight.delete(dedupeKey);
 			});
-			return ok(shouldValidate(options, "response") ? parseWith(responseSchema, out) : out);
-		} catch (error) {
-			return err(toPartnerError(error));
 		}
+		return run;
 	}
 	async function request(path, init = {}, responseSchema) {
 		return unwrapResult(await requestResult(path, init, responseSchema), partnerErrorToException);
@@ -248,13 +284,15 @@ function createAnalyticsClient(options = {}) {
 		analyze(siteId, params) {
 			return request(analyticsRoutes.site.analyze(siteId), {
 				method: "POST",
-				body: withDefaultSearchType(params)
+				body: withDefaultSearchType(params),
+				dedupe: true
 			});
 		},
 		queryRows(siteId, state) {
 			return request(analyticsRoutes.site.rows(siteId), {
 				method: "POST",
-				body: withDefaultSearchType(state)
+				body: withDefaultSearchType(state),
+				dedupe: true
 			}, analyticsEndpointSchemas.analyticsRows.response);
 		},
 		getRollup(siteId, rollupId, params) {
@@ -632,7 +670,8 @@ function createPartnerClient(options = {}) {
 			const body = shouldValidate("request") ? partnerEndpointSchemas.getKeywordSparklines.body.parse(withSearchType) : withSearchType;
 			return request(partnerRoutes.sites.keywordSparklines(siteId), {
 				method: "POST",
-				body
+				body,
+				dedupe: true
 			}, partnerEndpointSchemas.getKeywordSparklines.response);
 		},
 		getQueryTrend(siteId, params) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@gscdump/sdk",
   "type": "module",
-  "version": "0.36.0",
+  "version": "0.36.2",
   "description": "Consumer SDK for hosted gscdump.com integrations.",
   "author": {
     "name": "Harlan Wilton",
@@ -44,10 +44,10 @@
     "date-fns": "^4.4.0",
     "ofetch": "^1.5.1",
     "zod": "^4.4.3",
-    "@gscdump/contracts": "0.36.0",
-    "gscdump": "0.36.0",
-    "@gscdump/analysis": "0.36.0",
-    "@gscdump/engine": "0.36.0"
+    "@gscdump/engine": "0.36.2",
+    "@gscdump/analysis": "0.36.2",
+    "gscdump": "0.36.2",
+    "@gscdump/contracts": "0.36.2"
   },
   "devDependencies": {
     "typescript": "^6.0.3",