Merged
Conversation
Remove the catalog feature which provided policy registry and distribution: - Delete cupcake-core/src/catalog/ module (registry, index, manifest, lock, installer) - Delete engine/catalog_overlay.rs and Phase 1.5 evaluation - Delete cupcake-cli/src/catalog_cli.rs and CLI commands - Remove catalog feature flags from Cargo.toml files - Delete docs/docs/catalog/ documentation - Remove stale asciinema recordings - Update telemetry docs to remove catalog phase references 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
backnotprop
force-pushed
the
feat/reduce
branch
from
cfb084f to
024e7cf
Compare
Actions were fire-and-forget commands executed after policy violations. The feature was unused and unmaintained, adding unnecessary complexity. - Remove ActionConfig, ActionSection from rulebook - Remove execute_actions from executor - Remove actions paths from ProjectPaths and GlobalPaths - Delete all action-related test files (~3600 lines) - Update trust manifest to only track signals - Clean up documentation and config templates 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The trust system provided optional cryptographic integrity verification for signal scripts using SHA-256 hashing and HMAC-SHA256 signing. The feature was unused and added unnecessary complexity. - Remove cupcake-core/src/trust/ module (~1,200 lines) - Remove trust_cli.rs (~830 lines) - Remove trust_verifier from Engine and Executor - Remove deterministic-tests feature flag - Remove sha2/hmac dependencies (keep hex for trace IDs) - Remove #[cfg(feature = "deterministic-tests")] from all tests - Update CLI docs to remove trust command section 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove trust verification comment in executor.rs - Delete trust.yaml cast config - Update docs/assets/README.md to remove trust references - Remove "cupcake trust" from cupcake_exec_protection.rego patterns 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove asciinema cast recordings and generation infrastructure - Remove disabled test files (trust_cli_test, init_test) - Remove stale actions/ directory references from docs - Remove duplicate examples/claude-code/welcome directory - Remove committed debug files, add .cupcake/debug/ to gitignore - Remove deterministic-tests feature flag from justfile 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Delete trust_parser_test/ fixture directory (unused after trust removal) - Delete REFERENCE_HOOKS.md (redundant with internal docs) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The rulebook.yml file was accidentally removed from init during the catalog system cleanup. This restores it by including the base-config template from fixtures. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The deterministic-tests feature was removed with the trust system but CI workflows still referenced it, causing build failures. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove the catalog feature which provided policy registry and distribution: