Skip to content

fix: add missing libraries to TIER_1_LIBRARIES for CLI consistency#34

Merged
youssefragab99 merged 1 commit intomainfrom
fix/critical-migration-blockers
Jan 29, 2026
Merged

fix: add missing libraries to TIER_1_LIBRARIES for CLI consistency#34
youssefragab99 merged 1 commit intomainfrom
fix/critical-migration-blockers

Conversation

@youssefragab99
Copy link
Collaborator

Several libraries had working Tier 1 AST transformers but were not listed in TIER_1_LIBRARIES, causing the CLI to incorrectly fall back to Tier 3 LLM instead of using the deterministic transforms.

Libraries added:

  • celery (10 tests)
  • httpx (10 tests)
  • aiohttp (9 tests)
  • click (9 tests)
  • attrs (12 tests)
  • django (14 tests)

All 352 unit tests pass. The migration engine now correctly reports "Using Tier 1 (deterministic AST transforms)" for these libraries.

Summary

Brief description of the changes in this PR.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

Related Issues

Fixes #(issue number)

Changes Made

  • Change 1
  • Change 2
  • Change 3

Testing

  • I have added tests that prove my fix/feature works
  • All existing tests pass locally
  • I have tested this manually

Checklist

  • My code follows the project's coding standards
  • I have run black and ruff on my code
  • I have updated documentation if needed
  • My changes don't introduce new warnings

Screenshots (if applicable)

Add screenshots or terminal output showing the changes.

Several libraries had working Tier 1 AST transformers but were not
listed in TIER_1_LIBRARIES, causing the CLI to incorrectly fall back
to Tier 3 LLM instead of using the deterministic transforms.

Libraries added:
- celery (10 tests)
- httpx (10 tests)
- aiohttp (9 tests)
- click (9 tests)
- attrs (12 tests)
- django (14 tests)

All 352 unit tests pass. The migration engine now correctly reports
"Using Tier 1 (deterministic AST transforms)" for these libraries.
@youssefragab99 youssefragab99 merged commit 8abccfe into main Jan 29, 2026
5 checks passed
youssefragab99 added a commit that referenced this pull request Feb 7, 2026
…-blockers

fix: add missing libraries to TIER_1_LIBRARIES for CLI consistency
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