fundamental-ngx: Skill for Claude Code

Best Practices Audit: $ARGUMENTS

If $ARGUMENTS is empty, ask the user for a component path or folder before proceeding.

Audit the code at $ARGUMENTS against the project's conventions. Unlike /review-pr which checks diffs, this audits existing code as-is.

Checklist

1. Angular 22+ Patterns

• New code uses input() / output() / model() / linkedSignal(). Existing @Input() / @Output() decorators are acceptable.
• host: {} in decorator — no @HostBinding() / @HostListener()
• @if / @for / @switch — no *ngIf / *ngFor / *ngSwitch
• No standalone: true (default since Angular 19)
• No allowSignalWrites option in effect() (the option no longer exists)
• DestroyRef + takeUntilDestroyed() — no custom DestroyedService
• computed() + host: { '[class]': } — no CssClassBuilder / @applyCssClass
• No ngClass / ngStyle (use direct bindings)

2. State Management

• signal() only when reactive consumer exists
• Plain properties for internal bookkeeping
• No redundant markForCheck() after signal updates
• BehaviorSubject only for async streams, not local state
• effect() for signal side effects — no Subject<void> for trigger-only patterns
• No effect() used for state derivation — use computed() or linkedSignal instead
• linkedSignal used for mutable derived state (e.g., editable fields that reset on input change)
• No object/array mutation in place then signal.set() with same reference — always create new references
• No conditional signal reads creating invisible dependency gaps in effect() / computed() — tracked signals read before conditional logic

3. Dependency Injection

• InjectionToken for contextual defaults — not @ContentChild assigning to signal inputs
• Tokens defined near child component with { optional: true }
• FD_ prefix for component identity tokens
• Queries by token, not concrete class