Merge pull request #876 from Rahul-R79/chore/fix-unused-variables

chore: remove unused variables and imports

Author: brunobuddy

.changeset/fix-unused-variables.md

@@ -0,0 +1,5 @@
+---
+"manifest": patch
+---
+
+Remove unused variables

packages/backend/src/analytics/services/agent-analytics.service.spec.ts

@@ -7,14 +7,12 @@ describe('AgentAnalyticsService', () => {
   let service: AgentAnalyticsService;
   let mockGetRawOne: jest.Mock;
   let mockGetRawMany: jest.Mock;
-  let qbCallIndex: number;
 
   const scope = { tenantId: 't1', agentId: 'a1' };
 
   beforeEach(async () => {
     mockGetRawOne = jest.fn();
     mockGetRawMany = jest.fn();
-    qbCallIndex = 0;
 
     const mockQb = {
       select: jest.fn().mockReturnThis(),

packages/backend/src/auth/auth.instance.ts

@@ -31,7 +31,6 @@ if (!isLocalMode && nodeEnv !== 'test' && (!betterAuthSecret || betterAuthSecret
   throw new Error('BETTER_AUTH_SECRET must be set to a value of at least 32 characters');
 }
 
-import { getLocalAuthSecret } from '../common/constants/local-mode.constants';
 
 function buildTrustedOrigins(): string[] {
   const origins: string[] = [];

packages/backend/src/auth/local-auth.guard.spec.ts

@@ -7,7 +7,6 @@ jest.mock('../common/constants/local-mode.constants', () => ({
 import { ExecutionContext } from '@nestjs/common';
 import { Reflector } from '@nestjs/core';
 import { LocalAuthGuard } from './local-auth.guard';
-import { IS_PUBLIC_KEY } from '../common/decorators/public.decorator';
 import { readLocalNotificationEmail } from '../common/constants/local-mode.constants';
 
 function createMockContext(overrides: {

packages/backend/src/notifications/services/notification-rules.service.spec.ts

@@ -511,7 +511,7 @@ describe('NotificationRulesService (sql.js / local mode)', () => {
 
     await service.deleteRule('user-1', 'r1');
 
-    const deleteCall = mockQuery.mock.calls[2]; // verifyOwnership(2 queries) then DELETE
+    // verifyOwnership(2 queries) then DELETE
     // The DELETE should use ? placeholder
     const allCalls = mockQuery.mock.calls.map((c: unknown[]) => c[0] as string);
     const deleteQuery = allCalls.find((sql) => sql.includes('DELETE'));