Skip to content

fix: performance issue from @Copilot in #213#214

Merged
Groupguanfang merged 1 commit into
nextfrom
next-dev
Nov 26, 2025
Merged

fix: performance issue from @Copilot in #213#214
Groupguanfang merged 1 commit into
nextfrom
next-dev

Conversation

@Groupguanfang
Copy link
Copy Markdown
Collaborator

No description provided.

@Groupguanfang Groupguanfang self-assigned this Nov 26, 2025
Copilot AI review requested due to automatic review settings November 26, 2025 03:48
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Nov 26, 2025

⚠️ No Changeset found

Latest commit: d3d75f8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses a performance issue by optimizing AST traversal in semantic token processing. The fix consolidates two separate tree walks into a single traversal, significantly reducing computational overhead when processing semantic classifications.

Key changes:

  • Introduced a new findAnnotationOrDecoratorNodeInPosition function that checks for three node types in a single AST traversal
  • Replaced two sequential findNodeInPosition calls with one consolidated call in the semantic token processing loop

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Groupguanfang Groupguanfang merged commit d0fc1b3 into next Nov 26, 2025
9 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in ArkTS Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants