Back to all reviewers

Use semantic naming

google-gemini/gemini-cli
Based on 6 comments
TypeScript

Choose names that clearly communicate the purpose, behavior, and intent of variables, methods, classes, and interfaces. Avoid ambiguous or misleading names that require additional context to understand.

Naming Conventions TypeScript

Reviewer Prompt

Choose names that clearly communicate the purpose, behavior, and intent of variables, methods, classes, and interfaces. Avoid ambiguous or misleading names that require additional context to understand.

Names should be self-documenting and accurately reflect what the entity represents or does. Consider the actual behavior and usage patterns when naming, not just the initial implementation.

Examples of improvements:

  • SupportedIDEDetectedIde (clarifies this represents the current IDE, not all supported ones)
  • ActiveFileSchemaopenFilesSchema with filePathactiveFilePath (better reflects the expanding scope)
  • sourcekind (more semantic and extensible for different types)
  • interactiveforceInteractive (clarifies it only affects behavior when -p flag is present)
  • is_looploopDetected (more targeted and descriptive)

When naming conflicts arise or behavior changes, prioritize semantic accuracy over maintaining existing names. The name should help future developers understand the code without needing to read the implementation.

6
Comments Analyzed
TypeScript
Primary Language
Naming Conventions
Category

Source Discussions