【架构优化】删除 domain.WorkflowDefinition 中无调用方的 GroupMemberIDs/HasGroupMember 死代码

Đang mở
  • Có gì không ổn?
#408 0 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

@fx-kurumi đang làm issue này rồi.

Từ ngày 13/11/2026.

  • #409 của @fx-kurumi — đang mở

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
18/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
go
Lĩnh vực
backend

Hướng nghiên cứu

The two dead methods, GroupMemberIDs and HasGroupMember, live in internal/domain/workflow.go; run grep -rn "HasGroupMember\|GroupMemberIDs" first to confirm there are no callers. Keep the GroupMembers field and its JSON tag, since engine/run_start.go uses it. Done when both methods are removed and go build ./... and go test ./internal/domain/ ./internal/engine/ pass. A linked pull request (#409) is already open, so check it before starting.

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

Mô tả

现状问题

internal/domain/workflow.go 中 WorkflowDefinition 的两个方法没有任何调用方：

  • HasGroupMember(id string) bool：全仓库 0 处调用（含测试）；
  • GroupMemberIDs() []string：唯一调用方就是 HasGroupMember。

二者位于 internal/ 包，仓库外无法引用，属于死代码；同时 HasGroupMember 里还有一段可被 slices.Contains 替代的手写查找循环，留着会误导读者以为群组成员判断仍走这里。

重构方案

  • 删除 WorkflowDefinition.GroupMemberIDs 与 WorkflowDefinition.HasGroupMember 两个方法（共约 24 行）。
  • 保留 GroupMembers 字段及其 JSON 标签 groupMembers 不动（持久化/接口契约），engine/run_start.go 对该字段的使用不受影响。

影响范围

仅 internal/domain/workflow.go；无调用方，对外接口与运行行为不变。

验证方式

  • grep -rn "HasGroupMember\|GroupMemberIDs" 确认无残留引用；
  • gofmt、go vet ./internal/...、go build ./...、go test ./internal/domain/ ./internal/engine/ 通过。
Ngôn ngữ chính
Go
Star
141
Fork
28
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
113

Chuẩn bị môi trường

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 Tencent/LoopForge

Tất cả issue của Tencent/LoopForge

Issue tương tự

Thêm issue về Go

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.