@fern-api/fern-api-dev 4.90.0-13-g88979a56bb3 → 4.90.0-14-g871c1a60e90

package/cli.cjs

@@ -332867,21 +332867,33 @@ var require_AutoVersionStep = __commonJS({
       async handleNormalFlow(params2) {
         const { prepared, service, language, mappedMagicVersion, previousGenerationSha } = params2;
         const rawDiff = this.gitDiff(previousGenerationSha, prepared.currentGenerationSha);
-        if (rawDiff.trim().length === 0) {
-          this.logger.info("AutoVersionStep: empty diff between generations; skipping autoversion.");
-          return { executed: true, success: true };
-        }
         const previousVersion = await this.resolvePreviousVersion({ service, rawDiff, mappedMagicVersion });
         if (previousVersion == null) {
           return await this.handleFirstGeneration({ service, language, mappedMagicVersion });
         }
+        if (rawDiff.trim().length === 0) {
+          this.logger.info(`AutoVersionStep: empty diff between generations; rewriting placeholder to ${previousVersion}.`);
+          return await this.finalizeNoChange({
+            service,
+            language,
+            mappedMagicVersion,
+            previousVersion,
+            reason: "no diff between generations"
+          });
+        }
         const cleanedDiff = service.cleanDiffForAI(rawDiff, mappedMagicVersion);
         const rawBytes = Buffer.byteLength(rawDiff, "utf-8");
         const cleanedBytes = Buffer.byteLength(cleanedDiff, "utf-8");
         this.logger.debug(`AutoVersionStep: raw=${(0, index_1.formatSizeKB)(rawBytes)}KB (${(0, index_1.countFilesInDiff)(rawDiff)} files), cleaned=${(0, index_1.formatSizeKB)(cleanedBytes)}KB (${(0, index_1.countFilesInDiff)(cleanedDiff)} files)`);
         if (cleanedDiff.trim().length === 0) {
-          this.logger.info("AutoVersionStep: cleaned diff is empty; no semantic changes.");
-          return { executed: true, success: true };
+          this.logger.info(`AutoVersionStep: cleaned diff is empty; no semantic changes \u2014 rewriting placeholder to ${previousVersion}.`);
+          return await this.finalizeNoChange({
+            service,
+            language,
+            mappedMagicVersion,
+            previousVersion,
+            reason: "no semantic changes"
+          });
         }
         if (cleanedBytes > index_1.MAX_RAW_DIFF_BYTES) {
           this.logger.warn(`AutoVersionStep: diff too large for FAI (${(0, index_1.formatSizeKB)(cleanedBytes)}KB, limit ${(0, index_1.formatSizeKB)(index_1.MAX_RAW_DIFF_BYTES)}KB). Falling back to PATCH.`);
@@ -332907,13 +332919,14 @@ var require_AutoVersionStep = __commonJS({
           analysis = { versionBump: "PATCH", message: this.brandMessage("SDK regeneration") };
         }
         if (analysis == null) {
-          this.logger.info("AutoVersionStep: FAI returned NO_CHANGE; skipping autoversion commit.");
-          return {
-            executed: true,
-            success: true,
+          this.logger.info(`AutoVersionStep: FAI returned NO_CHANGE; rewriting placeholder to ${previousVersion}.`);
+          return await this.finalizeNoChange({
+            service,
+            language,
+            mappedMagicVersion,
             previousVersion,
-            versionBump: "NO_CHANGE"
-          };
+            reason: "FAI returned NO_CHANGE"
+          });
         }
         return await this.finalizeWithBump({
           service,
@@ -332923,6 +332936,38 @@ var require_AutoVersionStep = __commonJS({
           analysis
         });
       }
+      /**
+       * Terminal path for runs that determine no semver bump is needed. The
+       * current working tree still contains `0.0.0-fern-placeholder` from this
+       * run's generator output, so we must rewrite it to `previousVersion`
+       * before exiting — otherwise the SDK ships with the placeholder embedded
+       * in manifests (package.json, pyproject.toml, .fern/metadata.json) and
+       * user-agent strings (e.g. `X-Fern-SDK-Version`). Commits the rewrite as
+       * `[fern-autoversion]` so the replay + github steps see a stable base.
+       */
+      async finalizeNoChange(params2) {
+        const { service, language, mappedMagicVersion, previousVersion, reason } = params2;
+        if (!(0, index_1.isValidSemver)(previousVersion)) {
+          const errorMessage = `AutoVersionStep: resolved previousVersion ${JSON.stringify(previousVersion)} is not a valid semver string. Refusing to rewrite placeholder to avoid shell injection.`;
+          this.logger.error(errorMessage);
+          return { executed: true, success: false, errorMessage };
+        }
+        await service.replaceMagicVersion(this.outputDir, mappedMagicVersion, previousVersion);
+        if (language === "go") {
+          await service.addGoMajorVersionSuffix(this.outputDir, previousVersion);
+        }
+        const commitMessage = this.brandMessage(`SDK regeneration (no semver change: ${reason})`);
+        const commitSha = this.commitAutoversion(commitMessage);
+        return {
+          executed: true,
+          success: true,
+          version: previousVersion,
+          previousVersion,
+          versionBump: "NO_CHANGE",
+          commitMessage,
+          commitSha
+        };
+      }
       async finalizeWithBump(params2) {
         const { service, language, mappedMagicVersion, previousVersion, analysis } = params2;
         const newVersion = (0, index_1.incrementVersion)(previousVersion, analysis.versionBump);
@@ -343160,6 +343205,7 @@ var require_GithubStep = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.GithubStep = void 0;
+    exports2.resolvePrFields = resolvePrFields;
     exports2.enrichPrBodyForAutomation = enrichPrBodyForAutomation;
     exports2.shouldEnableAutomerge = shouldEnableAutomerge;
     exports2.shouldCheckNoDiff = shouldCheckNoDiff;
@@ -343185,8 +343231,10 @@ var require_GithubStep = __commonJS({
       }
       async execute(context3) {
         const replayResult = context3.previousStepResults.replay;
+        const autoVersionResult = context3.previousStepResults.autoVersion;
         const skipCommit = this.config.skipCommit ?? this.deriveSkipCommit(replayResult);
         const replayConflictInfo = this.config.replayConflictInfo ?? this.deriveReplayConflictInfo(replayResult);
+        const resolvedPrFields = this.resolvePrFields(autoVersionResult);
         try {
           this.logger.debug("Starting GitHub self-hosted flow in directory: " + this.outputDir);
           const repository = github_1.ClonedRepository.createAtPath(this.outputDir);
@@ -343200,9 +343248,9 @@ var require_GithubStep = __commonJS({
           const mode = this.config.mode;
           switch (mode) {
             case "pull-request":
-              return await this.executePullRequestMode(repository, newPrBranch, skipCommit, replayConflictInfo, replayResult);
+              return await this.executePullRequestMode(repository, newPrBranch, skipCommit, replayConflictInfo, replayResult, resolvedPrFields);
             case "push":
-              return await this.executePushMode(repository);
+              return await this.executePushMode(repository, resolvedPrFields);
             default: {
               const exhaustive = mode;
               throw new Error(`Unexpected GitHub mode: ${String(exhaustive)}`);
@@ -343218,7 +343266,7 @@ var require_GithubStep = __commonJS({
           };
         }
       }
-      async executePullRequestMode(repository, newPrBranch, skipCommit, replayConflictInfo, replayResult) {
+      async executePullRequestMode(repository, newPrBranch, skipCommit, replayConflictInfo, replayResult, resolved) {
         const baseBranch = this.config.branch ?? await repository.getDefaultBranch();
         const octokit = new rest_1.Octokit({ auth: this.config.token });
         const { owner, repo } = (0, github_1.parseRepository)(this.config.uri);
@@ -343245,7 +343293,7 @@ var require_GithubStep = __commonJS({
         switch (branchAction) {
           case "replay-branch":
             if (shouldPushGenerationBaseTag(replayResult)) {
-              generationBaseSha = await (0, createReplayBranch_1.createReplayBranch)(repository, prBranch, this.config.commitMessage, replayConflictInfo, this.logger);
+              generationBaseSha = await (0, createReplayBranch_1.createReplayBranch)(repository, prBranch, resolved.commitMessage, replayConflictInfo, this.logger);
             } else {
               await repository.createBranchFromHead(prBranch);
             }
@@ -343264,8 +343312,7 @@ var require_GithubStep = __commonJS({
         if (!skipCommit) {
           await this.ensureFernignore();
           this.logger.debug("Committing changes...");
-          const finalCommitMessage2 = this.config.commitMessage ?? "SDK Generation";
-          await repository.commitAllChanges(finalCommitMessage2);
+          await repository.commitAllChanges(resolved.commitMessage);
           this.logger.debug(`Committed changes to local copy of GitHub repository at ${this.outputDir}`);
         }
         if (shouldCheckNoDiff(this.config)) {
@@ -343305,13 +343352,12 @@ var require_GithubStep = __commonJS({
             }
           }
         }
-        const finalCommitMessage = this.config.commitMessage ?? "SDK Generation";
         const headSha = await repository.getHeadSha();
-        const changelogUrl = this.config.changelogEntry ? `https://github.com/${this.config.uri}/blob/${headSha}/changelog.md` : void 0;
-        const { prTitle, prBody } = (0, parseCommitMessage_1.parseCommitMessageForPR)(finalCommitMessage, this.config.changelogEntry, this.config.prDescription, this.config.versionBumpReason, this.config.previousVersion, this.config.newVersion, this.config.versionBump, changelogUrl);
+        const changelogUrl = resolved.changelogEntry ? `https://github.com/${this.config.uri}/blob/${headSha}/changelog.md` : void 0;
+        const { prTitle, prBody } = (0, parseCommitMessage_1.parseCommitMessageForPR)(resolved.commitMessage, resolved.changelogEntry, resolved.prDescription, resolved.versionBumpReason, resolved.previousVersion, resolved.newVersion, resolved.versionBump, changelogUrl);
         const replaySection = (0, replay_summary_1.formatReplayPrBody)(replayResult, { branchName: prBranch, repoUri: this.config.uri });
         let enrichedBody = replaySection != null ? prBody + "\n\n---\n\n" + replaySection : prBody;
-        enrichedBody = enrichPrBodyForAutomation(enrichedBody, this.config);
+        enrichedBody = enrichPrBodyForAutomation(enrichedBody, this.config, resolved);
         if (isUpdatingExistingPR && existingPR != null) {
           this.logger.info(`Updated existing pull request: ${existingPR.htmlUrl}`);
           result.prUrl = existingPR.htmlUrl;
@@ -343342,7 +343388,7 @@ var require_GithubStep = __commonJS({
             this.logger.info(`Created pull request: ${pullRequest.html_url}`);
             result.prUrl = pullRequest.html_url;
             result.prNumber = pullRequest.number;
-            if (shouldEnableAutomerge(this.config) && pullRequest.node_id) {
+            if (shouldEnableAutomerge(this.config, resolved) && pullRequest.node_id) {
               try {
                 await octokit.graphql(`mutation($pullRequestId: ID!) {
                                 enablePullRequestAutoMerge(input: {
@@ -343367,7 +343413,7 @@ var require_GithubStep = __commonJS({
         }
         return result;
       }
-      async executePushMode(repository) {
+      async executePushMode(repository, resolved) {
         const baseBranch = this.config.branch ?? await repository.getDefaultBranch();
         if (this.config.branch != null) {
           this.logger.debug(`Checking out branch ${this.config.branch}`);
@@ -343375,8 +343421,7 @@ var require_GithubStep = __commonJS({
         }
         await this.ensureFernignore();
         this.logger.debug("Committing changes...");
-        const finalCommitMessage = this.config.commitMessage ?? "SDK Generation";
-        await repository.commitAllChanges(finalCommitMessage);
+        await repository.commitAllChanges(resolved.commitMessage);
         this.logger.debug(`Committed changes to local copy of GitHub repository at ${this.outputDir}`);
         if (shouldCheckNoDiff(this.config)) {
           const noDiff = await repository.treeHashEquals(`origin/${baseBranch}`);
@@ -343439,15 +343484,34 @@ var require_GithubStep = __commonJS({
         }
         return void 0;
       }
+      resolvePrFields(autoVersion) {
+        return resolvePrFields(this.config, autoVersion);
+      }
     };
     exports2.GithubStep = GithubStep;
-    function enrichPrBodyForAutomation(body, config5) {
+    function resolvePrFields(config5, autoVersion) {
+      const autoVersionBreaking = autoVersion?.versionBump === "MAJOR";
+      return {
+        commitMessage: config5.commitMessage ?? autoVersion?.commitMessage ?? "SDK Generation",
+        changelogEntry: config5.changelogEntry ?? autoVersion?.changelogEntry,
+        prDescription: config5.prDescription ?? autoVersion?.prDescription,
+        versionBumpReason: config5.versionBumpReason ?? autoVersion?.versionBumpReason,
+        previousVersion: config5.previousVersion ?? autoVersion?.previousVersion,
+        newVersion: config5.newVersion ?? autoVersion?.version,
+        versionBump: config5.versionBump ?? autoVersion?.versionBump,
+        hasBreakingChanges: config5.hasBreakingChanges ?? autoVersionBreaking,
+        breakingChangesSummary: config5.breakingChangesSummary ?? autoVersion?.prDescription
+      };
+    }
+    function enrichPrBodyForAutomation(body, config5, breaking = {}) {
       if (!config5.automationMode) {
         return body;
       }
+      const hasBreakingChanges = breaking.hasBreakingChanges ?? config5.hasBreakingChanges;
+      const breakingChangesSummary = breaking.breakingChangesSummary ?? config5.breakingChangesSummary;
       let result = body;
-      if (config5.hasBreakingChanges && config5.breakingChangesSummary) {
-        result += "\n\n---\n\n## \u26A0\uFE0F Breaking Changes\n\nThis release contains breaking changes that require manual review:\n\n" + config5.breakingChangesSummary;
+      if (hasBreakingChanges && breakingChangesSummary) {
+        result += "\n\n---\n\n## \u26A0\uFE0F Breaking Changes\n\nThis release contains breaking changes that require manual review:\n\n" + breakingChangesSummary;
       }
       if (config5.runId) {
         result += `
@@ -343458,8 +343522,9 @@ _Fern Run ID: \`${config5.runId}\`_`;
       }
       return result;
     }
-    function shouldEnableAutomerge(config5) {
-      return config5.automationMode === true && config5.autoMerge === true && config5.hasBreakingChanges !== true;
+    function shouldEnableAutomerge(config5, breaking = {}) {
+      const hasBreakingChanges = breaking.hasBreakingChanges ?? config5.hasBreakingChanges;
+      return config5.automationMode === true && config5.autoMerge === true && hasBreakingChanges !== true;
     }
     function shouldCheckNoDiff(config5) {
       return config5.skipIfNoDiff === true;
@@ -618177,7 +618242,7 @@ var AccessTokenPosthogManager = class {
         properties: {
           ...event,
           ...event.properties,
-          version: "4.90.0-13-g88979a56bb3",
+          version: "4.90.0-14-g871c1a60e90",
           usingAccessToken: true
         }
       });
@@ -618231,7 +618296,7 @@ var UserPosthogManager = class {
       distinctId: this.userId ?? await this.getPersistedDistinctId(),
       event: "CLI",
       properties: {
-        version: "4.90.0-13-g88979a56bb3",
+        version: "4.90.0-14-g871c1a60e90",
         ...event,
         ...event.properties,
         usingAccessToken: false,
@@ -835799,7 +835864,7 @@ var LOCAL_STORAGE_FOLDER4 = ".fern-dev";
 var LOGS_FOLDER_NAME = "logs";
 var MAX_LOGS_DIR_SIZE_BYTES = 100 * 1024 * 1024;
 function getCliSource() {
-  const version7 = "4.90.0-13-g88979a56bb3";
+  const version7 = "4.90.0-14-g871c1a60e90";
   return `cli@${version7}`;
 }
 var DebugLogger = class {
@@ -847550,7 +847615,7 @@ var LegacyDocsPublisher = class {
         previewId,
         disableTemplates: void 0,
         skipUpload,
-        cliVersion: "4.90.0-13-g88979a56bb3"
+        cliVersion: "4.90.0-14-g871c1a60e90"
       });
       if (taskContext.getResult() === TaskResult.Failure) {
         return { success: false };
@@ -921588,7 +921653,7 @@ var CliContext = class _CliContext {
     if (false) {
       this.logger.error("CLI_VERSION is not defined");
     }
-    return "4.90.0-13-g88979a56bb3";
+    return "4.90.0-14-g871c1a60e90";
   }
   getCliName() {
     if (false) {

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "4.90.0-13-g88979a56bb3",
+  "version": "4.90.0-14-g871c1a60e90",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/fern-api/fern.git",