release-please 16.13.0 → 16.14.1

package/build/src/commit.js

@@ -127,6 +127,17 @@ function toConventionalChangelogFormat(ast) {
                 break;
         }
     });
+    // Add additional breaking change detection from commit body
+    if (body) {
+        const bodyString = String(body);
+        const breakingChangeMatch = bodyString.match(/BREAKING-CHANGE:\s*(.*)/);
+        if (breakingChangeMatch && breakingChangeMatch[1]) {
+            if (breaking.text) {
+                breaking.text += '\n';
+            }
+            breaking.text += breakingChangeMatch[1].trim();
+        }
+    }
     if (breaking.text !== '')
         headerCommit.notes.push(breaking);
     // Populates references array from footers:
@@ -241,6 +252,7 @@ function toConventionalChangelogFormat(ast) {
 // we should be able to move post processing into
 // to-conventional-changelog.ts.
 function postProcessCommits(commit) {
+    var _a;
     commit.notes.forEach(note => {
         let text = '';
         let i = 0;
@@ -262,6 +274,19 @@ function postProcessCommits(commit) {
         }
         note.text = text.trim();
     });
+    const breakingChangeMatch = (_a = commit.body) === null || _a === void 0 ? void 0 : _a.match(/BREAKING-CHANGE:\s*(.*)/);
+    if (breakingChangeMatch && breakingChangeMatch[1]) {
+        const existingNote = commit.notes.find(note => note.title === 'BREAKING CHANGE');
+        if (existingNote) {
+            existingNote.text += `\n${breakingChangeMatch[1].trim()}`;
+        }
+        else {
+            commit.notes.push({
+                title: 'BREAKING CHANGE',
+                text: breakingChangeMatch[1].trim(),
+            });
+        }
+    }
     return commit;
 }
 // If someone wishes to include additional contextual information for a
@@ -283,20 +308,29 @@ function hasExtendedContext(line) {
 function parseCommits(message) {
     return conventionalCommitsFilter(toConventionalChangelogFormat(parser.parser(message))).map(postProcessCommits);
 }
-// If someone wishes to aggregate multiple, complex commit messages into a
-// single commit, they can include one or more `BEGIN_NESTED_COMMIT`/`END_NESTED_COMMIT`
-// blocks into the body of the commit
+/**
+ * Splits a commit message into multiple messages based on conventional commit format and nested commit blocks.
+ * This function is capable of:
+ * 1. Separating conventional commits (feat, fix, docs, etc.) within the main message.
+ * 2. Extracting nested commits enclosed in BEGIN_NESTED_COMMIT/END_NESTED_COMMIT blocks.
+ * 3. Preserving the original message structure outside of nested commit blocks.
+ * 4. Handling multiple nested commits and conventional commits in a single message.
+ *
+ * @param message The input commit message string
+ * @returns An array of individual commit messages
+ */
 function splitMessages(message) {
     const parts = message.split('BEGIN_NESTED_COMMIT');
     const messages = [parts.shift()];
     for (const part of parts) {
         const [newMessage, ...rest] = part.split('END_NESTED_COMMIT');
         messages.push(newMessage);
-        // anthing outside of the BEGIN/END annotations are added to the original
-        // commit
-        messages[0] = messages[0] + rest.join();
+        messages[0] = messages[0] + rest.join('END_NESTED_COMMIT');
     }
-    return messages;
+    const conventionalCommits = messages[0]
+        .split(/\r?\n\r?\n(?=(?:feat|fix|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.*?\))?: )/)
+        .filter(Boolean);
+    return [...conventionalCommits, ...messages.slice(1)];
 }
 /**
  * Given a list of raw commits, parse and expand into conventional commits.

package/build/src/index.d.ts

@@ -14,4 +14,4 @@ export { Logger, setLogger } from './util/logger';
 export { GitHub } from './github';
 export declare const configSchema: any;
 export declare const manifestSchema: any;
-export declare const VERSION = "16.13.0";
+export declare const VERSION = "16.14.1";

package/build/src/index.js

@@ -36,6 +36,6 @@ Object.defineProperty(exports, "GitHub", { enumerable: true, get: function () {
 exports.configSchema = require('../../schemas/config.json');
 exports.manifestSchema = require('../../schemas/manifest.json');
 // x-release-please-start-version
-exports.VERSION = '16.13.0';
+exports.VERSION = '16.14.1';
 // x-release-please-end
 //# sourceMappingURL=index.js.map

package/build/src/updaters/generic-json.js

@@ -14,7 +14,7 @@
 // limitations under the License.
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.GenericJson = void 0;
-const jp = require("jsonpath");
+const jsonpath_plus_1 = require("jsonpath-plus");
 const json_stringify_1 = require("../util/json-stringify");
 const logger_1 = require("../util/logger");
 const VERSION_REGEX = /(?<major>\d+)\.(?<minor>\d+)\.(?<patch>\d+)(-(?<preRelease>[\w.]+))?(\+(?<build>[-\w.]+))?/;
@@ -30,21 +30,23 @@ class GenericJson {
      */
     updateContent(content, logger = logger_1.logger) {
         const data = JSON.parse(content);
-        const nodes = jp.apply(data, this.jsonpath, value => {
-            if (typeof value !== 'string') {
-                logger.warn(`No string in ${this.jsonpath}. Skipping.`);
-                return value;
-            }
-            if (!value.match(VERSION_REGEX)) {
-                logger.warn(`No version found in ${this.jsonpath}. Skipping.`);
-                return value;
-            }
-            return value.replace(VERSION_REGEX, this.version.toString());
+        (0, jsonpath_plus_1.JSONPath)({
+            resultType: 'all',
+            path: this.jsonpath,
+            json: data,
+            callback: (payload, _payloadType, _fullPayload) => {
+                if (typeof payload.value !== 'string') {
+                    logger.warn(`No string in ${this.jsonpath}. Skipping.`);
+                    return payload;
+                }
+                if (!payload.value.match(VERSION_REGEX)) {
+                    logger.warn(`No version found in ${this.jsonpath}. Skipping.`);
+                    return payload;
+                }
+                payload.parent[payload.parentProperty] = payload.parent[payload.parentProperty].replace(VERSION_REGEX, this.version.toString());
+                return payload;
+            },
         });
-        if (!nodes) {
-            logger.warn(`No entries modified in ${this.jsonpath}`);
-            return content;
-        }
         return (0, json_stringify_1.jsonStringify)(data, content);
     }
 }

package/build/src/updaters/generic-toml.js

@@ -14,7 +14,7 @@
 // limitations under the License.
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.GenericToml = void 0;
-const jp = require("jsonpath");
+const jsonpath_plus_1 = require("jsonpath-plus");
 const toml_edit_1 = require("../util/toml-edit");
 const logger_1 = require("../util/logger");
 /**
@@ -43,7 +43,12 @@ class GenericToml {
             logger.warn('Invalid toml, cannot be parsed', e);
             return content;
         }
-        const paths = jp.paths(data, this.jsonpath);
+        const pointers = (0, jsonpath_plus_1.JSONPath)({
+            path: this.jsonpath,
+            json: data,
+            resultType: 'pointer',
+        });
+        const paths = pointers.map(pointer => pointer.split('/').filter(Boolean));
         if (!paths || paths.length === 0) {
             logger.warn(`No entries modified in ${this.jsonpath}`);
             return content;

package/build/src/updaters/generic-yaml.js

@@ -14,7 +14,7 @@
 // limitations under the License.
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.GenericYaml = void 0;
-const jp = require("jsonpath");
+const jsonpath_plus_1 = require("jsonpath-plus");
 const yaml = require("js-yaml");
 const logger_1 = require("../util/logger");
 const DOCUMENT_SEPARATOR = '---\n';
@@ -50,12 +50,20 @@ class GenericYaml {
         // Update each document
         let modified = false;
         docs.forEach(data => {
-            const nodes = jp.apply(data, this.jsonpath, _val => {
-                return this.version.toString();
+            (0, jsonpath_plus_1.JSONPath)({
+                resultType: 'all',
+                path: this.jsonpath,
+                json: data,
+                callback: (payload, _payloadType, _fullPayload) => {
+                    if (typeof payload.value !== 'string') {
+                        logger.warn(`No string in ${this.jsonpath}. Skipping.`);
+                        return payload;
+                    }
+                    modified = true;
+                    payload.parent[payload.parentProperty] = this.version.toString();
+                    return payload;
+                },
             });
-            if (nodes && nodes.length) {
-                modified = true;
-            }
         });
         // If nothing was modified, return original content
         if (!modified) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "release-please",
-  "version": "16.13.0",
+  "version": "16.14.1",
   "description": "generate release PRs based on the conventionalcommits.org spec",
   "main": "./build/src/index.js",
   "bin": "./build/src/bin/release-please.js",
@@ -86,7 +86,7 @@
     "http-proxy-agent": "^7.0.0",
     "https-proxy-agent": "^7.0.0",
     "js-yaml": "^4.0.0",
-    "jsonpath": "^1.1.1",
+    "jsonpath-plus": "^9.0.0",
     "node-html-parser": "^6.0.0",
     "parse-github-repo-url": "^1.4.1",
     "semver": "^7.5.3",