add difficulty field to aggregate shot metrics operation

Merge pull request 'Add videoId field to playlist in ShotWithAllFeatures fragment' (#210 ) from loewy/add-video-id-to-playlist-shots-fragment into master
Reviewed-on: #210
2025-11-13 11:10:58 -08:00 · 2025-11-12 22:16:58 +00:00 · 2025-11-12 14:08:49 -08:00 · 2025-11-11 21:09:28 +00:00 · 2025-11-11 20:34:55 +00:00 · 2025-11-10 22:09:09 +00:00
7 changed files with 53 additions and 159 deletions
--- a/src/index.tsx
+++ b/src/index.tsx
@@ -3178,6 +3178,7 @@ export type UserGql = {
  following?: Maybe<Array<UserGql>>;
  id: Scalars["Int"]["output"];
  isAdmin?: Maybe<Scalars["Boolean"]["output"]>;
+  isFollowedByCurrentUser?: Maybe<Scalars["Boolean"]["output"]>;
  profileImageUri?: Maybe<Scalars["String"]["output"]>;
  stripeCustomerId?: Maybe<Scalars["String"]["output"]>;
  updatedAt?: Maybe<Scalars["DateTime"]["output"]>;
@@ -3227,16 +3228,25 @@ export type VideoFeedInputGql =
  | {
      allUsers: Scalars["Boolean"]["input"];
      followedByUserId?: never;
+      home?: never;
      userId?: never;
    }
  | {
      allUsers?: never;
      followedByUserId: Scalars["Int"]["input"];
+      home?: never;
      userId?: never;
    }
  | {
      allUsers?: never;
      followedByUserId?: never;
+      home: Scalars["Boolean"]["input"];
+      userId?: never;
+    }
+  | {
+      allUsers?: never;
+      followedByUserId?: never;
+      home?: never;
      userId: Scalars["Int"]["input"];
    };

@@ -3385,6 +3395,7 @@ export type GetAggregatedShotMetricsQuery = {
      __typename?: "TargetMetricsGQL";
      count: number;
      makePercentage?: number | null;
+      averageDifficulty?: number | null;
    };
  }>;
 };
@@ -3538,12 +3549,7 @@ export type GetFeedQuery = {
          id: number;
          username: string;
          profileImageUri?: string | null;
-          followers?: Array<{
-            __typename?: "UserGQL";
-            id: number;
-            username: string;
-            profileImageUri?: string | null;
-          }> | null;
+          isFollowedByCurrentUser?: boolean | null;
        };
      }>;
      comments: Array<{
@@ -3555,12 +3561,7 @@ export type GetFeedQuery = {
          id: number;
          username: string;
          profileImageUri?: string | null;
-          followers?: Array<{
-            __typename?: "UserGQL";
-            id: number;
-            username: string;
-            profileImageUri?: string | null;
-          }> | null;
+          isFollowedByCurrentUser?: boolean | null;
        };
        replies: Array<{
          __typename?: "CommentGQL";
@@ -3571,12 +3572,7 @@ export type GetFeedQuery = {
            id: number;
            username: string;
            profileImageUri?: string | null;
-            followers?: Array<{
-              __typename?: "UserGQL";
-              id: number;
-              username: string;
-              profileImageUri?: string | null;
-            }> | null;
+            isFollowedByCurrentUser?: boolean | null;
          };
        }>;
      }>;
@@ -3594,12 +3590,7 @@ export type UserSocialsFieldsFragment = {
  id: number;
  username: string;
  profileImageUri?: string | null;
-  followers?: Array<{
-    __typename?: "UserGQL";
-    id: number;
-    username: string;
-    profileImageUri?: string | null;
-  }> | null;
+  isFollowedByCurrentUser?: boolean | null;
 };

 export type VideoCardFieldsFragment = {
@@ -3656,12 +3647,7 @@ export type VideoCardFieldsFragment = {
      id: number;
      username: string;
      profileImageUri?: string | null;
-      followers?: Array<{
-        __typename?: "UserGQL";
-        id: number;
-        username: string;
-        profileImageUri?: string | null;
-      }> | null;
+      isFollowedByCurrentUser?: boolean | null;
    };
  }>;
  comments: Array<{
@@ -3673,12 +3659,7 @@ export type VideoCardFieldsFragment = {
      id: number;
      username: string;
      profileImageUri?: string | null;
-      followers?: Array<{
-        __typename?: "UserGQL";
-        id: number;
-        username: string;
-        profileImageUri?: string | null;
-      }> | null;
+      isFollowedByCurrentUser?: boolean | null;
    };
    replies: Array<{
      __typename?: "CommentGQL";
@@ -3689,12 +3670,7 @@ export type VideoCardFieldsFragment = {
        id: number;
        username: string;
        profileImageUri?: string | null;
-        followers?: Array<{
-          __typename?: "UserGQL";
-          id: number;
-          username: string;
-          profileImageUri?: string | null;
-        }> | null;
+        isFollowedByCurrentUser?: boolean | null;
      };
    }>;
  }>;
