maple-review-bot bot updated on push a05c12f 6/10
needs attention
quality 88/100
Digest scheduling is pure, read end to end and well tested; the AI key panel now loses its only Remove path for a lapsed add-on.
tests partialrisk medium2/2 new units observable
- Digest subscribers gain send_hour plus day/hour/timezone scheduling
- isDigestDue claims a row only inside the subscriber’s local send window
- AI provider and auto-triage merge into one AI settings tab
Before merge
Migration · Check it is safe on production data and with the code still deployed
packages/db/drizzle/20261017080931_digest_send_hour/migration.sql
Findings
Warning
F2 correctness
Lapsed add-on leaves the saved OpenRouter key with no way to remove it
apps/web/src/components/settings/ai-provider-section.tsx:179-181
OpenRouterKeyPanel returns AddOnUpsell before the saved-key branch whenever the add-on lapses, so the only Remove call site is unreachable. Pass onRemove down and keep the saved-key row next to the add-on button.
Note
F1 Save is a silent no-op for a user with no primary email still open
Production impact · last 7 days
changed file calls/day
services/digest/WebAnalyticsDigestService.ts 235
Observability coverage: 2 of 2 changes observable
a05c12f
Reply “won’t fix” to dismiss a finding, or mention @maple-review-bot to ask about one.
