Back to skills
SKILL.md
Controllers Services
ASecurityController/Service separation and Custom Decorators.
- 8 stars
- 0 votes
- 0 copies
- 2 views
- Added September 8, 2026
Works with
Security analysis
100/100Pro scans all 4 files and shows the line behind each finding
npx -y skills add ngxtm/devkit --skill controllers-services --agent claude-codeAre you the author of Controllers Services?
Add the live security badge to your README. It updates with every re-scan.
[](https://www.skillsdirectory.com/skills/ngxtm-controllers-services)---
name: NestJS Controllers & Services
description: Controller/Service separation and Custom Decorators.
metadata:
labels: [nestjs, controller, service]
triggers:
files: ['**/*.controller.ts', '**/*.service.ts']
keywords: [Controller, Injectable, ExecutionContext, createParamDecorator]
---
# NestJS Controllers & Services Standards
## Controllers
- **Role**: Handler only. Delegate **all** logic to Services.
- **Context**: Use `ExecutionContext` helpers (`switchToHttp()`) for platform-agnostic code.
- **Custom Decorators**:
- **Avoid**: `@Request() req` -> `req.user` (Not type-safe).
- **Pattern**: Create typed decorators like `@CurrentUser()`, `@DeviceIp()`.
```typescript
export const CurrentUser = createParamDecorator(
(data: unknown, ctx: ExecutionContext) =>
ctx.switchToHttp().getRequest().user,
);
```
## DTOs & Validation
- **Strictness**:
- `whitelist: true`: Strip properties without decorators.
- **Critical**: `forbidNonWhitelisted: true`: Throw error if unknown properties exist.
- **Transformation**:
- `transform: true`: Auto-convert primitives (String '1' -> Number 1) and instantiate DTO classes.
- **Documentation**:
- **Automation**: Use the `@nestjs/swagger` CLI plugin (`nest-cli.json`) to auto-detect DTO properties without manual `@ApiProperty()` tags.
## Interceptors (Response Mapping)
- **Standardization**: specific responses should be mapped in **Interceptors**, not Controllers.
- Use `map()` to wrap success responses (e.g. `{ data: T }`).
- Refer to **[API Standards](../api-standards/SKILL.md)** for `PageDto` and `ApiResponse`.
- Use `catchError()` to map low-level errors (DB constraints) to `HttpException` (e.g. `ConflictException`) _before_ they hit the global filter.
## Services & Business Logic
- **Singleton**: Default.
- **Stateless**: Do not store request-specific state in class properties unless identifying as `Scope.REQUEST`.
## Pipes & Validation
- **Global**: Register `ValidationPipe` globally.
- **Route Params**: Fail fast. Always use `ParseIntPipe`, `ParseUUIDPipe` on all ID parameters.
```typescript
@Get(':id')
findOne(@Param('id', ParseIntPipe) id: number) { ... }
```
## Lifecycle Events
- **Init**: Use `OnModuleInit` for connection setup.
- **Destroy**: Use `OnApplicationShutdown` for cleanup. (Requires `enableShutdownHooks()`).
Files in this skill
- SKILL.md
- references/REFERENCE.md
- references/controller-patterns.md
- references/service-patterns.md
Attribution
Comments
Loading comments…