@@ -3713,6 +3689,7 @@ export type GetVideoFeedQuery = {
  __typename?: "Query";
  getFeedVideos: {
    __typename?: "VideoHistoryGQL";
+    hasFollowing: boolean;
    videos: Array<{
      __typename?: "VideoGQL";
      id: number;
@@ -3770,12 +3747,7 @@ export type GetVideoFeedQuery = {
          id: number;
          username: string;
          profileImageUri?: string | null;
-          followers?: Array<{
-            __typename?: "UserGQL";
-            id: number;
-            username: string;
-            profileImageUri?: string | null;
-          }> | null;
+          isFollowedByCurrentUser?: boolean | null;
        };
      }>;
      comments: Array<{
@@ -3787,12 +3759,7 @@ export type GetVideoFeedQuery = {
          id: number;
          username: string;
          profileImageUri?: string | null;
-          followers?: Array<{
-            __typename?: "UserGQL";
-            id: number;
-            username: string;
-            profileImageUri?: string | null;
-          }> | null;
+          isFollowedByCurrentUser?: boolean | null;
        };
        replies: Array<{
          __typename?: "CommentGQL";
@@ -3803,12 +3770,7 @@ export type GetVideoFeedQuery = {
            id: number;
            username: string;
            profileImageUri?: string | null;
-            followers?: Array<{
-              __typename?: "UserGQL";
-              id: number;
-              username: string;
-              profileImageUri?: string | null;
-            }> | null;
+            isFollowedByCurrentUser?: boolean | null;
          };
        }>;
      }>;
@@ -4491,6 +4453,7 @@ export type GetShotsWithMetadataFilterResultQuery = {
        } | null;
        playlist?: {
          __typename?: "HLSPlaylistGQL";
+          videoId: number;
          segmentDurations: Array<number>;
        } | null;
      } | null;
@@ -4573,6 +4536,7 @@ export type GetShotsWithMetadataQuery = {
        } | null;
        playlist?: {
          __typename?: "HLSPlaylistGQL";
+          videoId: number;
          segmentDurations: Array<number>;
        } | null;
      } | null;
@@ -4644,6 +4608,7 @@ export type GetShotsByIdsQuery = {
      } | null;
      playlist?: {
        __typename?: "HLSPlaylistGQL";
+        videoId: number;
        segmentDurations: Array<number>;
      } | null;
    } | null;
@@ -4708,6 +4673,7 @@ export type ShotWithAllFeaturesFragment = {
    } | null;
    playlist?: {
      __typename?: "HLSPlaylistGQL";
+      videoId: number;
      segmentDurations: Array<number>;
    } | null;
  } | null;
@@ -4789,6 +4755,7 @@ export type EditShotMutation = {
        } | null;
        playlist?: {
          __typename?: "HLSPlaylistGQL";
+          videoId: number;
          segmentDurations: Array<number>;
        } | null;
      } | null;
@@ -4975,21 +4942,7 @@ export type FollowUserMutationVariables = Exact<{

 export type FollowUserMutation = {
  __typename?: "Mutation";
-  followUser: {
-    __typename?: "UserGQL";
-    username: string;
-    id: number;
-    following?: Array<{
-      __typename?: "UserGQL";
-      id: number;
-      username: string;
-    }> | null;
-    followers?: Array<{
-      __typename?: "UserGQL";
-      id: number;
-      username: string;
-    }> | null;
-  };
+  followUser: { __typename?: "UserGQL"; id: number; username: string };
 };

 export type UnfollowUserMutationVariables = Exact<{
@@ -4998,21 +4951,7 @@ export type UnfollowUserMutationVariables = Exact<{

 export type UnfollowUserMutation = {
  __typename?: "Mutation";
-  unfollowUser: {
-    __typename?: "UserGQL";
-    username: string;
-    id: number;
-    following?: Array<{
-      __typename?: "UserGQL";
-      id: number;
-      username: string;
-    }> | null;
-    followers?: Array<{
-      __typename?: "UserGQL";
-      id: number;
-      username: string;
-    }> | null;
-  };
+  unfollowUser: { __typename?: "UserGQL"; id: number; username: string };
 };

 export type GetUserFollowingFollowersQueryVariables = Exact<{
@@ -5029,12 +4968,14 @@ export type GetUserFollowingFollowersQuery = {
      id: number;
      username: string;
      profileImageUri?: string | null;
+      isFollowedByCurrentUser?: boolean | null;
    }> | null;
    followers?: Array<{
      __typename?: "UserGQL";
      id: number;
      username: string;
      profileImageUri?: string | null;
+      isFollowedByCurrentUser?: boolean | null;
    }> | null;
  } | null;
 };
@@ -5262,12 +5203,7 @@ export type GetVideoSocialDetailsByIdQuery = {
        id: number;
        username: string;
        profileImageUri?: string | null;
-        followers?: Array<{
-          __typename?: "UserGQL";
-          id: number;
-          username: string;
-          profileImageUri?: string | null;
-        }> | null;
+        isFollowedByCurrentUser?: boolean | null;
      };
    }>;
    comments: Array<{
@@ -5279,12 +5215,7 @@ export type GetVideoSocialDetailsByIdQuery = {
        id: number;
        username: string;
        profileImageUri?: string | null;
-        followers?: Array<{
-          __typename?: "UserGQL";
-          id: number;
-          username: string;
-          profileImageUri?: string | null;
-        }> | null;
+        isFollowedByCurrentUser?: boolean | null;
      };
      replies: Array<{
        __typename?: "CommentGQL";
@@ -5295,12 +5226,7 @@ export type GetVideoSocialDetailsByIdQuery = {
          id: number;
          username: string;
          profileImageUri?: string | null;
-          followers?: Array<{
-            __typename?: "UserGQL";
-            id: number;
-            username: string;
-            profileImageUri?: string | null;
-          }> | null;
+          isFollowedByCurrentUser?: boolean | null;
        };
      }>;
    }>;
@@ -5888,11 +5814,7 @@ export const UserSocialsFieldsFragmentDoc = gql`
    id
    username
    profileImageUri
-    followers {
-      id
-      username
-      profileImageUri
-    }
+    isFollowedByCurrentUser
  }
 `;
 export const VideoCardFieldsFragmentDoc = gql`
@@ -6067,6 +5989,7 @@ export const ShotWithAllFeaturesFragmentDoc = gql`
        streamSegmentType
      }
      playlist {
+        videoId
        segmentDurations
      }
    }
@@ -6225,6 +6148,7 @@ export const GetAggregatedShotMetricsDocument = gql`
      targetMetrics {
        count
        makePercentage
+        averageDifficulty
      }
    }
  }
@@ -6797,6 +6721,7 @@ export const GetVideoFeedDocument = gql`
        hasNextPage
        endCursor
      }
+      hasFollowing
    }
  }
  ${VideoCardFieldsFragmentDoc}
@@ -9571,17 +9496,9 @@ export type GetUserTagsQueryResult = Apollo.QueryResult<
 export const FollowUserDocument = gql`
  mutation followUser($followedUserId: Int!) {
    followUser(followedUserId: $followedUserId) {
-      username
-      id
-      following {
      id
      username
    }
-      followers {
-        id
-        username
-      }
-    }
  }
 `;
 export type FollowUserMutationFn = Apollo.MutationFunction<
@@ -9630,17 +9547,9 @@ export type FollowUserMutationOptions = Apollo.BaseMutationOptions<
 export const UnfollowUserDocument = gql`
  mutation unfollowUser($followedUserId: Int!) {
    unfollowUser(followedUserId: $followedUserId) {
-      username
-      id
-      following {
      id
      username
    }
-      followers {
-        id
-        username
-      }
-    }
  }
 `;
 export type UnfollowUserMutationFn = Apollo.MutationFunction<
