kodu 2.1.1 → 2.1.2

package/__tests__/core/fs/fs.service.test.ts

@@ -0,0 +1,72 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { ConfigService } from '../../../src/core/config/config.service';
+import { FsService } from '../../../src/core/file-system/fs.service';
+import { UiService } from '../../../src/core/ui/ui.service';
+
+vi.mock('../../../src/core/config/config.service', () => ({
+  ConfigService: vi.fn().mockImplementation(() => ({
+    getConfig: () => ({
+      cleaner: { whitelist: [], keepJSDoc: false },
+      packer: { ignore: ['node_modules', 'dist'], useGitignore: false },
+    }),
+  })),
+}));
+
+vi.mock('../../../src/core/ui/ui.service', () => ({
+  UiService: vi.fn().mockImplementation(() => ({
+    log: { warn: vi.fn() },
+  })),
+}));
+
+describe('FsService', () => {
+  let fsService: FsService;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    const configService = new ConfigService() as never;
+    const uiService = new UiService() as never;
+    fsService = new FsService(configService, uiService);
+  });
+
+  describe('findProjectFiles', () => {
+    it('should find ts files in current directory', async () => {
+      const files = await fsService.findProjectFiles({
+        ignore: ['node_modules', 'dist'],
+        useGitignore: false,
+      });
+
+      const tsFiles = files.filter((f) => f.endsWith('.ts'));
+      expect(tsFiles.length).toBeGreaterThan(0);
+    });
+
+    it('should exclude node_modules by default', async () => {
+      const files = await fsService.findProjectFiles({
+        ignore: ['node_modules', 'dist'],
+        useGitignore: false,
+      });
+
+      const hasNodeModules = files.some((f) => f.includes('node_modules'));
+      expect(hasNodeModules).toBe(false);
+    });
+
+    it('should return relative paths', async () => {
+      const files = await fsService.findProjectFiles({
+        ignore: [],
+        useGitignore: false,
+      });
+
+      const hasAbsolute = files.some((f) => f.startsWith('/'));
+      expect(hasAbsolute).toBe(false);
+    });
+
+    it('should return sorted paths', async () => {
+      const files = await fsService.findProjectFiles({
+        ignore: [],
+        useGitignore: false,
+      });
+
+      const sorted = [...files].sort((a, b) => a.localeCompare(b));
+      expect(files).toEqual(sorted);
+    });
+  });
+});

package/__tests__/shared/cleaner/cleaner.service.test.ts

@@ -0,0 +1,102 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { ConfigService } from '../../../src/core/config/config.service';
+
+vi.mock('../../../src/core/config/config.service', () => ({
+  ConfigService: vi.fn().mockImplementation(() => ({
+    getConfig: () => ({
+      cleaner: {
+        whitelist: [],
+        keepJSDoc: false,
+        useGitignore: false,
+      },
+      packer: {
+        ignore: [],
+        useGitignore: false,
+      },
+    }),
+  })),
+}));
+
+vi.mock('../../../src/core/file-system/fs.service', () => ({
+  FsService: vi.fn().mockImplementation(() => ({})),
+}));
+
+describe('CleanerService', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('should remove single-line comments', async () => {
+    const { CleanerService } = await import(
+      '../../../src/shared/cleaner/cleaner.service'
+    );
+    const configService = new ConfigService() as never;
+    const fsService = {} as never;
+    const cleaner = new CleanerService(configService, fsService);
+
+    const input = `const x = 1; // comment
+const y = 2;`;
+    const result = cleaner.cleanContent('test.ts', input);
+
+    expect(result).toBe('const x = 1; \nconst y = 2;');
+  });
+
+  it('should remove multi-line comments', async () => {
+    const { CleanerService } = await import(
+      '../../../src/shared/cleaner/cleaner.service'
+    );
+    const configService = new ConfigService() as never;
+    const fsService = {} as never;
+    const cleaner = new CleanerService(configService, fsService);
+
+    const input = `/* comment */
+const x = 1;`;
+    const result = cleaner.cleanContent('test.ts', input);
+
+    expect(result).toBe('\nconst x = 1;');
+  });
+
+  it('should preserve comments in whitelist', async () => {
+    const { CleanerService } = await import(
+      '../../../src/shared/cleaner/cleaner.service'
+    );
+    const configService = new ConfigService() as never;
+    const fsService = {} as never;
+    const cleaner = new CleanerService(configService, fsService);
+
+    const input = `const x = 1; // eslint-disable-line
+const y = 2;`;
+    const result = cleaner.cleanContent('test.ts', input);
+
+    expect(result).toBe('const x = 1; // eslint-disable-line\nconst y = 2;');
+  });
+
+  it('should keep JSDoc when option is set', async () => {
+    const { CleanerService } = await import(
+      '../../../src/shared/cleaner/cleaner.service'
+    );
+    const configService = new ConfigService() as never;
+    const fsService = {} as never;
+    const cleaner = new CleanerService(configService, fsService);
+
+    const input = `/** JSDoc */
+const x = 1;`;
+    const result = cleaner.cleanContent('test.ts', input, true);
+
+    expect(result).toBe('/** JSDoc */\nconst x = 1;');
+  });
+
+  it('should remove JSX expression without content', async () => {
+    const { CleanerService } = await import(
+      '../../../src/shared/cleaner/cleaner.service'
+    );
+    const configService = new ConfigService() as never;
+    const fsService = {} as never;
+    const cleaner = new CleanerService(configService, fsService);
+
+    const input = `const x = <>{/* comment */}</>;`;
+    const result = cleaner.cleanContent('test.tsx', input);
+
+    expect(result).toBe('const x = <></>;');
+  });
+});

