feat: rulebook standard structuring support#75
Conversation
There was a problem hiding this comment.
Pull request overview
This PR refactors the catalog bundle structure to simplify the organization and reduce nesting. The main change moves from a per-harness system directory structure to a single shared system entrypoint at the rulebook root, and flattens the policy directory structure.
Key changes:
- System entrypoint moved from
policies/<harness>/system/tosystem/at rulebook root - Policies now placed directly in
policies/<harness>/instead of nested subdirectories - Updated namespace patterns to reflect the flatter structure
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/docs/catalog/reference/namespace-conventions.md | Updated documentation to reflect new namespace patterns with system at root and simplified policy structure |
| docs/docs/catalog/reference/cli.md | Updated CLI validation checks to reflect system directory at root level |
| docs/docs/catalog/authoring/publishing.md | Updated directory structure example to show system at root |
| docs/docs/catalog/authoring/policies.md | Updated policy authoring guide with new namespace patterns and aggregation approach |
| docs/docs/catalog/authoring/manifest.md | Updated manifest validation requirements for new structure |
| docs/docs/catalog/authoring/index.md | Updated quick start guide and directory structure examples |
| cupcake-core/src/engine/catalog_overlay.rs | Added system directory scanning and parsing logic at rulebook root |
| cupcake-cli/src/catalog_cli.rs | Updated linting logic to validate new structure and namespace patterns |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
captjt
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 23 out of 26 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| assert!(cupcake_dir.is_dir(), ".cupcake should be a directory"); | ||
|
|
||
| // Verify all required subdirectories exist (only claude harness) | ||
| // New structure: system/ and helpers/ at root level, not per-harness |
There was a problem hiding this comment.
should probably remove comments like these
2 participants
No description provided.