From ab9b587018e03484f67febcaf38f2c11413d1e25 Mon Sep 17 00:00:00 2001
From: jutty <j@jutty.dev>
Date: Wed, 3 Jun 2026 14:35:50 -0300
Subject: [PATCH] Rename 'hidden' node option to 'listed'

---
 src/graph.rs                 |  2 +-
 src/graph/node.rs            | 17 ++++++++++-------
 src/router/handlers/graph.rs |  2 +-
 templates/base.html          |  4 ++--
 templates/index.html         |  2 +-
 templates/node.html          |  2 +-
 templates/tree.html          |  4 ++--
 7 files changed, 18 insertions(+), 15 deletions(-)

diff --git a/src/graph.rs b/src/graph.rs
index 67a32b5..0fd940c 100644
--- a/src/graph.rs
+++ b/src/graph.rs
@@ -665,7 +665,7 @@ mod tests {
                     "title": "JSON",
                     "links": [],
                     "id": "JSON",
-                    "hidden": false,
+                    "listed": true,
                     "connections": {}
                 }
             },
diff --git a/src/graph/node.rs b/src/graph/node.rs
index ff0c719..3472641 100644
--- a/src/graph/node.rs
+++ b/src/graph/node.rs
@@ -18,8 +18,8 @@ pub struct Node {
     pub links: Vec<String>,
     #[serde(default)]
     pub redirect: String,
-    #[serde(default)]
-    pub hidden: bool,
+    #[serde(default = "mktrue")]
+    pub listed: bool,
 
     #[serde(default)]
     pub connections: HashMap<String, Edge>,
@@ -28,6 +28,9 @@ pub struct Node {
     pub stats: Stats,
 }
 
+// See: https://github.com/serde-rs/serde/issues/368
+const fn mktrue() -> bool { true }
+
 #[derive(Serialize, Deserialize, Clone, Default, Eq, PartialEq, Debug)]
 pub struct Stats {
     pub outgoing: u32,
@@ -46,7 +49,7 @@ impl Node {
             connections: HashMap::default(),
             links: vec![],
             redirect: String::default(),
-            hidden: false,
+            listed: true,
             summary: String::default(),
             stats: Stats::default(),
         }
@@ -78,8 +81,8 @@ impl std::fmt::Display for Node {
             meta_elements.push(format!("links:{links}"));
         }
 
-        if self.hidden {
-            meta_elements.push(String::from("hidden"));
+        if !self.listed {
+            meta_elements.push(String::from("unlisted"));
         }
 
         let meta = meta_elements.join(" ");
@@ -143,7 +146,7 @@ mod tests {
             )
         );
 
-        node.hidden = true;
+        node.listed = false;
 
         assert_eq!(
             format!("{node}"),
@@ -153,7 +156,7 @@ mod tests {
                     text:15l summary:{} \
                     redirect:{redirect} \
                     links:{} \
-                    hidden\
+                    unlisted\
                     ]",
                 summary.len(),
                 node.links.len(),
diff --git a/src/router/handlers/graph.rs b/src/router/handlers/graph.rs
index bf0b112..0ebc539 100644
--- a/src/router/handlers/graph.rs
+++ b/src/router/handlers/graph.rs
@@ -93,7 +93,7 @@ mod tests {
 
     #[tokio::test]
     async fn docs_redirect() {
-        let response = wrap_node("docs").await;
+        let response = wrap_node("RedirectTest").await;
         assert_eq!(response.status(), StatusCode::PERMANENT_REDIRECT);
     }
 
diff --git a/templates/base.html b/templates/base.html
index 990f779..61043bd 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -52,8 +52,8 @@
                 <select class="node-selector" name="node">
                   <option value="">Nodes</option>
                   {% for _, node in graph.nodes %}
-                  {% if node.hidden or node.redirect %}{% continue %}{% endif %}
-                  {% if not node.hidden and not node.redirect %}
+                  {% if not node.listed or node.redirect %}{% continue %}{% endif %}
+                  {% if node.listed and not node.redirect %}
                   <option value="{{node.id}}">{{node.title}}</option>
                   {% endif %}
                   {% endfor %}
diff --git a/templates/index.html b/templates/index.html
index f2b1e75..69a0737 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -37,7 +37,7 @@
         {% if loop.index > graph.meta.config.index_node_count %}
         {% break %}
         {% endif %}
-        {% if node.id != graph.root_node and not node.hidden and not node.redirect %}
+        {% if node.id != graph.root_node and node.listed and not node.redirect %}
         <li><a href="/node/{{node.id}}">{{node.title}}</a></li>
         {% endif %}
         {% endfor %}
diff --git a/templates/node.html b/templates/node.html
index 6d19a8a..f3d9397 100644
--- a/templates/node.html
+++ b/templates/node.html
@@ -8,7 +8,7 @@
             <h1 class="node-title">{{ node.title }}</h1>
             <div class="labels">
                 {% if node.title != node.id %}<span class="label id-label">ID: {{ node.id }}</span>{% endif %}
-                {% if node.hidden %}<span class="label hidden-label">Hidden</span>{% endif %}
+                {% if not node.listed %}<span class="label unlisted-label">Unlisted</span>{% endif %}
                 {% if node.id == graph.root_node %}<span class="label root-label">Root</span>{% endif %}
             </div>
         </div>
diff --git a/templates/tree.html b/templates/tree.html
index 243bab5..3d47d40 100644
--- a/templates/tree.html
+++ b/templates/tree.html
@@ -6,7 +6,7 @@
     {% if graph.nodes or root_node %}
     <h1>Tree</h1>
     <ul>
-    {% if root_node and not root_node.hidden %}
+    {% if root_node and root_node.listed %}
     <li>
         <a href="/node/{{root_node.id}}">{{root_node.title}}</a>
         <span class="label root-label">Root</span>
@@ -34,7 +34,7 @@
     {% endif %}
     {% if graph.nodes %}
     {% for id, node in graph.nodes %}
-    {% if node.hidden or (root_node and node.id == root_node.id ) %}{% continue %}{% endif %}
+    {% if not node.listed or (root_node and node.id == root_node.id ) %}{% continue %}{% endif %}
     <li>
         <a href="/node/{{node.id}}">{{node.title}}</a>
         {% if node.connections or graph.meta.config.tree_node_summary %}