Skip to content

fix: improve stability - timeouts, parameter setting, Camoufox retry, error handling#12

Open
maebahesioru wants to merge 7 commits intoMag1cFall:mainfrom
maebahesioru:main
Open

fix: improve stability - timeouts, parameter setting, Camoufox retry, error handling#12
maebahesioru wants to merge 7 commits intoMag1cFall:mainfrom
maebahesioru:main

Conversation

@maebahesioru
Copy link
Contributor

Summary

Comprehensive stability improvements based on real-world log analysis.

Fixes

Parameter Setting

  • Use fill() instead of type() for spinbutton inputs (fixes decimal truncation: 1.010 → clamped to 2)
  • Re-expand advanced settings panel if parameter inputs not visible
  • scroll_into_view_if_needed before interacting with Google Search toggle and stop sequence input

Submission

  • Submit Button wait timeout: 2s → 10s
  • Textarea locator timeout: 5s → 15s
  • If shortcut fails but response already started, treat as success (avoids false 500 errors)

Camoufox Launch

  • Retry up to 3 times on WebSocket endpoint timeout
  • Find free port before each attempt to avoid EADDRINUSE
  • Early exit on EADDRINUSE detection (no need to wait full 90s)

Error Handling / Log Noise

  • Downgrade TargetClosedError in model management and localStorage init to DEBUG
  • Downgrade noisy button check warnings to DEBUG
  • STREAM proxy: retry with port increment on failure

Other

  • Clear chat verification: fallback to textarea when zero_state not visible
  • Tools panel / Google Search: skip if button not found (new UI layout)
  • Fuzzy model ID matching when model not in parsed_model_list
  • SLEEP_LONG 0.5s → 1.0s

… error handling

Parameter Setting:
- Use fill() instead of type() for spinbutton inputs (fixes decimal truncation)
- Re-expand advanced settings panel if parameter inputs not visible
- scroll_into_view_if_needed for Google Search toggle and stop sequence input
- Google Search: JS label click fallback when force click fails

Submission:
- Submit Button wait timeout: 2s -> 10s
- Textarea locator timeout: 5s -> 15s
- If shortcut fails but response already started, treat as success

Camoufox Launch:
- Retry up to 3 times on WebSocket endpoint timeout
- Find free port before each attempt to avoid EADDRINUSE
- Early exit on EADDRINUSE detection (no need to wait full 90s)

Error Handling / Log Noise:
- Downgrade TargetClosedError in model management and localStorage init to DEBUG
- Downgrade noisy button check warnings to DEBUG
- STREAM proxy: retry with port increment on failure

Other:
- Clear chat verification: fallback to textarea when zero_state not visible
- Tools panel / Google Search: skip if button not found (new UI layout)
- Fuzzy model ID matching when model not in parsed_model_list
- SLEEP_LONG 0.5s -> 1.0s
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.

1 participant