feat: browse artists by area

[lydavid]

No description provided.

[sweep-ai]

Sweep: PR Review

data/database/src/commonMain/kotlin/ly/david/musicsearch/data/database/DatabaseDaoModule.kt

The changes add ArtistsByEntityDao to the import statements and the dependency injection module, ensuring it can be used throughout the application.

---

data/database/src/commonMain/kotlin/ly/david/musicsearch/data/database/dao/ArtistsByEntityDao.kt

A new DAO class ArtistsByEntityDao has been added to manage database operations for the Artists_by_entity table, including insertions, deletions, and queries with support for coroutines and paging.

Sweep Found These Issues

• The getNumberOfArtistsByEntity method uses a hardcoded query string "%%" which may not be necessary and could lead to unexpected results.

MusicSearch/data/database/src/commonMain/kotlin/ly/david/musicsearch/data/database/dao/ArtistsByEntityDao.kt

Line 53 in d401004

query = "%%",


View Diff
• The insertAll method does not handle potential exceptions that could occur during the transaction, which might leave the database in an inconsistent state.

MusicSearch/data/database/src/commonMain/kotlin/ly/david/musicsearch/data/database/dao/ArtistsByEntityDao.kt

Lines 36 to 43 in d401004

	transacter.transaction {
	entityAndArtistIds.forEach { (entityId, artistId) ->
	insert(
	entityId = entityId,
	artistId = artistId,
	)
	}
	}


View Diff

---

data/database/src/commonMain/sqldelight/ly.david.musicsearch.data.database/artists_by_entity.sq

Introduced a new table artists_by_entity and associated SQL operations for inserting, deleting, counting, and retrieving artists by entity.

Sweep Found These Issues

• The deleteArtistsByEntity query incorrectly references the artist table instead of artists_by_entity for deletion, which could lead to unintended data loss.

MusicSearch/data/database/src/commonMain/sqldelight/ly.david.musicsearch.data.database/artists_by_entity.sq

Lines 12 to 18 in d401004

	deleteArtistsByEntity:
	DELETE FROM artist WHERE id IN (
	SELECT a.id
	FROM artists_by_entity ae
	INNER JOIN artist a ON a.id = ae.artist_id
	WHERE ae.entity_id = :entityId
	);


View Diff

---

data/database/src/commonMain/sqldelight/migrations/5.sqm

A new table artists_by_entity is created with entity_id and artist_id as non-null columns and a composite primary key.

---

data/musicbrainz/src/commonMain/kotlin/ly/david/musicsearch/data/musicbrainz/api/BrowseApi.kt

The changes generalize the method for browsing artists to support various entity types by introducing a new parameter for the entity type and updating the request URL construction accordingly.

Sweep Found These Issues

• The change from collectionId to entityId and the addition of entity requires careful handling to ensure that the correct resourceUri is used for different entity types, which could introduce bugs if not properly managed.

MusicSearch/data/musicbrainz/src/commonMain/kotlin/ly/david/musicsearch/data/musicbrainz/api/BrowseApi.kt

Lines 45 to 46 in d401004

	entityId: String,
	entity: MusicBrainzEntity,


View Diff

Potential Issues

Sweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.

• The dynamic construction of the request URL using entity.resourceUri could fail if resourceUri is not correctly defined for all possible MusicBrainzEntity values.

MusicSearch/data/musicbrainz/src/commonMain/kotlin/ly/david/musicsearch/data/musicbrainz/api/BrowseApi.kt

Lines 209 to 210 in d401004

	entity.resourceUri,
	entityId,


View Diff

---

data/repository/src/commonMain/kotlin/ly/david/musicsearch/data/repository/artist/ArtistsByEntityRepositoryImpl.kt

The changes introduce the ArtistsByEntityDao to handle database operations for artists by entity, expanding the functionality of several methods to support more entity types without raising errors.

Sweep Found These Issues

• The deleteLinkedEntitiesByEntity method now deletes artists for all entity types without checking if the entity type is valid, which could lead to unintended deletions.

MusicSearch/data/repository/src/commonMain/kotlin/ly/david/musicsearch/data/repository/artist/ArtistsByEntityRepositoryImpl.kt

