Skip to content

Commit cfb49e0

Browse files
committed
fix(cli): only update when there are successful installs
1 parent 80f257a commit cfb49e0

2 files changed

Lines changed: 7 additions & 6 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
@@ -108,8 +108,6 @@ describe('install service', () => {
108108
expect(report.phases.skipped).toBe(1);
109109
expect(report.skills.installed).toBe(1);
110110
expect(mockConfigManager.update).toHaveBeenCalledWith({
111-
environments: [],
112-
phases: [],
113111
skills: [{ registry: 'codeaholicguy/ai-devkit', name: 'debug' }],
114112
});
115113
});

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

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,10 +130,13 @@ export async function reconcileAndInstall(
130130
}
131131
}
132132

133-
const updates: Record<string, unknown> = {
134-
environments: successfulEnvironments,
135-
phases: successfulPhases,
136-
};
133+
const updates: Record<string, unknown> = {};
134+
if (successfulEnvironments.length > 0) {
135+
updates.environments = successfulEnvironments;
136+
}
137+
if (successfulPhases.length > 0) {
138+
updates.phases = successfulPhases;
139+
}
137140
if (successfulSkills.length > 0) {
138141
updates.skills = successfulSkills;
139142
}

0 commit comments

Comments
 (0)