Replies: 1 comment
|
Thanks for considering this. Local need: when Herdr restores Claude/Codex after a server restart it runs bare Ready branch (with CI clippy fix): https://github.com/Chibionos/herdr/tree/feat/session-resume-extra-args @ogulcancelik — if this direction is OK, could you convert this discussion into an accepted issue (or reopen PR #2105 with a scope override)? Happy to iterate on naming/validation if you prefer a different config shape. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Problem
Native agent session restore hardcodes resume argv:
claude --resume <id>codex resume <id>Users who start agents with flags such as Claude
--dangerously-skip-permissions/--permission-mode bypassPermissionsor Codex--yolo/--dangerously-bypass-approvals-and-sandboxlose those flags when Herdr restores after a server restart. Workarounds today are PATH wrappers or agent-global config defaults.Proposal
Add optional per-agent extra args in
config.toml:Semantics: insert the list immediately after the executable name, so global flags apply before subcommands:
claude --resume <id>claude --dangerously-skip-permissions --resume <id>codex resume <id>codex --yolo resume <id>Why this design
ui.sidebar.agents.rows_by_agent).claude,codex, …), rejected at load time if unknown.agent startdefault args (can be a follow-up).Implementation status
A ready branch on a fork implements this with unit tests and next-release docs:
SessionConfig,agent_resume::plan_with_extra_args, restore plumbing,docs/nextconfig/session-state/config-reference, default-config comments.Happy to open/adjust a PR once this direction is accepted (per CONTRIBUTING, feature work needs maintainer alignment first).
Alternatives considered
claude/codex— works today, but global and outside Herdr.approval_policy/ sandbox) — does not fully cover Claude bypass flags.Ask
Does this fit Herdr’s product direction for session restore? If yes, happy to open the PR (or have a maintainer take the branch) under whatever title/scope override you prefer.
All reactions