Title: [isolatedDeclarations] Add a syntactic form of computed property name which is always emitted as a computed property name · Issue #58800 · microsoft/TypeScript · GitHub
Open Graph Title: [isolatedDeclarations] Add a syntactic form of computed property name which is always emitted as a computed property name · Issue #58800 · microsoft/TypeScript
X Title: [isolatedDeclarations] Add a syntactic form of computed property name which is always emitted as a computed property name · Issue #58800 · microsoft/TypeScript
Description: 🔍 Search Terms isolatedDeclarations transpileModule computed property name ✅ Viability Checklist This wouldn't be a breaking change in existing TypeScript/JavaScript code This wouldn't change the runtime behavior of existing JavaScript c...
Open Graph Description: 🔍 Search Terms isolatedDeclarations transpileModule computed property name ✅ Viability Checklist This wouldn't be a breaking change in existing TypeScript/JavaScript code This wouldn't change the r...
X Description: 🔍 Search Terms isolatedDeclarations transpileModule computed property name ✅ Viability Checklist This wouldn't be a breaking change in existing TypeScript/JavaScript code This wouldn't chan...
Opengraph URL: https://github.com/microsoft/TypeScript/issues/58800
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[isolatedDeclarations] Add a syntactic form of computed property name which is always emitted as a computed property name","articleBody":"### 🔍 Search Terms\r\n\r\nisolatedDeclarations transpileModule computed property name\r\n\r\n### ✅ Viability Checklist\r\n\r\n- [X] This wouldn't be a breaking change in existing TypeScript/JavaScript code\r\n- [X] This wouldn't change the runtime behavior of existing JavaScript code\r\n- [X] This could be implemented without emitting different JS based on the types of the expressions\r\n- [X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)\r\n- [X] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types\r\n- [X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals\r\n\r\n### ⭐ Suggestion\r\n\r\n# Background\r\n\r\nComputed 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.\r\n\r\nThus 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.\r\n\r\n# Proposal\r\nWhat 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.\r\n\r\nI 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\r\n```ts\r\nexport const a = {\r\n [something satisfies keyof]: () =\u003e {}\r\n}\r\n```\r\nand we would always emit\r\n```ts\r\nexport const a: {\r\n [something]: () =\u003e void;\r\n};\r\n```\r\nand 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).\r\n\r\n# Compatibility\r\n\r\nOnly `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.\r\n\r\n# Addenda: Making error cases better\r\n\r\nOnce 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\r\n1. Is it worth supporting this scenario with a special case? and\r\n2. 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?\r\n","author":{"url":"https://github.com/weswigham","@type":"Person","name":"weswigham"},"datePublished":"2024-06-07T16:17:56.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":7},"url":"https://github.com/58800/TypeScript/issues/58800"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:08a57de0-ccaf-eca1-ec33-403c5aab1601 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | EA00:39B1FE:220A9CB:2FA223C:6A5F5202 |
| html-safe-nonce | b853afbe8c0afb32ec33400fdd5fdc2eb5b5932565b62268c48e3736eb13da40 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFQTAwOjM5QjFGRToyMjBBOUNCOjJGQTIyM0M6NkE1RjUyMDIiLCJ2aXNpdG9yX2lkIjoiMjY0OTUzMDgyOTUwOTcxMDMzOCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 955a54ee5d5a55747a4b85721c4d33c36489e567560931670ca342d30c089ab5 |
| hovercard-subject-tag | issue:2340791006 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/microsoft/TypeScript/58800/issue_layout |
| twitter:image | https://opengraph.githubassets.com/b6825a84634e4c342d110be43e888c07dfbc9ca990f6af88ba675ccc00a89ae1/microsoft/TypeScript/issues/58800 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/b6825a84634e4c342d110be43e888c07dfbc9ca990f6af88ba675ccc00a89ae1/microsoft/TypeScript/issues/58800 |
| og:image:alt | 🔍 Search Terms isolatedDeclarations transpileModule computed property name ✅ Viability Checklist This wouldn't be a breaking change in existing TypeScript/JavaScript code This wouldn't change the r... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | weswigham |
| hostname | github.com |
| expected-hostname | github.com |
| None | 9a0797c11799dbfd7df57fa3de31c7ea3a4c4bd055eb1a0427a5cde1d4d7ba3f |
| turbo-cache-control | no-preview |
| go-import | github.com/microsoft/TypeScript git https://github.com/microsoft/TypeScript.git |
| octolytics-dimension-user_id | 6154722 |
| octolytics-dimension-user_login | microsoft |
| octolytics-dimension-repository_id | 20929025 |
| octolytics-dimension-repository_nwo | microsoft/TypeScript |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 20929025 |
| octolytics-dimension-repository_network_root_nwo | microsoft/TypeScript |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | b099d96da147aea31036cd16b2fba0083a6defe4 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width