chore: decommission the Amplitude analytics-summary cron and its table - #2015
#2015Open
paustint wants to merge 1 commit into
mainjetstreamapp/jetstream:mainfrom
chore/decommission-analytics-summaryjetstreamapp/jetstream:chore/decommission-analytics-summaryCopy head branch name to clipboard
Open
chore: decommission the Amplitude analytics-summary cron and its table#2015paustint wants to merge 1 commit intomainjetstreamapp/jetstream:mainfrom chore/decommission-analytics-summaryjetstreamapp/jetstream:chore/decommission-analytics-summaryCopy head branch name to clipboard
paustint wants to merge 1 commit into
mainjetstreamapp/jetstream:mainfrom
chore/decommission-analytics-summaryjetstreamapp/jetstream:chore/decommission-analytics-summaryCopy head branch name to clipboard
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR decommissions the legacy Amplitude “analytics summary” cron pipeline and removes its persisted AnalyticsSummary database table, which no longer has any consumers and has been failing due to Amplitude retention-window constraints.
Changes:
- Removes the
AnalyticsSummaryPrisma model and adds a migration to drop the backing table. - Deletes the cron task entrypoint and Amplitude dashboard API client/types used to populate the summary.
- Removes landing/types artifacts related to rendering the analytics summary.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| prisma/schema.prisma | Removes the AnalyticsSummary model from the Prisma schema. |
| prisma/migrations/20260827142303_remove_analytics_summary/migration.sql | Drops the AnalyticsSummary table (but currently also drops an unrelated user email index). |
| libs/types/src/lib/types.ts | Removes the AnalyticStat interface that was used by the landing analytics summary. |
| apps/landing/components/landing/AnalyticsSummary.tsx | Deletes the landing-page component that rendered analytics summary stats. |
| apps/cron-tasks/src/utils/types.ts | Deletes Amplitude chart response typing used by the removed cron task. |
| apps/cron-tasks/src/utils/amplitude-dashboard-api.ts | Deletes the Amplitude dashboard API client wrapper. |
| apps/cron-tasks/src/save-analytics-summary.ts | Deletes the cron task that upserted AnalyticsSummary rows. |
| apps/cron-tasks/src/config/env-config.ts | Removes Amplitude credential env vars from cron-tasks ENV mapping. |
| apps/cron-tasks/project.json | Removes the analytics-summary entrypoint from the cron-tasks build outputs. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
paustint
force-pushed
the
chore/decommission-analytics-summary
branch
from
August 28, 2026 12:58
337e9ec to
c3aac77
Compare
The cron fed homepage usage stats that were removed from the landing page long ago; nothing reads the AnalyticsSummary table anymore. It had also been failing daily since at least May: its YEAR charts query a month- snapped 12-month window, which Amplitude's rolling 365-day retention cutoff rejects on every day except the 1st-2nd of each month.
paustint
force-pushed
the
chore/decommission-analytics-summary
branch
from
August 28, 2026 12:59
c3aac77 to
307ed8b
Compare
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
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.
The cron fed homepage usage stats that were removed from the landing page long ago; nothing reads the AnalyticsSummary table anymore. It had also been failing daily since at least May: its YEAR charts query a month- snapped 12-month window, which Amplitude's rolling 365-day retention cutoff rejects on every day except the 1st-2nd of each month.