feat(decorators): support array of paths in @DController#801
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (20)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Adds multi-mount support to the decorators controller mounting logic so a single @DController() can expose the same controller under multiple base paths, and aligns the repo to routup ^5.2.0 accordingly.
Changes:
- Extend
@DController()/ decorator metadata to acceptstring[]in addition tostring. - Update controller mounting to mount the same controller router under each provided base path.
- Bump routup peer/dev dependency ranges to
^5.2.0, update lockfile, and document + test the new behavior.
Reviewed changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/swagger-ui/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/rate-limit/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/rate-limit-redis/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/query/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/prometheus/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/logger/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/i18n/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/docs/src/decorators/controllers.md | Documents @DController(string[]) multi-mount usage. |
| packages/docs/package.json | Adds routup ^5.2.0 as a peer dependency for the docs workspace. |
| packages/decorators/test/unit/multi-path.spec.ts | Adds unit test coverage for multi-mount controllers. |
| packages/decorators/src/type.ts | Updates DecoratorMeta.url type to `string |
| packages/decorators/src/mount.ts | Implements multi-path mounting via router building + per-path mount. |
| packages/decorators/src/class/module.ts | Updates DController signature to `string |
| packages/decorators/README.md | Notes that @DController can take string[]. |
| packages/decorators/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/cors/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/cookie/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/body/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/basic/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| packages/assets/package.json | Bumps routup peer/dev dependency to ^5.2.0. |
| package-lock.json | Updates routup to 5.2.0 and related transitive dependency versions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.