package/__tests__/shared/git/git.service.test.ts

@@ -0,0 +1,84 @@
+import { execa } from 'execa';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { GitService } from '../../../src/shared/git/git.service';
+
+vi.mock('execa');
+
+const mockExeca = vi.mocked(execa);
+
+describe('GitService', () => {
+  let gitService: GitService;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    gitService = new GitService();
+  });
+
+  afterEach(() => {
+    vi.resetAllMocks();
+  });
+
+  describe('ensureRepo', () => {
+    it('should resolve when inside git repo', async () => {
+      mockExeca.mockResolvedValue({ stdout: '' } as never);
+
+      await expect(gitService.ensureRepo()).resolves.not.toThrow();
+    });
+
+    it('should reject when not in git repo', async () => {
+      mockExeca.mockRejectedValue(new Error('fatal: not a git repository'));
+
+      await expect(gitService.ensureRepo()).rejects.toThrow();
+    });
+  });
+
+  describe('getChangedFiles', () => {
+    it('should return empty array when no changes', async () => {
+      mockExeca.mockResolvedValue({ stdout: '' } as never);
+
+      const files = await gitService.getChangedFiles();
+
+      expect(files).toEqual([]);
+    });
+
+    it('should return changed files from all sources', async () => {
+      const mockCalls = [
+        { stdout: '' }, // ensureRepo
+        { stdout: 'src/a.ts\nsrc/b.ts' }, // diff
+        { stdout: '' }, // diff --staged
+        { stdout: '' }, // ls-files
+      ];
+      mockExeca
+        .mockResolvedValueOnce(mockCalls[0] as never)
+        .mockResolvedValueOnce(mockCalls[1] as never)
+        .mockResolvedValueOnce(mockCalls[2] as never)
+        .mockResolvedValueOnce(mockCalls[3] as never);
+
+      const files = await gitService.getChangedFiles();
+
+      expect(files).toEqual(['src/a.ts', 'src/b.ts']);
+    });
+  });
+
+  describe('getStagedFiles', () => {
+    it('should return empty array when no staged files', async () => {
+      mockExeca
+        .mockResolvedValueOnce({ stdout: '' } as never)
+        .mockResolvedValueOnce({ stdout: '' } as never);
+
+      const files = await gitService.getStagedFiles();
+
+      expect(files).toEqual([]);
+    });
+
+    it('should return staged files', async () => {
+      mockExeca
+        .mockResolvedValueOnce({ stdout: '' } as never)
+        .mockResolvedValueOnce({ stdout: 'src/new.ts' } as never);
+
+      const files = await gitService.getStagedFiles();
+
+      expect(files).toEqual(['src/new.ts']);
+    });
+  });
+});

