Skip to content

Commit dc54301

Browse files
committed
fix: empty mcpServers
1 parent ec356be commit dc54301

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

packages/cli/src/__tests__/services/install/install.service.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ describe('install service', () => {
8585
environments: ['codex'],
8686
phases: ['requirements'],
8787
skills: [{ registry: 'codeaholicguy/ai-devkit', name: 'debug' }],
88-
mcpServers: {}
8988
});
9089
expect(report.environments.installed).toBe(1);
9190
expect(report.phases.installed).toBe(1);
@@ -112,7 +111,6 @@ describe('install service', () => {
112111
environments: [],
113112
phases: [],
114113
skills: [{ registry: 'codeaholicguy/ai-devkit', name: 'debug' }],
115-
mcpServers: {}
116114
});
117115
});
118116

packages/cli/src/services/install/install.service.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,12 +130,15 @@ export async function reconcileAndInstall(
130130
}
131131
}
132132

133-
await configManager.update({
133+
const updates: Record<string, unknown> = {
134134
environments: successfulEnvironments,
135135
phases: successfulPhases,
136136
skills: successfulSkills,
137-
mcpServers: config.mcpServers
138-
});
137+
};
138+
if (config.mcpServers && Object.keys(config.mcpServers).length > 0) {
139+
updates.mcpServers = config.mcpServers;
140+
}
141+
await configManager.update(updates as any);
139142

140143
return report;
141144
}

0 commit comments

Comments
 (0)