py(deps) ruff 0.15.22 -> 0.16.0

[tony]

Summary

• Bump the ruff floor to >=0.16.0 in the dev and lint dependency groups so contributors and CI see the same diagnostics.
• Adopt ruff 0.16.0's Markdown formatting: ruff format now formats Python code blocks inside .md files by default.
• Adopt ruff's default rule set by replacing [tool.ruff.lint] select with extend-select, taking the enabled rule count from 351 to 565.
• Fix every finding that surfaced, one commit per rule, and scope the two deliberate idioms as per-file ignores that carry their reason in the config.

Adopting the default rule set

ruff 0.16 ships a curated default rule set as its recommended baseline. An explicit select replaces that set rather than extending it, so this project was running only the linters it named and silently opting out of the rest. Renaming select to extend-select layers the project's own linters on top of the default set instead of in place of it, and every entry now carries a trailing comment naming the linter it selects.

select stays unset deliberately, and the config says so. RuleSelector has no DEFAULT token, so an explicit select cannot name the default set — leaving it unset is the only way to get defaults-plus-extras. Expanding to whole prefixes instead was rejected: it drags in all of D, PL, and S rather than the curated subset.

Bumping the floor rather than pinning an exact version keeps the lockfile the single source of the resolved version while still refusing anything older than the release whose formatter output and default rule set this repo is now checked against.

Rules fixed

Each landed as its own commit.

• PYI034 — PrivatePath.__new__ and two __enter__ methods were annotated with their own concrete class, so a subclass instance was inferred as the base. They return Self now. typing.Self needs 3.11 and the package floor is 3.10, so Self is imported from typing_extensions under TYPE_CHECKING; the runtime dependency smoke test still passes with only the declared runtime deps installed.
• PYI036 — __exit__ tail parameters typed as types.TracebackType | None and object rather than t.Any.
• PLE0101 — CouldNotGuessVCSFromURL.__init__ returned the result of super().__init__(...). A constructor may only return None.
• DTZ005 — the per-invocation debug log file name was stamped from a naive datetime.now(), whose meaning depends on the ambient system zone. .astimezone() attaches the local zone without changing the digits in the name.
• ISC004 — two multi-part strings sitting unparenthesized inside a list and a tuple. Wrapping them keeps a stray comma from silently splitting one value into two.
• PLW1510 — two test probes deliberately let git fail so they can assert on returncode; they now say check=False.
• TC005 — an if t.TYPE_CHECKING: block holding nothing but pass, plus the typing import that existed only to guard it.
• PIE790 — a pass trailing a method whose body was already a docstring.
• EXE001 — src/vcspull/__init__.py carried #!/usr/bin/env python despite being a package initializer that is never run as a script and is not executable. Note that ruff suppresses this rule under WSL, where the executable bit is unreliable, so it surfaces on CI but not on a WSL checkout.

Ignores added

Each landed as its own commit, with the reason written into pyproject.toml next to the entry.

• S102 in docs/conf.py — the Sphinx idiom of execing src/vcspull/__about__.py to read version, title, and repo URLs without importing (and therefore installing) the package. The exec'd input is a file in this repository, not user data.
• BLE001 in four files, each a boundary where the broad catch is the point rather than an oversight. scripts/runtime_dep_smoketest.py exists to report arbitrary import and CLI failures. src/vcspull/cli/_progress.py restores the terminal cursor from an atexit handler, whose raise the interpreter would swallow silently. src/vcspull/cli/sync.py catches inside a worker thread to hand the failure back to the main thread, which decides what to do with it. src/vcspull/log.py keeps a caller's %-format failure from taking the application down with the log formatter. Every one of these sites already carried a comment saying so.

Test plan

• uv run ruff check . — all checks passed
• uv run ruff format . --check — all files already formatted
• uv run mypy . — no issues found
• uv run py.test — full suite passes
• uvx --isolated --no-cache --from . python scripts/runtime_dep_smoketest.py — clean, confirming the typing_extensions import stays type-checking-only

tests/cli/test_sync_progress.py::test_stop_repo_panel_with_no_final_line_still_collapses is intermittently flaky — it races the running spinner thread against the test's direct writes to the indicator's internals. It reproduces at the same rate with this branch's change to that file reverted, so it predates this work and is not addressed here.

[codecov]

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.72%. Comparing base (40109c5) to head (b042904).

Files with missing lines	Patch %	Lines
src/vcspull/cli/sync.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #568   +/-   ##
=======================================
  Coverage   83.72%   83.72%           
=======================================
  Files          32       32           
  Lines        4596     4596           
  Branches      928      928           
=======================================
  Hits         3848     3848           
  Misses        497      497           
  Partials      251      251           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.