Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Computed names in declarations files are resolved even when non-literal, preserve computed names when expressions are entity names #60052

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

weswigham
Copy link
Member

This PR basically removes the grammar error requiring computed names in declaration files/other contexts resolve to literal types, and allows them to resolve to whatever so long as they're an entity name expression (a.b.c), and endeavors in declaration emit to preserve those input computed names that imply index signatures wherever possible. This clears the way for isolatedDeclarations to allow always emitting a computed property name on the output for a computed property name on the input, so long as the expression is an entity name expression.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Sep 24, 2024
@weswigham
Copy link
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Sep 24, 2024

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

@typescript-bot
Copy link
Collaborator

Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page.

Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up.

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

effect

tsconfig.json

tsconfig.base.json

packages/schema/dtslint/tsconfig.json

@typescript-bot
Copy link
Collaborator

@weswigham
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 30 30 ~ ~ ~ p=1.000 n=6
Symbols 62,340 62,340 ~ ~ ~ p=1.000 n=6
Types 50,378 50,378 ~ ~ ~ p=1.000 n=6
Memory used 193,214k (± 0.82%) 195,011k (± 0.98%) ~ 192,422k 196,310k p=0.298 n=6
Parse Time 1.93s (± 0.89%) 1.96s (± 0.42%) +0.03s (+ 1.64%) 1.95s 1.97s p=0.006 n=6
Bind Time 1.05s (± 0.79%) 1.07s (± 0.98%) ~ 1.05s 1.08s p=0.122 n=6
Check Time 13.98s (± 0.48%) 13.97s (± 0.27%) ~ 13.91s 14.00s p=0.872 n=6
Emit Time 4.09s (± 4.96%) 4.04s (± 3.26%) ~ 3.97s 4.31s p=0.572 n=6
Total Time 21.05s (± 0.79%) 21.03s (± 0.54%) ~ 20.94s 21.25s p=0.686 n=6
angular-1 - node (v18.15.0, x64)
Errors 7 7 ~ ~ ~ p=1.000 n=6
Symbols 947,102 947,102 ~ ~ ~ p=1.000 n=6
Types 410,738 410,738 ~ ~ ~ p=1.000 n=6
Memory used 1,224,040k (± 0.00%) 1,224,060k (± 0.00%) ~ 1,224,033k 1,224,144k p=0.689 n=6
Parse Time 6.62s (± 0.57%) 6.61s (± 0.67%) ~ 6.56s 6.69s p=0.688 n=6
Bind Time 1.87s (± 0.22%) 1.87s (± 0.40%) ~ 1.86s 1.88s p=1.000 n=6
Check Time 31.30s (± 0.25%) 31.41s (± 0.34%) ~ 31.27s 31.56s p=0.064 n=6
Emit Time 15.05s (± 0.41%) 15.03s (± 0.28%) ~ 14.98s 15.07s p=0.288 n=6
Total Time 54.84s (± 0.25%) 54.93s (± 0.31%) ~ 54.68s 55.14s p=0.378 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,521,651 2,521,652 +1 (+ 0.00%) ~ ~ p=0.001 n=6
Types 936,037 936,038 +1 (+ 0.00%) ~ ~ p=0.001 n=6
Memory used 2,351,731k (± 0.00%) 2,351,955k (± 0.00%) +224k (+ 0.01%) 2,351,914k 2,351,986k p=0.005 n=6
Parse Time 9.29s (± 0.33%) 9.29s (± 0.22%) ~ 9.26s 9.32s p=0.871 n=6
Bind Time 2.14s (± 0.35%) 2.14s (± 0.55%) ~ 2.12s 2.15s p=0.933 n=6
Check Time 73.26s (± 0.29%) 73.48s (± 0.76%) ~ 73.00s 74.46s p=0.689 n=6
Emit Time 0.28s (± 3.19%) 0.27s (± 3.31%) ~ 0.26s 0.28s p=0.113 n=6
Total Time 84.97s (± 0.27%) 85.17s (± 0.66%) ~ 84.68s 86.18s p=0.688 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,250,056 1,250,396 +340 (+ 0.03%) ~ ~ p=0.001 n=6
Types 264,970 265,020 +50 (+ 0.02%) ~ ~ p=0.001 n=6
Memory used 2,404,455k (± 0.03%) 2,405,163k (± 0.03%) +708k (+ 0.03%) 2,404,332k 2,405,839k p=0.045 n=6
Parse Time 5.12s (± 0.51%) 5.12s (± 0.43%) ~ 5.08s 5.14s p=1.000 n=6
Bind Time 1.90s (± 0.47%) 1.91s (± 0.39%) ~ 1.90s 1.92s p=0.149 n=6
Check Time 34.91s (± 0.30%) 34.95s (± 0.37%) ~ 34.70s 35.07s p=0.422 n=6
Emit Time 3.02s (± 3.84%) 2.99s (± 0.89%) ~ 2.95s 3.03s p=0.748 n=6
Total Time 44.97s (± 0.45%) 44.98s (± 0.33%) ~ 44.70s 45.11s p=0.298 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,250,056 1,250,396 +340 (+ 0.03%) ~ ~ p=0.001 n=6
Types 264,970 265,020 +50 (+ 0.02%) ~ ~ p=0.001 n=6
Memory used 2,478,660k (± 0.02%) 2,479,162k (± 0.02%) ~ 2,478,578k 2,480,282k p=0.093 n=6
Parse Time 6.37s (± 0.30%) 6.35s (± 0.94%) ~ 6.25s 6.41s p=1.000 n=6
Bind Time 2.05s (± 0.57%) 2.06s (± 0.60%) ~ 2.04s 2.07s p=0.625 n=6
Check Time 41.61s (± 0.92%) 41.82s (± 0.83%) ~ 41.44s 42.19s p=0.298 n=6
Emit Time 3.58s (± 1.28%) 3.67s (± 3.57%) ~ 3.51s 3.85s p=0.173 n=6
Total Time 53.63s (± 0.71%) 53.92s (± 0.58%) ~ 53.50s 54.21s p=0.128 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 260,936 261,221 +285 (+ 0.11%) ~ ~ p=0.001 n=6
Types 106,385 106,435 +50 (+ 0.05%) ~ ~ p=0.001 n=6
Memory used 436,588k (± 0.03%) 436,904k (± 0.01%) +316k (+ 0.07%) 436,835k 436,951k p=0.005 n=6
Parse Time 2.85s (± 0.31%) 2.86s (± 0.97%) ~ 2.83s 2.90s p=0.683 n=6
Bind Time 1.08s (± 1.43%) 1.07s (± 1.40%) ~ 1.06s 1.09s p=0.423 n=6
Check Time 15.37s (± 0.35%) 15.46s (± 0.25%) +0.09s (+ 0.56%) 15.39s 15.51s p=0.020 n=6
Emit Time 1.28s (± 1.45%) 1.29s (± 1.61%) ~ 1.26s 1.32s p=0.744 n=6
Total Time 20.58s (± 0.30%) 20.68s (± 0.36%) ~ 20.56s 20.75s p=0.065 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 68 68 ~ ~ ~ p=1.000 n=6
Symbols 225,916 225,916 ~ ~ ~ p=1.000 n=6
Types 94,414 94,414 ~ ~ ~ p=1.000 n=6
Memory used 370,841k (± 0.02%) 370,962k (± 0.02%) +121k (+ 0.03%) 370,873k 371,066k p=0.013 n=6
Parse Time 2.76s (± 1.24%) 2.79s (± 0.67%) ~ 2.76s 2.81s p=0.259 n=6
Bind Time 1.57s (± 1.38%) 1.58s (± 1.06%) ~ 1.56s 1.60s p=0.463 n=6
Check Time 15.78s (± 0.40%) 15.77s (± 0.22%) ~ 15.73s 15.82s p=0.467 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.11s (± 0.44%) 20.14s (± 0.12%) ~ 20.12s 20.18s p=0.872 n=6
vscode - node (v18.15.0, x64)
Errors 1 1 ~ ~ ~ p=1.000 n=6
Symbols 3,099,775 3,099,775 ~ ~ ~ p=1.000 n=6
Types 1,068,973 1,068,973 ~ ~ ~ p=1.000 n=6
Memory used 3,196,811k (± 0.00%) 3,197,123k (± 0.00%) +312k (+ 0.01%) 3,197,050k 3,197,255k p=0.005 n=6
Parse Time 17.20s (± 0.22%) 17.23s (± 0.41%) ~ 17.17s 17.37s p=0.332 n=6
Bind Time 5.42s (± 2.44%) 5.39s (± 2.61%) ~ 5.27s 5.58s p=0.936 n=6
Check Time 100.10s (± 0.42%) 100.11s (± 0.39%) ~ 99.73s 100.78s p=1.000 n=6
Emit Time 27.43s (± 0.72%) 27.45s (± 0.48%) ~ 27.31s 27.67s p=0.689 n=6
Total Time 150.15s (± 0.33%) 150.18s (± 0.39%) ~ 149.55s 151.03s p=0.689 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 278,920 278,920 ~ ~ ~ p=1.000 n=6
Types 113,837 113,837 ~ ~ ~ p=1.000 n=6
Memory used 428,349k (± 0.02%) 428,371k (± 0.02%) ~ 428,277k 428,542k p=0.936 n=6
Parse Time 3.95s (± 0.44%) 3.94s (± 0.31%) ~ 3.93s 3.96s p=0.622 n=6
Bind Time 1.72s (± 0.52%) 1.72s (± 0.57%) ~ 1.71s 1.73s p=0.798 n=6
Check Time 17.69s (± 0.24%) 17.61s (± 0.39%) ~ 17.54s 17.72s p=0.065 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 23.36s (± 0.18%) 23.28s (± 0.32%) ~ 23.19s 23.39s p=0.073 n=6
xstate-main - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 540,027 540,027 ~ ~ ~ p=1.000 n=6
Types 181,292 181,292 ~ ~ ~ p=1.000 n=6
Memory used 483,548k (± 0.01%) 483,586k (± 0.01%) ~ 483,525k 483,677k p=0.298 n=6
Parse Time 4.02s (± 0.52%) 4.01s (± 0.37%) ~ 3.99s 4.03s p=0.514 n=6
Bind Time 1.42s (± 1.45%) 1.43s (± 0.53%) ~ 1.42s 1.44s p=0.935 n=6
Check Time 22.90s (± 0.55%) 22.91s (± 0.36%) ~ 22.79s 23.04s p=0.873 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 28.35s (± 0.43%) 28.35s (± 0.29%) ~ 28.25s 28.50s p=1.000 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

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

Something interesting changed - please have a look.

Details

alibaba/lowcode-engine

40 of 42 projects failed to build with the old tsc and were ignored

modules/code-generator/tsconfig.json

FormidableLabs/victory

9 of 63 projects failed to build with the old tsc and were ignored

packages/victory-axis/tsconfig.json

packages/victory-axis/tsconfig.build.json

tinacms/tinacms

13 of 25 projects failed to build with the old tsc and were ignored

packages/@tinacms/schema-tools/tsconfig.json

packages/@tinacms/graphql/tsconfig.json

packages/@tinacms/cli/tsconfig.json

tusen-ai/naive-ui

3 of 6 projects failed to build with the old tsc and were ignored

src/tsconfig.demo.json

vuetifyjs/vuetify

3 of 7 projects failed to build with the old tsc and were ignored

packages/vuetify/tsconfig.json

packages/vuetify/tsconfig.dist.json

packages/vuetify/tsconfig.checks.json

@MichaelMitchell-at
Copy link

MichaelMitchell-at commented Sep 25, 2024

Amazing! Has the team already discussed the intent to ship this feature or is this only an experiment? Asking because I'm wondering if https://github.com/oxc-project/oxc/tree/main/crates/oxc_isolated_declarations could bypass the need to implement something like #58771 (oxc-project/oxc#4016) and implement the behavior in this PR directly.

@weswigham
Copy link
Member Author

You can't ship this before we do because it produces declaration files that can't be correctly read by current versions of TS 😉

@weswigham weswigham force-pushed the computed-names-in-declarations-files-are-resolved branch from 7fec56c to 22c5c89 Compare October 1, 2024 17:31
@weswigham weswigham force-pushed the computed-names-in-declarations-files-are-resolved branch from f1fcd56 to 587f9e0 Compare October 1, 2024 17:58
@weswigham
Copy link
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 1, 2024

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

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

effect

tsconfig.json

tsconfig.base.json

packages/schema/dtslint/tsconfig.json

@typescript-bot
Copy link
Collaborator

