Fix enableGitInfo being silently discarded by TOML scoping

[vpetersson-bot]

You asked whether the missing dates were a shallow clone. They weren't — it's a TOML scoping bug, and the setting has never done anything.

The bug

[pagination]
  pagerSize = 12
  path = "page"

enableGitInfo = true          # <-- parsed as pagination.enableGitInfo
ignoreFiles   = [...]         # <-- parsed as pagination.ignoreFiles

TOML assigns every bare key to the table that precedes it. Both keys were being folded into [pagination] and dropped — hugo config has no root-level enablegitinfo among its 666 lines, and the resolved [pagination] contains only pagersize and path.

Ruled out the alternatives before concluding: the deploy workflow already uses fetch-depth: 0, the local clone has full history (508 commits), and security.exec.allow already permits ^git$. Building with the --enableGitInfo flag populates .GitInfo immediately from the same working tree — so only the config key was at fault.

What it was costing

• 28 FAQ pages shipping "dateModified":"0001-01-01T00:00:00Z" in live structured data
• sitemap.xml with zero <lastmod> entries — now 314
• every guide and FAQ page dateless, which is why the uploadDate work in Add required thumbnailUrl and TechArticle headline to schema #123 needed dates reconstructed from generator commits

Three follow-on corrections

Enabling it naively would have introduced new inaccuracies, so:

1. The guides TechArticle no longer takes datePublished from .Lastmod. .Lastmod is the file's latest commit — with GitInfo live it would restamp a guide as newly published on every edit. datePublished now comes only from an explicit frontmatter date; .Lastmod supplies dateModified alone.
2. The FAQ WebPage dateModified is guarded on .Lastmod.IsZero — that is the exact field that shipped the 28 zero dates, so it can't do so again if the config regresses.
3. Blog post meta and BlogPosting schema restricted to .Kind "page". /blog/ and its 7 paginated pages share Type "posts" but have no date, so they were emitting BlogPosting and article:published_time with zero dates while already carrying their own CollectionPage.

On (3): the listing's breadcrumb came from that same gated block, so restricting it removed breadcrumbs from /blog/. I caught that in the entity diff and re-emit the breadcrumb alongside the CollectionPage instead.

Verification

Production build, entity counts before vs after:

type	before	after
BlogPosting	99	91
BreadcrumbList	172	172
TechArticle	64	64
FAQPage	62	62
VideoObject	33	33
(all others)	—	unchanged

The single delta is the intended one. Also:

• zero dates site-wide: 28 → 0
• sitemap <lastmod>: 0 → 314
• invalid JSON-LD: 0, entities missing a required property: 0

bun run lint and the production build both pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FbqYj6ztzG6yautDpRWDXS