fix(env): PR #25 cross-review round 1 gemini 指摘の修正

- `_generate_default_filename` を新設し `_default_dest` / `_default_filename`
  の結合を解消 (major / 設計)
- `_complete_dir_dest` の `endswith` をタプル形式に統一 (minor / 可読性)
- `test_complete_dir_dest_local_trailing_slash` を `os.sep` 完全パス比較
  (`==`) に変更 (minor / テスト)

Author: takemi-ohama

lib/devbase/env/io_export.py

@@ -45,17 +45,23 @@ class ExportOptions:
     unsafe_allow_unencrypted_bucket: bool = False
 
 
-def _default_dest(force_unencrypted: bool) -> str:
+def _generate_default_filename(force_unencrypted: bool) -> str:
+    """既定ファイル名 (prefix なし) を生成する共通ヘルパー。
+    `_default_dest` / `_default_filename` の両方から呼ぶことで結合度を下げる
+    (PR #25 cross-review round 1 gemini 指摘)。"""
     # microsecond まで含めて衝突を回避する (PR #22 codex round 3 指摘)
     ts = datetime.now().strftime('%Y%m%d-%H%M%S-%f')
     suffix = '.dbenv.tar.gz' if force_unencrypted else '.dbenv'
-    return f'./devbase-env-{ts}{suffix}'
+    return f'devbase-env-{ts}{suffix}'
+
+
+def _default_dest(force_unencrypted: bool) -> str:
+    return f'./{_generate_default_filename(force_unencrypted)}'
 
 
 def _default_filename(force_unencrypted: bool) -> str:
-    """`_default_dest` の `./` prefix を除いたファイル名部分のみを返す。
-    dest がディレクトリ的なときに append する用途。"""
-    return _default_dest(force_unencrypted).removeprefix('./')
+    """既定ファイル名 (prefix なし) を返す。dest がディレクトリ的なときに append する用途。"""
+    return _generate_default_filename(force_unencrypted)
 
 
 def _complete_dir_dest(dest: str, force_unencrypted: bool) -> str:
@@ -72,7 +78,7 @@ def _complete_dir_dest(dest: str, force_unencrypted: bool) -> str:
         return dest + name if dest.endswith('/') else dest
     # ローカル: 既存ディレクトリか末尾 `/` ならディレクトリ扱い
     p = Path(dest)
-    if dest.endswith('/') or dest.endswith(os.sep) or p.is_dir():
+    if dest.endswith(('/', os.sep)) or p.is_dir():
         return str(p / name)
     return dest
 

tests/cli/test_env_export.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import io
+import os
 from pathlib import Path
 
 import pyrage
@@ -390,8 +391,10 @@ def test_complete_dir_dest_local_trailing_slash(tmp_path, monkeypatch):
         "devbase.env.io_export._default_filename",
         lambda fu: "devbase-env-FIXED.dbenv",
     )
-    target = str(tmp_path / "nodir") + "/"
-    assert _complete_dir_dest(target, False).endswith("/nodir/devbase-env-FIXED.dbenv")
+    target_dir = tmp_path / "nodir"
+    target_path_with_slash = str(target_dir) + os.sep
+    expected_path = str(target_dir / "devbase-env-FIXED.dbenv")
+    assert _complete_dir_dest(target_path_with_slash, False) == expected_path
 
 
 def test_complete_dir_dest_local_normal_file_unchanged(tmp_path, monkeypatch):