fix: exclude archived features in segments count (#7897)

src/lib/features/segment/admin-segment.e2e.test.ts

@@ -948,4 +948,39 @@ describe('detect strategy usage in change requests', () => {
             { usedInFeatures: 0, usedInProjects: 0 },
         ]);
     });
+
+    test('If a segment is used in an archived feature it should be excluded from count - enterprise version', async () => {
+        await app.createSegment({
+            name: 'a',
+            constraints: [],
+        });
+        const flag = mockFeatureFlag();
+        await createFeatureFlag(enterpriseApp, flag);
+        const [segment] = await enterpriseFetchSegments();
+
+        await addStrategyToFeatureEnv(
+            enterpriseApp,
+            { ...flag.strategies[0] },
+            'default',
+            flag.name,
+        );
+
+        const [feature] = await fetchFeatures();
+        const [strategy] = await fetchFeatureStrategies(feature.name);
+
+        const strategyId = strategy.id;
+        await addSegmentsToStrategy([segment.id], strategyId!);
+
+        const segments = await enterpriseFetchSegments();
+        expect(segments).toMatchObject([
+            { usedInFeatures: 1, usedInProjects: 1 },
+        ]);
+
+        await enterpriseApp.archiveFeature(flag.name, 'default');
+
+        const segmentsAfter = await enterpriseFetchSegments();
+        expect(segmentsAfter).toMatchObject([
+            { usedInFeatures: 0, usedInProjects: 0 },
+        ]);
+    });
 });

src/lib/features/segment/segment-store.ts

@@ -187,7 +187,13 @@ export default class SegmentStore implements ISegmentStore {
                 T.featureStrategies,
                 `${T.featureStrategies}.id`,
                 `${T.featureStrategySegment}.feature_strategy_id`,
-            );
+            )
+            .leftJoin(
+                T.features,
+                `${T.featureStrategies}.feature_name`,
+                `${T.features}.name`,
+            )
+            .where(`${T.features}.archived_at`, null);
 
         this.mergeCurrentUsageWithCombinedData(
             combinedUsageData,