@weswigham
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 31 31 ~ ~ ~ p=1.000 n=6
Symbols 62,340 62,340 ~ ~ ~ p=1.000 n=6
Types 50,379 50,379 ~ ~ ~ p=1.000 n=6
Memory used 193,378k (± 0.73%) 192,924k (± 0.09%) ~ 192,779k 193,152k p=0.471 n=6
Parse Time 1.31s (± 0.57%) 1.31s (± 0.57%) ~ 1.30s 1.32s p=1.000 n=6
Bind Time 0.72s 0.72s ~ ~ ~ p=1.000 n=6
Check Time 9.75s (± 0.53%) 9.74s (± 0.25%) ~ 9.71s 9.78s p=0.872 n=6
Emit Time 2.72s (± 0.59%) 2.73s (± 0.51%) ~ 2.70s 2.74s p=0.315 n=6
Total Time 14.50s (± 0.35%) 14.50s (± 0.18%) ~ 14.46s 14.53s p=1.000 n=6
angular-1 - node (v18.15.0, x64)
Errors 33 33 ~ ~ ~ p=1.000 n=6
Symbols 947,886 947,886 ~ ~ ~ p=1.000 n=6
Types 410,840 410,840 ~ ~ ~ p=1.000 n=6
Memory used 1,224,800k (± 0.01%) 1,224,879k (± 0.00%) +78k (+ 0.01%) 1,224,780k 1,224,952k p=0.031 n=6
Parse Time 8.05s (± 0.83%) 8.06s (± 0.86%) ~ 7.97s 8.16s p=0.810 n=6
Bind Time 2.26s (± 0.72%) 2.27s (± 1.24%) ~ 2.24s 2.32s p=0.739 n=6
Check Time 37.72s (± 0.43%) 37.62s (± 0.41%) ~ 37.39s 37.79s p=0.296 n=6
Emit Time 18.08s (± 0.82%) 18.17s (± 0.42%) ~ 18.09s 18.25s p=0.468 n=6
Total Time 66.11s (± 0.24%) 66.11s (± 0.22%) ~ 65.89s 66.27s p=0.873 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,522,038 2,522,038 ~ ~ ~ p=1.000 n=6
Types 936,053 936,053 ~ ~ ~ p=1.000 n=6
Memory used 2,352,501k (± 0.00%) 2,352,657k (± 0.00%) +156k (+ 0.01%) 2,352,576k 2,352,706k p=0.005 n=6
Parse Time 9.37s (± 0.23%) 9.33s (± 0.25%) -0.04s (- 0.46%) 9.29s 9.36s p=0.014 n=6
Bind Time 2.16s (± 0.24%) 2.16s (± 0.29%) ~ 2.15s 2.17s p=0.386 n=6
Check Time 75.87s (± 0.27%) 75.65s (± 0.17%) ~ 75.49s 75.79s p=0.066 n=6
Emit Time 0.28s (± 2.95%) 0.28s (± 2.26%) ~ 0.27s 0.29s p=0.432 n=6
Total Time 87.67s (± 0.23%) 87.41s (± 0.15%) ~ 87.26s 87.54s p=0.066 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,106 1,258,459 +353 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,266 266,354 +88 (+ 0.03%) ~ ~ p=0.001 n=6
Memory used 3,146,790k (± 0.03%) 3,146,909k (± 0.02%) ~ 3,146,389k 3,147,720k p=1.000 n=6
Parse Time 6.59s (± 0.55%) 6.59s (± 0.54%) ~ 6.54s 6.63s p=0.810 n=6
Bind Time 2.35s (± 0.73%) 2.34s (± 1.82%) ~ 2.28s 2.41s p=0.468 n=6
Check Time 43.00s (± 0.25%) 43.08s (± 0.33%) ~ 42.86s 43.28s p=0.378 n=6
Emit Time 3.51s (± 2.00%) 3.52s (± 1.40%) ~ 3.45s 3.60s p=0.936 n=6
Total Time 55.45s (± 0.31%) 55.54s (± 0.31%) ~ 55.27s 55.79s p=0.471 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,106 1,258,459 +353 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,266 266,354 +88 (+ 0.03%) ~ ~ p=0.001 n=6
Memory used 2,496,029k (± 0.01%) 2,496,975k (± 0.01%) +946k (+ 0.04%) 2,496,399k 2,497,391k p=0.008 n=6
Parse Time 5.41s (± 0.54%) 5.37s (± 1.00%) ~ 5.32s 5.47s p=0.107 n=6
Bind Time 1.74s (± 0.48%) 1.74s (± 0.32%) ~ 1.73s 1.74s p=0.855 n=6
Check Time 35.83s (± 0.20%) 35.82s (± 0.16%) ~ 35.75s 35.92s p=0.810 n=6
Emit Time 3.05s (± 1.10%) 3.05s (± 2.13%) ~ 2.93s 3.12s p=1.000 n=6
Total Time 46.03s (± 0.21%) 45.98s (± 0.27%) ~ 45.91s 46.23s p=0.378 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 261,785 262,083 +298 (+ 0.11%) ~ ~ p=0.001 n=6
Types 106,508 106,596 +88 (+ 0.08%) ~ ~ p=0.001 n=6
Memory used 438,475k (± 0.01%) 438,838k (± 0.01%) +363k (+ 0.08%) 438,803k 438,875k p=0.005 n=6
Parse Time 2.90s (± 0.42%) 2.88s (± 0.42%) ~ 2.87s 2.90s p=0.138 n=6
Bind Time 1.10s 1.10s ~ ~ ~ p=1.000 n=6
Check Time 15.71s (± 0.25%) 15.72s (± 0.27%) ~ 15.66s 15.78s p=0.809 n=6
Emit Time 1.31s (± 1.31%) 1.32s (± 0.96%) ~ 1.31s 1.34s p=0.359 n=6
Total Time 21.02s (± 0.22%) 21.03s (± 0.19%) ~ 20.98s 21.07s p=0.809 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 68 68 ~ ~ ~ p=1.000 n=6
Symbols 225,919 225,919 ~ ~ ~ p=1.000 n=6
Types 94,415 94,415 ~ ~ ~ p=1.000 n=6
Memory used 371,091k (± 0.02%) 371,081k (± 0.01%) ~ 371,048k 371,129k p=0.575 n=6
Parse Time 2.88s (± 1.13%) 2.90s (± 0.55%) ~ 2.89s 2.93s p=0.224 n=6
Bind Time 1.60s (± 1.84%) 1.58s (± 0.89%) ~ 1.56s 1.60s p=0.258 n=6
Check Time 16.32s (± 0.24%) 16.30s (± 0.28%) ~ 16.25s 16.38s p=0.573 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.80s (± 0.37%) 20.79s (± 0.20%) ~ 20.74s 20.85s p=1.000 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,110,299 3,110,299 ~ ~ ~ p=1.000 n=6
Types 1,072,179 1,072,179 ~ ~ ~ p=1.000 n=6
Memory used 3,202,479k (± 0.01%) 3,203,049k (± 0.00%) +570k (+ 0.02%) 3,202,916k 3,203,338k p=0.005 n=6
Parse Time 14.02s (± 0.33%) 14.02s (± 0.45%) ~ 13.94s 14.09s p=0.872 n=6
Bind Time 4.44s (± 0.44%) 4.43s (± 0.24%) ~ 4.42s 4.45s p=0.871 n=6
Check Time 85.29s (± 1.42%) 86.61s (± 3.30%) ~ 84.75s 92.09s p=0.575 n=6
Emit Time 24.92s (± 9.42%) 24.08s (± 9.10%) ~ 22.53s 26.93s p=0.377 n=6
Total Time 128.67s (± 2.09%) 129.14s (± 2.74%) ~ 125.83s 133.40s p=0.936 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 279,484 279,484 ~ ~ ~ p=1.000 n=6
Types 114,024 114,024 ~ ~ ~ p=1.000 n=6
Memory used 429,365k (± 0.03%) 429,460k (± 0.04%) ~ 429,295k 429,739k p=0.230 n=6
Parse Time 5.02s (± 0.98%) 5.06s (± 0.44%) ~ 5.04s 5.10s p=0.225 n=6
Bind Time 2.14s (± 1.56%) 2.15s (± 1.28%) ~ 2.12s 2.19s p=0.746 n=6
Check Time 22.47s (± 0.56%) 22.48s (± 0.54%) ~ 22.35s 22.63s p=0.936 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 29.64s (± 0.30%) 29.70s (± 0.42%) ~ 29.53s 29.85s p=0.572 n=6
xstate-main - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 540,560 540,560 ~ ~ ~ p=1.000 n=6
Types 181,448 181,448 ~ ~ ~ p=1.000 n=6
Memory used 484,360k (± 0.01%) 484,386k (± 0.01%) ~ 484,284k 484,461k p=0.298 n=6
Parse Time 3.36s (± 0.72%) 3.37s (± 0.56%) ~ 3.34s 3.39s p=0.571 n=6
Bind Time 1.18s (± 0.89%) 1.18s (± 1.16%) ~ 1.16s 1.19s p=0.804 n=6
Check Time 19.12s (± 0.40%) 19.04s (± 0.27%) ~ 18.97s 19.10s p=0.091 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 23.66s (± 0.25%) 23.59s (± 0.22%) ~ 23.51s 23.65s p=0.125 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

