fix(changelog): prevent duplicate changelog entries

- Improved changelog update logic
- Handles existing entries for same version
- Places new entries after header
- Uses separator for better placement
- Improved logging for duplicate entries

Author: taj54

CHANGELOG.md

@@ -1,4 +1,3 @@
-
 # 📦 Changelog for Universal Version Bump
 
 All notable changes for each version of the Ambient Music extension.
@@ -10,7 +9,7 @@ All notable changes for each version of the Ambient Music extension.
 ### Added
 
 - add changelog generation
-- version bumpupdated this  taj54/[email protected]
+- version bumpupdated this taj54/[email protected]
 
 ### Changed
 
@@ -42,6 +41,11 @@ All notable changes for each version of the Ambient Music extension.
 
 - **changelog:** Strip prefixes like `feat:` and `fix:` from changelog entries.
 
+### Added
+
+- **changelog:** Prevent duplicate changelog entries for the same version.
+- **changelog:** Place new changelog entries after the header in `CHANGELOG.md`.
+
 ## v0.9.0 2025 08 25
 
 ### Added

dist/index.js

@@ -1,12 +1,12 @@
 /**
- * universal-version-bump v0.9.2
+ * universal-version-bump v0.9.3
  * Universal Version Bump
  *
  * Description: A GitHub Action to automatically bump versions across any app (Node, Python, PHP, Docker, etc.)
  * Author: Taj <[email protected]>
  * Homepage: https://github.com/taj54/universal-version-bump#readme
  * License: MIT
- * Generated on Tue, 26 Aug 2025 10:43:40 GMT
+ * Generated on Tue, 26 Aug 2025 13:13:01 GMT
  */
 require('./sourcemap-register.js');/******/ (() => { // webpackBootstrap
 /******/ 	var __webpack_modules__ = ({
@@ -32997,8 +32997,26 @@ class ChangelogService {
     async updateChangelog(changelogContent) {
         const changelogPath = 'CHANGELOG.md';
         const existingChangelog = await this.fileHandler.readFile(changelogPath);
-        const newChangelog = changelogContent + existingChangelog;
-        await this.fileHandler.writeFile(changelogPath, newChangelog);
+        const versionMatch = changelogContent.match(/## v(\d+\.\d+\.\d+)/);
+        if (versionMatch) {
+            const newVersion = versionMatch[0];
+            if (existingChangelog.includes(newVersion)) {
+                console.log(`Changelog for version ${newVersion} already exists. Skipping.`);
+                return;
+            }
+        }
+        const separator = '\n---\n\n';
+        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);
+        }
+        else {
+            const newChangelog = changelogContent + existingChangelog;
+            await this.fileHandler.writeFile(changelogPath, newChangelog);
+        }
     }
 }
 exports.ChangelogService = ChangelogService;

dist/index.js.map

@@ -53,4 +53,4 @@
     "vitest": "^3.2.4",
     "yaml": "^2.8.1"
   }
-}
+}

package.json

@@ -81,7 +81,26 @@ export class ChangelogService {
   async updateChangelog(changelogContent: string): Promise<void> {
     const changelogPath = 'CHANGELOG.md';
     const existingChangelog = await this.fileHandler.readFile(changelogPath);
-    const newChangelog = changelogContent + existingChangelog;
-    await this.fileHandler.writeFile(changelogPath, newChangelog);
+    const versionMatch = changelogContent.match(/## v(\d+\.\d+\.\d+)/);
+    if (versionMatch) {
+      const newVersion = versionMatch[0];
+      if (existingChangelog.includes(newVersion)) {
+        console.log(`Changelog for version ${newVersion} already exists. Skipping.`);
+        return;
+      }
+    }
+
+    const separator = '\n---\n\n';
+    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);
+    } else {
+      const newChangelog = changelogContent + existingChangelog;
+      await this.fileHandler.writeFile(changelogPath, newChangelog);
+    }
   }
 }

src/services/changelogService.ts

@@ -64,12 +64,34 @@ describe('ChangelogService', () => {
   });
 
   it('should update the changelog file', async () => {
-    const readFileSpy = vi.spyOn(fileHandler, 'readFile').mockResolvedValue('existing content');
+    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();
+
+    await changelogService.updateChangelog(newContent);
+
+    expect(readFileSpy).toHaveBeenCalledWith('CHANGELOG.md');
+    expect(writeFileSpy).toHaveBeenCalledWith('CHANGELOG.md', expectedContent);
+  });
+
+  it('should not update the changelog file if the version already exists', async () => {
+    const existingChangelog = '## v1.1.0 2025-08-26\n\n### Added\n\n- existing feature\n\n';
+    const readFileSpy = vi.spyOn(fileHandler, 'readFile').mockResolvedValue(existingChangelog);
     const writeFileSpy = vi.spyOn(fileHandler, 'writeFile').mockResolvedValue();
+    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
 
-    await changelogService.updateChangelog('new content');
+    const newChangelog = '## v1.1.0 2025-08-26\n\n### Fixed\n\n- a bug fix\n\n';
+    await changelogService.updateChangelog(newChangelog);
 
     expect(readFileSpy).toHaveBeenCalledWith('CHANGELOG.md');
-    expect(writeFileSpy).toHaveBeenCalledWith('CHANGELOG.md', 'new contentexisting content');
+    expect(writeFileSpy).not.toHaveBeenCalled();
+    expect(consoleSpy).toHaveBeenCalledWith(
+      'Changelog for version ## v1.1.0 already exists. Skipping.',
+    );
   });
 });