fix(snapshot): mtime 判定をアーカイブ実体に限定 + 未来日時ガード + テスト追加

- last_snapshot_time() の mtime 集計対象を full.tar.zst / incr-*.tar.zst に限定
  (meta.yml / snapshot.snar 等で誤スキップしないように)
- _auto_snapshot() で経過時間が負 (last が未来) の場合はスキップしない
- _snapshot_min_interval_minutes / last_snapshot_time の単体テスト追加

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: takemi-ohama

lib/devbase/commands/container.py

@@ -412,16 +412,18 @@ def _auto_snapshot() -> None:
         mgr = SnapshotManager(Path(devbase_root))
         min_interval = _snapshot_min_interval_minutes()
         last = mgr.last_snapshot_time()
-        if (
-            min_interval > 0
-            and last is not None
-            and datetime.now() - last < timedelta(minutes=min_interval)
-        ):
-            logger.info(
-                "[0/6] 直近のスナップショット (%s) から%d分以内のためスキップします",
-                last.strftime('%Y-%m-%d %H:%M:%S'), min_interval,
-            )
-            return
+        if min_interval > 0 and last is not None:
+            # 経過時間が負 (last が未来) の場合はスキップしない。システム時計の
+            # ズレや他環境からのリストアで last が未来になると delta が負になり、
+            # 常に閾値未満と判定されて無期限にスキップされてしまうため、
+            # timedelta(0) <= delta の下限ガードを設ける。
+            delta = datetime.now() - last
+            if timedelta(0) <= delta < timedelta(minutes=min_interval):
+                logger.info(
+                    "[0/6] 直近のスナップショット (%s) から%d分以内のためスキップします",
+                    last.strftime('%Y-%m-%d %H:%M:%S'), min_interval,
+                )
+                return
         if mgr.should_start_new_generation():
             logger.info("[0/6] 新しいスナップショット世代を作成中...")
             mgr.create()

lib/devbase/snapshot/manager.py

@@ -98,10 +98,16 @@ def list(self) -> list[dict]:
     def last_snapshot_time(self) -> Optional[datetime]:
         """直近のスナップショット取得 (フル/差分) 日時を返す。
 
-        各スナップショットディレクトリ内のアーカイブファイルの mtime のうち
-        最新のものを採用する。差分更新は既存ディレクトリ名を再利用するため
-        (ディレクトリ名の日付は世代作成時のまま) ファイルの mtime を実測する方が
-        正確で、メタデータの整合性にも依存しない。
+        各スナップショットディレクトリ内のアーカイブ実体
+        (``full.tar.zst`` / ``incr-*.tar.zst``) の mtime のうち最新のものを採用する。
+        差分更新は既存ディレクトリ名を再利用するため (ディレクトリ名の日付は世代
+        作成時のまま) ファイルの mtime を実測する方が正確で、メタデータの整合性にも
+        依存しない。
+
+        ``meta.yml`` / ``snapshot.snar`` (listed-incremental 状態ファイル) や
+        ``.bak`` 等の付随ファイルは集計対象から除外する。これらはバックアップ本体の
+        作成に失敗 (コピーや差分作成失敗) しても残りうるため、これらの mtime を採用
+        すると「成功したバックアップ本体が無いのに up がスキップされる」状態を招く。
 
         スナップショットが存在しない場合は None。
         """
@@ -114,6 +120,12 @@ def last_snapshot_time(self) -> Optional[datetime]:
             for f in snap_dir.iterdir():
                 if not f.is_file():
                     continue
+                # アーカイブ実体 (full.tar.zst / incr-NNN.tar.zst) のみを対象とし、
+                # meta.yml / snapshot.snar / *.bak 等は除外する。
+                if f.name != 'full.tar.zst' and not (
+                    f.name.startswith('incr-') and f.name.endswith('.tar.zst')
+                ):
+                    continue
                 mtime = f.stat().st_mtime
                 if latest is None or mtime > latest:
                     latest = mtime

tests/snapshot/__init__.py

