microsoft / microsoft/TypeScript
Error not generated for nested generic array
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.3k
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 132
Description
🔎 Search Terms
error, nested, array, order
🕗 Version & Regression Information
- This changed between versions 4.4.4 and 4.5.5 (from playground testing - all later version exhibit this behaviour).
⏯ Playground Link
💻 Code
type TypeA = number;
type TypeB = boolean;
type DataTypeTwo<T extends TypeA | TypeB> = {
one: Array<{ two: Array<T> }>;
};
type DataTypeThree<T extends TypeA | TypeB> = {
one: Array<{ two: Array<{ three: Array<T> }> }>;
};
let testingThreeErr: DataTypeThree<TypeA> = {
one: [{ two: [ {three: [ true ] } ]}] // errors as expected
};
let testingTwoErr: DataTypeTwo<TypeA> = {
one: [{ two: [ false ] }] // errors as expected
};
// Uncomment these lines to see all errors
// let testingTwoPre: DataTypeTwo<TypeA> = {
// one: [{ two: [ 5 ] }]
// };
// let t2aPre: DataTypeTwo<TypeB> = testingTwoPre;
let testingThree: DataTypeThree<TypeA> = {
one: [{ two: [ {three: [ 5 ] } ]}]
};
// Comment out this line to see error for t2a assignment
let t3a: DataTypeThree<TypeB> = testingThree; // Should error, but does not
let testingTwo: DataTypeTwo<TypeA> = {
one: [{ two: [ 5 ] }]
};
let t2a: DataTypeTwo<TypeB> = testingTwo; // errors only if section above commented out
let check: number = "asdf"; // always errors, as it should
// Comment out L27 to see correct error on L32
// Uncomment L27 and lines 18-21 to see all errors correctly
// Works as expected in v4.4.4, but not any later versions
🙁 Actual behavior
No errors on lines 27 and 32 (assigning an incompatible type).
🙂 Expected behavior
Should error on those lines.
Additional information about the issue
The code works (i.e. gives correct errors) in version 4.4.4 (pg).
I assume this is to with depth limits, as it works fine for the two level case (DataTypeTwo) - though if the bad assignment comes after the three level case then even this error is not shown. (i.e. need to comment out line 27 in the example for the bad assignment to the DataTypeTwo variable to correctly error).
Even more oddly, if a bad assignment to a DataTypeTwo variable is made before the bad assignment to a DataTypeThree, the DataTypeThree assignment begins to error correctly. (Uncomment lines 18-21 in the example).
Possibly related to #56291.
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
Start with the linked TypeScript Playground and compare the reproduction in 4.4.4 with 5.6.3 or later. Investigate why the nested DataTypeTwo and DataTypeThree assignments affect whether incompatible assignments are reported, using the examples and the related issue #56291 as context. Done means the assignments on lines 27 and 32 reliably produce errors without depending on code order, with regression coverage added in the compiler tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100