---

name: plugin-doctor description: Diagnose and fix quality issues in marketplace components with automated safe fixes and prompted risky fixes user-invocable: true allowed-tools: Read, Edit, Write, Bash, AskUserQuestion, Glob, Grep, Skill

Plugin Doctor Skill

Comprehensive diagnostic and fix skill for marketplace components. Combines diagnosis, automated safe fixes, prompted risky fixes, and verification into a single workflow.

Purpose

Provides unified doctor workflows following the pattern: Diagnose → Auto-Fix Safe → Prompt Risky → Verify

Workflow Decision Tree

MANDATORY: Select workflow based on input and execute IMMEDIATELY.

If scope = "agents" or agent-name specified

→ EXECUTE Workflow 1: doctor-agents (jump to that section)

If scope = "commands" or command-name specified

→ EXECUTE Workflow 2: doctor-commands (jump to that section)

If scope = "skills" or skill-name specified

→ EXECUTE Workflow 3: doctor-skills (jump to that section)

If scope = "metadata"

→ EXECUTE Workflow 4: doctor-metadata (jump to that section)

If scope = "scripts" or script-name specified

→ EXECUTE Workflow 5: doctor-scripts (jump to that section)

If scope = "marketplace" (full marketplace health check)

→ EXECUTE Workflow 7: doctor-marketplace (jump to that section)

If scope = "skill-content" or skill-path specified with content analysis

→ EXECUTE Workflow 6: doctor-skill-content (jump to that section)

---

7 Doctor Workflows:

1. doctor-agents: Analyze and fix agent issues
2. doctor-commands: Analyze and fix command issues
3. doctor-skills: Analyze and fix skill issues
4. doctor-metadata: Analyze and fix plugin.json issues
5. doctor-scripts: Analyze and fix script issues
6. doctor-skill-content: Analyze and reorganize skill content files
7. doctor-marketplace: Full marketplace batch analysis with report

Each workflow performs the complete cycle: discover → analyze → categorize → fix → verify.

Progressive Disclosure Strategy

Load ONE reference guide per workflow (not all 10):

| Workflow | Diagnosis Reference | Fix Reference | |----------|---------------------|---------------| | doctor-agents | agents-guide.md | fix-catalog.md | | doctor-commands | commands-guide.md | fix-catalog.md | | doctor-skills | skills-guide.md | fix-catalog.md | | doctor-metadata | metadata-guide.md | fix-catalog.md | | doctor-scripts | scripts-guide.md | fix-catalog.md |

Context Efficiency: ~800 lines per workflow vs ~4,000 lines if loading everything.

Common Workflow Pattern

All 5 workflows follow the same pattern:

Phase 1: Discover and Analyze

1. MANDATORY - Load Prerequisites

   EXECUTE these skill loads before proceeding:

   Skill: plan-marshall:ref-development-standards
   Skill: pm-plugin-development:plugin-architecture
   Skill: pm-plugin-development:tools-marketplace-inventory
   

2. MANDATORY - Load Component Reference (progressive disclosure)

   READ: references/{component}-guide.md

3. Discover Components (based on scope parameter)

   • marketplace scope: Use marketplace-inventory
   • global scope: Glob ~/.claude/{component}/
   • project scope: Glob .claude/{component}/

4. MANDATORY - Analyze Components (using doctor-marketplace)

   Use the batch analyze command with appropriate filters:

   EXECUTE:

   python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze \
     --bundles {bundle} --type {component_type}
   

   This performs markdown analysis, coverage extraction, and reference validation for all matching components. The output includes per-component analysis results in JSON format.

Phase 2: Categorize Issues

Safe Fixes (auto-apply):

