Skip to content

Conversation

DavidVujic
Copy link
Owner

Description

The poly check command performs AST parsing to find any missing bricks or third-party imports. A module that already have been parsed should be cached during the session. This is done since before, but with the default max size of lru_cache. For large Polylith workspaces, the cache need to be bigger.

Motivation and Context

See discussion #354

How Has This Been Tested?

✅ CI
✅ running the poly check command for a repo with 18+ projects, 350+ bricks and with large python files (20K LOC).

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

Copy link

@DavidVujic DavidVujic merged commit 6a008bb into main Jul 27, 2025
4 checks passed
@DavidVujic DavidVujic deleted the poly-check-performance branch August 4, 2025 08:14
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.

1 participant