[davegaeddert]

Summary

• ruff 0.16.10 → 0.17.1 (plain-code pin >=0.17.1,<0.18), ty 0.0.84 → 0.0.89
• ruff 0.17 autofixes; @contextmanager returns switched from Iterator to Generator (ty deprecation)
• CI fix: the lint job ran a bare uv python install, which now picks Python 3.15, and psycopg-binary has no cp315 wheel, so uv sync failed before lint ran. It's now pinned to 3.14, like the test matrix.
• ty 0.0.86 added attribute/property override checks and 0.0.87 turned on invalid-mutable-override by default, which produced 51 new diagnostics. Where the annotation was wrong, I fixed the annotation:
  • admin cards / parent_view_class hold classes (type[...]), and card filters holds an Enum class
  • background_command is a tuple, FormView.success_url is str | None
  • OAuthCallbackView.oauth_error is declared (two getattr workarounds gone)
  • UploadedFile(size=None) no longer shadows File.size with None (a latent bug)
  • streaming responses get a raising content setter that matches the read-only getter
  • plain-postgres: Lookup.lookup_name is now ClassVar[str] (4 asserts removed), on_delete lives only on ForeignKeyRel, SQLWriteCompiler is generic over its query, the ChoicesMeta properties are typed against TextChoices (2 ignores removed), and Query.chain checks isinstance(UpdateQuery) (1 ignore removed)
  • the admin list base accepts the dict[str, Q] form of filters, so the model view's get_filter_names override is gone
• Narrowings ty can't verify keep targeted ignores, each with a reason: FK/M2M remote_field and rel.field, AdminView.user, the documented MCP mcp: AppMCP pattern, Lookup.identity / Query.output_field (can't be cached like the base), and one test that deliberately exercises a property override.

Net: ty ignores +9/−3, asserts −4, casts −4.

Testing

• ./scripts/test (all packages, local Postgres 16): all pass
• plain-code check . --skip-ty, ./scripts/type-validate, ./scripts/test-package-imports: clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01SDZ9zhE4BiReFLFQ9sH2tW

[pullapprove5]

PASS: 1 review scope passed

	Scope	Progress
✅	code	1/1

View in PullApprove

Next steps:

• @davegaeddert: Merge or close