chore(cli): update envCode for consistency

Author: codeaholicguy

packages/cli/src/__tests__/lib/TemplateManager.test.ts

@@ -325,7 +325,7 @@ This is the prompt content.`;
 
   describe('setupMultipleEnvironments', () => {
     it('should setup multiple environments successfully', async () => {
-      const envIds: EnvironmentCode[] = ['cursor', 'gemini'];
+      const envCodes: EnvironmentCode[] = ['cursor', 'gemini'];
       const cursorEnv = {
         code: 'cursor',
         name: 'Cursor',
@@ -352,7 +352,7 @@ This is the prompt content.`;
 
       (templateManager as any).setupSingleEnvironment = mockSetupSingleEnvironment;
 
-      const result = await templateManager.setupMultipleEnvironments(envIds);
+      const result = await templateManager.setupMultipleEnvironments(envCodes);
 
       expect(mockGetEnvironment).toHaveBeenCalledWith('cursor');
       expect(mockGetEnvironment).toHaveBeenCalledWith('gemini');
@@ -366,7 +366,7 @@ This is the prompt content.`;
     });
 
     it('should skip invalid environments and continue with valid ones', async () => {
-      const envIds: EnvironmentCode[] = ['cursor', 'invalid' as any, 'gemini'];
+      const envCodes: EnvironmentCode[] = ['cursor', 'invalid' as any, 'gemini'];
       const cursorEnv = {
         code: 'cursor',
         name: 'Cursor',
@@ -393,7 +393,7 @@ This is the prompt content.`;
 
       (templateManager as any).setupSingleEnvironment = mockSetupSingleEnvironment;
 
