# NousResearch/hermes-agent · Fix optional-dependency tests to skip not fail

## Open now

- Nothing open on GitHub.

A bug was identified where tests requiring optional extras would fail rather than skip on a default install, preventing a clean test run. The related members address this by updating tests to properly skip when optional dependencies are absent.

## TL;DR

- 4 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 2 PR(s) closed without merging.
- A fix was merged to main in PR #114922. Verify it addresses your case.
- **Recommended:** read PR #114922 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `resolved` · deciding PR(s): `pr-114922`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.50`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Closed / duplicate issues

- #114058: \[Bug\]: tests that need an optional extra fail instead of skipping, so a default install can never run the suite green

## Prior & competing PRs

- **#114922: fix(tests): tests needing an optional extra skip instead of failing on a default install (#114058, salvage #114068 #114211)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `tests/plugins/memory/test_hindsight_provider.py`, `tests/plugins/video_gen/test_fal_plugin.py`, `tests/tools/test_daytona_environment.py`, `tests/tools/test_image_generation.py`, `tests/tools/test_managed_media_gateways.py`, `tests/tools/test_mcp_capability_gating.py`, `tests/tools/test_mcp_dashboard_oauth.py`, `tests/tools/test_mcp_device_flow.py`, `tests/tools/test_mcp_oauth.py`, `tests/tools/test_mcp_oauth_manager.py`, `tests/tools/test_mcp_oauth_metadata.py`, `tests/tools/test_modal_snapshot_isolation.py`, `tests/tools/test_video_generation_tool_surface_matrix.py`, `tests/tools/test_web_tools_config.py`
- **#114068: test: skip optional SDK test dependencies**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `f685eab0d31a`, `#114922`, `#114058`, `#114068`, `#114211`
- **#114211: test(tools): keep mocked SDK paths off the real lazy-dep gate (#114058 remaining files)**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `f685eab0d31a`, `#114922`, `#114058`, `#114068`, `#114211`

## Fix status

- Merged to main in PR #114922 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-09-18`.