Creepso/webstreamr-github
1
0
Fork 0
Code Issues 27 Pull requests 7 Projects Releases 240 Packages Wiki Activity Actions

fix(source): consider season and episode for cache key

Browse source
This commit is contained in:
WebStreamr 2025-09-01 20:10:30 +00:00
parent b47a512587
commit ceb520d49d
No known key found for this signature in database
5 changed files with 48 additions and 27 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/source/Source.ts
View file

@ -32,8 +32,9 @@ export abstract class Source {
protected abstract handleInternal(ctx: Context, type: ContentType, id: Id): Promise<(SourceResult[])>;
public async handle(ctx: Context, type: ContentType, id: Id): Promise<(SourceResult[])> {
if (this.sourceResultCache.has(id.id)) {
return this.sourceResultCache.get(id.id) as SourceResult[];
const cacheKey = id.toString();
if (this.sourceResultCache.has(cacheKey)) {
return this.sourceResultCache.get(cacheKey) as SourceResult[];
}
let sourceResults: SourceResult[];
@ -47,7 +48,7 @@ export abstract class Source {
}
}
this.sourceResultCache.set(id.id, sourceResults);
this.sourceResultCache.set(cacheKey, sourceResults);
return sourceResults;
}

30
src/utils/id/ImdbId.test.ts
View file

@ -2,27 +2,33 @@ import { ImdbId } from './ImdbId';
describe('can be created from string', () => {
test('splits id properly', () => {
const { id, season, episode } = ImdbId.fromString('tt2085059:2:4');
const imdbId = ImdbId.fromString('tt2085059:2:4');
expect(id).toBe('tt2085059');
expect(season).toBe(2);
expect(episode).toBe(4);
expect(imdbId.id).toBe('tt2085059');
expect(imdbId.season).toBe(2);
expect(imdbId.episode).toBe(4);
expect(imdbId.toString()).toBe('tt2085059:2:4');
});
test('handles weird 0 prefixes in series and episode', () => {
const { id, season, episode } = ImdbId.fromString('tt2085059:02:04');
const imdbId = ImdbId.fromString('tt2085059:02:04');
expect(id).toBe('tt2085059');
expect(season).toBe(2);
expect(episode).toBe(4);
expect(imdbId.id).toBe('tt2085059');
expect(imdbId.season).toBe(2);
expect(imdbId.episode).toBe(4);
expect(imdbId.toString()).toBe('tt2085059:2:4');
});
test('supports movie with missing series and episode', () => {
const { id, season, episode } = ImdbId.fromString('tt2085059');
const imdbId = ImdbId.fromString('tt2085059');
expect(id).toBe('tt2085059');
expect(season).toBeUndefined();
expect(episode).toBeUndefined();
expect(imdbId.id).toBe('tt2085059');
expect(imdbId.season).toBeUndefined();
expect(imdbId.episode).toBeUndefined();
expect(imdbId.toString()).toBe('tt2085059');
});
test('throws for empty ids', () => {

4
src/utils/id/ImdbId.ts
View file

@ -22,4 +22,8 @@ export class ImdbId {
idParts[2] ? parseInt(idParts[2]) : undefined,
);
}
public toString(): string {
return this.season ? `${this.id}:${this.season}:${this.episode}` : this.id;
}
}

30
src/utils/id/TmdbId.test.ts
View file

@ -2,27 +2,33 @@ import { TmdbId } from './TmdbId';
describe('can be created from string', () => {
test('splits id properly', () => {
const { id, season, episode } = TmdbId.fromString('2085059:2:4');
const tmdbId = TmdbId.fromString('2085059:2:4');
expect(id).toBe(2085059);
expect(season).toBe(2);
expect(episode).toBe(4);
expect(tmdbId.id).toBe(2085059);
expect(tmdbId.season).toBe(2);
expect(tmdbId.episode).toBe(4);
expect(tmdbId.toString()).toBe('2085059:2:4');
});
test('handles weird 0 prefixes in series and episode', () => {
const { id, season, episode } = TmdbId.fromString('2085059:02:04');
const tmdbId = TmdbId.fromString('2085059:02:04');
expect(id).toBe(2085059);
expect(season).toBe(2);
expect(episode).toBe(4);
expect(tmdbId.id).toBe(2085059);
expect(tmdbId.season).toBe(2);
expect(tmdbId.episode).toBe(4);
expect(tmdbId.toString()).toBe('2085059:2:4');
});
test('supports movie with missing series and episode', () => {
const { id, season, episode } = TmdbId.fromString('2085059');
const tmdbId = TmdbId.fromString('2085059');
expect(id).toBe(2085059);
expect(season).toBeUndefined();
expect(episode).toBeUndefined();
expect(tmdbId.id).toBe(2085059);
expect(tmdbId.season).toBeUndefined();
expect(tmdbId.episode).toBeUndefined();
expect(tmdbId.toString()).toBe('2085059');
});
test('throws for empty ids', () => {

4
src/utils/id/TmdbId.ts
View file

@ -22,4 +22,8 @@ export class TmdbId {
idParts[2] ? parseInt(idParts[2]) : undefined,
);
}
public toString(): string {
return this.season ? `${this.id}:${this.season}:${this.episode}` : `${this.id}`;
}
}