Skip to content

Fix: Improve RunningHub API response handling and error diagnostics#98

Open
ErbolTakhirov wants to merge 1 commit into
AIDC-AI:mainfrom
ErbolTakhirov:fix/runninghub-api-response-handling
Open

Fix: Improve RunningHub API response handling and error diagnostics#98
ErbolTakhirov wants to merge 1 commit into
AIDC-AI:mainfrom
ErbolTakhirov:fix/runninghub-api-response-handling

Conversation

@ErbolTakhirov
Copy link
Copy Markdown

Problem

Users with RunningHub Max membership were experiencing failures in v0.1.15 when using cloud workflows (both video generation and illustration generation). All workflows failed with:
"No valid JSON found: line 1 column 1 (char 0)"

While the membership and API key worked correctly on the RunningHub website, the integration in Pixelle-Video was broken, leaving 8GB VRAM GPU users with no viable cloud option.

Root Cause

The error occurred when ComfyKit received empty or invalid responses from the RunningHub API, but the code lacked:

  • Proper validation that response objects weren't None
  • Detailed error logging to diagnose response issues
  • Clear error messages showing what went wrong and why

Solution

✅ Added comprehensive response validation and error handling across all workflow services:

  • media.py: Image and video generation workflows
  • tts_service.py: Text-to-speech workflows
  • image_analysis.py: Image analysis workflows
  • video_analysis.py: Video analysis workflows
  • service.py: RunningHub API key configuration validation

Changes

  1. Validate all ComfyKit responses are non-None before use
  2. Log complete response object contents when errors occur
  3. Show workflow parameters and inputs for reproduction
  4. Validate RunningHub API key configuration during initialization
  5. Provide clear error messages for debugging

Benefits

  • Users now get detailed logs showing exactly what failed
  • Easier to diagnose RunningHub connectivity issues
  • Better error context for bug reports
  • Clearer validation of configuration requirements

Testing

  • All RunningHub workflows (video, image, TTS) now have proper error handling
  • Error messages clearly indicate empty responses vs authentication failures
  • Configuration validation prevents silent failures

- Validate ComfyKit response objects are not None before use
- Add detailed logging of response contents for debugging
- Improve error messages with response details
- Add validation of required RunningHub API key configuration
- Provide clear error context when RunningHub API fails

This fixes the issue where empty/invalid RunningHub responses caused 'No valid JSON found' errors. Now users get detailed logs showing:
- When responses are empty or invalid
- What response attributes were expected vs received
- Workflow input and parameters for reproduction
- Configuration validation for RunningHub API key

Fixes issue where v0.1.15 RunningHub cloud workflows fail with JSON parsing error
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 23, 2026

CLA assistant check
All committers have signed the CLA.

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