Skip to content

chore: cleanup project root to match Claude plugin standards - #4

Open
tommy-ca wants to merge 13 commits into
ginobefun:mainfrom
tommy-ca:cleanup/root-structure
Open

tommy-ca wants to merge 13 commits into
ginobefun:mainfrom
tommy-ca:cleanup/root-structure

Conversation

@tommy-ca

Copy link
Copy Markdown

🎯 Summary

This PR cleans up the project root directory to align with Claude plugin standard directory structure.

📋 Changes Made

Removed Files

Updated Files

  • .gitignore - Added Claude plugin build artifacts:
    • *.skill (legacy skill file format)
    • *.claude-plugin.zip (plugin distribution archives)

✅ Benefits

  • Cleaner root structure - Only essential files remain
  • Better organization - Follows Claude plugin conventions
  • Prevents build artifacts - Updated .gitignore prevents accidental commits

📁 Final Root Structure

Root Directory:
├── README.md              # Main documentation
├── LICENSE                 # License file
├── CLAUDE.md              # Development guide
├── build.sh               # Build/validation script
├── plugins/               # Plugin directory
├── .claude-plugin/        # Marketplace registry
├── .github/               # GitHub templates
└── .gitignore            # Updated with plugin ignores

🧪 Testing

  • Verified all essential files remain
  • Confirmed .github/PULL_REQUEST_TEMPLATE.md still exists
  • Validated .gitignore changes
  • Checked plugin structure intact

📝 Commit History

This PR contains 3 atomic commits:

  1. chore: remove duplicate PULL_REQUEST_TEMPLATE.md from root
  2. chore: remove temporary PR_REVIEW_RESULTS.md
  3. chore: add Claude plugin build artifacts to .gitignore

- Add plugins/deep-reading-analyst/ directory with complete plugin bundle
- Include plugin manifest, skills, commands, and agents
- Add marketplace registry configuration
- Follow Every Marketplace plugin format standards
- Update README.md with marketplace installation instructions
- Update CLAUDE.md to reflect cleaned repository structure
- Remove references to legacy .skill file distribution
- Add marketplace-specific development workflows
- Update build.sh to validate plugin structure only
- Remove legacy .skill file building functionality
- Add comprehensive plugin component validation
- Enhance JSON syntax checking and error reporting
- Remove duplicate src/ directory (content moved to plugins/)
- Remove legacy deep-reading-analyst.skill file
- Remove README_CN.md (optional multilingual docs)
- Achieve single source of truth for plugin content
- Include detailed overview of changes
- Document breaking changes and migration guide
- Provide installation instructions and benefits
- Outline testing and deployment readiness
- Update .claude/skills/deep-reading-analyst/references/ to skills/deep-reading-analyst/references/
- Ensures consistent relative path from plugin root
- Aligns with marketplace plugin directory structure
- Update 8 framework reference paths from .claude/skills/deep-reading-analyst/ to skills/deep-reading-analyst/references/
- Ensures consistent relative paths from plugin root
- Covers all 8 thinking frameworks: SCQA, 5W2H, Critical Thinking, Inversion, Mental Models, First Principles, Systems Thinking, Six Hats
- Update /references/ to skills/deep-reading-analyst/references/ for consistency
- Fixes 2 occurrences: framework docs reference and comparison matrix link
- Ensures all reference links use consistent relative path from plugin root
feat: refactor repository to Claude plugins marketplace spec
- GitHub uses .github/PULL_REQUEST_TEMPLATE.md for PR templates
- Root file was a duplicate PR description, not a template
- Aligns with Claude plugin standard directory structure
- Temporary review document from PR #1
- Not part of Claude plugin standard structure
- Cleanup improves repository organization
- Add *.skill (legacy skill file format)
- Add *.claude-plugin.zip (plugin distribution archives)
- Prevents accidental commit of build artifacts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant