Refactor media search & persistence logic

Unify and improve media search/persistence behavior and fix external reference strategy.

- GamesAndVNsApiImpl: switch to Game proto responses, adjust requested fields (name, summary, cover, artworks, genres, alternative names), add genre mapping, and simplify VN filter logic.
- ExternalReferenceModel: change ID generation to GenerationType.IDENTITY.
- MediaServiceImpl: extract shared logic into findGenericMediaByIdOrName and updateIfStale; centralize paging, fallback to external search when local results are missing, and refresh stale records before returning.
- Tests: update MediaServiceImplTest (mocks, setup, and new unit tests for external fallback and stale-refresh behavior) and add MediaServicePersistenceIntegrationTest to verify persistence and reuse of fetched media.

These changes reduce duplicated code, ensure stale data is refreshed automatically, fix ID generation for external references, and add coverage for the new behaviors.

Author: vitorhugo-java

src/main/java/com/espacogeek/geek/data/api/impl/GamesAndVNsApiImpl.java

@@ -31,7 +31,6 @@
 
 import jakarta.annotation.PostConstruct;
 import proto.Game;
-import proto.Search;
 
 import static com.espacogeek.geek.data.api.MediaApi.ApiKey.*;
 
@@ -128,29 +127,39 @@ public MediaModel getDetails(Integer id) {
     @Override
     @Retryable(maxAttempts = 2, backoff = @Backoff(delay = 2000), retryFor = com.espacogeek.geek.exception.RequestException.class)
     public List<MediaModel> doSearch(String search, MediaCategoryModel mediaCategoryModel) {
-        var apicalypse = new APICalypse().search(search).fields("game.age_ratings, game.aggregated_rating, game.alternative_names.name, game.artworks.image_id, game.cover.image_id, game.name").where("game.genres " + (mediaCategoryModel.getId() == MediaDataController.MediaType.GAME.getId() ? "!=" : "=") + " [" + VN_ID_IGDB + "]").limit(10);
+        String vnFilterOperator = mediaCategoryModel.getId() == MediaDataController.MediaType.GAME.getId() ? "!=" : "=";
+        var apicalypse = new APICalypse()
+                .search(search)
+                .fields("name, summary, alternative_names.name, artworks.image_id, cover.image_id, genres.name")
+                .where("genres " + vnFilterOperator + " [" + VN_ID_IGDB + "]")
+                .limit(10);
         List<MediaModel> medias = new ArrayList<>();
 
         try {
-            var searchGames = ProtoRequestKt.search(wrapper, apicalypse);
+            var searchGames = ProtoRequestKt.games(wrapper, apicalypse);
 
-            for (Search result : searchGames) {
-                if ((long) result.getGame().getId() != (long) 0L) {
+            for (Game result : searchGames) {
+                if ((long) result.getId() != (long) 0L) {
                     var media = new MediaModel();
-                    var reference = new ExternalReferenceModel(null, String.valueOf(result.getGame().getId()), media, typeReference);
+                    var reference = new ExternalReferenceModel(null, String.valueOf(result.getId()), media, typeReference);
 
-                    media.setName(result.getGame().getName());
+                    media.setName(result.getName());
+                    media.setAbout(result.getSummary());
                     media.setCover(
-                        !result.getGame().getCover().getImageId().isEmpty()
-                                    ? ImageBuilderKt.imageBuilder(result.getGame().getCover().getImageId(),
+                        !result.getCover().getImageId().isEmpty()
+                                    ? ImageBuilderKt.imageBuilder(result.getCover().getImageId(),
                                             ImageSize.COVER_BIG, ImageType.PNG)
                                     : null);
-                    media.setBanner(result.getGame().getArtworksList().isEmpty() ? null
-                            : ImageBuilderKt.imageBuilder(result.getGame().getArtworksList().getFirst().getImageId(),
+                    media.setBanner(result.getArtworksList().isEmpty() ? null
+                            : ImageBuilderKt.imageBuilder(result.getArtworksList().getFirst().getImageId(),
                                     ImageSize.SCREENSHOT_HUGE, ImageType.PNG));
 
+                    List<String> genresName = new ArrayList<>();
+                    result.getGenresList().forEach(genre -> genresName.add(genre.getName()));
+                    media.setGenre(new java.util.LinkedHashSet<>(genreService.findAllByNames(genresName)));
+
                     var alternativeTitles = new ArrayList<AlternativeTitleModel>();
-                    for (proto.AlternativeName title : result.getGame().getAlternativeNamesList()) {
+                    for (proto.AlternativeName title : result.getAlternativeNamesList()) {
                         if (!title.getName().isEmpty()) alternativeTitles.add(new AlternativeTitleModel(null, title.getName(), media));
                     }
                     media.setAlternativeTitles(new java.util.LinkedHashSet<>(alternativeTitles));

src/main/java/com/espacogeek/geek/models/ExternalReferenceModel.java

@@ -19,8 +19,7 @@
 @AllArgsConstructor
 public class ExternalReferenceModel implements Serializable {
     @Id
-    @GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "gen_externals_references")
-    @SequenceGenerator(name = "gen_externals_references", sequenceName = "seq_externals_references", allocationSize = 50)
+    @GeneratedValue(strategy = GenerationType.IDENTITY)
     @Column(name = "id_external_reference")
     private Integer id;
 

src/main/java/com/espacogeek/geek/services/impl/MediaServiceImpl.java

@@ -146,32 +146,13 @@ public MediaPage findSerieByIdOrName(Integer id, String name, Pageable pageable)
     @SuppressWarnings("unchecked")
     @Override
     public MediaPage findGameByIdOrName(Integer id, String name, Pageable pageable) {
-        Page<MediaModel> results;
-        if (id != null) {
-            List<MediaModel> medias = new ArrayList<MediaModel>();
-            this.mediaRepository.findById(id).ifPresent(media -> medias.add((MediaModel) media));
-            Pageable safePageable = pageable != null ? pageable : Pageable.unpaged();
-            results = new PageImpl<>(medias, safePageable, medias.size());
-        } else {
-            results = this.mediaRepository.findMediaByNameOrAlternativeTitleAndMediaCategory(name, name, mediaCategoryService.findById(MediaDataController.MediaType.GAME.getId()).get().getId(), pageable);
-        }
-        return mountMediaPage(results);
+        return findGenericMediaByIdOrName(id, name, pageable, MediaDataController.MediaType.GAME);
     }
 
     @Override
     @SuppressWarnings("unchecked")
     public MediaPage findVisualNovelByIdOrName(Integer id, String name, Pageable pageable) {
-        Page<MediaModel> results;
-
-        if(id != null) {
-            List<MediaModel> medias = new ArrayList<MediaModel>();
-            this.mediaRepository.findById(id).ifPresent(media -> medias.add((MediaModel) media));
-            Pageable safePageable = pageable != null ? pageable : Pageable.unpaged();
-            results = new PageImpl<>(medias, safePageable, medias.size());
-        } else {
-            results = this.mediaRepository.findMediaByNameOrAlternativeTitleAndMediaCategory(name, name, mediaCategoryService.findById(MediaDataController.MediaType.VN.getId()).get().getId(), pageable);
-        }
-        return mountMediaPage(results);
+        return findGenericMediaByIdOrName(id, name, pageable, MediaDataController.MediaType.VN);
     }
 
     /**
@@ -322,6 +303,59 @@ private MediaModel update(MediaModel media) {
 
     }
 
+    private MediaPage findGenericMediaByIdOrName(Integer id, String name, Pageable pageable, MediaDataController.MediaType mediaType) {
+        Pageable safePageable = pageable != null ? pageable : Pageable.unpaged();
+        var mediaCategory = mediaCategoryService.findById(mediaType.getId()).orElseThrow();
+        Page<MediaModel> results;
+
+        if (id != null) {
+            List<MediaModel> medias = new ArrayList<>();
+            this.mediaRepository.findById(id)
+                    .map(media -> updateIfStale((MediaModel) media))
+                    .ifPresent(medias::add);
+            results = new PageImpl<>(medias, safePageable, medias.size());
+            return mountMediaPage(results);
+        }
+
+        results = this.mediaRepository.findMediaByNameOrAlternativeTitleAndMediaCategory(name, name, mediaCategory.getId(), safePageable);
+
+        if (results.hasContent()) {
+            List<MediaModel> refreshed = results.getContent().stream()
+                    .map(this::updateIfStale)
+                    .toList();
+            return mountMediaPage(new PageImpl<>(refreshed, safePageable, results.getTotalElements()));
+        }
+
+        if (name == null || name.isBlank()) {
+            return mountMediaPage(Page.empty(safePageable));
+        }
+
+        var igdbTypeReference = typeReferenceService.findById(MediaDataController.ExternalReferenceType.IGDB.getId()).orElseThrow();
+        List<MediaModel> fetched = genericMediaDataController.searchMedia(name, gamesAndVNsAPI, igdbTypeReference, mediaCategory);
+
+        if (fetched.isEmpty()) {
+            return mountMediaPage(Page.empty(safePageable));
+        }
+
+        int fromIndex = Math.min((int) safePageable.getOffset(), fetched.size());
+        int toIndex = safePageable.isPaged()
+                ? Math.min(fromIndex + safePageable.getPageSize(), fetched.size())
+                : fetched.size();
+        List<MediaModel> pagedContent = fetched.subList(fromIndex, toIndex);
+
+        return mountMediaPage(new PageImpl<>(pagedContent, safePageable, fetched.size()));
+    }
+
+    private MediaModel updateIfStale(MediaModel media) {
+        if (media == null) {
+            return null;
+        }
+
+        return MediaUtils.updateMediaWhenLastTimeUpdateMoreThanOneDay(media)
+                ? update(media)
+                : media;
+    }
+
     private MediaPage mountMediaPage(Page<MediaModel> medias) {
         MediaPage response = new MediaPage();