@weswigham
Copy link
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 1, 2024

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

@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

effect

tsconfig.json

tsconfig.base.json

packages/schema/dtslint/tsconfig.json

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@weswigham
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 31 31 ~ ~ ~ p=1.000 n=6
Symbols 62,340 62,340 ~ ~ ~ p=1.000 n=6
Types 50,379 50,379 ~ ~ ~ p=1.000 n=6
Memory used 194,035k (± 0.98%) 193,440k (± 0.74%) ~ 192,753k 196,365k p=0.575 n=6
Parse Time 1.32s (± 0.48%) 1.30s (± 0.97%) -0.02s (- 1.52%) 1.29s 1.32s p=0.019 n=6
Bind Time 0.72s 0.72s ~ ~ ~ p=1.000 n=6
Check Time 9.73s (± 0.28%) 9.73s (± 0.26%) ~ 9.69s 9.77s p=0.566 n=6
Emit Time 2.73s (± 0.59%) 2.71s (± 1.15%) ~ 2.66s 2.75s p=0.191 n=6
Total Time 14.50s (± 0.31%) 14.46s (± 0.27%) ~ 14.41s 14.50s p=0.125 n=6
angular-1 - node (v18.15.0, x64)
Errors 33 33 ~ ~ ~ p=1.000 n=6
Symbols 947,886 947,886 ~ ~ ~ p=1.000 n=6
Types 410,840 410,840 ~ ~ ~ p=1.000 n=6
Memory used 1,224,561k (± 0.00%) 1,224,687k (± 0.00%) +126k (+ 0.01%) 1,224,644k 1,224,743k p=0.005 n=6
Parse Time 6.66s (± 0.89%) 6.63s (± 0.83%) ~ 6.55s 6.69s p=0.514 n=6
Bind Time 1.89s (± 0.44%) 1.88s (± 0.29%) ~ 1.88s 1.89s p=0.855 n=6
Check Time 31.81s (± 0.38%) 31.86s (± 0.15%) ~ 31.77s 31.91s p=0.377 n=6
Emit Time 15.21s (± 0.88%) 15.18s (± 0.40%) ~ 15.10s 15.26s p=1.000 n=6
Total Time 55.56s (± 0.45%) 55.56s (± 0.17%) ~ 55.40s 55.65s p=0.575 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,522,038 2,522,038 ~ ~ ~ p=1.000 n=6
Types 936,053 936,053 ~ ~ ~ p=1.000 n=6
Memory used 2,352,496k (± 0.00%) 2,352,669k (± 0.00%) +173k (+ 0.01%) 2,352,612k 2,352,740k p=0.005 n=6
Parse Time 9.37s (± 0.55%) 9.35s (± 0.30%) ~ 9.30s 9.37s p=0.517 n=6
Bind Time 2.17s (± 0.41%) 2.16s (± 0.65%) ~ 2.14s 2.18s p=0.215 n=6
Check Time 76.04s (± 0.38%) 76.14s (± 0.26%) ~ 75.79s 76.35s p=0.575 n=6
Emit Time 0.28s 0.28s (± 1.47%) ~ 0.27s 0.28s p=0.405 n=6
Total Time 87.86s (± 0.36%) 87.92s (± 0.23%) ~ 87.57s 88.17s p=0.575 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,106 1,258,500 +394 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,266 266,376 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 2,422,532k (± 0.01%) 2,423,377k (± 0.02%) +845k (+ 0.03%) 2,423,035k 2,424,182k p=0.005 n=6
Parse Time 5.20s (± 0.89%) 5.20s (± 0.76%) ~ 5.14s 5.24s p=0.748 n=6
Bind Time 1.93s (± 0.39%) 1.93s (± 0.72%) ~ 1.91s 1.94s p=0.740 n=6
Check Time 35.52s (± 0.28%) 35.37s (± 0.34%) ~ 35.21s 35.52s p=0.054 n=6
Emit Time 3.01s (± 1.85%) 3.02s (± 0.71%) ~ 2.98s 3.04s p=1.000 n=6
Total Time 45.68s (± 0.15%) 45.52s (± 0.26%) -0.16s (- 0.34%) 45.35s 45.64s p=0.020 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,258,106 1,258,500 +394 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,266 266,376 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 2,859,100k (±13.77%) 2,766,705k (±12.90%) ~ 2,499,732k 3,222,096k p=0.689 n=6
Parse Time 6.72s (± 2.99%) 6.63s (± 2.17%) ~ 6.47s 6.89s p=0.748 n=6
Bind Time 2.14s (± 1.99%) 2.19s (± 3.98%) ~ 2.11s 2.36s p=0.336 n=6
Check Time 43.19s (± 0.48%) 43.07s (± 0.40%) ~ 42.81s 43.33s p=0.471 n=6
Emit Time 3.57s (± 2.43%) 3.49s (± 1.70%) ~ 3.41s 3.59s p=0.173 n=6
Total Time 55.63s (± 0.46%) 55.39s (± 0.29%) ~ 55.20s 55.65s p=0.128 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 261,785 262,124 +339 (+ 0.13%) ~ ~ p=0.001 n=6
Types 106,508 106,618 +110 (+ 0.10%) ~ ~ p=0.001 n=6
Memory used 438,789k (± 0.01%) 439,077k (± 0.01%) +288k (+ 0.07%) 439,040k 439,137k p=0.005 n=6
Parse Time 3.54s (± 0.67%) 3.54s (± 1.41%) ~ 3.48s 3.63s p=0.809 n=6
Bind Time 1.31s (± 0.89%) 1.32s (± 1.12%) ~ 1.30s 1.33s p=0.219 n=6
Check Time 18.94s (± 0.42%) 18.84s (± 0.36%) ~ 18.72s 18.92s p=0.064 n=6
Emit Time 1.55s (± 1.27%) 1.54s (± 1.26%) ~ 1.52s 1.57s p=0.466 n=6
Total Time 25.33s (± 0.40%) 25.24s (± 0.24%) ~ 25.14s 25.31s p=0.172 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 68 68 ~ ~ ~ p=1.000 n=6
Symbols 225,919 225,919 ~ ~ ~ p=1.000 n=6
Types 94,415 94,415 ~ ~ ~ p=1.000 n=6
Memory used 371,077k (± 0.01%) 371,092k (± 0.01%) ~ 371,027k 371,165k p=0.630 n=6
Parse Time 3.59s (± 1.59%) 3.62s (± 1.41%) ~ 3.52s 3.66s p=0.260 n=6
Bind Time 1.96s (± 1.66%) 1.96s (± 0.95%) ~ 1.93s 1.98s p=1.000 n=6
Check Time 20.27s (± 0.28%) 20.31s (± 0.32%) ~ 20.24s 20.41s p=0.145 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 25.82s (± 0.25%) 25.88s (± 0.29%) ~ 25.78s 25.97s p=0.227 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,110,459 3,110,459 ~ ~ ~ p=1.000 n=6
Types 1,072,226 1,072,226 ~ ~ ~ p=1.000 n=6
Memory used 3,202,640k (± 0.01%) 3,203,131k (± 0.00%) +491k (+ 0.02%) 3,202,948k 3,203,256k p=0.008 n=6
Parse Time 14.10s (± 0.70%) 14.00s (± 0.28%) ~ 13.94s 14.05s p=0.077 n=6
Bind Time 4.44s (± 0.23%) 4.43s (± 0.26%) ~ 4.41s 4.44s p=0.491 n=6
Check Time 86.30s (± 1.71%) 85.42s (± 1.41%) ~ 84.13s 87.72s p=0.173 n=6
Emit Time 24.62s (± 8.98%) 23.45s (± 7.52%) ~ 22.60s 27.04s p=0.471 n=6
Total Time 129.44s (± 2.70%) 127.31s (± 2.30%) ~ 125.26s 133.20s p=0.093 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 279,484 279,484 ~ ~ ~ p=1.000 n=6
Types 114,024 114,024 ~ ~ ~ p=1.000 n=6
Memory used 429,471k (± 0.03%) 429,554k (± 0.02%) ~ 429,430k 429,687k p=0.230 n=6
Parse Time 4.07s (± 0.36%) 4.06s (± 0.93%) ~ 3.99s 4.09s p=0.806 n=6
Bind Time 1.74s (± 0.63%) 1.73s (± 1.69%) ~ 1.69s 1.78s p=0.319 n=6
Check Time 18.11s (± 0.58%) 18.21s (± 0.23%) ~ 18.13s 18.25s p=0.197 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 23.92s (± 0.41%) 23.99s (± 0.29%) ~ 23.90s 24.08s p=0.199 n=6
xstate-main - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 540,560 540,560 ~ ~ ~ p=1.000 n=6
Types 181,448 181,448 ~ ~ ~ p=1.000 n=6
Memory used 484,316k (± 0.01%) 484,365k (± 0.01%) ~ 484,311k 484,418k p=0.066 n=6
Parse Time 3.35s (± 0.74%) 3.35s (± 1.09%) ~ 3.31s 3.41s p=0.934 n=6
Bind Time 1.17s (± 0.84%) 1.18s (± 1.03%) ~ 1.17s 1.20s p=0.734 n=6
Check Time 19.05s (± 0.28%) 19.06s (± 0.39%) ~ 18.99s 19.18s p=0.872 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 23.57s (± 0.26%) 23.59s (± 0.29%) ~ 23.48s 23.67s p=0.689 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

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

