fix: show correct section name in changelog validation error

When validating changelogs for a release version, the error message
now correctly shows the version section (e.g., [1.2.3]) instead of
always showing [Unreleased].

src/changelog-validator.test.ts

@@ -51,6 +51,7 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: false,
           missingEntries: mockChanges['controller-utils'].dependencyChanges,
           existingEntries: [],
+          checkedVersion: null,
         },
       ]);
     });
@@ -73,6 +74,7 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: false,
           missingEntries: mockChanges['controller-utils'].dependencyChanges,
           existingEntries: [],
+          checkedVersion: null,
         },
       ]);
     });
@@ -102,6 +104,7 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: false,
           missingEntries: mockChanges['controller-utils'].dependencyChanges,
           existingEntries: [],
+          checkedVersion: null,
         },
       ]);
     });
@@ -131,6 +134,7 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: true,
           missingEntries: mockChanges['controller-utils'].dependencyChanges,
           existingEntries: [],
+          checkedVersion: null,
         },
       ]);
     });
@@ -166,6 +170,7 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: true,
           missingEntries: [],
           existingEntries: ['@metamask/transaction-controller'],
+          checkedVersion: null,
         },
       ]);
 
@@ -217,6 +222,50 @@ describe('changelog-validator', () => {
           hasUnreleasedSection: true,
           missingEntries: [],
           existingEntries: ['@metamask/transaction-controller'],
+          checkedVersion: '1.1.0',
+        },
+      ]);
+    });
+
+    it('catches error when release version section does not exist', async () => {
+      when(jest.spyOn(fsModule, 'fileExists'))
+        .calledWith('/path/to/project/packages/controller-utils/CHANGELOG.md')
+        .mockResolvedValue(true);
+      when(jest.spyOn(fsModule, 'readFile'))
+        .calledWith('/path/to/project/packages/controller-utils/CHANGELOG.md')
+        .mockResolvedValue('# Changelog\n## [Unreleased]');
+      jest.spyOn(packageModule, 'formatChangelog').mockResolvedValue('');
+
+      const mockChangelog = {
+        getUnreleasedChanges: jest.fn().mockReturnValue({ Changed: [] }),
+        getReleaseChanges: jest.fn().mockImplementation(() => {
+          throw new Error('Version not found');
+        }),
+      };
+      (parseChangelog as jest.Mock).mockReturnValue(mockChangelog);
+
+      const changesWithVersion = {
+        'controller-utils': {
+          ...mockChanges['controller-utils'],
+          newVersion: '1.2.3',
+        },
+      };
+
+      const results = await validateChangelogs(
+        changesWithVersion,
+        '/path/to/project',
+        'https://github.com/example-org/example-repo',
+      );
+
+      expect(mockChangelog.getReleaseChanges).toHaveBeenCalledWith('1.2.3');
+      expect(results).toStrictEqual([
+        {
+          package: 'controller-utils',
+          hasChangelog: true,
+          hasUnreleasedSection: false,
+          missingEntries: mockChanges['controller-utils'].dependencyChanges,
+          existingEntries: [],
+          checkedVersion: '1.2.3',
         },
       ]);
     });
@@ -314,6 +363,7 @@ describe('changelog-validator', () => {
             '@metamask/transaction-controller',
             '@metamask/transaction-controller',
           ],
+          checkedVersion: null,
         },
       ]);
     });

src/changelog-validator.ts

@@ -18,6 +18,8 @@ type ChangelogValidationResult = {
   hasUnreleasedSection: boolean;
   missingEntries: DependencyChange[];
   existingEntries: string[];
+  /** Version that was checked (null for [Unreleased] section) */
+  checkedVersion?: string | null;
 };
 
 /**
@@ -169,6 +171,7 @@ export async function validateChangelogs(
         hasUnreleasedSection: false,
         missingEntries: packageChanges,
         existingEntries: [],
+        checkedVersion: packageVersion ?? null,
       });
       continue;
     }
@@ -213,6 +216,7 @@ export async function validateChangelogs(
         hasUnreleasedSection,
         missingEntries,
         existingEntries,
+        checkedVersion: packageVersion ?? null,
       });
     } catch (error) {
       // If parsing fails, assume changelog is malformed
@@ -222,6 +226,7 @@ export async function validateChangelogs(
         hasUnreleasedSection: false,
         missingEntries: packageChanges,
         existingEntries: [],
+        checkedVersion: packageVersion ?? null,
       });
     }
   }

src/check-dependency-bumps.test.ts

@@ -235,6 +235,7 @@ index 1234567..890abcd 100644
             hasUnreleasedSection: true,
             missingEntries: [],
             existingEntries: ['@metamask/transaction-controller'],
+            checkedVersion: null,
           },
         ]);
 
@@ -616,6 +617,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
               },
             ],
             existingEntries: [],
+            checkedVersion: null,
           },
         ]);
 
@@ -684,6 +686,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
             hasUnreleasedSection: false,
             missingEntries: [],
             existingEntries: [],
+            checkedVersion: null,
           },
         ]);
 
@@ -701,6 +704,83 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
       );
     });
 
+    it('reports correct section name when checking release version', async () => {
+      const stderrWriteSpy = jest.spyOn(stderr, 'write');
+      const getStdoutSpy = jest.spyOn(miscUtilsModule, 'getStdoutFromCommand');
+
+      const diffWithVersion = `
+diff --git a/packages/controller-utils/package.json b/packages/controller-utils/package.json
+index 1234567..890abcd 100644
+--- a/packages/controller-utils/package.json
++++ b/packages/controller-utils/package.json
+@@ -1,6 +1,6 @@
+ {
+   "name": "@metamask/controller-utils",
+-  "version": "1.2.2",
++  "version": "1.2.3",
+   "dependencies": {
+-    "@metamask/transaction-controller": "^61.0.0"
++    "@metamask/transaction-controller": "^62.0.0"
+   }
+ }
+`;
+
+      when(getStdoutSpy)
+        .calledWith(
+          'git',
+          ['diff', '-U9999', 'abc123', 'HEAD', '--', '**/package.json'],
+          { cwd: '/path/to/project' },
+        )
+        .mockResolvedValue(diffWithVersion);
+
+      when(jest.spyOn(packageManifestModule, 'readPackageManifest'))
+        .calledWith('/path/to/project/package.json')
+        .mockResolvedValue({
+          unvalidated: {},
+          validated: buildMockManifest(),
+        });
+
+      when(jest.spyOn(packageManifestModule, 'readPackageManifest'))
+        .calledWith('/path/to/project/packages/controller-utils/package.json')
+        .mockResolvedValue({
+          unvalidated: {},
+          validated: buildMockManifest({
+            name: '@metamask/controller-utils',
+          }),
+        });
+
+      jest
+        .spyOn(projectModule, 'getValidRepositoryUrl')
+        .mockResolvedValue('https://github.com/example-org/example-repo');
+
+      const validateChangelogsSpy = jest
+        .spyOn(changelogValidatorModule, 'validateChangelogs')
+        .mockResolvedValue([
+          {
+            package: 'controller-utils',
+            hasChangelog: true,
+            hasUnreleasedSection: false,
+            missingEntries: [],
+            existingEntries: [],
+            checkedVersion: '1.2.3',
+          },
+        ]);
+
+      await checkDependencyBumps({
+        fromRef: 'abc123',
+        projectRoot: '/path/to/project',
+        stdout,
+        stderr,
+      });
+
+      expect(validateChangelogsSpy).toHaveBeenCalled();
+      expect(stderrWriteSpy).toHaveBeenCalledWith(
+        expect.stringContaining(
+          '❌ controller-utils: No [1.2.3] section found',
+        ),
+      );
+    });
+
     it('reports validation errors for missing changelog entries', async () => {
       const stderrWriteSpy = jest.spyOn(stderr, 'write');
       const getStdoutSpy = jest.spyOn(miscUtilsModule, 'getStdoutFromCommand');
@@ -759,6 +839,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
               },
             ],
             existingEntries: [],
+            checkedVersion: null,
           },
         ]);
 
@@ -846,6 +927,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
               },
             ],
             existingEntries: [],
+            checkedVersion: null,
           },
         ]);
 
@@ -919,6 +1001,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
             hasUnreleasedSection: true,
             missingEntries: [],
             existingEntries: ['@metamask/transaction-controller'],
+            checkedVersion: null,
           },
         ]);
 
@@ -992,6 +1075,7 @@ diff --git a/packages/controller-utils/package.json b/packages/controller-utils/
               },
             ],
             existingEntries: [],
+            checkedVersion: null,
           },
         ]);
 

src/check-dependency-bumps.ts

@@ -354,7 +354,10 @@ export async function checkDependencyBumps({
       stderr.write(`❌ ${result.package}: CHANGELOG.md not found\n`);
       hasErrors = true;
     } else if (!result.hasUnreleasedSection) {
-      stderr.write(`❌ ${result.package}: No [Unreleased] section found\n`);
+      const sectionName = result.checkedVersion
+        ? `[${result.checkedVersion}]`
+        : '[Unreleased]';
+      stderr.write(`❌ ${result.package}: No ${sectionName} section found\n`);
       hasErrors = true;
     } else if (result.missingEntries.length > 0) {
       stderr.write(