Skip to content

fix: normalize Hugo page bundles - #311

Merged
kpango merged 2 commits into
mainfrom
codex/fix-hugo-page-bundles
Sep 9, 2026
Merged

fix: normalize Hugo page bundles#311
kpango merged 2 commits into
mainfrom
codex/fix-hugo-page-bundles

Conversation

@Matts966

@Matts966 Matts966 commented Sep 8, 2026

Copy link
Copy Markdown
Member

Summary

  • normalize source layouts where foo.md and foo/ coexist into the Hugo branch bundle foo/_index.md
  • rewrite links from the moved overview page to its child pages
  • fail instead of overwriting an existing bundle index
  • propagate archive download and extraction errors
  • add regression tests for bundle conversion, standalone pages, and index conflicts

Why

Vald v1.8.0 added docs/user-guides/vald-operator.md together with detailed pages under docs/user-guides/vald-operator/. The web sync currently generates both vald-operator.md and vald-operator/_index.md, which resolve to the same Hugo URL. Depending on archive traversal order, the conflict was either logged and ignored or caused make all to fail.

This change normalizes that source layout immediately after archive extraction, before Hugo content generation begins.

Failed workflow: https://github.com/vdaas/web/actions/runs/32932596482

Validation

  • go test -race -tags=mage ./magefile
  • synced the current Vald main archive and confirmed only vald-operator/_index.md plus its child pages remain
  • synced the Vald v1.8.0 archive and confirmed the same normalized layout
  • confirmed overview links resolve to design.md, controller-spec.md, resource-and-topology.md, and test-strategy.md within the bundle

Matts966 and others added 2 commits September 8, 2026 12:16
Signed-off-by: Matts966 <matts966@vdaas.org>
Signed-off-by: vdaas-ci <ci@vdaas.org>
@kpango
kpango merged commit 61227bd into main Sep 9, 2026
4 checks passed
@kpango
kpango deleted the codex/fix-hugo-page-bundles branch September 9, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants