Skip to content

fix(gateway): make /reasoning session-scoped by default - #15533

Merged
teknium1 merged 1 commit into
NousResearch:mainfrom
irispillars:fix/reasoning-session-default
Apr 26, 2026
Merged

fix(gateway): make /reasoning session-scoped by default#15533
teknium1 merged 1 commit into
NousResearch:mainfrom
irispillars:fix/reasoning-session-default

Conversation

@irispillars

Copy link
Copy Markdown

Summary

  • Makes gateway /reasoning <level> session-scoped by default for Slack/Telegram/etc.
  • Adds /reasoning <level> --global to explicitly persist the global agent.reasoning_effort config value
  • Adds /reasoning reset to clear the current session override and fall back to global config
  • Routes normal agent runs, /background, and /btw through the session-aware reasoning resolver
  • Clears session reasoning overrides on /new, session auto-reset, and compression auto-reset

Design notes

Test Plan

  • python -m py_compile gateway/run.py
  • scripts/run_tests.sh tests/gateway/test_reasoning_command.py tests/gateway/test_session_model_reset.py tests/gateway/test_session_model_override_routing.py tests/gateway/test_running_agent_session_toggles.py -q

Closes #9586

@alt-glitch alt-glitch added type/feature New feature or request P2 Medium — degraded but workaround exists comp/gateway Gateway runner, session dispatch, delivery labels Apr 25, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Related to #14558 and #15522 — competing implementations for #9586. This is the gateway-only approach (narrower scope than #14558).

@teknium1
teknium1 merged commit 25ba6a4 into NousResearch:main Apr 26, 2026
teknium1 added a commit that referenced this pull request Apr 26, 2026
Follow-up to #15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
donald131 pushed a commit to donald131/hermes-agent that referenced this pull request May 2, 2026
…h#15825)

Follow-up to NousResearch#15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…h#15825)

Follow-up to NousResearch#15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
dannyJ848 pushed a commit to dannyJ848/hermes-agent that referenced this pull request May 17, 2026
…h#15825)

Follow-up to NousResearch#15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
…h#15825)

Follow-up to NousResearch#15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
…h#15825)

Follow-up to NousResearch#15533 (merged). Prevents release notes CI from
attributing the contributor to the placeholder.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Session-only reasoning effort override via /reasoning command

3 participants