dont match on disabled learned patterns

apps/web/utils/ai/choose-rule/match-rules.ts

@@ -2,6 +2,7 @@ import { getConditionTypes, isAIRule } from "@/utils/condition";
 import {
   findMatchingGroup,
   getGroupsWithRules,
+  type GroupsWithRules,
 } from "@/utils/group/find-matching-group";
 import type { ParsedMessage, RuleWithActions } from "@/utils/types";
 import {
@@ -296,7 +297,7 @@ export function evaluateRuleConditions({
 
 // Lazy load learned patterns when needed
 class LearnedPatternsLoader {
-  private groups?: Awaited<ReturnType<typeof getGroupsWithRules>> | null;
+  private groups?: GroupsWithRules | null;
 
   async getGroups(emailAccountId: string) {
     if (this.groups === undefined)
@@ -496,7 +497,7 @@ export function splitEmailPatterns(pattern: string): string[] {
 
 function matchesGroupRule(
   rule: RuleWithActions,
-  groups: Awaited<ReturnType<typeof getGroupsWithRules>>,
+  groups: GroupsWithRules,
   message: ParsedMessage,
 ) {
   const ruleGroup = groups.find((g) => g.rule?.id === rule.id);

apps/web/utils/group/find-matching-group.ts

@@ -3,15 +3,20 @@ import { generalizeSubject } from "@/utils/string";
 import type { ParsedMessage } from "@/utils/types";
 import { type GroupItem, GroupItemType } from "@prisma/client";
 
-type GroupsWithRules = Awaited<ReturnType<typeof getGroupsWithRules>>;
+export type GroupsWithRules = Awaited<ReturnType<typeof getGroupsWithRules>>;
 
 export async function getGroupsWithRules({
   emailAccountId,
+  enabledOnly = true,
 }: {
   emailAccountId: string;
+  enabledOnly?: boolean;
 }) {
   return prisma.group.findMany({
-    where: { emailAccountId, rule: { isNot: null } },
+    where: {
+      emailAccountId,
+      rule: enabledOnly ? { enabled: true } : { isNot: null },
+    },
     include: { items: true, rule: { include: { actions: true } } },
   });
 }

version.txt

@@ -1 +1 @@
-v2.17.8
+v2.17.9