Repository navigation
feat: sync silo-plugin-metadata-tvdb upstream (v1.3.0) - #10
Merged
Merged
Conversation
* fix: report TVDB artwork text presence Preserve the TVDB includesText field through image metadata so the server can distinguish text-bearing and textless artwork. Use a pointer to retain the difference between false and a missing provider value. * refactor: reuse structFromMap and share artwork test fixtures Address review feedback on #12: build image record metadata through the existing structFromMap helper instead of duplicating its empty-check and discarding the structpb.NewStruct error, and replace the copy-pasted artwork JSON literals in the provider image tests with a shared fixture helper that makes an omitted includesText explicit. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Quick104 <31828688+Quick104@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.79.3 to 1.82.1. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.79.3...v1.82.1) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-version: 1.82.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.53.0 to 0.55.0. - [Commits](golang/net@v0.53.0...v0.55.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-version: 0.55.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat(metadata): return full TVDB season artwork galleries * fix(metadata): validate TVDB season poster candidates * fix(metadata): classify season posters by artwork type and address review findings - Classify season gallery artwork by TVDB season-poster type (7) instead of an aspect-ratio heuristic; keep a landscape guard only when dimensions are known, so typed posters with unknown dimensions are no longer dropped. - Skip appending the primary season poster when it matches artwork the filter rejected, so non-poster primaries cannot bypass the gallery contract. - Extract the season gallery flow into seasonGalleryImages, restoring the shared movie/series artwork loop to its previous shape. - Merge preferPrimaryImage/includePrimaryImage into ensurePrimaryImage with a boost flag, preserving both rating semantics. - Resolve duplicate official seasons deterministically (lowest ID) and share the official-season predicate across all three call sites. - Cache GetSeriesExtended for five minutes, mirroring the episodes cache, so per-season gallery requests reuse one series fetch. - Re-pin the temporary SDK replace to the current head of Silo-Server/silo-plugin-sdk#16 and mark it as blocked on an official tagged release. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: blurbery <41898282+blurbery@users.noreply.github.com> Co-authored-by: Quick104 <31828688+Quick104@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Replace the temporary fork pin with the official SDK release that ships the season-scoped image contract, and bump the manifest version for the season artwork gallery feature release. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sync upstream TVDB metadata improvements while preserving configurable API keys: - Exact TVDB season artwork galleries with extended-series cache - Artwork includes_text reporting - SDK and dependency updates Co-authored-by: Jonah May <JonahMMay@users.noreply.github.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merges Silo TVDB metadata plugin upstream while preserving Prairie's configurable API key model.
Changes
includes_textreportingDependencies
prairie-plugin-sdkseason gallery proto PRVerification
make testAI-assisted contribution.