Something interesting changed - please have a look.

Details

alibaba/lowcode-engine

40 of 42 projects failed to build with the old tsc and were ignored

modules/code-generator/tsconfig.json

1 similar comment
@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

alibaba/lowcode-engine

40 of 42 projects failed to build with the old tsc and were ignored

modules/code-generator/tsconfig.json

@weswigham
Copy link
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Dec 6, 2024

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

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

@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

effect

tsconfig.json

tsconfig.build.json

tsconfig.base.json

packages/effect/dtslint/tsconfig.json

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@weswigham
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 34 34 ~ ~ ~ p=1.000 n=6
Symbols 62,363 62,363 ~ ~ ~ p=1.000 n=6
Types 50,395 50,395 ~ ~ ~ p=1.000 n=6
Memory used 194,236k (± 0.98%) 193,683k (± 0.75%) ~ 193,011k 196,642k p=0.575 n=6
Parse Time 1.30s (± 2.05%) 1.31s (± 0.57%) ~ 1.30s 1.32s p=0.406 n=6
Bind Time 0.72s 0.72s ~ ~ ~ p=1.000 n=6
Check Time 9.78s (± 0.26%) 9.76s (± 0.37%) ~ 9.71s 9.79s p=0.332 n=6
Emit Time 2.74s (± 0.55%) 2.73s (± 0.50%) ~ 2.72s 2.76s p=0.788 n=6
Total Time 14.53s (± 0.24%) 14.52s (± 0.22%) ~ 14.47s 14.55s p=0.872 n=6
angular-1 - node (v18.15.0, x64)
Errors 37 37 ~ ~ ~ p=1.000 n=6
Symbols 947,936 947,936 ~ ~ ~ p=1.000 n=6
Types 410,955 410,955 ~ ~ ~ p=1.000 n=6
Memory used 1,225,797k (± 0.00%) 1,225,891k (± 0.00%) +94k (+ 0.01%) 1,225,846k 1,225,955k p=0.005 n=6
Parse Time 6.62s (± 0.34%) 6.66s (± 0.83%) ~ 6.60s 6.74s p=0.165 n=6
Bind Time 1.88s (± 0.43%) 1.88s (± 0.27%) ~ 1.88s 1.89s p=0.929 n=6
Check Time 32.01s (± 0.41%) 31.99s (± 0.22%) ~ 31.88s 32.07s p=0.936 n=6
Emit Time 15.15s (± 0.64%) 15.15s (± 0.71%) ~ 15.01s 15.28s p=0.872 n=6
Total Time 55.66s (± 0.26%) 55.68s (± 0.37%) ~ 55.47s 55.94s p=0.936 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,501,827 2,501,827 ~ ~ ~ p=1.000 n=6
Types 909,625 909,625 ~ ~ ~ p=1.000 n=6
Memory used 2,319,348k (± 0.00%) 2,319,494k (± 0.00%) +146k (+ 0.01%) 2,319,447k 2,319,538k p=0.005 n=6
Parse Time 9.30s (± 0.32%) 9.29s (± 0.36%) ~ 9.26s 9.35s p=0.466 n=6
Bind Time 2.16s (± 0.39%) 2.15s (± 0.54%) ~ 2.14s 2.17s p=0.555 n=6
Check Time 74.71s (± 0.23%) 74.95s (± 0.35%) ~ 74.62s 75.30s p=0.199 n=6
Emit Time 0.28s (± 4.15%) 0.29s (± 1.92%) ~ 0.28s 0.29s p=0.498 n=6
Total Time 86.45s (± 0.19%) 86.67s (± 0.34%) ~ 86.31s 87.04s p=0.471 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,330 1,225,725 +395 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,569 266,679 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 2,354,347k (± 0.01%) 2,416,131k (± 6.14%) +61,784k (+ 2.62%) 2,355,031k 2,719,238k p=0.005 n=6
Parse Time 5.22s (± 0.97%) 5.24s (± 1.26%) ~ 5.15s 5.32s p=0.687 n=6
Bind Time 1.78s (± 1.28%) 1.79s (± 0.76%) ~ 1.77s 1.81s p=0.416 n=6
Check Time 35.28s (± 0.48%) 35.18s (± 0.97%) ~ 34.79s 35.80s p=0.261 n=6
Emit Time 2.99s (± 1.17%) 2.92s (± 2.50%) ~ 2.82s 3.00s p=0.077 n=6
Total Time 45.28s (± 0.54%) 45.13s (± 0.74%) ~ 44.78s 45.77s p=0.128 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,330 1,225,725 +395 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,569 266,679 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 3,055,968k (± 7.58%) 3,152,028k (± 0.02%) +96,060k (+ 3.14%) 3,151,278k 3,152,841k p=0.005 n=6
Parse Time 6.98s (± 1.48%) 7.03s (± 1.68%) ~ 6.86s 7.21s p=0.689 n=6
Bind Time 2.15s (± 2.09%) 2.13s (± 1.17%) ~ 2.11s 2.17s p=0.936 n=6
Check Time 42.80s (± 1.04%) 42.96s (± 0.56%) ~ 42.56s 43.22s p=0.575 n=6
Emit Time 3.41s (± 2.81%) 3.48s (± 2.22%) ~ 3.38s 3.58s p=0.199 n=6
Total Time 55.36s (± 0.81%) 55.60s (± 0.38%) ~ 55.32s 55.78s p=0.230 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,269 262,609 +340 (+ 0.13%) ~ ~ p=0.001 n=6
Types 106,628 106,738 +110 (+ 0.10%) ~ ~ p=0.001 n=6
Memory used 439,898k (± 0.02%) 440,481k (± 0.02%) +582k (+ 0.13%) 440,311k 440,560k p=0.005 n=6
Parse Time 3.54s (± 0.52%) 3.53s (± 0.92%) ~ 3.49s 3.59s p=0.293 n=6
Bind Time 1.32s (± 1.11%) 1.30s (± 0.64%) ~ 1.30s 1.32s p=0.055 n=6
Check Time 18.92s (± 0.28%) 18.89s (± 0.62%) ~ 18.72s 19.06s p=0.575 n=6
Emit Time 1.54s (± 1.34%) 1.54s (± 0.54%) ~ 1.53s 1.55s p=0.503 n=6
Total Time 25.32s (± 0.16%) 25.27s (± 0.42%) ~ 25.11s 25.44s p=0.148 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 70 70 ~ ~ ~ p=1.000 n=6
Symbols 226,062 226,062 ~ ~ ~ p=1.000 n=6
Types 94,488 94,488 ~ ~ ~ p=1.000 n=6
Memory used 371,632k (± 0.01%) 371,726k (± 0.05%) ~ 371,577k 372,043k p=0.378 n=6
Parse Time 2.88s (± 1.02%) 2.90s (± 0.84%) ~ 2.86s 2.93s p=0.361 n=6
Bind Time 1.60s (± 0.61%) 1.59s (± 0.94%) ~ 1.58s 1.62s p=0.404 n=6
Check Time 16.55s (± 0.49%) 16.53s (± 0.39%) ~ 16.41s 16.60s p=1.000 n=6
Emit Time 0.00s (±244.70%) 0.00s ~ ~ ~ p=0.405 n=6
Total Time 21.03s (± 0.52%) 21.02s (± 0.40%) ~ 20.89s 21.15s p=0.872 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,197,899 3,197,899 ~ ~ ~ p=1.000 n=6
Types 1,099,330 1,099,330 ~ ~ ~ p=1.000 n=6
Memory used 3,271,798k (± 0.01%) 3,271,993k (± 0.01%) ~ 3,271,675k 3,272,382k p=0.378 n=6
Parse Time 14.20s (± 0.25%) 14.17s (± 0.36%) ~ 14.10s 14.22s p=0.193 n=6
Bind Time 4.57s (± 2.10%) 4.52s (± 0.34%) ~ 4.50s 4.54s p=0.333 n=6
Check Time 88.87s (± 3.11%) 87.58s (± 1.45%) ~ 86.42s 89.41s p=0.572 n=6
Emit Time 26.63s (±11.14%) 28.17s (± 2.20%) ~ 27.33s 28.76s p=0.873 n=6
Total Time 134.28s (± 1.11%) 134.44s (± 0.51%) ~ 133.82s 135.49s p=0.471 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 288,747 288,747 ~ ~ ~ p=1.000 n=6
Types 117,158 117,158 ~ ~ ~ p=1.000 n=6
Memory used 440,916k (± 0.02%) 441,025k (± 0.03%) ~ 440,859k 441,153k p=0.093 n=6
Parse Time 4.08s (± 0.70%) 4.09s (± 1.42%) ~ 4.00s 4.16s p=0.686 n=6
Bind Time 1.75s (± 1.75%) 1.76s (± 1.17%) ~ 1.74s 1.79s p=0.462 n=6
Check Time 18.82s (± 0.64%) 18.79s (± 0.45%) ~ 18.69s 18.88s p=0.520 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.65s (± 0.63%) 24.64s (± 0.47%) ~ 24.52s 24.80s p=0.936 n=6
xstate-main - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 552,389 552,389 ~ ~ ~ p=1.000 n=6
Types 185,093 185,093 ~ ~ ~ p=1.000 n=6
Memory used 492,430k (± 0.02%) 492,545k (± 0.01%) +115k (+ 0.02%) 492,413k 492,630k p=0.031 n=6
Parse Time 3.42s (± 0.80%) 3.42s (± 1.13%) ~ 3.38s 3.48s p=0.936 n=6
Bind Time 1.18s (± 1.28%) 1.17s (± 0.70%) ~ 1.16s 1.18s p=0.164 n=6
Check Time 19.59s (± 1.07%) 19.42s (± 0.14%) -0.18s (- 0.91%) 19.39s 19.46s p=0.006 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.19s (± 0.93%) 24.00s (± 0.20%) -0.19s (- 0.76%) 23.94s 24.06s p=0.008 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

