Skip to content

Commit aebb49e

Browse files
committed
Add module version to the source parameter
1 parent f6459f2 commit aebb49e

3 files changed

Lines changed: 9 additions & 6 deletions

File tree

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ and this project adheres to
1010

1111
### Added
1212

13+
- Add module version to the `source` parameter.
14+
1315
### Changed
1416

1517
### Fixed

src/utils.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { config } from "./config.ts";
22
import { InvalidTimeoutError, MissingApiKeyError } from "./errors.ts";
3+
import { version } from "../version.ts";
34

45
type UrlParameters = Record<
56
string,
@@ -59,22 +60,22 @@ function getBaseUrl() {
5960
return "https://serpapi.com";
6061
}
6162

62-
// TODO(seb): get module version too
6363
function getSource() {
64+
const moduleSource = `serpapi@${version}`;
6465
try {
6566
// Check if running in Node.js
6667
// deno-lint-ignore no-explicit-any
6768
if ((globalThis as any)?.process?.version) {
6869
// deno-lint-ignore no-explicit-any
6970
const nodeVersion = (globalThis as any).process.version.replace("v", "");
70-
return `nodejs@${nodeVersion}`;
71+
return `nodejs@${nodeVersion},${moduleSource}`;
7172
}
7273

7374
// Assumes running in Deno instead
74-
return `deno@${Deno.version.deno}`;
75+
return `deno@${Deno.version.deno},${moduleSource}`;
7576
} catch {
7677
// If something unexpectedly occurs, revert to "nodejs".
77-
return "nodejs";
78+
return `nodejs,${moduleSource}`;
7879
}
7980
}
8081

tests/utils_test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,10 +182,10 @@ describe("execute", {
182182

183183
assertSpyCalls(fetchStub, 1);
184184
const url = fetchStub.calls[0].args[0] as string;
185-
// e.g. deno@1.28.2
185+
// e.g. deno@1.28.2,serpapi@1.0.0
186186
assertMatch(
187187
url,
188-
/source=(nodejs|deno)%40\d+\.\d+\.\d+$/,
188+
/source=(nodejs|deno)%40\d+\.\d+\.\d+%2Cserpapi%40\d+\.\d+\.\d+$/,
189189
);
190190
});
191191

0 commit comments

Comments
 (0)