Skip to content

Conversation

@BruinGrowly
Copy link
Owner

Unifies configuration loading, enables async function analysis, and optimizes vocabulary initialization to improve Harmonizer's reliability, coverage, and performance.

This PR resolves several key issues:

  1. Configuration: Ensures the CLI correctly loads and applies settings (excludes, thresholds, custom vocabulary) from .harmonizer.yml, harmonizer.yaml, or pyproject.toml by using a unified ConfigLoader. It also correctly roots exclude patterns.
  2. Async Function Analysis: Extends the AST walk to include ast.AsyncFunctionDef, ensuring coroutines are analyzed for ICE metrics and disharmony.
  3. Performance: Caches the large keyword vocabulary, reducing startup time and silencing verbose initialization messages by default.

Open in Cursor Open in Web

Co-authored-by: taurekaw <taurekaw@gmail.com>
@cursor
Copy link

cursor bot commented Nov 27, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@BruinGrowly BruinGrowly marked this pull request as ready for review November 27, 2025 09:41
@BruinGrowly BruinGrowly merged commit 0836278 into main Nov 27, 2025
15 checks passed
@BruinGrowly BruinGrowly deleted the cursor/review-code-for-issues-and-improvements-gemini-3-pro-preview-4500 branch November 27, 2025 09:41
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.

3 participants