Skip to content

[codex] Clear Ruff baseline violations#446

Merged
proboscis merged 35 commits into
mainfrom
issue/ruff-clean-baseline/run-20260611-195750
Jun 11, 2026
Merged

[codex] Clear Ruff baseline violations#446
proboscis merged 35 commits into
mainfrom
issue/ruff-clean-baseline/run-20260611-195750

Conversation

@proboscis

Copy link
Copy Markdown
Owner

概要

ruff-clean-baseline の指示に従い、doeff/tests/packages/ の Ruff baseline 違反を解消しました。主な変更は Ruff の自動修正、未使用 import/変数の整理、相対 import の絶対 import 化、既存挙動を変えない範囲での check=False/pytest.raises(match=...)/contextlib.suppress などの lint 対応です。

公開 API の re-export は削除せず、Name as Name 形式や __init__.py の明示 re-export として維持しています。doeff.program の VM node re-export も復元し、doeff.__init__ は循環 import を避ける順序にしています。

noqa の扱い

挙動変更を避けるため、次のカテゴリは理由付き noqa として残しています。

  • public/compat API: N802A002N812N817N818 は既存公開名や互換 alias を維持するため。
  • legacy/spec tests: F821 は削除済み API を参照する仕様・回帰テストを、その意図のまま残すため。
  • baseline complexity: PLR0911PLR0912PLR0915 は lint cleanup で制御フローを変えないため。
  • import order/late import: E402I001 は fixture の sys.path セットアップや循環 import 回避のため。
  • fixture literals: RUF001 は traceback や UI fixture の literal glyph を保持するため。
  • framework/static metadata: RUF012 は Textual などの class-level metadata を維持するため。
  • existing runtime behavior: DTZ005PLW0603 は既存の local wall-clock や module-level state model を維持するため。

差分内の新規 noqa は理由なしで残らないことも確認済みです。

検証

  • uv run ruff check doeff/ tests/ packages/ -> All checks passed!
  • uv run pytest -> 842 passed, 84 skipped, 54327 warnings
  • make lint-pyright -> 0 errors, 0 warnings, 0 informations
  • public API import smoke -> public api imports ok
  • git diff --check origin/main...HEAD -> clean

影響

挙動変更は意図していません。変更は Ruff baseline cleanup と re-export 維持に限定しています。

@proboscis proboscis marked this pull request as ready for review June 11, 2026 11:21
@proboscis proboscis merged commit 71e4ac6 into main Jun 11, 2026
1 check 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