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

fix: add missing TEAM_GET_LOGS_FOR_TASK to test mock and remove unused variable

Browse source 
- Add TEAM_GET_LOGS_FOR_TASK to ipcChannels mock in teams.test.ts
- Add handler registration/removal assertions for the new channel
- Remove unused knownMembers destructuring in TeamMemberLogsFinder
This commit is contained in:
iliya 2026-02-23 11:44:41 +02:00 committed by Илия
parent 9948c01e20
commit 3865d7455e
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/services/team/TeamMemberLogsFinder.ts
View file

@ -298,7 +298,7 @@ export class TeamMemberLogsFinder {
} | null> { } | null> {
const discovery = await this.discoverProjectSessions(teamName); const discovery = await this.discoverProjectSessions(teamName);
if (!discovery) return null; if (!discovery) return null;
const { config, knownMembers } = discovery; const { config } = discovery;
const leadMemberName = const leadMemberName =
config.members?.find((m) => m?.agentType === 'team-lead')?.name?.trim() || 'team-lead'; config.members?.find((m) => m?.agentType === 'team-lead')?.name?.trim() || 'team-lead';
const isLeadMember = leadMemberName.toLowerCase() === memberName.trim().toLowerCase(); const isLeadMember = leadMemberName.toLowerCase() === memberName.trim().toLowerCase();

4
test/main/ipc/teams.test.ts
View file

@ -21,6 +21,7 @@ vi.mock('@preload/constants/ipcChannels', () => ({
TEAM_PROCESS_ALIVE: 'team:processAlive', TEAM_PROCESS_ALIVE: 'team:processAlive',
TEAM_ALIVE_LIST: 'team:aliveList', TEAM_ALIVE_LIST: 'team:aliveList',
TEAM_GET_MEMBER_LOGS: 'team:getMemberLogs', TEAM_GET_MEMBER_LOGS: 'team:getMemberLogs',
TEAM_GET_LOGS_FOR_TASK: 'team:getLogsForTask',
TEAM_GET_MEMBER_STATS: 'team:getMemberStats', TEAM_GET_MEMBER_STATS: 'team:getMemberStats',
TEAM_UPDATE_CONFIG: 'team:updateConfig', TEAM_UPDATE_CONFIG: 'team:updateConfig',
TEAM_START_TASK: 'team:startTask', TEAM_START_TASK: 'team:startTask',
@ -45,6 +46,7 @@ import {
TEAM_REQUEST_REVIEW, TEAM_REQUEST_REVIEW,
TEAM_SEND_MESSAGE, TEAM_SEND_MESSAGE,
TEAM_GET_ALL_TASKS, TEAM_GET_ALL_TASKS,
TEAM_GET_LOGS_FOR_TASK,
TEAM_GET_MEMBER_LOGS, TEAM_GET_MEMBER_LOGS,
TEAM_GET_MEMBER_STATS, TEAM_GET_MEMBER_STATS,
TEAM_START_TASK, TEAM_START_TASK,
@ -135,6 +137,7 @@ describe('ipc teams handlers', () => {
expect(handlers.has(TEAM_ALIVE_LIST)).toBe(true); expect(handlers.has(TEAM_ALIVE_LIST)).toBe(true);
expect(handlers.has(TEAM_CREATE_CONFIG)).toBe(true); expect(handlers.has(TEAM_CREATE_CONFIG)).toBe(true);
expect(handlers.has(TEAM_GET_MEMBER_LOGS)).toBe(true); expect(handlers.has(TEAM_GET_MEMBER_LOGS)).toBe(true);
expect(handlers.has(TEAM_GET_LOGS_FOR_TASK)).toBe(true);
expect(handlers.has(TEAM_GET_MEMBER_STATS)).toBe(true); expect(handlers.has(TEAM_GET_MEMBER_STATS)).toBe(true);
expect(handlers.has(TEAM_UPDATE_CONFIG)).toBe(true); expect(handlers.has(TEAM_UPDATE_CONFIG)).toBe(true);
expect(handlers.has(TEAM_GET_ALL_TASKS)).toBe(true); expect(handlers.has(TEAM_GET_ALL_TASKS)).toBe(true);
@ -303,6 +306,7 @@ describe('ipc teams handlers', () => {
expect(handlers.has(TEAM_ALIVE_LIST)).toBe(false); expect(handlers.has(TEAM_ALIVE_LIST)).toBe(false);
expect(handlers.has(TEAM_CREATE_CONFIG)).toBe(false); expect(handlers.has(TEAM_CREATE_CONFIG)).toBe(false);
expect(handlers.has(TEAM_GET_MEMBER_LOGS)).toBe(false); expect(handlers.has(TEAM_GET_MEMBER_LOGS)).toBe(false);
expect(handlers.has(TEAM_GET_LOGS_FOR_TASK)).toBe(false);
expect(handlers.has(TEAM_GET_MEMBER_STATS)).toBe(false); expect(handlers.has(TEAM_GET_MEMBER_STATS)).toBe(false);
expect(handlers.has(TEAM_UPDATE_CONFIG)).toBe(false); expect(handlers.has(TEAM_UPDATE_CONFIG)).toBe(false);
expect(handlers.has(TEAM_GET_ALL_TASKS)).toBe(false); expect(handlers.has(TEAM_GET_ALL_TASKS)).toBe(false);