fix(e2e): harden workflow readiness invariants

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: 2 people

apps/sim/app/workspace/[workspaceId]/settings/components/team-management/team-management.tsx

@@ -42,8 +42,8 @@ export function TeamManagement({
   const { data: session, isPending: isSessionPending, error: sessionError } = useSession()
   const {
     isInvitationsDisabled,
-    isLoading: isPermissionConfigLoading,
-    isError: isPermissionConfigError,
+    isPermissionLoading: isPermissionConfigLoading,
+    isPermissionError: isPermissionConfigError,
   } = usePermissionConfig()
 
   const { data: userSubscriptionData } = useSubscriptionData()
@@ -73,14 +73,14 @@ export function TeamManagement({
   const hasLoadError =
     Boolean(sessionError) ||
     isOrganizationError ||
-    isOrganizationBillingError ||
+    (adminOrOwner && isOrganizationBillingError) ||
     isRosterError ||
     isPermissionConfigError
   const isMembersLoading =
     !hasLoadError &&
     (isSessionPending ||
       isLoading ||
-      isOrgBillingLoading ||
+      (adminOrOwner && isOrgBillingLoading) ||
       isLoadingRoster ||
       isPermissionConfigLoading)
   const dataState = hasLoadError ? 'error' : isMembersLoading ? 'loading' : 'ready'

apps/sim/app/workspace/[workspaceId]/settings/components/teammates/teammates.tsx

@@ -98,8 +98,8 @@ export function Teammates() {
   const queryClient = useQueryClient()
   const {
     isInvitationsDisabled: isInvitationsDisabledByConfig,
-    isLoading: permissionConfigLoading,
-    isError: permissionConfigError,
+    isPermissionLoading: permissionConfigLoading,
+    isPermissionError: permissionConfigError,
   } = usePermissionConfig()
 
   const resendInvitation = useResendWorkspaceInvitation()

apps/sim/e2e/scripts/seed-world.ts

@@ -641,13 +641,18 @@ async function assertTrustedWorldInvariants(world: E2EWorld): Promise<void> {
       throw new Error(`Persisted invitation grants do not match scenario: ${definition.key}`)
     }
   }
-  await assertWorkflowPersonaInvariants(world)
+  if (world.scenario.definition.namespace.world === 'settings-primary') {
+    await assertWorkflowPersonaInvariants(world)
+  }
 }
 
 async function assertWorkflowPersonaInvariants(world: E2EWorld): Promise<void> {
-  const teamTarget = world.records.users.get('team-workflow-member')
-  const enterpriseTarget = world.records.users.get('enterprise-workflow-member')
-  if (!teamTarget || !enterpriseTarget) return
+  const teamTarget = required(world.records.users, 'team-workflow-member', 'Team workflow member')
+  const enterpriseTarget = required(
+    world.records.users,
+    'enterprise-workflow-member',
+    'Enterprise workflow member'
+  )
 
   const teamOrganization = required(
     world.records.organizations,

apps/sim/hooks/use-permission-config.ts

@@ -18,6 +18,9 @@ export interface PermissionConfigResult {
   config: PermissionGroupConfig
   isLoading: boolean
   isError: boolean
+  /** Permission-group policy state only; excludes the independent integration allowlist query. */
+  isPermissionLoading: boolean
+  isPermissionError: boolean
   isInPermissionGroup: boolean
   filterBlocks: <T extends { type: string }>(blocks: T[]) => T[]
   filterProviders: (providerIds: string[]) => string[]
@@ -167,6 +170,8 @@ export function usePermissionConfig(): PermissionConfigResult {
       config: mergedConfig,
       isLoading,
       isError,
+      isPermissionLoading,
+      isPermissionError,
       isInPermissionGroup,
       filterBlocks,
       filterProviders,
@@ -181,6 +186,8 @@ export function usePermissionConfig(): PermissionConfigResult {
       mergedConfig,
       isLoading,
       isError,
+      isPermissionLoading,
+      isPermissionError,
       isInPermissionGroup,
       filterBlocks,
       filterProviders,