fix(pkgdown): remove CLAUDE.md before build instead of post-build html-only cleanup - #30
Merged
Merged
Conversation
…l-only cleanup The existing post-build "Remove internal-only pages" step (rm docs/CLAUDE.html docs/CLAUDE.md) only removed the rendered .html and its markdown copy inside docs/ -- it missed the companion artifacts pkgdown also generates for every root .md (an llms.txt-era markdown twin, a sitemap.xml entry, a full-text search.json entry), which still leaked the content. pkgdown::package_mds() hardcodes README/LICENSE/LICENCE/NEWS/cran-comments as the only skipped root .md files, so CLAUDE.md always gets rendered unless it's gone before the build runs. Move the removal to a pre-build step on the disposable CI checkout; the tracked file is untouched. Verified locally via pkgdown::build_site_github_pages() (the exact function CI calls) with CLAUDE.md removed first: confirmed docs/CLAUDE.html and docs/CLAUDE.md both absent, no build errors.
Data-Wise
added a commit
that referenced
this pull request
Aug 23, 2026
Source fix in flight (PR #30) for the CLAUDE.md pattern; additionally purging 20 other stale internal docs under dev/ (PLAN-*, SPEC-*, ISSUE-*, optimization/setup notes) from a ~2026-06 build -- these were removed from repo root long ago but never purged from gh-pages. clean: false on the deploy action means nothing self-heals without explicit removal. Part of cross-repo pkgdown-leak remediation.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Part of a cross-repo pkgdown-leak remediation (same pattern already applied to medfit
#60, probmed#36, mediationverse#22).Test plan
pkgdown::build_site_github_pages()(the exact function CI calls) withCLAUDE.mdremoved first: confirmeddocs/CLAUDE.htmlanddocs/CLAUDE.mdboth absent.CLAUDE.mdrestored viagit checkoutafter local verification (source file untouched).docs/is gitignored in this repo — no build-artifact cleanup needed.