• Missing frontmatter fields
• Invalid YAML syntax
• Unused tools in frontmatter
• Trailing whitespace
• Missing blank lines
• Missing foundation skill loading (plugin-architecture, diagnostic-patterns)
• Incorrect section header case (e.g., ## Workflow → ## WORKFLOW)
• Missing CONTINUOUS IMPROVEMENT RULE section (commands only)
• Legacy CONTINUOUS IMPROVEMENT RULE (uses /plugin-update-* or /plugin-maintain instead of manage-lessons)

Risky Fixes (require confirmation):

• Rule 6 violations (Task tool in agents)
• Rule 7 violations (Direct Maven usage - should use builder-maven skill)
• Rule 8 violations (Hardcoded script paths - should use script-runner)
• Rule 9 violations (Missing explicit script calls in workflows)
• Pattern 22 violations (self-invocation)
• Structural changes
• Content removal

Phase 3: Apply Fixes

1. Auto-Apply Safe Fixes (NO PROMPT)

   CRITICAL: Safe fixes are applied automatically WITHOUT user confirmation. Do NOT use AskUserQuestion for safe fixes.

   • Apply each safe fix immediately using Edit tool
   • Track success/failure
   • Log: "✅ Fixed: {description}"

2. Prompt for Risky Fixes ONLY

   AskUserQuestion:
     question: "Apply fix for {issue}?"
     options:
       - label: "Yes" description: "Apply this fix"
       - label: "No" description: "Skip this fix"
       - label: "Skip All" description: "Skip remaining risky fixes"
   

   Only risky fixes prompt - safe fixes never prompt.

Phase 4: Verify and Report

1. Verify Fixes

   Re-run analysis to verify fixes resolved issues:

   python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze \
     --bundles {bundle} --type {component_type}
   

   Compare issue counts before and after to verify resolution.

2. Generate Summary

   Read references/reporting-templates.md
   

   Use summary template with metrics.

---

Workflow 1: doctor-agents

Parameters

• scope (optional, default: "marketplace"): "marketplace" | "global" | "project"
• agent-name (optional): Analyze specific agent
• --no-fix (optional): Diagnosis only, no fixes

Step 1: Load Prerequisites and Standards

Skill: plan-marshall:ref-development-standards
Skill: pm-plugin-development:plugin-architecture
Read references/agents-guide.md
Read references/fix-catalog.md

Step 2: Discover Agents

marketplace scope (default):

Skill: pm-plugin-development:tools-marketplace-inventory

global/project scope:

Glob: pattern="*.md", path="{scope_path}/agents"

Step 3: Analyze Agents

Use the batch analyze command filtered to agents:

python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze \
  --bundles {bundle} --type agents

This returns JSON with per-agent analysis including markdown structure, tool coverage, and reference validation.

Check against agents-guide.md:

• Tool fit score >= 70% (good) or >= 90% (excellent)
• No Rule 6 violations (agents CANNOT use Task tool)
• No Rule 7 violations (only maven-builder can use Maven)
• No Pattern 22 violations (must use manage-lessons skill, not self-invoke)
• Bloat thresholds (component-type specific):
  • Agents: NORMAL (<300), LARGE (300-500), BLOATED (500-800), CRITICAL (>800)
  • Commands: NORMAL (<100), LARGE (100-150), BLOATED (150-200), CRITICAL (>200)
  • Skills: NORMAL (<400), LARGE (400-800), BLOATED (800-1200), CRITICAL (>1200)

Step 4: Categorize and Fix

Safe fixes (auto-apply unless --no-fix):

• Missing frontmatter fields
• Unused tools in frontmatter
• Invalid YAML syntax

Risky fixes (always prompt):

• Rule 6 violations (requires architectural refactoring)
• Rule 7 violations (Maven usage restriction)
• Pattern 22 violations (self-invocation)
• Bloat issues (agents >500 lines)

Step 5: Verify and Report

git status --short

Display summary using reporting-templates.md format.

---

Workflow 2: doctor-commands

Parameters

• scope (optional, default: "marketplace"): "marketplace" | "global" | "project"
• command-name (optional): Analyze specific command
• --no-fix (optional): Diagnosis only, no fixes

Step 1: Load Prerequisites and Standards

Skill: plan-marshall:ref-development-standards
Skill: pm-plugin-development:plugin-architecture
Read references/commands-guide.md
Read references/fix-catalog.md

Step 2: Discover Commands

Same pattern as doctor-agents.

Step 3: Analyze Commands

Use the batch analyze command filtered to commands:

python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze \
  --bundles {bundle} --type commands

This returns JSON with per-command analysis including markdown structure and reference validation.

Check against commands-guide.md:

Rule 0 - Thin Wrapper Check (CRITICAL):

• Line count thresholds:
  • IDEAL: < 100 lines (proper thin wrapper)
  • ACCEPTABLE: 100-150 lines (minor workflow logic OK)
  • BLOATED: 150-200 lines (too much logic, needs refactoring)
  • CRITICAL: > 200 lines (severe bloat, MUST refactor immediately)
• Commands MUST delegate to skills (check for Skill: invocation)
• Commands should NOT contain:
  • Step-by-step workflow logic (### Step 1, ### Step 2, etc. with implementation)
  • For loops or iteration logic
  • File processing code
  • Analysis algorithms
• Flag as CRITICAL if command contains workflow implementation instead of skill delegation

Other Checks:

• Verify proper Skill invocation format (Skill: bundle:skill-name)
• Check parameter documentation (PARAMETERS section exists)
• Foundation skill loading via invoked skills (see below)

Step 3b: Verify Foundation Skills in Invoked Skills

Commands are thin orchestrators - they delegate to skills via Skill: invocation.

Check criteria:

1. Extract skill invocations from command (e.g., Skill: pm-plugin-development:plugin-create)
2. For each invoked skill, verify it loads foundation skills (plugin-architecture, diagnostic-patterns)
3. Report if invoked skill is missing foundation skills (fix the skill, not the command)

If invoked skill missing foundation skills:

• Report: "Command invokes skill '{skill}' which is missing foundation skill loading"
• Recommendation: "Run /plugin-doctor skill-name={skill} to fix the skill"

This is NOT a command fix - it's a skill fix. Commands don't load foundation skills directly; their skills do.

Step 4: Categorize and Fix

Safe fixes (auto-apply unless --no-fix):

• Incorrect section header case (## Workflow → ## WORKFLOW, ## Parameter Validation → ## PARAMETERS)
• Missing CONTINUOUS IMPROVEMENT RULE section
• Legacy CONTINUOUS IMPROVEMENT RULE (uses /plugin-update-* or /plugin-maintain instead of manage-lessons)

Risky fixes (require confirmation):

• Rule 0 violations (command contains workflow logic instead of skill delegation)
  • Severity: CRITICAL if > 200 lines or contains implementation logic
  • Fix: Refactor command to thin wrapper, move logic to skill
  • This is architectural refactoring - requires manual intervention

Auto-fix pattern for section headers: Search for ## Workflow, ## Parameter Validation, ## Parameters and replace with uppercase versions.

Auto-fix pattern for missing CONTINUOUS IMPROVEMENT RULE:

## CONTINUOUS IMPROVEMENT RULE

If you discover issues or improvements during execution, record them:

1. **Activate skill**: \`Skill: plan-marshall:manage-lessons\`
2. **Record lesson** with:
   - Component: \`{type: "command", name: "{command-name}", bundle: "{bundle}"}\`
   - Category: bug | improvement | pattern | anti-pattern
   - Summary and detail of the finding

Insert before ## Related section (or at end if no Related section).

Auto-fix pattern for legacy CONTINUOUS IMPROVEMENT RULE: If CONTINUOUS IMPROVEMENT RULE section contains /plugin-update-command, /plugin-maintain, or /plugin-apply-lessons-learned, replace entire section with the new pattern above.

Rule 0 violation reporting (no auto-fix - requires manual refactoring):

⚠️ CRITICAL: Command '{command}' violates Rule 0 (thin wrapper requirement)
   - Line count: {lines} (threshold: 200)
   - Contains workflow implementation instead of skill delegation
   - Required action: Refactor to thin wrapper pattern

   Refactoring steps:
   1. Create/identify skill to contain workflow logic
   2. Move all ### Step sections to skill's workflow
   3. Replace command content with skill invocation
   4. Target: < 100 lines (parameters + skill invocation + examples)

   Example thin wrapper:
   ## Workflow
   Activate \`bundle:skill-name\` and execute the **Workflow Name** workflow.

Step 5: Verify and Report

Same pattern as doctor-agents with command-specific thresholds.

---

Workflow 3: doctor-skills

Parameters

• scope (optional, default: "marketplace"): "marketplace" | "global" | "project"
• skill-name (optional): Analyze specific skill
• --no-fix (optional): Diagnosis only, no fixes

Step 1: Load Prerequisites and Standards

Skill: plan-marshall:ref-development-standards
Skill: pm-plugin-development:plugin-architecture
Read references/skills-guide.md
Read references/fix-catalog.md

Step 2: Discover Skills

marketplace scope:

Skill: pm-plugin-development:tools-marketplace-inventory

Step 3: Analyze Skills

Use the batch analyze command filtered to skills:

python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze \
  --bundles {bundle} --type skills

This returns JSON with per-skill analysis including structure validation, markdown analysis, and reference checking.

Check against skills-guide.md:

• Structure score >= 70 (good) or >= 90 (excellent)
• Progressive disclosure compliance
• Relative path usage
• No missing referenced files
• No unreferenced files
• Foundation skill loading (see below)
• Rule 9 compliance (explicit script calls in workflows)

Step 3b: Validate Foundation Skill Loading

CRITICAL: Skills with workflows MUST load foundation skills.

Required foundation skills:

Skill: pm-plugin-development:plugin-architecture
Skill: plan-marshall:ref-development-standards

Check criteria:

1. Search SKILL.md for Skill: pm-plugin-development:plugin-architecture
2. Search SKILL.md for Skill: plan-marshall:ref-development-standards
3. Exempt skills (skip check):
   • plugin-architecture (is itself the architecture skill)
   • marketplace-inventory (pure Pattern 1 script automation, no component operations)
   • Skills with allowed-tools: Read only (pure reference libraries)

If missing: Flag as safe fix (auto-apply).

Step 3c: Validate Rule 9 - Explicit Script Calls

CRITICAL: Workflow steps that perform script operations MUST have explicit bash code blocks.

Detection logic:

1. Find workflow steps (### Step N: ...)
2. For each step, check if it contains action verbs: "read", "write", "display", "check", "validate", "get", "list", "create", "update", "delete"
3. If action verb present WITHOUT a bash code block containing execute-script.py, flag as Rule 9 violation

Violations examples:

• "Display the solution outline for review" (no bash block)
• "Read the config to get domains" (no bash block)
• "Validate the output" (no bash block)

Exempt patterns:

• Steps that use Task: (agent delegation)
• Steps that use Skill: (skill loading)
• Steps that use Read: or Glob: (Claude Code tools)
• Steps with explicit bash blocks containing execute-script.py

If violation found: Flag as risky fix (requires manual intervention to add proper script call).

Step 3d: Validate plan-marshall-plugin Manifest

Conditional: Only execute if skill name is plan-marshall-plugin.

Load reference:

Read references/plan-marshall-plugin-validation.md

Validation:

1. Extract bundle name from skill path: marketplace/bundles/{bundle}/skills/plan-marshall-plugin
2. Run manifest validation:

   python3 .plan/execute-script.py plan-marshall:domain-extension-api:validate_manifest validate \
     --bundle {bundle}
   

3. Parse validation output for issues
4. Add findings to issue list with appropriate fix categories

Issue categorization:

• Schema/structure issues → Safe fix
• Missing extension skills → Risky fix
• Invalid skill references → Risky fix

Step 4: Categorize and Fix

Safe fixes (auto-apply unless --no-fix):

• Missing frontmatter fields
• Unused tools in frontmatter
• Invalid YAML syntax
• Missing foundation skill loading (add Step 0 to each workflow)

Risky fixes (require confirmation):

• Rule 9 violations (missing explicit script calls in workflows)

Auto-fix pattern for missing foundation skills:

#### Step 0: Load Foundation Skills

\`\`\`
Skill: pm-plugin-development:plugin-architecture
Skill: plan-marshall:ref-development-standards
\`\`\`

These provide architecture principles and non-prompting tool usage patterns.

Insert this before the first step of each workflow section (after ### Steps line).

Step 5: Verify and Report

Same pattern with skill-specific checks.

---

Workflow 4: doctor-metadata

Parameters

• scope (optional, default: "marketplace"): "marketplace" | "global" | "project"
• --no-fix (optional): Diagnosis only, no fixes

Step 1: Load Prerequisites and Standards

Skill: plan-marshall:ref-development-standards
Read references/metadata-guide.md
Read references/fix-catalog.md

Step 2: Discover plugin.json Files

Glob: pattern="**/plugin.json", path="marketplace/bundles"

Step 3: Analyze Each plugin.json

• Verify JSON syntax
• Check required fields (name, version, description)
• Validate component arrays (commands, skills, agents)
• Cross-check declared components vs actual files

Step 4-5: Categorize, Fix, Verify, Report

Safe fixes:

• Missing required fields
• Extra entries (files don't exist)
• Missing entries (files exist but not listed)

---

Workflow 5: doctor-scripts

Parameters

• scope (optional, default: "marketplace"): "marketplace" | "global" | "project"
• script-name (optional): Analyze specific script
• --no-fix (optional): Diagnosis only, no fixes

Step 1: Load Prerequisites and Standards

Skill: plan-marshall:ref-development-standards
Skill: pm-plugin-development:plugin-architecture
Skill: pm-plugin-development:plugin-script-architecture
Read references/fix-catalog.md

Step 2: Discover Scripts

Glob: pattern="scripts/*.{sh,py}", path="marketplace/bundles/*/skills"

Step 3: Analyze Each Script

• Verify SKILL.md documentation
• Check test file exists
• Verify --help output
• Check stdlib-only compliance

Step 4-5: Categorize, Fix, Verify, Report

Same pattern with script-specific checks.

---

Workflow 6: doctor-skill-content

Comprehensive analysis and refactoring of skill subdirectory content (references/, workflows/, templates/).

Parameters

• skill-path (required): Path to skill directory
• --no-fix (optional): Analysis only, no reorganization
• --skip-quality (optional): Skip Phase 3 quality analysis

Overview

This workflow analyzes all markdown files within a skill's subdirectories for proper organization, quality, and consistency. It uses LLM-based semantic analysis for classification and quality assessment.

Phases:

1. Inventory - Discover all files (SCRIPT)
2. Classify - Categorize each file as reference/workflow/template (LLM)
3. Analyze - Content quality analysis (LLM)
4. Reorganize - Move files to correct directories (LLM + Bash)
5. Verify - Link verification (SCRIPT)
6. Report - Generate findings report (LLM)

Step 1: Load Prerequisites

Skill: plan-marshall:ref-development-standards
Skill: pm-plugin-development:plugin-architecture
Read references/content-classification-guide.md
Read references/content-quality-guide.md

Step 2: Phase 1 - Inventory (LLM)

Use Glob and Read tools to inventory skill content:

# List all markdown files in skill subdirectories
Glob: pattern="**/*.md", path={skill_path}

For each file, collect:

• File path and directory
• Line count (via Read tool)
• Extension statistics

Step 3: Phase 2 - Classify (LLM)

For each .md file discovered in subdirectories:

1. Read file content
2. Apply classification criteria from content-classification-guide.md
3. Determine category: reference | workflow | template | mixed
4. Record confidence level: high | medium | low

Classification Output (for each file):

File: {relative_path}
Classification: {category}
Confidence: {level}
Reasoning:
  - {observation 1}
  - {observation 2}
Current Location: {directory}
Correct Location: {references/|workflows/|templates/}
Needs Move: {yes|no}
Needs Splitting: {yes|no}

Step 4: Phase 3 - Analyze Quality (LLM-Hybrid)

Skip if --skip-quality specified.

Step 4a: Cross-File Analysis (LLM)

Perform cross-file analysis by reading and comparing content:

1. Read all markdown files in skill subdirectories
2. Compute content hashes to detect exact duplicates
3. Compare section headers and content blocks for similarity
4. Identify extraction candidates (repeated patterns)

Produce analysis with:

• exact_duplicates: Report directly (no LLM needed - hash-verified matches)
• similarity_candidates: Queue for LLM semantic analysis
• extraction_candidates: Queue for LLM extraction recommendations
• terminology_variants: Queue for LLM consistency analysis

Step 4b: LLM Semantic Analysis

For each similarity_candidate (40-95% similarity):

1. Read both content blocks from file locations
2. Classify: true_duplicate | similar_concept | false_positive
3. Recommend: consolidate | cross_reference | keep_both

For each extraction_candidate:

1. Review detected patterns (placeholders, step sequences)
2. Recommend: extract_to_templates | extract_to_workflows | keep_inline

For each terminology_variant:

1. Review variant terms and their files
2. Recommend standardization term or keep variants

Step 4c: Single-File Quality Analysis

Read each content file and analyze:

Completeness:

• TODO markers, placeholder text
• Missing examples, incomplete sections

Contradictions:

• Conflicting rules
• Examples violating stated rules

Step 4d: Verify LLM Findings

Verify LLM findings by re-reading referenced files:

1. For each claimed duplicate, read both files and compare content
2. For each similarity claim, verify the specific sections exist
3. For each extraction candidate, verify the pattern appears as claimed

Reject any LLM claims that can't be verified against actual content.

Output: Verified quality report with:

• Exact duplicates (auto-detected)
• Verified similarity findings
• Verified extraction candidates
• Verified terminology issues
• Single-file quality scores

Step 5: Phase 4 - Reorganize (LLM + Bash)

Based on Phase 2 classification results:

Safe Reorganizations (auto-apply unless --no-fix):

• Move file to correct directory (same name)
• Rename to remove redundant suffix (e.g., -protocol, -framework)

mv {old_path} {new_path}

Risky Reorganizations (require confirmation):

• Split mixed-content file into multiple files
• Delete duplicate file
• Merge similar files

AskUserQuestion:
  question: "Split {file} into reference and workflow components?"
  options:
    - label: "Yes" description: "Split file"
    - label: "No" description: "Keep as-is"

After moves: Update cross-references in all affected files:

1. Grep for old paths in SKILL.md and all content files
2. Update references using Edit tool

Step 6: Phase 5 - Verify Links (LLM)

Use Grep and Read tools to verify references:

1. Extract all relative path references from SKILL.md (pattern: Read , references/, scripts/)
2. For each reference, verify the target file exists using Glob
3. Report any broken references

For each content file, verify:

• Internal cross-references valid
• SKILL.md references point to existing files

Step 7: Phase 6 - Report (LLM)

Generate comprehensive report:

# Skill Content Analysis Report

**Skill**: {skill_name}
**Path**: {skill_path}
**Date**: {timestamp}

## Summary

| Metric | Value |
|--------|-------|
| Total Files | {count} |
| Total Lines | {count} |
| Content Quality Score | {score}/100 |
| Reorganizations Applied | {count} |
| Links Verified | {count} |

## File Classification

| File | Current Dir | Classification | Confidence | Action |
|------|-------------|----------------|------------|--------|
| {file} | {dir} | {type} | {level} | {action} |

## Quality Analysis

### Completeness
{findings}

### Duplication
{findings}

### Consistency
{findings}

### Contradictions
{findings}

## Reorganizations Applied

### Safe (Auto-Applied)
- ✅ {description}

### Risky (User Confirmed)
- ✅ {description}
- ❌ Skipped: {description}

## Link Verification

| Status | Count |
|--------|-------|
| ✅ Valid | {count} |
| ⚠️ Updated | {count} |
| ❌ Broken | {count} |

## Recommendations

1. {recommendation}

---

Workflow 7: doctor-marketplace

Full marketplace batch analysis using hybrid two-phase workflow.

Parameters

• --no-fix (optional): Generate report only, skip fix phase

Step 1: Phase 1 - Script Batch Processing

EXECUTE the batch script to scan, analyze, and apply safe fixes.

If executor exists, use notation:

python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace fix
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace report

Otherwise, use bootstrap pattern with ${PLUGIN_ROOT} (see script-executor skill):

python3 ${PLUGIN_ROOT}/pm-plugin-development/*/skills/plugin-doctor/scripts/doctor-marketplace.py fix
python3 ${PLUGIN_ROOT}/pm-plugin-development/*/skills/plugin-doctor/scripts/doctor-marketplace.py report

Parse the JSON output to get:

• report_dir: Directory containing report files
• report_file: Path to JSON report
• findings_file: Path where LLM should write findings.md
• summary: Issue counts and categorization

Step 2: Phase 2 - LLM Analysis

1. Read the JSON report:

   Read: {report_file}
   

2. Tool Coverage Analysis via Agents (for items in components_for_tool_analysis):

   Spawn tool-coverage-agent for each component. Use parallel spawning for efficiency:

   # Spawn multiple agents in parallel (single message with multiple Task calls)
   Task: tool-coverage-agent (file1)
   Task: tool-coverage-agent (file2)
   Task: tool-coverage-agent (file3)
   ...
   

   Each agent receives:

   • file_path: {file}
   • declared_tools: {declared_tools}
   • component_type: {type}

   The agent semantically determines:

   • Which tools are actually USED (not just mentioned in docs)
   • Missing tools (used but not declared)
   • Unused tools (declared but not used)
   • False positives (tool mentioned in documentation, not actual usage)

   Why agents? Script-based regex detection causes false positives:

   • "Global settings" matched "Glob"
   • "task=" parameter matched "Task"
   • Documentation about tools matched as usage

3. Aggregate results using TOON format:

   Use templates/tool-coverage-results.toon template to aggregate agent results:

   analysis_timestamp: 2025-12-11T10:30:00Z
   total_components: 5
   
   results[5]{file,type,bundle,declared_tools,used_tools,missing_tools,unused_tools,confidence}:
   agents/foo.md,agent,pm-dev-java,"Read,Write","Read,Write",,Write,high
   commands/bar.md,command,pm-workflow,"Skill,Read","Skill,Read,Bash",,Bash,medium
   ...
   
   summary:
     components_analyzed: 5
     with_missing_tools: 1
     with_unused_tools: 2
     false_positives_detected: 3
   

   Why TOON? Uniform arrays of analysis results achieve ~50% token reduction vs JSON.

4. Create findings.md with:

   • Executive summary with statistics
   • Bundle-by-bundle analysis
   • Issue categorization:
     • Fixed: Safe fixes already applied by script
     • False Positive: Rule violations that are intentional
     • Intentional: Design decisions (e.g., Task tool for orchestration)
     • Needs Review: Actual issues requiring attention
   • Tool coverage findings from aggregated TOON
   • Recommendations for manual review

5. Write findings.md:

   Write: {findings_file}
   

Step 3: Process Risky Fixes

For each item in llm_review_items from the JSON report:

1. Evaluate context - Is this a real issue or false positive?
2. If real issue, prompt for risky fix:

   AskUserQuestion:
     question: "Fix {issue_type} in {file}?"
     options:
       - label: "Yes" description: "Apply fix"
       - label: "No" description: "Skip"
       - label: "Skip All" description: "Skip remaining"
   

3. Apply fix if approved using Edit tool

Step 4: Report Summary

Display final summary:

## Marketplace Health Report

**Report Location**: {report_dir}

| Metric | Value |
|--------|-------|
| Total Bundles | X |
| Total Components | X |
| Safe Fixes Applied | X |
| Issues Reviewed | X |
| False Positives | X |

**Files Created**:
- {report_file}
- {findings_file}

---

External Resources

Scripts (scripts/)

IMPORTANT: Only doctor-marketplace.py is registered in the executor. The other scripts (_analyze.py, _validate.py, _fix.py) are internal modules with underscore prefix and are accessed via doctor-marketplace subcommands.

Registered Script (callable via executor):

| Script | Subcommand | Mode | Purpose | |--------|------------|------|---------| | doctor-marketplace.py | scan | EXECUTE | Batch discovery of all marketplace components | | doctor-marketplace.py | analyze | EXECUTE | Batch analysis of all components for issues | | doctor-marketplace.py | fix | EXECUTE | Auto-apply safe fixes across marketplace | | doctor-marketplace.py | report | EXECUTE | Generate comprehensive report for LLM review |

Notation: pm-plugin-development:plugin-doctor:doctor-marketplace {subcommand}

Internal Modules (NOT directly callable - used internally by doctor-marketplace):

| Module | Purpose | |--------|---------| | _analyze.py | Structural analysis, bloat, Rule 6/7/Pattern 22 | | _analyze_markdown.py | Markdown structure analysis | | _analyze_coverage.py | Tool coverage extraction | | _analyze_structure.py | Skill directory structure validation | | _analyze_crossfile.py | Cross-file duplication analysis | | _validate.py | Reference extraction and validation | | _fix.py | Fix application and verification |

Hybrid Batch Processing (doctor-marketplace.py)

The doctor-marketplace.py script provides Phase 1 of the hybrid doctor workflow for full marketplace operations:

Phase 1 (Script - Deterministic):

# Scan entire marketplace
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace scan

# Analyze all components
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace analyze

# Preview safe fixes (dry run)
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace fix --dry-run

# Apply safe fixes
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace fix

# Generate report for LLM review (writes to: .plan/temp/plugin-doctor-report/)
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace report

# Or specify custom output directory
python3 .plan/execute-script.py pm-plugin-development:plugin-doctor:doctor-marketplace report --output .plan/temp/my-review

Report Output: Reports are written to a fixed directory with timestamped, scoped files:

.plan/temp/plugin-doctor-report/
├── 20251213-155927-pm-plugin-development-report.json   # Single bundle
├── 20251213-155927-pm-plugin-development-findings.md
├── 20251213-160530-marketplace-report.json             # All bundles
└── 20251213-160530-marketplace-findings.md

Filename includes scope: single bundle name, multiple bundle names (up to 3), or "marketplace" for all. Use --output to specify a custom directory path. Multiple reports accumulate in the directory.

Phase 2 (LLM - Semantic): After Phase 1 creates the report directory and JSON, the LLM:

1. Reads {timestamp}-{scope}-report.json for structured data
2. Applies contextual judgment (identifies false positives, priorities)
3. Creates {timestamp}-{scope}-findings.md in the same directory with:
   • Executive summary and statistics
   • Bundle-by-bundle analysis
   • Categorization of remaining issues (fixed, false positive, intentional)
   • Recommendations for manual review
4. Processes risky fixes with user confirmation
5. Documents complex refactoring recommendations

Workflow for /plugin-doctor marketplace:

1. Run doctor-marketplace scan to discover components
2. Run doctor-marketplace analyze to find issues
3. Run doctor-marketplace fix to auto-apply safe fixes
4. Run doctor-marketplace report to generate LLM review items
5. LLM processes remaining risky/unfixable issues

References (references/)

Diagnosis References (7) - READ before analyzing:

• agents-guide.md - Agent quality standards
• commands-guide.md - Command quality standards
• skills-guide.md - Skill structure standards
• metadata-guide.md - plugin.json schema
• content-classification-guide.md - Content type classification criteria (for doctor-skill-content)
• content-quality-guide.md - Content quality analysis dimensions (for doctor-skill-content)
• plan-marshall-plugin-validation.md - Domain manifest validation (for plan-marshall-plugin skills)

External Standards (from plugin-architecture) - READ for script analysis:

• script-standards.md - Script documentation, testing, and quality standards

Fix References (4) - READ before applying fixes:

• fix-catalog.md - Fix categorization rules
• safe-fixes-guide.md - Safe fix patterns
• risky-fixes-guide.md - Risky fix patterns
• verification-guide.md - Verification procedures

Reporting (1) - REFERENCE for output formatting:

• reporting-templates.md - Summary report templates

Assets (assets/)

• fix-templates.json - Fix templates and rules

Templates (templates/)

• tool-coverage-results.toon - TOON template for aggregating tool-coverage-agent results

---

Critical Rules

Rule 6: Task Tool Prohibition in Agents

Agents CANNOT use Task tool (unavailable at runtime).

Rule 7: Maven Execution Restriction

Only maven-builder agent may execute Maven commands.

Pattern 22: Agent Lessons Learned Requirement

Agents MUST record lessons via manage-lessons skill, not self-invoke commands.

Rule 9: Explicit Script Calls in Workflows

All script/tool invocations in workflow documentation MUST have explicit bash code blocks. Vague instructions like "read the file", "display the content", or "check the status" are NOT acceptable. Every operation requiring a script call MUST document the exact python3 .plan/execute-script.py command.

Detection: Scan workflow steps for action verbs (read, write, display, check, validate, get, list) without accompanying bash code blocks containing execute-script.py.

Examples of violations:

• "Display the solution outline for review" (missing bash block)
• "Read the config to get domains" (missing bash block)
• "Validate the output" (missing bash block)

Correct pattern:

### Step N: Read the solution outline

```bash
python3 .plan/execute-script.py pm-workflow:manage-solution-outline:manage-solution-outline read \
  --plan-id {plan_id}

### Rule 10: Self-Contained Command Definition

Components that execute scripts MUST have the EXACT notation (`bundle:skill:script`) explicitly defined WITHIN themselves.

**Four Detection Modes**:

| Mode | Catches |
|------|---------|
| A: Delegation | "Execute command from section Nb" - parent-passed |
| B: Notation | `execute-script.py artifact_store` - missing bundle:skill |
| C: Missing Section | "Log the assessment" without ## Logging Command section |
| D: Parameters | `--plan-id` when should be positional (via --help) |

**Notation Format**:
- ✅ `pm-workflow:manage-plan-artifacts:manage-artifacts`
- ❌ `artifact_store` (missing bundle:skill)
- ❌ `manage-files:artifact-store` (missing bundle)

**Required Pattern**:
- Explicit "## Logging Command" or "## Script Commands" section
- Full bash block with `python3 .plan/execute-script.py {bundle}:{skill}:{script}`
- Every notation MUST match format: `bundle:skill:script`
- Parameter table showing where values come from

**CRITICAL**: Do NOT invent notations. Use only documented notations from the skill being called.

---

## Non-Prompting Requirements

This skill is designed to run without user prompts for safe operations. Required permissions:

**Skill Invocations (covered by bundle wildcards):**
- `Skill(plan-marshall:*)` - diagnostic-patterns
- `Skill(pm-plugin-development:*)` - plugin-architecture, marketplace-inventory

**Script Execution:**
- Script paths resolved from `.plan/scripts-library.toon` (system convention)
- Permissions managed by `tools-setup-project-permissions`

**File Operations (covered by project permissions):**
- `Read(//marketplace/**)` - Read marketplace files
- `Edit(//marketplace/**)` - Apply fixes to components
- `Glob(//marketplace/**)` - Discover components

**Prompting Behavior:**
- **Safe fixes**: Applied automatically WITHOUT prompts (no AskUserQuestion)
- **Risky fixes**: ONLY these require AskUserQuestion confirmation
- All other operations (read, analyze, glob) are non-prompting

**Ensuring Non-Prompting for Safe Operations:**
- All file reads/edits use relative paths within marketplace/
- Script paths resolved from `.plan/scripts-library.toon` (system convention)
- Skill invocations use bundle-qualified names covered by `Skill({bundle}:*)` wildcards
- AskUserQuestion is ONLY used for risky fix confirmations

## Notes

- **Unified workflow**: Diagnose → Auto-Fix → Prompt Risky → Verify
- **Progressive disclosure**: Load 2 references per workflow (~800 lines)
- **Stdlib-only scripts**: No external dependencies
- **Backup before modify**: `fix apply` creates backups
- **User control**: Risky fixes require explicit approval
- **Non-prompting safe fixes**: Safe fixes never prompt - applied automatically