Skip to content

fix: remove unused imports across 20 source files#464

Open
Kailigithub wants to merge 1 commit into
lsdefine:mainfrom
Kailigithub:optimize/cleanup-unused-imports
Open

fix: remove unused imports across 20 source files#464
Kailigithub wants to merge 1 commit into
lsdefine:mainfrom
Kailigithub:optimize/cleanup-unused-imports

Conversation

@Kailigithub
Copy link
Copy Markdown
Contributor

Remove 38 unused import statements across 20 source files, as flagged by ruff check --select F401.

Changes:

  • memory/ljqCtrl.py — Removed os, sys, random, math, ImageEnhance, ImageFilter, ImageDraw
  • frontends/qtapp.py — Removed Signal, QMetaObject, Q_ARG, QObject, clean_reply, build_done_text, os
  • assets/code_run_header.py — Removed json, re, time
  • memory/L4_raw_sessions/compress_session.py — Removed json, ast
  • Plus 16 other files with 1-2 unused imports each

Note: The readline import in agentmain.py is kept with # noqa: F401 as it is a side-effect import that enables REPL command history.

Verification:

  • All 57 Python files pass python3 -m py_compile syntax check
  • ruff check --select F401 shows 0 remaining violations

Remove 38 unused import statements flagged by ruff F401. The only
exception is the  import in agentmain.py which is kept with
a noqa comment since it is a side-effect import for REPL history.
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