Skip to content

fix: remove remaining CORA references missed by #121 #127

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
terry-li-hm wants to merge 2 commits into
base: main
Choose a base branch
from
+7 −7
Open

fix: remove remaining CORA references missed by #121 #127

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0f354dd
fix: remove remaining hardcoded CORA references in schema.yaml
terry-li-hm Jan 26, 2026
15d2d68
fix: rename cora-test-reviewer to test-quality-reviewer
terry-li-hm Jan 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions plugins/compound-engineering/commands/workflows/compound.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ This command launches multiple specialized subagents IN PARALLEL to maximize eff
- **performance_issue** → `performance-oracle`
- **security_issue** → `security-sentinel`
- **database_issue** → `data-integrity-guardian`
- **test_failure** → `cora-test-reviewer`
- **test_failure** → `test-quality-reviewer`
- Any code-heavy issue → `kieran-rails-reviewer` + `code-simplicity-reviewer`

## What It Captures
Expand Down Expand Up @@ -184,7 +184,7 @@ Based on problem type, these agents can enhance documentation:
### Specific Domain Experts
- **performance-oracle**: Analyzes performance_issue category solutions
- **security-sentinel**: Reviews security_issue solutions for vulnerabilities
- **cora-test-reviewer**: Creates test cases for prevention strategies
- **test-quality-reviewer**: Creates test cases for prevention strategies
- **data-integrity-guardian**: Reviews database_issue migrations and queries

### Enhancement & Documentation
Expand Down
2 changes: 1 addition & 1 deletion plugins/compound-engineering/commands/workflows/work.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ This command takes a work document (plan, specification, or todo file) and execu
- **kieran-rails-reviewer**: Verify Rails conventions (Rails projects)
- **performance-oracle**: Check for performance issues
- **security-sentinel**: Scan for security vulnerabilities
- **cora-test-reviewer**: Review test quality (Rails projects with comprehensive test coverage)
- **test-quality-reviewer**: Review test quality (Rails projects with comprehensive test coverage)

Run reviewers in parallel with Task tool:

Expand Down
8 changes: 4 additions & 4 deletions plugins/compound-engineering/skills/compound-docs/schema.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# CORA Documentation Schema
# Documentation Schema
# This schema MUST be validated before writing any documentation file

required_fields:
module:
type: string
description: "Module/area of CORA (e.g., 'Email Processing', 'Brief System', 'Authentication')"
description: "Module/area (e.g., 'Email Processing', 'Brief System', 'Authentication')"
examples:
- "Email Processing"
- "Brief System"
Expand Down Expand Up @@ -54,7 +54,7 @@ required_fields:
- testing_framework # Test setup, fixtures, VCR
- documentation # README, guides, inline docs
- tooling # Scripts, generators, CLI tools
description: "CORA component involved"
description: "Component involved"

symptoms:
type: array[string]
Expand Down Expand Up @@ -133,7 +133,7 @@ optional_fields:
- "turbo-stream"

validation_rules:
- "module must be a valid CORA module name"
- "module must be a valid module name"
- "date must be in YYYY-MM-DD format"
- "problem_type must match one of the enum values"
- "component must match one of the enum values"
Expand Down