Fix #200: Store project config in local directory

Author: bhouston

packages/cli/src/commands/config.ts

@@ -114,7 +114,18 @@ export const command: CommandModule<SharedOptions, ConfigOptions> = {
 
     // Handle 'list' command
     if (argv.command === 'list') {
+      // Import directly to avoid circular dependency
+      const { getSettingsDir } = await import('../settings/settings.js');
+      const { getProjectConfigFile } = await import('../settings/config.js');
+      
+      const globalConfigFile = path.join(getSettingsDir(), 'config.json');
+      const projectConfigFile = getProjectConfigFile();
+      
       logger.info('Current configuration:');
+      logger.info(`Global config file: ${globalConfigFile}`);
+      logger.info(`Project config file: ${projectConfigFile}`);
+      logger.info('');
+      
       const defaultConfig = getDefaultConfig();
 
       // Get all valid config keys

packages/cli/src/settings/config.ts

@@ -23,6 +23,17 @@ const globalConfigFile = path.join(getSettingsDir(), 'config.json');
 // Export for testing
 export const getProjectConfigFile = (): string => {
   const projectDir = getProjectSettingsDir();
+  
+  // Ensure the project directory exists
+  if (projectDir && !fs.existsSync(projectDir)) {
+    try {
+      fs.mkdirSync(projectDir, { recursive: true });
+    } catch (error) {
+      console.error(`Error creating project settings directory: ${error}`);
+      return '';
+    }
+  }
+  
   return projectDir ? path.join(projectDir, 'config.json') : '';
 };
 

packages/cli/src/settings/settings.ts

@@ -33,7 +33,20 @@ export const getProjectSettingsDir = (): string => {
   }
 
   // If we're creating a new project config, use the current directory
-  return path.join(process.cwd(), '.mycoder');
+  const projectDir = path.join(process.cwd(), '.mycoder');
+  
+  // Ensure directory exists when it's requested
+  if (!fs.existsSync(projectDir)) {
+    try {
+      fs.mkdirSync(projectDir, { recursive: true });
+    } catch (error) {
+      console.error(`Error creating project settings directory: ${error}`);
+      // Still return the path even if we couldn't create it,
+      // as other code will handle the error when trying to use it
+    }
+  }
+  
+  return projectDir;
 };
 
 /**

packages/cli/tests/settings/project-config.test.ts

@@ -0,0 +1,111 @@
+import * as fs from 'fs';
+import * as path from 'path';
+
+import { beforeEach, afterEach, describe, expect, it, vi } from 'vitest';
+
+import { getProjectConfigFile } from '../../src/settings/config.js';
+import { getProjectSettingsDir } from '../../src/settings/settings.js';
+
+// Mock fs module
+vi.mock('fs', () => ({
+  existsSync: vi.fn(),
+  mkdirSync: vi.fn(),
+  statSync: vi.fn(),
+}));
+
+// Mock path module
+vi.mock('path', async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    ...actual,
+    parse: vi.fn(),
+  };
+});
+
+// Only mock specific functions from settings.js
+vi.mock('../../src/settings/settings.js', async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    ...actual,
+    getProjectSettingsDir: vi.fn(),
+  };
+});
+
+describe('Project Config File', () => {
+  const mockCwd = '/mock/project/dir';
+  const mockProjectDir = path.join(mockCwd, '.mycoder');
+  const expectedConfigFile = path.join(mockProjectDir, 'config.json');
+
+  beforeEach(() => {
+    // Reset mocks
+    vi.resetAllMocks();
+    
+    // Mock process.cwd()
+    vi.spyOn(process, 'cwd').mockReturnValue(mockCwd);
+    
+    // Mock path.parse
+    vi.mocked(path.parse).mockReturnValue({
+      root: '/',
+      dir: '/mock',
+      base: 'dir',
+      name: 'dir',
+      ext: '',
+    });
+    
+    // Default mock for existsSync
+    vi.mocked(fs.existsSync).mockReturnValue(false);
+    
+    // Default mock for statSync
+    vi.mocked(fs.statSync).mockReturnValue({
+      isDirectory: () => true,
+    } as unknown as fs.Stats);
+  });
+
+  afterEach(() => {
+    vi.resetAllMocks();
+  });
+
+  it('should return project config file path in current directory', () => {
+    // Mock getProjectSettingsDir to return the project dir
+    vi.mocked(getProjectSettingsDir).mockReturnValue(mockProjectDir);
+    
+    const result = getProjectConfigFile();
+    
+    expect(result).toBe(expectedConfigFile);
+  });
+
+  it('should create project directory if it does not exist', () => {
+    // Mock getProjectSettingsDir to return the project dir
+    vi.mocked(getProjectSettingsDir).mockReturnValue(mockProjectDir);
+    
+    // Mock directory does not exist
+    vi.mocked(fs.existsSync).mockReturnValue(false);
+    
+    getProjectConfigFile();
+    
+    // Verify directory creation was attempted
+    expect(fs.mkdirSync).toHaveBeenCalledWith(mockProjectDir, { recursive: true });
+  });
+
+  it('should not create project directory if it already exists', () => {
+    // Mock getProjectSettingsDir to return the project dir
+    vi.mocked(getProjectSettingsDir).mockReturnValue(mockProjectDir);
+    
+    // Mock directory already exists
+    vi.mocked(fs.existsSync).mockReturnValue(true);
+    
+    getProjectConfigFile();
+    
+    // Verify directory creation was not attempted
+    expect(fs.mkdirSync).not.toHaveBeenCalled();
+  });
+
+  it('should return empty string if project directory cannot be determined', () => {
+    // Mock getProjectSettingsDir to return empty string (error case)
+    vi.mocked(getProjectSettingsDir).mockReturnValue('');
+    
+    const result = getProjectConfigFile();
+    
+    expect(result).toBe('');
+  });
+});