Repository navigation
fix(zsh): add ~/.local/bin to PATH in .zshenv for uv tool shims - #513
Merged
Merged
Conversation
uv installs tool shims into ~/.local/bin (radian, arxiv_latex_cleaner), but nothing in the zsh config put that directory on PATH, so the tools were installed yet unreachable by name. Placed in .zshenv rather than .zshrc so non-interactive shells (scripts, Claude Code) resolve them too — matching the file's stated purpose of holding essential PATH modifications. Guarded with a substring test because .zshenv is sourced for every zsh invocation; an unconditional prepend would grow PATH once per nested shell. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Data-Wise
added a commit
that referenced
this pull request
Sep 5, 2026
Version bump for the 5 commits merged to dev since 7.17.1: 2 fixes (#513 PATH, #499 tap-push GH006), 3 CI/deps changes (#512, #514, #515). No feat commits -- patch bump. - package.json, package-lock.json, flow.plugin.zsh (FLOW_VERSION), CLAUDE.md, man/man1/*.1 (.TH version line) via scripts/release.sh - CHANGELOG.md + docs/CHANGELOG.md: new 7.17.2 entry (mirrored, per the project's dual-changelog convention) - docs/index.md: footer version stamp only. The "What's New in v7.17.1" banner is left as-is -- it describes actual 7.17.1 user-facing features, and 7.17.2 has none to add (pure CI/infra). Test-run side effects (.STATUS wins/streak counters, a test fixture's timestamp) surfaced as unrelated dirty files from running ./tests/run-all.sh and were excluded from this commit.
Merged
Data-Wise
added a commit
that referenced
this pull request
Sep 5, 2026
.STATUS had not been touched since 2026-07-07 (f740eb5): the header still read Phase: Released (v7.16.0) while v7.17.0, v7.17.1 and v7.17.2 had all shipped, and Focus: held the literal string "--help" -- a bad write, not a focus. - Header: Phase -> Released (v7.17.2); Focus -> a real one-line focus. - Added four session entries reconstructed from CHANGELOG.md, the merged PR bodies (#505, #506, #509, #513, #499, #514, #515) and the release tags: v7.17.0 (teach deploy safety + CI coverage), v7.17.1 (alias-proof pipelines), v7.17.2 (CI hardening + PATH fix), plus the 2026-07-08..07-19 housekeeping commits and a session entry for this resync itself. - Demoted the stale "Current Session (2026-07-07)" and two older headings that had been left as "Current Session" against the file's own convention. - Test baseline refreshed from evidence rather than carried forward: 84 passed / 0 failed / 0 timeout / 1 skipped across 85 registered suites, per CI run 33946019980 on dev. - Next Action re-verified against repo state: added stale-issue triage (#487/#488/#489), the PR #500 rebase decision, .gitignore for .token-optimizer, and the advisory markdown-lint backlog. Verified: scripts/check-status.zsh clean on the updated file (validator confirmed non-vacuous via a planted "Progress: banana" defect, which it caught); _flow_status_field reads all four header fields correctly. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
Problem
uvinstalls tool shims into~/.local/bin(currentlyradianandarxiv_latex_cleaner), but nothing in the zsh config put that directory onPATH. The tools were installed yet unreachable by name —uvitself warns about this on every install.Surfaced while restoring a new Mac:
radian(the R console) is listed in the captured machine state and is load-bearing, but would not resolve.Change
One guarded line in
zsh/.zshenv..zshenv, not.zshrc— so non-interactive shells (scripts, Claude Code) resolve the tools too. This matches the file's own stated purpose: "Use this for: Essential PATH modifications"..zshenvis sourced for every zsh invocation, so an unconditional prepend would growPATHonce per nested shell.Verification
zsh -n zsh/.zshenvparses~/.local/binpresent onPATHcommand -v radianandcommand -v arxiv_latex_cleanerboth resolveTest suite
zsh/tests/run-all-tests.zsh— 2 passed, 1 failed.The failure is pre-existing and unrelated: the duplicate-function check flags
_log_error()defined in bothlib/hooks/pre-commit-template.zsh:39andlib/hooks/pre-push-template.zsh:37. Confirmed by stashing this change and re-running — identical failure on a clean tree. Not addressed here; worth its own issue.