Skip to content

feat(herdr): added session-context pane rename on prefix+shift+p - #72

Merged
vanducng merged 2 commits into
mainfrom
feat/herdr-pane-rename-session-context
Jul 29, 2026
Merged

feat(herdr): added session-context pane rename on prefix+shift+p#72
vanducng merged 2 commits into
mainfrom
feat/herdr-pane-rename-session-context

Conversation

@vanducng

Copy link
Copy Markdown
Owner
  • Why: Restored the removed C-x Shift-P pane rename, upgraded from static repo:branch labels to names that reflect what the session is actually doing.
  • What: herdr-pane-rename asks the pane's own agent CLI (codex exec for Codex panes, claude -p haiku otherwise, both on existing subscription auth) to label the pane <project>:<intent> from session title + scrollback; falls back to <repo>:<branch> / folder name without an agent CLI; herdr toasts on start/finish/failure; sandboxed tool-disabled LLM calls with 60/120s timeouts and non-tty stderr logging.
  • Risks: none - display metadata only; keybinding change needs a one-time C-x Shift-R config reload.

Tests: ✓ make test (35 platform + 5 script suites; new test-herdr-pane-rename.sh covers 7 cases incl. agent routing, truncation, failover)
Docs: ✓ docs/content/herdr.md
Breaking: – none

Rebind prefix+shift+p to an LLM-powered pane rename. The script labels
the pane <project>:<intent> by asking the pane's own agent CLI (codex
exec for Codex panes, claude -p haiku otherwise, both on subscription
auth) to summarize the session title and recent scrollback. Falls back
to <repo>:<branch> or the folder name when no agent CLI is available.

Word-boundary truncation within the 40-char label budget, herdr
notifications on start/finish/failure, 60/120s timeouts, tool-disabled
sandboxed LLM calls, and stderr logging to ~/.local/state/herdr when
run non-interactively.

@munmiu munmiu Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 OpenCodeReview found 6 issue(s) in this PR.

  • ✅ 6 posted as inline comment(s)
  • 📝 0 posted as summary

Comment thread scripts/ci/test-herdr-pane-rename.sh Outdated
Comment thread scripts/ci/test-herdr-pane-rename.sh
Comment thread dotfiles/bin/.local/bin/herdr-pane-rename
Comment thread dotfiles/bin/.local/bin/herdr-pane-rename
Comment thread dotfiles/bin/.local/bin/herdr-pane-rename
Comment thread dotfiles/bin/.local/bin/herdr-pane-rename
Also unset OPENAI_API_KEY for the codex call so both LLM paths use
their subscription login regardless of ambient credentials.
@vanducng

Copy link
Copy Markdown
Owner Author

/review

@vanducng
vanducng merged commit 01742dc into main Jul 29, 2026
@munmiu munmiu Bot mentioned this pull request Jul 29, 2026
@vanducng
vanducng deleted the feat/herdr-pane-rename-session-context branch July 29, 2026 16:49
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