Skip to content

fix(scanner): clean dead-code warnings#6

Merged
duriantaco merged 1 commit into
mainfrom
fix/skylos-dead-code-cleanup
May 28, 2026
Merged

fix(scanner): clean dead-code warnings#6
duriantaco merged 1 commit into
mainfrom
fix/skylos-dead-code-cleanup

Conversation

@duriantaco

Copy link
Copy Markdown
Owner

Summary

  • remove genuinely unused pickle static scan symbols reported by Skylos
  • replace dynamic eval-config AST visitor hooks with explicit ast.walk checks so Skylos no longer flags them as unused
  • add regression coverage for Python assignment, annotated assignment, and dict-literal eval config scanning

Verification

  • python -m pytest -q
  • python -m ruff check .
  • python -m compileall -q ceres tests scripts
  • skylos . --quality --no-upload --file-filter ceres/analyzers/model/pickle_static.py --format concise (no dead-code findings remain for this file; other quality findings still reported)
  • skylos . --quality --no-upload --file-filter ceres/analyzers/eval/safety_config.py --format concise (no dead-code findings remain for this file; other quality findings still reported)

Remove unused pickle scan symbols and make eval config AST scanning explicit while preserving assignment coverage.
@duriantaco duriantaco merged commit 036e46a into main May 28, 2026
4 checks passed
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