fix: resolve AI response format error and error handling issues

CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+### Fixed
+- Fixed OpenAI API error: only use response_format for models that support it (not gpt-4o-mini)
+- Fixed commit message escaping to handle newlines and quotes in git commands
+- Fixed error handling in ReleaseWorkflow to properly rethrow errors instead of returning default values
+- Tool now properly reports failures instead of misleading success messages
+
 ## [1.0.3] - 2025-08-12
 
 ### Fixed

dist/index.js

@@ -27659,7 +27659,8 @@ class GitOperations {
                 // There are staged changes, proceed with commit
             }
             // Commit changes
-            this.executeGitCommand(`commit -m "${message}"`);
+            const escapedMessage = message.replace(/"/g, '\\"').replace(/\n/g, '\\n');
+            this.executeGitCommand(`commit -m "${escapedMessage}"`);
             const commitSha = this.executeGitCommand('rev-parse HEAD').trim();
             core.info(`✅ Committed changes: ${commitSha.substring(0, 7)}`);
             return commitSha;
@@ -36175,7 +36176,10 @@ Respond with valid JSON in this exact format:
                 ],
                 temperature: this.config.temperature,
                 max_tokens: this.config.maxTokens,
-                response_format: { type: 'json_object' },
+                // Only use response_format for models that support it
+                ...(this.config.model.includes('gpt-4') && !this.config.model.includes('mini')
+                    ? { response_format: { type: 'json_object' } }
+                    : {}),
             });
             const choice = response.choices[0];
             if (choice?.message?.content === undefined ||
@@ -36909,18 +36913,7 @@ class ReleaseWorkflow {
                 ? error
                 : new WorkflowError(error instanceof Error ? error.message : 'Unknown workflow error', 'WORKFLOW_ERROR');
             core.setFailed(workflowError.message);
-            return {
-                version: '0.0.0',
-                releaseBranch: '',
-                versionBump: 'patch',
-                changelogEntry: '',
-                analysisStrategy: this.config.versioningStrategy,
-                aiConfidence: undefined,
-                reasoning: [workflowError.message],
-                commitCount: 0,
-                breakingChanges: false,
-                changelogData: undefined,
-            };
+            throw workflowError;
         }
     }
     /**

src/ReleaseWorkflow.ts

@@ -211,19 +211,7 @@ export class ReleaseWorkflow {
             );
 
       core.setFailed(workflowError.message);
-
-      return {
-        version: '0.0.0',
-        releaseBranch: '',
-        versionBump: 'patch',
-        changelogEntry: '',
-        analysisStrategy: this.config.versioningStrategy,
-        aiConfidence: undefined,
-        reasoning: [workflowError.message],
-        commitCount: 0,
-        breakingChanges: false,
-        changelogData: undefined,
-      };
+      throw workflowError;
     }
   }
 

src/ai-analyzer.ts

@@ -310,7 +310,10 @@ Respond with valid JSON in this exact format:
         ],
         temperature: this.config.temperature,
         max_tokens: this.config.maxTokens,
-        response_format: { type: 'json_object' },
+        // Only use response_format for models that support it
+        ...(this.config.model.includes('gpt-4') && !this.config.model.includes('mini')
+          ? { response_format: { type: 'json_object' } }
+          : {}),
       });
 
       const choice = response.choices[0];

src/git-operations.ts

@@ -209,7 +209,7 @@ export class GitOperations {
             // First, fetch all remote refs to ensure we have the latest information
             this.executeGitCommand('fetch origin --prune');
             core.debug(`Fetched latest remote references`);
-            
+
             // Try remote branch again after fetch
             this.executeGitCommand(`rev-parse --verify origin/${branch}`);
             core.debug(`Branch '${branch}' found as remote branch 'origin/${branch}' after fetch`);
@@ -218,7 +218,7 @@ export class GitOperations {
             try {
               this.executeGitCommand(`fetch origin ${branch}`);
               core.debug(`Fetched specific branch '${branch}' from remote`);
-              
+
               // Verify it exists as a remote branch now
               this.executeGitCommand(`rev-parse --verify origin/${branch}`);
               core.debug(`Branch '${branch}' verified as remote branch after specific fetch`);
@@ -251,7 +251,8 @@ export class GitOperations {
       }
 
       // Commit changes
-      this.executeGitCommand(`commit -m "${message}"`);
+      const escapedMessage = message.replace(/"/g, '\\"').replace(/\n/g, '\\n');
+      this.executeGitCommand(`commit -m "${escapedMessage}"`);
       const commitSha = this.executeGitCommand('rev-parse HEAD').trim();
 
       core.info(`✅ Committed changes: ${commitSha.substring(0, 7)}`);