Skip to content

feat(nestjs): add NestJS enterprise patterns skill#5

Open
Andiveli wants to merge 1 commit intoGentleman-Programming:mainfrom
Andiveli:feat/nestjs-skill
Open

feat(nestjs): add NestJS enterprise patterns skill#5
Andiveli wants to merge 1 commit intoGentleman-Programming:mainfrom
Andiveli:feat/nestjs-skill

Conversation

@Andiveli
Copy link
Copy Markdown

Summary

  • Add comprehensive NestJS skill with enterprise patterns
  • Include strict TypeScript rules and SOLID principles
  • Cover controllers, services, DTOs, repositories patterns
  • Add testing patterns and anti-patterns

Changes

  • Created curated/nestjs/SKILL.md with complete guidelines
  • Includes architecture flow: Controllers → Services → Repositories → Database
  • Added validation patterns, error handling, and security rules
  • Included code examples and testing requirements

Testing

  • Validated skill follows template format
  • Checked YAML frontmatter is correct
  • Verified all required sections are present
  • Manual testing complete

Features

  • TypeScript strict mode enforcement (ZERO exceptions)
  • Complete CRUD patterns with validation
  • Enterprise project structure
  • Comprehensive testing strategies
  • Security best practices

Keywords

nestjs, typescript, enterprise, patterns, architecture, testing, validation

@Alan-TheGentleman
Copy link
Copy Markdown
Contributor

Review Summary

Hey @Andiveli! Thanks for the NestJS skill - the content looks solid. However, there are some issues that need fixing:

1. Wrong Directory (Security Check will fail)

You're adding to curated/nestjs/SKILL.md but community contributions must go in the community/ folder:

❌ curated/nestjs/SKILL.md      # Maintainers only
✅ community/nestjs/SKILL.md    # Community contributions

2. PR Title Format

For community skills, please use this title format:

❌ feat(nestjs): add NestJS enterprise patterns skill
✅ [Community Skill] NestJS Enterprise Patterns

3. Missing Labels

The community-skill and voting labels weren't added because:

  1. Wrong directory (not in community/)
  2. Wrong title format (no [Community Skill] prefix)

How to Fix

  1. Move your file: community/nestjs/SKILL.md
  2. Update PR title to: [Community Skill] NestJS Enterprise Patterns
  3. Push the changes

What's Good

  • Great content structure
  • Proper conventional commit
  • Excellent code examples and anti-patterns

Looking forward to the updated PR!

@gudo09
Copy link
Copy Markdown

gudo09 commented Apr 15, 2026

Hey @Andiveli, this looks really solid 👏

The PR is failing due to repository rules (not the content). To fix it:

  1. Update the PR title:
    [Community Skill] NestJS Enterprise Patterns

  2. Move the file to the correct path:
    community/nestjs/SKILL.md

    (Currently it's in curated/, which is restricted to maintainers only.)

After these changes, the CI checks should pass and the PR will enter the voting process 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants