Skip to content

feat: add NestJS rules and skills - #30

Closed
HieuNT44 wants to merge 2 commits into
TOMOSIA-VIETNAM:mainfrom
HieuNT44:feat/nestjs-content
Closed

HieuNT44 wants to merge 2 commits into
TOMOSIA-VIETNAM:mainfrom
HieuNT44:feat/nestjs-content

Conversation

@HieuNT44

Copy link
Copy Markdown
Contributor

Thay đổi

  • Thêm nestjs vào catalog.yaml. Trước đây dự án NestJS chỉ nhận layer global, không có gì về module, dependency injection, validation hay ORM dự án đang dùng.
  • 7 rule: conventions, architecture, http, persistence, security, config, testing. Trừ conventions, mỗi rule có front matter paths: nên Claude Code chỉ nạp khi làm với file khớp. Mỗi rule kết thúc bằng mục ## Project-specific, architecture.md có mục Order of precedence.
  • 4 skill: nestjs-endpoint, nestjs-migration, nestjs-verify và nestjs-infer-conventions. Skill cuối khảo sát codebase rồi ghi convention của dự án vào AGENTS.md và mục Project-specific của rule. verify và infer có disable-model-invocation: true.
  • detect nhận @nestjs/core trong package.json. Express hoặc Fastify đơn thuần không bị gán thành nestjs.
  • tests/pipeline.test.ts: thêm test khẳng định chỉ cần --framework nestjs là chọn đủ 7 rule và 4 skill.
  • README, knowledge/README.md, AGENTS.md, CONTRIBUTING.md: catalog giờ có Rails, Laravel, FastAPI, React, Vue và NestJS.

Kiểm chứng

  • npm run check pass (191 test, coverage 100% của src/). dist/ không đổi vì không sửa src/.
  • generate --framework nestjs sinh 7 rule, 4 skill của Nest, cộng global thành 7 │ 30 │ 9, và AGENTS.md.
  • Front matter của 4 skill hợp lệ (frontmatter.mjs).
  • Các claim gắn version (NestJS 11 Express 5, NestJS 12 ESM, @Optional(), Standard Schema, validationError.target) đã đối chiếu với migration guide và release notes của Nest.

Made with Cursor

A NestJS project got only the global layer, which says nothing about
modules, dependency injection, validation, or the ORM the app uses.

Seven rules and four skills, gated on nestjs. Version-specific advice
names its release. Express or Fastify alone is not detected as NestJS.
The @optional, @global, unhandled-rejection, and shutdown-hook
sentences could be read as applying to a later release, or as a
convention the project had not chosen.
@HieuNT44
HieuNT44 force-pushed the feat/nestjs-content branch from 122e404 to f120198 Compare October 29, 2026 04:01
@HieuNT44 HieuNT44 closed this Oct 29, 2026
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.

1 participant