pytest_plugin(refactor[types]): Extract env types to public TypeAliases

why: `_ENV` from `_internal/run.py` leaked into public API signatures,
causing Sphinx to render its full expanded form
(`Mapping[bytes, ...] | Mapping[str, ...]`) in the docs instead of a
linkable name. `dict[str, str]` was also used ad-hoc in test
annotations without a shared name.
what:
- Add `Env: t.TypeAlias = _ENV` as public alias for the subprocess env
  type; replace `env: _ENV | None` in all public signatures
- Add `GitCommitEnvVars: t.TypeAlias = dict[str, str]` for the
  narrower git-commit env type; replace `git_commit_envvars: _ENV`
  in fixture parameters
- Update test TYPE_CHECKING imports to use `GitCommitEnvVars`

Author: tony

src/libvcs/pytest_plugin.py

@@ -66,7 +66,7 @@ def vcs_user(vcs_name: str, vcs_email: str) -> str:
 
 
 @pytest.fixture(scope="session")
-def git_commit_envvars(vcs_name: str, vcs_email: str) -> _ENV:
+def git_commit_envvars(vcs_name: str, vcs_email: str) -> GitCommitEnvVars:
     """Return environment variables for `git commit`.
 
     For some reason, `GIT_CONFIG` via {func}`set_gitconfig` doesn't work for `git
@@ -266,6 +266,8 @@ def unique_repo_name(remote_repos_path: pathlib.Path, max_retries: int = 15) ->
 
 
 InitCmdArgs: t.TypeAlias = list[str] | None
+GitCommitEnvVars: t.TypeAlias = dict[str, str]
+Env: t.TypeAlias = _ENV
 
 
 class CreateRepoPostInitFn(t.Protocol):
@@ -274,7 +276,7 @@ class CreateRepoPostInitFn(t.Protocol):
     def __call__(
         self,
         remote_repo_path: pathlib.Path,
-        env: _ENV | None = None,
+        env: Env | None = None,
     ) -> None:
         """Ran after creating a repo from pytest fixture."""
         ...
@@ -301,7 +303,7 @@ def _create_git_remote_repo(
     remote_repo_path: pathlib.Path,
     remote_repo_post_init: CreateRepoPostInitFn | None = None,
     init_cmd_args: InitCmdArgs = DEFAULT_GIT_REMOTE_REPO_CMD_ARGS,
-    env: _ENV | None = None,
+    env: Env | None = None,
 ) -> pathlib.Path:
     if init_cmd_args is None:
         init_cmd_args = []
@@ -434,7 +436,7 @@ def fn(
 
 def git_remote_repo_single_commit_post_init(
     remote_repo_path: pathlib.Path,
-    env: _ENV | None = None,
+    env: Env | None = None,
 ) -> None:
     """Post-initialization: Create a test git repo with a single commit."""
     testfile_filename = "testfile.test"
@@ -456,7 +458,7 @@ def git_remote_repo_single_commit_post_init(
 def git_remote_repo(
     create_git_remote_repo: CreateRepoPytestFixtureFn,
     gitconfig: pathlib.Path,
-    git_commit_envvars: _ENV,
+    git_commit_envvars: GitCommitEnvVars,
 ) -> pathlib.Path:
     """Copy the session-scoped Git repository to a temporary directory."""
     # TODO: Cache the effect of of this in a session-based repo
@@ -490,7 +492,7 @@ def _create_svn_remote_repo(
 
 def svn_remote_repo_single_commit_post_init(
     remote_repo_path: pathlib.Path,
-    env: _ENV | None = None,
+    env: Env | None = None,
 ) -> None:
     """Post-initialization: Create a test SVN repo with a single commit."""
     assert remote_repo_path.exists()
@@ -610,7 +612,7 @@ def _create_hg_remote_repo(
 
 def hg_remote_repo_single_commit_post_init(
     remote_repo_path: pathlib.Path,
-    env: _ENV | None = None,
+    env: Env | None = None,
 ) -> None:
     """Post-initialization: Create a test mercurial repo with a single commit."""
     testfile_filename = "testfile.test"
@@ -787,7 +789,7 @@ def add_doctest_fixtures(
     doctest_namespace: dict[str, t.Any],
     tmp_path: pathlib.Path,
     set_home: pathlib.Path,
-    git_commit_envvars: _ENV,
+    git_commit_envvars: GitCommitEnvVars,
     hgconfig: pathlib.Path,
     create_git_remote_repo: CreateRepoPytestFixtureFn,
     create_svn_remote_repo: CreateRepoPytestFixtureFn,

tests/cmd/test_git.py

@@ -12,7 +12,7 @@
 from libvcs.cmd import git
 
 if t.TYPE_CHECKING:
-    from libvcs.pytest_plugin import CreateRepoPytestFixtureFn
+    from libvcs.pytest_plugin import CreateRepoPytestFixtureFn, GitCommitEnvVars
     from libvcs.sync.git import GitSync
 
 
@@ -2287,7 +2287,7 @@ def test_reflog_entry_delete(git_repo: GitSync) -> None:
 @pytest.fixture
 def submodule_repo(
     tmp_path: pathlib.Path,
-    git_commit_envvars: dict[str, str],
+    git_commit_envvars: GitCommitEnvVars,
     set_gitconfig: pathlib.Path,
 ) -> git.Git:
     """Create a git repository to use as a submodule source."""

tests/test_pytest_plugin.py

@@ -15,7 +15,7 @@
 if t.TYPE_CHECKING:
     import pathlib
 
-    from libvcs.pytest_plugin import CreateRepoPytestFixtureFn
+    from libvcs.pytest_plugin import CreateRepoPytestFixtureFn, GitCommitEnvVars
     from libvcs.sync.git import GitSync
 
 
@@ -265,7 +265,7 @@ def test_gitconfig_submodule_file_protocol(
 def test_git_repo_fixture_submodule_file_protocol(
     git_repo: GitSync,
     create_git_remote_repo: CreateRepoPytestFixtureFn,
-    git_commit_envvars: dict[str, str],
+    git_commit_envvars: GitCommitEnvVars,
     user_path: pathlib.Path,
     monkeypatch: pytest.MonkeyPatch,
 ) -> None: