Skip to content

feat: add /side side-question command#7733

Closed
gaixianggeng wants to merge 2 commits into
NousResearch:mainfrom
gaixianggeng:feat/side-command
Closed

feat: add /side side-question command#7733
gaixianggeng wants to merge 2 commits into
NousResearch:mainfrom
gaixianggeng:feat/side-command

Conversation

@gaixianggeng

Copy link
Copy Markdown
Contributor

Summary

  • add /side <question> as a side-question command in CLI and gateway
  • let /side bypass the active-session guard so it can run without interrupting the main task
  • add gateway regression tests for registration, bypass behavior, and session-race handling

Behavior

  • uses current session context
  • does not interrupt the active task
  • runs with no tools
  • does not persist side-question history into the main session

Test Plan

  • python -m pytest tests/gateway/test_side_command.py tests/gateway/test_command_bypass_active_session.py tests/gateway/test_session_race_guard.py tests/hermes_cli/test_commands.py -q
  • python -m py_compile cli.py gateway/run.py gateway/platforms/base.py hermes_cli/commands.py tests/gateway/test_side_command.py tests/gateway/test_command_bypass_active_session.py tests/gateway/test_session_race_guard.py

Notes

  • reviewed the branch diff and cleaned the branch base so this PR contains only the /side feature commit

@gaixianggeng

Copy link
Copy Markdown
Contributor Author

先收口这批 open PR,优先保留更容易被接收的高价值 bugfix。这个 /side 功能方向本身有意思,但体量偏大、优先级低于当前 bug backlog,先关闭,后续等 Claude/Codex 再做一轮收敛 review 后按更成熟形态重提。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant