feat: promote project task comment modules and swagger to main#6
Merged
Conversation
…ng, and full test coverage - create project module with GET all and GET by id for any authenticated user, and POST PATCH DELETE restricted to admin via RolesGuard and Roles ADMIN - create task module with project-scoped task list returning all for admin and assigned only for member, GET by id with same scoping, admin-only create update delete, and assign and unassign endpoints restricted to admin - add canAccessTask shared helper for cross-module task access checks with 4 tests covering bypass, own, forbidden, and not-found cases - create comment module with task-access scoped list and get by id, task-access required create, author-only update, and author-or-admin delete - simplify unassign route from DELETE :id/assign/:userId to DELETE :id/assign removing unused param - add full swagger documentation at /docs with ApiTags, ApiOperation, ApiProperty, and bearer auth across all modules - enable bearer plugin on better-auth so login returns set-auth-token header for bearer auth support - create AllExceptionsFilter with Prisma.PrismaClientKnownRequestError guard, Logger, and consistent statusCode message data error shape - create src/common/prisma.ts barrel file replacing raw relative prisma imports across modules - reformat long single-line imports to multi-line per project style - add 74 tests across 7 suites covering project, task, comment, and shared access logic - update PLAN.md marking swagger, exception filter, negative tests, and review items complete
…ng, and full test coverage Completes the TaskForge API CRUD surface with ProjectModule restricted to admin for mutations and open for reads, TaskModule with project and member-scoped access including a shared canAccessTask helper with 4 tests, admin-only assign and unassign endpoints, CommentModule with task-access scoping, author-only update, and author-or-admin delete, full Swagger documentation at /docs with Bearer auth, the bearer() Better Auth plugin enabling token-based auth, an AllExceptionsFilter with proper Prisma error type guarding and consistent error response shape, a Prisma barrel file replacing scattered relative imports, reformatted multi-line imports, 74 tests across 7 suites, and a fully checked off PLAN.md.
Dependency ReviewThe following issues were found:
License Issuespackage.json
OpenSSF Scorecard
Scanned Files
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promotes the complete Project, Task, and Comment modules, Swagger docs, exception filter, Bearer auth, and full test suite from develop into main.
What's Changed
Why
This completes the core API surface and should be on the stable branch as TaskForge API moves from foundation to a fully functional backend.