From 78429a709f6f53d89116ebbc02d5de9e903a703d Mon Sep 17 00:00:00 2001 From: lmiranda Date: Sat, 7 Feb 2026 15:23:57 -0500 Subject: [PATCH] fix(marketplace): correct stale hook references and migration guide errors MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - claude-config-audit-settings.md: update hook inventory to post-Decision #29 state - maintainer.md: remove PostToolUse references, update to current hook types - settings-optimization.md: update review layer table and hooks.json format - claude-config-optimize-settings.md: fix stale doc-guardian PostToolUse reference - project-hygiene/claude-md-integration.md: rewrite for manual /hygiene check - doc-guardian: update doc-sync.md and sync-workflow.md hook references - MIGRATION-v9.md: mark deleted commands as Removed, not renamed - projman/task-sizing.md: PostToolUse → PreToolUse in example - scripts/setup.sh: /labels-sync → /labels sync - docs/CONFIGURATION.md: doc-guardian "Commands and hooks" → "Commands only" - docs/prompts/INDEX.md: add prompt execution index Version: 9.0.1 Co-Authored-By: Claude Opus 4.6 --- .claude-plugin/marketplace.json | 2 +- CHANGELOG.md | 19 ++++++++++ CLAUDE.md | 2 +- README.md | 2 +- docs/CONFIGURATION.md | 2 +- docs/MIGRATION-v9.md | 6 +-- docs/prompts/INDEX.md | 9 +++++ .../agents/maintainer.md | 4 +- .../commands/claude-config-audit-settings.md | 38 +++++++++---------- .../claude-config-optimize-settings.md | 2 +- .../skills/settings-optimization.md | 21 ++++++---- plugins/doc-guardian/commands/doc-sync.md | 2 +- plugins/doc-guardian/skills/sync-workflow.md | 2 +- .../project-hygiene/claude-md-integration.md | 32 ++++++++-------- plugins/projman/skills/task-sizing.md | 2 +- scripts/setup.sh | 4 +- 16 files changed, 91 insertions(+), 58 deletions(-) diff --git a/.claude-plugin/marketplace.json b/.claude-plugin/marketplace.json index 08f9c33..a76426c 100644 --- a/.claude-plugin/marketplace.json +++ b/.claude-plugin/marketplace.json @@ -6,7 +6,7 @@ }, "metadata": { "description": "Project management plugins with Gitea and NetBox integrations", - "version": "9.0.0" + "version": "9.0.1" }, "plugins": [ { diff --git a/CHANGELOG.md b/CHANGELOG.md index 52ee265..fd0f007 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,25 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). ## [Unreleased] +## [9.0.1] - 2026-02-06 + +### Fixed + +- **claude-config-maintainer:** `claude-config-audit-settings.md` Step 4 referenced deleted hooks.json files (doc-guardian, project-hygiene, data-platform, contract-validator) — updated to current hook inventory (code-sentinel, git-flow, cmdb-assistant, clarity-assist) +- **claude-config-maintainer:** `maintainer.md` agent referenced project-hygiene PostToolUse hooks — updated to current hook types +- **claude-config-maintainer:** `claude-config-audit-settings.md` output format referenced doc-guardian review layer — updated to git-flow, cmdb-assistant, clarity-assist +- **claude-config-maintainer:** `claude-config-audit-settings.md` Mermaid diagram referenced doc-guardian — updated to git-flow +- **claude-config-maintainer:** `claude-config-optimize-settings.md` reviewed profile prerequisites referenced doc-guardian PostToolUse — updated to git-flow PreToolUse +- **project-hygiene:** `claude-md-integration.md` described PostToolUse hook behavior that was removed in v8.1.0 — rewritten for manual `/hygiene check` command +- **doc-guardian:** `doc-sync.md` referenced doc-guardian hooks — updated to reference `/doc audit` +- **doc-guardian:** `sync-workflow.md` referenced PostToolUse hook — updated to note removal per Decision #29 +- **projman:** `task-sizing.md` example referenced PostToolUse — updated to PreToolUse +- **docs:** `MIGRATION-v9.md` listed `/pm-debug`, `/suggest-version`, `/proposal-status` as renamed to `/projman` sub-commands — corrected to show as **Removed** (these were deleted in v8.1.0, not renamed in v9.0.0) +- **docs:** `CONFIGURATION.md` listed doc-guardian as "Commands and hooks only" — corrected to "Commands only" +- **scripts:** `setup.sh` referenced old `/labels-sync` command — updated to `/labels sync` + +## [9.0.0] - 2026-02-06 + ### Added - **Phase 3: 8 new plugin scaffolds** diff --git a/CLAUDE.md b/CLAUDE.md index 229ce16..fca171a 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -154,7 +154,7 @@ When user says "fix the sprint plan command", edit the SOURCE code. ## Project Overview **Repository:** leo-claude-mktplace -**Version:** 9.0.0 +**Version:** 9.0.1 **Status:** Production Ready A plugin marketplace for Claude Code containing: diff --git a/README.md b/README.md index 7367845..82ffa3f 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,4 @@ -# Leo Claude Marketplace - v9.0.0 +# Leo Claude Marketplace - v9.0.1 A collection of Claude Code plugins for project management, infrastructure automation, and development workflows. diff --git a/docs/CONFIGURATION.md b/docs/CONFIGURATION.md index 5eb27cd..49b8843 100644 --- a/docs/CONFIGURATION.md +++ b/docs/CONFIGURATION.md @@ -485,7 +485,7 @@ Not all plugins have MCP servers. The install script handles this automatically: | projman | ✓ (via gitea) | Issue, wiki, PR tools | | pr-review | ✓ (via gitea) | PR review tools | | git-flow | ✗ | Commands only | -| doc-guardian | ✗ | Commands and hooks only | +| doc-guardian | ✗ | Commands only | | code-sentinel | ✗ | Commands and hooks only | | clarity-assist | ✗ | Commands only | diff --git a/docs/MIGRATION-v9.md b/docs/MIGRATION-v9.md index a285685..48aca73 100644 --- a/docs/MIGRATION-v9.md +++ b/docs/MIGRATION-v9.md @@ -21,10 +21,10 @@ v9.0.0 standardizes all commands to the `/ ` sub-command pattern. | `/pm-review` | `/sprint review` | Moved under `/sprint` | | `/pm-test` | `/sprint test` | Moved under `/sprint` | | `/pm-setup` | `/projman setup` | Moved under `/projman` | -| `/pm-debug` | `/projman debug` | Moved under `/projman` | +| `/pm-debug` | **Removed** | Deleted in v8.1.0 — migrated to `debug-mcp` plugin (Decision #11) | | `/labels-sync` | `/labels sync` | | -| `/suggest-version` | `/projman suggest-version` | Moved under `/projman` | -| `/proposal-status` | `/projman proposal-status` | Moved under `/projman` | +| `/suggest-version` | **Removed** | Deleted in v8.1.0 — migrated to `ops-release-manager` plugin (Decision #18) | +| `/proposal-status` | **Removed** | Deleted in v8.1.0 — absorbed into `/project status` (Decision #19) | | `/rfc ` | `/rfc ` | Unchanged | | `/project ` | `/project ` | Unchanged | | `/adr ` | `/adr ` | Unchanged | diff --git a/docs/prompts/INDEX.md b/docs/prompts/INDEX.md index e69de29..0f2c21c 100644 --- a/docs/prompts/INDEX.md +++ b/docs/prompts/INDEX.md @@ -0,0 +1,9 @@ +# Prompt Index + +Tracks execution status of structured prompts used for marketplace evolution and maintenance. + +| ID | Title | Status | Date | Version | +|----|-------|--------|------|---------| +| 001 | Phase 1 Evolution — Domain Metadata + Hook Migration | executed | 2026-02-06 | 8.0.0 → 8.1.0 | +| 002 | Phase 1b — Command Consolidation + 8 New Plugins | executed | 2026-02-06 | 8.1.0 → 9.0.0 | +| 003 | Post-Evolution Cleanup — Stale References and Migration Guide Fixes | executed | 2026-02-06 | 9.0.0 → 9.0.1 | diff --git a/plugins/claude-config-maintainer/agents/maintainer.md b/plugins/claude-config-maintainer/agents/maintainer.md index 0fa5709..8ba3e66 100644 --- a/plugins/claude-config-maintainer/agents/maintainer.md +++ b/plugins/claude-config-maintainer/agents/maintainer.md @@ -96,7 +96,7 @@ Use this mapping to identify active plugins: | `gitea` | projman | | `netbox` | cmdb-assistant | -Also check for hook-based plugins (project-hygiene uses `PostToolUse` hooks). +Also check for hook-based plugins (code-sentinel, git-flow, cmdb-assistant use `PreToolUse` safety hooks; clarity-assist uses `UserPromptSubmit` quality hook). **Step 2: Check CLAUDE.md for Plugin References** @@ -151,7 +151,7 @@ Evaluate using `skills/settings-optimization.md`: Before recommending auto-allow patterns, verify active review layers: 1. Read `plugins/*/hooks/hooks.json` for each installed plugin -2. Map hook types (PreToolUse, PostToolUse) to tool matchers (Write, Edit, Bash) +2. Map hook types (PreToolUse, UserPromptSubmit) to tool matchers (Write, Edit, MultiEdit, Bash, MCP patterns) 3. Confirm plugins are listed in `.claude-plugin/marketplace.json` 4. Only recommend auto-allow for scopes covered by ≥2 verified review layers diff --git a/plugins/claude-config-maintainer/commands/claude-config-audit-settings.md b/plugins/claude-config-maintainer/commands/claude-config-audit-settings.md index c03ce53..390de36 100644 --- a/plugins/claude-config-maintainer/commands/claude-config-audit-settings.md +++ b/plugins/claude-config-maintainer/commands/claude-config-audit-settings.md @@ -62,21 +62,20 @@ Using `settings-optimization.md` Section 3, detect: ### Step 4: Detect Active Marketplace Hooks -Read `plugins/*/hooks/hooks.json` files: +Read `plugins/*/hooks/hooks.json` files (post-Decision #29 — only PreToolUse safety hooks and UserPromptSubmit quality hooks exist): ```bash -# Check each plugin's hooks -plugins/code-sentinel/hooks/hooks.json # PreToolUse security -plugins/doc-guardian/hooks/hooks.json # PostToolUse drift detection -plugins/project-hygiene/hooks/hooks.json # PostToolUse cleanup -plugins/data-platform/hooks/hooks.json # PostToolUse schema diff -plugins/contract-validator/hooks/hooks.json # Plugin validation +# Check each plugin's hooks (exhaustive post-v8.1.0 inventory) +plugins/code-sentinel/hooks/hooks.json # PreToolUse: Write|Edit|MultiEdit → security-check.sh +plugins/git-flow/hooks/hooks.json # PreToolUse: Bash → branch-check.sh, commit-msg-check.sh +plugins/cmdb-assistant/hooks/hooks.json # PreToolUse: MCP create/update → validate-input.sh +plugins/clarity-assist/hooks/hooks.json # UserPromptSubmit → vagueness-check.sh ``` Parse each to identify: -- Hook event type (PreToolUse, PostToolUse) -- Tool matchers (Write, Edit, MultiEdit, Bash) -- Whether hook is command type (reliable) or prompt type (unreliable) +- Hook event type (PreToolUse or UserPromptSubmit only — no other types should exist) +- Tool matchers (Write, Edit, MultiEdit, Bash, MCP patterns) +- Whether hook is command type (must be — prompt type is forbidden) ### Step 5: Map Review Layers to Directory Scopes @@ -118,9 +117,9 @@ Issues Found: Active Review Layers Detected: ✓ code-sentinel (PreToolUse: Write|Edit|MultiEdit) - ✓ doc-guardian (PostToolUse: Write|Edit|MultiEdit) - ✓ project-hygiene (PostToolUse: Write|Edit) - ✗ data-platform schema-diff (not detected) + ✓ git-flow (PreToolUse: Bash — branch naming + commit format) + ✓ cmdb-assistant (PreToolUse: MCP create/update) + ✓ clarity-assist (UserPromptSubmit: vagueness detection) Recommendations: 1. [specific action with pattern] @@ -146,7 +145,7 @@ When `--diagram` is specified, generate a Mermaid flowchart showing: **Color coding:** - PreToolUse hooks: Blue -- PostToolUse hooks: Green +- UserPromptSubmit hooks: Green - Sprint Approval: Amber - PR Review: Purple @@ -161,7 +160,7 @@ flowchart LR subgraph Review Layers CS[code-sentinel] - DG[doc-guardian] + GF[git-flow] PR[pr-review] end @@ -172,18 +171,17 @@ flowchart LR end W --> CS - W --> DG E --> CS - E --> DG + B --> GF CS --> A - DG --> A + GF --> A B --> P classDef preHook fill:#e3f2fd - classDef postHook fill:#e8f5e9 + classDef userPrompt fill:#e8f5e9 classDef prReview fill:#f3e5f5 class CS preHook - class DG postHook + class GF preHook class PR prReview ``` diff --git a/plugins/claude-config-maintainer/commands/claude-config-optimize-settings.md b/plugins/claude-config-maintainer/commands/claude-config-optimize-settings.md index 6d081c8..922c843 100644 --- a/plugins/claude-config-maintainer/commands/claude-config-optimize-settings.md +++ b/plugins/claude-config-maintainer/commands/claude-config-optimize-settings.md @@ -171,7 +171,7 @@ Switching to reviewed profile... Prerequisites verified: ✓ code-sentinel hook active (PreToolUse) - ✓ doc-guardian hook active (PostToolUse) + ✓ git-flow hook active (PreToolUse) ✓ 2+ review layers detected This profile: diff --git a/plugins/claude-config-maintainer/skills/settings-optimization.md b/plugins/claude-config-maintainer/skills/settings-optimization.md index 8cd3c51..e53df48 100644 --- a/plugins/claude-config-maintainer/skills/settings-optimization.md +++ b/plugins/claude-config-maintainer/skills/settings-optimization.md @@ -351,14 +351,19 @@ To verify which review layers are active, read these files: ```json { - "hooks": [ - { - "event": "PreToolUse", - "type": "command", - "command": "./hooks/security-check.sh", - "tools": ["Write", "Edit", "MultiEdit"] - } - ] + "hooks": { + "PreToolUse": [ + { + "matcher": "Write|Edit|MultiEdit", + "hooks": [ + { + "type": "command", + "command": "./hooks/security-check.sh" + } + ] + } + ] + } } ``` diff --git a/plugins/doc-guardian/commands/doc-sync.md b/plugins/doc-guardian/commands/doc-sync.md index aad0c49..98fd713 100644 --- a/plugins/doc-guardian/commands/doc-sync.md +++ b/plugins/doc-guardian/commands/doc-sync.md @@ -5,7 +5,7 @@ description: Synchronize all pending documentation updates in a single commit # /doc sync -Apply all pending documentation updates detected by doc-guardian hooks. +Apply all pending documentation updates detected by `/doc audit`. ## Skills to Load diff --git a/plugins/doc-guardian/skills/sync-workflow.md b/plugins/doc-guardian/skills/sync-workflow.md index 17d8800..7084dff 100644 --- a/plugins/doc-guardian/skills/sync-workflow.md +++ b/plugins/doc-guardian/skills/sync-workflow.md @@ -12,7 +12,7 @@ Defines how to synchronize documentation with code changes. ## When to Use - **doc sync**: Apply pending documentation updates -- **PostToolUse hook**: Queue drift for later sync +- **doc audit**: Detect drift manually (PostToolUse hook removed per Decision #29) --- diff --git a/plugins/project-hygiene/claude-md-integration.md b/plugins/project-hygiene/claude-md-integration.md index 4efafaa..387e97d 100644 --- a/plugins/project-hygiene/claude-md-integration.md +++ b/plugins/project-hygiene/claude-md-integration.md @@ -1,14 +1,23 @@ ## Project Cleanup (project-hygiene) -This project uses the **project-hygiene** plugin for automated post-task cleanup. +This project uses the **project-hygiene** plugin for file organization and cleanup checks. ### How It Works -The plugin automatically runs after file Write or Edit operations to: +Run `/hygiene check` to scan for common project cleanliness issues: -1. **Delete temporary files** - Removes `*.tmp`, `*.bak`, `__pycache__/`, `.pytest_cache/`, etc. -2. **Warn about unexpected root files** - Alerts when files are created outside expected locations -3. **Identify orphaned files** - Detects supporting files that may no longer be needed +1. **Temp file detection** — finds `*.tmp`, `*.bak`, `*.swp`, `*~` files +2. **Misplaced files** — files outside their expected directories +3. **Empty directories** — directories with no files +4. **Large files** — files exceeding reasonable size thresholds +5. **Debug artifacts** — leftover debug logs, console.log, print statements + +### Usage + +``` +/hygiene check # Run all checks +/hygiene check --fix # Auto-fix safe issues (delete temp files, remove empty dirs) +``` ### Configuration @@ -23,14 +32,7 @@ The plugin can be configured via `.hygiene.json` in the project root: } ``` -### Hook Events +### Notes -The plugin registers on the following events: -- `PostToolUse` (matcher: `Write|Edit`) - Runs cleanup after file modifications - -### Usage Guidelines - -- Let the hook run automatically - no manual intervention needed -- Review warnings about unexpected root files -- Configure `.hygiene.json` to customize cleanup behavior for your project -- Check cleanup output if files seem to disappear unexpectedly +- This was previously a PostToolUse hook (automatic). Since v8.1.0 (Decision #29), it runs manually via `/hygiene check`. +- Add `/hygiene check` as an explicit step in your prompt files where project cleanliness matters. diff --git a/plugins/projman/skills/task-sizing.md b/plugins/projman/skills/task-sizing.md index 3c06ce5..0f9a643 100644 --- a/plugins/projman/skills/task-sizing.md +++ b/plugins/projman/skills/task-sizing.md @@ -82,7 +82,7 @@ Labels: Efforts/S [Sprint 3] chore: Register hook in hooks.json Labels: Efforts/XS -- [ ] Add PostToolUse:Edit hook entry +- [ ] Add PreToolUse:Edit hook entry ``` --- diff --git a/scripts/setup.sh b/scripts/setup.sh index 7b0ebf9..16140e0 100755 --- a/scripts/setup.sh +++ b/scripts/setup.sh @@ -219,10 +219,10 @@ setup_labels() { if [[ -f "$labels_file" ]]; then log_success "Label reference file exists" - log_info "Run '/labels-sync' command after setup to sync with Gitea" + log_info "Run '/labels sync' command after setup to sync with Gitea" else log_error "Label reference file not found at $labels_file" - log_todo "Run '/labels-sync' to create label reference from Gitea" + log_todo "Run '/labels sync' to create label reference from Gitea" fi } -- 2.49.1