Skip to content

fix(plugins/honcho): monkeypatch SDK to ignore new server fields #67104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
archdex-art wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix(plugins/honcho): monkeypatch SDK to ignore new server fields #67104

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
60008f2
fix(nix): include apps/shared in TUI source filter
archdex-art Jul 18, 2026
1d1eb7b
fix(plugins/honcho): monkeypatch SDK to ignore new server fields
archdex-art Jul 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion nix/tui.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# nix/tui.nix — Hermes TUI (Ink/React) compiled with tsc and bundled
{ pkgs, hermesNpmLib, ... }:
let
npm = hermesNpmLib.mkNpmPassthru { dirs = [ "ui-tui" ]; };
npm = hermesNpmLib.mkNpmPassthru { dirs = [ "ui-tui" "apps/shared" ]; };

@teknium1 teknium1 Jul 19, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is independent of the Honcho SDK workaround and duplicates the focused Nix repair tracked in #67082. Please split it out so the Honcho compatibility fix can be reviewed and tested independently.


packageJson = builtins.fromJSON (builtins.readFile (npm.src + "/ui-tui/package.json"));
version = packageJson.version;
Expand Down
12 changes: 12 additions & 0 deletions plugins/memory/honcho/session.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,18 @@
if TYPE_CHECKING:
from honcho import Honcho

# Workaround for issue #67013: newer Honcho servers send fields (like observe_others)
# that the honcho-ai<=2.2.0 SDK strictly rejects. We monkeypatch the Pydantic models
# to ignore extra fields until the upstream SDK is updated on PyPI.
try:

@teknium1 teknium1 Jul 19, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a regression test that parses a representative newer-server payload through the pinned honcho-ai==2.2.0 model. The PR currently changes global Pydantic behavior without coverage of the reported PeerResponse failure.

from honcho.api_types import PeerConfig, PeerResponse
if PeerConfig.model_config.get("extra") == "forbid":
PeerConfig.model_config["extra"] = "ignore"
PeerConfig.model_rebuild(force=True)
PeerResponse.model_rebuild(force=True)
except ImportError:
pass

logger = logging.getLogger(__name__)

# Sentinel to signal the async writer thread to shut down
Expand Down