microsoft / microsoft/TypeScript
A way to specify class properties using JSDoc
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.3k
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 132
Description
Search Terms
JSDoc class properties any key @property
Suggestion
I would like to be able to extend the type of an ES6 class using JSDoc comments, such that TypeScript understands there are other properties that might be added to the object later.
Alternatively, I would like a way to indicate that a given class acts as a container that can contain any possible key.
Use Cases
I have a class that is used like a DTO. It has some properties that are guaranteed to be there, but is also used for a lot of optional properties. Take, for example:
class DTO {
constructor(id) {
/**
* @type {string}
*/
this.id = id;
}
}
TypeScript now recognizes the object type, but whenever I try to use other properties, it complains. Currently, I'm resorting to something like this as a work-around:
class DTO {
constructor(id) {
/**
* @type {string}
*/
this.id = id;
/**
* @type {Object?}
*/
this.otherProperty;
}
}
But it's ugly and verbose, and worse, it includes actual JavaScript code, that serves no purpose other than to provide type information.
Examples
Rather, what I would like to do is something like this (that I would like to be equivalent to the snippet above):
/**
* @property {Object} [otherProperty]
*/
class DTO {
constructor(id) {
/**
* @type {string}
*/
this.id = id;
}
}
Another equivalent alternative could be (but currently doesn't compile because of a "Duplicate identifier" error):
/**
* @typedef {Object} DTO
* @property {string} id
* @property {Object} [otherProperty]
*/
class DTO {
constructor(id) {
this.id = id;
}
}
Or, otherwise, some way to indicate this class can be extended with anything. Which means I would like a way to specify the following TypeScript using JSDoc and I want it to apply to an existing ES6 class (because I do use the class to be able to do instanceof checks):
interface DTO {
id: string,
[key: string]: any,
}
Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript / JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. new expression-level syntax)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are named; begin by reading the JSDoc class and property examples and the requested ES6-class behavior. Done means an agreed JSDoc form lets TypeScript recognize declared or arbitrary additional properties on an existing class without adding runtime code.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, typescript
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100