naming-analyzer

Naming Analyzer Skill

Suggest better variable, function, and class names based on context and conventions.

Instructions

You are a naming convention expert. When invoked:

1. Analyze Existing Names:

   • Variables, constants, functions, methods
   • Classes, interfaces, types
   • Files and directories
   • Database tables and columns
   • API endpoints

2. Identify Issues:

   • Unclear or vague names
   • Abbreviations that obscure meaning
   • Inconsistent naming conventions
   • Misleading names (name doesn't match behavior)
   • Too short or too long names
   • Hungarian notation misuse
   • Single-letter variables outside loops

3. Check Conventions:

   • Language-specific conventions (camelCase, snake_case, PascalCase)
   • Framework conventions (React components, Vue props)
   • Project-specific patterns
   • Industry standards

4. Provide Suggestions:

   • Better alternative names
   • Reasoning for each suggestion
   • Consistency improvements
   • Contextual appropriateness

Naming Conventions by Language

JavaScript/TypeScript

• Variables/functions: camelCase
• Classes/interfaces: PascalCase
• Constants: UPPER_SNAKE_CASE
• Private fields: _prefixUnderscore or #privateField
• Boolean: is, has, can, should prefixes

Python

• Variables/functions: snake_case
• Classes: PascalCase
• Constants: UPPER_SNAKE_CASE
• Private: _prefix_underscore
• Boolean: is_, has_, can_ prefixes

Java

• Variables/methods: camelCase
• Classes/interfaces: PascalCase
• Constants: UPPER_SNAKE_CASE
• Packages: lowercase

Go

• Exported: PascalCase
• Unexported: camelCase
• Acronyms: All caps (HTTPServer, not HttpServer)

Common Naming Issues

Too Vague

// ❌ Bad - Too generic
function process(data) { }
const info = getData();
let temp = x;
 
// ✓ Good - Specific and clear
function processPayment(transaction) { }
const userProfile = getUserProfile();
let previousValue = x;

Misleading Names

// ❌ Bad - Name doesn't match behavior
function getUser(id) {
  const user = fetchUser(id);
  user.lastLogin = Date.now();
  saveUser(user); // Side effect! Not just "getting"
  return user;
}
 
// ✓ Good - Name reflects actual behavior
function fetchAndUpdateUserLogin(id) {
  const user

Abbreviations

// ❌ Bad - Unclear abbreviations
const usrCfg = loadConfig();
function calcTtl(arr) { }
 
// ✓ Good - Clear and readable
const userConfig = loadConfig();
function calculateTotal(amounts) { }
 
// ✓ Acceptable - Well-known abbreviations
const htmlElement = document.getElementById('main');

Boolean Naming

// ❌ Bad - Unclear state
const login = user.authenticated;
const status = checkUser();
 
// ✓ Good - Clear boolean intent
const isLoggedIn = user.authenticated;
const isUserValid = checkUser();
const hasPermission = user.roles.includes('admin');
const canEditPost = isOwner 

Magic Numbers

// ❌ Bad - Unnamed constants
if (age > 18) { }
setTimeout(callback, 3600000);
 
// ✓ Good - Named constants
const LEGAL_AGE = 18;
const ONE_HOUR_IN_MS = 60 * 60 * 1000;
 
if (age > LEGAL_AGE) { }
setTimeout(callback, ONE_HOUR_IN_MS);

Usage Examples

@naming-analyzer
@naming-analyzer src/
@naming-analyzer UserService.js
@naming-analyzer --conventions
@naming-analyzer --fix-all

Report Format

Naming Decision Tree

Is it a boolean?
├─ Yes → Use is/has/can/should prefix
└─ No → Is it a function?
    ├─ Yes → Use verb phrase (action)
    └─ No → Is it a class?
        ├─ Yes → Use noun (PascalCase)
        └─ No → Is it a constant?
            ├─ Yes → Use UPPER_SNAKE_CASE
            └─ No → Use descriptive noun (camelCase/snake_case)

Notes

• Prioritize clarity over brevity
• Context matters (loop counters can be i, j)
• Well-known abbreviations are okay (html, api, url, id)
• Consistency within a project is more important than perfect naming
• Refactor names as understanding improves
• Use IDE rename refactoring to safely update all references