Repository navigation
chore: remove unused files and update vulnerable dependencies - #37
Merged
Merged
Conversation
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
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.
What changed
Remove the unreferenced motion review and deferred threats-page planning note, four unused Astro components, unused CSS selectors, and illustration counter bookkeeping. Retained illustration exports produce identical output; removed selectors match no elements in the built site. Page markup, wording, browser behavior, and scanner source are unchanged.
Update vulnerable npm dependencies, Brotli to 1.2.6, and the Go toolchain and digest-pinned sandbox builder to 1.26.9. Include the pinned workflow-action updates from Dependabot PR #32. Supersedes dependency PRs #29, #32, and #35 after merging; #30 and #34 have already merged.
Security and detection impact
No scanner safety boundary, detection rule, output schema, or application logic changes. Dependency and standard-library security fixes are included.
Verification
Passed locally:
GOTOOLCHAIN=go1.26.9 make check(gofmt, vet, full race-test suite)GOTOOLCHAIN=go1.26.9 go run golang.org/x/vuln/cmd/govulncheck@v1.7.0 ./...(no vulnerabilities found)npm audit --audit-level=lowinsite/(zero vulnerabilities)npm --prefix site run check,npm --prefix site run format:check,npm --prefix site run buildpython3 site/check_docs.py(16 routes, 255 search destinations, 6 artifacts)python3 -m unittest discover -s scripts -p 'test_*.py'(13 tests)GOTOOLCHAIN=go1.26.9 go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.12The whole-repository formatting check still reports pre-existing formatting drift in untouched files. Broad formatting-only changes were excluded to keep this cleanup focused. The existing deprecated clipboard-fallback hint remains. Docker and GitHub workflow execution are validated by PR CI.