Merge branch 'main' into feat/incident-detection-implementation

waltergalvao · web-flow · commit c9d7a08b18c4 · 2026-02-27T04:49:00.000-03:00
diff --git a/apps/api/package.json b/apps/api/package.json
@@ -57,7 +57,7 @@
     "got": "^11.8.3",
     "graphql": "^16.8.1",
     "graphql-scalars": "^1.22.4",
-    "graphql-yoga": "^5.0.0",
+    "graphql-yoga": "^5.18.0",
     "hashids": "^2.3.0",
     "ioredis": "^5.9.3",
     "jsonwebtoken": "^9.0.2",
diff --git a/apps/api/src/app/pull-requests/resolvers/transformers/pull-request-tracking.transformer.ts b/apps/api/src/app/pull-requests/resolvers/transformers/pull-request-tracking.transformer.ts
@@ -53,6 +53,12 @@ export const transformPullRequestTracking = ({
     firstReviewAt: tracking.firstReviewAt?.toISOString(),
     timeToCode: tracking.timeToCode,
     firstDeployedAt: tracking.firstDeployedAt?.toISOString(),
+    timeToFirstReview: calculateTimeForEvent({
+      from: tracking.firstReadyAt || pullRequest.createdAt,
+      duration: tracking.timeToFirstReview,
+      pullRequest,
+      uselessAfterMerge: true,
+    }),
     timeToFirstApproval: calculateTimeForEvent({
       from:
         tracking.firstReviewAt ||
@@ -62,14 +68,12 @@ export const transformPullRequestTracking = ({
       pullRequest,
       uselessAfterMerge: true,
     }),
-    timeToFirstReview: calculateTimeForEvent({
-      from: tracking.firstReadyAt || pullRequest.createdAt,
-      duration: tracking.timeToFirstReview,
-      pullRequest,
-      uselessAfterMerge: true,
-    }),
     timeToMerge: calculateTimeForEvent({
-      from: tracking.firstReadyAt || pullRequest.createdAt,
+      from:
+        tracking.firstApprovalAt ||
+        tracking.firstReviewAt ||
+        tracking.firstReadyAt ||
+        pullRequest.createdAt,
       duration: tracking.timeToMerge,
       pullRequest,
       uselessAfterMerge: true,
diff --git a/apps/api/src/fastify.ts b/apps/api/src/fastify.ts
@@ -51,11 +51,21 @@ export async function buildApp(opts: FastifyServerOptions = {}) {
   app.route({
     url: yoga.graphqlEndpoint,
     method: ["GET", "POST", "OPTIONS"],
-    handler: (req, reply) =>
-      yoga.handleNodeRequestAndResponse(req, reply, {
+    handler: async (req, reply) => {
+      const response = await yoga.handleNodeRequestAndResponse(req, reply, {
         req,
         reply,
-      } as never),
+      } as never);
+
+      response.headers.forEach((value, key) => {
+        reply.header(key, value);
+      });
+
+      reply.status(response.status);
+      reply.send(response.body);
+
+      return reply;
+    },
   });
 
   setupFastifyErrorHandler(app);
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/activity/code-review-distribution/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/activity/code-review-distribution/page.tsx
@@ -83,8 +83,7 @@ export const TeamCodeReviewDistributionPage = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/code-reviews/time-to-approve/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/code-reviews/time-to-approve/page.tsx
@@ -79,8 +79,7 @@ export const TeamCodeReviewsTimeToApprovePage = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/code-reviews/time-to-first-review/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/code-reviews/time-to-first-review/page.tsx
@@ -78,8 +78,7 @@ export const TeamCodeReviewsTimeToFirstReviewPage = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/cycle-time/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/cycle-time/page.tsx
@@ -80,8 +80,7 @@ export const TeamPullRequestsCycleTimePage = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/size-distribution/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/size-distribution/page.tsx
@@ -84,8 +84,7 @@ export const TeamPullRequestsSizeDistribution = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/time-to-merge/page.tsx b/apps/web/src/app/humans/teams/[id]/health-and-performance/pull-requests/time-to-merge/page.tsx
@@ -84,8 +84,7 @@ export const TeamPullRequestsTimeToMergePage = () => {
 
                 filters.setFieldValue("from", from);
                 filters.setFieldValue("to", to);
-                searchParams.set("from", from);
-                searchParams.set("to", to);
+                searchParams.setMany({ from, to });
               }}
               value={[
                 parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/humans/teams/[id]/work-log/page.tsx b/apps/web/src/app/humans/teams/[id]/work-log/page.tsx
@@ -35,8 +35,7 @@ export const TeamWorkLogPage = () => {
 
               filters.setFieldValue("from", from);
               filters.setFieldValue("to", to);
-              searchParams.set("from", from);
-              searchParams.set("to", to);
+              searchParams.setMany({ from, to });
             }}
             value={[
               parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/metrics-and-insights/page.tsx b/apps/web/src/app/metrics-and-insights/page.tsx
@@ -81,8 +81,7 @@ export const MetricsAndInsightsPage = () => {
 
             filters.setFieldValue("from", from);
             filters.setFieldValue("to", to);
-            searchParams.set("from", from);
-            searchParams.set("to", to);
+            searchParams.setMany({ from, to });
           }}
           value={[
             parseNullableISO(filters.values.from) || null,
diff --git a/apps/web/src/app/systems/deployments/page.tsx b/apps/web/src/app/systems/deployments/page.tsx
@@ -124,8 +124,7 @@ export const DeploymentsPage = () => {
 
             filters.setFieldValue("deployedAtFrom", deployedAtFrom);
             filters.setFieldValue("deployedAtTo", deployedAtTo);
-            searchParams.set("deployedAtFrom", deployedAtFrom);
-            searchParams.set("deployedAtTo", deployedAtTo);
+            searchParams.setMany({ deployedAtFrom, deployedAtTo });
           }}
           value={[
             parseNullableISO(filters.values.deployedAtFrom) || null,
diff --git a/apps/web/src/app/systems/incidents/page.tsx b/apps/web/src/app/systems/incidents/page.tsx
@@ -159,8 +159,7 @@ export const IncidentsPage = () => {
 
             filters.setFieldValue("detectedAtFrom", detectedAtFrom);
             filters.setFieldValue("detectedAtTo", detectedAtTo);
-            searchParams.set("detectedAtFrom", detectedAtFrom);
-            searchParams.set("detectedAtTo", detectedAtTo);
+            searchParams.setMany({ detectedAtFrom, detectedAtTo });
           }}
           value={[
             parseNullableISO(filters.values.detectedAtFrom) || null,
diff --git a/apps/web/src/providers/filter.provider.ts b/apps/web/src/providers/filter.provider.ts
@@ -43,6 +43,26 @@ export const useFilterSearchParameters = () => {
         { replace: true },
       );
     },
+    setMany: (entries: Record<string, string[] | string | null>) => {
+      setSearchParams(
+        (parameters) => {
+          for (const [key, value] of Object.entries(entries)) {
+            parameters.delete(key);
+
+            if (Array.isArray(value)) {
+              for (const v of value) {
+                parameters.append(key, v);
+              }
+            } else if (value) {
+              parameters.set(key, value);
+            }
+          }
+
+          return parameters;
+        },
+        { replace: true },
+      );
+    },
     hasAny: searchParameters.entries().toArray().length > 0,
   };
 };
diff --git a/apps/web/src/providers/pull-request-list.provider.ts b/apps/web/src/providers/pull-request-list.provider.ts
@@ -128,8 +128,7 @@ export const usePullRequestList = ({
 
     filters.setFieldValue("createdAtFrom", createdAtFrom);
     filters.setFieldValue("createdAtTo", createdAtTo);
-    searchParams.set("createdAtFrom", createdAtFrom);
-    searchParams.set("createdAtTo", createdAtTo);
+    searchParams.setMany({ createdAtFrom, createdAtTo });
   };
 
   const handleCompletedAtChange = (dates: [Date | null, Date | null]) => {
@@ -138,15 +137,20 @@ export const usePullRequestList = ({
 
     filters.setFieldValue("completedAtFrom", completedAtFrom);
     filters.setFieldValue("completedAtTo", completedAtTo);
-    searchParams.set("completedAtFrom", completedAtFrom);
-    searchParams.set("completedAtTo", completedAtTo);
 
     if (completedAtFrom) {
       const selectedCompletedStates = filters.values.states.filter((state) =>
         completedStates.includes(state as PullRequestState),
       );
 
-      handleStateChange(selectedCompletedStates);
+      filters.setFieldValue("states", selectedCompletedStates as PullRequestState[]);
+      searchParams.setMany({
+        completedAtFrom,
+        completedAtTo,
+        state: selectedCompletedStates,
+      });
+    } else {
+      searchParams.setMany({ completedAtFrom, completedAtTo });
     }
   };
 
diff --git a/package-lock.json b/package-lock.json
