Enh: domain/ layer imports isar_community directly in 3 files — violates Clean Architecture rule
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/100
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
"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).
- Ngôn ngữ chính
- Dart
- Star
- 2
- Fork
- 2
- Merge trung bình
- 1 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 35
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của basictech01/uniun
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
basictech01/uniun#300 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue P2-Medium
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
basictech01/uniun#296 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug good first issue P2-Medium
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 22/100
basictech01/uniun#293 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug P1-High
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 18/100
basictech01/uniun#286 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
P3-Low
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 30/100
basictech01/uniun#282 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của basictech01/uniun
Issue tương tự
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
chriskelly/wish-together#36 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] Web Guide has broken linksĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
AppFlowy-IO/AppFlowy#9075 ·
-
[web] Semantics `lang` attribute uses `Locale.toString()`, producing invalid tags like `es_US`Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Paste asks for iOS paste permission because the default context menu ignores SystemContextMenuCó thể đã có người làm @dorklein đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
maheshj01/searchfield#267 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
jhomlala/betterplayer#1616 ·
Maintainer thường phản hồi trong vòng 1 ngày