Skip to content

[codex] Fix search index generation#81

Merged
tuxmea merged 1 commit into
OpenVoxProject:masterfrom
miharp:codex/fix-search-index
Apr 23, 2026
Merged

[codex] Fix search index generation#81
tuxmea merged 1 commit into
OpenVoxProject:masterfrom
miharp:codex/fix-search-index

Conversation

@miharp
Copy link
Copy Markdown
Contributor

@miharp miharp commented Apr 23, 2026

Summary

Add a repo-level search.json so local site search indexes the actual documentation pages instead of only the homepage.

Root cause

jekyll-vitepress-theme generates /search.json from site.data.sidebar, but this repo does not define _data/sidebar.*.

This docs site uses _data/nav/*.yml and collection defaults in _config.yml, so the theme's generated search index never iterates the documentation collections.

What changed

  • Added a custom root search.json
  • Indexed the homepage, regular titled pages, and output collection documents from site.documents
  • Overrode the theme's broken default index generation for this repo structure

Validation

  • Ran bundle exec jekyll build
  • Verified _site/search.json contains 881 entries after the change
  • Verified OpenVox docs pages are present in the generated index

Closes #80

Signed-off-by: Michael Harp <mike@mikeharp.com>
@miharp miharp force-pushed the codex/fix-search-index branch from 7e77bad to 6524838 Compare April 23, 2026 12:08
@tuxmea
Copy link
Copy Markdown
Contributor

tuxmea commented Apr 23, 2026

That's great. I stumpled upon search not working by today, too.
Please undraft so I can approve. We can always improve later

@tuxmea tuxmea marked this pull request as ready for review April 23, 2026 14:50
@tuxmea tuxmea requested a review from a team as a code owner April 23, 2026 14:50
@tuxmea tuxmea merged commit d3a40bb into OpenVoxProject:master Apr 23, 2026
2 checks passed
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.

Search index only includes homepage because theme expects _data/sidebar

2 participants