Enable FIX002 and check alphabetical sort of ALL_TOOLS (#496)

Author: nathanjmcdougall

pyproject.toml

@@ -97,7 +97,7 @@ lint.select = [
   "EM",
   "ERA",
   "F",
-  # "FIX002",
+  "FIX002",
   "FURB",
   "I",
   "INP",

src/usethis/_core/author.py

@@ -1,13 +1,10 @@
-# TODO can we default to git info?
-# TODO neither name or email is strictly required, but at least one is.
-
-
 from usethis._integrations.file.pyproject_toml.io_ import PyprojectTOMLManager
 from usethis._integrations.uv.init import ensure_pyproject_toml
 
 
 def add_author(
     *,
+    # Strictly, a name is not required by the TOML spec, but we do require it
     name: str,
     email: str | None = None,
     overwrite: bool = False,

src/usethis/_tool.py

@@ -1563,4 +1563,3 @@ def _ensure_exists(file_manager: KeyValueFileManager) -> None:
     RequirementsTxtTool(),
     RuffTool(),
 ]
-# TODO test that the list is sorted alphabetically by CLI name

tests/usethis/test_usethis_tool.py

@@ -12,6 +12,7 @@
 from usethis._integrations.uv.deps import Dependency, add_deps_to_group
 from usethis._test import change_cwd
 from usethis._tool import (
+    ALL_TOOLS,
     ConfigEntry,
     ConfigItem,
     ConfigSpec,
@@ -1370,3 +1371,9 @@ def test_no_rules(self, tmp_path: Path):
 
                 # Assert
                 assert RuffTool().get_ignored_rules() == ["A"]
+
+
+class TestAllTools:
+    def test_sorted_alphabetically(self):
+        names = [tool.name.lower() for tool in ALL_TOOLS]
+        assert names == sorted(names)