Skip to content

fix(vision): surface actual error reason instead of generic message#1338

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-1fc28d17
Mar 14, 2026
Merged

fix(vision): surface actual error reason instead of generic message#1338
teknium1 merged 1 commit into
mainfrom
hermes/hermes-1fc28d17

Conversation

@teknium1

@teknium1 teknium1 commented Mar 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • python -m pytest tests/tools/test_vision_tools.py -n0 -q
  • python -m pytest tests/ -n0 -q
  • python -m hermes_cli.main chat -q "Reply with OK."

When vision_analyze_tool fails, the except block was returning a
generic 'could not be analyzed' message that gave the agent no
actionable information about the failure cause.

Replace the generic message with the actual exception string so the
agent can distinguish between backend errors, missing dependencies,
network failures, and unsupported image paths.

Also add an 'error' field to the failure response for structured
error handling by callers.

Fixes #1034
@teknium1 teknium1 merged commit c207a6b into main Mar 14, 2026
1 check passed
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
…1fc28d17

fix(vision): surface actual error reason instead of generic message
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…1fc28d17

fix(vision): surface actual error reason instead of generic message
olympus-terminal pushed a commit to olympus-terminal/hermes-agent that referenced this pull request May 16, 2026
…1fc28d17

fix(vision): surface actual error reason instead of generic message
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
…1fc28d17

fix(vision): surface actual error reason instead of generic message
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