Lines 58 to 60 in d401004

	else -> {
	artistsByEntityDao.deleteArtistsByEntity(entityId)
	}


View Diff
• The getLinkedEntitiesPagingSource method now retrieves artists for all entity types without checking if the entity type is valid, which could lead to incorrect data retrieval.

MusicSearch/data/repository/src/commonMain/kotlin/ly/david/musicsearch/data/repository/artist/ArtistsByEntityRepositoryImpl.kt

Lines 78 to 83 in d401004

	else -> {
	artistsByEntityDao.getArtistsByEntity(
	entityId = entityId,
	query = listFilters.query,
	)
	}


View Diff
• The insertAllLinkingModels method now inserts artist-entity links for all entity types without checking if the entity type is valid, which could lead to incorrect data insertion.

MusicSearch/data/repository/src/commonMain/kotlin/ly/david/musicsearch/data/repository/artist/ArtistsByEntityRepositoryImpl.kt

Lines 113 to 119 in d401004

	else -> {
	artistsByEntityDao.insertAll(
	entityAndArtistIds = musicBrainzModels.map { event ->
	entityId to event.id
	},
	)
	}


View Diff

---

docs/all_features.md

The change updates the documentation to reflect that users can now browse artists when viewing the details of an area.

---

shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/DetailsFeatureModule.kt

Added a new dependency artistsByEntityPresenter to the AreaPresenter in the detailsFeatureModule.

---

shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaPresenter.kt

The changes introduce functionality to handle artist-related data within the AreaPresenter by adding new dependencies, initializing UI state, handling events, and updating the overall state management.

Potential Issues

Sweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.

• The new artistsByEntityPresenter dependency is not checked for nullability, which could lead to a NullPointerException if it is not properly initialized.

MusicSearch/shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaPresenter.kt

Line 39 in d401004

private val artistsByEntityPresenter: ArtistsByEntityPresenter,


View Diff

---

shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaTab.kt

A new enum constant ARTISTS has been added to the AreaTab enum class.

---

shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaUi.kt

The changes introduce a new tab for displaying a list of artists associated with an area, using the ArtistsListScreen composable.

Potential Issues

Sweep isn't 100% sure if the following are issues or not but they may be worth taking a look at.

• The lazyListState for the ArtistsListScreen is set to eventsLazyListState, which may cause issues as it should likely have its own distinct state.

MusicSearch/shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaUi.kt

Line 193 in d401004

lazyListState = eventsLazyListState,


View Diff

---

shared/feature/details/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/details/area/AreaUiState.kt

The changes include adding an import statement for ArtistsByEntityUiState and reordering the properties within the AreaUiState data class.

---

shared/feature/stats/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/stats/AreaStatsPresenter.kt

The changes introduce functionality to collect and include artist statistics in the StatsUiState by adding a new DAO dependency and state variables for remote and local artist counts.

---

shared/feature/stats/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/stats/Stats.kt

A new ArtistStats data class has been added and integrated into the Stats data class to include artist-related statistics.

---

shared/feature/stats/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/stats/StatsFeatureModule.kt

A new dependency artistsByEntityDao was added to the AreaStatsPresenter in the statsFeatureModule.

---

shared/feature/stats/src/commonMain/kotlin/ly/david/musicsearch/shared/feature/stats/StatsUi.kt

A new case for Tab.ARTISTS has been added to display artist statistics in the StatsUi function.

---

strings/src/commonMain/kotlin/ly/david/musicsearch/strings/AppStrings.kt

Two new properties, artists and cachedArtists, were added to the AppStrings data class to enhance string resource management and formatting capabilities.

---

strings/src/commonMain/kotlin/ly/david/musicsearch/strings/EnStrings.kt

The changes introduce new localized strings for "Artists" and a template for displaying cached artists information.

---

test-data/src/commonMain/kotlin/ly/david/data/test/api/FakeMusicBrainzApi.kt

The method browseArtistsByCollection was renamed to browseArtistsByEntity, with the parameter collectionId replaced by entityId and a new parameter entity added.

---

ui/common/src/commonMain/kotlin/ly/david/ui/common/topappbar/Tab.kt

The changes introduce a new ARTISTS value to the Tab enum and update the getTitle function to return the appropriate title for this new tab.

---