Type Parameter Modernization

docs/developer/feature-test-map/core.md

@@ -58,11 +58,13 @@ https://github.com/TNO/Renaissance.Py/blob/main/LICENSE
     - `test/recipes/test_python_refactoring.py`
     - `test/utils/test_cst_utils.py`
     - `test/utils/test_import_resolution.py`
+    - `test/utils/test_ruff_utils.py`
     - `test/rejuvenation/test_migration_type_recipes.py`
 - **Code file(s):** `src/renaissance/recipes/type_parameter_check.py`, `src/renaissance/recipes/type_parameter_rules.py`,
   `src/renaissance/recipes/type_parameter_declarations.py`, `src/renaissance/recipes/type_parameter_localization.py`,
   `src/renaissance/recipes/type_parameter_uses.py`,
   `src/renaissance/recipes/step_runner.py`,
   `src/renaissance/recipes/python_refactoring.py`,
   `src/renaissance/utils/import_resolution.py`, `src/renaissance/utils/cst_utils.py`,
+  `src/renaissance/utils/ruff_utils.py`,
   `src/rejuvenation/migration_type_recipes.py`

docs/developer/modules/recipes.md

@@ -34,11 +34,9 @@ safely cover. This page covers `TypeParameterCheck`, the recipe behind
   names each project file imports from the others.
 - `src/renaissance/utils/cst_utils.py`: a module's top-level statements, the names they read or bind and the comment
   trailing a statement's line.
-
-<!-- TODO: FrancescoPezzella - list ruff_utils.py here once a recipe uses run_ruff_import_cleanup (#111):
-- `src/renaissance/utils/ruff_utils.py`: `run_ruff_import_cleanup`, which removes unused imports and sorts the imports
-  of the files a recipe changed, with ruff. -->
-
+- `src/renaissance/utils/ruff_utils.py`: `run_ruff_import_cleanup` and `run_ruff_format`, which the CLI runs on the
+  files the recipe changed. The first removes unused imports and sorts the imports, the second formats the files.
+
 ## Public entry points
 
 - `TypeParameterCheck.check()` (and `run()`): runs the three phases through `run_steps`, committing to disk between them.

docs/user/features/type-parameter-modernization.md

@@ -404,14 +404,15 @@ run the tool again.
 - `test/recipes/test_python_refactoring.py`
 - `test/utils/test_cst_utils.py`
 - `test/utils/test_import_resolution.py`
+- `test/utils/test_ruff_utils.py`
 - `test/rejuvenation/test_migration_type_recipes.py`
 
 ## Implemented by code modules
 
 - [Refactoring recipes](../../developer/modules/recipes.md)
 - `src/renaissance/recipes/type_parameter_check.py`, `type_parameter_rules.py`, `type_parameter_declarations.py`,
   `type_parameter_localization.py`, `type_parameter_uses.py`, `step_runner.py` and `python_refactoring.py`.
-- `src/renaissance/utils/import_resolution.py` and `cst_utils.py`
+- `src/renaissance/utils/import_resolution.py`, `cst_utils.py` and `ruff_utils.py`
 - `src/rejuvenation/migration_type_recipes.py` (the CLI)
 
 ## API entry points
@@ -432,12 +433,10 @@ python src/rejuvenation/migration_type_recipes.py <path> --py MAJOR.MINOR \
 - `--second-pass`: process again only the files that kept a declaration because another file imported it when the run
   started, and merge the result into the same report.
 
-After the recipe, the CLI runs `ruff check --fix --select F401,I001,UP049` on the changed files. This removes **every**
+After the recipe, the CLI runs `ruff check --fix --select F401,I001` on the changed files. This removes **every**
 unused import in them, including ones that were unused before (such as `from typing import TypeVar` after the
 declaration is gone). It also merges and sorts **every** import block in them, including imports that were unsorted
-before, using the `ruff` settings of the project. It removes the leading underscores of **every** PEP 695 type
-parameter in them, so `def f[_T]` becomes `def f[T]`, unless the new name would shadow another one. Docstrings,
-comments and the report keep the old name. The project's `select` and `ignore` lists do not apply to these three
+before, using the `ruff` settings of the project. The project's `select` and `ignore` lists do not apply to these two
 rules. Then it runs `ruff format` on the same files, which reformats **every** line in them with the project's
 `ruff format` settings, not only the changed ones. `ruff` must be installed in the same environment. Files are changed
 directly, so run it on a git checkout.