@m3hti/commit-genie 3.0.1 → 3.1.1

package/dist/services/fileContentAnalyzer.js

@@ -0,0 +1,170 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.analyzeFileContent = analyzeFileContent;
+const gitService_1 = require("./gitService");
+/**
+ * Analyze file content changes for non-JS/TS files.
+ * Returns structural descriptions of what changed (keys, selectors, headings, etc.)
+ */
+function analyzeFileContent(filePath, diff) {
+    const ext = filePath.split('.').pop()?.toLowerCase();
+    switch (ext) {
+        case 'json':
+            return analyzeJsonChanges(filePath);
+        case 'css':
+        case 'scss':
+        case 'sass':
+        case 'less':
+            return analyzeCssChanges(diff);
+        case 'md':
+        case 'mdx':
+            return analyzeMarkdownChanges(diff);
+        case 'yml':
+        case 'yaml':
+            return analyzeYamlChanges(diff);
+        default:
+            return null;
+    }
+}
+function analyzeJsonChanges(filePath) {
+    try {
+        const oldContent = gitService_1.GitService.getFileAtHead(filePath);
+        const newContent = gitService_1.GitService.getStagedFileContent(filePath);
+        // New file
+        if (!oldContent && newContent) {
+            const newObj = JSON.parse(newContent);
+            const keys = Object.keys(newObj).slice(0, 5);
+            return { changedElements: keys, changeKind: 'added' };
+        }
+        // Deleted file
+        if (oldContent && !newContent) {
+            const oldObj = JSON.parse(oldContent);
+            const keys = Object.keys(oldObj).slice(0, 5);
+            return { changedElements: keys, changeKind: 'removed' };
+        }
+        if (!oldContent || !newContent)
+            return null;
+        const oldObj = JSON.parse(oldContent);
+        const newObj = JSON.parse(newContent);
+        const oldKeys = new Set(Object.keys(oldObj));
+        const newKeys = new Set(Object.keys(newObj));
+        const addedKeys = [];
+        const removedKeys = [];
+        const modifiedKeys = [];
+        for (const key of newKeys) {
+            if (!oldKeys.has(key))
+                addedKeys.push(key);
+        }
+        for (const key of oldKeys) {
+            if (!newKeys.has(key))
+                removedKeys.push(key);
+        }
+        for (const key of newKeys) {
+            if (oldKeys.has(key) && JSON.stringify(oldObj[key]) !== JSON.stringify(newObj[key])) {
+                modifiedKeys.push(key);
+            }
+        }
+        const changedElements = [...addedKeys, ...removedKeys, ...modifiedKeys];
+        if (changedElements.length === 0)
+            return null;
+        let changeKind;
+        if (addedKeys.length > 0 && removedKeys.length === 0 && modifiedKeys.length === 0) {
+            changeKind = 'added';
+        }
+        else if (removedKeys.length > 0 && addedKeys.length === 0 && modifiedKeys.length === 0) {
+            changeKind = 'removed';
+        }
+        else if (modifiedKeys.length > 0 && addedKeys.length === 0 && removedKeys.length === 0) {
+            changeKind = 'modified';
+        }
+        else {
+            changeKind = 'mixed';
+        }
+        return { changedElements: changedElements.slice(0, 5), changeKind };
+    }
+    catch {
+        return null;
+    }
+}
+function analyzeCssChanges(diff) {
+    if (!diff)
+        return null;
+    const selectors = [];
+    const lines = diff.split('\n');
+    for (const line of lines) {
+        if (!line.startsWith('+') && !line.startsWith('-'))
+            continue;
+        if (line.startsWith('+++') || line.startsWith('---'))
+            continue;
+        const content = line.substring(1).trim();
+        // Match CSS selectors: .class {, #id {, element {
+        const selectorMatch = content.match(/^([.#@]?[a-zA-Z_-][\w-]*(?:\s*[>,+~]\s*[.#]?[\w-]+)*)\s*\{/);
+        if (selectorMatch) {
+            const selector = selectorMatch[1].trim();
+            if (!selectors.includes(selector)) {
+                selectors.push(selector);
+            }
+            continue;
+        }
+        // Match @-rules: @media (...), @keyframes name
+        const atRuleMatch = content.match(/^(@(?:media|keyframes|font-face|supports|layer)\b[^{]*)/);
+        if (atRuleMatch) {
+            const rule = atRuleMatch[1].trim();
+            const short = rule.length > 30 ? rule.substring(0, 30) + '...' : rule;
+            if (!selectors.includes(short)) {
+                selectors.push(short);
+            }
+        }
+    }
+    if (selectors.length === 0)
+        return null;
+    return { changedElements: selectors.slice(0, 5), changeKind: 'modified' };
+}
+function analyzeMarkdownChanges(diff) {
+    if (!diff)
+        return null;
+    const headings = [];
+    const lines = diff.split('\n');
+    for (const line of lines) {
+        if (!line.startsWith('+') && !line.startsWith('-'))
+            continue;
+        if (line.startsWith('+++') || line.startsWith('---'))
+            continue;
+        const content = line.substring(1);
+        const headingMatch = content.match(/^(#{1,6})\s+(.+)$/);
+        if (headingMatch) {
+            const title = headingMatch[2].trim();
+            if (!headings.includes(title)) {
+                headings.push(title);
+            }
+        }
+    }
+    if (headings.length === 0)
+        return null;
+    return { changedElements: headings.slice(0, 5), changeKind: 'modified' };
+}
+function analyzeYamlChanges(diff) {
+    if (!diff)
+        return null;
+    const keys = [];
+    const lines = diff.split('\n');
+    for (const line of lines) {
+        if (!line.startsWith('+') && !line.startsWith('-'))
+            continue;
+        if (line.startsWith('+++') || line.startsWith('---'))
+            continue;
+        const content = line.substring(1);
+        // Match top-level keys (no leading whitespace)
+        const topLevelMatch = content.match(/^([a-zA-Z_][\w.-]*)\s*:/);
+        if (topLevelMatch) {
+            const key = topLevelMatch[1];
+            if (!keys.includes(key)) {
+                keys.push(key);
+            }
+        }
+    }
+    if (keys.length === 0)
+        return null;
+    return { changedElements: keys.slice(0, 5), changeKind: 'modified' };
+}
+//# sourceMappingURL=fileContentAnalyzer.js.map

package/dist/services/fileContentAnalyzer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"fileContentAnalyzer.js","sourceRoot":"","sources":["../../src/services/fileContentAnalyzer.ts"],"names":[],"mappings":";;AAWA,gDAuBC;AAlCD,6CAA0C;AAO1C;;;GAGG;AACH,SAAgB,kBAAkB,CAChC,QAAgB,EAChB,IAAY;IAEZ,MAAM,GAAG,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,WAAW,EAAE,CAAC;IAErD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,MAAM;YACT,OAAO,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACtC,KAAK,KAAK,CAAC;QACX,KAAK,MAAM,CAAC;QACZ,KAAK,MAAM,CAAC;QACZ,KAAK,MAAM;YACT,OAAO,iBAAiB,CAAC,IAAI,CAAC,CAAC;QACjC,KAAK,IAAI,CAAC;QACV,KAAK,KAAK;YACR,OAAO,sBAAsB,CAAC,IAAI,CAAC,CAAC;QACtC,KAAK,KAAK,CAAC;QACX,KAAK,MAAM;YACT,OAAO,kBAAkB,CAAC,IAAI,CAAC,CAAC;QAClC;YACE,OAAO,IAAI,CAAC;IAChB,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CAAC,QAAgB;IAC1C,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,uBAAU,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC;QACtD,MAAM,UAAU,GAAG,uBAAU,CAAC,oBAAoB,CAAC,QAAQ,CAAC,CAAC;QAE7D,WAAW;QACX,IAAI,CAAC,UAAU,IAAI,UAAU,EAAE,CAAC;YAC9B,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;YAC7C,OAAO,EAAE,eAAe,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,CAAC;QACxD,CAAC;QAED,eAAe;QACf,IAAI,UAAU,IAAI,CAAC,UAAU,EAAE,CAAC;YAC9B,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;YAC7C,OAAO,EAAE,eAAe,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,CAAC;QAC1D,CAAC;QAED,IAAI,CAAC,UAAU,IAAI,CAAC,UAAU;YAAE,OAAO,IAAI,CAAC;QAE5C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QACtC,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;QAEtC,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC;QAC7C,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC;QAC7C,MAAM,SAAS,GAAa,EAAE,CAAC;QAC/B,MAAM,WAAW,GAAa,EAAE,CAAC;QACjC,MAAM,YAAY,GAAa,EAAE,CAAC;QAElC,KAAK,MAAM,GAAG,IAAI,OAAO,EAAE,CAAC;YAC1B,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC7C,CAAC;QAED,KAAK,MAAM,GAAG,IAAI,OAAO,EAAE,CAAC;YAC1B,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC/C,CAAC;QAED,KAAK,MAAM,GAAG,IAAI,OAAO,EAAE,CAAC;YAC1B,IAAI,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;gBACpF,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACzB,CAAC;QACH,CAAC;QAED,MAAM,eAAe,GAAG,CAAC,GAAG,SAAS,EAAE,GAAG,WAAW,EAAE,GAAG,YAAY,CAAC,CAAC;QACxE,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,IAAI,CAAC;QAE9C,IAAI,UAA6C,CAAC;QAClD,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAClF,UAAU,GAAG,OAAO,CAAC;QACvB,CAAC;aAAM,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzF,UAAU,GAAG,SAAS,CAAC;QACzB,CAAC;aAAM,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzF,UAAU,GAAG,UAAU,CAAC;QAC1B,CAAC;aAAM,CAAC;YACN,UAAU,GAAG,OAAO,CAAC;QACvB,CAAC;QAED,OAAO,EAAE,eAAe,EAAE,eAAe,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,UAAU,EAAE,CAAC;IACtE,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,SAAS,iBAAiB,CAAC,IAAY;IACrC,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IAEvB,MAAM,SAAS,GAAa,EAAE,CAAC;IAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAE/B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC;YAAE,SAAS;QAC7D,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YAAE,SAAS;QAE/D,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAEzC,kDAAkD;QAClD,MAAM,aAAa,GAAG,OAAO,CAAC,KAAK,CAAC,4DAA4D,CAAC,CAAC;QAClG,IAAI,aAAa,EAAE,CAAC;YAClB,MAAM,QAAQ,GAAG,aAAa,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YACzC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAClC,SAAS,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC3B,CAAC;YACD,SAAS;QACX,CAAC;QAED,+CAA+C;QAC/C,MAAM,WAAW,GAAG,OAAO,CAAC,KAAK,CAAC,yDAAyD,CAAC,CAAC;QAC7F,IAAI,WAAW,EAAE,CAAC;YAChB,MAAM,IAAI,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YACnC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,GAAG,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC;YACtE,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC/B,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxB,CAAC;QACH,CAAC;IACH,CAAC;IAED,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IACxC,OAAO,EAAE,eAAe,EAAE,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;AAC5E,CAAC;AAED,SAAS,sBAAsB,CAAC,IAAY;IAC1C,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IAEvB,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAE/B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC;YAAE,SAAS;QAC7D,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YAAE,SAAS;QAE/D,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;QAElC,MAAM,YAAY,GAAG,OAAO,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;QACxD,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,KAAK,GAAG,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YACrC,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC9B,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACvB,CAAC;QACH,CAAC;IACH,CAAC;IAED,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IACvC,OAAO,EAAE,eAAe,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;AAC3E,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY;IACtC,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IAEvB,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAE/B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC;YAAE,SAAS;QAC7D,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YAAE,SAAS;QAE/D,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;QAElC,+CAA+C;QAC/C,MAAM,aAAa,GAAG,OAAO,CAAC,KAAK,CAAC,yBAAyB,CAAC,CAAC;QAC/D,IAAI,aAAa,EAAE,CAAC;YAClB,MAAM,GAAG,GAAG,aAAa,CAAC,CAAC,CAAC,CAAC;YAC7B,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;gBACxB,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACjB,CAAC;QACH,CAAC;IACH,CAAC;IAED,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IACnC,OAAO,EAAE,eAAe,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;AACvE,CAAC"}

package/dist/services/fileContentAnalyzer.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=fileContentAnalyzer.test.d.ts.map

package/dist/services/fileContentAnalyzer.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"fileContentAnalyzer.test.d.ts","sourceRoot":"","sources":["../../src/services/fileContentAnalyzer.test.ts"],"names":[],"mappings":""}

package/dist/services/fileContentAnalyzer.test.js

@@ -0,0 +1,118 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const fileContentAnalyzer_1 = require("./fileContentAnalyzer");
+const gitService_1 = require("./gitService");
+jest.mock('./gitService');
+const mockedGitService = gitService_1.GitService;
+describe('fileContentAnalyzer', () => {
+    beforeEach(() => {
+        jest.clearAllMocks();
+    });
+    describe('JSON parser', () => {
+        it('should detect changed top-level keys in package.json', () => {
+            mockedGitService.getFileAtHead.mockReturnValue(JSON.stringify({ name: 'app', version: '1.0.0', scripts: { test: 'jest' } }, null, 2));
+            mockedGitService.getStagedFileContent.mockReturnValue(JSON.stringify({ name: 'app', version: '1.1.0', scripts: { test: 'jest', build: 'tsc' } }, null, 2));
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('package.json', '');
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('version');
+            expect(result.changedElements).toContain('scripts');
+        });
+        it('should detect added top-level keys', () => {
+            mockedGitService.getFileAtHead.mockReturnValue(JSON.stringify({ name: 'app' }, null, 2));
+            mockedGitService.getStagedFileContent.mockReturnValue(JSON.stringify({ name: 'app', description: 'My app' }, null, 2));
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('package.json', '');
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('description');
+            expect(result.changeKind).toBe('added');
+        });
+        it('should detect removed top-level keys', () => {
+            mockedGitService.getFileAtHead.mockReturnValue(JSON.stringify({ name: 'app', deprecated: true }, null, 2));
+            mockedGitService.getStagedFileContent.mockReturnValue(JSON.stringify({ name: 'app' }, null, 2));
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('package.json', '');
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('deprecated');
+            expect(result.changeKind).toBe('removed');
+        });
+        it('should return null for unparseable JSON', () => {
+            mockedGitService.getFileAtHead.mockReturnValue('not json');
+            mockedGitService.getStagedFileContent.mockReturnValue('also not json');
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('config.json', '');
+            expect(result).toBeNull();
+        });
+        it('should handle new JSON files (no old content)', () => {
+            mockedGitService.getFileAtHead.mockReturnValue(null);
+            mockedGitService.getStagedFileContent.mockReturnValue(JSON.stringify({ name: 'app', version: '1.0.0' }, null, 2));
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('config.json', '');
+            expect(result).not.toBeNull();
+            expect(result.changeKind).toBe('added');
+        });
+    });
+    describe('CSS parser', () => {
+        it('should detect changed selectors from diff', () => {
+            const diff = '+.header {\n' +
+                '+  background: red;\n' +
+                '+}\n' +
+                '-#footer {\n' +
+                '-  display: none;\n' +
+                '-}\n';
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('styles.css', diff);
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('.header');
+            expect(result.changedElements).toContain('#footer');
+        });
+        it('should detect @media and @keyframes', () => {
+            const diff = '+@media (max-width: 768px) {\n' +
+                '+  .container { width: 100%; }\n' +
+                '+}\n';
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('styles.css', diff);
+            expect(result).not.toBeNull();
+            expect(result.changedElements.some((e) => e.includes('@media'))).toBe(true);
+        });
+        it('should return null for empty diff', () => {
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('styles.css', '');
+            expect(result).toBeNull();
+        });
+    });
+    describe('Markdown parser', () => {
+        it('should detect changed headings', () => {
+            const diff = '+## Installation\n' +
+                '+\n' +
+                '+Run npm install to get started.\n' +
+                '-## Setup\n' +
+                '-\n' +
+                '-Follow these steps.\n';
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('README.md', diff);
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('Installation');
+            expect(result.changedElements).toContain('Setup');
+        });
+        it('should handle multiple heading levels', () => {
+            const diff = '+# Overview\n+### Quick Start\n';
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('docs.md', diff);
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('Overview');
+            expect(result.changedElements).toContain('Quick Start');
+        });
+    });
+    describe('YAML parser', () => {
+        it('should detect changed top-level keys', () => {
+            const diff = '+services:\n' +
+                '+  web:\n' +
+                '+    ports:\n' +
+                '+      - "8080:80"\n' +
+                '-volumes:\n' +
+                '-  data:\n';
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('docker-compose.yml', diff);
+            expect(result).not.toBeNull();
+            expect(result.changedElements).toContain('services');
+            expect(result.changedElements).toContain('volumes');
+        });
+    });
+    describe('unsupported files', () => {
+        it('should return null for unknown file types', () => {
+            const result = (0, fileContentAnalyzer_1.analyzeFileContent)('binary.exe', '');
+            expect(result).toBeNull();
+        });
+    });
+});
+//# sourceMappingURL=fileContentAnalyzer.test.js.map

package/dist/services/fileContentAnalyzer.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"fileContentAnalyzer.test.js","sourceRoot":"","sources":["../../src/services/fileContentAnalyzer.test.ts"],"names":[],"mappings":";;AAAA,+DAA2D;AAC3D,6CAA0C;AAE1C,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;AAC1B,MAAM,gBAAgB,GAAG,uBAA4C,CAAC;AAEtE,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,UAAU,CAAC,GAAG,EAAE;QACd,IAAI,CAAC,aAAa,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;YAC9D,gBAAgB,CAAC,aAAa,CAAC,eAAe,CAC5C,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACtF,CAAC;YACF,gBAAgB,CAAC,oBAAoB,CAAC,eAAe,CACnD,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACpG,CAAC;YAEF,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,cAAc,EAAE,EAAE,CAAC,CAAC;YAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;YACrD,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,gBAAgB,CAAC,aAAa,CAAC,eAAe,CAC5C,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACzC,CAAC;YACF,gBAAgB,CAAC,oBAAoB,CAAC,eAAe,CACnD,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAChE,CAAC;YAEF,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,cAAc,EAAE,EAAE,CAAC,CAAC;YAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;YACzD,MAAM,CAAC,MAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAC3C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;YAC9C,gBAAgB,CAAC,aAAa,CAAC,eAAe,CAC5C,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAC3D,CAAC;YACF,gBAAgB,CAAC,oBAAoB,CAAC,eAAe,CACnD,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CACzC,CAAC;YAEF,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,cAAc,EAAE,EAAE,CAAC,CAAC;YAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;YACxD,MAAM,CAAC,MAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,gBAAgB,CAAC,aAAa,CAAC,eAAe,CAAC,UAAU,CAAC,CAAC;YAC3D,gBAAgB,CAAC,oBAAoB,CAAC,eAAe,CAAC,eAAe,CAAC,CAAC;YAEvE,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,aAAa,EAAE,EAAE,CAAC,CAAC;YAErD,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC5B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;YACvD,gBAAgB,CAAC,aAAa,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;YACrD,gBAAgB,CAAC,oBAAoB,CAAC,eAAe,CACnD,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAC3D,CAAC;YAEF,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,aAAa,EAAE,EAAE,CAAC,CAAC;YAErD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAC3C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,YAAY,EAAE,GAAG,EAAE;QAC1B,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;YACnD,MAAM,IAAI,GACR,cAAc;gBACd,uBAAuB;gBACvB,MAAM;gBACN,cAAc;gBACd,qBAAqB;gBACrB,MAAM,CAAC;YAET,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,YAAY,EAAE,IAAI,CAAC,CAAC;YAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;YACrD,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,IAAI,GACR,gCAAgC;gBAChC,kCAAkC;gBAClC,MAAM,CAAC;YAET,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,YAAY,EAAE,IAAI,CAAC,CAAC;YAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvF,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,YAAY,EAAE,EAAE,CAAC,CAAC;YACpD,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC5B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,IAAI,GACR,oBAAoB;gBACpB,KAAK;gBACL,oCAAoC;gBACpC,aAAa;gBACb,KAAK;gBACL,wBAAwB,CAAC;YAE3B,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,WAAW,EAAE,IAAI,CAAC,CAAC;YAErD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;YAC1D,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QACrD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,IAAI,GAAG,iCAAiC,CAAC;YAE/C,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YAEnD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;YACtD,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;QAC3D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;YAC9C,MAAM,IAAI,GACR,cAAc;gBACd,WAAW;gBACX,eAAe;gBACf,sBAAsB;gBACtB,aAAa;gBACb,YAAY,CAAC;YAEf,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,oBAAoB,EAAE,IAAI,CAAC,CAAC;YAE9D,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;YACtD,MAAM,CAAC,MAAO,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;YACnD,MAAM,MAAM,GAAG,IAAA,wCAAkB,EAAC,YAAY,EAAE,EAAE,CAAC,CAAC;YACpD,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC5B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/services/gitService.test.js

@@ -35,6 +35,21 @@ describe('GitService', () => {
             expect(status.staged[1]).toEqual({ status: 'M', path: 'src/modified.ts' });
             expect(status.staged[2]).toEqual({ status: 'D', path: 'src/deleted.ts' });
         });
+        it('should parse unstaged files correctly', () => {
+            mockedExecSync.mockReturnValueOnce(' M src/unstaged.ts\n');
+            const status = gitService_1.GitService.getStatus();
+            expect(status.unstaged).toHaveLength(1);
+            expect(status.unstaged[0]).toEqual({ status: 'M', path: 'src/unstaged.ts' });
+            expect(status.staged).toHaveLength(0);
+            expect(status.hasStagedChanges).toBe(false);
+        });
+        it('should not include untracked files in staged or unstaged', () => {
+            mockedExecSync.mockReturnValueOnce('?? untracked.ts\n');
+            const status = gitService_1.GitService.getStatus();
+            expect(status.staged).toHaveLength(0);
+            expect(status.unstaged).toHaveLength(0);
+            expect(status.hasStagedChanges).toBe(false);
+        });
         it('should return empty arrays when no changes', () => {
             mockedExecSync.mockReturnValueOnce('');
             const status = gitService_1.GitService.getStatus();
@@ -42,11 +57,25 @@ describe('GitService', () => {
             expect(status.staged).toHaveLength(0);
             expect(status.unstaged).toHaveLength(0);
         });
+        it('should handle mixed staged and unstaged changes in the same file', () => {
+            mockedExecSync.mockReturnValueOnce('MM src/both.ts\n');
+            const status = gitService_1.GitService.getStatus();
+            expect(status.staged).toHaveLength(1);
+            expect(status.staged[0]).toEqual({ status: 'M', path: 'src/both.ts' });
+            // Source only adds to unstaged when stagedStatus === ' '
+            expect(status.unstaged).toHaveLength(0);
+        });
         it('should handle renamed files', () => {
             mockedExecSync.mockReturnValueOnce('R  old.ts -> new.ts\n');
             const status = gitService_1.GitService.getStatus();
             expect(status.staged[0].status).toBe('R');
         });
+        it('should throw on git error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('fatal: not a git repository');
+            });
+            expect(() => gitService_1.GitService.getStatus()).toThrow('Failed to get git status');
+        });
     });
     describe('getDiffStats', () => {
         it('should parse diff stats correctly', () => {
@@ -63,13 +92,26 @@ describe('GitService', () => {
             expect(stats.insertions).toBe(0);
             expect(stats.deletions).toBe(0);
         });
-        it('should handle single file change', () => {
+        it('should handle single file with insertions only', () => {
             mockedExecSync.mockReturnValueOnce(' 1 file changed, 5 insertions(+)\n');
             const stats = gitService_1.GitService.getDiffStats();
             expect(stats.filesChanged).toBe(1);
             expect(stats.insertions).toBe(5);
             expect(stats.deletions).toBe(0);
         });
+        it('should handle single file with deletions only', () => {
+            mockedExecSync.mockReturnValueOnce(' 1 file changed, 3 deletions(-)\n');
+            const stats = gitService_1.GitService.getDiffStats();
+            expect(stats.filesChanged).toBe(1);
+            expect(stats.insertions).toBe(0);
+            expect(stats.deletions).toBe(3);
+        });
+        it('should throw on git error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('fatal: not a git repository');
+            });
+            expect(() => gitService_1.GitService.getDiffStats()).toThrow('Failed to get diff stats');
+        });
     });
     describe('getDiff', () => {
         it('should return diff content', () => {
@@ -80,9 +122,24 @@ describe('GitService', () => {
             const diff = gitService_1.GitService.getDiff();
             expect(diff).toBe(mockDiff);
         });
+        it('should throw on git error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('fatal: not a git repository');
+            });
+            expect(() => gitService_1.GitService.getDiff()).toThrow('Failed to get diff');
+        });
+    });
+    describe('getStagedFiles', () => {
+        it('should return staged files from getStatus', () => {
+            mockedExecSync.mockReturnValueOnce('M  src/modified.ts\nA  src/new.ts\n');
+            const files = gitService_1.GitService.getStagedFiles();
+            expect(files).toHaveLength(2);
+            expect(files[0]).toEqual({ status: 'M', path: 'src/modified.ts' });
+            expect(files[1]).toEqual({ status: 'A', path: 'src/new.ts' });
+        });
     });
     describe('commit', () => {
-        it('should execute git commit with message', () => {
+        it('should execute git commit with message via stdin', () => {
             mockedSpawnSync.mockReturnValueOnce({
                 status: 0,
                 stdout: '',
@@ -111,7 +168,7 @@ describe('GitService', () => {
                 input: multiLineMessage,
             }));
         });
-        it('should throw error when commit fails', () => {
+        it('should throw error with stderr when commit fails', () => {
             mockedSpawnSync.mockReturnValueOnce({
                 status: 1,
                 stdout: '',
@@ -122,6 +179,186 @@ describe('GitService', () => {
             });
             expect(() => gitService_1.GitService.commit('test')).toThrow('Failed to commit');
         });
+        it('should use stdout when stderr is empty on failure', () => {
+            mockedSpawnSync.mockReturnValueOnce({
+                status: 1,
+                stdout: 'stdout error message',
+                stderr: '',
+                pid: 123,
+                output: [],
+                signal: null,
+            });
+            expect(() => gitService_1.GitService.commit('test')).toThrow('Failed to commit: stdout error message');
+        });
+    });
+    describe('amendCommit', () => {
+        it('should execute git commit --amend with message via stdin', () => {
+            mockedSpawnSync.mockReturnValueOnce({
+                status: 0,
+                stdout: '',
+                stderr: '',
+                pid: 123,
+                output: [],
+                signal: null,
+            });
+            expect(() => gitService_1.GitService.amendCommit('fix: correct typo')).not.toThrow();
+            expect(mockedSpawnSync).toHaveBeenCalledWith('git', ['commit', '--amend', '-F', '-'], expect.objectContaining({
+                input: 'fix: correct typo',
+            }));
+        });
+        it('should throw error when amend fails', () => {
+            mockedSpawnSync.mockReturnValueOnce({
+                status: 1,
+                stdout: '',
+                stderr: 'cannot amend',
+                pid: 123,
+                output: [],
+                signal: null,
+            });
+            expect(() => gitService_1.GitService.amendCommit('test')).toThrow('Failed to amend commit');
+        });
+    });
+    describe('getLastCommitMessage', () => {
+        it('should return the last commit message trimmed', () => {
+            mockedExecSync.mockReturnValueOnce('feat: initial commit\n');
+            const msg = gitService_1.GitService.getLastCommitMessage();
+            expect(msg).toBe('feat: initial commit');
+        });
+        it('should return null when output is empty', () => {
+            mockedExecSync.mockReturnValueOnce('  \n');
+            const msg = gitService_1.GitService.getLastCommitMessage();
+            expect(msg).toBeNull();
+        });
+        it('should return null on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('no commits yet');
+            });
+            expect(gitService_1.GitService.getLastCommitMessage()).toBeNull();
+        });
+    });
+    describe('getLastCommitFiles', () => {
+        it('should parse tab-separated file change lines', () => {
+            mockedExecSync.mockReturnValueOnce('M\tsrc/index.ts\nA\tsrc/new.ts\nD\tsrc/old.ts\n');
+            const files = gitService_1.GitService.getLastCommitFiles();
+            expect(files).toHaveLength(3);
+            expect(files[0]).toEqual({ status: 'M', path: 'src/index.ts' });
+            expect(files[1]).toEqual({ status: 'A', path: 'src/new.ts' });
+            expect(files[2]).toEqual({ status: 'D', path: 'src/old.ts' });
+        });
+        it('should return empty array on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('no commits');
+            });
+            expect(gitService_1.GitService.getLastCommitFiles()).toEqual([]);
+        });
+    });
+    describe('getLastCommitDiff', () => {
+        it('should return diff string', () => {
+            const mockDiff = 'diff --git a/f.ts b/f.ts\n+new line\n';
+            mockedExecSync.mockReturnValueOnce(mockDiff);
+            expect(gitService_1.GitService.getLastCommitDiff()).toBe(mockDiff);
+        });
+        it('should return empty string on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('no commits');
+            });
+            expect(gitService_1.GitService.getLastCommitDiff()).toBe('');
+        });
+    });
+    describe('hasCommits', () => {
+        it('should return true when there are commits', () => {
+            mockedExecSync.mockReturnValueOnce(Buffer.from('abc123'));
+            expect(gitService_1.GitService.hasCommits()).toBe(true);
+        });
+        it('should return false when there are no commits', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('unknown revision HEAD');
+            });
+            expect(gitService_1.GitService.hasCommits()).toBe(false);
+        });
+    });
+    describe('getGitRoot', () => {
+        it('should return the trimmed root path', () => {
+            mockedExecSync.mockReturnValueOnce('/home/user/project\n');
+            expect(gitService_1.GitService.getGitRoot()).toBe('/home/user/project');
+        });
+        it('should throw on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('not a git repository');
+            });
+            expect(() => gitService_1.GitService.getGitRoot()).toThrow('Failed to get git root directory');
+        });
+    });
+    describe('getGitDir', () => {
+        it('should resolve relative .git path to absolute', () => {
+            mockedExecSync.mockReturnValueOnce('.git\n');
+            const result = gitService_1.GitService.getGitDir();
+            // Should resolve to an absolute path ending with .git
+            expect(result.endsWith('.git')).toBe(true);
+            expect(result.length).toBeGreaterThan(4); // more than just ".git"
+        });
+        it('should return absolute Unix paths as-is', () => {
+            mockedExecSync.mockReturnValueOnce('/home/user/project/.git\n');
+            const result = gitService_1.GitService.getGitDir();
+            expect(result).toBe('/home/user/project/.git');
+        });
+        it('should throw on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('not a git repository');
+            });
+            expect(() => gitService_1.GitService.getGitDir()).toThrow('Failed to get git directory');
+        });
+    });
+    describe('getCurrentBranch', () => {
+        it('should return the current branch name', () => {
+            mockedExecSync.mockReturnValueOnce('main\n');
+            expect(gitService_1.GitService.getCurrentBranch()).toBe('main');
+        });
+        it('should return null for detached HEAD', () => {
+            mockedExecSync.mockReturnValueOnce('HEAD\n');
+            expect(gitService_1.GitService.getCurrentBranch()).toBeNull();
+        });
+        it('should return null on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('not a git repository');
+            });
+            expect(gitService_1.GitService.getCurrentBranch()).toBeNull();
+        });
+    });
+    describe('getCommitHistory', () => {
+        it('should parse null-delimited commit entries', () => {
+            // Format: hash\x00subject\x00body\x00 per commit, separated by \x00\x00
+            const output = [
+                'abc123\x00feat: add feature\x00Some body text\x00',
+                'def456\x00fix: bug fix\x00\x00',
+            ].join('\x00');
+            mockedExecSync.mockReturnValueOnce(output);
+            const commits = gitService_1.GitService.getCommitHistory(10);
+            expect(commits).toHaveLength(2);
+            expect(commits[0]).toEqual({
+                hash: 'abc123',
+                subject: 'feat: add feature',
+                message: 'feat: add feature\n\nSome body text',
+                body: 'Some body text',
+            });
+            expect(commits[1]).toEqual({
+                hash: 'def456',
+                subject: 'fix: bug fix',
+                message: 'fix: bug fix',
+                body: undefined,
+            });
+        });
+        it('should return empty array on error', () => {
+            mockedExecSync.mockImplementationOnce(() => {
+                throw new Error('no commits');
+            });
+            expect(gitService_1.GitService.getCommitHistory()).toEqual([]);
+        });
+        it('should use default count of 50', () => {
+            mockedExecSync.mockReturnValueOnce('');
+            gitService_1.GitService.getCommitHistory();
+            expect(mockedExecSync).toHaveBeenCalledWith(expect.stringContaining('-50'), expect.any(Object));
+        });
     });
     describe('getFileAtHead', () => {
         it('should return file contents from HEAD', () => {
@@ -130,18 +367,12 @@ describe('GitService', () => {
             expect(result).toBe('export function foo() {}');
             expect(mockedExecSync).toHaveBeenCalledWith('git show HEAD:src/foo.ts', expect.objectContaining({ encoding: 'utf-8' }));
         });
-        it('should return null for new files (not in HEAD)', () => {
+        it('should return null for files not in HEAD', () => {
             mockedExecSync.mockImplementationOnce(() => {
                 throw new Error('path does not exist');
             });
             expect(gitService_1.GitService.getFileAtHead('src/new.ts')).toBeNull();
         });
-        it('should return null when no commits exist', () => {
-            mockedExecSync.mockImplementationOnce(() => {
-                throw new Error('unknown revision');
-            });
-            expect(gitService_1.GitService.getFileAtHead('src/any.ts')).toBeNull();
-        });
     });
     describe('getStagedFileContent', () => {
         it('should return staged file contents from index', () => {
@@ -150,25 +381,12 @@ describe('GitService', () => {
             expect(result).toBe('export const X = 1;');
             expect(mockedExecSync).toHaveBeenCalledWith('git show :src/x.ts', expect.objectContaining({ encoding: 'utf-8' }));
         });
-        it('should return null for deleted files', () => {
+        it('should return null for deleted files in index', () => {
             mockedExecSync.mockImplementationOnce(() => {
                 throw new Error('path does not exist in index');
             });
             expect(gitService_1.GitService.getStagedFileContent('src/deleted.ts')).toBeNull();
         });
     });
-    describe('getGitDir', () => {
-        it('should resolve relative .git path', () => {
-            mockedExecSync.mockReturnValueOnce('.git\n');
-            const result = gitService_1.GitService.getGitDir();
-            // Should resolve to absolute path ending with .git
-            expect(result.endsWith('.git')).toBe(true);
-        });
-        it('should handle absolute paths', () => {
-            mockedExecSync.mockReturnValueOnce('/home/user/project/.git\n');
-            const result = gitService_1.GitService.getGitDir();
-            expect(result).toBe('/home/user/project/.git');
-        });
-    });
 });
 //# sourceMappingURL=gitService.test.js.map