Skip to content

experiment: remove SymbolConstructor ESSymbol transform #61893

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

Conversation

Renegade334
Copy link
Contributor

@Renegade334 Renegade334 commented Jun 18, 2025

ref: #42543
ref: code search

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Jun 18, 2025
@jakebailey
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jun 18, 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

@typescript-bot
Copy link
Collaborator

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

There were interesting changes:

Branch only errors:

Package: jquery
Error:

Error: 
/mnt/vss/_work/1/DefinitelyTyped/types/jquery/misc.d.ts
  7359:14  error  TypeScript@local compile error: 
Subsequent property declarations must have the same type.  Property 'toStringTag' must be of type 'unique symbol', but here has type 'symbol'  @definitelytyped/expect

✖ 1 problem (1 error, 0 warnings)

    at combineErrorsAndWarnings (/mnt/vss/_work/1/DefinitelyTyped/node_modules/.pnpm/@[email protected][email protected]/node_modules/@definitelytyped/dtslint/dist/index.js:199:28)
    at runTests (/mnt/vss/_work/1/DefinitelyTyped/node_modules/.pnpm/@[email protected][email protected]/node_modules/@definitelytyped/dtslint/dist/index.js:191:20)

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/61893/merge:

Everything looks good!

@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 34 34 ~ ~ ~ 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 195,943k (± 0.73%) 195,375k (± 0.94%) ~ 192,968k 196,647k p=0.810 n=6
Parse Time 1.62s (± 1.72%) 1.60s (± 1.09%) ~ 1.58s 1.63s p=0.331 n=6
Bind Time 0.88s (± 1.67%) 0.88s (± 0.58%) ~ 0.88s 0.89s p=0.784 n=6
Check Time 11.73s (± 0.58%) 11.79s (± 0.58%) ~ 11.71s 11.91s p=0.229 n=6
Emit Time 3.32s (± 3.89%) 3.29s (± 0.74%) ~ 3.26s 3.32s p=0.872 n=6
Total Time 17.56s (± 0.93%) 17.57s (± 0.61%) ~ 17.48s 17.77s p=0.688 n=6
angular-1 - node (v18.15.0, x64)
Errors 56 56 ~ ~ ~ p=1.000 n=6
Symbols 948,750 948,706 -44 (- 0.00%) ~ ~ p=0.001 n=6
Types 410,846 410,886 +40 (+ 0.01%) ~ ~ p=0.001 n=6
Memory used 1,224,644k (± 0.00%) 1,224,640k (± 0.01%) ~ 1,224,530k 1,224,734k p=0.936 n=6
Parse Time 6.58s (± 0.87%) 6.60s (± 0.94%) ~ 6.54s 6.69s p=0.517 n=6
Bind Time 1.88s (± 0.93%) 1.89s (± 0.56%) ~ 1.87s 1.90s p=0.869 n=6
Check Time 31.75s (± 0.36%) 31.81s (± 0.20%) ~ 31.75s 31.90s p=0.230 n=6
Emit Time 14.91s (± 0.73%) 14.87s (± 0.37%) ~ 14.81s 14.95s p=0.335 n=6
Total Time 55.11s (± 0.41%) 55.17s (± 0.19%) ~ 55.01s 55.32s p=0.630 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,561,651 2,561,651 ~ ~ ~ p=1.000 n=6
Types 892,630 892,630 ~ ~ ~ p=1.000 n=6
Memory used 2,836,536k (± 0.00%) 2,836,524k (± 0.00%) ~ 2,836,468k 2,836,559k p=1.000 n=6
Parse Time 9.19s (± 0.35%) 9.17s (± 0.27%) ~ 9.14s 9.21s p=0.417 n=6
Bind Time 2.31s (± 0.36%) 2.30s (± 0.27%) ~ 2.29s 2.31s p=0.340 n=6
Check Time 83.66s (± 0.18%) 83.68s (± 0.47%) ~ 83.09s 84.28s p=0.936 n=6
Emit Time 0.30s (± 1.37%) 0.30s ~ ~ ~ p=0.405 n=6
Total Time 95.45s (± 0.18%) 95.45s (± 0.43%) ~ 94.84s 96.09s p=0.936 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,227,126 1,227,126 ~ ~ ~ p=1.000 n=6
Types 267,408 267,407 -1 (- 0.00%) ~ ~ p=0.001 n=6
Memory used 2,973,547k (±10.04%) 2,999,101k (± 7.83%) ~ 2,518,973k 3,095,658k p=0.298 n=6
Parse Time 6.64s (± 1.39%) 6.63s (± 1.10%) ~ 6.53s 6.74s p=0.471 n=6
Bind Time 2.17s (± 0.79%) 2.13s (± 1.75%) -0.04s (- 1.92%) 2.06s 2.17s p=0.019 n=6
Check Time 42.52s (± 0.21%) 42.56s (± 0.55%) ~ 42.11s 42.75s p=0.298 n=6
Emit Time 3.39s (± 3.35%) 3.42s (± 1.86%) ~ 3.31s 3.49s p=0.630 n=6
Total Time 54.74s (± 0.28%) 54.75s (± 0.40%) ~ 54.31s 54.90s p=0.689 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,227,126 1,227,126 ~ ~ ~ p=1.000 n=6
Types 267,408 267,407 -1 (- 0.00%) ~ ~ p=0.001 n=6
Memory used 2,428,975k (± 0.01%) 2,549,995k (± 7.34%) ~ 2,428,680k 2,792,055k p=0.298 n=6
Parse Time 5.38s (± 1.21%) 5.45s (± 1.36%) +0.08s (+ 1.46%) 5.37s 5.56s p=0.045 n=6
Bind Time 1.82s (± 1.86%) 1.82s (± 1.33%) ~ 1.79s 1.85s p=0.807 n=6
Check Time 35.20s (± 0.46%) 35.02s (± 0.97%) ~ 34.47s 35.38s p=0.378 n=6
Emit Time 3.04s (± 0.54%) 2.98s (± 0.63%) -0.05s (- 1.70%) 2.96s 3.01s p=0.005 n=6
Total Time 45.44s (± 0.38%) 45.29s (± 0.66%) ~ 44.81s 45.65s p=0.471 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 262,474 262,474 ~ ~ ~ p=1.000 n=6
Types 107,135 107,134 -1 (- 0.00%) ~ ~ p=0.001 n=6
Memory used 441,735k (± 0.01%) 441,699k (± 0.01%) ~ 441,640k 441,795k p=0.298 n=6
Parse Time 3.52s (± 0.80%) 3.54s (± 1.13%) ~ 3.47s 3.57s p=0.627 n=6
Bind Time 1.32s (± 1.62%) 1.32s (± 0.83%) ~ 1.31s 1.34s p=0.868 n=6
Check Time 18.84s (± 0.47%) 18.91s (± 0.24%) ~ 18.85s 18.96s p=0.148 n=6
Emit Time 1.53s (± 1.47%) 1.51s (± 0.80%) ~ 1.50s 1.53s p=0.370 n=6
Total Time 25.22s (± 0.43%) 25.28s (± 0.26%) ~ 25.23s 25.37s p=0.467 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 71 71 ~ ~ ~ p=1.000 n=6
Symbols 225,327 225,327 ~ ~ ~ p=1.000 n=6
Types 94,288 94,288 ~ ~ ~ p=1.000 n=6
Memory used 371,024k (± 0.01%) 370,985k (± 0.02%) ~ 370,923k 371,056k p=0.230 n=6
Parse Time 2.31s (± 0.65%) 2.32s (± 0.63%) ~ 2.30s 2.34s p=0.462 n=6
Bind Time 1.35s (± 1.69%) 1.33s (± 1.48%) ~ 1.32s 1.36s p=0.161 n=6
Check Time 13.64s (± 0.19%) 13.60s (± 0.42%) ~ 13.50s 13.66s p=0.324 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 17.30s (± 0.24%) 17.26s (± 0.40%) ~ 17.13s 17.34s p=0.261 n=6
vscode - node (v18.15.0, x64)
Errors 33 33 ~ ~ ~ p=1.000 n=6
Symbols 3,471,654 3,471,654 ~ ~ ~ p=1.000 n=6
Types 1,167,299 1,167,299 ~ ~ ~ p=1.000 n=6
Memory used 3,518,645k (± 0.00%) 3,518,609k (± 0.00%) ~ 3,518,338k 3,518,726k p=0.936 n=6
Parse Time 14.98s (± 0.73%) 15.02s (± 0.22%) ~ 14.99s 15.08s p=0.520 n=6
Bind Time 4.87s (± 0.69%) 4.84s (± 0.83%) ~ 4.77s 4.88s p=0.226 n=6
Check Time 96.11s (± 2.61%) 94.83s (± 4.88%) ~ 92.30s 104.24s p=0.230 n=6
Emit Time 29.95s (± 2.80%) 29.44s (± 1.39%) ~ 29.15s 30.23s p=0.575 n=6
Total Time 145.91s (± 2.10%) 144.12s (± 3.48%) ~ 141.37s 154.32s p=0.298 n=6
webpack - node (v18.15.0, x64)
Errors 2 2 ~ ~ ~ p=1.000 n=6
Symbols 320,000 320,000 ~ ~ ~ p=1.000 n=6
Types 140,356 140,356 ~ ~ ~ p=1.000 n=6
Memory used 474,133k (± 0.01%) 474,224k (± 0.03%) ~ 474,034k 474,369k p=0.298 n=6
Parse Time 4.35s (± 0.43%) 4.32s (± 0.40%) ~ 4.29s 4.34s p=0.054 n=6
Bind Time 1.76s (± 1.64%) 1.75s (± 1.75%) ~ 1.71s 1.80s p=1.000 n=6
Check Time 20.68s (± 0.34%) 20.72s (± 0.40%) ~ 20.62s 20.82s p=0.521 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 26.79s (± 0.36%) 26.80s (± 0.33%) ~ 26.68s 26.91s p=0.873 n=6
xstate-main - node (v18.15.0, x64)
Errors 30 30 ~ ~ ~ p=1.000 n=6
Symbols 667,036 666,295 -741 (- 0.11%) ~ ~ p=0.001 n=6
Types 198,508 198,987 +479 (+ 0.24%) ~ ~ p=0.001 n=6
Memory used 569,745k (± 0.01%) 569,731k (± 0.01%) ~ 569,675k 569,771k p=0.936 n=6
Parse Time 5.54s (± 0.82%) 5.53s (± 0.85%) ~ 5.46s 5.58s p=1.000 n=6
Bind Time 1.67s (± 0.49%) 1.66s (± 1.12%) ~ 1.64s 1.68s p=0.288 n=6
Check Time 25.36s (± 0.45%) 25.34s (± 0.53%) ~ 25.19s 25.54s p=0.688 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 32.57s (± 0.28%) 32.53s (± 0.43%) ~ 32.40s 32.75s p=0.298 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/61893/merge:

Something interesting changed - please have a look.

Details

pmndrs/jotai

tsconfig.json

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

Successfully merging this pull request may close these issues.

3 participants