chore(ci): fix ruff import-sorting across repo (46 files)#57
Merged
Conversation
46 pre-existing I001 violations across test files and 3 non-test files blocked all PR CI runs. Applied ruff 0.15.14 --fix --select I to the full repo. Mechanical change, no logic touched. Breakdown: - 43 test files (advanced_rag_system/tests/, tests/, ghl_real_estate_ai/tests/, insight_engine/tests/) - 3 non-test files with 1-line whitespace/blank-line trims: - advanced_rag_system/hybrid_search.py - ghl_real_estate_ai/streamlit_demo/components/customer_intelligence_dashboard.py - conftest.py (root) Verified: ruff check . is now clean. (ruff format --check is still red for 44 unrelated files; out of scope for this PR.) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
4 tasks
ChunkyTortoise
added a commit
that referenced
this pull request
May 23, 2026
ruff format --check was red on ~44 files unrelated to recent PRs. This is the second of two CI rehab PRs (first was #57 for import sorting). All changes are mechanical formatter output, no logic touched. auto-claude/ excluded per pyproject.toml. Co-authored-by: Cayman <chunktort@gmail.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Clears 46 pre-existing ruff I001 (import block un-sorted) violations that were blocking the Code Quality Checks job on every open PR (#48 through #56). Auto-fix only, no logic changes.
Scope
advanced_rag_system/tests/,tests/,ghl_real_estate_ai/tests/,insight_engine/tests/advanced_rag_system/hybrid_search.pyghl_real_estate_ai/streamlit_demo/components/customer_intelligence_dashboard.pyconftest.py(root)Total: 46 files, 25 insertions, 60 deletions (85 lines touched).
auto-claude/is excluded bypyproject.toml[tool.ruff].exclude, so its 157 I001 errors are intentionally not in CI scope and not fixed here.Method
ruff 0.15.14pinned to match CI (requirements-dev.txtresolves to that version). Localruff 0.6.9does not flag these errors. Future repo work should useruff>=0.15.14to match CI behavior.Before / After
origin/main):ruff check .->Found 46 errors.ruff check .->All checks passed!Out of scope
ruff format --check .is also failing on 44 unrelated files. That's a separate cleanup and not addressed here. The Code Quality Checks job will still fail at the format step until that is fixed in a follow-up.Test plan
ruff check .clean on this branchgit diffreviewed: only import-block re-ordering and blank-line removal, no functional changesRun ruff checkpasses on this PRCo-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com