- Notifications
You must be signed in to change notification settings - Fork113
feat(api): add feed sentiment backend support#3596
Open
Daily-Dave wants to merge 2 commits intomainfrom
Open
Conversation
Implement backend infrastructure for capturing and storing user feed sentiment feedback.**Features:**- FeedSentiment entity with TypeORM- GraphQL mutation submitFeedSentiment with rate limiting- Database migration for feed_sentiment table- Foreign key constraint to user table with CASCADE delete- Indexes on userId, sentiment, and createdAt for efficient querying**Technical Details:**- Rate limited to 5 submissions per hour per user- Validates sentiment values (good, neutral, bad)- Registered in GraphQL schema with typeDefs and resolvers- Migration: 1771326220000-AddFeedSentimentTable.tsCo-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
**Critical Fixes:**- Fix User import path from './user' to './user/User' (CLAUDE.md compliance)- Use ValidationError instead of Error for GraphQL validation- Remove unnecessary comments per CLAUDE.md guidelines**Changes:**- FeedSentiment.ts: Corrected import path to avoid barrel file- feedSentiment.ts: Added ValidationError import from apollo-server-errors- feedSentiment.ts: Use ValidationError for invalid sentiment values- Removed redundant comments that restate codeCo-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
CLAassistant commentedFeb 17, 2026
|
🍹 TheUpdate (preview) fordailydotdev/api/prod (ate6b1bcd) was successful. Resource ChangesName Type Operation+ vpc-native-api-db-migration-9aabb55d kubernetes:batch/v1:Job create~ vpc-native-generate-search-invites-cron kubernetes:batch/v1:CronJob update~ vpc-native-sync-subscription-with-cio-cron kubernetes:batch/v1:CronJob update~ vpc-native-daily-digest-cron kubernetes:batch/v1:CronJob update~ vpc-native-calculate-top-readers-cron kubernetes:batch/v1:CronJob update~ vpc-native-deployment kubernetes:apps/v1:Deployment update~ vpc-native-check-analytics-report-cron kubernetes:batch/v1:CronJob update~ vpc-native-personalized-digest-deployment kubernetes:apps/v1:Deployment update~ vpc-native-update-achievement-rarity-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-highlighted-views-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-zombie-opportunities-cron kubernetes:batch/v1:CronJob update~ vpc-native-user-profile-updated-sync-cron kubernetes:batch/v1:CronJob update+ vpc-native-api-clickhouse-migration-9aabb55d kubernetes:batch/v1:Job create~ vpc-native-validate-active-users-cron kubernetes:batch/v1:CronJob update~ vpc-native-user-profile-analytics-history-clickhouse-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-tag-recommendations-cron kubernetes:batch/v1:CronJob update~ vpc-native-user-profile-analytics-clickhouse-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-zombie-images-cron kubernetes:batch/v1:CronJob update~ vpc-native-expire-super-agent-trial-cron kubernetes:batch/v1:CronJob update- vpc-native-api-clickhouse-migration-ab5b80e1 kubernetes:batch/v1:Job delete~ vpc-native-ws-deployment kubernetes:apps/v1:Deployment update~ vpc-native-personalized-digest-cron kubernetes:batch/v1:CronJob update~ vpc-native-generic-referral-reminder-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-source-public-threshold-cron kubernetes:batch/v1:CronJob update~ vpc-native-post-analytics-clickhouse-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-source-tag-view-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-zombie-user-companies-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-gifted-plus-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-trending-cron kubernetes:batch/v1:CronJob update~ vpc-native-user-posts-analytics-refresh-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-zombie-users-cron kubernetes:batch/v1:CronJob update~ vpc-native-clean-stale-user-transactions-cron kubernetes:batch/v1:CronJob update~ vpc-native-bg-deployment kubernetes:apps/v1:Deployment update~ vpc-native-update-current-streak-cron kubernetes:batch/v1:CronJob update- vpc-native-api-db-migration-ab5b80e1 kubernetes:batch/v1:Job delete~ vpc-native-update-views-cron kubernetes:batch/v1:CronJob update~ vpc-native-update-tags-str-cron kubernetes:batch/v1:CronJob update~ vpc-native-post-analytics-history-day-clickhouse-cron kubernetes:batch/v1:CronJob update~ vpc-native-hourly-notification-cron kubernetes:batch/v1:CronJob update~ vpc-native-private-deployment kubernetes:apps/v1:Deployment update~ vpc-native-temporal-deployment kubernetes:apps/v1:Deployment update... and 2 other changes |
| unknown, | ||
| BaseContext | ||
| >({ | ||
| Mutation: { |
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
this could be in existing feed schema
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
It should just be frontend mock for now I think.
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
yeah but still, no need to create whole new gql schema for this?
| export * from './campaign'; | ||
| export * from './PersonalAccessToken'; | ||
| export * from './Feedback'; | ||
| export * from './FeedSentiment'; |
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
we don't use barrel files, so this needs to be removed, its in AGENTS.md but agent ignored it...
Comment on lines +29 to +32
| const validSentiments = ['good', 'neutral', 'bad']; | ||
| if (!validSentiments.includes(sentiment)) { | ||
| throw new ValidationError('Invalid sentiment value'); | ||
| } |
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
this should be validated with zod
Comment on lines +33 to +37
| const feedSentimentRepo = ctx.con.getRepository(FeedSentiment); | ||
| await feedSentimentRepo.save({ | ||
| userId: ctx.userId, | ||
| sentiment, | ||
| }); |
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
So user can add single sentiment? since user can have multiple feeds maybe better to also add feedId, but not blocking more of a expansion opportunity
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 freeto join this conversation on GitHub. Already have an account?Sign in to comment
Labels
None yet
4 participants
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.
This PR implements backend infrastructure for capturing and storing user feed sentiment feedback to support the frontend sentiment popup feature.
Changes
submitFeedSentimentmutation with rate limiting (5 submissions/hour)feed_sentimenttable with indexes on userId, sentiment, and createdAtKey Decisions
ClosesENG-720
Created by Huginn 🐦⬛