From 34de68d2604f12ab625b1207fef949db2bc284da 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 4608d3077..5924f897c 100644
--- a/src/libstore/dummy-store.cc
+++ b/src/libstore/dummy-store.cc
@@ -168,6 +168,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! :)
      */