Use strict optional checking in legacy resolver

The assert in _set_req_to_reinstall is definitely correct, the three
call sites have guards

The assert in _populate_link is a little trickier. It's not obvious to
me how to prove it will never trigger. Note that if link were ever None,
it could potentially cause AttributeError in Cache._get_cache_path_parts
and direct_url_from_link

The assert in _resolve_one is safe, if req_to_install.name was None it
would raise TypeError in canonicalize_name

I can't prove that req.name is not None in get_installation_order
and the code would work fine if that were the case (since it's a
defaultdict), so I opted to just ignore

Author: hauntsaninja

news/4CCE4788-B8B3-402E-9A88-2981AD074999.trivial.rst

@@ -11,7 +11,6 @@
 """
 
 # The following comment should be removed at some point in the future.
-# mypy: strict-optional=False
 
 import logging
 import sys
@@ -325,6 +324,7 @@ def _set_req_to_reinstall(self, req: InstallRequirement) -> None:
         """
         # Don't uninstall the conflict if doing a user install and the
         # conflict is not a user install.
+        assert req.satisfied_by is not None
         if not self.use_user_site or req.satisfied_by.in_usersite:
             req.should_reinstall = True
         req.satisfied_by = None
@@ -423,6 +423,8 @@ def _populate_link(self, req: InstallRequirement) -> None:
 
         if self.wheel_cache is None or self.preparer.require_hashes:
             return
+
+        assert req.link is not None, "_find_requirement_link unexpectedly returned None"
         cache_entry = self.wheel_cache.get_cache_entry(
             link=req.link,
             package_name=req.name,
@@ -536,6 +538,7 @@ def add_req(subreq: Requirement, extras_requested: Iterable[str]) -> None:
         with indent_log():
             # We add req_to_install before its dependencies, so that we
             # can refer to it when adding dependencies.
+            assert req_to_install.name is not None
             if not requirement_set.has_requirement(req_to_install.name):
                 # 'unnamed' requirements will get added here
                 # 'unnamed' requirements can only come from being directly
@@ -591,7 +594,7 @@ def schedule(req: InstallRequirement) -> None:
             if req.constraint:
                 return
             ordered_reqs.add(req)
-            for dep in self._discovered_dependencies[req.name]:
+            for dep in self._discovered_dependencies[req.name]:  # type: ignore[index]
                 schedule(dep)
             order.append(req)