feat!: align hot-path session memory with context-mode

Author: vicary

.github/scripts/version.test.ts

@@ -11,6 +11,7 @@ import {
   applyBump,
   calculateVersion,
   findReleaseAs,
+  hasBreakingChangeBody,
   hasNonTestChanges,
   parseSemver,
 } from "./version.ts";
@@ -45,6 +46,7 @@ describe("analyzeCommits", () => {
 
   it("returns 'major' for breaking change with ! suffix", () => {
     assertEquals(analyzeCommits(["feat!: breaking change"]), "major");
+    assertEquals(analyzeCommits(["fix!: breaking fix"]), "major");
   });
 
   it("returns 'major' for breaking change with BREAKING CHANGE in subject", () => {
@@ -61,6 +63,15 @@ describe("analyzeCommits", () => {
     );
   });
 
+  it("returns 'major' when a commit body contains BREAKING CHANGE", () => {
+    assertEquals(
+      analyzeCommits(["feat: keep subject normal"], [
+        "BREAKING CHANGE: api changed",
+      ]),
+      "major",
+    );
+  });
+
   it("returns highest bump when mixed commits (feat + fix → minor)", () => {
     const subjects = [
       "fix: bug fix",
@@ -116,6 +127,24 @@ describe("analyzeCommits", () => {
   });
 });
 
+describe("hasBreakingChangeBody", () => {
+  it("returns true for semantic-release style breaking change bodies", () => {
+    assertEquals(
+      hasBreakingChangeBody([
+        "Some text\n\nBREAKING CHANGE: changed output format",
+      ]),
+      true,
+    );
+  });
+
+  it("returns false when commit bodies do not include the breaking footer", () => {
+    assertEquals(
+      hasBreakingChangeBody(["Regular body", "Another body"]),
+      false,
+    );
+  });
+});
+
 describe("findReleaseAs", () => {
   it("returns undefined for empty array", () => {
     assertEquals(findReleaseAs([]), undefined);
@@ -349,6 +378,16 @@ describe("calculateVersion", () => {
       assertEquals(result, { skip: false, version: "2.0.0", tag: "latest" });
     });
 
+    it("creates release version for BREAKING CHANGE in commit body", () => {
+      const result = calculateVersion({
+        ...baseOpts,
+        subjects: ["feat: keep subject stable"],
+        bodies: ["BREAKING CHANGE: api changed"],
+        eventName: "push",
+      });
+      assertEquals(result, { skip: false, version: "2.0.0", tag: "latest" });
+    });
+
     it("skips when no triggering commits", () => {
       const result = calculateVersion({
         ...baseOpts,
@@ -468,6 +507,20 @@ describe("calculateVersion", () => {
       });
     });
 
+    it("creates canary for BREAKING CHANGE in commit body", () => {
+      const result = calculateVersion({
+        ...baseOpts,
+        subjects: ["fix: preserve subject format"],
+        bodies: ["BREAKING CHANGE: cache schema changed"],
+        eventName: "pull_request",
+      });
+      assertEquals(result, {
+        skip: false,
+        version: "2.0.0-canary.abc123d.20260212091429",
+        tag: "canary",
+      });
+    });
+
     it("shortens commit SHA to 7 characters", () => {
       const result = calculateVersion({
         ...baseOpts,
@@ -618,5 +671,20 @@ describe("calculateVersion", () => {
         tag: "canary",
       });
     });
+
+    it("creates a 0.x canary minor bump from BREAKING CHANGE in body", () => {
+      const result = calculateVersion({
+        ...baseOpts,
+        currentVersion: "0.1.12",
+        subjects: ["feat: keep subject stable"],
+        bodies: ["BREAKING CHANGE: overhaul session-memory semantics"],
+        eventName: "pull_request",
+      });
+      assertEquals(result, {
+        skip: false,
+        version: "0.2.0-canary.abc123d.20260212091429",
+        tag: "canary",
+      });
+    });
   });
 });

.github/scripts/version.ts

@@ -19,15 +19,31 @@ export type VersionResult =
   | { skip: false; version: string; tag: "latest" | "canary" };
 
 /**
- * Analyze conventional commit subjects and return the highest bump type.
+ * Returns true when any commit body contains a semantic-release style breaking
+ * change footer/header such as `BREAKING CHANGE: details`.
+ */
+export function hasBreakingChangeBody(bodies: string[]): boolean {
+  return bodies.some((body) => /^BREAKING CHANGE:/im.test(body));
+}
+
+/**
+ * Analyze conventional commits and return the highest bump type.
+ *
+ * Supported formats:
+ * - `feat: add feature` -> minor
+ * - `fix: resolve bug` / `perf: speed up path` -> patch
+ * - `feat!: breaking api change` / `fix!: breaking bugfix` -> major
+ * - `BREAKING CHANGE: explanation` in a commit body -> major
+ * - `Release-As: x.y.z` is handled separately as an exact override
  *
- * Rules:
- * - `BREAKING CHANGE` in body or `type!:` → major
- * - `feat:` → minor
- * - `fix:` / `perf:` → patch
- * - Anything else → none
+ * In `0.x`, a major bump resolves to the next minor version.
  */
-export function analyzeCommits(subjects: string[]): Bump {
+export function analyzeCommits(
+  subjects: string[],
+  bodies: string[] = [],
+): Bump {
+  if (hasBreakingChangeBody(bodies)) return "major";
+
   let bump: Bump = "none";
 
   for (const msg of subjects) {
@@ -121,7 +137,7 @@ export function calculateVersion(opts: {
   currentVersion: string;
   /** Conventional commit subjects since last release. */
   subjects: string[];
-  /** Commit bodies (for Release-As detection). */
+  /** Commit bodies (for Release-As and BREAKING CHANGE detection). */
   bodies: string[];
   /** Whether this is a "push" (release) or "pull_request" (canary). */
   eventName: "push" | "pull_request";
@@ -151,8 +167,8 @@ export function calculateVersion(opts: {
     return { skip: false, version, tag } as const;
   }
 
-  // Analyze commits
-  let bump = analyzeCommits(opts.subjects);
+  // Analyze commits using subjects plus semantic-release style body footers.
+  let bump = analyzeCommits(opts.subjects, opts.bodies);
 
   // When no git tags, default to patch bump from npm baseline
   if (opts.noGitTags && bump === "none") {