From 650ca7a3725cdd90b1057d292ea7d7cd75e21e9b Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 25 Nov 2025 13:10:51 +0000
Subject: [PATCH] docs(test): clarify purpose of underscore-prefixed fields in
 SharedServer

Co-authored-by: jnsgruk <668505+jnsgruk@users.noreply.github.com>
---
 tests/cli/helpers.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tests/cli/helpers.rs b/tests/cli/helpers.rs
index 0349ba5..7cfb14e 100644
--- a/tests/cli/helpers.rs
+++ b/tests/cli/helpers.rs
@@ -4,7 +4,11 @@ use std::sync::Mutex;
 use std::time::Duration;
 use tempfile::TempDir;
 
-/// Shared test server state - kept alive for the duration of all tests
+/// Shared test server state
+/// Fields prefixed with `_` are kept alive to prevent cleanup:
+/// - `_temp_dir`: keeps temporary database file from being deleted
+/// - `_process`: keeps server process running
+/// - `_db_url`: retained for consistency
 struct SharedServer {
     address: String,
     admin_password: String,