From 2f4fc54257d5cff42688f758aca47d3ed43d6dec Mon Sep 17 00:00:00 2001
From: CMGS <ilskdw@gmail.com>
Date: Thu, 9 Jul 2026 15:59:05 +0800
Subject: [PATCH] fix(pool): warn when archive is enabled with checkpoint TTL
 off

Archive relies on the checkpoint TTL to reclaim a checkpoint whose physical
delete failed (the claim is gone, so the ck is unpinned). With
checkpoint_ttl_hours=0 that orphan is never reclaimed; warn at startup.
---
 sandboxd/pool/pool.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sandboxd/pool/pool.go b/sandboxd/pool/pool.go
index 9232839..f0c2441 100644
--- a/sandboxd/pool/pool.go
+++ b/sandboxd/pool/pool.go
@@ -265,6 +265,9 @@ func NewManager(ctx context.Context, cfg *config.Config, eng Engine) (*Manager,
 			m.archiveEnabled = true
 		}
 	}
+	if m.archiveEnabled && m.ckptTTL == 0 {
+		log.WithFunc("pool.NewManager").Warn(ctx, "archive enabled with checkpoint_ttl_hours=0: a checkpoint whose delete fails is not reclaimed")
+	}
 	return m, nil
 }