Conversation
* feat: add internal endpoint for task status change notifications Adds POST /v1/internal/tasks/notify-status-change so enterprise-api can delegate email + in-app notifications to the comp API instead of duplicating Resend infrastructure. Makes changedByUserId optional in TaskNotifierService.notifyStatusChange() with 'Automation' fallback. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: add validation decorators to DTO and return 500 on failure Adds @IsString() and @IsEnum() decorators so the global ValidationPipe doesn't reject all requests. Throws InternalServerErrorException instead of returning 200 with { success: false } on notification failures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: update openapi.json with validated DTO schema Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: add automation failure notifications for partial failures Notify users when ANY automation fails (not just when all fail). New email template, service method, and internal endpoint for automation-specific failure notifications. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: use Object.values(TaskStatus) instead of hardcoded array Prevents stale validation when new statuses are added to the schema. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
PR SummaryMedium Risk Overview
Written by Cursor Bugbot for commit 97cebf8. This will update automatically on new commits. Configure here. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| ); | ||
|
|
||
| const organizationName = organization?.name ?? 'your organization'; | ||
| const changedByName = 'Automation'; |
There was a problem hiding this comment.
Unused variable changedByName in new method
Low Severity
The variable changedByName is assigned the value 'Automation' inside notifyAutomationFailures but is never read anywhere in the method. Neither the AutomationFailuresEmail template props nor the Novu in-app notification payload reference it. This is dead code, likely left over from copy-pasting the notifyStatusChange pattern.
1 participant
This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.