package/__tests__/shared/tokenizer/tokenizer.service.test.ts

@@ -0,0 +1,45 @@
+import { beforeEach, describe, expect, it } from 'vitest';
+
+describe('TokenizerService', () => {
+  let tokenizer: {
+    count: (text: string) => { tokens: number; usdEstimate: number };
+  };
+
+  beforeEach(async () => {
+    const { TokenizerService } = await import(
+      '../../../src/shared/tokenizer/tokenizer.service'
+    );
+    tokenizer = new TokenizerService();
+  });
+
+  it('should count tokens for empty string', () => {
+    const result = tokenizer.count('');
+
+    expect(result.tokens).toBe(0);
+    expect(result.usdEstimate).toBe(0);
+  });
+
+  it('should count tokens for simple text', () => {
+    const result = tokenizer.count('hello world');
+
+    expect(result.tokens).toBeGreaterThan(0);
+    expect(result.usdEstimate).toBeGreaterThan(0);
+  });
+
+  it('should estimate cost correctly based on DEFAULT_PRICE_PER_MILLION', () => {
+    const result = tokenizer.count('a'.repeat(1000));
+
+    expect(result.tokens).toBeGreaterThan(0);
+    // DEFAULT_PRICE_PER_MILLION = 5 means $5 per 1M tokens
+    const expectedCost = (result.tokens / 1_000_000) * 5;
+    expect(result.usdEstimate).toBe(expectedCost);
+  });
+
+  it('should handle large text', () => {
+    const longText = 'test '.repeat(10000);
+    const result = tokenizer.count(longText);
+
+    expect(result.tokens).toBeGreaterThan(10000);
+    expect(result.usdEstimate).toBeGreaterThan(0);
+  });
+});

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "kodu",
-    "version": "2.1.1",
+    "version": "2.1.2",
     "description": "High-performance CLI to prepare codebase for LLMs, automate reviews, and draft commits.",
     "repository": {
         "type": "git",
@@ -34,6 +34,7 @@
         "generate:schema": "ts-node scripts/generate-json-schema.ts",
         "postbuild": "npm run generate:schema",
         "start:prod": "node dist/main.js",
+        "start:dev": "nest start --watch",
         "new:command": "nest g -c nest-commander-schematics command",
         "new:question": "nest g -c nest-commander-schematics question",
         "________________ FORMAT AND LINT ________________": "",
@@ -43,8 +44,14 @@
         "ts:check": "tsc --noEmit",
         "knip": "knip --production",
         "check": "run-p ts:check lint:fix knip",
+        "________________ TEST ________________": "",
+        "test": "vitest run",
+        "test:watch": "vitest",
+        "test:cov": "vitest run --coverage",
         "________________ OTHER ________________": "",
-        "prepare": "lefthook install"
+        "prepare": "is-ci || lefthook install",
+        "update": "npx npm-check-updates -u && rimraf node_modules package-lock.json && npm i",
+        "postupdate": "npm run lint:fix && npm run check"
     },
     "dependencies": {
         "@inquirer/confirm": "^6.0.4",
@@ -73,13 +80,17 @@
         "@nestjs/schematics": "^11.0.0",
         "@nestjs/testing": "^11.0.1",
         "@types/node": "^22.10.7",
+        "is-ci": "^4.1.0",
         "knip": "^5.82.1",
         "lefthook": "^2.0.15",
         "nest-commander-schematics": "^3.2.0",
+        "npm-check-updates": "^18.3.1",
         "npm-run-all": "^4.1.5",
+        "rimraf": "^6.1.3",
         "ts-loader": "^9.5.2",
         "ts-node": "^10.9.2",
         "tsconfig-paths": "^4.2.0",
-        "typescript": "^5.7.3"
+        "typescript": "^5.7.3",
+        "vitest": "^3.2.4"
     }
 }

package/dist/src/core/config/config.service.js

