Skip to content

Conversation

@gmorpheme
Copy link
Member

Summary

Fix the Claude API release notes generation that's currently producing "null" output by adding comprehensive debugging and improving error handling.

Problem

The Claude API integration is failing silently, producing "null" in CHANGELOG.md instead of proper release notes. The API key is present but something in the request/response handling is broken.

Changes Made

  • Comprehensive logging: Added step-by-step debugging output to trace the entire process
  • Proper JSON escaping: Use jq to create API payload instead of string interpolation to avoid JSON syntax errors
  • HTTP status logging: Show the actual HTTP response code from the API
  • Full response tracing: Display the complete API response in build logs for debugging
  • Improved error handling: Clear success/failure messages and proper fallback logic
  • Output verification: Show the final CHANGELOG.md content for validation

Debugging Output

The new workflow will show:

  • 🔍 Payload creation status
  • 📏 Request size in bytes
  • 🚀 API call initiation
  • 📥 Complete API response
  • ✅/❌ Success/failure status
  • 📝 Final generated content

This will help identify whether the issue is authentication, malformed requests, unexpected response format, or parsing problems.

Test Plan

  • Test workflow syntax locally
  • Trigger release build to see full debugging output
  • Analyze API response to identify root cause
  • Apply fix based on debugging results

🤖 Generated with Claude Code

- Add comprehensive logging to trace API payload and response
- Use jq for proper JSON escaping instead of string interpolation
- Add HTTP status code logging
- Show full API response in build logs for debugging
- Improve fallback error handling with clear status messages
- Add final CHANGELOG.md output display for verification

This will help debug why we're getting 'null' in release notes.
@gmorpheme gmorpheme merged commit e5b9eba into master Jul 6, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants