fix: self-reference detection and type alias dependency extraction (follow-up to #123)#125
Merged
Duslia merged 13 commits intocloudwego:mainfrom Dec 7, 2025
Merged
fix: self-reference detection and type alias dependency extraction (follow-up to #123)#125Duslia merged 13 commits intocloudwego:mainfrom
Duslia merged 13 commits intocloudwego:mainfrom
Conversation
… export check Fixed an issue where arrow function default export detection was checking the arrow function's symbol instead of the variable declaration's symbol. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed type dependency recognition for type aliases (especially union types) across all parsers by extracting symbols from TypeReference node's typeName child instead of the node itself. Changes: - DependencyUtils: capture union/intersection type aliases before recursing into members - TypeParser: extract symbols from typeName for TypeReference nodes, handle ExpressionWithTypeArguments, put type alias dependencies in InlineStruct instead of Implements - FunctionParser: extract symbols from TypeReference typeName for function parameter/return types - VarParser: extract symbols from TypeReference typeName for variable/property type annotations - Added comprehensive unit tests for all three parsers 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…sistency - Resolved version conflicts in package.json and index.ts (keep 0.0.22) - Added function property parsing feature: arrow functions and function expressions in class properties are now added to Methods - Fixed constructor naming: use assignSymbolName consistently to avoid duplicates between FunctionParser and TypeParser - Updated tests to use correct constructor name format (ClassName.__constructor) - All tests passing (33/33)
angrychow
changed the title
Duslia
approved these changes
Dec 7, 2025
Duslia
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What type of PR is this?
fix: self-reference detection and type alias dependency extraction (follow-up to #123)
Check the PR title.
(Optional) Translate the PR title into Chinese.
PR #123 后续 feature
(Optional) More detailed description for this PR(en: English/zh: Chinese).
en:
This is a follow-up to PR #123, which was merged prematurely.
This commit completes the remaining work.
zh(optional):
PR #123 忘记加 WIP 了，将剩下的 feature 补完。