@@ -25,10 +25,8 @@ let ConfigService = class ConfigService {
     loadConfig() {
         const explorer = (0, lilconfig_1.lilconfigSync)('kodu', { searchPlaces: ['kodu.json'] });
         const result = explorer.search(process.cwd());
-        if (!result || result.isEmpty || !result.config) {
-            this.terminate('kodu.json not found. Create it in the project root to configure kodu.');
-        }
-        const parsed = config_schema_1.configSchema.safeParse(result.config);
+        const rawConfig = result && !result.isEmpty && result.config ? result.config : {};
+        const parsed = config_schema_1.configSchema.safeParse(rawConfig);
         if (!parsed.success) {
             console.error(picocolors_1.default.red('kodu.json is invalid:'));
             parsed.error.issues.forEach((issue) => {

package/dist/src/core/config/config.service.js.map

@@ -1 +1 @@
-{"version":3,"file":"config.service.js","sourceRoot":"","sources":["../../../../src/core/config/config.service.ts"],"names":[],"mappings":";;;;;;;;;;;;AAAA,2CAA4C;AAC5C,yCAA0C;AAC1C,4DAA4B;AAC5B,mDAAgE;AAGzD,IAAM,aAAa,GAAnB,MAAM,aAAa;IAChB,MAAM,CAAc;IAE5B,SAAS;QACP,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC;YACjB,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,UAAU,EAAE,CAAC;QAClC,CAAC;QAED,OAAO,IAAI,CAAC,MAAM,CAAC;IACrB,CAAC;IAEO,UAAU;QAChB,MAAM,QAAQ,GAAG,IAAA,yBAAa,EAAC,MAAM,EAAE,EAAE,YAAY,EAAE,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC;QACxE,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QAE9C,IAAI,CAAC,MAAM,IAAI,MAAM,CAAC,OAAO,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC;YAChD,IAAI,CAAC,SAAS,CACZ,uEAAuE,CACxE,CAAC;QACJ,CAAC;QAED,MAAM,MAAM,GAAG,4BAAY,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;QAErD,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YACpB,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,uBAAuB,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,EAAE;gBACpC,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,QAAQ,CAAC;gBAC9C,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,KAAK,IAAI,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC;YACvD,CAAC,CAAC,CAAC;YACH,IAAI,CAAC,SAAS,CAAC,2CAA2C,CAAC,CAAC;QAC9D,CAAC;QAED,OAAO,MAAM,CAAC,IAAI,CAAC;IACrB,CAAC;IAEO,SAAS,CAAC,OAAe;QAC/B,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;CACF,CAAA;AAvCY,sCAAa;wBAAb,aAAa;IADzB,IAAA,mBAAU,GAAE;GACA,aAAa,CAuCzB"}
+{"version":3,"file":"config.service.js","sourceRoot":"","sources":["../../../../src/core/config/config.service.ts"],"names":[],"mappings":";;;;;;;;;;;;AAAA,2CAA4C;AAC5C,yCAA0C;AAC1C,4DAA4B;AAC5B,mDAAgE;AAGzD,IAAM,aAAa,GAAnB,MAAM,aAAa;IAChB,MAAM,CAAc;IAE5B,SAAS;QACP,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC;YACjB,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,UAAU,EAAE,CAAC;QAClC,CAAC;QAED,OAAO,IAAI,CAAC,MAAM,CAAC;IACrB,CAAC;IAEO,UAAU;QAChB,MAAM,QAAQ,GAAG,IAAA,yBAAa,EAAC,MAAM,EAAE,EAAE,YAAY,EAAE,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC;QACxE,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;QAE9C,MAAM,SAAS,GACb,MAAM,IAAI,CAAC,MAAM,CAAC,OAAO,IAAI,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;QAElE,MAAM,MAAM,GAAG,4BAAY,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QAEjD,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YACpB,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,uBAAuB,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,EAAE;gBACpC,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,QAAQ,CAAC;gBAC9C,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,KAAK,IAAI,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC;YACvD,CAAC,CAAC,CAAC;YACH,IAAI,CAAC,SAAS,CAAC,2CAA2C,CAAC,CAAC;QAC9D,CAAC;QAED,OAAO,MAAM,CAAC,IAAI,CAAC;IACrB,CAAC;IAEO,SAAS,CAAC,OAAe;QAC/B,OAAO,CAAC,KAAK,CAAC,oBAAE,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;CACF,CAAA;AApCY,sCAAa;wBAAb,aAAa;IADzB,IAAA,mBAAU,GAAE;GACA,aAAa,CAoCzB"}