sessionbudget diverges from the documented Configure pattern
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 72/100
Hướng nghiên cứu
Đọc mẫu Configure trong docs/plugin-reference.md, sau đó kiểm tra core/plugins/sessionbudget/plugin.go quanh dòng 143 và cách xử lý tạm ở dòng 167-170. So sánh với một trong các plugin đã theo thứ tự strict-decode → applyDefaults → validate → state, rồi định hình lại Configure theo cách tương tự. Hoàn thành khi các khóa cấu hình gõ sai bị từ chối và các kiểm thử sessionbudget hiện có vẫn chạy thành công.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
What
docs/plugin-reference.md mandates the Configure pattern: strict decode (DisallowUnknownFields) → applyDefaults → validate → state. 11 of the 13 Configurable plugins follow it (verified). sessionbudget is the one that doesn't:
core/plugins/sessionbudget/plugin.go:143 — plain json.Unmarshal (no strict decode), defaults set before decode, then the "explicit 0 (or unset via struct-marshal) — restore default" workaround at :167-170 — the exact bug applyDefaults-after-decode exists to prevent — plus ~90 lines of validation inlined into Configure.
Fix
A mechanical re-shape to the documented pattern. No behavior change for correct configs; misspelled config keys start being rejected like every other plugin's.
Found in the 2026-10-29 structural audit of code organization.
- Ngôn ngữ chính
- Go
- Star
- 21
- Fork
- 43
- Merge trung bình
- 13 giờ 8 phút
- Pull request đã merge (30 ngày)
- 267
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 rossoctl/cortex
-
UI shows costs for all sessions even when drilled into a sessionCó thể đã có người làm @esnible đã nhận 16 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
rossoctl/cortex#1116 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 Hơn một tuần Mức phù hợp với người mới 18/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Cost/session layering: ledger snapshot logic in the HTTP layer; import edges point both waysĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 45/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của rossoctl/cortex
Issue tương tự
-
automation doc-healer documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
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 90/100
tsukumogami/tsuku#2804 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Anthropic /v1/messages: only the last thinking block of an assistant message is keptCó thể đã có người làm @Shayan12456 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
ollama/ollama#19415 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
raised-by:verifier
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
medici-finance/assay#5039 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
metax CheckHealth reads Status.Capacity, so unhealthy Metax-GPU nodes are never cleaned upCó thể đã có người làm @barbierajput378-pixel đã nhận hôm nay. Đang mởkind/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Project-HAMi/HAMi#3343 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày