Executive summary
shinhyo/Compose-BreakingBad scores 55 out of 100 and is rated Adequate, while four of its six lenses score higher still: Code Health and Performance at 100 and Architecture at 97, all rated Exemplary, and Security and Compliance at 68, rated Adequate.
The score is lower than those four because the weakest lens counts most, and here that is Production Readiness, which scores 39 and is rated Weak. Its Security & performance tooling dimension scores 0 out of 10, and until that reaches 4, Production Readiness can't be rated higher than Adequate. The first actions are to resolve the 1 Leaked secret finding(s) in Secret Scanning and to resolve the 1 No ADRs found finding(s) in ADR Quality, and if Production Readiness rose to 70, the score would rise from 55 to about 65.
22 findings are critical: 20 in Static Analysis (SAST), one in Secret Scanning and one in Secrets (history).
Watchdog couldn't measure 13 dimensions in this code, among them Code Coverage, Test Reliability and Dependency Hygiene, so they are left out of the score. The product code is 4,220 lines, which would take an estimated 0.1 person-years, around €4,600, to rebuild.
If Production Readiness rose from 39 to 70, the lowest score rated Strong, the score would rise from 55 to about 65.
New since the last survey: 2 findings
- Supply-chain Provenance & Signing · REDACTED
- Supply-chain Provenance & Signing · REDACTED
Fixed since the last survey: 1 finding
- Supply-chain Provenance & Signing REDACTED
Rebuild cost: about €4,600 · Estimate
| Cost to rebuild | €1,500 to €7,600, for 25 to 80 hours of work, about 0.1 person-years for one engineer |
|---|---|
| Difficulty of the problem | Standard, judged from the kind of software and how it is built. Code that solves a harder problem, such as payments or medical bookings, costs more per line to write. |
| Quality factor | ×0.8 at a score of 55. Better code takes more work to write, so the factor is ×1.0 at a score of 70, lower below it and higher above it. |
| Size | Small: 4,220 lines of product code |
Top priorities
Diagnosis: what the measurements show together
Architecture — module dependency graph
Architecture: which parts of the code depend on which
| uses → | 1 brba | 2 brba.core.common.di | 3 brba.core.common.result | 4 brba.core.model | 5 brba.core.testing.rule | 6 brba.feature.bottombar | 7 brba.core.common.di.BrbaDispatcher | 8 brba.core.common.result.Result | 9 brba.core.database.model | 10 brba.core.datastore | 11 brba.core.datastore.model | 12 brba.core.domain.repository | 13 brba.core.model.BrbaThemeMode | 14 brba.core.network.model | 15 brba.core.ui | 16 brba.feature.bottombar.Tab | 17 brba.feature.detail.navigaion | 18 brba.feature.favorate.navigation | 19 brba.feature.list.navigation | 20 brba.core.data.model | 21 brba.core.database.dao | 22 brba.core.domain.usecase | 23 brba.core.network | 24 brba.feature.setting | 25 brba.main | 26 brba.core.data.repository | 27 brba.core.database | 28 brba.core.network.di | 29 brba.core.network.retrofit | 30 brba.core.testing | 31 brba.feature.detail | 32 brba.feature.favorate | 33 brba.feature.list | 34 brba.feature.setting.SettingUiState | 35 brba.main.MainUiState | 36 brba.core.data.di | 37 brba.core.database.di | 38 brba.feature.detail.DetailInfoState | 39 brba.feature.favorate.FavoriteUiState | 40 brba.feature.list.ListUiState |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1 brba | ||||||||||||||||||||||||||||||||||||||||
| 2 brba.core.common.di | ||||||||||||||||||||||||||||||||||||||||
| 3 brba.core.common.result | ||||||||||||||||||||||||||||||||||||||||
| 4 brba.core.model | ||||||||||||||||||||||||||||||||||||||||
| 5 brba.core.testing.rule | ||||||||||||||||||||||||||||||||||||||||
| 6 brba.feature.bottombar | ||||||||||||||||||||||||||||||||||||||||
| 7 brba.core.common.di.BrbaDispatcher | 2 | |||||||||||||||||||||||||||||||||||||||
| 8 brba.core.common.result.Result | 3 | |||||||||||||||||||||||||||||||||||||||
| 9 brba.core.database.model | 1 | |||||||||||||||||||||||||||||||||||||||
| 10 brba.core.datastore | 1 | |||||||||||||||||||||||||||||||||||||||
| 11 brba.core.datastore.model | 1 | |||||||||||||||||||||||||||||||||||||||
| 12 brba.core.domain.repository | 3 | |||||||||||||||||||||||||||||||||||||||
| 13 brba.core.model.BrbaThemeMode | 3 | |||||||||||||||||||||||||||||||||||||||
| 14 brba.core.network.model | 1 | |||||||||||||||||||||||||||||||||||||||
| 15 brba.core.ui | 3 | |||||||||||||||||||||||||||||||||||||||
| 16 brba.feature.bottombar.Tab | 3 | |||||||||||||||||||||||||||||||||||||||
| 17 brba.feature.detail.navigaion | 1 | |||||||||||||||||||||||||||||||||||||||
| 18 brba.feature.favorate.navigation | 1 | |||||||||||||||||||||||||||||||||||||||
| 19 brba.feature.list.navigation | 1 | |||||||||||||||||||||||||||||||||||||||
| 20 brba.core.data.model | 1 | |||||||||||||||||||||||||||||||||||||||
| 21 brba.core.database.dao | 1 | |||||||||||||||||||||||||||||||||||||||
| 22 brba.core.domain.usecase | 4 | 4 | ||||||||||||||||||||||||||||||||||||||
| 23 brba.core.network | 2 | |||||||||||||||||||||||||||||||||||||||
| 24 brba.feature.setting | 3 | 1 | ||||||||||||||||||||||||||||||||||||||
| 25 brba.main | 1 | |||||||||||||||||||||||||||||||||||||||
| 26 brba.core.data.repository | 3 | 1 | 2 | 1 | 1 | |||||||||||||||||||||||||||||||||||
| 27 brba.core.database | 1 | |||||||||||||||||||||||||||||||||||||||
| 28 brba.core.network.di | 2 | |||||||||||||||||||||||||||||||||||||||
| 29 brba.core.network.retrofit | 2 | 1 | ||||||||||||||||||||||||||||||||||||||
| 30 brba.core.testing | 1 | 2 | 3 | |||||||||||||||||||||||||||||||||||||
| 31 brba.feature.detail | 2 | 2 | ||||||||||||||||||||||||||||||||||||||
| 32 brba.feature.favorate | 2 | 2 | ||||||||||||||||||||||||||||||||||||||
| 33 brba.feature.list | 4 | 1 | 2 | |||||||||||||||||||||||||||||||||||||
| 34 brba.feature.setting.SettingUiState | 3 | |||||||||||||||||||||||||||||||||||||||
| 35 brba.main.MainUiState | 1 | 2 | ||||||||||||||||||||||||||||||||||||||
| 36 brba.core.data.di | 2 | 2 | ||||||||||||||||||||||||||||||||||||||
| 37 brba.core.database.di | 1 | 2 | ||||||||||||||||||||||||||||||||||||||
| 38 brba.feature.detail.DetailInfoState | 1 | 3 | ||||||||||||||||||||||||||||||||||||||
| 39 brba.feature.favorate.FavoriteUiState | 1 | 4 | ||||||||||||||||||||||||||||||||||||||
| 40 brba.feature.list.ListUiState | 2 | 3 |
7→2 brba.core.common.di.BrbaDispatcher uses brba.core.common.di✕
- Connections
- 2 pairs of types, where a type in brba.core.common.di.BrbaDispatcher uses a type in brba.core.common.di.
- The types involved
io.github.shinhyo.brba.core.common.di.BrbaDispatcher.IO→io.github.shinhyo.brba.core.common.di.BrbaDispatcherio.github.shinhyo.brba.core.common.di.BrbaDispatcher.MAIN→io.github.shinhyo.brba.core.common.di.BrbaDispatcher
- Direction
- brba.core.common.di.BrbaDispatcher uses brba.core.common.di, so a change to brba.core.common.di can break brba.core.common.di.BrbaDispatcher, while a change to brba.core.common.di.BrbaDispatcher can't break brba.core.common.di.
- Position
- Below the diagonal: brba.core.common.di.BrbaDispatcher uses a module listed above it, which is the healthy direction.
8→3 brba.core.common.result.Result uses brba.core.common.result✕
- Connections
- 3 pairs of types, where a type in brba.core.common.result.Result uses a type in brba.core.common.result.
- The types involved
io.github.shinhyo.brba.core.common.result.Result.Error→io.github.shinhyo.brba.core.common.result.Resultio.github.shinhyo.brba.core.common.result.Result.Loading→io.github.shinhyo.brba.core.common.result.Resultio.github.shinhyo.brba.core.common.result.Result.Success→io.github.shinhyo.brba.core.common.result.Result
- Direction
- brba.core.common.result.Result uses brba.core.common.result, so a change to brba.core.common.result can break brba.core.common.result.Result, while a change to brba.core.common.result.Result can't break brba.core.common.result.
- Position
- Below the diagonal: brba.core.common.result.Result uses a module listed above it, which is the healthy direction.
9→4 brba.core.database.model uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.core.database.model uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.database.model.CharacterEntityKt→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.core.database.model uses brba.core.model, so a change to brba.core.model can break brba.core.database.model, while a change to brba.core.database.model can't break brba.core.model.
- Position
- Below the diagonal: brba.core.database.model uses a module listed above it, which is the healthy direction.
10→4 brba.core.datastore uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.core.datastore uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.datastore.DeviceDataSource→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.datastore uses brba.core.model, so a change to brba.core.model can break brba.core.datastore, while a change to brba.core.datastore can't break brba.core.model.
- Position
- Below the diagonal: brba.core.datastore uses a module listed above it, which is the healthy direction.
11→4 brba.core.datastore.model uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.core.datastore.model uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.datastore.model.DeviceData→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.datastore.model uses brba.core.model, so a change to brba.core.model can break brba.core.datastore.model, while a change to brba.core.datastore.model can't break brba.core.model.
- Position
- Below the diagonal: brba.core.datastore.model uses a module listed above it, which is the healthy direction.
12→4 brba.core.domain.repository uses brba.core.model✕
- Connections
- 3 pairs of types, where a type in brba.core.domain.repository uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.domain.repository.CharactersRepository→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.domain.repository.DeviceRepository→io.github.shinhyo.brba.core.model.BrbaDeviceDataio.github.shinhyo.brba.core.domain.repository.DeviceRepository→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.domain.repository uses brba.core.model, so a change to brba.core.model can break brba.core.domain.repository, while a change to brba.core.domain.repository can't break brba.core.model.
- Position
- Below the diagonal: brba.core.domain.repository uses a module listed above it, which is the healthy direction.
13→4 brba.core.model.BrbaThemeMode uses brba.core.model✕
- Connections
- 3 pairs of types, where a type in brba.core.model.BrbaThemeMode uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.model.BrbaThemeMode.Dark→io.github.shinhyo.brba.core.model.BrbaThemeModeio.github.shinhyo.brba.core.model.BrbaThemeMode.Light→io.github.shinhyo.brba.core.model.BrbaThemeModeio.github.shinhyo.brba.core.model.BrbaThemeMode.System→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.model.BrbaThemeMode uses brba.core.model, so a change to brba.core.model can break brba.core.model.BrbaThemeMode, while a change to brba.core.model.BrbaThemeMode can't break brba.core.model.
- Position
- Below the diagonal: brba.core.model.BrbaThemeMode uses a module listed above it, which is the healthy direction.
14→4 brba.core.network.model uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.core.network.model uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.network.model.CharacterResponseKt→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.core.network.model uses brba.core.model, so a change to brba.core.model can break brba.core.network.model, while a change to brba.core.network.model can't break brba.core.model.
- Position
- Below the diagonal: brba.core.network.model uses a module listed above it, which is the healthy direction.
15→4 brba.core.ui uses brba.core.model✕
- Connections
- 3 pairs of types, where a type in brba.core.ui uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.ui.BrbaCharacterCardKt→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.ui.BrbaCharacterRowKt→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.ui.BrbaThemeSelectDialogKt→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.ui uses brba.core.model, so a change to brba.core.model can break brba.core.ui, while a change to brba.core.ui can't break brba.core.model.
- Position
- Below the diagonal: brba.core.ui uses a module listed above it, which is the healthy direction.
16→6 brba.feature.bottombar.Tab uses brba.feature.bottombar✕
- Connections
- 3 pairs of types, where a type in brba.feature.bottombar.Tab uses a type in brba.feature.bottombar.
- The types involved
io.github.shinhyo.brba.feature.bottombar.Tab.FAVORITE→io.github.shinhyo.brba.feature.bottombar.Tabio.github.shinhyo.brba.feature.bottombar.Tab.LIST→io.github.shinhyo.brba.feature.bottombar.Tabio.github.shinhyo.brba.feature.bottombar.Tab.SETTING→io.github.shinhyo.brba.feature.bottombar.Tab
- Direction
- brba.feature.bottombar.Tab uses brba.feature.bottombar, so a change to brba.feature.bottombar can break brba.feature.bottombar.Tab, while a change to brba.feature.bottombar.Tab can't break brba.feature.bottombar.
- Position
- Below the diagonal: brba.feature.bottombar.Tab uses a module listed above it, which is the healthy direction.
17→4 brba.feature.detail.navigaion uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.feature.detail.navigaion uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.detail.navigaion.DetailNavigationKt→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.detail.navigaion uses brba.core.model, so a change to brba.core.model can break brba.feature.detail.navigaion, while a change to brba.feature.detail.navigaion can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.detail.navigaion uses a module listed above it, which is the healthy direction.
18→4 brba.feature.favorate.navigation uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.feature.favorate.navigation uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.favorate.navigation.FavoriteNavigationKt→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.favorate.navigation uses brba.core.model, so a change to brba.core.model can break brba.feature.favorate.navigation, while a change to brba.feature.favorate.navigation can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.favorate.navigation uses a module listed above it, which is the healthy direction.
19→4 brba.feature.list.navigation uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.feature.list.navigation uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.list.navigation.ListNavigationKt→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.list.navigation uses brba.core.model, so a change to brba.core.model can break brba.feature.list.navigation, while a change to brba.feature.list.navigation can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.list.navigation uses a module listed above it, which is the healthy direction.
20→9 brba.core.data.model uses brba.core.database.model✕
- Connections
- 1 pair of types, where a type in brba.core.data.model uses a type in brba.core.database.model.
- The types involved
io.github.shinhyo.brba.core.data.model.CharacterKt→io.github.shinhyo.brba.core.database.model.CharacterEntity
- Direction
- brba.core.data.model uses brba.core.database.model, so a change to brba.core.database.model can break brba.core.data.model, while a change to brba.core.data.model can't break brba.core.database.model.
- Position
- Below the diagonal: brba.core.data.model uses a module listed above it, which is the healthy direction.
21→9 brba.core.database.dao uses brba.core.database.model✕
- Connections
- 1 pair of types, where a type in brba.core.database.dao uses a type in brba.core.database.model.
- The types involved
io.github.shinhyo.brba.core.database.dao.CharacterDao→io.github.shinhyo.brba.core.database.model.CharacterEntity
- Direction
- brba.core.database.dao uses brba.core.database.model, so a change to brba.core.database.model can break brba.core.database.dao, while a change to brba.core.database.dao can't break brba.core.database.model.
- Position
- Below the diagonal: brba.core.database.dao uses a module listed above it, which is the healthy direction.
22→4 brba.core.domain.usecase uses brba.core.model✕
- Connections
- 4 pairs of types, where a type in brba.core.domain.usecase uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.domain.usecase.GetCharacterListUseCase→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.domain.usecase.GetCharacterUseCase→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.domain.usecase.GetFavoriteListUseCase→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.core.domain.usecase uses brba.core.model, so a change to brba.core.model can break brba.core.domain.usecase, while a change to brba.core.domain.usecase can't break brba.core.model.
- Position
- Below the diagonal: brba.core.domain.usecase uses a module listed above it, which is the healthy direction.
22→12 brba.core.domain.usecase uses brba.core.domain.repository✕
- Connections
- 4 pairs of types, where a type in brba.core.domain.usecase uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.core.domain.usecase.GetCharacterListUseCase→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.domain.usecase.GetCharacterUseCase→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.domain.usecase.GetFavoriteListUseCase→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase→io.github.shinhyo.brba.core.domain.repository.CharactersRepository
- Direction
- brba.core.domain.usecase uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.core.domain.usecase, while a change to brba.core.domain.usecase can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.core.domain.usecase uses a module listed above it, which is the healthy direction.
23→14 brba.core.network uses brba.core.network.model✕
- Connections
- 2 pairs of types, where a type in brba.core.network uses a type in brba.core.network.model.
- The types involved
io.github.shinhyo.brba.core.network.LocalDataSource→io.github.shinhyo.brba.core.network.model.CharacterResponseio.github.shinhyo.brba.core.network.NetworkDataSource→io.github.shinhyo.brba.core.network.model.CharacterResponse
- Direction
- brba.core.network uses brba.core.network.model, so a change to brba.core.network.model can break brba.core.network, while a change to brba.core.network can't break brba.core.network.model.
- Position
- Below the diagonal: brba.core.network uses a module listed above it, which is the healthy direction.
24→4 brba.feature.setting uses brba.core.model✕
- Connections
- 3 pairs of types, where a type in brba.feature.setting uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.setting.SettingData→io.github.shinhyo.brba.core.model.BrbaDeviceDataio.github.shinhyo.brba.feature.setting.SettingScreenKt→io.github.shinhyo.brba.core.model.BrbaThemeModeio.github.shinhyo.brba.feature.setting.SettingViewModel→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.feature.setting uses brba.core.model, so a change to brba.core.model can break brba.feature.setting, while a change to brba.feature.setting can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.setting uses a module listed above it, which is the healthy direction.
24→12 brba.feature.setting uses brba.core.domain.repository✕
- Connections
- 1 pair of types, where a type in brba.feature.setting uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.feature.setting.SettingViewModel→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.feature.setting uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.feature.setting, while a change to brba.feature.setting can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.feature.setting uses a module listed above it, which is the healthy direction.
25→12 brba.main uses brba.core.domain.repository✕
- Connections
- 1 pair of types, where a type in brba.main uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.main.MainViewModel→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.main uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.main, while a change to brba.main can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.main uses a module listed above it, which is the healthy direction.
26→4 brba.core.data.repository uses brba.core.model✕
- Connections
- 3 pairs of types, where a type in brba.core.data.repository uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.core.data.repository.CharactersRepositoryImpl→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.core.data.repository.DeviceRepositoryImpl→io.github.shinhyo.brba.core.model.BrbaDeviceDataio.github.shinhyo.brba.core.data.repository.DeviceRepositoryImpl→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.core.data.repository uses brba.core.model, so a change to brba.core.model can break brba.core.data.repository, while a change to brba.core.data.repository can't break brba.core.model.
- Position
- Below the diagonal: brba.core.data.repository uses a module listed above it, which is the healthy direction.
26→10 brba.core.data.repository uses brba.core.datastore✕
- Connections
- 1 pair of types, where a type in brba.core.data.repository uses a type in brba.core.datastore.
- The types involved
io.github.shinhyo.brba.core.data.repository.DeviceRepositoryImpl→io.github.shinhyo.brba.core.datastore.DeviceDataSource
- Direction
- brba.core.data.repository uses brba.core.datastore, so a change to brba.core.datastore can break brba.core.data.repository, while a change to brba.core.data.repository can't break brba.core.datastore.
- Position
- Below the diagonal: brba.core.data.repository uses a module listed above it, which is the healthy direction.
26→12 brba.core.data.repository uses brba.core.domain.repository✕
- Connections
- 2 pairs of types, where a type in brba.core.data.repository uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.core.data.repository.CharactersRepositoryImpl→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.data.repository.DeviceRepositoryImpl→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.core.data.repository uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.core.data.repository, while a change to brba.core.data.repository can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.core.data.repository uses a module listed above it, which is the healthy direction.
26→21 brba.core.data.repository uses brba.core.database.dao✕
- Connections
- 1 pair of types, where a type in brba.core.data.repository uses a type in brba.core.database.dao.
- The types involved
io.github.shinhyo.brba.core.data.repository.CharactersRepositoryImpl→io.github.shinhyo.brba.core.database.dao.CharacterDao
- Direction
- brba.core.data.repository uses brba.core.database.dao, so a change to brba.core.database.dao can break brba.core.data.repository, while a change to brba.core.data.repository can't break brba.core.database.dao.
- Position
- Below the diagonal: brba.core.data.repository uses a module listed above it, which is the healthy direction.
26→23 brba.core.data.repository uses brba.core.network✕
- Connections
- 1 pair of types, where a type in brba.core.data.repository uses a type in brba.core.network.
- The types involved
io.github.shinhyo.brba.core.data.repository.CharactersRepositoryImpl→io.github.shinhyo.brba.core.network.NetworkDataSource
- Direction
- brba.core.data.repository uses brba.core.network, so a change to brba.core.network can break brba.core.data.repository, while a change to brba.core.data.repository can't break brba.core.network.
- Position
- Below the diagonal: brba.core.data.repository uses a module listed above it, which is the healthy direction.
27→21 brba.core.database uses brba.core.database.dao✕
- Connections
- 1 pair of types, where a type in brba.core.database uses a type in brba.core.database.dao.
- The types involved
io.github.shinhyo.brba.core.database.AppDatabase→io.github.shinhyo.brba.core.database.dao.CharacterDao
- Direction
- brba.core.database uses brba.core.database.dao, so a change to brba.core.database.dao can break brba.core.database, while a change to brba.core.database can't break brba.core.database.dao.
- Position
- Below the diagonal: brba.core.database uses a module listed above it, which is the healthy direction.
28→23 brba.core.network.di uses brba.core.network✕
- Connections
- 2 pairs of types, where a type in brba.core.network.di uses a type in brba.core.network.
- The types involved
io.github.shinhyo.brba.core.network.di.NetworkModule→io.github.shinhyo.brba.core.network.LocalDataSourceio.github.shinhyo.brba.core.network.di.NetworkModule→io.github.shinhyo.brba.core.network.NetworkDataSource
- Direction
- brba.core.network.di uses brba.core.network, so a change to brba.core.network can break brba.core.network.di, while a change to brba.core.network.di can't break brba.core.network.
- Position
- Below the diagonal: brba.core.network.di uses a module listed above it, which is the healthy direction.
29→14 brba.core.network.retrofit uses brba.core.network.model✕
- Connections
- 2 pairs of types, where a type in brba.core.network.retrofit uses a type in brba.core.network.model.
- The types involved
io.github.shinhyo.brba.core.network.retrofit.BaBrApi→io.github.shinhyo.brba.core.network.model.CharacterResponseio.github.shinhyo.brba.core.network.retrofit.RetrofitNetwork→io.github.shinhyo.brba.core.network.model.CharacterResponse
- Direction
- brba.core.network.retrofit uses brba.core.network.model, so a change to brba.core.network.model can break brba.core.network.retrofit, while a change to brba.core.network.retrofit can't break brba.core.network.model.
- Position
- Below the diagonal: brba.core.network.retrofit uses a module listed above it, which is the healthy direction.
29→23 brba.core.network.retrofit uses brba.core.network✕
- Connections
- 1 pair of types, where a type in brba.core.network.retrofit uses a type in brba.core.network.
- The types involved
io.github.shinhyo.brba.core.network.retrofit.RetrofitNetwork→io.github.shinhyo.brba.core.network.NetworkDataSource
- Direction
- brba.core.network.retrofit uses brba.core.network, so a change to brba.core.network can break brba.core.network.retrofit, while a change to brba.core.network.retrofit can't break brba.core.network.
- Position
- Below the diagonal: brba.core.network.retrofit uses a module listed above it, which is the healthy direction.
30→5 brba.core.testing uses brba.core.testing.rule✕
- Connections
- 1 pair of types, where a type in brba.core.testing uses a type in brba.core.testing.rule.
- The types involved
io.github.shinhyo.brba.core.testing.ViewModelTest→io.github.shinhyo.brba.core.testing.rule.MainDispatcherRule
- Direction
- brba.core.testing uses brba.core.testing.rule, so a change to brba.core.testing.rule can break brba.core.testing, while a change to brba.core.testing can't break brba.core.testing.rule.
- Position
- Below the diagonal: brba.core.testing uses a module listed above it, which is the healthy direction.
30→12 brba.core.testing uses brba.core.domain.repository✕
- Connections
- 2 pairs of types, where a type in brba.core.testing uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.core.testing.BaseViewModelTest→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.testing.BaseViewModelTest→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.core.testing uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.core.testing, while a change to brba.core.testing can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.core.testing uses a module listed above it, which is the healthy direction.
30→22 brba.core.testing uses brba.core.domain.usecase✕
- Connections
- 3 pairs of types, where a type in brba.core.testing uses a type in brba.core.domain.usecase.
- The types involved
io.github.shinhyo.brba.core.testing.BaseViewModelTest→io.github.shinhyo.brba.core.domain.usecase.GetCharacterListUseCaseio.github.shinhyo.brba.core.testing.BaseViewModelTest→io.github.shinhyo.brba.core.domain.usecase.GetFavoriteListUseCaseio.github.shinhyo.brba.core.testing.BaseViewModelTest→io.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase
- Direction
- brba.core.testing uses brba.core.domain.usecase, so a change to brba.core.domain.usecase can break brba.core.testing, while a change to brba.core.testing can't break brba.core.domain.usecase.
- Position
- Below the diagonal: brba.core.testing uses a module listed above it, which is the healthy direction.
31→4 brba.feature.detail uses brba.core.model✕
- Connections
- 2 pairs of types, where a type in brba.feature.detail uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.detail.DetailScreenKt→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.feature.detail.DetailViewModel→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.detail uses brba.core.model, so a change to brba.core.model can break brba.feature.detail, while a change to brba.feature.detail can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.detail uses a module listed above it, which is the healthy direction.
31→22 brba.feature.detail uses brba.core.domain.usecase✕
- Connections
- 2 pairs of types, where a type in brba.feature.detail uses a type in brba.core.domain.usecase.
- The types involved
io.github.shinhyo.brba.feature.detail.DetailViewModel→io.github.shinhyo.brba.core.domain.usecase.GetCharacterUseCaseio.github.shinhyo.brba.feature.detail.DetailViewModel→io.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase
- Direction
- brba.feature.detail uses brba.core.domain.usecase, so a change to brba.core.domain.usecase can break brba.feature.detail, while a change to brba.feature.detail can't break brba.core.domain.usecase.
- Position
- Below the diagonal: brba.feature.detail uses a module listed above it, which is the healthy direction.
32→4 brba.feature.favorate uses brba.core.model✕
- Connections
- 2 pairs of types, where a type in brba.feature.favorate uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.favorate.FavoriteScreenKt→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.feature.favorate.FavoriteViewModel→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.favorate uses brba.core.model, so a change to brba.core.model can break brba.feature.favorate, while a change to brba.feature.favorate can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.favorate uses a module listed above it, which is the healthy direction.
32→22 brba.feature.favorate uses brba.core.domain.usecase✕
- Connections
- 2 pairs of types, where a type in brba.feature.favorate uses a type in brba.core.domain.usecase.
- The types involved
io.github.shinhyo.brba.feature.favorate.FavoriteViewModel→io.github.shinhyo.brba.core.domain.usecase.GetFavoriteListUseCaseio.github.shinhyo.brba.feature.favorate.FavoriteViewModel→io.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase
- Direction
- brba.feature.favorate uses brba.core.domain.usecase, so a change to brba.core.domain.usecase can break brba.feature.favorate, while a change to brba.feature.favorate can't break brba.core.domain.usecase.
- Position
- Below the diagonal: brba.feature.favorate uses a module listed above it, which is the healthy direction.
33→4 brba.feature.list uses brba.core.model✕
- Connections
- 4 pairs of types, where a type in brba.feature.list uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.list.ListScreenKt→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.feature.list.ListScreenKt→io.github.shinhyo.brba.core.model.BrbaThemeModeio.github.shinhyo.brba.feature.list.ListViewModel→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.feature.list.ListViewModel→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.feature.list uses brba.core.model, so a change to brba.core.model can break brba.feature.list, while a change to brba.feature.list can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.list uses a module listed above it, which is the healthy direction.
33→12 brba.feature.list uses brba.core.domain.repository✕
- Connections
- 1 pair of types, where a type in brba.feature.list uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.feature.list.ListViewModel→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.feature.list uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.feature.list, while a change to brba.feature.list can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.feature.list uses a module listed above it, which is the healthy direction.
33→22 brba.feature.list uses brba.core.domain.usecase✕
- Connections
- 2 pairs of types, where a type in brba.feature.list uses a type in brba.core.domain.usecase.
- The types involved
io.github.shinhyo.brba.feature.list.ListViewModel→io.github.shinhyo.brba.core.domain.usecase.GetCharacterListUseCaseio.github.shinhyo.brba.feature.list.ListViewModel→io.github.shinhyo.brba.core.domain.usecase.UpdateFavoriteUseCase
- Direction
- brba.feature.list uses brba.core.domain.usecase, so a change to brba.core.domain.usecase can break brba.feature.list, while a change to brba.feature.list can't break brba.core.domain.usecase.
- Position
- Below the diagonal: brba.feature.list uses a module listed above it, which is the healthy direction.
34→24 brba.feature.setting.SettingUiState uses brba.feature.setting✕
- Connections
- 3 pairs of types, where a type in brba.feature.setting.SettingUiState uses a type in brba.feature.setting.
- The types involved
io.github.shinhyo.brba.feature.setting.SettingUiState.Loading→io.github.shinhyo.brba.feature.setting.SettingUiStateio.github.shinhyo.brba.feature.setting.SettingUiState.Success→io.github.shinhyo.brba.feature.setting.SettingDataio.github.shinhyo.brba.feature.setting.SettingUiState.Success→io.github.shinhyo.brba.feature.setting.SettingUiState
- Direction
- brba.feature.setting.SettingUiState uses brba.feature.setting, so a change to brba.feature.setting can break brba.feature.setting.SettingUiState, while a change to brba.feature.setting.SettingUiState can't break brba.feature.setting.
- Position
- Below the diagonal: brba.feature.setting.SettingUiState uses a module listed above it, which is the healthy direction.
35→4 brba.main.MainUiState uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.main.MainUiState uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.main.MainUiState.Success→io.github.shinhyo.brba.core.model.BrbaDeviceData
- Direction
- brba.main.MainUiState uses brba.core.model, so a change to brba.core.model can break brba.main.MainUiState, while a change to brba.main.MainUiState can't break brba.core.model.
- Position
- Below the diagonal: brba.main.MainUiState uses a module listed above it, which is the healthy direction.
35→25 brba.main.MainUiState uses brba.main✕
- Connections
- 2 pairs of types, where a type in brba.main.MainUiState uses a type in brba.main.
- The types involved
io.github.shinhyo.brba.main.MainUiState.Loading→io.github.shinhyo.brba.main.MainUiStateio.github.shinhyo.brba.main.MainUiState.Success→io.github.shinhyo.brba.main.MainUiState
- Direction
- brba.main.MainUiState uses brba.main, so a change to brba.main can break brba.main.MainUiState, while a change to brba.main.MainUiState can't break brba.main.
- Position
- Below the diagonal: brba.main.MainUiState uses a module listed above it, which is the healthy direction.
36→12 brba.core.data.di uses brba.core.domain.repository✕
- Connections
- 2 pairs of types, where a type in brba.core.data.di uses a type in brba.core.domain.repository.
- The types involved
io.github.shinhyo.brba.core.data.di.RepositoryModule→io.github.shinhyo.brba.core.domain.repository.CharactersRepositoryio.github.shinhyo.brba.core.data.di.RepositoryModule→io.github.shinhyo.brba.core.domain.repository.DeviceRepository
- Direction
- brba.core.data.di uses brba.core.domain.repository, so a change to brba.core.domain.repository can break brba.core.data.di, while a change to brba.core.data.di can't break brba.core.domain.repository.
- Position
- Below the diagonal: brba.core.data.di uses a module listed above it, which is the healthy direction.
36→26 brba.core.data.di uses brba.core.data.repository✕
- Connections
- 2 pairs of types, where a type in brba.core.data.di uses a type in brba.core.data.repository.
- The types involved
io.github.shinhyo.brba.core.data.di.RepositoryModule→io.github.shinhyo.brba.core.data.repository.CharactersRepositoryImplio.github.shinhyo.brba.core.data.di.RepositoryModule→io.github.shinhyo.brba.core.data.repository.DeviceRepositoryImpl
- Direction
- brba.core.data.di uses brba.core.data.repository, so a change to brba.core.data.repository can break brba.core.data.di, while a change to brba.core.data.di can't break brba.core.data.repository.
- Position
- Below the diagonal: brba.core.data.di uses a module listed above it, which is the healthy direction.
37→21 brba.core.database.di uses brba.core.database.dao✕
- Connections
- 1 pair of types, where a type in brba.core.database.di uses a type in brba.core.database.dao.
- The types involved
io.github.shinhyo.brba.core.database.di.DaoModule→io.github.shinhyo.brba.core.database.dao.CharacterDao
- Direction
- brba.core.database.di uses brba.core.database.dao, so a change to brba.core.database.dao can break brba.core.database.di, while a change to brba.core.database.di can't break brba.core.database.dao.
- Position
- Below the diagonal: brba.core.database.di uses a module listed above it, which is the healthy direction.
37→27 brba.core.database.di uses brba.core.database✕
- Connections
- 2 pairs of types, where a type in brba.core.database.di uses a type in brba.core.database.
- The types involved
io.github.shinhyo.brba.core.database.di.DaoModule→io.github.shinhyo.brba.core.database.AppDatabaseio.github.shinhyo.brba.core.database.di.DatabaseModule→io.github.shinhyo.brba.core.database.AppDatabase
- Direction
- brba.core.database.di uses brba.core.database, so a change to brba.core.database can break brba.core.database.di, while a change to brba.core.database.di can't break brba.core.database.
- Position
- Below the diagonal: brba.core.database.di uses a module listed above it, which is the healthy direction.
38→4 brba.feature.detail.DetailInfoState uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.feature.detail.DetailInfoState uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.detail.DetailInfoState.Success→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.detail.DetailInfoState uses brba.core.model, so a change to brba.core.model can break brba.feature.detail.DetailInfoState, while a change to brba.feature.detail.DetailInfoState can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.detail.DetailInfoState uses a module listed above it, which is the healthy direction.
38→31 brba.feature.detail.DetailInfoState uses brba.feature.detail✕
- Connections
- 3 pairs of types, where a type in brba.feature.detail.DetailInfoState uses a type in brba.feature.detail.
- The types involved
io.github.shinhyo.brba.feature.detail.DetailInfoState.Error→io.github.shinhyo.brba.feature.detail.DetailInfoStateio.github.shinhyo.brba.feature.detail.DetailInfoState.Loading→io.github.shinhyo.brba.feature.detail.DetailInfoStateio.github.shinhyo.brba.feature.detail.DetailInfoState.Success→io.github.shinhyo.brba.feature.detail.DetailInfoState
- Direction
- brba.feature.detail.DetailInfoState uses brba.feature.detail, so a change to brba.feature.detail can break brba.feature.detail.DetailInfoState, while a change to brba.feature.detail.DetailInfoState can't break brba.feature.detail.
- Position
- Below the diagonal: brba.feature.detail.DetailInfoState uses a module listed above it, which is the healthy direction.
39→4 brba.feature.favorate.FavoriteUiState uses brba.core.model✕
- Connections
- 1 pair of types, where a type in brba.feature.favorate.FavoriteUiState uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.favorate.FavoriteUiState.Success→io.github.shinhyo.brba.core.model.BrbaCharacter
- Direction
- brba.feature.favorate.FavoriteUiState uses brba.core.model, so a change to brba.core.model can break brba.feature.favorate.FavoriteUiState, while a change to brba.feature.favorate.FavoriteUiState can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.favorate.FavoriteUiState uses a module listed above it, which is the healthy direction.
39→32 brba.feature.favorate.FavoriteUiState uses brba.feature.favorate✕
- Connections
- 4 pairs of types, where a type in brba.feature.favorate.FavoriteUiState uses a type in brba.feature.favorate.
- The types involved
io.github.shinhyo.brba.feature.favorate.FavoriteUiState.Empty→io.github.shinhyo.brba.feature.favorate.FavoriteUiStateio.github.shinhyo.brba.feature.favorate.FavoriteUiState.Error→io.github.shinhyo.brba.feature.favorate.FavoriteUiStateio.github.shinhyo.brba.feature.favorate.FavoriteUiState.Loading→io.github.shinhyo.brba.feature.favorate.FavoriteUiStateio.github.shinhyo.brba.feature.favorate.FavoriteUiState.Success→io.github.shinhyo.brba.feature.favorate.FavoriteUiState
- Direction
- brba.feature.favorate.FavoriteUiState uses brba.feature.favorate, so a change to brba.feature.favorate can break brba.feature.favorate.FavoriteUiState, while a change to brba.feature.favorate.FavoriteUiState can't break brba.feature.favorate.
- Position
- Below the diagonal: brba.feature.favorate.FavoriteUiState uses a module listed above it, which is the healthy direction.
40→4 brba.feature.list.ListUiState uses brba.core.model✕
- Connections
- 2 pairs of types, where a type in brba.feature.list.ListUiState uses a type in brba.core.model.
- The types involved
io.github.shinhyo.brba.feature.list.ListUiState.Success→io.github.shinhyo.brba.core.model.BrbaCharacterio.github.shinhyo.brba.feature.list.ListUiState.Success→io.github.shinhyo.brba.core.model.BrbaThemeMode
- Direction
- brba.feature.list.ListUiState uses brba.core.model, so a change to brba.core.model can break brba.feature.list.ListUiState, while a change to brba.feature.list.ListUiState can't break brba.core.model.
- Position
- Below the diagonal: brba.feature.list.ListUiState uses a module listed above it, which is the healthy direction.
40→33 brba.feature.list.ListUiState uses brba.feature.list✕
- Connections
- 3 pairs of types, where a type in brba.feature.list.ListUiState uses a type in brba.feature.list.
- The types involved
io.github.shinhyo.brba.feature.list.ListUiState.Error→io.github.shinhyo.brba.feature.list.ListUiStateio.github.shinhyo.brba.feature.list.ListUiState.Loading→io.github.shinhyo.brba.feature.list.ListUiStateio.github.shinhyo.brba.feature.list.ListUiState.Success→io.github.shinhyo.brba.feature.list.ListUiState
- Direction
- brba.feature.list.ListUiState uses brba.feature.list, so a change to brba.feature.list can break brba.feature.list.ListUiState, while a change to brba.feature.list.ListUiState can't break brba.feature.list.
- Position
- Below the diagonal: brba.feature.list.ListUiState uses a module listed above it, which is the healthy direction.
At a glance — Code Health
At a glance — Architecture
At a glance — Maturity
At a glance — Readiness
At a glance — Security
At a glance — Performance
Security & Compliance — OWASP Top-10 mapping
| OWASP category | Findings | Severity |
|---|---|---|
| A05:2025 — Injection | 21 | High / Critical |
| A03:2025 — Software Supply Chain Failures | 7 | Medium |
| A07:2025 — Authentication Failures | 2 | High / Critical |
Roadmap
Immediately resolve the leaked REDACTED secret and address the 21 orphaned files in the codebase to eliminate security risks and technical debt. Establish a CI security gate using SAST tools to prevent regressions and document architectural decisions with an ADR. Finally, harden outbound HTTP calls by enforcing strict timeouts and implementing resilience patterns like retries and circuit breakers.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Resolve the 1 Leaked secret finding(s) in Secret Scanning — start with REDACTED. | +13.2 pts | Low | Secret Scanning |
| Resolve the 1 No ADRs found finding(s) in ADR Quality. | +6.4 pts | Low | ADR Quality |
| Resolve the 21 Orphaned file finding(s) in Knowledge Freshness — start with DetailScreen.kt, ListScreen.kt, Theme.kt. | +8.2 pts | Medium | Knowledge Freshness |
| Add a SAST step to CI running what this repository's stack ships: spotbugs with find-sec-bugs — or `semgrep --config=auto`, which runs on any language — so a security regression fails the build instead of landing. | +8.0 pts | Medium | Security & performance tooling |
| Set connect and read timeouts on every client (`RestTemplateBuilder.connectTimeout/readTimeout`, WebClient `responseTimeout`, `HttpRequest.Builder.timeout`, `HttpURLConnection.setReadTimeout`, Feign `connectTimeout`/`readTimeout` in configuration), and wrap calls to dependencies in resilience4j `@Retry`/`@CircuitBreaker`/`@TimeLimiter` (or Spring Retry). | +5.1 pts | Medium | Outbound HTTP resilience |
| Record your significant design decisions. Write one short document per decision, giving its date, the situation, the decision and its consequences, and keep them together with your other design documents. A docs/adr folder with each file named by number and title is the easiest place to find them. | +4.3 pts | Medium | Architecture documentation |
| Nothing pauses a release for a human: publish as a draft release (or gate the release job on a protected tag/manual dispatch) so a bad build can be stopped before users can download it. | +3.0 pts | Medium | Deployment & Rollback |
| Improve Documentation Quality — currently 7.0/10. | +2.5 pts | Medium | Documentation Quality |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| REDACTED | 1.3 | Slop | Static Analysis (SAST): High: REDACTED |
| REDACTED | 2.3 | Slop | Static Analysis (SAST): High: REDACTED |
| REDACTED | 4.4 | Mixed | Secret Scanning: Leaked secret: REDACTED |
| feature/detail/src/main/kotlin/io/github/shinhyo/brba/feature/detail/DetailScreen.kt | 8.2 | Near-clean | Explicit Debt: TodoComment |
How the grades work
A definite problem that already costs you something and drags the score down: a missing authorisation check, a dependency with a known exploit, a build that does not reproduce. Failure here tends to cause failures elsewhere.
Likely wrong, but not failing yet. It degrades the codebase over a longer horizon and can cause failures elsewhere — not urgent this week, not something to carry for two years either.
Recorded, with no effect on how the codebase functions. Present so the survey is complete, not because it needs doing.
Something this survey could not settle from the outside, and which could be critical or serious. Either a control was required and no positive evidence of it exists in the repository — a backup job that nothing shows was ever restored from proves nothing about restores — or our own analysis could not run over that part of the tree. This is not a clean result. These are excluded from the score rather than awarded a pass, so the number on the cover neither rewards nor penalises them: if you act on this survey without resolving them, you carry that risk yourself. Each one is named under Limitations.
Methodology & how to trust this report
Watchdog is a deep, periodic assessment — run each sprint, monthly, or quarterly, taking the time to go wider and deeper than a quick check and surfacing in one coherent report what you'd otherwise piece together from a dozen separate tools. It scores deterministically: the same commit yields the same score, every run. 51 of 54 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 3 documentation/naming judgement(s) are LLM-assisted and labelled advisory. Overall confidence is 0.8 — the weighted average across measured dimensions; it falls as more of the score leans on LLM-assisted judgement and rises when it's fully tool-backed.
What we checked — 54 dimensions across the health lenses
- Can you open the finding? Real findings cite a repo-relative file and line you can open at the cited line — never an absolute scratch path. Here, 48 of 64 do; the remainder are repo-wide signals — a dimension-level measurement, not a single line.
- Is there a tool behind the number? Every score below names the method that produced it — Roslyn, git, a scanner, or (for a handful of documentation/naming dimensions) an LLM labelled sampled · advisory — not a narrative.
- Does re-running give the same result? Run it again on the same commit and the score — and this report, byte for byte — is identical. A report whose numbers move between runs is describing the run, not the code.
Tools & methods
| Method | Backs | Version | Evaluator |
|---|---|---|---|
| Roslyn static analysis | Complexity, cohesion, coupling, dead code, API surface, layering | 5.3.0 | ✓ deterministic |
| Native secret scanner | Hardcoded secrets / credentials | 1.0.0 | ✓ deterministic |
| Watchdog duplication detector (in-process) | Code duplication | 1.0.0 | ✓ deterministic |
| Coverage (coverlet / dotnet-coverage) | Line & branch coverage | 10.0.400 | ✓ deterministic |
| NuGet / dotnet | Outdated, vulnerable & deprecated dependencies | 10.0.400 | ✓ deterministic |
| git / LibGit2Sharp | Churn hotspots, knowledge concentration, history | 2.43.0 · 0.31.0 | ✓ deterministic |
| gitleaks · semgrep · trivy | Secrets in history, SAST, CVEs, IaC & container, PII / GDPR | 1.86.0 · 0.69.3 | ✓ deterministic |
| LLM (sampled · advisory) | Documentation quality, ADR conformance, naming — sampled over a bounded sample; advisory, never a deterministic measurement | Local LLM | ◐ LLM · sampled · advisory |
Run transparency — what happened this run
- D8 Code Coverage — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Coverage NOT READ here — but this repository measures it: a coverage step in CI (`coverage report`) shows that coverage is collected and tracked in your own CI. The built-in collector has no runner for this ecosystem (.kt), so the analyzer could not read the number — a gap in the analyzer's language coverage, not an unmeasured repo. Not scored. To have the real number read, produce a coverage report in a standard format (JaCoCo XML — the Gradle `jacocoTestReport` task) and commit it — a hosted scan measures a clone of the repository, so a report that exists only in a working tree, a CI runner's or your own, never reaches it; the artefact is commonly gitignored, so `git add -f` that one file (or un-ignore its path) and commit it alongside the code it measures. You can widen what we reach: optional: produce a coverage report in a standard format (JaCoCo XML — the Gradle `jacocoTestReport` task) and commit it — a hosted scan measures a clone of the repository, so a report that exists only in a working tree, a CI runner's or your own, never reaches it; the artefact is commonly gitignored, so `git add -f` that one file (or un-ignore its path) and commit it alongside the code it measures — then the real number is read on the next scan.
- D11 Test Reliability — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Test source is present (.kt) and this repository declares a JVM test suite (repository root, Gradle), but it was not re-run: no test result was produced. Not scored — this is a gap in the analyzer's language coverage, not a finding about this repository.
- D12 Dependency Hygiene — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored — 19 Maven/Gradle manifest(s) were read, stating a version for 45 dependency(ies) and leaving 12 to a parent or an imported BOM, but no build resolved the dependency graph, so the outdated signal was not established. NOT a finding that these dependencies are current or healthy.
- D16 Bus Factor — not measured this run — This is a true statement about the repository that carries nothing for its owner to act on, so it is reported here rather than as a defect in their code. Single-contributor repository — bus factor is not applicable (1 contributor(s) across 113 commit(s) sampled, automation and bot accounts excluded). Concentration needs a team to concentrate: with one contributor there is nobody to spread the knowledge to, so there is nothing here for the owner to act on.
- D22 Internal API Consistency — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. D22 identifies the intentionally-exposed surface from `IsPackable` and `.Contracts` project names, MSBuild conventions read off the loaded project set. This target exposed no such projects, so the probe never ran; this says nothing about whether the repository has a public API. This repository commits no C#/VB source at all, so there was never an MSBuild project set to read these conventions off. That is OUR side and it is an ADMISSION gap, not an environment fault: it declares a published package (core/common/build.gradle.kts), but no published-package marker D22 reads admitted any project here. D22 DOES collect the public API of every language in this tree, and admits a project only when its manifest makes that ecosystem's publication act in a form the marker recognises — so what is missing is not a collector but an admission rule for the publication act this repository makes, if it makes one. The remedy is a better discriminator for that marker — not a collector, and no change to the scan image can close it. The marker's rules are measured to refuse applications, so one refused repository is the rule working as calibrated; several are the evidence that it is too strict.
- D27 Navigability — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Navigability is taken over call sites whose callee resolved. For C# that population includes calls into referenced assemblies; for this language only calls declared inside the repository count, so the abstention threshold reads the repository's use of its standard library as a resolution failure. That is a limit of this analyzer, not a build problem in this repository.
- D32 Data Compliance (PII/GDPR) — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. `core/database/src/main/kotlin/io/github/shinhyo/brba/core/database/AppDatabase.kt` produced a parse error, so every rule in this engine's `gdpr.yml` was absent there. That absence is NOT a clean result: these rules detect personal data crossing a boundary into a log sink, a URL or browser storage, and a file that was never parsed cannot report any of the three. The rest of the tree analysed normally and its rows above stand; only these files are unaccounted for. You can widen what we reach: fix the syntax error (or exclude the file deliberately) and re-scan to cover it.
- AX6 Interface segregation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over the public interfaces this run's compilations declare, and none was loaded, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- AXB1 Runtime evidence locked — no reproducible boot — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Subject: a native UI project (app/src/main/AndroidManifest.xml). The Runtime Evidence tier boots an app only via docker-compose, an Aspire AppHost, a Dockerfile, an npm dev script, a web service's own toolchain entry point (cargo, go, dotnet, Python, Spring Boot), or an Android app's own Gradle wrapper (run on an emulator); an Apple or Flutter app needs a host it lacks. None applied, so no live runtime a11y/egress/header evidence was collected. You can widen what we reach: add a docker-compose.yml (or an Aspire AppHost) that brings the app up with its dependencies. Watchdog then boots it in an isolated sandbox and gathers real runtime evidence — you change nothing in your pipeline (no CI step, no SDK).
- C2 Access Controls — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These authorization controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks authorization controls.
- GD1 Unfinished & placeholder code — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check reads C# syntax, F#, JavaScript/TypeScript, Dart, Ruby and Scala source only, and no C#, F#, JavaScript/TypeScript, Dart, Ruby or Scala was found, and this repository's Kotlin is not read yet, so it had nothing of this repository's product to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- IC1 Incompleteness & stubs — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check reads C# syntax, JavaScript/TypeScript, Java, Dart, Ruby and Scala source only, and no C#, JavaScript/TypeScript, Java, Dart, Ruby or Scala was found, and this repository's Kotlin is not read yet, so it had nothing of this repository's product to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- P5 DR & Backup — not measured this run — This is a true statement about the repository that carries nothing for its owner to act on, so it is reported here rather than as a defect in their code. No backup/snapshot/replication config, RTO/RPO or restore-procedure documentation was found — and no production persistence was detected either (no data-access packages, no data-store services, no database resources), so there is nothing in this repository whose loss a DR control would recover. If this system's data lives in a platform or ops repo we can't see, that's where the DR evidence belongs.
- S1 Web-Security Posture — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These web-security controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks web-security controls.
- X24 Document value interpolated into markup unescaped — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check reads C# syntax, and JavaScript/TypeScript, Java, Scala, Ruby and Python source only, and no C# was loaded and no JavaScript/TypeScript, Java, Scala, Ruby or Python was found in this repository, so it had nothing of this repository's product to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
Limitations & what we did not check
Watchdog assesses the repository exactly as committed, and only the repository. By design it does not reach outside the source tree: the live cloud account, the running CI/CD pipeline, the host's branch-protection and approval rules, the production configuration, or a restore actually exercised against a backup are all out of scope. That boundary is a feature, not a gap — a repo-relative, deterministic scan re-runs identically on any commit and every finding opens at a real file and line, where a live audit can neither be reproduced nor traced. The visible consequence is that controls which leave no in-repo evidence are reported as "not evidenced" and excluded from the score rather than awarded a number a static scan cannot justify.
Per-dimension blind spots
- D1 Cyclomatic Complexity: Cyclomatic complexity counts branches statically — it cannot tell an essential decision tree from accidental tangle, nor see complexity that lives in data or configuration (large switch-case token tables, DSL lexers/parsers, data-as-code rule tables) rather than control flow: a tokenizer's many single-character cases read as high complexity though each branch is trivial.
- D2 Cognitive Complexity: Cognitive-complexity heuristics approximate how hard code is to follow; genuine domain difficulty and well-named intent that eases reading are not captured.
- D3 God Classes: "God class" is sized by members and responsibilities visible in the type — a deliberately broad facade over a coherent subsystem can read the same as an accidental grab-bag. For front-end JS the file-length check is cohesion-aware (a single-responsibility module — one class/IIFE — earns a 3× threshold), but cohesion is approximated from top-level declarations, not true dependency structure.
- D4 Code Duplication: Duplication is token-similarity — an in-process token-stream comparison over sliding windows, with type-aware normalization — so it finds copy-paste, not semantic duplication expressed differently. Committed machine-written code (scaffolded migrations, designer/codegen output, protobuf/OpenAPI stubs, model snapshots) is EXCLUDED — its repetition is the tool's, not the team's — so the score reflects hand-written duplication only.
- D5 Coupling: Coupling is measured between projects/assemblies — runtime coupling through DI, reflection, messaging or shared databases is invisible to a static reference graph.
- D6 Cohesion (LCOM4): LCOM4 cohesion is syntactic — it infers connectivity from which methods touch which fields/methods by name, not from real runtime behaviour or intent.
- D13 Secret Scanning: Secret detection is signature- and entropy-based on the current tree — a secret that does not match a known pattern, or one already rotated, will not be flagged (a clean scan is "nothing matched", not "no secrets exist").
- D15 Churn × Complexity Hotspots: Churn hotspots come from git history — a freshly imported or squashed repository has no churn signal, and recent rewrites can mask a historically risky file.
- D17 Explicit Debt: Acknowledged-debt signals (TODO/FIXME, suppressions, dead code) are textual — undocumented debt that nobody marked, and debt that lives in design rather than annotations, is invisible. Committed machine-written code (scaffolded migrations, designer/codegen output, generated stubs) is excluded — it is never the team's dead code to delete.
- D19 Documentation Quality: Documentation quality is judged by an LLM over a bounded sample of docs — it reads what is written, not whether the docs match the running system, and it is advisory, not a measurement. Its critique rows are drawn from a closed category vocabulary and each row means the same thing in every run, so two scans can be compared row by row; the SET that fires is still a sample, and does not repeat exactly. Measured on one frozen input, six scans at one engine SHA: 2-5 critique rows per scan, 8 distinct rows across the six, 3 of those 8 seen in only one scan. So a D19 row is evidence about the documentation, but a COUNT of D19 rows is not a quantity — never read a change in it as an improvement or a regression.
- D20 ADR Quality: ADR quality is an LLM read of the decision records present — it cannot know about decisions made and never recorded, and its verdict is sampled and advisory.
- D21 Naming Consistency: Naming quality is an LLM judgement over a bounded sample — it assesses clarity/consistency of the names it sees, not domain-correctness, and is advisory.
- D26 Project Cohesion: Project focus is sized from members/namespaces per project — a project that is broad by deliberate design reads the same as one that has sprawled.
- D28 Secrets (history): Secret-history scanning sweeps the git log for known patterns — a secret that predates the available history, or never matched a signature, is not found (clean means "nothing matched in the history we can see").
- D29 Static Analysis (SAST): SAST findings are pattern-based (semgrep) — it finds classes of bug it has rules for; logic flaws, auth/authorization gaps and issues needing runtime context are out of reach (and clean means "no rule matched").
- D34 Knowledge Freshness: Freshness is decayed commit RECENCY, not comprehension — code read often but rarely committed reads as orphaned, and stable code that genuinely needs no changes is penalised the same as forgotten code; bot/squash commits distort it like the bus factor.
- D35 Change Coupling: Change coupling is co-change in COMMITS — files split across separate commits, or coupled only through a shared config/build step, read as uncoupled, and a sweeping commit (rename/format) is excluded so it doesn't couple everything. It shows that files change together, not WHY: a high coupling can be a healthy cohesive pair as readily as a hidden leak.
- D44 Platform End-of-Life: The support table is FROZEN, so it goes out of date by losing RECALL: a release that ended support after the table was written is missed until the table is refreshed, and this dimension reading 10 is not evidence that a platform is current. Only platforms the repository DECLARES in a place this pass reads are seen — a runtime named only in a Dockerfile (D31's subject), in a CI workflow (D29's), or in a file this pass does not parse (go.mod, a Gemfile ruby directive) is invisible here, which is why a repository declaring none of them abstains rather than scoring. Only frameworks with a PUBLISHED support policy are tracked: React, Flask and Express publish none, so their age cannot be judged and their absence from a report is not a statement that they are supported.
- AX10 Code composition: Role is inferred from namespace/folder convention, not semantics — a domain concept living in a folder named "Services" reads as application, and the split is lines-of-code, not business value. The business-logic-share score is a SOFT, FLOORED signal: it contributes to the Architecture lens but is floored at the Critical gate, so an infrastructure-heavy design (a gateway, an ETL, a driver) is legitimately low without being nuked to zero.
- M4 Documentation accuracy: Onboarding quality is an LLM read of the docs/setup present — it cannot run the onboarding or measure how long a real new joiner takes; the verdict is sampled and advisory.
- P4 Deployment & Rollback: Approval/branch-protection rules live in repository settings the scan cannot see — only their in-repo evidence (config files, workflows) is checked, so a control enforced purely in the host's settings reads as "not evidenced".
- P6 Release Hygiene: Rollback/observability controls are inferred from repo artefacts (pipelines, dashboards-as-code) — controls configured in external tooling, with no in-repo trace, cannot be credited.
The LLM boundary
Dimensions
D1 · Cyclomatic Complexity
0 method(s) exceeded the cyclomatic complexity threshold of 15.
D2 · Cognitive Complexity
0 method(s) exceeded the cognitive complexity threshold of 15.
D3 · God Classes
0 god class(es) detected.
D4 · Code Duplication
0 duplicated block group(s) detected.
D5 · Coupling
18 production modules (Gradle), 0 dependency cycle(s), 0 unstable depended-on module(s). Read from the build's own module declarations; edges a convention plugin adds from buildSrc or build-logic are not visible there; 6 module(s) off the main sequence, with abstractness counted on 17 of the 18 (the rest declare no modelled class or interface, export only macros, declare types in source no model reads, or have no source directory of their own).
What to do
- Resolve the 5 Off the main sequence finding(s) in Coupling. — One of this dimension's main actionable groups (5 warning-level).
- Enforce Coupling in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D6 · Cohesion (LCOM4)
0 of 4 classes have LCOM4 above 3.
D13 · Secret Scanning
1 secret(s) detected.
What to do
- Resolve the 1 Leaked secret finding(s) in Secret Scanning — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
- Enforce Secret Scanning in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D15 · Churn × Complexity Hotspots
No churn × complexity hotspots in the window.
D17 · Explicit Debt
1 deducted task-comment markers across 4220 LoC (0.0/KLoC) → score 10.0. Task comments only: this repository's language is read without a compiler, so D17's suppression, dead-code and commented-out-code arms did not run and this score counts fewer marker kinds than a .NET repository's would.
What to do
- Resolve the 1 TodoComment finding(s) in Explicit Debt — start with DetailScreen.kt. — One of this dimension's main actionable groups (1 warning-level).
- Enforce Explicit Debt in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D19 · Documentation Quality
The repository's root README is a well-structured overview of the project (a breaking-bad Android Jetpack Compose app) with an impressive tech-stack list and a centered image showing the UI. It describes what it does (programmatic composable UI, Hilt injection, Room persistence, ViewModel lifecycle), but it lacks installation/build/usage examples and contribution guidance for an open-source library; those belong to the root README rather than this single-file doc.
What to do
- Improve Documentation Quality — currently 7.0/10. — The repository's root README is a well-structured overview of the project (a breaking-bad Android Jetpack Compose app) with an impressive tech-stack list and a centered image showing the UI. It describes what it does (programmatic composable UI, Hilt injection, Room persistence, ViewModel lifecycle), but it lacks installation/build/usage examples and contribution guidance for an open-source library; those belong to the root README rather than this single-file doc.
D20 · ADR Quality
No architecture decision records were found.
What to do
- Resolve the 1 No ADRs found finding(s) in ADR Quality. — One of this dimension's main actionable groups (1 recommendation-level).
D21 · Naming Consistency
0 naming inconsistencies across 0 sampled symbols.
D26 · Project Cohesion
0 of 18 build units (Gradle) flagged as possibly oversized/incoherent.
D28 · Secrets (history)
1 finding(s): 0 critical, 1 high, 0 medium, 0 low.
What to do
- Resolve the 1 High secret finding(s) in Secrets (history) — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
D29 · Static Analysis (SAST)
21 finding(s): 0 critical, 20 high, 1 medium, 0 low. 17 unpinned-GitHub-Actions row(s) are reported here but scored by D36 (supply-chain provenance), which measures that posture as `pinned_actions` — one pinning decision is charged once, not once per lens. Separately, one or more rules could not re-parse an embedded snippet in 1 file(s) (e.g. a workflow `run:` block read as shell). Those files WERE scanned and their other rows are unaffected; only those rules' view of those snippets is missing.
What to do
- Resolve the 3 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED (2), REDACTED. — One of this dimension's main actionable groups (3 issue-level).
- Resolve the 1 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED. — One of this dimension's main actionable groups (1 warning-level).
- No action in Static Analysis (SAST) — all 17 REDACTED finding(s) are reported here at file:line but scored by D36 (supply-chain provenance), so none is charged to this dimension. — One of this dimension's main actionable groups (17 issue-level, 0 of them charged here).
D34 · Knowledge Freshness
21 of 21 significant source file(s) are orphaned — their living knowledge has decayed to nothing, so no one currently understands them. The largest is feature/detail/src/main/kotlin/io/github/shinhyo/brba/feature/detail/DetailScreen.kt. Counted over 21 of the 100 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over.
What to do
- Resolve the 21 Orphaned file finding(s) in Knowledge Freshness — start with DetailScreen.kt, ListScreen.kt, Theme.kt. — One of this dimension's main actionable groups (21 recommendation-level).
- Resolve the 1 Dormant codebase finding(s) in Knowledge Freshness. — One of this dimension's main actionable groups (1 recommendation-level).
D35 · Change Coupling
No strong hidden change-coupling between production files.
D36 · Supply-chain Provenance & Signing
1/4 supply-chain integrity signals present (provenance, signing, SBOM, pinned actions).
What to do
- Resolve the 2 REDACTED finding(s) in Supply-chain Provenance & Signing — start with REDACTED, REDACTED. — One of this dimension's main actionable groups (2 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 recommendation-level).
D44 · Platform End-of-Life
0 end-of-life runtime(s) and 0 end-of-life framework(s), read from 1 platform declaration(s) and 45 dependency declaration(s). This dimension reads what the repository says about ITSELF — a pinned target framework, a version file, a capped requires-python, a Rust toolchain pin, a framework major a constraint cannot move off. A FLOOR is deliberately never charged: `requires-python = ">=3.8"` states what the package SUPPORTS, not what it runs on, and a well-maintained library declares exactly that while running its own CI on a current release. The end-of-life facts are FROZEN and dated, so this dimension needs no network and answers identically inside a closed scan fence; as the table ages it loses recall and never precision, because a statement about support that ended in the past cannot become false. The OS layer of a container image is D31's question and the toolchain a CI workflow installs is D29's; this row is neither.
Frontend & cross-cutting dimensions
AX10 · Code composition
What to do
- The domain core is a small share of production code, but most of the rest matched no layer vocabulary at all — so this is not yet an anemic-domain finding. The namespace/path convention could not place that code, which makes the composition above a statement about the naming, not about the design. Name the layers (or check that the repository's conventions differ from the ones this check knows) before reading a thin domain into it.
AX2 · Stateful singletons
AX3 · Project dependency cycles
AX4 · Dependency direction
M1 · Documentation (README)
What to do
- Add a build/run (quick start) section to the root README — the first thing a newcomer needs.
- Explain in the README how to run the tests. Add a section called Testing to the README in the repository's top folder, saying how to run the test suite.
- Add an 'Architecture' / 'How it works' section to the root README — the high-level shape.
- Add a README to the 17 of 17 project(s) that lack one — worth up to 2 pts.
M2 · Architecture documentation
- No Architecture Decision Records found — no conventional ADR directory, no numbered `NNNN-title` documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer.
What to do
- Record your significant design decisions. Write one short document per decision, giving its date, the situation, the decision and its consequences, and keep them together with your other design documents. A docs/adr folder with each file named by number and title is the easiest place to find them.
M3 · Folder & project structure
M4 · Documentation accuracy
P1 · CI/CD gates
P3 · Security & performance tooling
- No static application security testing detected. For this repository's stack, add spotbugs with find-sec-bugs (or `semgrep --config=auto`, which runs on any language) as a CI step. What was searched, so you can tell an absence from a miss: the 5274 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge's web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen — if that is your case the row is wrong, and saying so is more useful than adding a second scanner.
What to do
- Add a SAST step to CI running what this repository's stack ships: spotbugs with find-sec-bugs — or `semgrep --config=auto`, which runs on any language — so a security regression fails the build instead of landing.
- Enable Dependabot/Renovate or a dependency-review gate.
- Add gitleaks/trufflehog in CI to block PRs that introduce committed secrets.
P4 · Deployment & Rollback
- The release is automated and no gate that pauses it for a human is DECLARED IN THIS REPOSITORY'S PIPELINE FILES. What was read: every file under `.github/workflows/`, `.forgejo/workflows/`, `.gitea/workflows/`, `.azuredevops/` and `.azure-pipelines/`, plus `.gitlab-ci*` and `azure-pipelines*` — with comment text stripped, so documenting a gate is not declaring one. What would have counted: GitLab's `when: manual`, CircleCI's `type: approval`, an Azure `ManualValidation@` task or an `approvals:` block, a Jenkins `input` step, a `uses:` step naming an approval action, an `environment:` paired with `reviewers` / `required_reviewers` / `protection` / `wait-timer` / `deployment_branch_policy`, a draft-release step, a `workflow_dispatch` promotion, or a release-event gate. ★ What this cannot see, because none of it is a file: a GitHub environment whose required reviewers are configured in repo SETTINGS, a branch protection rule, or an organisation deployment policy — all of them real, enforced gates that live outside the repository. If yours is one of those, this row is wrong and nothing in the tree could have told us. Otherwise: whatever the release trigger points at is published to users unreviewed, so a mistagged or unverified commit ships and the only remedy is a follow-up release.
What to do
- Nothing pauses a release for a human: publish as a draft release (or gate the release job on a protected tag/manual dispatch) so a bad build can be stopped before users can download it.
P6 · Release Hygiene
P7 · Outbound HTTP resilience
- `Retrofit.Builder(` makes an outbound HTTP call, and nothing bounds it: no timeout, deadline, retry or circuit breaker is set for it here, and the client has no process-wide default. A slow or failing dependency will hold this service's request, thread or connection until the call gives up on its own — or never, for a client with no default timeout. — core/network/src/main/kotlin/io/github/shinhyo/brba/core/network/retrofit/RetrofitNetwork.kt:62
What to do
- Set connect and read timeouts on every client (`RestTemplateBuilder.connectTimeout/readTimeout`, WebClient `responseTimeout`, `HttpRequest.Builder.timeout`, `HttpURLConnection.setReadTimeout`, Feign `connectTimeout`/`readTimeout` in configuration), and wrap calls to dependencies in resilience4j `@Retry`/`@CircuitBreaker`/`@TimeLimiter` (or Spring Retry).
PF3 · Async & latency hygiene
X1 · Async correctness
X12 · Unreachable branch
X13 · Undrained process stream
X14 · Bypassable address classification
X15 · Unvalidated length from an untrusted reader
X16 · Unfloored truncation loop
X17 · Uncapped recursion over a caller-supplied document
X18 · Disposal-pattern correctness
X19 · Unrestored process-global state
X20 · Mistyped argument guard
X21 · Side-effecting pattern guard
X22 · Contradicted release guard
X23 · Unguarded diagnostic materialisation
X25 · Inert configuration knob
X26 · Unsynchronised callback handoff
X27 · Collection changed while being enumerated
X28 · Index access outside its own emptiness guard
X29 · Per-element action decided by a fixed element
X3 · Exception handling
X30 · Support guard that admits what it rejects
X5 · Nullable reference types
Reference — by lens
| Lens | Score | Rating | Impact |
|---|---|---|---|
| Code Health | 100% | Exemplary | Solid. |
| Architecture | 97% | Exemplary | Solid. |
| Maturity | 52% | Adequate — gated by D34, M2 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Readiness | 39% | Weak — gated by P3, P7 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 68% | Adequate — gated by D36 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Performance | 100% | Exemplary | Strongest area. |
Not evidenced — 1 control(s) we could not find positive evidence for
- P5 DR & Backup — not evidenced — repo shows no backup/RTO/RPO controls; absence of evidence is not evidence of a working control
Not included — 64 check(s) not relevant to this codebase
- AC1 Text alternatives — No web markup found — accessibility is not applicable to this repository.
- AC2 Forms & labels — No web markup found — accessibility is not applicable to this repository.
- AC3 Page structure — No web markup found — accessibility is not applicable to this repository.
- AC4 Keyboard semantics — No web markup found — accessibility is not applicable to this repository.
- AC5 ARIA correctness — No web markup found — accessibility is not applicable to this repository.
- AC6 Visual & motion safety — No web markup found — accessibility is not applicable to this repository.
- AC7 A11y enforcement — No web markup found — accessibility is not applicable to this repository.
- AX1 Captive dependencies — Not applicable: Dagger rejects, at compile time, a component that holds a binding with a narrower scope than its own.
- AX5 Architecture & structure — not assessed — architecture style/structure is computed from a project graph (projects, types, module namespaces) that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX6 Interface segregation — not assessed — interface segregation is computed over a type surface that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — no test/production split to check
- AX9 CQS / query purity — no CQRS query handlers detected — query purity is not applicable to this codebase
- AXB1 Runtime evidence locked — no reproducible boot — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- AXB2 Runtime readiness — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- C1 Data Protection — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a Room, JPA or MongoDB entity, an Exposed or Ktorm table, or in a SQL CREATE TABLE or ALTER TABLE — and no database or data-store client (Room, Exposed, SQLDelight, Realm, JPA, JDBC, MongoDB, Firebase, …) imported either, so this repository keeps no data at rest for these controls to protect.
- C2 Access Controls — Not assessed: these authorization controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks authorization controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C3 Audit Trail — No personal data detected in a persisted data model — no PII-named column (email, first_name, date_of_birth, phone_number, …) or stored credential on a Room, JPA or MongoDB entity, an Exposed or Ktorm table, or a SQL CREATE TABLE or ALTER TABLE — and no database or data-store client (Room, Exposed, SQLDelight, Realm, JPA, JDBC, MongoDB, Firebase, …) imported either, so this repository keeps no sensitive data whose changes an audit trail would record. If it does (through a store configured outside this repository, for instance), the trail belongs to wherever that data is stored.
- C4 Data Retention — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a Room, JPA or MongoDB entity, an Exposed or Ktorm table, or in a SQL CREATE TABLE or ALTER TABLE — and no database or data-store client (Room, Exposed, SQLDelight, Realm, JPA, JDBC, MongoDB, Firebase, …) imported either, so this repository keeps no personal data whose lifetime a retention limit would bound.
- C5 Data-Subject Rights — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a Room, JPA or MongoDB entity, an Exposed or Ktorm table, or in a SQL CREATE TABLE or ALTER TABLE — and no database or data-store client (Room, Exposed, SQLDelight, Realm, JPA, JDBC, MongoDB, Firebase, …) imported either, so this repository keeps no personal data for a data subject to erase, export or consent over.
- D10 Test Quality — ~368 lines of test source are present (.kt) but no test cases reached the test census for this repository — no test root we could resolve declared them, or the files we read declare no cases in a test framework we recognise — so skipped/assertion-free tests couldn't be counted. Not scored — this is a gap in the analyzer, not a finding about this repository.
- D11 Test Reliability — Test reliability not included — the .kt suite was found but not re-run
- D12 Dependency Hygiene — Not scored — 19 Maven/Gradle manifest(s) were read, stating a version for 45 dependency(ies) and leaving 12 to a parent or an imported BOM, but no build resolved the dependency graph, so the outdated signal was not established. NOT a finding that these dependencies are current or healthy.
- D14 License Compliance — Not scored — this repository's 45 declared JVM dependency coordinate(s) were read from its 19 Maven/Gradle manifest(s), but Maven Central could not be asked for the licence of 45 of them (HTTP 429 Too Many Requests from https://repo1.maven.org/maven2/org/jetbrains/kotlin/kotlin-stdlib-jdk8/2.1.21/kotlin-stdlib-jdk8-2.1.21.pom), and a licence verdict over part of a dependency graph is not a licence verdict. Nothing is asserted about this repository's licensing in either direction.
- D16 Bus Factor — single-contributor repository — bus factor is not applicable
- D18 Solution Shape — D18 scores the shape of a .NET solution; this repository has no .NET solution or project files, so the dimension does not apply.
- D22 Internal API Consistency — The exposed public-API surface could not be collected — no C#/VB projects loaded.
- D23 Boundary Type-Coupling — No bounded-context organisation was detected either — neither a context-shaped layout, nor 2+ sibling source directories each declaring an aggregate root, nor 2+ projects each declaring an entity of their own. Declaring this codebase's bounded contexts (≥2) would let cross-boundary type coupling be assessed. Declare them in `.codehealth/config.yaml` at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it — e.g. `architecture:` → `contexts:` → `Billing: ["src/billing", "Acme.Billing"]`, `Catalog: ["src/catalog", "Acme.Catalog"]`.
- D24 Comment Value — No inline comments to assess — comment value is not applicable here.
- D25 ADR Conformance — no ADRs to check
- D27 Navigability — D27 counts a call into the standard library or a dependency as unresolved for this repository's language (.kt), so too few call sites counted as resolved to assess navigability. Not scored: this is a gap in the analyzer, not a verdict about this repository.
- D30 Dependency Vulnerabilities — Not scored — no dependency manifest in a supported ecosystem was read for this repository. A gap in the analyzer's language coverage, NOT a finding that the repository is free of vulnerable dependencies (a Gradle build (build.gradle/build.gradle.kts) and a Gradle version catalogue — not scanned yet).
- D31 IaC & Container Security — No Infrastructure-as-Code or container manifests found (Dockerfile, Docker Compose, Terraform, Kubernetes/Helm, CloudFormation, ARM, Bicep, Ansible); nothing to scan.
- D32 Data Compliance (PII/GDPR) — 1 file(s) were not parsed by semgrep — the PII/GDPR ruleset never ran over them
- D37 Vulnerability-disclosure Policy — No vulnerability-disclosure policy file found (SECURITY.md/.markdown/.rst/.txt at root or under .github/.forgejo/.gitea/docs, .well-known/security.txt). A coordinated-disclosure policy may live off-repo, so this is not evidenced rather than failed.
- D39 IL Efficiency — D39 measures the IL emitted by a .NET build; this repository has no .NET solution or project files, so the dimension does not apply.
- D40 Network Egress Confinement — No Kubernetes/orchestration workloads found in the repository manifests; network egress policy is a cluster-native control that may live at the platform/firewall layer, so there is nothing to assess here.
- D41 Kernel & Syscall Confinement — No Kubernetes/orchestration workloads found in the repository manifests; seccomp/AppArmor/SELinux confinement is a workload-level control, so there is nothing to assess here.
- D42 Runtime Threat Enforcement — No Kubernetes/orchestration workloads found in the repository manifests; runtime threat-detection and admission-control policy are cluster-level controls, so there is nothing to assess here.
- D43 Malicious Dependencies — Not scored — no dependency manifest in a supported ecosystem was read for this repository. A gap in the analyzer's language coverage, NOT a finding that the repository is free of vulnerable dependencies (a Gradle build (build.gradle/build.gradle.kts) and a Gradle version catalogue — not scanned yet).
- D7 Architectural Integrity — no checkable ADRs, and no project-reference graph for the cycle pass to read — so this dimension makes no claim about dependency cycles in either direction (where this repository's language has an import-cycle lens, cycles are reported there). Architectural integrity not assessed
- D8 Code Coverage — Coverage not included — suite not readable by the collector
- D9 Test Distribution — Test source is present (.kt) but no test cases reached the test census for this repository — no test root we could resolve declared them, or the files we read declare no cases in a test framework we recognise — so its unit/integration/BDD/E2E split couldn't be counted. Not scored — this is a gap in the analyzer, not a finding about this repository.
- DM1 Domain Modelling — applicable but not scored (2 of 3 signals for this style — below the bar we score at): 23 value object(s); a Domain/Aggregates/ValueObjects layer
- ED1 Event-Driven — not scored — this repository shows none of the 3 signals this lens looks for
- ED5 Idempotency — no mutating command handlers or message consumers detected — idempotency check not applicable
- ES1 Event Sourcing — not scored — this repository shows none of the 3 signals this lens looks for
- GD1 Unfinished & placeholder code — no source files were read — this check reads C#, F#, JavaScript/TypeScript, Dart, Ruby and Scala, and none was read for this repository's product. That is a limit of the analyzer, not a finding about your code.
- IC1 Incompleteness & stubs — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- P12 CI test-gate honesty — Reported, not scored — this card publishes what the CI gate does with the test inventory rather than grading it. The findings above are its output.
- P2 Observability — This repository's JVM source (17 module(s), 73 file(s) read) declares no entry point and bootstraps no server, and nothing here deploys a service — it is a library, run inside whatever hosts it, so production observability (structured logging, tracing/metrics, health checks) is N/A. If it grows a binary or a service, the dimension reactivates.
- P8 Schema migrations — no ORM, schema-migration tool or schema auto-create was found in this repository's dependency manifests or source, so there is no database schema for this check to judge
- P9 Domain vs controller coverage — no coverage report found on disk — produce a coverage report in a standard format (JaCoCo XML — the Gradle `jacocoTestReport` task) and commit it — a hosted scan measures a clone of the repository, so a report that exists only in a working tree, a CI runner's or your own, never reaches it; the artefact is commonly gitignored, so `git add -f` that one file (or un-ignore its path) and commit it alongside the code it measures, or wire coverage collection into CI, to enable this cross-layer check
- PF1 Benchmark discipline — Not applicable: no benchmark suite was found. This check searched for `@Benchmark` in a Java, Kotlin or Scala file importing JMH or kotlinx-benchmark, Jetpack Benchmark's `measureRepeated` in a file importing androidx.benchmark, or JMH/kotlinx-benchmark/androidx.benchmark in a pom.xml, Gradle or sbt build, and for a `*benchmark*` script that this repository's CI runs. Benchmarks are credited as a bonus, so their absence is neither scored nor deducted.
- PF2 Allocation hygiene — Not applicable: allocation awareness is rated where code engineers for it — at least 400 production lines that ship benchmarks or already use allocation-aware idioms (8 or more). This repository's Kotlin code has 4,559 production line(s), 0 such use(s) and no benchmarks, so there is no allocation work to rate. The card is reward-only: its absence costs nothing.
- S1 Web-Security Posture — Not assessed: these web-security controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks web-security controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- X10 Duplicated predicate — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X2 Cancellation propagation — no C# was loaded; this repository's Kotlin was read, and no request in it can be cancelled by a caller (Ktor, OkHttp's newCall or a URL stream, from a suspend function) — a Kotlin request outside a suspend function has no caller's cancellation to honour
- X24 Document value interpolated into markup unescaped — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X32 Type resolved by simple name across every loaded assembly — This check is about how a .NET program searches the assemblies loaded into its process for a type, and this repository contains no .NET source, so there is nothing here for it to assess. Not a gap in the analyzer and not a finding about your code.
- X4 Structured logging — This check asks whether log calls keep a message's values apart from its text, which needs a logger that can carry them as fields. This repository's Kotlin was read, and it logs through none that an assembled message defeats: Android's `Log` and Timber, java.util.logging and log4j-api-kotlin take one finished message (as log4net does in C#), or it makes no call through a named logger at all. Calls made through SLF4J, log4j 2 or kotlin-logging (`LoggerFactory.getLogger`, `KotlinLogging.logger {}`) would be judged.
- X6 Hand-rolled structured-format parsing — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X7 Silent fallback defaults — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X9 Subsumed condition operand — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
Appendix A — Findings (grouped)
Critical — 22 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
Serious — 11 finding(s)
- Off the main sequence: BrBa-Compose:core:model
- Off the main sequence: BrBa-Compose:feature:detail
- Off the main sequence: BrBa-Compose:feature:favorite
- Off the main sequence: BrBa-Compose:feature:list
- Off the main sequence: BrBa-Compose:feature:setting
- REDACTED
- REDACTED
- TodoComment feature/detail/src/main/kotlin/io/github/shinhyo/brba/feature/detail/DetailScreen.kt:171
- REDACTED
- REDACTED
- Outbound HTTP without resilience core/network/src/main/kotlin/io/github/shinhyo/brba/core/network/retrofit/RetrofitNetwork.kt:62
Minor — 30 finding(s)
- Orphaned file feature/detail/src/main/kotlin/io/github/shinhyo/brba/feature/detail/DetailScreen.kt:1
- Orphaned file feature/list/src/main/kotlin/io/github/shinhyo/brba/feature/list/ListScreen.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/theme/Theme.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaCharacterCard.kt:1
- Orphaned file feature/favorite/src/main/kotlin/io/github/shinhyo/brba/feature/favorate/FavoriteScreen.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaCharacterRow.kt:1
- Orphaned file gradle/projectDependencyGraph.gradle:1
- Orphaned file feature/setting/src/main/kotlin/io/github/shinhyo/brba/feature/setting/SettingScreen.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaThemeSelectDialog.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaPreference.kt:1
- Orphaned file feature/bottombar/src/main/kotlin/io/github/shinhyo/brba/feature/bottombar/BottomBarScreen.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/theme/Color.kt:1
- Orphaned file feature/list/src/main/kotlin/io/github/shinhyo/brba/feature/list/ListViewModel.kt:1
- Orphaned file core/network/src/main/kotlin/io/github/shinhyo/brba/core/network/retrofit/RetrofitNetwork.kt:1
- Orphaned file feature/detail/src/main/kotlin/io/github/shinhyo/brba/feature/detail/DetailViewModel.kt:1
- Orphaned file feature/main/src/main/kotlin/io/github/shinhyo/brba/main/MainActivity.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaChips.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaTopAppBar.kt:1
- Orphaned file feature/favorite/src/main/kotlin/io/github/shinhyo/brba/feature/favorate/FavoriteViewModel.kt:1
- Orphaned file core/designsystem/src/main/kotlin/io/github/shinhyo/brba/core/ui/BrbaIconFavorite.kt:1
- Orphaned file core/data/src/main/kotlin/io/github/shinhyo/brba/core/data/repository/CharactersRepositoryImpl.kt:1
- No ADRs found
- Dormant codebase
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- No ADRs
- No SAST
- No release approval gate
Minor — 1 finding(s)
- Off the main sequence: BrBa-Compose:core:common
Appendix B — Reproduction & audit trail
| Dimension | Tool | Version | Command | Findings | Raw output |
|---|---|---|---|---|---|
| D28 · Secrets (history) | gitleaks | — | gitleaks detect --no-banner --config /opt/gitleaks-rules/watchdog-gitleaks.toml --report-format json --report-path /tmp/watchdog-gitleaks-273b1248f3e64f19b650a859a64d5a53/history.json --exit-code 0 --source . | 0 | artifacts/raw/gitleaks-history.json |
| D28 · Secrets (history) | gitleaks | — | gitleaks detect --no-git --no-banner --config /opt/gitleaks-rules/watchdog-gitleaks.toml --report-format json --report-path /tmp/watchdog-gitleaks-273b1248f3e64f19b650a859a64d5a53/tree.json --exit-code 0 --source . | 0 | artifacts/raw/gitleaks-tree.json |
| D29 · Static Analysis (SAST) | semgrep | — | semgrep --config /opt/semgrep-rules/security-audit.yml --config /opt/semgrep-rules/owasp-top-ten.yml --config /opt/semgrep-rules/watchdog-sast.yml --json --quiet --timeout 10 --timeout-threshold 3 --metrics off . | 21 | artifacts/raw/semgrep.json |
| D30 · Dependency Vulnerabilities | none (dependency manifest found, not scanned for vulnerabilities here) | — | none (dependency manifest found, not scanned for vulnerabilities here): not applicable — Not scored — no dependency manifest in a supported ecosystem was read for this repository. A gap in the analyzer's language coverage, NOT a finding that the repository is free of vulnerable dependencies (a Gradle build (build.gradle/build.gradle.kts) and a Gradle version catalogue — not scanned yet). | 0 | — |
| D31 · IaC & Container Security | trivy | — | trivy: not applicable — No Infrastructure-as-Code or container manifests found (Dockerfile, Docker Compose, Terraform, Kubernetes/Helm, CloudFormation, ARM, Bicep, Ansible); nothing to scan. | 0 | — |
| D32 · Data Compliance (PII/GDPR) | semgrep | — | semgrep: not applicable — No personal data was found crossing a boundary the PII/GDPR ruleset checks — nothing written to a log or console sink, placed in a URL or query string, or persisted to browser storage. That is a clean result for the LEAK surface only: this ruleset detects personal data escaping, it does not inventory the personal data a repository holds, so it is not evidence that this repository has no personal-data surface. The personal-data map (Appendix C) and the C1-C5 compliance cards are what speak to that. semgrep could not parse 1 file(s) — `core/database/src/main/kotlin/io/github/shinhyo/brba/core/database/AppDatabase.kt` — so the PII/GDPR sweep did not cover the unparsed regions of them; rows reported elsewhere in those files are real. | 0 | — |
| D37 · Vulnerability-disclosure Policy | disclosure | — | disclosure: not applicable — No vulnerability-disclosure policy file found (SECURITY.md/.markdown/.rst/.txt at root or under .github/.forgejo/.gitea/docs, .well-known/security.txt). A coordinated-disclosure policy may live off-repo, so this is not evidenced rather than failed. | 0 | — |
| D40 · Network Egress Confinement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; network egress policy is a cluster-native control that may live at the platform/firewall layer, so there is nothing to assess here. | 0 | — |
| D41 · Kernel & Syscall Confinement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; seccomp/AppArmor/SELinux confinement is a workload-level control, so there is nothing to assess here. | 0 | — |
| D42 · Runtime Threat Enforcement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; runtime threat-detection and admission-control policy are cluster-level controls, so there is nothing to assess here. | 0 | — |
| D43 · Malicious Dependencies | none (dependency manifest found, not scanned for vulnerabilities here) | — | none (dependency manifest found, not scanned for vulnerabilities here): not applicable — Not scored — no dependency manifest in a supported ecosystem was read for this repository. A gap in the analyzer's language coverage, NOT a finding that the repository is free of vulnerable dependencies (a Gradle build (build.gradle/build.gradle.kts) and a Gradle version catalogue — not scanned yet). | 0 | — |