From 5540282dcbe6a3de6ba8aba064e34fdbd826f8a6 Mon Sep 17 00:00:00 2001
From: seto <hirokazu.seto@gmail.com>
Date: Mon, 13 Apr 2026 15:41:58 +0900
Subject: [PATCH] fix: remove unnecessary disk I/O + fix test cleanup
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- isChecked() no longer calls saveState() — read-only operation
  should not write to disk (was causing 3x writes per tool call)
- Test cleanup uses fs.rmSync(recursive) instead of fs.rmdirSync
  which failed with ENOTEMPTY when .tmp files remained

9/9 tests pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 scripts/hooks/gateguard-fact-force.js    | 1 -
 tests/hooks/gateguard-fact-force.test.js | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/scripts/hooks/gateguard-fact-force.js b/scripts/hooks/gateguard-fact-force.js
index c200b83f..4290c7d3 100644
--- a/scripts/hooks/gateguard-fact-force.js
+++ b/scripts/hooks/gateguard-fact-force.js
@@ -84,7 +84,6 @@ function markChecked(key) {
 
 function isChecked(key) {
   const state = loadState();
-  saveState(state);
   return state.checked.includes(key);
 }
 
diff --git a/tests/hooks/gateguard-fact-force.test.js b/tests/hooks/gateguard-fact-force.test.js
index 98b1c909..e48b4ca1 100644
--- a/tests/hooks/gateguard-fact-force.test.js
+++ b/tests/hooks/gateguard-fact-force.test.js
@@ -308,10 +308,9 @@ function runTests() {
   })) passed++; else failed++;
 
   // Cleanup: remove test-isolated state directory
-  clearState();
   try {
     if (fs.existsSync(stateDir)) {
-      fs.rmdirSync(stateDir);
+      fs.rmSync(stateDir, { recursive: true, force: true });
     }
   } catch (err) {
     console.error(`  [cleanup] failed to remove ${stateDir}: ${err.message}`);