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

[archdex-art]

Fixes #67013

The Honcho server has updated its schema (adding fields like observe_others), but the latest Python SDK available on PyPI (honcho-ai==2.2.0) is strictly typed to reject unknown fields, causing a 1 validation error for PeerResponse crash upon session initialization.

Since we cannot bump the dependency until upstream publishes a new release to PyPI, this applies a targeted Pydantic monkeypatch to PeerConfig at module load time. By setting extra = 'ignore' and recompiling the model via model_rebuild(force=True), Hermes gracefully ignores any new API fields sent by newer servers while still remaining compatible with the pinned 2.2.0 SDK.

[alt-glitch]

This was generated by AI during triage.

This bundle contains the same Nix TUI source-filter repair as open #67082, plus a separate Honcho workaround for #67013. Please split the independent Honcho change into a focused PR.

[teknium1]

Thanks for isolating a plausible SDK/server compatibility failure. Current main still pins honcho-ai==2.2.0 (pyproject.toml:186) and configures observe_others before session.add_peers (plugins/memory/honcho/session.py:200-210), so the affected SDK path remains relevant.

Problems

• This bundles an unrelated Nix source-filter change at nix/tui.nix:4. The existing maintainer comment links that same change to open #67082 and requests a focused Honcho PR.
• The diff adds no regression coverage for the SDK response-model compatibility path. Current tests do not cover PeerResponse, PeerConfig, or model_rebuild.

Suggested changes

• Split the Nix line into #67082 and keep this PR limited to Honcho.
• Add a regression test using a representative newer-server response payload against the pinned honcho-ai==2.2.0 model, demonstrating that the intended parse succeeds and known fields remain available.

Automated hermes-sweeper review.

[teknium1]

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.

[teknium1]

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.

[GottZ]

This was generated by AI during triage.

Summary

Six PRs address this three-issue complex. Only #67104 changes the Honcho failure path by relaxing SDK model handling of new server fields; #67082, #67090, #67098, #67475, and #67885 address the separate Nix TUI source-filter omission, while #67104 also duplicates that Nix change.

Related pull requests

• fix(build): Include apps/shared in nix npm package #67082 best fix — (+1/-1) — n/a: Adds apps/shared to mkNpmPassthru.dirs, directly restoring the workspace source omitted from the Nix TUI build. Consistent with its automated keep-open verdict, this focused diff is the salvage path for [Bug]: Nix TUI source filter omits @hermes/shared sources #67056 and [Bug]: TUI fails to build in nix environment #67079.
• fix(nix): include apps/shared in TUI source filter #67090 [closed] partial — (+4/-0) — n/a: Attempts the same Nix correction as fix(build): Include apps/shared in nix npm package #67082, but the displayed diff contains unresolved conflict markers. Although its automated review said keep open based on the intended correction, it remains relevant after closure as a contributor-confirmed duplicate superseded by the clean fix(build): Include apps/shared in nix npm package #67082 diff.
• fix(nix): include apps/shared in tui build source #67098 [closed] fixes — (+8/-1) — n/a: Adds apps/shared to the TUI source filter with explanatory formatting, addressing the same workspace-resolution cause as fix(build): Include apps/shared in nix npm package #67082. It remains relevant after closure as a functionally equivalent duplicate of fix(build): Include apps/shared in nix npm package #67082.
• fix(plugins/honcho): monkeypatch SDK to ignore new server fields #67104 best fix — (+13/-1) — n/a; recorded best fix for Bug: Honcho plugin fails to parse PeerResponse after Honcho server update #67013: Changes PeerConfig to ignore extra fields and rebuilds PeerConfig and PeerResponse, directly targeting the reported SDK parse failure, but also bundles fix(build): Include apps/shared in nix npm package #67082's unrelated Nix fix. Consistent with the contributor keep-open review, salvage requires removing the Nix line and adding a representative newer-server PeerResponse regression test.
• fix(nix): include apps/shared in TUI source filter (#67079) #67475 [closed] fixes — (+6/-1) — n/a: Adds the same missing apps/shared source scope as fix(build): Include apps/shared in nix npm package #67082. Despite its automated keep-open verdict, a contributor explicitly closed it as a duplicate of fix(build): Include apps/shared in nix npm package #67082, and its diff supplies no distinct behavior.
• fix(tui): include apps/shared in npm build source for @hermes/shared workspace dependency #67885 [closed] fixes — (+1/-1) — n/a: Applies the byte-identical one-line Nix source-filter correction from fix(build): Include apps/shared in nix npm package #67082. It remains relevant after closure as a contributor-confirmed duplicate of fix(build): Include apps/shared in nix npm package #67082.

Duplicates

#67082, #67098, #67475, and #67885 implement the same functional Nix TUI source-filter correction; #67090 targets that correction but its displayed diff contains conflict markers. The Nix portion of #67104 also duplicates #67082, but #67104 is not a full duplicate because its Honcho workaround uniquely targets #67013.

Suggested consolidation

Author action: split out the part that can merge. Keep #67104 open as the recorded best existing fix for #67013, but remove its duplicate nix/tui.nix change and add the PeerResponse regression coverage requested by the contributor review; keep #67082 open as the focused salvage path for #67056/#67079, while #67090, #67098, #67475, and #67885 remain closed as duplicates of #67082.

Complex graph

flowchart TD
    classDef open fill:#dbeafe,stroke:#1d4ed8,color:#1e3a8a
    classDef merged fill:#dcfce7,stroke:#15803d,color:#14532d
    classDef closed fill:#e5e7eb,stroke:#6b7280,color:#1f2937
    classDef unverified fill:#f3f4f6,stroke:#9ca3af,color:#374151
    classDef best stroke-width:3px,stroke:#b45309
    classDef target stroke-width:3px,stroke:#4338ca
    I67013(["issue #67013 (open)"])
    I67056(["issue #67056 (open)"])
    I67079(["issue #67079 (open)"])
    subgraph Dup67082 ["PRs duplicating each other"]
        P67082["PR #67082 (open)"]
        P67090["PR #67090 (closed)"]
        P67098["PR #67098 (closed)"]
        P67104["PR #67104 (open)"]
        P67475["PR #67475 (closed)"]
        P67885["PR #67885 (closed)"]
    end
    P67104 -->|best fix| I67013
    P67104 -->|fixes| I67056
    P67104 -->|fixes| I67079
    class I67013 open
    class I67056 open
    class I67079 open
    class P67082 open
    class P67090 closed
    class P67098 closed
    class P67104 open
    class P67475 closed
    class P67885 closed
    class P67082 best
    class P67082 best
    class P67104 best
    class P67104 target
    click I67013 "https://github.com/NousResearch/hermes-agent/issues/67013"
    click I67056 "https://github.com/NousResearch/hermes-agent/issues/67056"
    click I67079 "https://github.com/NousResearch/hermes-agent/issues/67079"
    click P67082 "https://github.com/NousResearch/hermes-agent/pull/67082"
    click P67090 "https://github.com/NousResearch/hermes-agent/pull/67090"
    click P67098 "https://github.com/NousResearch/hermes-agent/pull/67098"
    click P67104 "https://github.com/NousResearch/hermes-agent/pull/67104"
    click P67475 "https://github.com/NousResearch/hermes-agent/pull/67475"
    click P67885 "https://github.com/NousResearch/hermes-agent/pull/67885"

Loading

Graph: solid arrow = fixes / best fix, dashed arrow = partial or unverified (see edge label); boxed group = PRs duplicating each other; amber border = best fix; indigo border = target; gray node = closed (state tag in the node label).

Cross-PR triage: Reviewed 6 pull requests and 3 issues in this complex. Each diff was read against this issue; Assessment working set: 4 kB of PR diffs, 15 kB of issue/PR text, 6 kB of discussion (13 comments), 16 verify verdicts. verdicts reflect diff content, not PR titles. Part of an automated triage batch.