fix(tools,cli): normalise MCP schemas + expand session list columns#2102
Merged
Conversation
Show complete session IDs in 'hermes sessions list' instead of truncating to 20 characters. Widens title column from 20→30 chars and adjusts header widths accordingly. Fixes #2068. Based on PR #2085 by @Nebula037 with a correction to preserve the no-titles layout (the original PR accidentally replaced the Preview/Src header with a duplicate Title/Preview header).
This was referenced Mar 20, 2026
angelburgosrosado
pushed a commit
to angelburgosrosado/hermes-agent
that referenced
this pull request
Apr 27, 2026
…6757a563 fix(tools,cli): normalise MCP schemas + expand session list columns
02356abc
pushed a commit
to 02356abc/hermes-agent
that referenced
this pull request
May 14, 2026
…6757a563 fix(tools,cli): normalise MCP schemas + expand session list columns
olympus-terminal
pushed a commit
to olympus-terminal/hermes-agent
that referenced
this pull request
May 16, 2026
…6757a563 fix(tools,cli): normalise MCP schemas + expand session list columns
gweeteve
pushed a commit
to gweeteve/hermes-agent
that referenced
this pull request
Jun 2, 2026
…6757a563 fix(tools,cli): normalise MCP schemas + expand session list columns
Egavasyug
pushed a commit
to Egavasyug/hermes-agent
that referenced
this pull request
Jun 10, 2026
…6757a563 fix(tools,cli): normalise MCP schemas + expand session list columns
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two salvaged bug fixes on current main.
1. Normalise MCP object schemas without properties (PR #2095 by @sammcf)
Normalises MCP tool input schemas that declare
{"type": "object"}without apropertiesfield before forwarding them to the LLM tool-calling API. Fixes a400 Bad Requestfrom OpenAI when an MCP server exposes a bare object schema (e.g. Crawl4AI'sasktool)._normalize_mcp_input_schema()intools/mcp_tool.pyCherry-picked from #2095 with authorship preserved.
2. Expand session list columns for full ID visibility (PR #2085 by @Nebula037, fixes #2068)
hermes sessions listwas truncating session IDs to 20 chars ([:20]), cutting off the last 2-4 characters. This made it impossible to copy the correct ID for--resume.[:20]truncation — full IDs now shownBased on #2085 with a correction: the original PR accidentally replaced the no-titles layout (
Preview/Srcheader) with a duplicate of the has-titles layout (Title/Previewheader), misaligning columns when sessions have no titles.Test plan
python -m pytest tests/ -n0 -q # 5524 passed