Skip to content

Add ClaudeCodeFormat to unified build system#46

Merged
santosomar merged 1 commit intodevelopfrom
feature/plugin-fixes
Oct 21, 2025
Merged

Add ClaudeCodeFormat to unified build system#46
santosomar merged 1 commit intodevelopfrom
feature/plugin-fixes

Conversation

@thomas-bartlett
Copy link
Copy Markdown
Collaborator

Integrate Claude Code plugin generation into the Python converter architecture. Replaces bash script with ClaudeCodeFormat class that outputs to skills/ while preserving YAML frontmatter. Fixes plugin manifest schema issues (repository field format).

@thomas-bartlett thomas-bartlett self-assigned this Oct 20, 2025
Copilot AI review requested due to automatic review settings October 20, 2025 15:36
@thomas-bartlett thomas-bartlett added the enhancement New feature or request label Oct 20, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR integrates Claude Code plugin generation into the unified build system by replacing a bash script with a Python-based ClaudeCodeFormat class. The implementation outputs to skills/software-security/rules/ at project root (rather than ide_rules/), preserves YAML frontmatter with security rule metadata, and fixes plugin manifest schema issues by changing the repository field from an object to a string format.

Key Changes

  • Added ClaudeCodeFormat class to generate .md files with preserved YAML frontmatter for Claude Code plugins
  • Introduced outputs_to_ide_rules() method in base format to support formats outputting to project root vs ide_rules/
  • Fixed plugin manifest schema by changing repository field from object format to simple string URL

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/unified_to_all.py Registers ClaudeCodeFormat and uses format's output preference for path construction
src/prepare-claude-code-plugin.sh Removed bash script replaced by Python converter
src/formats/claudecode.py New ClaudeCodeFormat implementation with YAML frontmatter preservation
src/formats/base.py Added outputs_to_ide_rules() method for flexible output location
src/formats/init.py Exports ClaudeCodeFormat in module interface
src/converter.py Updated FormatOutput to include outputs_to_ide_rules field
skills/software-security/rules/*.md Added rule_id after frontmatter, removed empty languages arrays
.claude-plugin/plugin.json Changed repository from object to string format
.claude-plugin/marketplace.json Moved repository field to plugin level, fixed source path

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/formats/claudecode.py
Comment thread skills/software-security/rules/codeguard-1-safe-c-functions.md
Comment thread .claude-plugin/marketplace.json
Copy link
Copy Markdown
Contributor

@santosomar santosomar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thomas-bartlett — thanks for fixing and enhancing the elements from #45 ... Looks good!

@santosomar santosomar merged commit 4088444 into develop Oct 21, 2025
@santosomar santosomar deleted the feature/plugin-fixes branch October 21, 2025 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants