Skip to content

Fix 42 ruff lint violations (repo is not lint-clean) #7

Description

@williamzujkowski

AGENTS.md requires every PR to be lint-clean, but ruff check . currently reports 42 errors:

  • 15 I001 unsorted-imports
  • 9 F401 unused-import (including asdict in replay/trace_log.py)
  • 5 TC001 typing-only-first-party-import
  • 4 UP042 replace-str-enum
  • 2 B905 zip-without-explicit-strict
  • 2 F841 unused-variable
  • 2 UP035 deprecated-import
  • 1 RUF012 mutable-class-default (grid.py _action_handlers)
  • 1 RUF022, 1 TC003

27 are auto-fixable via ruff check --fix. Suggested fix: run the autofix, manually address B905 (add explicit strict= to zip), F841 (remove dead variables), UP042 (StrEnum migration or noqa), and RUF012 (annotate the class-level dict with ClassVar or build it in init). Then wire this into CI so it stays clean.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions