microsoft / microsoft/TypeScript

[isolatedDeclarations] Add a syntactic form of computed property name which is always emitted as a computed property name

Đang mở
#58,800 7 bình luận 2 reaction 0 người được giao Xem trên GitHub

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

Domain: flag: isolatedDeclarations
Ngôn ngữ chính
Go
Star
111k
Fork
14.4k
Merge trung bình
1 ngày 19 giờ
Pull request đã merge (30 ngày)
117

Mô tả

🔍 Search Terms

isolatedDeclarations transpileModule computed property name

✅ Viability Checklist
⭐ Suggestion

Background

Computed property names under isolatedDeclarations are very limited right now. Today, you can write {[Symbol.iterator]: ...} and that's about it. This restriction is in place because for an arbitrary {[expression]: ...} we don't know if the type should be {[expression]: something}, {[expression: string]: something} or even {} (or a future {f1: something} | {f2: something}). Computed property names in types today have to exactly be a single late bindable name - nothing more, nothing less - meanwhile computed property names in object expressions (and class declarations) are much more flexible in what we allow.

Thus far, this has worked pretty well for TS users, since we basically pre-solve and cache whatever the expression computed name resolves to into our declaration files. Unfortunately, for isolatedDeclarations users, this poses a problem, since the expression in the computed property name may be from or rely on type information from another file. In such a case, it's impossible to know how to emit the type for the expression. You could optimistically emit {[expression]: something}, but if expression ends up evaluating to string or any in a whole-program context, the declaration file will produce an error and incorrect type information.

Proposal

What we could use in such a scenario is a syntactic opt-in to guaranteeing the preservation of a computed property name in the calculated type for an expression. A form of computed property name that, when you see it, always ensures a computed property name appears in the output, and issues checker errors if the types when checked cannot produce a valid computed property name in a declaration file.

I propose we reuse some existing syntax with a bit of a new meaning to accomplish this - a satisfies keyof postfix assertion, only valid in computed property name positions, and only on dotted entity name expressions. This would mean you could write

export const a = {
  [something satisfies keyof]: () => {}
}

and we would always emit

export const a: {
  [something]: () => void;
};

and issue an error on something satisfies keyof if something isn't exactly a single unique symbol, string literal, or number literal type (as is valid in the type position computed property name).

Compatibility

Only isolatedDeclarations-concerned authors really need to think about this feature - it's erased from declaration files, since they already check this constraint, so library consumers will never see it. People not using isolatedDeclarations will never be driven to use it, since they will always be able to produce a declaration type without an assertion. This is pretty easy to integrate into the isolatedDeclarations quickfixer. This doesn't conflict with existing satisfies keyof T assertions, since they require a type argument for keyof. There is also the possibility of allowing satisfies keyof in other locations and on arbitrary expression kinds in the future to check the same invariant - that the expression is exactly a single literal key type - if we think such a check has use in broader contexts than just computed property names.

Addenda: Making error cases better

Once we have {[expression satsifies keyof]: ...} in place, we know that that computed property name should always produce exactly one object key, even if expression doesn't produce a valid key type (and thus an error). In such a scenario, it could be beneficial to override the type of expression with a property key unique to the expression symbol, and then fallback to using such a symbol whenever later obj[expression] lookups fail. In this way, we can preserve as much user intent as possible, without rapidly reverting to an unchecked any state. This is neat (I have a working prototype), especially in the context of single-file checking modes like what our language service does when loading the full program in the background, but isn't really necessary for the feature. The open questions I have for this are just

  1. Is it worth supporting this scenario with a special case? and
  2. Should the keyof result of a type containing one of the fallback property keys be adjusted to be string | number | symbol? Should the fallback error property just be filtered from keyof entirely?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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.

Hướng nghiên cứu

Bắt đầu bằng cách truy vết việc xử lý isolatedDeclarations đối với các tên thuộc tính được tính toán và các quy tắc của checker đối với các khóa được tính toán trong declaration files. Xem xét cách isolatedDeclarations quickfixer có thể tích hợp cú pháp được đề xuất. Được xem là hoàn tất khi cú pháp, việc xác thực, declaration emit và các trường hợp lỗi liên quan đã được đặc tả và được kiểm thử.

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

Đánh giá

Công nghệ
typescript
Lĩnh vực
compilers
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
25/100

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.