fix(browser): default non-finite Camofox vision config

[lin-hongkuan]

Summary

• reject non-finite Camofox vision timeout and temperature config values
• fall back to the existing 120s / 0.1 defaults for inf and nan
• add regression coverage ensuring non-finite config is not passed to call_llm

Tests

• python -m pytest tests/tools/test_browser_camofox.py::TestCamofoxVisionConfig::test_camofox_vision_defaults_nonfinite_config -q
• NO_PROXY=localhost,127.0.0.1,::1 python -m pytest tests/tools/test_browser_camofox.py -q
• python -m py_compile tools/browser_camofox.py
• python scripts/check-windows-footguns.py --all

[teknium1]

Thanks for isolating the Camofox failure mode. The Camofox premise is confirmed on current main: tools/browser_camofox.py:894-915 converts auxiliary.vision values with float() and forwards them explicitly to call_llm.

Problems

• This is a shared-config bug class, but the patch covers only Camofox. The regular browser path still does unrestricted float() conversion in tools/browser_tool.py:4118-4121; image and video vision do the same in tools/vision_tools.py:1236-1239 and :1739-1742. Explicit timeouts are preserved by agent/auxiliary_client.py:6077 and passed into the SDK request at :6529-6533.

Suggested changes

• Reuse one finite-value coercion helper across every auxiliary.vision config-to-call boundary, preserving the video flow's 180-second floor.
• Add regression coverage for the non-Camofox browser/image/video paths as well as this Camofox case.

Automated hermes-sweeper review.

[teknium1]

Please apply this finite-value coercion to the sibling auxiliary.vision parsers too: tools/browser_tool.py:4118-4121 and tools/vision_tools.py:1236-1239,1739-1742 still forward nan/inf as explicit LLM request values.