From 79ace278ec8d4c42883095512d15b32a0a5b7d65 Mon Sep 17 00:00:00 2001
From: Matt Sturgeon <matt@sturgeon.me.uk>
Date: Wed, 17 Dec 2025 18:33:36 +0000
Subject: [PATCH] docs/mdbook: refactor `isVisible`'s "has visible" check

---
 docs/mdbook/default.nix | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/docs/mdbook/default.nix b/docs/mdbook/default.nix
index 39dc396c..36e748b6 100644
--- a/docs/mdbook/default.nix
+++ b/docs/mdbook/default.nix
@@ -66,6 +66,10 @@ let
           visible' = if lib.isBool visible then visible else visible != "transparent";
         in
         visible' && !internal;
+
+      # FIXME: isVisible is not a perfect check;
+      # it will false-positive on `visible = "transparent"`
+      hasVisible = opts: lib.any (v: lib.isAttrs v -> isVisible v) (lib.attrValues opts);
     in
     opts:
     if lib.isOption opts then
@@ -73,14 +77,10 @@ let
     else if opts.isOption then
       test opts.index.options
     else
-      let
-        # FIXME: isVisible is not a perfect check;
-        # it will false-positive on `visible = "transparent"`
-        filterFunc = lib.filterAttrs (_: v: if lib.isAttrs v then isVisible v else true);
-        hasEmptyIndex = (filterFunc opts.index.options) == { };
-        hasEmptyComponents = (filterFunc opts.components) == { };
-      in
-      !hasEmptyIndex || !hasEmptyComponents;
+      lib.any hasVisible [
+        opts.index.options
+        opts.components
+      ];
 
   wrapOptionDocPage = path: opts: isOpt: rec {
     index = {