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
36110mo 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
3463mo 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
code change that neither fixes a bug nor adds a feature
- `perf`: A code change that improves performance
- `test`: Adding missing tests or correcting existing tests
- `chore`: Other changes that
28817mo agoReviewMountCreate a free account to mount
typescript
// Wait for Modal/Drawer to fully render
await new Promise(resolve => setTimeout(resolve, 100))
```
## CodeReview Checklist
- [ ] Confirm that tests for Modal/Drawer components use the correct DOM query method
28817mo agoReviewMountCreate a free account to mount