Fix SIM103 lint and CI git identity in parallel worktree test

Author: johnteee

teaagent/cli/_handlers/_doctor.py

@@ -52,11 +52,11 @@ def _looks_like_sensitive_string(value: str) -> bool:
         return True
     if candidate.startswith(('sk-', 'rk-', 'pk-', 'ghp_', 'xoxb-', 'xoxp-')):
         return True
-    if len(candidate) >= 20 and any(ch.isdigit() for ch in candidate) and any(
-        ch.isalpha() for ch in candidate
-    ):
-        return True
-    return False
+    return (
+        len(candidate) >= 20
+        and any(ch.isdigit() for ch in candidate)
+        and any(ch.isalpha() for ch in candidate)
+    )
 
 
 def _redact_sensitive_fields(

tests/acceptance/test_subagent_parallel_worktree_merge_flow.py

@@ -24,11 +24,18 @@ def test_parallel_worktree_children_expose_lineage_for_parent_review(self) -> No
             subprocess.run(
                 ['git', 'add', 'README.md'], cwd=root, check=True, capture_output=True
             )
+            env = {
+                'GIT_AUTHOR_NAME': 'test',
+                'GIT_AUTHOR_EMAIL': 'test@test.com',
+                'GIT_COMMITTER_NAME': 'test',
+                'GIT_COMMITTER_EMAIL': 'test@test.com',
+            }
             subprocess.run(
                 ['git', 'commit', '-m', 'init'],
                 cwd=root,
                 check=True,
                 capture_output=True,
+                env=env,
             )
             (root / '.teaagent').mkdir()
             registry = ToolRegistry()