admin-valentin/agent-ecosystem
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(team): mark permission_request inbox messages as read after scan

Browse source 
Restored markInboxMessagesRead for permission_request messages in the
early scan. Without this, processed permission_request messages stayed
read=false forever, potentially showing stale unread indicators in UI.
This commit is contained in:
iliya 2026-03-28 20:36:10 +02:00
parent e431cfd02c
commit ccd11a5eb0
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/main/services/team/TeamProvisioningService.ts
View file

@ -4042,12 +4042,22 @@ export class TeamProvisioningService {
const leadName = config.members?.find((m) => isLeadMember(m))?.name?.trim() || 'team-lead'; const leadName = config.members?.find((m) => isLeadMember(m))?.name?.trim() || 'team-lead';
try { try {
const leadInboxMessages = await this.inboxReader.getMessagesFor(teamName, leadName); const leadInboxMessages = await this.inboxReader.getMessagesFor(teamName, leadName);
const permMsgsToMarkRead: { messageId: string }[] = [];
for (const msg of leadInboxMessages) { for (const msg of leadInboxMessages) {
if (typeof msg.text !== 'string') continue; if (typeof msg.text !== 'string') continue;
const perm = parsePermissionRequest(msg.text); const perm = parsePermissionRequest(msg.text);
if (!perm) continue; if (!perm) continue;
// Dedup is handled inside handleTeammatePermissionRequest via processedPermissionRequestIds // Dedup is handled inside handleTeammatePermissionRequest via processedPermissionRequestIds
this.handleTeammatePermissionRequest(run, perm, msg.timestamp); this.handleTeammatePermissionRequest(run, perm, msg.timestamp);
// Mark unread permission_request messages as read to prevent stale unread indicators
if (!msg.read && this.hasStableMessageId(msg)) {
permMsgsToMarkRead.push({ messageId: msg.messageId });
}
}
if (permMsgsToMarkRead.length > 0) {
await this.markInboxMessagesRead(teamName, leadName, permMsgsToMarkRead).catch(
() => {}
);
} }
} catch { } catch {
// best-effort — inbox may not exist yet // best-effort — inbox may not exist yet