@weswigham
Copy link
Member Author

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Dec 6, 2024

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

@typescript-bot
Copy link
Collaborator

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

Something interesting changed - please have a look.

Details

effect

tsconfig.json

tsconfig.build.json

tsconfig.base.json

packages/effect/dtslint/tsconfig.json

@typescript-bot
Copy link
Collaborator

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

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@weswigham
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 34 34 ~ ~ ~ p=1.000 n=6
Symbols 62,363 62,363 ~ ~ ~ p=1.000 n=6
Types 50,395 50,395 ~ ~ ~ p=1.000 n=6
Memory used 195,543k (± 0.93%) 194,863k (± 1.04%) ~ 192,999k 196,767k p=0.336 n=6
Parse Time 1.30s (± 0.94%) 1.31s (± 0.92%) ~ 1.29s 1.32s p=0.300 n=6
Bind Time 0.72s 0.72s ~ ~ ~ p=1.000 n=6
Check Time 9.79s (± 0.43%) 9.80s (± 0.43%) ~ 9.76s 9.87s p=1.000 n=6
Emit Time 2.73s (± 0.49%) 2.73s (± 0.75%) ~ 2.70s 2.76s p=0.605 n=6
Total Time 14.54s (± 0.32%) 14.56s (± 0.36%) ~ 14.50s 14.64s p=0.747 n=6
angular-1 - node (v18.15.0, x64)
Errors 37 37 ~ ~ ~ p=1.000 n=6
Symbols 947,936 947,936 ~ ~ ~ p=1.000 n=6
Types 410,955 410,955 ~ ~ ~ p=1.000 n=6
Memory used 1,225,834k (± 0.00%) 1,225,865k (± 0.00%) ~ 1,225,827k 1,225,927k p=0.471 n=6
Parse Time 6.66s (± 0.61%) 6.66s (± 0.67%) ~ 6.59s 6.71s p=0.872 n=6
Bind Time 1.89s (± 0.33%) 1.89s (± 0.27%) ~ 1.88s 1.89s p=0.386 n=6
Check Time 32.05s (± 0.43%) 31.97s (± 0.39%) ~ 31.79s 32.11s p=0.297 n=6
Emit Time 15.17s (± 0.18%) 15.19s (± 0.42%) ~ 15.09s 15.28s p=0.318 n=6
Total Time 55.78s (± 0.29%) 55.71s (± 0.20%) ~ 55.56s 55.84s p=0.378 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,501,827 2,501,827 ~ ~ ~ p=1.000 n=6
Types 909,625 909,625 ~ ~ ~ p=1.000 n=6
Memory used 2,319,533k (± 0.00%) 2,319,775k (± 0.00%) +242k (+ 0.01%) 2,319,672k 2,319,971k p=0.005 n=6
Parse Time 11.21s (± 0.48%) 11.21s (± 0.73%) ~ 11.08s 11.32s p=1.000 n=6
Bind Time 2.61s (± 0.58%) 2.62s (± 0.74%) ~ 2.60s 2.65s p=0.408 n=6
Check Time 91.22s (± 2.56%) 90.44s (± 1.49%) ~ 88.96s 92.19s p=0.471 n=6
Emit Time 0.35s (± 3.32%) 0.35s (± 3.32%) ~ 0.34s 0.37s p=1.000 n=6
Total Time 105.39s (± 2.18%) 104.62s (± 1.25%) ~ 103.17s 106.37s p=0.575 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,330 1,225,725 +395 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,569 266,679 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 3,084,924k (± 0.03%) 2,842,889k (±13.22%) ~ 2,356,448k 3,085,999k p=0.810 n=6
Parse Time 6.79s (± 0.77%) 6.72s (± 1.14%) ~ 6.61s 6.82s p=0.298 n=6
Bind Time 2.11s (± 1.57%) 2.14s (± 0.42%) ~ 2.13s 2.15s p=0.065 n=6
Check Time 42.81s (± 0.37%) 42.70s (± 0.40%) ~ 42.49s 42.95s p=0.378 n=6
Emit Time 3.44s (± 2.41%) 3.57s (± 3.73%) ~ 3.47s 3.79s p=0.109 n=6
Total Time 55.15s (± 0.39%) 55.13s (± 0.44%) ~ 54.80s 55.43s p=0.810 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,225,330 1,225,725 +395 (+ 0.03%) ~ ~ p=0.001 n=6
Types 266,569 266,679 +110 (+ 0.04%) ~ ~ p=0.001 n=6
Memory used 2,421,934k (± 0.02%) 2,422,995k (± 0.02%) +1,062k (+ 0.04%) 2,422,352k 2,423,695k p=0.013 n=6
Parse Time 5.45s (± 1.18%) 5.48s (± 1.01%) ~ 5.41s 5.56s p=0.336 n=6
Bind Time 1.83s (± 0.67%) 1.82s (± 0.41%) ~ 1.81s 1.83s p=0.929 n=6
Check Time 35.26s (± 0.17%) 35.30s (± 0.24%) ~ 35.21s 35.42s p=0.423 n=6
Emit Time 3.02s (± 3.83%) 3.01s (± 1.24%) ~ 2.95s 3.05s p=0.422 n=6
Total Time 45.57s (± 0.47%) 45.61s (± 0.16%) ~ 45.53s 45.73s p=0.092 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,269 262,609 +340 (+ 0.13%) ~ ~ p=0.001 n=6
Types 106,628 106,738 +110 (+ 0.10%) ~ ~ p=0.001 n=6
Memory used 439,863k (± 0.02%) 440,355k (± 0.01%) +492k (+ 0.11%) 440,275k 440,440k p=0.005 n=6
Parse Time 2.88s (± 0.72%) 2.89s (± 0.44%) ~ 2.88s 2.91s p=0.867 n=6
Bind Time 1.10s (± 0.37%) 1.10s ~ ~ ~ p=0.405 n=6
Check Time 15.78s (± 0.37%) 15.76s (± 0.32%) ~ 15.70s 15.84s p=0.519 n=6
Emit Time 1.30s (± 0.90%) 1.31s (± 0.92%) ~ 1.30s 1.33s p=0.138 n=6
Total Time 21.06s (± 0.38%) 21.06s (± 0.32%) ~ 20.99s 21.17s p=0.936 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 70 70 ~ ~ ~ p=1.000 n=6
Symbols 226,062 226,062 ~ ~ ~ p=1.000 n=6
Types 94,488 94,488 ~ ~ ~ p=1.000 n=6
Memory used 371,608k (± 0.01%) 371,625k (± 0.01%) ~ 371,574k 371,648k p=0.575 n=6
Parse Time 2.90s (± 0.98%) 2.88s (± 1.17%) ~ 2.84s 2.92s p=0.333 n=6
Bind Time 1.58s (± 0.66%) 1.59s (± 1.19%) ~ 1.56s 1.62s p=0.549 n=6
Check Time 16.49s (± 0.36%) 16.47s (± 0.26%) ~ 16.42s 16.53s p=0.575 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.98s (± 0.32%) 20.94s (± 0.29%) ~ 20.85s 21.03s p=0.294 n=6
vscode - node (v18.15.0, x64)
Errors 3 3 ~ ~ ~ p=1.000 n=6
Symbols 3,197,901 3,197,901 ~ ~ ~ p=1.000 n=6
Types 1,099,331 1,099,331 ~ ~ ~ p=1.000 n=6
Memory used 3,271,892k (± 0.01%) 3,271,915k (± 0.01%) ~ 3,271,523k 3,272,515k p=0.936 n=6
Parse Time 14.16s (± 0.33%) 14.17s (± 0.27%) ~ 14.13s 14.22s p=1.000 n=6
Bind Time 4.54s (± 0.64%) 4.52s (± 0.44%) ~ 4.49s 4.54s p=0.195 n=6
Check Time 88.08s (± 2.16%) 86.79s (± 0.57%) ~ 86.18s 87.42s p=0.575 n=6
Emit Time 27.06s (± 7.58%) 28.24s (± 2.44%) ~ 27.26s 28.84s p=0.173 n=6
Total Time 133.84s (± 1.20%) 133.71s (± 0.80%) ~ 132.31s 134.59s p=0.810 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 288,747 288,747 ~ ~ ~ p=1.000 n=6
Types 117,158 117,158 ~ ~ ~ p=1.000 n=6
Memory used 440,950k (± 0.02%) 441,056k (± 0.02%) ~ 440,907k 441,165k p=0.093 n=6
Parse Time 4.08s (± 0.94%) 4.10s (± 1.27%) ~ 4.02s 4.16s p=0.871 n=6
Bind Time 1.76s (± 0.91%) 1.76s (± 1.63%) ~ 1.72s 1.79s p=0.627 n=6
Check Time 18.85s (± 0.37%) 18.79s (± 0.49%) ~ 18.67s 18.89s p=0.226 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.69s (± 0.27%) 24.64s (± 0.66%) ~ 24.41s 24.84s p=0.748 n=6
xstate-main - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 552,389 552,389 ~ ~ ~ p=1.000 n=6
Types 185,093 185,093 ~ ~ ~ p=1.000 n=6
Memory used 492,432k (± 0.02%) 492,546k (± 0.02%) ~ 492,413k 492,676k p=0.066 n=6
Parse Time 3.42s (± 0.75%) 3.41s (± 0.49%) ~ 3.39s 3.43s p=0.869 n=6
Bind Time 1.18s (± 1.49%) 1.17s (± 1.04%) ~ 1.15s 1.18s p=0.328 n=6
Check Time 19.57s (± 1.12%) 19.50s (± 0.35%) ~ 19.39s 19.58s p=0.809 n=6
Emit Time 0.00s 0.00s (±244.70%) ~ 0.00s 0.01s p=0.405 n=6
Total Time 24.17s (± 0.86%) 24.08s (± 0.32%) ~ 23.98s 24.17s p=0.423 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

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

Everything looks good!

@weswigham weswigham marked this pull request as ready for review December 9, 2024 17:13
@dragomirtitian
Copy link
Contributor

Computed properties that have a computed name of a literal type seem to be duplicated when another computed property that is not a literal type is present:

// index.ts
declare const x: string;
declare const y: "y";

export class Test {
    [x] = 10;
    [y] = 10;
}

// index.d.ts
declare const x: string;
declare const y: "y";
export declare class Test {
    [x]: number;
    [y]: number;
    [y]: number; // ❌
}
export {};

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants