personal-projects/leo-claude-mktplace
Template
0
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(hooks): check venv cache path before marketplace path #401

Merged
lmiranda merged 1 commits from fix/startup-hook-venv-cache-path into development 2026-02-03 03:15:01 +00:00
Conversation 0 Commits 1 Files Changed 3 +51 -9
lmiranda commented 2026-02-03 03:14:40 +00:00
Owner
Copy Link

Summary

  • Fixed startup hooks in data-platform and pr-review to check venv cache path first
  • Previously hooks checked marketplace path which gets wiped on updates
  • This caused false "MCP venv missing" errors wasting hours of debugging
  • Updated docs/CANONICAL-PATHS.md with correct venv path pattern

Root Cause

Hooks checked ~/.claude/plugins/marketplaces/.../mcp-servers/.venv/ but venvs live at ~/.cache/claude-mcp-venvs/leo-claude-mktplace/ which survives updates.

Lesson Learned

Wiki: Startup Hooks Must Check Venv Cache Path First

Test Plan

  • Verified venvs exist in cache directory
  • Tested data-platform startup hook - no false "venv missing" error
  • Tested pr-review startup hook - passes silently
  • Copied fixed hooks to installed marketplace

🤖 Generated with Claude Code

## Summary - Fixed startup hooks in `data-platform` and `pr-review` to check venv cache path first - Previously hooks checked marketplace path which gets wiped on updates - This caused false "MCP venv missing" errors wasting hours of debugging - Updated `docs/CANONICAL-PATHS.md` with correct venv path pattern ## Root Cause Hooks checked `~/.claude/plugins/marketplaces/.../mcp-servers/.venv/` but venvs live at `~/.cache/claude-mcp-venvs/leo-claude-mktplace/` which survives updates. ## Lesson Learned Wiki: [Startup Hooks Must Check Venv Cache Path First](https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/wiki/lessons/patterns/startup-hooks-must-check-venv-cache-path-first) ## Test Plan - [x] Verified venvs exist in cache directory - [x] Tested data-platform startup hook - no false "venv missing" error - [x] Tested pr-review startup hook - passes silently - [x] Copied fixed hooks to installed marketplace --- 🤖 Generated with [Claude Code](https://claude.com/claude-code)
lmiranda added 1 commit 2026-02-03 03:14:41 +00:00
fix(hooks): check venv cache path before marketplace path c9e054e013 
Startup hooks in data-platform and pr-review were checking for venvs
at the marketplace path (~/.claude/plugins/marketplaces/.../mcp-servers/)
which gets wiped on updates. The actual venvs live in the cache directory
(~/.cache/claude-mcp-venvs/) which survives updates.

This caused false "MCP venv missing" errors even when venvs existed,
wasting hours of debugging time.

Fixed hooks now check cache path first, matching the pattern used
by run.sh scripts.

Also updated docs/CANONICAL-PATHS.md with the correct venv path pattern
to prevent future occurrences.

Lesson learned: lessons/patterns/startup-hooks-must-check-venv-cache-path-first

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
lmiranda merged commit 39556dbb59 into development 2026-02-03 03:15:01 +00:00
lmiranda deleted branch fix/startup-hook-venv-cache-path 2026-02-03 03:15:02 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Component/API
API endpoints, contracts, and integration
 Component/Auth
Authentication and authorization
 Component/Backend
Backend service code and business logic
 Component/Database
Database schemas, migrations, queries
 Component/Deploy
Deployment, infrastructure, DevOps
 Component/Docs
Documentation and guides
 Component/Frontend
User interface and client-side code
 Component/Infra
Infrastructure and system configuration
 Component/Testing
Test infrastructure and frameworks
 Tech/Docker
Docker containers and compose
 Tech/FastAPI
FastAPI backend framework
 Tech/JavaScript
JavaScript/Node.js code
 Tech/PostgreSQL
PostgreSQL database
 Tech/Python
Python language and libraries
 Tech/Redis
Redis cache and pub/sub
 Tech/Vue
Vue.js frontend framework
Complexity/Complex
High uncertainty or many dependencies
 Complexity/Medium
Some unknowns or dependencies
 Complexity/Simple
Straightforward, well-understood
 Effort/L
3-5 days
 Effort/M
1-2 days
 Effort/S
2-4 hours
 Effort/XL
> 1 week
 Effort/XS
< 2 hours
 Priority/Critical
Immediate attention required
 Priority/High
Important, needs attention soon
 Priority/Low
Nice to have, can wait
 Priority/Medium
Normal priority
 Type/Bug
Something isn't working
 Type/Chore
Maintenance tasks
 Type/Documentation
Documentation improvements
 Type/Feature
New feature or request
 Type/Refactor
Code refactoring without changing behavior
 Type/Test
Testing related
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
lmiranda
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: personal-projects/leo-claude-mktplace#401
Delete Branch "fix/startup-hook-venv-cache-path"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?