fix: uv lock release sync and local skill install - #147
Merged
Conversation
The 0.8.2 bump (#146) updated pyproject.toml, __init__.py, the cloud function requirements, and npm/package.json, but left uv.lock pinning the editable parallel-web-tools entry at 0.8.1 — the same slip the 0.8.1 release made (fixed by hand in #145). Any uv sync regenerates the entry, so the lock has been showing as dirty in working trees since the release. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Every release bump has left uv.lock stale on main: release.sh rewrites pyproject.toml but never re-locks, so the lock's own-package entry keeps the previous version and the first uv sync in any working tree dirties it. It happened at 0.8.1, was hand-patched in #145, and happened again at 0.8.2. Run uv lock as part of update_version_files and stage uv.lock in the release commit. Add a `uv lock --check` step to the CI lint job so a stale lock fails the release PR instead of surfacing later — `uv sync` alone silently re-locks in the runner and can never catch it. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
`skills install --project` only wrote `<root>/.claude/skills` when `<root>/.claude` already existed. A repo that has never stored Claude settings has no `.claude` directory, so on exactly the machines that do run Claude Code, project installs landed only in `.agents/skills` — a location Claude Code never reads — and the skills were silently invisible. A missing `<root>/.claude` doesn't mean the user isn't running Claude Code there; a missing machine-level configuration directory does. Project installs now create `<root>/.claude/skills` whenever the machine-level Claude config dir (CLAUDE_CONFIG_DIR or ~/.claude) exists. The mirror stays project-local — the global config dir is only consulted as the Claude-Code-is-present signal, never as a target. Global installs are unchanged. Also refresh the --project help strings, which still described the pre-#145 single-directory behavior. Co-Authored-By: Claude Fable 5 <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.
skill install --projectnow install.claude/skillsif Claude is detected. Previously this only works for the globalskill install