Skip to content

fix(go): fill File-level metadata for pre-parsed packages#198

Merged
wzekin merged 1 commit into
cloudwego:mainfrom
wzekin:fix/go-preparsed-pkg-metadata
Jun 1, 2026
Merged

fix(go): fill File-level metadata for pre-parsed packages#198
wzekin merged 1 commit into
cloudwego:mainfrom
wzekin:fix/go-preparsed-pkg-metadata

Conversation

@wzekin
Copy link
Copy Markdown
Collaborator

@wzekin wzekin commented Jun 1, 2026

What type of PR is this?

fix

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.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
When a package was pre-parsed via cross-module referCodes, only Functions were populated but File.Package/Imports were left empty. This broke downstream reachability analysis based on the package import graph. Now we still fill File-level metadata for such packages while skipping duplicate function-body parsing.

zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@wzekin wzekin force-pushed the fix/go-preparsed-pkg-metadata branch from b633713 to 6dbb3c2 Compare June 1, 2026 07:40
When a package was pre-parsed via cross-module referCodes, only Functions
were populated but File.Package/Imports were left empty. This broke
downstream reachability analysis based on the package import graph.
Now we still fill File-level metadata for such packages while skipping
duplicate function-body parsing.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@wzekin wzekin force-pushed the fix/go-preparsed-pkg-metadata branch from 6dbb3c2 to d24c104 Compare June 1, 2026 07:48
@wzekin wzekin merged commit b1c638a into cloudwego:main Jun 1, 2026
3 of 4 checks passed
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