# NousResearch/hermes-agent · MCP server name normalization and platform toolset fixes

## Open now

- Nothing open on GitHub.

Multiple bugs and fixes related to numeric MCP server names causing TypeErrors and stale \`no_mcp\` entries persisting incorrectly in \`_save_platform_tools\`. One unrelated Discord toolset member is also present.

## TL;DR

- 8 member(s): 0 open issue(s), 2 closed issue(s), 0 open PR(s), 1 merged PR(s), 5 PR(s) closed without merging.
- A fix was merged to main in PR #7654. Verify it addresses your case.
- **Recommended:** read PR #7654 (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: `unresolved`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_PARTIAL`): some members are connected by structural evidence (shared changed files, or a closing/superseded reference); the rest are joined by an automated relevance judgement over their text.
- Members outside that structural core may belong to a different underlying problem.
- Pairwise link density `0.23`: 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

- #6901: \[Bug\]:  TypeError: '<' not supported between instances of 'int' and 'str' when using numeric MCP server names (e.g., "12306")
- #13028: \[Bug\]: _save_platform_tools preserves stale \`no_mcp\` and still crashes on numeric entries

## Prior & competing PRs

- **#7654: fix: final batch — MCP names, zombies, vision hardening, test isolation (Buckets S+T)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `agent/skill_commands.py`, `cron/scheduler.py`, `environments/tool_call_parsers/hermes_parser.py`, `environments/tool_call_parsers/mistral_parser.py`, `hermes_cli/tools_config.py`, `tests/gateway/test_approve_deny_commands.py`, `tests/hermes_cli/test_tools_config.py`, `tests/tools/test_approval.py`, `tests/tools/test_managed_media_gateways.py`, `tests/tools/test_vision_tools.py`, `tools/process_registry.py`, `tools/vision_tools.py`
- **#13086: fix(cli): clear stale no_mcp and normalize numeric toolset names on save (#13028)**. closed without merge · fix tier `suggested`
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `9d7b64b5dd394a60df7d38824256e2f213eac5c6`, `#13028`
  - _suggested-tier fix detail withheld; re-request with `?tier=all` to include it._
- **#15091: fix(discord): end-to-end Discord support — toolset loading, tool split, and context injection**. closed without merge · fix tier `suggested`
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`
  - _suggested-tier fix detail withheld; re-request with `?tier=all` to include it._
- **#6962: fix: normalize numeric MCP server names to str (fixes #6901)**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `#7654`
- **#14101: fix(cli): normalize saved platform tool entries**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `9d7b64b5d`, `9d7b64b5dd394a60df7d38824256e2f213eac5c6`, `#13086`, `#13028`
- **#14149: fix(cli): clear stale no_mcp when saving platform toolsets**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `9d7b64b5d`, `#13086`, `#13086`

## Fix status

- Merged to main in PR #7654 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-04-11`. This confirmation is old and may be out of date (a revert/reopen would not have invalidated it here); re-verify.
- Subsystem: `gateway`