only used as a type in the file, use 'import type' instead of 'import'
## CodeReview Checklist
- Ensure proper typing
- Check for code duplication
- Verify error handling
- Confirm test coverage
3977d agoReviewMountCreate a free account to mount
result.success) {
throw new Error(result.error)
}
return result.webhook // This will be of type Webhook
}
})
}
```
## Enforcement
### CodeReview Checklist
- [ ] Types are imported from feature's `types/index.ts`
- [ ] No duplicate type definitions across files
36111mo agoReviewMountCreate a free account to mount
technology/pattern is used in 3+ files
- Common bugs could be prevented by a rule
- Codereviews repeatedly mention the same feedback
- New security or performance patterns emerge
- **Modify Existing Rules
3513mo agoReviewMountCreate a free account to mount
Integration
Loaded skills add their instructions to the prompt:
```xml
{skills_section}
Instructions for codereview...
```
## Best Practices
1. **Dynamic over static** - Generate tool lists from definitions
2. **Keep template
3318mo agoReviewMountCreate a free account to mount
reused independently
4. **Collaboration**: Multiple developers can work on different files simultaneously
5. **CodeReview**: Smaller changes are easier to review
## Refactoring Patterns
### When a File Gets Too Large
31811mo agoReviewMountCreate a free account to mount
Eight-step epic build cycle — reads state.yaml, execution-status.yaml, and one epic capsule; updates status via bp-yaml-set or direct edit. Resume mode runs one step per invocation. Use instead of ad-hoc execute-plan for release work.
2888d agoReviewMountCreate a free account to mount
Chain multiple bigpowers skills into a custom workflow recipe saved in specs/. Use when a project repeats a non-standard skill sequence, or user wants a documented playbook beyond orchestrate-project modes.
2888d agoReviewMountCreate a free account to mount