Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

BL-15: One-shot dispute resolution (one dispute per batch, ever) is undocumented

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#248 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

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ó
1/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
82/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
solidity
Lĩnh vực
blockchain, documentation

Hướng nghiên cứu

Đọc DINTaskCoordinator.sol tại openDispute và resolveDispute, sau đó xem lại phần giải quyết tranh chấp trong Developer/design/slashing-taxonomy.md. Thêm chú thích về ràng buộc chỉ thực hiện một lần và ghi lại quyết định thiết kế, bao gồm cả nơi cần xem xét lại nếu muốn hỗ trợ nhiều tranh chấp. Hoàn tất khi cả hai bản cập nhật tài liệu đều mô tả rằng một batch đã được giải quyết không thể bị tranh chấp lần nữa; không dự kiến thay đổi hành vi.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Approved documentation

Summary

DINTaskCoordinator.resolveDispute sets disputes[gi][tierKind][batchId].resolved = true with no path back. Once a dispute on a batch is resolved — upheld or rejected — no further dispute can be opened for that batch:

// DINTaskCoordinator.sol — openDispute
if (disputes[_GI][_tierKind][_batchId].resolved) revert TC_DisputeAlreadyResolved();

This means exactly one dispute per batch is possible in a GI's lifetime. This is a reasonable design choice at the scaffold stage, but it was never recorded:

  • No comment in resolveDispute or openDispute explains the one-shot constraint.
  • Developer/design/slashing-taxonomy.md describes the dispute flow without noting the limit.

If a future reviewer or implementer assumes disputes can be reopened (e.g. if new evidence surfaces), they will either silently hit TC_DisputeAlreadyResolved or incorrectly treat it as a bug to fix, potentially introducing unintended multi-dispute behaviour.

Fix

Documentation-only — no behaviour change.

  1. Add a one-line comment above resolveDispute in DINTaskCoordinator.sol: 
    // One dispute per batch per GI: once resolved (upheld or rejected), the batch cannot be re-disputed.
  2. Add a note to Developer/design/slashing-taxonomy.md in the dispute resolution section recording this as an explicit design decision and noting where to revisit it if multi-dispute-per-batch is ever desired.

When to upgrade

If multi-dispute-per-batch is later needed, the fix is to replace the boolean resolved flag with a counter or a re-openable state enum. That decision belongs in a separate issue.

References

  • Developer/BACK_LOG.md BL-15
  • PR #66 review (original observation)
  • DINTaskCoordinator.sol — openDispute / resolveDispute
  • Developer/design/slashing-taxonomy.md — dispute resolution section
Ngôn ngữ chính
Python
Star
4
Fork
7
Merge trung bình
3 ngày 4 giờ
Pull request đã merge (30 ngày)
34

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

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của InfiniteZeroFoundation/DevNet

Tất cả issue của InfiniteZeroFoundation/DevNet

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.