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

perf(startup): avoid provider refresh version probe

Browse source
This commit is contained in:
777genius 2026-05-25 23:37:12 +03:00
parent 320cee2d29
commit 0d4e6f5047
2 changed files with 26 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/main/services/infrastructure/CliInstallerService.ts
View file

@ -931,11 +931,6 @@ export class CliInstallerService {
}
const generation = this.statusGatherGeneration;
const versionProbe = await this.probeCliVersion(binaryPath);
if (!versionProbe.ok) {
return null;
}
const providerStatus = await this.multimodelBridgeService.getProviderStatus(
binaryPath,
providerId,

26
test/main/services/infrastructure/CliInstallerService.test.ts
View file

@ -426,6 +426,32 @@ describe('CliInstallerService', () => {
expect(status.installedVersion).toBeNull();
});
it('does not run a redundant version probe before an explicit multimodel provider refresh', async () => {
allowConsoleLogs();
vi.mocked(getConfiguredCliFlavor).mockReturnValue('agent_teams_orchestrator');
vi.mocked(getCliFlavorUiOptions).mockReturnValue({
displayName: 'agent_teams_orchestrator',
supportsSelfUpdate: false,
showVersionDetails: false,
showBinaryPath: false,
});
vi.mocked(ClaudeBinaryResolver.resolve).mockResolvedValue('/mock/agent_teams_orchestrator');
const providerStatus = createTestProviderStatus('codex', true, 'chatgpt');
const getProviderStatusSpy = vi
.spyOn(ClaudeMultimodelBridgeService.prototype, 'getProviderStatus')
.mockResolvedValue(providerStatus);
const status = await service.getProviderStatus('codex');
expect(status).toBe(providerStatus);
expect(execCli).not.toHaveBeenCalled();
expect(getProviderStatusSpy).toHaveBeenCalledWith(
'/mock/agent_teams_orchestrator',
'codex',
expect.any(Function)
);
});
it('retries the version probe once before marking the runtime unhealthy', async () => {
allowConsoleLogs();
vi.mocked(ClaudeBinaryResolver.resolve).mockResolvedValue('/usr/local/bin/claude');