Skip to content

fix(gateway): profile editors hide opt-in toolsets the profile hasn't enabled - #86239

Merged
teknium1 merged 1 commit into
mainfrom
fix/hide-optin-toolsets
Aug 14, 2026
Merged

teknium1 merged 1 commit into
mainfrom
fix/hide-optin-toolsets

Conversation

@teknium1

Copy link
Copy Markdown
Collaborator

Follow-up to #86227 (tester report: yuanbao and a2a still listed). _DEFAULT_OFF_TOOLSETS entries and the region-specific yuanbao are global opt-ins that hermes tools/Settings own; a per-profile capabilities editor listing them unchecked is noise at best and confusing at worst (a2a means nothing to a user who never set up agent-to-agent serving). Rule: opt-in toolsets appear only when actually enabled for the profile — hiding an active one would misrepresent the profile, so enabled ones always show.

E2E: describe on a live profile now lists only core user-facing toolsets; yuanbao/a2a absent unless enabled.

… enabled

Follow-up to #86227: _DEFAULT_OFF_TOOLSETS entries (a2a, spotify,
discord, video, x_search, ...) and the region-specific yuanbao are
global opt-ins configured via `hermes tools`/Settings; showing them
unchecked in every per-profile capabilities editor is noise, and
showing a2a at all confuses users who never enabled agent-to-agent
serving (tester report). Enabled ones still show — hiding an ACTIVE
toolset would misrepresent the profile.
@github-actions

github-actions Bot commented Aug 14, 2026 •

Copy link
Copy Markdown

૮ >ﻌ< ა ci review

ran on 790c973 — fix(gateway): profile editors hide opt-in toolsets the profi

⚠️ Warnings

OSV vulnerability scan · View job

5 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 2m55s vs 3m24s (-14.2%). 13 job(s) slower, 10 faster, 1 unchanged.

  • Python tests / Run tests slice 9/12: -39.0s
  • OS-specific tests / Windows-only tests: -37.0s
  • Python tests / Run tests slice 12/12: +28.0s
  • Python tests / Run tests slice 3/12: +22.0s
  • Python tests / Run tests slice 4/12: +20.0s

@teknium1
teknium1 merged commit ed653d3 into main Aug 14, 2026
45 checks passed
@teknium1
teknium1 deleted the fix/hide-optin-toolsets branch August 14, 2026 17:49
@alt-glitch alt-glitch added type/bug Something isn't working comp/tui Terminal UI (ui-tui/ + tui_gateway/) area/profiles Multi-profile isolation, HERMES_HOME scoping P3 Low — cosmetic, nice to have labels Aug 14, 2026
skappafrost pushed a commit to skappafrost/hermes-agent that referenced this pull request Aug 15, 2026
… enabled (NousResearch#86239)

Follow-up to NousResearch#86227: _DEFAULT_OFF_TOOLSETS entries (a2a, spotify,
discord, video, x_search, ...) and the region-specific yuanbao are
global opt-ins configured via `hermes tools`/Settings; showing them
unchecked in every per-profile capabilities editor is noise, and
showing a2a at all confuses users who never enabled agent-to-agent
serving (tester report). Enabled ones still show — hiding an ACTIVE
toolset would misrepresent the profile.
bobaba76 pushed a commit to bobaba76/hermes-agent that referenced this pull request Aug 27, 2026
… enabled (NousResearch#86239)

Follow-up to NousResearch#86227: _DEFAULT_OFF_TOOLSETS entries (a2a, spotify,
discord, video, x_search, ...) and the region-specific yuanbao are
global opt-ins configured via `hermes tools`/Settings; showing them
unchecked in every per-profile capabilities editor is noise, and
showing a2a at all confuses users who never enabled agent-to-agent
serving (tester report). Enabled ones still show — hiding an ACTIVE
toolset would misrepresent the profile.
melon-xf added a commit to melon-xf/hermes-agent that referenced this pull request Sep 3, 2026
… enabled (NousResearch#86239)

Follow-up to NousResearch#86227: _DEFAULT_OFF_TOOLSETS entries (a2a, spotify,
discord, video, x_search, ...) and the region-specific yuanbao are
global opt-ins configured via `hermes tools`/Settings; showing them
unchecked in every per-profile capabilities editor is noise, and
showing a2a at all confuses users who never enabled agent-to-agent
serving (tester report). Enabled ones still show — hiding an ACTIVE
toolset would misrepresent the profile.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/profiles Multi-profile isolation, HERMES_HOME scoping comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants