Skip to content

fix: several bugs detected in ts-parser#101

Merged
AsterDY merged 5 commits intocloudwego:mainfrom
angrychow:main
Oct 11, 2025
Merged

fix: several bugs detected in ts-parser#101
AsterDY merged 5 commits intocloudwego:mainfrom
angrychow:main

Conversation

@angrychow
Copy link
Copy Markdown
Collaborator

@angrychow angrychow commented Oct 3, 2025

What type of PR is this?

fix: several bugs detected in ts-parser

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

  1. 修复 ts-parser 无法识别形如
export default () => {
  console.log("test")
}

的问题

  1. 修复同一 package 下符号重名没有赋 mangled name 的问题。
  2. 修复了错误的符号解析缓存机制。

@angrychow angrychow requested a review from AsterDY October 11, 2025 01:53
@angrychow angrychow self-assigned this Oct 11, 2025
@angrychow angrychow changed the title [WIP] fix: ts-parser cannot identify export default arrow function fix:several bugs detected in ts-parser Oct 11, 2025
@angrychow angrychow changed the title fix:several bugs detected in ts-parser fix: several bugs detected in ts-parser Oct 11, 2025
@AsterDY AsterDY merged commit 7b4eb5b into cloudwego:main Oct 11, 2025
2 of 4 checks passed
simuleite pushed a commit to simuleite/abcoder that referenced this pull request Feb 28, 2026
* fix bug: cannot identify export default arrow function

* fix bug: if two files has same symbol name, should give a mangled name

* 1. revert import path changes 2. fix bug: falsy caching strategy in  function,

* chore: modify package version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants