Skip to content

Conversation

@Andarist
Copy link
Contributor

@Andarist Andarist commented Nov 5, 2025

fixes #62720

@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Nov 5, 2025
@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Nov 5, 2025
@jakebailey
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Nov 5, 2025

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top400 ✅ Started ✅ Results
user test this ✅ Started ✅ Results
run dt ✅ Started ✅ Results
perf test this faster ✅ Started 👀 Results

const inference = createInferenceInfo(typeParameter);
inferTypes([inference], sourceType, templateType);
return getTypeFromInference(inference) || unknownType;
return getWidenedType(getTypeFromInference(inference) || unknownType);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note for reviewers: this is what getCovariantInference already does

@typescript-bot
Copy link
Collaborator

Hey @jakebailey, the results of running the DT tests are ready.

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@jakebailey Here are the results of running the user tests with tsc comparing main and refs/pull/62722/merge:

There were infrastructure failures potentially unrelated to your change:

  • 1 instance of "Git clone failed"

Otherwise...

Everything looks good!

@typescript-bot typescript-bot added For Backlog Bug PRs that fix a backlog bug and removed For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Nov 5, 2025
@typescript-bot
Copy link
Collaborator

@jakebailey
The results of the perf run you requested are in!

Here they are:

tsc

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-Unions - node (v18.15.0, x64)
Errors 1 1 ~ ~ ~ p=1.000 n=6
Symbols 62,370 62,370 ~ ~ ~ p=1.000 n=6
Types 50,386 50,386 ~ ~ ~ p=1.000 n=6
Memory used 193,880k (± 0.95%) 195,057k (± 0.95%) ~ 192,645k 196,350k p=0.689 n=6
Parse Time 1.30s (± 0.80%) 1.30s (± 0.63%) ~ 1.29s 1.31s p=0.181 n=6
Bind Time 0.72s 0.72s (± 0.57%) ~ 0.72s 0.73s p=0.405 n=6
Check Time 9.87s (± 0.31%) 9.88s (± 0.33%) ~ 9.83s 9.92s p=0.628 n=6
Emit Time 2.73s (± 0.60%) 2.74s (± 0.61%) ~ 2.72s 2.76s p=0.615 n=6
Total Time 14.63s (± 0.25%) 14.65s (± 0.25%) ~ 14.60s 14.69s p=0.572 n=6
angular-1 - node (v18.15.0, x64)
Errors 2 2 ~ ~ ~ p=1.000 n=6
Symbols 956,047 956,049 +2 (+ 0.00%) ~ ~ p=0.001 n=6
Types 415,881 415,885 +4 (+ 0.00%) ~ ~ p=0.001 n=6
Memory used 1,255,044k (± 0.00%) 1,255,067k (± 0.00%) ~ 1,254,996k 1,255,125k p=0.336 n=6
Parse Time 6.54s (± 0.70%) 6.52s (± 0.44%) ~ 6.47s 6.55s p=0.624 n=6
Bind Time 1.88s (± 0.22%) 1.88s (± 0.34%) ~ 1.87s 1.89s p=0.673 n=6
Check Time 32.22s (± 0.25%) 32.16s (± 0.29%) ~ 32.01s 32.29s p=0.378 n=6
Emit Time 14.91s (± 0.26%) 14.92s (± 0.23%) ~ 14.87s 14.97s p=0.627 n=6
Total Time 55.55s (± 0.19%) 55.49s (± 0.16%) ~ 55.34s 55.61s p=0.261 n=6
mui-docs - node (v18.15.0, x64)
Errors 1 1 ~ ~ ~ p=1.000 n=6
Symbols 552,568 552,568 ~ ~ ~ p=1.000 n=6
Types 89 89 ~ ~ ~ p=1.000 n=6
Memory used 827,063k (± 0.00%) 827,078k (± 0.00%) ~ 827,056k 827,112k p=0.423 n=6
Parse Time 8.39s (± 0.51%) 8.37s (± 0.19%) ~ 8.34s 8.38s p=0.198 n=6
Bind Time 2.18s (± 0.54%) 2.19s (± 0.29%) ~ 2.18s 2.20s p=0.177 n=6
Check Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Emit Time 0.28s 0.28s ~ ~ ~ p=1.000 n=6
Total Time 10.85s (± 0.43%) 10.83s (± 0.08%) ~ 10.82s 10.84s p=0.568 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,237,277 1,237,673 +396 (+ 0.03%) ~ ~ p=0.001 n=6
Types 259,883 259,883 ~ ~ ~ p=1.000 n=6
Memory used 2,547,232k (±11.94%) 2,365,472k (± 0.01%) ~ 2,365,139k 2,366,028k p=0.471 n=6
Parse Time 5.14s (± 1.53%) 5.12s (± 0.45%) ~ 5.09s 5.16s p=1.000 n=6
Bind Time 1.76s (± 0.77%) 1.78s (± 1.30%) ~ 1.76s 1.82s p=0.361 n=6
Check Time 35.30s (± 0.84%) 35.20s (± 0.38%) ~ 34.98s 35.36s p=0.173 n=6
Emit Time 3.01s (± 1.20%) 2.99s (± 0.66%) ~ 2.98s 3.03s p=0.254 n=6
Total Time 45.22s (± 0.62%) 45.12s (± 0.31%) ~ 44.91s 45.32s p=0.378 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,237,277 1,237,673 +396 (+ 0.03%) ~ ~ p=0.001 n=6
Types 259,883 259,883 ~ ~ ~ p=1.000 n=6
Memory used 3,159,604k (± 0.02%) 3,159,135k (± 0.02%) ~ 3,158,123k 3,160,230k p=0.378 n=6
Parse Time 6.86s (± 0.85%) 6.87s (± 0.89%) ~ 6.79s 6.95s p=0.689 n=6
Bind Time 2.14s (± 1.31%) 2.13s (± 0.46%) ~ 2.12s 2.15s p=0.171 n=6
Check Time 42.78s (± 0.20%) 42.86s (± 0.29%) ~ 42.68s 43.02s p=0.298 n=6
Emit Time 3.51s (± 3.34%) 3.56s (± 1.60%) ~ 3.52s 3.67s p=0.688 n=6
Total Time 55.30s (± 0.27%) 55.41s (± 0.34%) ~ 55.11s 55.58s p=0.298 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,522 262,918 +396 (+ 0.15%) ~ ~ p=0.001 n=6
Types 104,048 104,048 ~ ~ ~ p=1.000 n=6
Memory used 440,693k (± 0.01%) 440,815k (± 0.01%) +122k (+ 0.03%) 440,751k 440,892k p=0.008 n=6
Parse Time 3.49s (± 0.93%) 3.51s (± 0.59%) ~ 3.48s 3.54s p=0.196 n=6
Bind Time 1.33s (± 1.20%) 1.32s (± 0.57%) ~ 1.31s 1.33s p=0.457 n=6
Check Time 19.06s (± 0.33%) 19.03s (± 0.47%) ~ 18.87s 19.13s p=0.518 n=6
Emit Time 1.53s (± 1.52%) 1.52s (± 0.54%) ~ 1.51s 1.53s p=0.100 n=6
Total Time 25.42s (± 0.31%) 25.38s (± 0.29%) ~ 25.26s 25.49s p=0.520 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 72 72 ~ ~ ~ p=1.000 n=6
Symbols 225,386 225,386 ~ ~ ~ p=1.000 n=6
Types 94,304 94,304 ~ ~ ~ p=1.000 n=6
Memory used 370,088k (± 0.04%) 370,083k (± 0.03%) ~ 369,994k 370,285k p=0.810 n=6
Parse Time 2.83s (± 1.38%) 2.81s (± 1.44%) ~ 2.75s 2.85s p=0.687 n=6
Bind Time 1.60s (± 1.02%) 1.60s (± 1.08%) ~ 1.58s 1.63s p=0.452 n=6
Check Time 16.53s (± 0.23%) 16.56s (± 0.44%) ~ 16.48s 16.69s p=0.630 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 20.96s (± 0.26%) 20.97s (± 0.17%) ~ 20.92s 21.02s p=1.000 n=6
vscode - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 4,016,547 4,016,547 ~ ~ ~ p=1.000 n=6
Types 1,260,033 1,260,033 ~ ~ ~ p=1.000 n=6
Memory used 3,797,947k (± 0.00%) 3,797,813k (± 0.00%) ~ 3,797,541k 3,798,009k p=0.471 n=6
Parse Time 15.56s (± 0.54%) 15.54s (± 0.49%) ~ 15.45s 15.65s p=0.809 n=6
Bind Time 5.02s (± 0.63%) 5.05s (± 0.78%) ~ 4.99s 5.09s p=0.172 n=6
Check Time 104.91s (± 3.93%) 105.54s (± 3.66%) ~ 101.88s 110.75s p=1.000 n=6
Emit Time 41.64s (±27.71%) 40.49s (±15.53%) ~ 31.40s 48.95s p=0.936 n=6
Total Time 167.13s (± 5.84%) 166.62s (± 5.11%) ~ 160.21s 177.82s p=0.810 n=6
webpack - node (v18.15.0, x64)
Errors 40 40 ~ ~ ~ p=1.000 n=6
Symbols 379,091 379,091 ~ ~ ~ p=1.000 n=6
Types 166,343 166,343 ~ ~ ~ p=1.000 n=6
Memory used 538,010k (± 0.01%) 538,010k (± 0.02%) ~ 537,882k 538,162k p=0.873 n=6
Parse Time 4.60s (± 0.53%) 4.61s (± 0.57%) ~ 4.57s 4.65s p=0.935 n=6
Bind Time 1.95s (± 0.39%) 1.92s (± 0.97%) -0.02s (- 1.28%) 1.90s 1.95s p=0.032 n=6
Check Time 22.63s (± 0.27%) 22.63s (± 0.36%) ~ 22.49s 22.70s p=0.686 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 29.18s (± 0.21%) 29.16s (± 0.26%) ~ 29.03s 29.24s p=0.683 n=6
xstate-main - node (v18.15.0, x64)
Errors 30 30 ~ ~ ~ p=1.000 n=6
Symbols 690,485 690,485 ~ ~ ~ p=1.000 n=6
Types 208,757 208,757 ~ ~ ~ p=1.000 n=6
Memory used 586,574k (± 0.03%) 586,541k (± 0.02%) ~ 586,413k 586,768k p=0.810 n=6
Parse Time 4.18s (± 0.89%) 4.18s (± 0.49%) ~ 4.15s 4.21s p=1.000 n=6
Bind Time 1.34s (± 1.28%) 1.35s (± 1.28%) ~ 1.32s 1.37s p=0.625 n=6
Check Time 20.69s (± 2.08%) 20.87s (± 2.05%) ~ 20.29s 21.24s p=0.229 n=6
Emit Time 0.00s 0.00s (±154.76%) ~ 0.00s 0.01s p=0.174 n=6
Total Time 26.21s (± 1.74%) 26.40s (± 1.68%) ~ 25.83s 26.80s p=0.377 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Compiler-Unions - node (v18.15.0, x64)
  • angular-1 - node (v18.15.0, x64)
  • mui-docs - node (v18.15.0, x64)
  • self-build-src - node (v18.15.0, x64)
  • self-build-src-public-api - node (v18.15.0, x64)
  • self-compiler - node (v18.15.0, x64)
  • ts-pre-modules - node (v18.15.0, x64)
  • vscode - node (v18.15.0, x64)
  • webpack - node (v18.15.0, x64)
  • xstate-main - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

@typescript-bot
Copy link
Collaborator

@jakebailey Here are the results of running the top 400 repos with tsc comparing main and refs/pull/62722/merge:

Everything looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

Status: Not started

Development

Successfully merging this pull request may close these issues.

Conditional type inference fails when generic parameter has Readonly<T> in function parameter position

3 participants