@@ -0,0 +1,128 @@
+"""自動スナップショットの最小間隔判定まわりの単体テスト。
+
+カバー対象:
+  - ``_snapshot_min_interval_minutes`` (commands/container.py): 環境変数の
+    パースと不正値フォールバック。
+  - ``SnapshotManager.last_snapshot_time`` (snapshot/manager.py): アーカイブ実体
+    (full.tar.zst / incr-*.tar.zst) の mtime のみを集計し、meta.yml /
+    snapshot.snar 等は除外することの確認。
+"""
+
+from __future__ import annotations
+
+import os
+from datetime import datetime
+
+import pytest
+
+from devbase.commands.container import (
+    _SNAPSHOT_MIN_INTERVAL_MINUTES_DEFAULT,
+    _snapshot_min_interval_minutes,
+)
+from devbase.snapshot.manager import SnapshotManager
+
+
+# ---------------------------------------------------------------------------
+# _snapshot_min_interval_minutes
+# ---------------------------------------------------------------------------
+
+_ENV = 'DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES'
+
+
+def test_min_interval_unset_returns_default(monkeypatch):
+    monkeypatch.delenv(_ENV, raising=False)
+    assert _snapshot_min_interval_minutes() == _SNAPSHOT_MIN_INTERVAL_MINUTES_DEFAULT
+    assert _SNAPSHOT_MIN_INTERVAL_MINUTES_DEFAULT == 60
+
+
+def test_min_interval_valid_value(monkeypatch):
+    monkeypatch.setenv(_ENV, '30')
+    assert _snapshot_min_interval_minutes() == 30
+
+
+def test_min_interval_zero_disables(monkeypatch):
+    monkeypatch.setenv(_ENV, '0')
+    assert _snapshot_min_interval_minutes() == 0
+
+
+def test_min_interval_negative_falls_back_with_warning(monkeypatch, caplog):
+    monkeypatch.setenv(_ENV, '-5')
+    with caplog.at_level('WARNING'):
+        result = _snapshot_min_interval_minutes()
+    assert result == _SNAPSHOT_MIN_INTERVAL_MINUTES_DEFAULT
+    assert any('Invalid' in r.getMessage() for r in caplog.records)
+
+
+def test_min_interval_non_numeric_falls_back_with_warning(monkeypatch, caplog):
+    monkeypatch.setenv(_ENV, 'abc')
+    with caplog.at_level('WARNING'):
+        result = _snapshot_min_interval_minutes()
+    assert result == _SNAPSHOT_MIN_INTERVAL_MINUTES_DEFAULT
+    assert any('Invalid' in r.getMessage() for r in caplog.records)
+
+
+# ---------------------------------------------------------------------------
+# SnapshotManager.last_snapshot_time
+# ---------------------------------------------------------------------------
+
+def _touch(path, mtime: float) -> None:
+    path.write_bytes(b'x')
+    os.utime(path, (mtime, mtime))
+
+
+def test_last_snapshot_time_empty_backups(tmp_path):
+    mgr = SnapshotManager(tmp_path)
+    # __init__ で backups ディレクトリは作成されるが中身は空。
+    assert mgr.last_snapshot_time() is None
+
+
+def test_last_snapshot_time_missing_backups_dir(tmp_path):
+    mgr = SnapshotManager(tmp_path)
+    # backups ディレクトリ自体を消した場合も None。
+    import shutil
+    shutil.rmtree(mgr.backups_dir)
+    assert mgr.last_snapshot_time() is None
+
+
+def test_last_snapshot_time_uses_newest_archive(tmp_path):
+    mgr = SnapshotManager(tmp_path)
+    snap_dir = mgr.backups_dir / '20240101-000000'
+    snap_dir.mkdir()
+
+    old = 1_700_000_000.0
+    new = 1_700_086_400.0  # +1 day
+    _touch(snap_dir / 'full.tar.zst', old)
+    _touch(snap_dir / 'incr-001.tar.zst', new)
+
+    result = mgr.last_snapshot_time()
+    assert result == datetime.fromtimestamp(new)
+
+
+def test_last_snapshot_time_ignores_meta_and_snar(tmp_path):
+    """meta.yml / snapshot.snar が新しくても、アーカイブ実体の mtime を返す。"""
+    mgr = SnapshotManager(tmp_path)
+    snap_dir = mgr.backups_dir / '20240101-000000'
+    snap_dir.mkdir()
+
+    archive_old = 1_700_000_000.0
+    noise_new = 1_700_200_000.0  # アーカイブより新しい付随ファイル
+
+    _touch(snap_dir / 'full.tar.zst', archive_old)
+    _touch(snap_dir / 'meta.yml', noise_new)
+    _touch(snap_dir / 'snapshot.snar', noise_new)
+    _touch(snap_dir / 'snapshot.snar.bak', noise_new)
+
+    result = mgr.last_snapshot_time()
+    # 付随ファイルは除外され、アーカイブ実体の古い mtime が返るはず。
+    assert result == datetime.fromtimestamp(archive_old)
+
+
+def test_last_snapshot_time_only_noise_returns_none(tmp_path):
+    """アーカイブ実体が一切無く付随ファイルだけの場合は None。"""
+    mgr = SnapshotManager(tmp_path)
+    snap_dir = mgr.backups_dir / '20240101-000000'
+    snap_dir.mkdir()
+    _touch(snap_dir / 'meta.yml', 1_700_200_000.0)
+    _touch(snap_dir / 'snapshot.snar', 1_700_200_000.0)
+
+    assert mgr.last_snapshot_time() is None