Feature/collect

[LucienShui]

Route under auth.controller.ts changed but not verified.

[Copilot]

Pull Request Overview

This PR introduces a new "collect" feature and refactors task handling while renaming endpoints and controllers to improve consistency. Key changes include the implementation of a new collect method in the tasks service, the migration and renaming of internal task controllers, and several updates across authentication, mail, and namespace modules to support the new functionality.

Reviewed Changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/utils/each.ts	Minor formatting update with a trailing comma in the callback.
src/tasks/tasks.service.ts	Added new collect, taskDoneCallback, raw SQL query for task fetching, and removed deprecated callback.
src/tasks/tasks.module.ts	Updated imports to reference renamed internal tasks controller and added ResourcesModule.
src/tasks/tasks.entity.ts	Changed default priority and removed concurrencyThreshold column.
src/tasks/tasks.controller.ts	Added new /collect endpoint for handling collect requests.
src/tasks/internal.tasks.controller.ts	Introduced new internal controller for tasks callbacks and fetching.
src/tasks/internal-tasks.controller.ts	Removed legacy controller.
src/tasks/dto/*.ts	Added new DTOs for task callback and collect request validation.
src/resources/dto/update-resource.dto.ts	Updated validation imports.
src/namespaces/*.ts	Added findByName service method and new columns in the namespace entity.
src/mail/mail.service.ts	Renamed email methods and updated email templates.
src/auth/*.ts	Added internal auth controller and renamed endpoints, updating signup flows.
compose.yaml	Updated port mapping to use an environment variable.

Files not reviewed (1)

• .env.example: Language not supported

Comments suppressed due to low confidence (2)

src/tasks/tasks.service.ts:59

• [nitpick] Using the property name 'function' may be confusing due to its reserved status in JavaScript. Consider renaming it to a more descriptive name such as 'taskType' to improve clarity.

      function: 'collect',

src/tasks/tasks.module.ts:6

• [nitpick] The renamed file 'internal.tasks.controller.ts' deviates from the previous kebab-case convention ('internal-tasks.controller.ts'). Please ensure that the file naming is consistent with the project's established practices.

import { InternalTasksController } from 'src/tasks/internal.tasks.controller';

[hewenguang]

/lgtm