Enh: domain/ layer imports isar_community directly in 3 files — violates Clean Architecture rule
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
"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).
- 主要言語
- Dart
- スター
- 2
- フォーク
- 2
- 平均マージ
- 1日 7時間
- マージ済み PR(30日)
- 35
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
basictech01/uniun のほかの issue
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
basictech01/uniun#300 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue P2-Medium
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
basictech01/uniun#296 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue P2-Medium
難易度 4/5 1〜2日 初心者へのやさしさ 22/100
basictech01/uniun#293 ·
メンテナーはふだん 1 日以内に返信
-
bug P1-High
難易度 4/5 3〜5日 初心者へのやさしさ 18/100
basictech01/uniun#286 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
P3-Low
難易度 4/5 1〜2日 初心者へのやさしさ 30/100
basictech01/uniun#282 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
basictech01/uniun の issue をすべて見る
似ている issue
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
chriskelly/wish-together#36 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
AppFlowy-IO/AppFlowy#9075 ·
-
Paste asks for iOS paste permission because the default context menu ignores SystemContextMenu対応中かも @dorklein が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
maheshj01/searchfield#267 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hajisensai/Fushi#2340 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信