@@ -9694,11 +9603,13 @@ export const GetUserFollowingFollowersDocument = gql`
        id
        username
        profileImageUri
+        isFollowedByCurrentUser
      }
      followers {
        id
        username
        profileImageUri
+        isFollowedByCurrentUser
      }
    }
  }
--- a/src/operations/aggregate.gql
+++ b/src/operations/aggregate.gql
@@ -7,6 +7,7 @@ query GetAggregatedShotMetrics($aggregateInput: AggregateInputGQL!) {
    targetMetrics {
      count
      makePercentage
+      averageDifficulty
    }
  }
 }
--- a/src/operations/feed.gql
+++ b/src/operations/feed.gql
@@ -19,11 +19,7 @@ fragment UserSocialsFields on UserGQL {
  id
  username
  profileImageUri
-  followers {
-    id
-    username
-    profileImageUri
-  }
+  isFollowedByCurrentUser
 }

 fragment VideoCardFields on VideoGQL {
@@ -116,5 +112,6 @@ query GetVideoFeed(
      hasNextPage
      endCursor
    }
+    hasFollowing
  }
 }
--- a/src/operations/shots.gql
+++ b/src/operations/shots.gql
@@ -192,6 +192,7 @@ fragment ShotWithAllFeatures on ShotGQL {
      streamSegmentType
    }
    playlist {
+      videoId
      segmentDurations
    }
  }
--- a/src/operations/user.gql
+++ b/src/operations/user.gql
@@ -88,32 +88,16 @@ query GetUserTags {

 mutation followUser($followedUserId: Int!) {
  followUser(followedUserId: $followedUserId) {
-    username
-    id
-    following {
    id
    username
  }
-    followers {
-      id
-      username
-    }
-  }
 }

 mutation unfollowUser($followedUserId: Int!) {
  unfollowUser(followedUserId: $followedUserId) {
-    username
-    id
-    following {
    id
    username
  }
-    followers {
-      id
-      username
-    }
-  }
 }

 query getUserFollowingFollowers {
@@ -123,11 +107,13 @@ query getUserFollowingFollowers {
      id
      username
      profileImageUri
+      isFollowedByCurrentUser
    }
    followers {
      id
      username
      profileImageUri
+      isFollowedByCurrentUser
    }
  }
 }
--- a/src/operations/video.gql
+++ b/src/operations/video.gql
@@ -88,11 +88,7 @@ fragment UserSocialsFields on UserGQL {
  id
  username
  profileImageUri
-  followers {
-    id
-    username
-    profileImageUri
-  }
+  isFollowedByCurrentUser
 }

 query GetVideoSocialDetailsById($videoId: Int!) {
--- a/src/schema.gql
+++ b/src/schema.gql
@@ -375,6 +375,7 @@ type UserGQL {
  agreesToMarketing: Boolean
  following: [UserGQL!]
  followers: [UserGQL!]
+  isFollowedByCurrentUser: Boolean
 }

 type ShotGQL {
@@ -654,6 +655,7 @@ input VideoFeedInputGQL @oneOf {
  followedByUserId: Int
  userId: Int
  allUsers: Boolean
+  home: Boolean
 }

 type MakePercentageIntervalGQL {
Author	SHA1	Message	Date
Loewy	beeba9207c	add difficulty field to aggregate shot metrics operation All checks were successful Tests / Tests (pull_request) Successful in 22s Details	2025-11-13 11:10:58 -08:00
loewy	f14e117416	Merge pull request 'Add videoId field to playlist in ShotWithAllFeatures fragment' (#210 ) from loewy/add-video-id-to-playlist-shots-fragment into master Reviewed-on: #210	2025-11-12 22:16:58 +00:00
Loewy	3d86c13291	add videoId field to playlist in shot w features fragment All checks were successful Tests / Tests (pull_request) Successful in 10s Details	2025-11-12 14:08:49 -08:00
loewy	e63f8600aa	Merge pull request 'Add pocket size field to GraphQL schema and operations' (#209 ) from dean/add-pocket-size-clean into master Reviewed-on: #209	2025-11-11 21:09:28 +00:00
loewy	7e822446da	Merge branch 'master' into dean/add-pocket-size-clean All checks were successful Tests / Tests (pull_request) Successful in 9s Details	2025-11-11 20:34:55 +00:00
loewy	75ecc5894b	Merge pull request 'dean/add-home-feed-option' (#208 ) from dean/add-home-feed-option into master Reviewed-on: #208	2025-11-10 22:09:09 +00:00
loewy	73c8dd5f57	Merge branch 'master' into dean/add-home-feed-option All checks were successful Tests / Tests (pull_request) Successful in 10s Details	2025-11-10 19:55:16 +00:00
dean	e4223a1a17	Add isFollowedByCurrentUser field and hasFollowing to feed query All checks were successful Tests / Tests (pull_request) Successful in 9s Details Updated GraphQL operations to support efficient follow status checking: - Added isFollowedByCurrentUser field to UserSocialsFields fragment - Removed nested followers array from UserSocialsFields (over-fetching) - Simplified followUser/unfollowUser mutations to return minimal data - Added hasFollowing field to GetVideoFeed query for feed mode detection - Updated getUserFollowingFollowers query to include isFollowedByCurrentUser These changes enable the mobile app to: - Display correct follow/unfollow button states without client-side lookups - Differentiate between "Following" and "Popular" feed modes - Reduce payload size by removing unnecessary nested data Backend handles efficient resolution via request-scoped caching. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-11-09 23:44:39 -08:00
dean	83b7f0d0f6	Add home feed and isFollowedByCurrentUser field - Add home option to VideoFeedInputGQL for automatic feed selection - Add hasFollowing field to VideoHistoryGQL response - Add isFollowedByCurrentUser field to UserGQL to replace followers N+1 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-11-08 09:15:39 -08:00
dean	a882f98d91	Add home option to VideoFeedInputGQL The home feed option enables smart feed selection on the backend: - If user has following: returns FOLLOWING feed - If user has no following: returns ALL feed - Always includes hasFollowing flag so frontend knows which feed it got This allows frontend to make a single query instead of needing to check following status separately.	2025-11-07 12:16:01 -08:00