Skip to content

fix: resolve symlinks for pagefind index #100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 19, 2025
Merged

fix: resolve symlinks for pagefind index #100

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5e18efc
fix: resolve symlinks for pagefind index
pfitzseb May 19, 2025
144fdb6
Merge branch 'main' into sp/pagefind-symlinks
pfitzseb May 19, 2025
e75cf2a
fix: `node() do` should take one argument
asinghvi17 May 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 11 additions & 5 deletions src/search/pagefind.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,22 +29,28 @@ function render()
end

function build_search_index(root, docs, config, rootpath)
# npx and npm are distributed as FileProducts,
# npx and npm are distributed as FileProducts,
# so the JLL does not bundle environment information into them.
# To fix this, we wrap all uses of npx and npm inside `node() do ...`
# which will automatically adjust the necessary environment variables.
node() do
node() do _
if !success(Cmd(`$(npx) pagefind -V`; dir = root))
@info "Installing pagefind into $root."
if !success(Cmd(`$(npm) install pagefind`; dir = root))
error("Could not install pagefind.")
end
end

pattern = "*/{$(join(config.index_versions, ","))}/**/*.{html}"

run(`$(npx) pagefind --site $(root) --glob $(pattern) --root-selector article`)

out_path = joinpath(root, "pagefind")
mktempdir() do dir
# pagefind doesn't look at symlinks, so we resolve them here:
cp(root, dir; follow_symlinks = true, force = true)
run(`$(npx) pagefind --site $(dir) --output-path $(out_path) --glob $(pattern) --root-selector article`)
end
end

return nothing
end

Expand Down
Loading