[Spec 0056] Relocate consult review types to consult-types/#108
Merged
waleedkadous merged 5 commits intomainfrom Dec 15, 2025
Merged
[Spec 0056] Relocate consult review types to consult-types/#108waleedkadous merged 5 commits intomainfrom
waleedkadous merged 5 commits intomainfrom
Conversation
Builders now automatically get a symlink to the project's .env file, giving them access to environment variables needed for running tests and local development.
0056: Consult Types Refactor - Move review types from roles/review-types/ to consult-types/ 0057: Dashboard Tab Overhaul - Rename Projects to Dashboard, add two-column layout
- Update command docs with consult-types/ location and migration note - Add test for deprecation warning functionality - Fix loadReviewTypePrompt to properly distinguish local vs skeleton files - Skip CLI existence check in dry-run mode for testability
Contributor
Author
Architect Integration Review (3-Way)Verdict: REQUEST_CHANGES Review Summary
Critical IssueThe embedded skeleton ( Problem: Published npm installs use
Fix Required: cd packages/codev
npm run copy-skeleton
git add skeleton/
git commit --amendMinor Notes (from Gemini)
🏗️ Architect integration review |
Contributor
Author
Update: ApprovedBuilder correctly clarified the build workflow:
Codex's concern was based on checking the local build artifact state, not the committed source. Verdict: APPROVE Builder can merge when ready. 🏗️ Architect |
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.
Summary
Relocates consult review types from
codev/roles/review-types/tocodev/consult-types/to provide a cleaner, more intuitive location for review type prompts.Key changes:
consult-types/directory in skeleton and local projectroles/review-types/locationcodev doctorchecks for proper migrationTest plan
consult --type spec-reviewworks with new locationcodev doctorwarns about missingconsult-types/codev doctorwarns about deprecatedroles/review-types/Migration Note (for users upgrading to v1.4.0+)
mkdir -p codev/consult-types mv codev/roles/review-types/* codev/consult-types/ rm -r codev/roles/review-types