fix: install lychee in post-code and post-fix scripts - #831
Merged
Merged
Conversation
Agent runs were failing when pre-commit hooks tried to run lychee for
markdown link checking. The lychee binary wasn't available on GitHub
Actions runners, causing the authoritative pre-commit check to fail.
Changes:
- Add lychee v0.24.2 installation to post-code.sh and post-fix.sh
- Download and verify lychee binary before running pre-commit hooks
- Follow same pattern as existing gitleaks installation
- Extract to ${HOME}/.local/bin and add to PATH
This ensures the lint-md-links pre-commit hook (added in a recent commit)
can execute successfully.
Related: #830 (documents broader security issue with pre-commit hooks
running outside sandbox)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
fullsend review is working on this — view logs |
Site previewPreview: https://f6d75d36-site.fullsend-ai.workers.dev Commit: |
Review: #831Head SHA: 81412ac SummaryThis PR adds lychee v0.24.2 installation to FindingsInfo
FooterOutcome: approve |
![fullsend-ai-review[bot]](https://avatars.githubusercontent.com/in/3478501?s=60&v=4){kind=small}
ascerra
approved these changes
May 12, 2026
This was referenced May 12, 2026
This was referenced May 12, 2026
ben-alkov
pushed a commit
to ben-alkov/fullsend
that referenced
this pull request
May 15, 2026
Agent runs were failing when the ty pre-commit hook tried to run type
checking via uvx. The uvx binary wasn't available on GitHub Actions
runners, causing authoritative pre-commit checks to fail with
"Executable 'uvx' not found".
Changes:
- Add UV_VERSION (0.11.14) configuration constant to both scripts
- Install uv/uvx before running pre-commit hooks
- Download and verify uv tarball with SHA256 checksum
- Extract both uv and uvx binaries to ${HOME}/.local/bin
- Add to PATH for pre-commit hook execution
- Follow same pattern as existing gitleaks and lychee installations
- Update section numbering in both scripts (5→6, 6→7, 7→8)
This completes the tooling setup that PR fullsend-ai#831 started by adding lychee.
Both lychee (markdown link checking) and uvx (Python tooling via ty hook)
are now installed before pre-commit runs, preventing agent failures.
Related: fullsend-ai#830 (pre-commit security), fullsend-ai#100 (ty hook configuration)
Builds on: fullsend-ai#831 (lychee installation), fullsend-ai#783 (markdown link linter)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2 tasks
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
2 participants
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
Agent runs were failing when pre-commit hooks tried to run lychee for markdown link checking. The lychee binary wasn't available on GitHub Actions runners, causing the authoritative pre-commit check to fail.
Changes
post-code.shandpost-fix.sh${HOME}/.local/binand add to PATHThis ensures the
lint-md-linkspre-commit hook can execute successfully.Test plan
Related
🤖 Generated with Claude Code