@entro314labs/release-kit 2.3.1 → 2.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@entro314labs/release-kit",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "description": "Single-file, zero-dependency release mechanism for JS/TS/Node projects: version bump, changelog roll, commit, annotated tag, push, publish, GitHub release",
   "keywords": [
     "changelog",

package/release.mjs

@@ -756,9 +756,18 @@ function changelogSection(text, version) {
  * @returns {string | null} the updated document, or null when there is nothing to roll
  */
 function rollUnreleased(text, version, date) {
+  // A heading for this version already exists — possibly with an empty body, which
+  // `changelogSection` reports as absent. Rolling again would duplicate the heading.
+  if (new RegExp(`^##\\s+\\[?v?${escapeRe(version)}\\]?(?![\\w.-])`, 'm').test(text)) return null
+
   const heading = /^##\s+\[?Unreleased\]?[^\n]*$/im
   const match = heading.exec(text)
   if (!match) return null
+
+  // An empty [Unreleased] has nothing to promote; rolling it produces an empty section.
+  const rest = text.slice(match.index + match[0].length)
+  const body = (/^## /m.exec(rest) ? rest.slice(0, /^## /m.exec(rest).index) : rest).trim()
+  if (!body) return null
   const released = `## [Unreleased]\n\n## [${version}] - ${date}`
   return text.slice(0, match.index) + released + text.slice(match.index + match[0].length)
 }
@@ -1515,6 +1524,18 @@ for (const asset of config.assets) {
   else fail(`asset ${asset} does not exist`)
 }
 
+// Reusing a tag is the resume path, and a resume writes nothing. If this run would still
+// produce a commit, that commit moves HEAD past the tag and the release ends up tagged at
+// the wrong revision — which is silent until someone checks out the tag.
+if (taggedCommit && runs('tag') && (bumping || rolledChangelog)) {
+  fail(
+    `tag ${tag} already exists at HEAD, but this run would still commit ` +
+      `${[bumping && 'a version bump', rolledChangelog && 'a changelog entry'].filter(Boolean).join(' and ')}.\n` +
+      '       That commit would leave the tag behind HEAD. Release a new version, or use ' +
+      '--only with the steps that remain.',
+  )
+}
+
 if (problems.length) {
   const summary = `${problems.length} preflight check(s) failed:\n  - ${problems.join('\n  - ')}`
   if (!dryRun) abort(summary)