Skip to content

feat(cli): show FULL ACCESS / RESTRICTED ACCESS label in welcome banner#2721

Closed
amethystani wants to merge 1 commit into
NousResearch:mainfrom
amethystani:feat/cli-permissions-warning
Closed

feat(cli): show FULL ACCESS / RESTRICTED ACCESS label in welcome banner#2721
amethystani wants to merge 1 commit into
NousResearch:mainfrom
amethystani:feat/cli-permissions-warning

Conversation

@amethystani

Copy link
Copy Markdown
Contributor

Summary

Adds a permissions mode indicator to the welcome banner so users immediately know whether tool approvals are active when Hermes starts.

Changes

hermes_cli/banner.py

  • Add yolo_mode: bool = False parameter to build_welcome_banner()
  • When yolo_mode=True: show ⚠ FULL ACCESS — tool approvals bypassed in red
  • Otherwise: show ✓ RESTRICTED ACCESS — tool approvals enabled in green

cli.py

  • Detect yolo mode from HERMES_YOLO_MODE env var or approvals.mode == "off" config key
  • Pass yolo_mode to build_welcome_banner()

Impact

Small cosmetic addition to the banner. No behavioral changes. Helps users running with --yolo or approvals.mode: off immediately see their permission state without checking config.

Closes #2663

Add a FULL ACCESS / RESTRICTED ACCESS label to the welcome banner so
users can immediately see whether tool approvals are active.

- FULL ACCESS (red, ⚠) when --yolo flag or approvals.mode=off is set
- RESTRICTED ACCESS (green, ✓) in default (manual approval) mode

Adds yolo_mode parameter to build_welcome_banner(); cli.py reads it
from HERMES_YOLO_MODE env var or the approvals.mode config key.
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard labels May 3, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Competing PR with #2665 for the same feature request #2663.

1 similar comment
@alt-glitch

Copy link
Copy Markdown
Collaborator

Competing PR with #2665 for the same feature request #2663.

@teknium1

teknium1 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Automated hermes-sweeper review: this PR is superseded by the implementation already on current main.

Evidence:

  • Current main shows a startup banner warning when approval bypass is active: hermes_cli/banner.py:566 renders ⚠ YOLO mode — all approval prompts bypassed when HERMES_YOLO_MODE is set.
  • Current main also carries the persistent status-bar indicator: cli.py:4053 checks _is_session_yolo_active() and appends ⚠ YOLO in the status text.
  • The implementation landed in b6e07417c5242f7a3d6af1c8d8f0173248b4253f (feat(cli): show YOLO mode warning in banner and status bar), whose commit message explicitly closes [Feature]: Add a permissions warning to the CLI #2663.
  • git tag --contains b6e07417c shows the fix is included in v2026.5.16.
  • The competing PR feat(cli): add permissions mode label to banner and status bar #2665 discussion notes the maintainer salvage intentionally refined the wording from FULL ACCESS to YOLO and dropped the always-on RESTRICTED ACCESS default banner line to keep the normal banner quiet.

Thanks for the contribution — the underlying UX issue is covered on main now.

@teknium1 teknium1 closed this Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add a permissions warning to the CLI

3 participants