config: Skip taxonomy entries with empty keys or values

When non-taxonomy keys (e.g. disableKinds = []) are placed after
[taxonomies] in TOML, they become part of the taxonomies table.
An empty-valued entry creates a phantom taxonomy with an empty
pluralTreeKey, causing .Ancestors to loop indefinitely.

Fixes #14550

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

diff --git a/AGENTS.md b/AGENTS.md
index 8bc6860c642731c58bb414100e82abd9253333f7..42c0706b63779098a97620342e982f80833fca2c 100644 (file)
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -8,6 +8,7 @@
 * Avoid global state at (almost) all cost.
 * This is a project with a long history; assume that a similiar problem has been solved before, look hard for helper functions before creating new ones.
 * In tests, use `qt` matchers (e.g. `b.Assert(err, qt.ErrorMatches, ...)`) instead of raw `if`/`t.Fatal` checks.
+* In tests, always use the latest Hugo specification, e.g. for layouts, it's `layouts/page.html` and not `layouts/_default/single.html`, `layouts/list.html` and not `layouts/_default/list.html`
 * Brevity is good. This applies to code, comments and commit messages. Don't write a novel.
 * Use `./check.sh ./somepackage/...` when iterating.
 * Use `./check.sh` when you're done.

diff --git a/config/allconfig/alldecoders.go b/config/allconfig/alldecoders.go
index b7c55258ddaccb56eb53e82b55ecc27d423adc4c..2fb0276d2f9fed33fc08bf5acad05704e63381e7 100644 (file)
--- a/config/allconfig/alldecoders.go
+++ b/config/allconfig/alldecoders.go
@@ -335,7 +335,14 @@ var allDecoderSetups = map[string]decodeWeight{
                key: "taxonomies",
                decode: func(d decodeWeight, p decodeConfig) error {
                        if p.p.IsSet(d.key) {
-                               p.c.Taxonomies = hmaps.CleanConfigStringMapString(p.p.GetStringMapString(d.key))
+                               m := hmaps.CleanConfigStringMapString(p.p.GetStringMapString(d.key))
+                               // Remove invalid entries (e.g. non-taxonomy keys placed inside [taxonomies] in TOML).
+                               for k, v := range m {
+                                       if k == "" || v == "" {
+                                               delete(m, k)
+                                       }
+                               }
+                               p.c.Taxonomies = m
                        }
                        return nil
                },

diff --git a/hugolib/disableKinds_test.go b/hugolib/disableKinds_test.go
index 9b02dd4c5f39e322ccebf89bbb181bdbe12a2413..a063d891f3eac65196631904f697134642223737 100644 (file)
--- a/hugolib/disableKinds_test.go
+++ b/hugolib/disableKinds_test.go
@@ -334,3 +334,29 @@ title: "Page 2 nn"
        b.AssertFileExists("public/nn/p1/index.html", false)
        b.AssertFileExists("public/nn/p2/index.html", false)
 }
+
+// Issue #14550
+// disableKinds = [] after [taxonomies] is inside the taxonomies TOML table,
+// creating a phantom taxonomy that causes .Ancestors to hang.
+func TestDisableKindsEmptySliceAncestors(t *testing.T) {
+       files := `
+-- hugo.toml --
+baseURL = "http://example.com/"
+title = "Bug repro"
+[taxonomies]
+  tag = "tags"
+disableKinds = []
+-- content/posts/hello.md --
+---
+title: Hello
+tags: [demo]
+---
+Hello.
+-- layouts/page.html --
+Ancestors: {{ len .Ancestors }}|{{ range .Ancestors }}{{ .Kind }}|{{ end }}
+-- layouts/list.html --
+{{ .Title }}
+`
+       b := Test(t, files)
+       b.AssertFileContent("public/posts/hello/index.html", "Ancestors: 2|section|home|")
+}