Remove dead exec-aide-reminder Lambda (INFRA-37) #54

Merged

amoussa1229 merged 2 commits from INFRA-37-remove-dead-reminder-lambda into main

2026-06-10 18:38:20 +00:00

amoussa1229 commented

2026-06-10 17:31:13 +00:00

(Migrated from github.com)

Summary

Removes the exec-aide-reminder Lambda and all resources that exclusively supported it. The Lambda had 0 invocations in 90 days and no EventBridge rule or schedule targets it. Adam approved the deletion (INFRA-37).

Resources removed:

exec-aide-reminder Lambda function (EmailPipelineReminderAE2EAC7C)
Lambda execution role EmailPipeline/Reminder/ServiceRole and its default policy
exec-aide-reminder-scheduler IAM role (EmailPipeline/ReminderSchedulerRole) and its default policy
LogRetention custom resource for the reminder Lambda log group

Conversation Lambda updated:

Removed REMINDER_FN_ARN and REMINDER_SCHEDULER_ROLE_ARN env vars
Removed scheduler:CreateSchedule, scheduler:DeleteSchedule, and iam:PassRole grants from conversation Lambda's policy
Removed create_reminder tool definition and _handle_create_reminder handler from conversation/tools.py
Removed reminder mention from conversation/app.py system prompt

Source deleted: src/reminder/ directory (exclusively this Lambda's handler — confirmed by grep, nothing else imports it).

Validation

npx tsc --noEmit: clean, no errors
cdk synth: succeeds
cdk diff: shows only reminder-related resources destroyed ([-] for Lambda, ServiceRole, ServiceRole/DefaultPolicy, LogRetention, ReminderSchedulerRole, ReminderSchedulerRoleDefaultPolicy) and reminder env vars + policies removed from conversation Lambda. No other Lambda's configuration changed.

Tests

No automated tests exist in this repo. The stack synthesizes cleanly and diff confirms the scope is correct.

Notes

Ready for deploy-then-merge per handbook workflow. cdk deploy will be run before merging. This is a net removal of IAM roles and policies — lower-risk direction.

## Summary Removes the `exec-aide-reminder` Lambda and all resources that exclusively supported it. The Lambda had 0 invocations in 90 days and no EventBridge rule or schedule targets it. Adam approved the deletion (INFRA-37). **Resources removed:** - `exec-aide-reminder` Lambda function (`EmailPipelineReminderAE2EAC7C`) - Lambda execution role `EmailPipeline/Reminder/ServiceRole` and its default policy - `exec-aide-reminder-scheduler` IAM role (`EmailPipeline/ReminderSchedulerRole`) and its default policy - `LogRetention` custom resource for the reminder Lambda log group **Conversation Lambda updated:** - Removed `REMINDER_FN_ARN` and `REMINDER_SCHEDULER_ROLE_ARN` env vars - Removed `scheduler:CreateSchedule`, `scheduler:DeleteSchedule`, and `iam:PassRole` grants from conversation Lambda's policy - Removed `create_reminder` tool definition and `_handle_create_reminder` handler from `conversation/tools.py` - Removed reminder mention from `conversation/app.py` system prompt **Source deleted:** `src/reminder/` directory (exclusively this Lambda's handler — confirmed by grep, nothing else imports it). ## Validation - `npx tsc --noEmit`: clean, no errors - `cdk synth`: succeeds - `cdk diff`: shows only reminder-related resources destroyed (`[-]` for Lambda, ServiceRole, ServiceRole/DefaultPolicy, LogRetention, ReminderSchedulerRole, ReminderSchedulerRoleDefaultPolicy) and reminder env vars + policies removed from conversation Lambda. No other Lambda's configuration changed. ## Tests No automated tests exist in this repo. The stack synthesizes cleanly and diff confirms the scope is correct. ## Notes Ready for deploy-then-merge per handbook workflow. `cdk deploy` will be run before merging. This is a net removal of IAM roles and policies — lower-risk direction.

This repo is archived. You cannot comment on pull requests.

No reviewers