Blog post Leaderboard MCP Deploy agents
Back to all reviewers

Standardize null safety patterns

nestjs/nest
Based on 5 comments
TypeScript

Use modern JavaScript features and utility functions consistently for null/undefined checks. Prefer optional chaining (?.), nullish coalescing (??), and dedicated utility functions like isNil() over direct null checks or OR (||) operators.

Null Handling TypeScript

Reviewer Prompt

Use modern JavaScript features and utility functions consistently for null/undefined checks. Prefer optional chaining (?.), nullish coalescing (??), and dedicated utility functions like isNil() over direct null checks or OR (   ) operators.

Key practices:

  1. Use optional chaining for potentially null properties: ```typescript // Instead of if (pattern && pattern.test(str))

// Use if (pattern?.test(str))


2. Use nullish coalescing for default values:
```typescript
// Instead of
const ttlValueOrFactory = ttlMetadata || null;

// Use
const ttlValueOrFactory = ttlMetadata ?? null;
  1. Use utility functions for explicit null/undefined checks: ```typescript // Instead of if (value === null || value === undefined)

// Use import { isNil } from ‘@nestjs/common/utils’; if (isNil(value)) ```

This approach improves code readability, reduces potential null reference errors, and maintains consistency across the codebase. It leverages TypeScript’s type system and modern JavaScript features for better null safety.

5
Comments Analyzed
TypeScript
Primary Language
Null Handling
Category

Source Discussions