LogoSkills

code-review-suppression

Project-specific rules to suppress false positives during code reviews

Code Review Suppression List#

gstack pattern reference: Specify "do not flag this" in the review checklist to reduce alert fatigue.

Default Suppressions (Dart Full-Stack)#

  • Freezed generated code (*.freezed.dart, *.g.dart): Auto-generated files are excluded from review
  • Serverpod generated code (*_server/lib/src/generated/): Server auto-generated code excluded
  • slang i18n generated files (*_gen/*.g.dart): Translation auto-generated files excluded
  • BLoC boilerplate: sealed class Event/State definitions are not flagged as boilerplate
  • context.read<>(): Usage in BLoC event dispatch is not an anti-pattern (only problematic inside build)
  • Freezed copyWith: Long .copyWith() chains are not flagged for refactoring
  • sealed class pattern: Long case lists in when/map methods are a normal pattern
  • No Co prefix: CoUI widgets use Button, Card etc. without prefix (differs from Flutter Material)
  • No ButtonSize.medium: ButtonSize.normal is the default, do not suggest using medium
  • Dot shorthand: Dart 3.10+ .start, .all() etc. shorthand syntax is normal
  • melos monorepo import: Relative path imports between packages are normal in melos structure
  • feature module barrel export: feature.dart barrel file is convention
  • GetIt DI registration: getIt.registerFactory/Singleton pattern does not need flagging
  • Duplicate code for readability: Repetitive code explicitly for clarity is not flagged as DRY violation
  • Missing const (inside widgets): Do not enforce const when it hurts readability
  • Korean comments/strings: Korean usage is the project standard

Project-Specific Custom Suppressions#

Create .code-review-suppress.md at the user project root to define additional suppressions:

# Project-Specific Suppressions

- **Legacy API calls**: Usage of deprecated API at `api/v1/` path is not flagged as it is under migration
- **TODO comments**: TODOs planned for the current sprint are ignored

Suppression Application#

  1. This file and the project's .code-review-suppress.md are automatically loaded during review
  2. Each flagged item is compared against the suppression list
  3. Matching items are excluded from review results
  4. Number of excluded items displayed at bottom: "N items excluded by suppression"