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

fix(hooks): correct venv path in startup-check scripts #399

Merged
lmiranda merged 1 commits from fix/startup-hook-venv-paths into development 2026-02-03 02:58:10 +00:00
Conversation 0 Commits 1 Files Changed 2 +6 -2
lmiranda commented 2026-02-03 02:57:48 +00:00
Owner
Copy Link

Summary

  • Fixed data-platform and pr-review startup hooks looking for MCP venvs in wrong path
  • Hooks were looking relative to plugin directory instead of marketplace root
  • This caused false "venv missing" errors on every session start

Test plan

  • Restart Claude session
  • Verify no false "venv missing" errors appear
  • Verify MCP tools still work
## Summary - Fixed data-platform and pr-review startup hooks looking for MCP venvs in wrong path - Hooks were looking relative to plugin directory instead of marketplace root - This caused false "venv missing" errors on every session start ## Test plan - [ ] Restart Claude session - [ ] Verify no false "venv missing" errors appear - [ ] Verify MCP tools still work
lmiranda added 1 commit 2026-02-03 02:57:48 +00:00
fix(hooks): correct venv path in startup-check scripts ba1dee4553 
The startup hooks were looking for MCP venvs relative to the plugin
directory instead of the marketplace root, causing false "venv missing"
errors on every session start.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
lmiranda merged commit db8fec42f2 into development 2026-02-03 02:58:10 +00:00
lmiranda deleted branch fix/startup-hook-venv-paths 2026-02-03 02:58:10 +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#399
Delete Branch "fix/startup-hook-venv-paths"

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?