fix(release): strip prefix before comparing version

lib/releases.js

@@ -5,11 +5,17 @@ const { getVersionInfo } = require('./versions')
 const { template } = require('./template')
 const { log } = require('./log')
 
-const sortReleases = (releases) => {
+const sortReleases = (releases, tagPrefix) => {
   // For semver, we find the greatest release number
   // For non-semver, we use the most recently merged
   try {
-    return releases.sort((r1, r2) => compareVersions(r1.tag_name, r2.tag_name))
+    const tagPrefixRexExp = new RegExp(`^${regexEscape(tagPrefix)}`)
+    return releases.sort((r1, r2) =>
+      compareVersions(
+        r1.tag_name.replace(tagPrefixRexExp, ''),
+        r2.tag_name.replace(tagPrefixRexExp, '')
+      )
+    )
   } catch {
     return releases.sort(
       (r1, r2) => new Date(r1.created_at) - new Date(r2.created_at)
@@ -57,7 +63,8 @@ const findReleases = async ({
     ? commitishFilteredReleases.filter((r) => r.tag_name.startsWith(tagPrefix))
     : commitishFilteredReleases
   const sortedPublishedReleases = sortReleases(
-    filteredReleases.filter((r) => !r.draft)
+    filteredReleases.filter((r) => !r.draft),
+    tagPrefix
   )
   const draftRelease = filteredReleases.find((r) => r.draft)
   const lastRelease =

test/releases.test.js

@@ -1,4 +1,4 @@
-const { generateChangeLog } = require('../lib/releases')
+const { generateChangeLog, findReleases } = require('../lib/releases')
 const { DEFAULT_CONFIG } = require('../lib/default-config')
 
 const pullRequests = [
@@ -234,4 +234,46 @@ describe('releases', () => {
       `)
     })
   })
+
+  describe('findReleases', () => {
+    it('should retrieve last release respecting semver, stripped prefix', async () => {
+      const paginate = jest.fn().mockResolvedValue([
+        {
+          tag_name: 'test-1.0.1',
+          target_commitish: 'master',
+          created_at: new Date('2022-01-01'),
+        },
+        {
+          tag_name: 'test-1.0.0',
+          target_commitish: 'master',
+          created_at: new Date('2022-02-01'),
+        },
+      ])
+
+      const context = {
+        log: {
+          info: jest.fn(),
+        },
+        repo: jest.fn(),
+        payload: {
+          repository: 'test',
+        },
+        octokit: {
+          paginate,
+          repos: { listReleases: { endpoint: { merge: jest.fn() } } },
+        },
+      }
+      const targetCommitish = 'refs/heads/master'
+      const filterByCommitish = ''
+      const tagPrefix = 'test-'
+
+      const { lastRelease } = await findReleases({
+        context,
+        targetCommitish,
+        filterByCommitish,
+        tagPrefix,
+      })
+      expect(lastRelease.tag_name).toEqual('test-1.0.1')
+    })
+  })
 })