fix(changelog): improve changelog update logic

- Handle cases with no existing header
- Handle cases where new version already exists
- Add more robust version detection
- Improve changelog content generation
- Add logging for skipped updates

Author: taj54

dist/index.js

@@ -6,7 +6,7 @@
  * Author: Taj <[email protected]>
  * Homepage: https://github.com/taj54/universal-version-bump#readme
  * License: MIT
- * Generated on Tue, 26 Aug 2025 13:13:01 GMT
+ * Generated on Tue, 26 Aug 2025 13:20:32 GMT
  */
 require('./sourcemap-register.js');/******/ (() => { // webpackBootstrap
 /******/ 	var __webpack_modules__ = ({
@@ -33009,11 +33009,21 @@ class ChangelogService {
         const headerIndex = existingChangelog.indexOf(separator);
         if (headerIndex !== -1) {
             const header = existingChangelog.substring(0, headerIndex + separator.length);
-            const restOfChangelog = existingChangelog.substring(headerIndex + separator.length);
-            const newChangelog = header + changelogContent + restOfChangelog;
-            await this.fileHandler.writeFile(changelogPath, newChangelog);
+            const contentAfterHeader = existingChangelog.substring(headerIndex + separator.length);
+            const firstVersionIndex = contentAfterHeader.search(/## v\d+\.\d+\.\d+/);
+            if (firstVersionIndex !== -1) {
+                const oldChangelog = contentAfterHeader.substring(firstVersionIndex);
+                const newChangelog = header + changelogContent + oldChangelog;
+                await this.fileHandler.writeFile(changelogPath, newChangelog);
+            }
+            else {
+                // No version found after header, so just append the new changelog
+                const newChangelog = header + changelogContent;
+                await this.fileHandler.writeFile(changelogPath, newChangelog);
+            }
         }
         else {
+            // No separator found, so prepend the new changelog
             const newChangelog = changelogContent + existingChangelog;
             await this.fileHandler.writeFile(changelogPath, newChangelog);
         }

dist/index.js.map

@@ -95,10 +95,20 @@ export class ChangelogService {
 
     if (headerIndex !== -1) {
       const header = existingChangelog.substring(0, headerIndex + separator.length);
-      const restOfChangelog = existingChangelog.substring(headerIndex + separator.length);
-      const newChangelog = header + changelogContent + restOfChangelog;
-      await this.fileHandler.writeFile(changelogPath, newChangelog);
+      const contentAfterHeader = existingChangelog.substring(headerIndex + separator.length);
+      const firstVersionIndex = contentAfterHeader.search(/## v\d+\.\d+\.\d+/);
+
+      if (firstVersionIndex !== -1) {
+        const oldChangelog = contentAfterHeader.substring(firstVersionIndex);
+        const newChangelog = header + changelogContent + oldChangelog;
+        await this.fileHandler.writeFile(changelogPath, newChangelog);
+      } else {
+        // No version found after header, so just append the new changelog
+        const newChangelog = header + changelogContent;
+        await this.fileHandler.writeFile(changelogPath, newChangelog);
+      }
     } else {
+      // No separator found, so prepend the new changelog
       const newChangelog = changelogContent + existingChangelog;
       await this.fileHandler.writeFile(changelogPath, newChangelog);
     }

src/services/changelogService.ts

@@ -63,19 +63,18 @@ describe('ChangelogService', () => {
     expect(changelog).toContain('- maintenance');
   });
 
-  it('should update the changelog file', async () => {
-    const existingChangelogWithHeader = '# Header\n\n---\n\nexisting content';
-    const newContent = 'new content';
-    const expectedContent = '# Header\n\n---\n\nnew contentexisting content';
-
-    const readFileSpy = vi
-      .spyOn(fileHandler, 'readFile')
-      .mockResolvedValue(existingChangelogWithHeader);
-    const writeFileSpy = vi.spyOn(fileHandler, 'writeFile').mockResolvedValue();
+  it('should update the changelog file, skipping last change noted info', async () => {
+    const existingChangelog =
+      '# Changelog\n\n---\n\nThis is the last change noted info.\n\n## v1.0.0\n\n- Initial release';
+    const newContent = '## v1.1.0\n\n- New feature';
+    const expectedContent =
+      '# Changelog\n\n---\n\n## v1.1.0\n\n- New feature## v1.0.0\n\n- Initial release';
+
+    vi.spyOn(fileHandler, 'readFile').mockResolvedValue(existingChangelog);
+    const writeFileSpy = vi.spyOn(fileHandler, 'writeFile').mockResolvedValue(undefined);
 
     await changelogService.updateChangelog(newContent);
 
-    expect(readFileSpy).toHaveBeenCalledWith('CHANGELOG.md');
     expect(writeFileSpy).toHaveBeenCalledWith('CHANGELOG.md', expectedContent);
   });