Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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!

@Andarist Andarist force-pushed the fix/widen-reverse-mapped-type-properties branch from 3da0a6d to f395121 Compare December 3, 2025 11:36
@github-project-automation github-project-automation bot moved this from Not started to Needs merge in PR Backlog Dec 5, 2025
@jakebailey jakebailey merged commit 0752278 into microsoft:main Dec 5, 2025
33 checks passed
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Dec 5, 2025
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: Done

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.