Skip to content
Snippets Groups Projects
Unverified Commit 674b757f authored by Sergei Zharinov's avatar Sergei Zharinov Committed by GitHub
Browse files

refactor(deno): Use single cache namespace (#31034)

parent ddcef9cf
Branches
Tags
No related merge requests found
...@@ -37,7 +37,7 @@ export class DenoDatasource extends Datasource { ...@@ -37,7 +37,7 @@ export class DenoDatasource extends Datasource {
namespace: `datasource-${DenoDatasource.id}`, namespace: `datasource-${DenoDatasource.id}`,
key: ({ packageName, registryUrl }: GetReleasesConfig) => key: ({ packageName, registryUrl }: GetReleasesConfig) =>
// TODO: types (#22198) // TODO: types (#22198)
`${registryUrl}:${packageName}`, `getReleases:${registryUrl}:${packageName}`,
}) })
async getReleases({ async getReleases({
packageName, packageName,
...@@ -70,14 +70,15 @@ export class DenoDatasource extends Datasource { ...@@ -70,14 +70,15 @@ export class DenoDatasource extends Datasource {
} }
@cache({ @cache({
namespace: `datasource-${DenoDatasource.id}-versions`, namespace: `datasource-${DenoDatasource.id}`,
key: (moduleAPIURL) => moduleAPIURL, key: (moduleAPIURL) => `getReleaseResult:${moduleAPIURL}`,
}) })
async getReleaseResult(moduleAPIURL: string): Promise<ReleaseResult> { async getReleaseResult(moduleAPIURL: string): Promise<ReleaseResult> {
const detailsCacheKey = `details:${moduleAPIURL}`;
const releasesCache: Record<string, Release> = const releasesCache: Record<string, Release> =
(await packageCache.get( (await packageCache.get(
`datasource-${DenoDatasource.id}-details`, `datasource-${DenoDatasource.id}`,
moduleAPIURL, detailsCacheKey,
)) ?? {}; )) ?? {};
let cacheModified = false; let cacheModified = false;
...@@ -119,8 +120,8 @@ export class DenoDatasource extends Datasource { ...@@ -119,8 +120,8 @@ export class DenoDatasource extends Datasource {
if (cacheModified) { if (cacheModified) {
// 1 week. Releases at Deno are immutable, therefore we can use a long term cache here. // 1 week. Releases at Deno are immutable, therefore we can use a long term cache here.
await packageCache.set( await packageCache.set(
`datasource-${DenoDatasource.id}-details`, `datasource-${DenoDatasource.id}`,
moduleAPIURL, detailsCacheKey,
releasesCache, releasesCache,
10080, 10080,
); );
......
...@@ -41,8 +41,6 @@ export type PackageCacheNamespace = ...@@ -41,8 +41,6 @@ export type PackageCacheNamespace =
| 'datasource-cpan' | 'datasource-cpan'
| 'datasource-crate-metadata' | 'datasource-crate-metadata'
| 'datasource-crate' | 'datasource-crate'
| 'datasource-deno-details'
| 'datasource-deno-versions'
| 'datasource-deno' | 'datasource-deno'
| 'datasource-docker-architecture' | 'datasource-docker-architecture'
| 'datasource-docker-hub-cache' | 'datasource-docker-hub-cache'
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment