IndexRepository: retire scope-blind read methods (find_by_owner, …)
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ó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Start with find_by_owner in infra/persistence/index/protocols.py and its implementation in IndexRepository, then follow the router pass-through to confirm nothing in src/ calls it. Decide whether to delete it or add app_id and project_id parameters, and check for sibling scope-blind reads. Done when no method addresses index rows without scope, the router no longer exposes the blind path, and the existing tests pass.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The primary-key fix removes get_by_id from IndexRepository because id is unique only within (app_id, project_id). find_by_owner(owner_id, *, limit) (infra/persistence/index/protocols.py) has the same blind spot — it reads across every app/project for an owner — and currently has no caller in src/ besides the router pass-through.
Remove it (and any sibling that addresses rows without scope) or give it scope parameters. Left out of the fix on purpose: zero callers, not on the bleeding path.
- Ngôn ngữ chính
- Python
- Star
- 13.3k
- Fork
- 926
- Merge trung bình
- 17 giờ 8 phút
- Pull request đã merge (30 ngày)
- 32
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 EverMind-AI/EverOS
-
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 62/100
EverMind-AI/EverOS#414 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Offer] Free independent recomputation of your published memory benchmarks (assayed-tier upgrade)Đang mở
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 3/100
EverMind-AI/EverOS#504 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
EverMind-AI/EverOS#501 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
EverMind-AI/EverOS#498 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
EverMind-AI/EverOS#485 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của EverMind-AI/EverOS
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ok-oldking/ok-wuthering-waves#1785 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
healthchecks/healthchecks#1349 ·
-
deployment release-lag
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
nolte/kamerplanter#2374 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v1 v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/python-sdk#3725 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
batchterms() drops queries without a similar() clause, misaligning results with inputsCó thể đã có người làm @MohammadHijjawi97 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày