From 18941a2421f40efad264ad1fc4e07339075491c3 Mon Sep 17 00:00:00 2001
From: John Ericson <John.Ericson@Obsidian.Systems>
Date: Tue, 21 Oct 2025 11:37:14 -0400
Subject: [PATCH] Optimize `DummyStore::isValidPathUncached`

See the API docs for the rationale of why this is needed.
---
 src/libstore/dummy-store.cc | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/libstore/dummy-store.cc b/src/libstore/dummy-store.cc
index d11fef73f..c45a13cc3 100644
--- a/src/libstore/dummy-store.cc
+++ b/src/libstore/dummy-store.cc
@@ -164,6 +164,15 @@ struct DummyStoreImpl : DummyStore
         callback(nullptr);
     }
 
+    /**
+     * Do this to avoid `queryPathInfoUncached` computing `PathInfo`
+     * that we don't need just to return a `bool`.
+     */
+    bool isValidPathUncached(const StorePath & path) override
+    {
+        return path.isDerivation() ? derivations.contains(path) : Store::isValidPathUncached(path);
+    }
+
     /**
      * The dummy store is incapable of *not* trusting! :)
      */