Expensify/App项目中的Workspace管理员移除问题分析

问题概述

在Expensify/App项目中，发现了一个关于Workspace管理员移除功能的缺陷。当从Workspace中移除具有管理员或审计员角色的成员时，系统未能正确地从管理员房间中移除这些成员。这意味着即使这些成员已被移出Workspace，他们仍然会显示在管理员房间的成员列表中。

技术背景

在Expensify/App的架构设计中，Workspace成员管理是一个核心功能。当用户被授予管理员权限时，系统会自动将他们添加到专门的管理员房间。这个房间用于管理员之间的沟通和协作。成员权限变更和房间成员管理是通过一系列Onyx数据操作实现的。

问题根源

经过分析，问题出在removeOptimisticRoomMembers函数的实现上。该函数负责处理从房间中移除成员的操作，但当前实现存在以下缺陷：

1. 函数仅更新了报告的元数据中的pendingChatMembers字段
2. 没有实际更新报告participants对象中的成员状态
3. 缺少对参与者列表的完整清理机制

解决方案

要解决这个问题，需要对removeOptimisticRoomMembers函数进行以下改进：

1. 创建两个新对象来跟踪乐观更新和失败回滚的状态：

   • optimisticParticipants：用于临时存储将被移除的参与者
   • failureParticipants：用于在操作失败时恢复原始参与者状态

2. 更新乐观数据推送逻辑：

   • 在乐观更新中包含参与者状态的变更
   • 正确处理当前会话用户的特殊状态

3. 完善失败处理机制：

   • 在操作失败时正确恢复参与者状态
   • 确保所有相关状态都能回滚到操作前的值

实现细节

具体的代码修改需要：

1. 遍历所有待移除的账户ID
2. 检查并记录当前报告中的参与者
3. 准备乐观更新和失败回滚所需的数据结构
4. 将这些变更集成到现有的Onyx数据流中

影响评估

这个修复将确保：

1. 从Workspace移除管理员时，他们也会从管理员房间中正确移除
2. 系统状态保持一致性
3. 用户界面显示与实际权限状态匹配

总结

这个问题的解决不仅修复了一个功能缺陷，也增强了系统在处理成员权限变更时的健壮性。通过完善数据更新机制，确保了Workspace成员管理功能在各种边界条件下的正确行为。