Enh: domain/ layer imports isar_community directly in 3 files — violates Clean Architecture rule
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Start by reading the three named service/use-case files and the existing GanaRepository/GanaRepositoryImpl pattern. Run grep -rn "package:isar_community" lib/domain/ to confirm the violations. Done means Isar access in each file goes through a Domain repository interface implemented in lib/data/, and the grep finds no remaining Domain imports.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
"Domain layer... Has zero imports from isar_community, flutter, or any presentation package." Three existing files break this — the Domain layer
should only reach Isar through repository interfaces implemented in lib/data/, but these reach into Isar directly:
- lib/domain/services/nip17_encryption_service.dart:4
- lib/domain/services/marmot_transport_service.dart:3
- lib/domain/usecases/dm_usecases.dart:3
Verify with:
grep -rn "package:isar_community" lib/domain/
This predates the recent Gana cloud-support work (that work was checked against this exact rule and introduced no new violations — see docs/AUDIT.md, 2026-08-11
entry). Not urgent, not touched mid-feature — flagging for a deliberate cleanup pass: each of these three needs its Isar access moved behind a repository interface
(Domain) + implementation (Data), following the pattern already used everywhere else in the codebase (e.g. GanaRepository/GanaRepositoryImpl).
- Lenguaje dominante
- Dart
- Estrellas
- 2
- Forks
- 2
- Merge medio
- 1 d 7 h
- PR fusionados (30 d)
- 35
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de basictech01/uniun
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
basictech01/uniun#300 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue P2-Medium
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
basictech01/uniun#296 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue P2-Medium
Dificultad 4/5 1-2 días Aptitud para principiantes 22/100
basictech01/uniun#293 ·
Los mantenedores suelen responder en 1 día
-
bug P1-High
Dificultad 4/5 3-5 días Aptitud para principiantes 18/100
basictech01/uniun#286 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
P3-Low
Dificultad 4/5 1-2 días Aptitud para principiantes 30/100
basictech01/uniun#282 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de basictech01/uniun
Issues similares
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
chriskelly/wish-together#36 ·
Los mantenedores suelen responder en 1 día
-
[Bug] Web Guide has broken linksAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
AppFlowy-IO/AppFlowy#9075 ·
-
Paste asks for iOS paste permission because the default context menu ignores SystemContextMenuPosiblemente ocupada @dorklein la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
maheshj01/searchfield#267 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
hajisensai/Fushi#2340 · 2 comentarios ·
Los mantenedores suelen responder en 1 día