chore: improve monitoring for fix(ai-chat): use AGENT_NOT_FOUND instead of AGENT

Added a test case to the existing `agent-graphql-api-exception-handler.util.spec.ts` that verifies `AGENT_NOT_FOUND` maps to `ErrorCode.NOT_FOUND` (not `INTERNAL_SERVER_ERROR`). This ensures future regressions in the exception code mapping are caught in CI. After deploying the fix, "Thread not found" errors will surface as `NOT_FOUND` (404) instead of `INTERNAL_SERVER_ERROR` (500) in Sentry, which should be filtered by default alert rules since 404s are expected user behavior.
fix(ai-chat): use AGENT_NOT_FOUND instead of AGENT_EXECUTION_FAILED for missing threads
2026-04-17 16:11:10 +00:00 · 2026-04-17 16:11:09 +00:00
5 changed files with 22 additions and 7 deletions
@@ -32,4 +32,19 @@ describe('agentGraphqlApiExceptionHandler', () => {
    );
    expect(graphqlError.extensions.userFriendlyMessage).toBeDefined();
  });
+
+  it('maps AGENT_NOT_FOUND to NOT_FOUND so missing threads return 404', () => {
+    const error = new AgentException(
+      'Thread not found',
+      AgentExceptionCode.AGENT_NOT_FOUND,
+    );
+
+    const graphqlError = catchGraphqlError(error);
+
+    expect(graphqlError.extensions.code).toBe(ErrorCode.NOT_FOUND);
+    expect(graphqlError.extensions.subCode).toBe(
+      AgentExceptionCode.AGENT_NOT_FOUND,
+    );
+    expect(graphqlError.extensions.userFriendlyMessage).toBeDefined();
+  });
 });
@@ -58,7 +58,7 @@ export class AgentChatSubscriptionResolver {
    if (!isDefined(thread)) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -169,7 +169,7 @@ export class AgentChatResolver {
    if (!isDefined(thread)) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -247,7 +247,7 @@ export class AgentChatResolver {
    if (!isDefined(message)) {
      throw new AgentException(
        'Queued message not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -258,7 +258,7 @@ export class AgentChatResolver {
    if (!isDefined(thread)) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -79,7 +79,7 @@ export class AgentChatStreamingService {
    if (!thread) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -97,7 +97,7 @@ export class AgentChatService {
    if (!thread) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }

@@ -180,7 +180,7 @@ export class AgentChatService {
    if (!thread) {
      throw new AgentException(
        'Thread not found',
-        AgentExceptionCode.AGENT_EXECUTION_FAILED,
+        AgentExceptionCode.AGENT_NOT_FOUND,
      );
    }