-      const result = await templateManager.setupMultipleEnvironments(envIds);
+      const result = await templateManager.setupMultipleEnvironments(envCodes);
 
       expect(mockUi.warning).toHaveBeenCalledWith("Environment 'invalid' not found, skipping");
       expect(result).toEqual([
@@ -403,7 +403,7 @@ This is the prompt content.`;
     });
 
     it('should throw error when setupSingleEnvironment fails', async () => {
-      const envIds: EnvironmentCode[] = ['cursor'];
+      const envCodes: EnvironmentCode[] = ['cursor'];
       const cursorEnv = {
         code: 'cursor',
         name: 'Cursor',
@@ -416,26 +416,26 @@ This is the prompt content.`;
       const mockSetupSingleEnvironment = jest.fn().mockRejectedValue(new Error('Setup failed'));
       (templateManager as any).setupSingleEnvironment = mockSetupSingleEnvironment;
 
-      await expect(templateManager.setupMultipleEnvironments(envIds)).rejects.toThrow('Setup failed');
+      await expect(templateManager.setupMultipleEnvironments(envCodes)).rejects.toThrow('Setup failed');
 
       expect(mockUi.error).toHaveBeenCalledWith("Error setting up environment 'Cursor': Setup failed");
     });
   });
 
   describe('checkEnvironmentExists', () => {
     it('should return false when environment does not exist', async () => {
-      const envId: EnvironmentCode = 'cursor';
+      const envCode: EnvironmentCode = 'cursor';
 
       mockGetEnvironment.mockReturnValue(undefined);
 
-      const result = await templateManager.checkEnvironmentExists(envId);
+      const result = await templateManager.checkEnvironmentExists(envCode);
 
-      expect(mockGetEnvironment).toHaveBeenCalledWith(envId);
+      expect(mockGetEnvironment).toHaveBeenCalledWith(envCode);
       expect(result).toBe(false);
     });
 
     it('should return false when only context file exists', async () => {
-      const envId: EnvironmentCode = 'cursor';
+      const envCode: EnvironmentCode = 'cursor';
       const env = {
         code: 'cursor',
         name: 'Cursor',
@@ -447,7 +447,7 @@ This is the prompt content.`;
 
       (mockFs.pathExists as any).mockResolvedValueOnce(false); // command dir doesn't exist
 
-      const result = await templateManager.checkEnvironmentExists(envId);
+      const result = await templateManager.checkEnvironmentExists(envCode);
 
       expect(mockFs.pathExists).toHaveBeenCalledWith(
         path.join(templateManager['targetDir'], env.commandPath)
@@ -456,7 +456,7 @@ This is the prompt content.`;
     });
 
     it('should return true when command directory exists', async () => {
-      const envId: EnvironmentCode = 'cursor';
+      const envCode: EnvironmentCode = 'cursor';
       const env = {
         code: 'cursor',
         name: 'Cursor',
@@ -468,13 +468,13 @@ This is the prompt content.`;
 
       (mockFs.pathExists as any).mockResolvedValueOnce(true); // command dir exists
 
-      const result = await templateManager.checkEnvironmentExists(envId);
+      const result = await templateManager.checkEnvironmentExists(envCode);
 
       expect(result).toBe(true);
     });
 
     it('should return false when command directory does not exist', async () => {
-      const envId: EnvironmentCode = 'cursor';
+      const envCode: EnvironmentCode = 'cursor';
       const env = {
         code: 'cursor',
         name: 'Cursor',
@@ -486,7 +486,7 @@ This is the prompt content.`;
 
       (mockFs.pathExists as any).mockResolvedValueOnce(false); // command dir doesn't exist
 
-      const result = await templateManager.checkEnvironmentExists(envId);
+      const result = await templateManager.checkEnvironmentExists(envCode);
 
       expect(result).toBe(false);
     });

packages/cli/src/commands/init.ts

@@ -206,9 +206,9 @@ export async function initCommand(options: InitOptions) {
     }
   }
   const existingEnvironments: EnvironmentCode[] = [];
-  for (const envId of selectedEnvironments) {
-    if (await templateManager.checkEnvironmentExists(envId)) {
-      existingEnvironments.push(envId);
+  for (const envCode of selectedEnvironments) {
+    if (await templateManager.checkEnvironmentExists(envCode)) {
+      existingEnvironments.push(envCode);
     }
   }
 

packages/cli/src/lib/Config.ts

@@ -124,9 +124,9 @@ export class ConfigManager {
     return this.update({ environments });
   }
 
-  async hasEnvironment(envId: EnvironmentCode): Promise<boolean> {
+  async hasEnvironment(envCode: EnvironmentCode): Promise<boolean> {
     const environments = await this.getEnvironments();
-    return environments.includes(envId);
+    return environments.includes(envCode);
   }
 
   async addSkill(skill: ConfigSkill): Promise<DevKitConfig> {

packages/cli/src/lib/EnvironmentSelector.ts

@@ -78,8 +78,8 @@ export class EnvironmentSelector {
     }
 
     ui.text("\nSelected environments:");
-    selected.forEach((envId) => {
-      ui.text(`  ${getEnvironmentDisplayName(envId)}`);
+    selected.forEach((envCode) => {
+      ui.text(`  ${getEnvironmentDisplayName(envCode)}`);
     });
     ui.breakline();
   }

packages/cli/src/lib/TemplateManager.ts

@@ -44,14 +44,14 @@ export class TemplateManager {
   }
 
   async setupMultipleEnvironments(
-    environmentIds: EnvironmentCode[]
+    environmentCodes: EnvironmentCode[]
   ): Promise<string[]> {
     const copiedFiles: string[] = [];
 
-    for (const envId of environmentIds) {
-      const env = getEnvironment(envId);
+    for (const envCode of environmentCodes) {
+      const env = getEnvironment(envCode);
       if (!env) {
-        ui.warning(`Environment '${envId}' not found, skipping`);
+        ui.warning(`Environment '${envCode}' not found, skipping`);
         continue;
       }
 
@@ -67,8 +67,8 @@ export class TemplateManager {
     return copiedFiles;
   }
 
-  async checkEnvironmentExists(envId: EnvironmentCode): Promise<boolean> {
-    const env = getEnvironment(envId);
+  async checkEnvironmentExists(envCode: EnvironmentCode): Promise<boolean> {
+    const env = getEnvironment(envCode);
 
     if (!env) {
       return false;