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

fix: address critical issues from codebase analysis #340

Merged
lmiranda merged 1 commits from fix/critical-issues-from-analysis into development 2026-01-30 23:07:22 +00:00
Conversation 0 Commits 1 Files Changed 6 +53 -23
lmiranda commented 2026-01-30 23:07:02 +00:00
Owner
Copy Link

Summary

  • Add hooks declarations to 9 plugins missing them in marketplace.json
  • Change .mcp.json to use relative paths (portable across users)
  • Fix pr-review hook schema to use standard nested hooks structure
  • Fix token exposure in cmdb-assistant startup-check.sh (use curl -K stdin)
  • Update version to 5.4.1 in marketplace.json, README.md
  • Fix CANONICAL-PATHS.md version (was incorrectly showing 5.5.0)

Test plan

  • ./scripts/validate-marketplace.sh passes
  • ./scripts/verify-hooks.sh passes
  • All 12 plugins have hooks declared
  • Verify MCP servers still work with relative paths after update

🤖 Generated with Claude Code

## Summary - Add hooks declarations to 9 plugins missing them in marketplace.json - Change .mcp.json to use relative paths (portable across users) - Fix pr-review hook schema to use standard nested hooks structure - Fix token exposure in cmdb-assistant startup-check.sh (use curl -K stdin) - Update version to 5.4.1 in marketplace.json, README.md - Fix CANONICAL-PATHS.md version (was incorrectly showing 5.5.0) ## Test plan - [x] `./scripts/validate-marketplace.sh` passes - [x] `./scripts/verify-hooks.sh` passes - [x] All 12 plugins have hooks declared - [ ] Verify MCP servers still work with relative paths after update 🤖 Generated with [Claude Code](https://claude.ai/code)
lmiranda added 1 commit 2026-01-30 23:07:02 +00:00
fix: address critical issues from codebase analysis f082b78c0b 
- Add hooks declarations to 9 plugins missing them in marketplace.json
- Change .mcp.json to use relative paths (portable across users)
- Fix pr-review hook schema to use standard nested hooks structure
- Fix token exposure in cmdb-assistant startup-check.sh (use curl -K)
- Update version to 5.4.1 in marketplace.json, README.md
- Fix CANONICAL-PATHS.md version (was incorrectly showing 5.5.0)

All 12 plugins now have hooks properly registered.
All validations pass.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
lmiranda merged commit a78bde2e42 into development 2026-01-30 23:07:22 +00:00
lmiranda deleted branch fix/critical-issues-from-analysis 2026-01-30 23:07:22 +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#340
Delete Branch "fix/critical-issues-from-analysis"

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?