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

FeatureSet cleanup: simplify over-complex type-system remnants in compatibility_hints.py

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

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
45/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
python
Lĩnh vực
backend-api-design

Hướng nghiên cứu

Start with docs/design/TODO_COMPATIBILITY_HINTS.md and review the listed methods in caldav/compatibility_hints.py. The issue also calls for migrating old_flags in about 12 server config dictionaries, so identify those locations and check how their features are represented before changing them. Done means the listed cleanup and migration are complete, with the shim removed where no longer needed.

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

Mô tả

⚠️ This comment is AI-generated (Claude Sonnet 4.6 via Claude Code) on behalf of tobixen

The FeatureSet class in caldav/compatibility_hints.py was written with a more ambitious vision than what ended up being needed. A review of all its methods found several items that can be simplified or removed. Details are documented in docs/design/TODO_COMPATIBILITY_HINTS.md.

Summary of findings

1. Dead/buggy bool branch in _convert_node
The else branch for non-server-feature types has inverted logic (not node.get('enable') returns True when enable=False) and is unreachable in practice — no caller uses is_supported(bool) on client-feature or server-peculiarity features. Should be replaced with an AssertionError to surface accidental misuse.

2. Possibly redundant _derive_from_subfeatures call in is_supported
For features with no dots, the method is called twice in the same lookup path (once in the loop body, once in the post-loop block). Needs careful review to confirm if one call is truly redundant.

3. _old_flags migration (ongoing)
Already marked ## TODO: remove this when it can be removed. ~12 server config dicts still carry old_flags lists. Each needs to be translated into new-style dotted features, then the shim code can be deleted.

4. feature_tree / _dots_to_tree can be simplified
Builds and caches a full nested-dict tree only to answer one-level subfeature lookups in find_feature. Can be replaced with a one-liner list comprehension, eliminating both methods.

5. _collapse_key carries vestigial enable/observed fields
These slots exist for the complex type vision but collapse is only ever called on server-feature nodes. Simplify to just return feature_dict.get('support').

6. copyFeatureSet should be renamed copy_feature_set
Inconsistent camelCase already noted with a TODO comment. No external callers; safe rename.

Suggested order

  • Items 3 and 6 are independent and safe to do first.
  • Items 1, 2, 4, 5 are interrelated around the type-system simplification and are best tackled together.

⚠️ This comment is AI-generated (Claude Sonnet 4.6 via Claude Code) on behalf of tobixen

Ngôn ngữ chính
Python
Star
412
Fork
113
Merge trung bình
11 giờ 37 phút
Pull request đã merge (30 ngày)
50

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 python-caldav/caldav

Tất cả issue của python-caldav/caldav

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.