Skip to content

fix(ci): plugin-validate cache fix + msft-365 README update#239

Merged
mihilmy merged 2 commits into
mainfrom
fix/plugin-validate-cli-cache
May 19, 2026
Merged

fix(ci): plugin-validate cache fix + msft-365 README update#239
mihilmy merged 2 commits into
mainfrom
fix/plugin-validate-cli-cache

Conversation

@mihilmy
Copy link
Copy Markdown
Collaborator

@mihilmy mihilmy commented May 18, 2026

CI fix — plugin-validate cache restored a dangling symlink

plugin-validate failed on a cache hit with exit code 127.

claude.ai/install.sh installs ~/.local/bin/claude as a symlink into ~/.local/share/claude/versions/<version>/. The cache step only saved ~/.local/bin/claude, so on a cache hit the symlink was restored but its target wasn't — a dangling link → claude: command not found. The install step is gated on cache-hit != 'true', so it was correctly skipped; the restored cache was just unusable.

Fix: cache ~/.local/share/claude (the versioned binary) alongside the symlink, and bump the cache key to -v2 so the already-saved broken entry is discarded. Verified green on this PR's own run.

docs — claude-for-msft-365-install README

Folded in from #238 (was blocked by the broken workflow above):

  • Update section — claude plugin update … so admins can pull new plugin versions.
  • Bootstrap section — install + /…:bootstrap for per-user MCP servers, skills, dynamic config.
  • Added the existing debug command to the commands table (implemented but undocumented).

mihilmy added 2 commits May 18, 2026 17:01
~/.local/bin/claude is a symlink into ~/.local/share/claude/versions/<v>.
Caching only the symlink restored a dangling link on a cache hit, so
`claude` failed with exit 127. Cache the versioned dir too and bump the
cache key (-v2) to discard the broken saved entry.
@mihilmy mihilmy changed the title fix(ci): plugin-validate cache restored a dangling claude symlink fix(ci): plugin-validate cache fix + msft-365 README update May 18, 2026
@mihilmy mihilmy enabled auto-merge (squash) May 19, 2026 01:53
@mihilmy mihilmy merged commit 3edda1c into main May 19, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants