hermes - 💡(How to fix) Fix [mcp_lazy] HIGH: promote_server eager flag documented but never persisted

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Fix Action

Fix

Either store eager per server (Dict[str, bool]) and honor it, or remove the eager parameter, the docstring claim, and the warning.


Filed from opus-4.8 plugin audit (mcp_lazy).

RAW_BUFFERClick to expand / collapse

Severity: HIGH (correctness, silent state loss)

promote_server eager flag is documented as last-writer-wins but is never persisted.

Evidence

  • plugins/mcp_lazy/pool.py:83-100_promoted_servers is a plain Set[str]; the eager flag passed to promote_server is never stored. Per-server eager state is only inferred from _promoted membership.
  • plugins/mcp_lazy/promote.py:160 — calls promote_server(eager=...); docstring claims "last-writer-wins for the eager flag" but there is no per-server eager storage.
  • The WARNING at pool.py:96 fires on every harmless idempotent re-promote, not just genuine eager conflicts (noise).

Impact

Eager-promotion intent is silently dropped; misleading docstring; spurious warnings on benign idempotent calls.

Fix

Either store eager per server (Dict[str, bool]) and honor it, or remove the eager parameter, the docstring claim, and the warning.


Filed from opus-4.8 plugin audit (mcp_lazy).

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING