fix: improve static canFollowModifier checks #16761

JLHwung · 2024-08-22T16:20:10Z

Q	A
Fixed Issues?	Fixes #16759
Patch: Bug Fix?	Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass?	Yes
Documentation PR Link
Any Dependency Changes?
License	MIT

In this PR we also fixed the following case:

interface Foo {
  get
  foo(): string;
  set
  bar(v);
}

Previously Babel parsed it as two property and two method signatures, however, it diverts from the TS4.3+ behaviour.

nicolo-ribaudo · 2024-08-22T16:27:36Z

packages/babel-parser/src/plugins/typescript/index.ts

+      this.next();
+      if (this.hasPrecedingLineBreak()) {
+        return false;
+      }
+      return this.tsTokenCanFollowModifier();


This refactor can be done separately from this PR, but given the deleted comment, we could:

use hasFollowingLineBreak instead of hasPrecedingLineBreak

use lookaheadCharCode instead of tsTokenCanFollowModifier

do not use tsTryParse to call tsNextTokenCanFollowModifier

nicolo-ribaudo · 2024-08-22T16:28:59Z

packages/babel-parser/test/fixtures/typescript/class/static-asi/input.ts

@@ -0,0 +1,22 @@
+class A {
+    static


Can you add a test for static followed by a modifier and then ASI?

class A { static readonly foo }

should be a static field called readonly and an instance field called foo

babel-bot · 2024-08-22T17:39:49Z

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/57715

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@babel/traverse](https://babel.dev/docs/en/next/babel-traverse) ([source](https://github.com/babel/babel/tree/HEAD/packages/babel-traverse)) | resolutions | minor | [`7.24.8` -> `7.25.6`](https://renovatebot.com/diffs/npm/@babel%2ftraverse/7.24.8/7.25.6) | --- ### Release Notes <details> <summary>babel/babel (@babel/traverse)</summary> ### [`v7.25.6`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7256-2024-08-29) [Compare Source](babel/babel@v7.25.4...v7.25.6) ##### 🐛 Bug Fix - `babel-generator` - [#16783](babel/babel#16783) Properly print inner comments in TS array types ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - [#16775](babel/babel#16775) fix: jsx whitespace is not properly preserved when retainLines ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-traverse` - [#16727](babel/babel#16727) fix: `path.getAssignmentIdentifiers` may be `undefined` ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-parser` - [#16761](babel/babel#16761) fix: improve static canFollowModifier checks ([@JLHwung](https://github.com/JLHwung)) - `babel-helpers`, `babel-plugin-transform-optional-chaining`, `babel-runtime-corejs3` - [#16769](babel/babel#16769) Only wrap functions in `superPropertyGet` helper ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 💅 Polish - `babel-generator`, `babel-plugin-transform-async-to-generator`, `babel-plugin-transform-block-scoping`, `babel-plugin-transform-class-properties`, `babel-plugin-transform-classes`, `babel-plugin-transform-duplicate-named-capturing-groups-regex`, `babel-plugin-transform-named-capturing-groups-regex`, `babel-plugin-transform-react-jsx-development`, `babel-plugin-transform-react-jsx`, `babel-plugin-transform-react-pure-annotations`, `babel-plugin-transform-regenerator`, `babel-plugin-transform-runtime`, `babel-preset-env` - [#16780](babel/babel#16780) Do not enforce printing space between `(` and comments ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-plugin-syntax-import-assertions`, `babel-plugin-syntax-import-attributes` - [#16781](babel/babel#16781) Don't throw when enabling both syntax-import-{assertions,attributes} ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-generator` - [#16782](babel/babel#16782) TS union/intersection nested in union does not need parens ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 🏠 Internal - `babel-generator` - [#16777](babel/babel#16777) Remove unused `parent` params in the generator ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ### [`v7.25.4`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7254-2024-08-22) [Compare Source](babel/babel@v7.25.3...v7.25.4) ##### 🐛 Bug Fix - `babel-traverse` - [#16756](babel/babel#16756) fix: Skip computed key when renaming ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-helper-create-class-features-plugin`, `babel-plugin-proposal-decorators` - [#16755](babel/babel#16755) fix: Decorator 2018-09 may throw an exception ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-types` - [#16710](babel/babel#16710) Visit AST fields nodes according to their syntactical order ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-generator` - [#16709](babel/babel#16709) Print semicolon after TS `export namespace as A` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 💅 Polish - `babel-generator`, `babel-plugin-proposal-decorators`, `babel-plugin-proposal-destructuring-private`, `babel-plugin-proposal-pipeline-operator`, `babel-plugin-transform-class-properties`, `babel-plugin-transform-destructuring`, `babel-plugin-transform-optional-chaining`, `babel-plugin-transform-private-methods`, `babel-plugin-transform-private-property-in-object`, `babel-plugin-transform-typescript`, `babel-runtime-corejs2`, `babel-runtime`, `babel-traverse` - [#16722](babel/babel#16722) Avoid unnecessary parens around sequence expressions ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-generator`, `babel-plugin-transform-class-properties` - [#16714](babel/babel#16714) Avoid unnecessary parens around exported arrow functions ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-generator`, `babel-plugin-proposal-decorators`, `babel-plugin-proposal-destructuring-private`, `babel-plugin-transform-object-rest-spread` - [#16712](babel/babel#16712) Avoid printing unnecessary parens around object destructuring ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 🔬 Output optimization - `babel-generator` - [#16740](babel/babel#16740) Avoid extra spaces between comments/regexps in compact mode ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ### [`v7.25.3`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7253-2024-07-31) [Compare Source](babel/babel@v7.25.2...v7.25.3) ##### 🐛 Bug Fix - `babel-plugin-bugfix-firefox-class-in-computed-class-key`, `babel-traverse` - [#16699](babel/babel#16699) Avoid validating visitors produced by `traverse.visitors.merge` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 🏠 Internal - `babel-parser` - [#16688](babel/babel#16688) Add `@babel/types` as a dependency of `@babel/parser` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ### [`v7.25.2`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7252-2024-07-30) [Compare Source](babel/babel@v7.25.1...v7.25.2) ##### 🐛 Bug Fix - `babel-core`, `babel-traverse` - [#16695](babel/babel#16695) Ensure that `requeueComputedKeyAndDecorators` is available ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ### [`v7.25.1`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7251-2024-07-28) [Compare Source](babel/babel@v7.25.0...v7.25.1) ##### 🐛 Bug Fix - `babel-plugin-transform-function-name` - [#16683](babel/babel#16683) fix: `ensureFunctionName` may be undefined ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-plugin-transform-react-constant-elements` - [#16582](babel/babel#16582) fix plugin-transform-react-constant-elements transform JSXFrament but not add JSXExpressionContainer ([@keiseiTi](https://github.com/keiseiTi)) - `babel-traverse` - [#16587](babel/babel#16587) fix: fixed issue16583 + test ([@nerodesu017](https://github.com/nerodesu017)) ##### 🏠 Internal - [#16663](babel/babel#16663) Test eslint plugin against eslint 9 ([@JLHwung](https://github.com/JLHwung)) ### [`v7.25.0`](https://github.com/babel/babel/blob/HEAD/CHANGELOG.md#v7250-2024-07-26) [Compare Source](babel/babel@v7.24.8...v7.25.0) ##### 👓 Spec Compliance - `babel-helpers`, `babel-plugin-proposal-explicit-resource-management`, `babel-runtime-corejs3` - [#16537](babel/babel#16537) `await using` normative updates ([@JLHwung](https://github.com/JLHwung)) - `babel-plugin-transform-typescript` - [#16602](babel/babel#16602) Ensure enum members syntactically determinable to be strings do not get reverse mappings ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) ##### 🚀 New Feature - `babel-helper-create-class-features-plugin`, `babel-helper-function-name`, `babel-helper-plugin-utils`, `babel-helper-wrap-function`, `babel-plugin-bugfix-safari-class-field-initializer-scope`, `babel-plugin-bugfix-safari-id-destructuring-collision-in-function-expression`, `babel-plugin-transform-classes`, `babel-plugin-transform-function-name`, `babel-preset-env`, `babel-traverse`, `babel-types` - [#16658](babel/babel#16658) Move `ensureFunctionName` to `NodePath.prototype` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-helper-hoist-variables`, `babel-helper-plugin-utils`, `babel-plugin-proposal-async-do-expressions`, `babel-plugin-transform-modules-systemjs`, `babel-traverse` - [#16644](babel/babel#16644) Move `hoistVariables` to `Scope.prototype` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-helper-create-class-features-plugin`, `babel-helper-module-transforms`, `babel-helper-plugin-utils`, `babel-helper-split-export-declaration`, `babel-plugin-transform-classes`, `babel-traverse`, `babel-types` - [#16645](babel/babel#16645) Move `splitExportDeclaration` to `NodePath.prototype` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-helper-create-class-features-plugin`, `babel-helper-environment-visitor`, `babel-helper-module-transforms`, `babel-helper-plugin-utils`, `babel-helper-remap-async-to-generator`, `babel-helper-replace-supers`, `babel-plugin-bugfix-firefox-class-in-computed-class-key`, `babel-plugin-bugfix-v8-static-class-fields-redefine-readonly`, `babel-plugin-transform-async-generator-functions`, `babel-plugin-transform-classes`, `babel-traverse` - [#16649](babel/babel#16649) Move `environment-visitor` helper into `@babel/traverse` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-core`, `babel-parser` - [#16480](babel/babel#16480) Expose wether a module has TLA or not as `.extra.async` ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-compat-data`, `babel-plugin-bugfix-safari-class-field-initializer-scope`, `babel-preset-env` - [#16569](babel/babel#16569) Introduce `bugfix-safari-class-field-initializer-scope` ([@davidtaylorhq](https://github.com/davidtaylorhq)) - `babel-plugin-transform-block-scoping`, `babel-traverse`, `babel-types` - [#16551](babel/babel#16551) Add `NodePath#getAssignmentIdentifiers` ([@JLHwung](https://github.com/JLHwung)) - `babel-helper-import-to-platform-api`, `babel-plugin-proposal-json-modules` - [#16579](babel/babel#16579) Add `uncheckedRequire` option for JSON imports to CJS ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-helper-transform-fixture-test-runner`, `babel-node` - [#16642](babel/babel#16642) Allow using custom config in `babel-node --eval` ([@slatereax](https://github.com/slatereax)) - `babel-compat-data`, `babel-helper-create-regexp-features-plugin`, `babel-plugin-proposal-duplicate-named-capturing-groups-regex`, `babel-plugin-transform-duplicate-named-capturing-groups-regex`, `babel-preset-env`, `babel-standalone` - [#16445](babel/babel#16445) Add `duplicate-named-capturing-groups-regex` to `preset-env` ([@JLHwung](https://github.com/JLHwung)) ##### 🐛 Bug Fix - `babel-generator` - [#16678](babel/babel#16678) Print parens around as expressions on the LHS ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-template`, `babel-types` - [#15286](babel/babel#15286) fix: Props are lost when the template replaces the node ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) ##### 🏠 Internal - Other - [#16674](babel/babel#16674) bump gulp to 5 ([@JLHwung](https://github.com/JLHwung)) - `babel-generator` - [#16651](babel/babel#16651) Simplify the printing logic for `(` before ambiguous tokens ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) - `babel-helper-function-name`, `babel-plugin-transform-arrow-functions`, `babel-plugin-transform-function-name`, `babel-preset-env`, `babel-traverse` - [#16652](babel/babel#16652) Simplify `helper-function-name` logic ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) ##### 🏃‍♀️ Performance - `babel-parser`, `babel-plugin-proposal-pipeline-operator` - [#16461](babel/babel#16461) Some minor parser performance improvements for ts ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) ##### 🔬 Output optimization - `babel-plugin-transform-classes` - [#16670](babel/babel#16670) Reduce redundant `assertThisInitialized` ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-helper-create-class-features-plugin`, `babel-helper-replace-supers`, `babel-helpers`, `babel-plugin-proposal-decorators`, `babel-plugin-transform-class-properties`, `babel-plugin-transform-classes`, `babel-plugin-transform-exponentiation-operator`, `babel-plugin-transform-object-super`, `babel-plugin-transform-private-methods`, `babel-runtime-corejs2`, `babel-runtime-corejs3`, `babel-runtime` - [#16374](babel/babel#16374) Improve `super.x` output ([@liuxingbaoyu](https://github.com/liuxingbaoyu)) - `babel-plugin-transform-class-properties`, `babel-plugin-transform-classes` - [#16656](babel/babel#16656) Simplify output for anonymous classes with no methods ([@nicolo-ribaudo](https://github.com/nicolo-ribaudo)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.vylpes.xyz/RabbitLabs/random-bunny/pulls/212 Reviewed-by: Vylpes <ethan@vylpes.com> Co-authored-by: Renovate Bot <renovate@vylpes.com> Co-committed-by: Renovate Bot <renovate@vylpes.com>

fix: improve static canFollowModifier checks

59ffebe

JLHwung added PR: Bug Fix 🐛 A type of pull request used for our changelog categories pkg: parser labels Aug 22, 2024

nicolo-ribaudo reviewed Aug 22, 2024

View reviewed changes

add new asi test case

983b908

nicolo-ribaudo approved these changes Aug 23, 2024

View reviewed changes

liuxingbaoyu approved these changes Aug 23, 2024

View reviewed changes

JLHwung merged commit 7edd508 into babel:main Aug 24, 2024
52 checks passed

JLHwung deleted the fix-16759 branch August 24, 2024 02:32

dmaccormack mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , jsonurl/jsonurl-js#724

Open

mboocannon mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , tslib mboocannon/msgraph-sdk-javascript#96

Open

MrRio mentioned this pull request Sep 19, 2024

[Snyk] Upgrade @babel/runtime from 7.23.2 to 7.25.6 parallax/jsPDF#3771

Merged

WontonSam mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , , , , , , , , config, eslint, eslint-plugin-jest WontonSam/woocommerce#538

Open

rawdaymohamed mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , , , , body-parser, dotenv, express, mongoose, nodemon rawdaymohamed/Hot-Todo#375

Open

adam-snyk mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , , , , , snyk/sweater-comb#648

Open

AKJUS mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: , , , , , , , , classnames, docusaurus-lunr-search, react-markdown AKJUS/OpenLineage#30

Open

BitcoinOutput mentioned this pull request Sep 20, 2024

[Snyk] Upgrade: , , , , , , , , , typescript, web3-eth2-core BitcoinOutput/js-direct-routein#279

Open

ali8889 mentioned this pull request Sep 20, 2024

[Snyk] Upgrade @babel/runtime from 7.0.0-beta.42 to 7.25.6 ali8889/mist#665

Open

WontonSam mentioned this pull request Sep 20, 2024

[Snyk] Upgrade: , , , jest-circus, jest, puppeteer, commander, , , , , , , , config, jest-each, jest-puppeteer WontonSam/woocommerce#547

Open

snyk-io bot mentioned this pull request Sep 20, 2024

[Snyk] Upgrade @babel/runtime from 7.4.3 to 7.25.6 Playgirlkaybraz11/td#15

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: improve static canFollowModifier checks #16761

fix: improve static canFollowModifier checks #16761

JLHwung commented Aug 22, 2024

nicolo-ribaudo Aug 22, 2024

nicolo-ribaudo Aug 22, 2024

babel-bot commented Aug 22, 2024

fix: improve static canFollowModifier checks #16761

fix: improve static canFollowModifier checks #16761

Conversation

JLHwung commented Aug 22, 2024

nicolo-ribaudo Aug 22, 2024

Choose a reason for hiding this comment

nicolo-ribaudo Aug 22, 2024

Choose a reason for hiding this comment

babel-bot commented Aug 22, 2024