fix(chat): default access request list to pending

The settings tool parses list input before calling the reader, so an omitted
status reached it as undefined and listed every status. The list schema now
defaults status to pending, matching get; explicit statuses still apply.

Author: waleedlatif1

apps/sim/ee/access-requests/lib/application/copilot-review.integration.ts

@@ -254,6 +254,9 @@ describe('organization Chat review of access requests', () => {
     expect(await readGroupConfig()).toMatchObject({ hideTablesTab: false, hideFilesTab: true })
     expect(await decidedEvents(requestIds.tables)).toHaveLength(1)
     await waitForAudit(requestIds.tables, 'permission_access_request.fulfilled')
+
+    const { result: pending } = await asAdmin('list', {})
+    expect((pending.requests as { id: string }[]).map(({ id }) => id)).toEqual([requestIds.files])
   })
 
   it("lets an admin's Chat deny a request with a reason, changing no policy", async () => {

apps/sim/lib/api/contracts/access-requests.ts

@@ -30,7 +30,7 @@ export const ACCESS_REQUEST_STATUSES = [
   'closed',
 ] as const
 
-export const ACCESS_REQUEST_PAGE_SIZE = 50
+const ACCESS_REQUEST_PAGE_SIZE = 50
 export const ACCESS_REQUEST_MAX_PAGE_SIZE = 100
 
 const requestIdSchema = z

apps/sim/lib/mothership/tools/server/settings-access-requests.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod'
 import {
-  ACCESS_REQUEST_PAGE_SIZE,
+  ACCESS_REQUEST_STATUSES,
   accessRequestParamsSchema,
   listOrganizationAccessRequestsQuerySchema,
 } from '@/lib/api/contracts/access-requests'
@@ -19,15 +19,17 @@ import type { ResolveAccessRequestDecision } from '@/ee/access-requests/lib/type
 
 const requestTarget = { requestId: accessRequestParamsSchema.shape.requestId }
 const [applyDecisionSchema, declineDecisionSchema] = v2ResolveAccessRequestBodySchema.options
+const listQuerySchema = listOrganizationAccessRequestsQuerySchema.extend({
+  status: z
+    .enum(ACCESS_REQUEST_STATUSES)
+    .default('pending')
+    .describe('Request status to list; defaults to pending.'),
+})
 
 /** The first page of pending requests, or the page `query` selects. */
 export function readSettingsAccessRequests(
   context: SettingsContext,
-  query: z.output<typeof listOrganizationAccessRequestsQuerySchema> = {
-    status: 'pending',
-    limit: ACCESS_REQUEST_PAGE_SIZE,
-    offset: 0,
-  }
+  query: z.output<typeof listQuerySchema> = listQuerySchema.parse({})
 ) {
   return listOrganizationAccessRequests.execute({
     principal: context.principal,
@@ -49,11 +51,7 @@ async function resolve(
 
 /** Admin role, fingerprint freshness, grants, audit and the requester's notice stay in the use cases. */
 export const accessRequestSettingsActions = {
-  list: settingsOperation(
-    'read',
-    listOrganizationAccessRequestsQuerySchema,
-    readSettingsAccessRequests
-  ),
+  list: settingsOperation('read', listQuerySchema, readSettingsAccessRequests),
   preview: settingsOperation('read', z.strictObject(requestTarget), (context, input) =>
     previewAccessRequest.execute({
       principal: context.principal,