From 27907e6cac22a8b1070554fb6e02e918c2f0a8d4 Mon Sep 17 00:00:00 2001
From: John Ericson <John.Ericson@Obsidian.Systems>
Date: Fri, 18 Apr 2025 12:24:24 -0400
Subject: [PATCH] Add documentation for `DerivationBuilder::{parsedDrv,
 drvOptions}`

---
 .../include/nix/store/build/derivation-builder.hh   | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/libstore/unix/include/nix/store/build/derivation-builder.hh b/src/libstore/unix/include/nix/store/build/derivation-builder.hh
index 5fe528f65..c5382c90e 100644
--- a/src/libstore/unix/include/nix/store/build/derivation-builder.hh
+++ b/src/libstore/unix/include/nix/store/build/derivation-builder.hh
@@ -30,7 +30,20 @@ struct DerivationBuilderParams
      */
     const std::unique_ptr<Derivation> & drv;
 
+    /**
+     * The "structured attrs" of `drv`, if it has them.
+     *
+     * @todo this should be part of `Derivation`.
+     *
+     * @todo this should be renamed from `parsedDrv`.
+     */
     const std::unique_ptr<StructuredAttrs> & parsedDrv;
+
+    /**
+     * The derivation options of `drv`.
+     *
+     * @todo this should be part of `Derivation`.
+     */
     const std::unique_ptr<DerivationOptions> & drvOptions;
 
     // The remainder is state held during the build.